-
-
Notifications
You must be signed in to change notification settings - Fork 91
feat(enrichment): Cross-file caller-impact / dead-symbol analyzer #1509
Copy link
Copy link
Closed
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority — scores a 1.5x multiplier.Maintainer-selected Gittensor priority — scores a 1.5x multiplier.
Description
Metadata
Metadata
Assignees
Labels
gittensor:featureGittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority — scores a 1.5x multiplier.Maintainer-selected Gittensor priority — scores a 1.5x multiplier.
Projects
StatusShow more project fields
Done
Context
A REES (review-enrichment service) analyzer. Tier: solid.
Detects: A changed/removed/renamed export with live callers in unchanged files the PR didn't update (a hidden compile/runtime break), and conversely new exports never referenced anywhere (dead-on-arrival).
Data source: Parse added/removed top-level declarations from the diff; resolve usages via GitHub Code Search API (search/code, repo-scoped) + contents API at headSha. Token-scoped, free within rate budget.
This is heavy/external/historical analysis the no-checkout headless
claude --printreviewer cannot do; the REES returns it as a brief block the engine splices into the review (additive + fail-safe).Implementation (established pattern, all inside
review-enrichment/)BriefFindingskey insrc/types.tssrc/analyzers/<name>.ts— pure, injectfetchfor testssrc/brief.tsANALYZERS registrysrc/render.tsnode:testunits againstdist/+ a live smoke against the real data sourceDeliverables
Parent: #1499