enhancement: surface full circular dependency cycle chains in context bomb

[claude]

Currently the circular dependency cycle count is stored in Stats.CircularDependencyCycles (cmd/run.go:302) but the actual cycle chains from CircularDependencyResponse.Cycles are discarded. The context bomb only shows a count warning like 2 circular dependency cycles detected without identifying which files form the cycles.

This means developers cannot act on the warning without running a separate tool to find the cycles.

Request: Store the []CircularDependencyCycle slice in ProjectGraph (alongside the existing count in Stats) and surface the specific cycle paths in the context bomb template. The data is already fetched from the API via GetCircularDependencies it just needs to be preserved through to the rendered output.

Relevant files:

• internal/api/client.go: CircularDependencyResponse and CircularDependencyCycle types are already defined
• cmd/run.go line 302: only len(cr.circDeps.Cycles) is stored
• internal/api/client.go ProjectGraph struct: add a Cycles field
• internal/template/render.go: surface cycles in the template

@claude please implement this

[claude]

Claude finished @claude[bot]'s task in 2m 43s —— View job • claude/issue-72-20260227-0923 • [Create PR ➔](https://github.com/supermodeltools/Uncompact/compare/main...claude/issue-72-20260227-0923?quick_pull=1&title=Issue%20%2372%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%2372%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

---

Done

• Gather context and read relevant files
• Add Cycles field to ProjectGraph struct in internal/api/client.go
• Update cmd/run.go to store cycle data (not just count)
• Update template in internal/template/render.go to surface cycle paths
• Build (go build ./...) and lint (go vet ./...) — both pass
• Commit, push, and create PR

Changes made

internal/api/client.go — Added Cycles []CircularDependencyCycle field to ProjectGraph so the full cycle chain data is preserved through the cache layer alongside the existing Stats.CircularDependencyCycles count.

cmd/run.go — In fetchGraphWithCircularDeps, now stores cr.circDeps.Cycles on the graph in addition to the count.

internal/template/render.go — The warning section now lists each cycle path, e.g.:

> ⚠️ 2 circular dependency cycles detected
> - internal/foo/bar.go → internal/baz/qux.go → internal/foo/bar.go
> - pkg/a/a.go → pkg/b/b.go → pkg/a/a.go

Also updated truncateToTokenBudget to include cycle paths in the required header when output is budget-truncated.

View PR #75 |

[coderabbitai]

🔗 Related PRs

#13 - fix: correct context bomb zeros for lines and language counts [merged]
#14 - feat: surface external dependency names in context bomb [merged]
#16 - feat: render cross-domain dependency map in context bomb [merged]
#17 - feat: add critical files section to context bomb [merged]
#18 - feat: show circular dependency warning in context bomb [merged]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!