fix: validation findings — git correlation, waste categories, data accuracy#3
Merged
Mahiler1909 merged 1 commit intomainfrom Mar 30, 2026
Merged
Conversation
…ccuracy - Fix cross-project contamination in session git correlation by filtering sessions to the same project before calling correlateBranch - Separate waste signals into "avoidable" vs "platform_overhead" categories with distinct display in waste report - Use actual model pricing for context_rebuild cost (was hardcoded Sonnet) - Make outcome inference conservative when session-meta is missing - Add tokenBreakdown (input/output/cache) to session JSON output - Fix filesModified precision by using full paths instead of basenames Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
2 tasks
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.
Summary
sessioncommand was passing all sessions (across all projects) tocorrelateBranch, causing sessions onmainto inherit commit windows from unrelated projects. Backoffice-ui session (4 days) was getting 5 commits instead of 1.context_rebuild(74% of waste signals) is now categorized as "platform_overhead" and displayed separately from user-avoidable waste in the waste report.mostly_achievedinstead offully_achieved(can't confirm zero errors without evidence).tokenBreakdownto session JSON: Exposes input/output/cacheCreation/cacheRead separately for debugging.Test plan
npm run build— compiles cleannpm test— 197 tests pass, zero regressionsburnlog session 33712c85now shows 1 commit (was 5)burnlog wasteseparates avoidable ($20.90) from overhead ($92.06)fully_achievedtomostly_achievedtokenBreakdownpresent in session JSON, sum matches total🤖 Generated with Claude Code