fix(platform): land dead-code cleanup + AI Exposure disclosure (recover #149) - #152
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- computeOrgPulse: drop the commits/aiAdoption sparklines. commits was fed by an accessor that always returned [] (dead on arrival, never produced real data); aiAdoption was hardcoded to []. Neither was ever read by OrgPulse.tsx — only sparklines.stabilization is. Simplified buildOrgSparkline by dropping its now-unused "sum" mode, which only existed for the removed commits sparkline. - getRepoTimeSeries + TimeSeriesPoint: stop fetching pr_merged_count, pr_single_pass_rate, fix_latency_median_hours, and cascade_rate on the repo-detail time series. None of the four is read anywhere in page.tsx or charts.tsx — the visible metric cards and charts pull stabilization_ratio, revert_rate, churn_events, and commits_total. Also dropped charts.tsx's local chartData re-mapping of revert_rate and cascade_rate, which were computed into that array but never plotted (page.tsx reads revert_rate separately, directly off timeSeries, for the summary card). - investment-hotspots.tsx: translate the origin enum (HUMAN/AI_ASSISTED) before interpolating it into the fix-magnet copy, instead of dropping the raw constant into a translated sentence. Reuses the same repoCharts.origin.labels.* keys charts.tsx already uses for this enum on the same page. No behavior change for anything that was actually rendering; typecheck, tests, and lint all pass unchanged. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
/ai-exposure largely re-presents the dashboard's Attribution Gap card
(same underlying ratio, plus a per-repo breakdown and install-hook CTA).
The dashboard card already links here ("See Shadow AI Exposure →"); add
the reverse disclosure so a reader landing on /ai-exposure directly
understands it's the detail view, not a second, disconnected metric.
Considered removing the redundancy structurally, but the page's per-repo
ranking and install-hook flow are real differentiated value — disclosure
is the smaller, sufficient fix.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
clickmatos
force-pushed
the
fix/dashboard-dead-code
branch
from
August 13, 2026 18:20
267d058 to
2bbff7c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why this exists
PR #149 was opened with the wrong base branch (
fix/dashboard-medium-issuesinstead ofmain). It shows as "Merged" on GitHub, but it merged into that intermediate branch, which was itself never merged intomain— so none of #149's dead-code removal, and none of the later AI Exposure disclosure commit I pushed to the same branch after #149 had already closed, ever reachedmain.This PR is the same content, targeted correctly at
mainthis time. Confirmed the diff is clean and exactly what's missing (verifiedgit diff main origin/fix/dashboard-dead-codematches this PR's file list, nothing extra from #147/#148 since those are already onmain).Contents (unchanged from #149 + the follow-up disclosure commit)
computeOrgPulsesparklines, unusedgetRepoTimeSeries/TimeSeriesPointfields,investment-hotspots.tsxorigin-enum i18n fix.docs(platform): AI Exposure page subtitle now discloses its relationship to the dashboard's Attribution Gap card.Verification
npx tsc --noEmit: clean.npm run test: 241/241 passing.npm run lint: 0 errors, same 72 pre-existing warnings.Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com