Skip to content

Honor minimum community size in architecture reports - #305

Merged
forhappy merged 1 commit into
mainfrom
codex/filter-thin-communities-report
Aug 29, 2026
Merged

Honor minimum community size in architecture reports#305
forhappy merged 1 commit into
mainfrom
codex/filter-thin-communities-report

Conversation

@forhappy

Copy link
Copy Markdown
Contributor

Summary

  • make the existing min_community_size report option filter detailed architecture communities
  • retain singleton and thin-community nodes, edges, IDs, and assignments in the underlying graph
  • preserve raw graph totals and disclose filtered communities through typed omission counts
  • document that --min-community-size=1 restores singleton presentation

Evidence

Graphify does not delete singleton communities: it assigns each isolate to a one-node community, then omits communities below its default size threshold of 3 only from detailed reports. A synthetic execution confirmed all nodes remained assigned while thin communities were disclosed as omitted.

A real FastAPI graph measurement found 268 singleton communities, all with degree zero. It also found exact edges in some two-node communities, which demonstrates why small communities must remain queryable rather than being removed from graph.json.

Compatibility

This is a presentation-layer change using an existing option. It does not change graph schema, graph publication, community identity, query behavior, viewer data, or version numbers. Callers that need the previous full directory can set min_community_size to 1.

Verification

  • cargo fmt --all -- --check
  • cargo test -p compass-output --locked
  • cargo clippy -p compass-output --all-targets --all-features --locked -- -D warnings
  • cargo clippy --workspace --lib --bins --locked -- -D warnings
  • cargo test --workspace --lib --bins --locked
  • ./scripts/qualify_code_graph_v1.sh --fixtures-only
  • sh scripts/check_product_boundary.sh

@forhappy
forhappy merged commit 91dde6b into main Aug 29, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant