-
-
Notifications
You must be signed in to change notification settings - Fork 89
feat(enrichment): Heavy-dependency-for-trivial-use detector #1505
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: high-value.
Detects: A newly added heavyweight dep used only trivially in the diff (one import, one call site) — 'added lodash for .get once' / 'added axios for one GET' — plus the raw install/bundle-size delta.
Data source: Shipped manifest extractor + grep of the PR's own changed patches for import/require usage count + packagephobia/bundlephobia/deps.dev for weight. Inputs already in EnrichRequest + one free sizing call.
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