perf: run GetGraph and GetCircularDependencies in parallel to halve API latency

[claude]

Problem

In cmd/run.go:248–271 (fetchGraphWithCircularDeps) and internal/api/client.go:414–467, GetGraph and GetCircularDependencies are called sequentially:

graph, err := client.GetGraph(ctx, projectName, repoZip)         // uploads zip, polls ~10-15 min
circDeps, err := client.GetCircularDependencies(ctx, projectName, repoZip) // uploads zip again, polls again

The two calls are completely independent — neither depends on the other's result. Running them back-to-back means:

• The repo zip (up to 10 MB) is uploaded twice over the network.
• Total wall-clock time is time(GetGraph) + time(GetCircularDeps) instead of max(time(GetGraph), time(GetCircularDeps)).

Fix

Run both calls concurrently using goroutines and a sync.WaitGroup (or errgroup). Collect results from both, then merge circDeps into graph.Stats.CircularDependencyCycles exactly as today.

This would roughly halve the total time spent waiting for the API on a cache miss.

Relevant files:

• cmd/run.go:248–271 — fetchGraphWithCircularDeps
• internal/api/client.go:414–467 — GetGraph / GetCircularDependencies

@claude please implement this

[claude]

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

---

Tasks

• Gather context and understand the request
• Read relevant files (cmd/run.go, internal/api/client.go)
• Implement parallel execution of GetGraph and GetCircularDependencies
• Build and lint (go build ./..., go vet ./...)
• Commit and push changes
• Create PR

Summary

Refactored fetchGraphWithCircularDeps in cmd/run.go to launch both API calls concurrently using goroutines and buffered channels:

graphCh := make(chan graphResult, 1)
circCh  := make(chan circResult, 1)

go func() { graphCh <- graphResult{client.GetGraph(...)} }()
go func() { circCh  <- circResult{client.GetCircularDependencies(...)} }()

gr := <-graphCh   // wait for graph
cr := <-circCh    // wait for circ deps

Both goroutines start simultaneously, so wall-clock time is max(GetGraph, GetCircularDeps) instead of the sum. No new dependencies — uses stdlib goroutines and buffered channels only.

PR: #55

[coderabbitai]

🔗 Related PRs

#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!