8000 ci(v3): binary-size gate report-only until #551 by adityamukho · Pull Request #680 · project-minigraf/minigraf · GitHub
[go: up one dir, main page]

Skip to content

ci(v3): binary-size gate report-only until #551 - #680

Merged
adityamukho merged 1 commit into
v3from
ci/v3-size-gate-report-only
Oct 21, 2026
Merged

adityamukho merged 1 commit into
v3from
ci/v3-size-gate-report-only

Conversation

@adityamukho
Copy link
Copy Markdown
Collaborator

Makes the v3 binary-size check report-only, temporarily.

#549 PR 1 (#676) left the release binary at 1,259 of 1,260 KiB, with #549 PR 2, #550 and #551 still to land while the streaming engine and the old executor both ship.

  • binary-size.yml on v3 still builds the release binary and prints its size. Above the 1,260 KiB limit it emits a ::warning:: instead of failing (SIZE_GATE: report).
  • PHILOSOPHY.md §4 says the same.
  • Restore: when streaming engine 9/9: remove the old executor, measure, docs (#473) #551 removes the executor's query path, set SIZE_GATE back to fail with SIZE_LIMIT_BYTES: 1228800 (1,200 KiB).

main is unchanged: its gate stays enforced.

🤖 Generated with Claude Code

https://claude.ai/code/session_019kY61Y11gcRD91Y6b7jbGT

#549 PR 1 (#676) left the release binary at 1259 of 1260 KiB with #549 PR 2,
#550 and #551 still to land while the engine and the executor both ship. The
check still builds and prints the size and warns above the limit; it no longer
fails. It returns to an enforced 1200 KiB when #551 removes the executor's
query path.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019kY61Y11gcRD91Y6b7jbGT
@codecov
codecov Bot commented Oct 21, 2026
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@adityamukho
adityamukho merged commit 985171c into v3 Oct 21, 2026
24 of 25 checks passed
@adityamukho
adityamukho deleted the ci/v3-size-gate-report-only branch October 21, 2026 08:49
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.

1 participant

0