Skip to content

perf: run GetGraph and GetCircularDependencies in parallel#55

Merged
claude[bot] merged 1 commit intomainfrom
claude/issue-52-20260227-0625
Feb 27, 2026
Merged

perf: run GetGraph and GetCircularDependencies in parallel#55
claude[bot] merged 1 commit intomainfrom
claude/issue-52-20260227-0625

Conversation

@claude
Copy link
Contributor

@claude claude bot commented Feb 27, 2026

Summary

  • Refactors fetchGraphWithCircularDeps in cmd/run.go to launch GetGraph and GetCircularDependencies as concurrent goroutines using buffered channels
  • Both calls are fully independent, so total wall-clock time drops from time(GetGraph) + time(GetCircularDeps) to max(time(GetGraph), time(GetCircularDeps)) — roughly halving API latency on cache misses
  • No new dependencies required; uses only stdlib goroutines and channels
  • go build ./... and go vet ./... pass cleanly

Closes #52

Generated with Claude Code

Both API calls are independent; run them concurrently using goroutines
and buffered channels so total wall-clock time is max(GetGraph,
GetCircularDeps) instead of the sum. Fixes #52.

Co-Authored-By: Grey Newell <greyshipscode@gmail.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@claude claude bot merged commit 64b33d3 into main Feb 27, 2026
1 check passed
@claude claude bot deleted the claude/issue-52-20260227-0625 branch February 27, 2026 06:32
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.

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

0 participants