hurtener / harbor
gomain9ba5a96403.0K linessynced 1mo ago
Code health
7.7out of 10Good
This codebase scores 7.7 out of 10 on defect risk, which we rate good. It also scores maintainability 8.1 and static performance risk 9.9 out of 10. The three are scored separately and never blended into one number. The files you change most are the weak spot: 334 of 2,127 files are git hotspots, and they average 5.2.
- Lines of code
- 403.0K
Recent activity
All commitsCommits
No commits indexed yet.
Decisions
- alwaysRequirePolicy is a minimal approval policy that always requires approval. The gate's pending mproposed
- mkBridgeFixture constructs a real ApprovalGate + Bus + Coordinator triple sharing one set of dependeproposed
- Fire the apply path with an APPROVE control carrying the gate's token in the wire payload (the canonproposed
- PAUSE must NOT itself call Coordinator.Request — the planner's RequestPause decision does.proposed
- The owning gate called Coordinator.Resume on the gate's pause (wireToken). The RunLoop's own outstproposed
- advancePause routes a RESUME / APPROVE / REJECT control event to the unified pause/resume primitive.proposed
Needs attention
1,607 open- medium severity. ProposedalwaysRequirePolicy is a minimal approval policy that always requires approval. The gate's pending mAuto-proposed decision awaiting review
- medium severity. ProposedmkBridgeFixture constructs a real ApprovalGate + Bus + Coordinator triple sharing one set of dependeAuto-proposed decision awaiting review
- medium severity. ProposedFire the apply path with an APPROVE control carrying the gate's token in the wire payload (the canonAuto-proposed decision awaiting review
- medium severity. ProposedPAUSE must NOT itself call Coordinator.Request — the planner's RequestPause decision does.Auto-proposed decision awaiting review
- medium severity. ProposedThe owning gate called Coordinator.Resume on the gate's pause (`wireToken`). The RunLoop's own outstAuto-proposed decision awaiting review
Where the risk concentrates
All 334 hotspotsRanked by prior bug fixes and change frequency, mined from full git history rather than from the code alone.
| File | Churn | Prior fixes | Maintainers | Commits 90d |
|---|---|---|---|---|
| devstack.goharbortest/devstack | 99.9th | 9 | 2 | 71 |
| cmd_dev.gocmd/harbor | 99.9th | 8 | 2 | 82 |
| methods.gointernal/protocol/methods | 99.8th | 0 | 2 | 38 |
| cmd_dev_runloop.gocmd/harbor | 99.8th | 4 | 2bus factor 1 | 32 |
| validate.gointernal/config | 99.7th | 4 | 3 | 63 |
Composition
Open the graphgo 66%markdown 20%typescript 11%json 2%yaml 1%Other 1%
Explore this codebase
- Docs0 pages across 84 modules
- ChatAsk this codebase a question and get an answer with its sources
- Files2,127 files with per-file docs, health and history
- ArchitectureDependency graph, layers, and 76 entry points
- Code healthPer-file scores, 10,068 open findings, coverage and refactoring targets
- Knowledge graphEntities, communities, and the paths between them
- Change couplingFiles that keep changing together, mined from commit history
- CommitsChange-risk ranked history with agent provenance
- ContributorsBus factor, per-file maintainers, and the human/agent split
- StatsSize class, origin, lifetime churn, rhythm and records
- CostsWhat indexing this snapshot cost, by model and by run