Skip to content

feat(cat5): break out isolates by entity_type#38

Open
jphein wants to merge 1 commit into
M0nkeyFl0wer:mainfrom
techempower-org:feat/cat5-isolate-type-breakdown
Open

feat(cat5): break out isolates by entity_type#38
jphein wants to merge 1 commit into
M0nkeyFl0wer:mainfrom
techempower-org:feat/cat5-isolate-type-breakdown

Conversation

@jphein
Copy link
Copy Markdown
Contributor

@jphein jphein commented May 25, 2026

Summary

Closes #14.

  • Adds isolated_by_type: dict[str, int] to GapDetectionReport — counts isolated nodes grouped by entity_type
  • Computed during the existing component iteration, zero additional graph traversal
  • format_report() renders the breakdown when isolates exist (e.g., topic: 1)

Test plan

  • test_isolated_by_type — verifies the dict matches fixture ground truth
  • test_format_report_shows_isolate_types — verifies rendered output contains type labels
  • Existing Cat 5 tests pass unchanged

🫏 Generated with Claude Code

)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 25, 2026 01:18
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@M0nkeyFl0wer
Copy link
Copy Markdown
Owner

Ship. Isolate-by-entity-type is the right diagnostic surface — aggregate isolate count hides which entity classes the ingestion pipeline is dropping on the floor, and computing it during the existing component iteration is the zero-cost shape. Tests verify both population and formatting. Clean.

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.

Cat 5 — break down isolate count by node type

3 participants