Skip to content

Reduce stale auto-review ledger burn#378

Merged
cbusillo merged 1 commit into
mainfrom
fix/reduce-stale-auto-review-burn
Jun 5, 2026
Merged

Reduce stale auto-review ledger burn#378
cbusillo merged 1 commit into
mainfrom
fix/reduce-stale-auto-review-burn

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 5, 2026

Summary

  • suppress terminal stale/detached auto-review findings from the compact ledger prompt context while keeping current/matching findings actionable
  • add aggregate diagnostics for suppressed stale history and duplicate-review saved token estimates
  • record duplicate saved-token estimates only for auto-review runs that are actually skipped/reused

Fixes #375

Validation

  • cargo test --manifest-path code-rs/Cargo.toml -p code-core compact_ledger
  • cargo test --manifest-path code-rs/Cargo.toml -p code-core superseded_clean_duplicate_does_not_report_saved_tokens
  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui auto_review
  • ./build-fast.sh

Agent review

  • planning agents reviewed the issue shape and recommended a focused review-store/TUI patch
  • initial review agents found one medium duplicate-savings bug; fixed before final validation
  • final cleanup review agents reported no blocking/high/medium issues

@cbusillo cbusillo merged commit e5185af into main Jun 5, 2026
1 check passed
@cbusillo cbusillo deleted the fix/reduce-stale-auto-review-burn branch June 5, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reduce repeated high-token auto-review runs on stale snapshots

1 participant