feat(enrichment): unused-export / dead-on-arrival detector analyzer#3584
feat(enrichment): unused-export / dead-on-arrival detector analyzer#3584james3773 wants to merge 1 commit into
Conversation
Detect newly added exports with zero non-declaration references via bounded GitHub Code Search, closing the net-new export slice of caller-impact work tracked in JSONbored#2025. Co-authored-by: Cursor <cursoragent@cursor.com>
|
🚨 Contributor flagged. Click here for more info: Superagent Dashboard |
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3584 +/- ##
=======================================
Coverage 93.16% 93.16%
=======================================
Files 311 311
Lines 31750 31750
Branches 11612 11612
=======================================
Hits 29581 29581
Misses 1517 1517
Partials 652 652
🚀 New features to boost your workflow:
|
|
Caution 🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥 🛑 Gittensory review result - reject/close recommendedReview updated: 2026-07-05 18:05:04 UTC
🛑 Suggested Action - Reject/Close
Review summary Nits — 1 non-blocking
Why this is blocked
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
|
Gittensory is closing this pull request on the maintainer's behalf (Linked issue overlaps another open PR; duplicate of another open PR). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed. |
Summary
Closes #2025.
Adds a github-light REES analyzer that flags exports newly introduced by a PR with zero non-declaration references anywhere in the repo. Parses added top-level export declarations from the diff, resolves each symbol via bounded GitHub Code Search (repo-scoped, injected fetch), and reports dead-on-arrival surface for reviewers.
UnusedExportFinding+unusedExportbrief keyreview-enrichment/src/analyzers/unused-export.tswithmaxSymbols/maxSearchescaps and fail-safe skip on missing token/head SHA or search errorsTest plan
npm --prefix review-enrichment run buildnpm --prefix review-enrichment run metadatanpm --prefix review-enrichment test(1082 passed)npm run typechecknpm run rees:testMade with Cursor