Skip to content

chore(playground): bump main chunk size limit 1750 → 1800 KB#23107

Merged
Thunkar merged 1 commit into
merge-train/fairiesfrom
claudebox/fix-merge-train-fairies
May 8, 2026
Merged

chore(playground): bump main chunk size limit 1750 → 1800 KB#23107
Thunkar merged 1 commit into
merge-train/fairiesfrom
claudebox/fix-merge-train-fairies

Conversation

@AztecBot

@AztecBot AztecBot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fix CI failure on merge-train/fairies (run 25572561023, log 1778264338812903).

The merge of next into merge-train/fairies (78632af) brought in barretenberg changes — feat!: optimized Poseidon2 (#22652) and feat: n1 apps (#22974) via feat: merge-train/barretenberg (#23025) — which grew bb.js by enough to push the playground main entrypoint 0.02 KB over the 1750 KB hard limit:

❌ assets/index-BEz4wQVA.js: 1750.02 KB exceeds limit of 1750 KB (Main entrypoint, hard limit)

Bumping the limit to 1800 KB (matching the prior +50 KB increment from the bump log) restores headroom.

Test plan

  • CI green on this PR (chunk size check passes)

ClaudeBox log: https://claudebox.work/s/22af49d0486fc035?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 8, 2026
@Thunkar Thunkar marked this pull request as ready for review May 8, 2026 18:41
@Thunkar Thunkar enabled auto-merge (squash) May 8, 2026 18:41
@AztecBot

AztecBot commented May 8, 2026

Copy link
Copy Markdown
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/da3a4e5c6b70a2df�da3a4e5c6b70a2df8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/multiple_validators_sentinel.parallel.test.ts "collects attestations for validators in proposer node when block is not published" (264s) (code: 0) group:e2e-p2p-epoch-flakes

@Thunkar Thunkar merged commit 4ea70f6 into merge-train/fairies May 8, 2026
50 of 68 checks passed
@Thunkar Thunkar deleted the claudebox/fix-merge-train-fairies branch May 8, 2026 19:00
github-merge-queue Bot pushed a commit that referenced this pull request May 10, 2026
BEGIN_COMMIT_OVERRIDE
fix: include sqlite binary in its npm package (#23039)
fix: add sendMessagesAs to wallet api schemas (#23041)
refactor(pxe): deduplicate tx hash lookups in MessageContextService
(#23075)
refactor(pxe): batch tagged private log queries across all secrets
(#23048)
refactor(pxe): batch log RPC calls in LogService.fetchLogsByTag (#23088)
feat(pxe,nr): flesh out account stubs and don't exclude syncing for
overrides (#23054)
feat: deploy method refactor 2 (#23033)
feat: fastForwardContractUpdate cheatcode for simulating contract
updates (#22905)
refactor(stdlib): consolidate find-function-by-selector helpers (#23008)
feat(ci): Snapshots for aztec-nr contract compilation failures and nargo
expand (#23061)
chore: kv store test fully on vitest (#23096)
refactor(pxe): skip redundant getBlock RPC when querying at anchor block
(#23100)
chore(playground): bump main chunk size limit 1750 → 1800 KB (#23107)
feat(txe): allow authorizing cross-contract utility calls in nr tests
(#23064)
chore: bench public fns with emit repro (#23105)
END_COMMIT_OVERRIDE
rangozd pushed a commit to rangozd/aztec-packages that referenced this pull request May 16, 2026
…AztecProtocol#23223)

## What

Bumps the hard chunk-size cap on the playground's main entrypoint chunk
(`assets/index-*.js`) in `playground/vite.config.ts` from **1800 KB →
1850 KB**, plus a bump-log entry.

## Why

The `merge-train/barretenberg` PR (AztecProtocol#23137) is repeatedly failing in the
merge queue on a **deterministic** `yarn build` failure in
`playground/`:

```
❌ Chunk size validation failed:
  ❌ assets/index-Bc65D_YP.js: 1801.31 KB exceeds limit of 1800 KB (Main entrypoint, hard limit)
error during build: Error: Build failed: chunk size limits exceeded
```

(merge-queue log: `ci.aztec-labs.com/a34536031aaca0b9`)

The 1800 KB limit was only raised four days ago (AztecProtocol#23107, 1750 → 1800).
The merge train bundles further bb-side changes on top of `next` —
notably `feat: multi-app kernel circuits (AztecProtocol#23076)` — which nudged bb.js
~1.3 KB past the just-raised ceiling. It will fail on every queue
attempt until the cap is raised. 1850 KB restores a small amount of
headroom.

## Note

This lands the bump on `next`. To unblock AztecProtocol#23137 immediately, `next`
needs to be re-merged into `merge-train/barretenberg` once this merges
(or the same one-liner can be applied directly on that branch — note
that branch's earlier conflict resolution dropped one of the two `=>
1800` bump-log comment lines, so expect a trivial conflict on the
comment block when re-merging).

No behavior change; CI build only.

---
*Created by
[claudebox](https://claudebox.work/v2/sessions/7ad309a3cd56537a) ·
group: `slackbot`*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants