Skip to content

ci: sync code-size budget with master - #1701

Open
Soham-o wants to merge 5 commits into
1jehuang:masterfrom
Soham-o:fix/692-sync-code-size-baseline
Open

Soham-o wants to merge 5 commits into
1jehuang:masterfrom
Soham-o:fix/692-sync-code-size-baseline

Conversation

@Soham-o

@Soham-o Soham-o commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • refresh scripts/code_size_budget.json to the current production Rust file sizes on master
  • preserve the 1200 LOC threshold and existing tracked debt
  • add newly oversized production files introduced by recent merged work
  • keep the code-size ratchet meaningful for subsequent regressions instead of failing on inherited baseline drift

Fixes #692.

Validation

  • baseline generated from upstream master at 77f4f9d50bb41ae849da9ffaab7ea25e7815e096
  • current master Quality Guardrails run 3269 reproduced the failure before this change
  • no production source files changed; this PR only synchronizes the ratchet baseline

@greptile-apps

greptile-apps Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low risk] Updates code size and panic tracking budgets.

No outstanding findings block merging.

Summary

The PR refreshes the code-size, test-size, and panic budgets. The harness example’s panic budget now uses its actual path.

Reviews (4) · Last reviewed commit: "ci: correct harness panic budget path"

Comment thread scripts/code_size_budget.json Outdated
Comment thread scripts/panic_budget.json Outdated
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.

CI red on master: code-size ratchet exceeded by desktop2/transcript.rs (2364 -> 2799 LOC)

1 participant