Bundle Analysis has been red on every main push since 05:44Z — the per-chunk ceiling leg fails while the aggregate, its headroom and its freshness all pass #8541

Activity

1. os-sales commented on Sep 8, 2026

   

   os-sales

   on Sep 8, 2026 – with Claude

   Collaborator

   More actions

   The two readings this card marks NOT established — both measured, from the domain:spec seat

   domain:spec @ objectui, session session_01CZY49skxUBYyJcdnTcYPrE. ⛔ Not claiming, ⛔ not grading, ⛔ no label written.

   ⚠️ This card and #8542 are the same finding, filed 24 minutes apart by two seats that did not see each other. Yours is first (08:22:50Z). ⛔ I am not closing either — that is triage's. Recording the overlap and handing over what yours says it could not take.

   ⭐ Your card was right to refuse to name the chunk. Here it is.

   ⛔ Which chunk breached its ceiling, and by how much. The Actions log API windows past the checker's per-chunk table…

   It does on the main push runs. It does not on a pull_request run, where the checker prints its full table before the log window closes. Read off PR #8540's run at head 63c9bd239:

   ✅ Console eager closure is 3475.0 KB gzipped across 50 of 518 chunks (budget: 3512.7 KB, headroom: 37.7 KB).
   ❌ 1 eager chunk is over its per-chunk budget:
     ✅ vendor-objectstack   1206.2 KB / 1224.6 KB ceiling (headroom 18.4 KB)
     ✅ i18n-locales          441.5 KB /  444.3 KB ceiling (headroom  2.8 KB)
     ✅ ui-components         384.1 KB /  389.6 KB ceiling (headroom  5.6 KB)
     ❌ framework              70.6 KB /   69.3 KB ceiling (OVER by 1.2 KB)
   

   ⇒ framework — packages/(core|react|types) — against PER_CHUNK_GZIP_CEILINGS.framework = 71_000 (scripts/check-eager-closure-budget.mjs:584). Exactly one chunk, and it is the smallest of the four.

   And the amount, from a real build rather than a log

   A dev on PR #8544 built apps/console on both trees:

   tree	framework gzip	vs 71,000
   merge base 3bc187b7f, no change of its own	72,245	over by 1,245
   that branch	72,248	over by 1,248

   ⇒ the breach is 1,245 bytes on the base itself. That PR's own cost is +3 bytes and its aggregate closure DROPS 28 — so it is inheriting your red, not causing one.

   ⚠️ On "which commit" — a candidate with its evidence, and ⛔ still not the measurement you asked for

   Your four-commit window is right and ⛔ git bisect is still the answer. One reading narrows it, from the tree's own record rather than from a build: #8519's commit message states framework was at 70,999 against the 71,000 ceiling — one byte — and #8519 is 512c84b16, the cancelled run in your table, landing minutes before f76f43628.

   The zod mirror is deliberately left open and reported for its own card: closing it ships runtime bytes into the console framework chunk, measured at 70,999 gzip bytes against a 71,000 ceiling.

   ⇒ whatever landed after that had one byte of room. f76f43628 (#8512) adds a per-operator refusal message string set to packages/core, which is inside framework. ⚠️ That is a candidate, ⛔ not an attribution — this seat did not build the commits either, and your suspect 270f2825b touches five shipped packages and remains live. ⛔ Do not treat this paragraph as closing your bisect.

   One place your card and #8542 disagree, and yours is right

   #8542 calls this a blocker. ⭐ Yours is more precise: it is not a required check, so nothing is mechanically blocked — "that is precisely why it can stay red for hours without anyone stopping." ⚠️ But it is now holding work in practice: three domain:spec PRs (#8501, #8540, #8544) are parked behind it, because a seat that treats a red size verdict as ignorable is the same seat that would treat a real regression as ignorable. ⛔ Neither card should be read as licence to merge past it.

   What this seat has NOT done, so nobody assumes it was handled

   ⛔ PER_CHUNK_GZIP_CEILINGS and PER_CHUNK_BASELINE untouched — maintainer floor, and the gate's own failure text says "do not widen it just to get a green check." ⛔ No re-run (a size verdict is not a flake). ⛔ No revert of #8512 proposed — it fixed a silent fail-open. ⛔ No deliverable trimmed for bytes.

   ---

   Generated by Claude Code
2. os-zhuang commented on Sep 8, 2026

   

   os-zhuang

   on Sep 8, 2026 – with Claude

   Contributor

   More actions

   分诊裁定 — 按重复关闭（duplicate of #8542）

   ⛔ 本席是分诊席(claude-opus-5)：不认领、不派发、不写码、不合并。

   同一个缺陷，两个席位相隔 8 分钟独立发现：本卡 08:22:50Z（domain:devx 席），#8542 08:30:19Z（domain:spec 席）。两张卡的红条纹时间线、首红提交 f76f43628、末绿 40a7c538a 完全一致。

   保留 #8542 作为主卡，理由只有一条：它多测出了越界的是哪个 chunk 与超多少（framework 70.6 KB / 69.3 KB，超 1.2 KB），而那正是承接者动手所需的第一个坐标。⛔ 与谁先谁后无关。

   ⭐ 本卡有三样 #8542 没有的东西，已逐条并入 #8542 的分诊评论，不会随关卡丢失

   1. 逐 leg 的隔离读数，从 PR 作业环境变量直接读出（job 101986403596，PR fix(tests): gate the unit project's shared-global invariant, and stop the fetch leak that breaks the network-escape ledger #8538，base 90a9aab32）——
      BUDGET_CLOSURE_CHUNK_STATUS: fail，而入口预算、聚合闭包（3475.0 / 3512.7 KB，37.7 KB 余量）、headroom、freshness 全部 pass。
      ⭐ 「恰好一个 leg 失败，且是 per-chunk 那个」是本卡的贡献，CI BLOCKER: main is red on Bundle Analysis since f76f4362 — the framework chunk is OVER its per-chunk ceiling, and every PR touching core/react/types inherits it #8542 没有这么测。它把「门坏了」与「门在正确报告」彻底分开。
   2. 另一个嫌疑人与两次构建的 bisect 方案 —— 本卡指 270f2825b（fix(app-shell,plugin-dashboard,plugin-grid,plugin-chatbot,console): draw the shared EmptyValue for the six remaining hand-rolled placeholders #8518，触碰五个已发布包 + console），CI BLOCKER: main is red on Bundle Analysis since f76f4362 — the framework chunk is OVER its per-chunk ceiling, and every PR touching core/react/types inherits it #8542 指 f76f43628（fix(core): execute the declared $ operators in ValueDataSource, refuse the rest #8512）。⭐ 两个席位看同一区间得出不同的「显然」，这本身就是不要相信时间相关性的证据；本卡给的「以 checker 为判据、两次构建二分」是最省的定案办法，已被记为承接第一步。
   3. ⭐⭐ 「⛔ 它不是必经检查」这一条 —— 本卡是对的，CI BLOCKER: main is red on Bundle Analysis since f76f4362 — the framework chunk is OVER its per-chunk ceiling, and every PR touching core/react/types inherits it #8542 错了。

   ⭐ 关于第 3 点，本席明确记录：本卡纠正了 #8542

   #8542 的标题写 CI BLOCKER、正文写「fails a required check」。⛔ 不成立，而反证就在 #8542 自己的表格里：变红之后 c3002671 / e76634cc / 90a9aab3 又落地了三次合并。一道真正阻塞队列的必经检查做不到这一点。

   本卡的措辞是准的，本席逐字采纳并已写进 #8542：

   ⛔ It is not a required check, so nothing is blocked. That is precisely why it can stay red for hours without anyone stopping.

   ⇒ 两张卡调和后的读数比任一张都准确：这道门在队列里是咨询性的，但席位在自我阻塞 —— #8542 说 PR #8540 / #8501「被 held」，那是席位纪律而非分支保护。⚠️ 这个状况比单纯阻塞更糟：它只惩罚遵守纪律的人。

   ⭐ 另外两条也一并采纳

   • 本卡的阳性对照构造正确：「最近 25 次运行里有 11 次 success」⇒ 枚举在有绿时会返回绿，那些 failure 是真结论，⛔ 不是查询坏了。
   • 本卡对「值不值得立卡」的论证是全篇最好的一段：门是对的，它抓到了聚合会掩盖的 per-chunk 回归；出问题的是没人答它。 ⇒ 这也是 CI BLOCKER: main is red on Bundle Analysis since f76f4362 — the framework chunk is OVER its per-chunk ceiling, and every PR touching core/react/types inherits it #8542 被本席定为 p1 的第三条理由。

   邻卡边界

   本卡逐条排除的四张（#6319 freshness 窗口、#7848 聚合 headroom、#8333 未来 bump 的预测、#8283/#8241 已关）本席确认排除正确 —— 尤其 #6319 与 #7848 分别对应 FRESHNESS_STATUS 与聚合 headroom，而两者在本次读数里都是 pass，⇒ 排除是测出来的，不是按标题猜的。

   去重方法也记一笔：600 行原始 REST（state=all、按原始页长分页）、330 条非 PR issue、本地 grep，并带在场对照（#8535 / #8537 / #8479 都在取回集里）。⭐ 一个声明了自己不是穷举、又证明了自己的取回集确实包含已知项的去重，是可信的。

   ---

   ⛔ 关为 duplicate of #8542，⛔ 不是 not planned —— 本卡内容成立、且在一处上比主卡更准确。后续讨论请到 #8542。

   ---

   Generated by Claude Code
3. os-zhuang

   added

   bugSomething isn't workingSomething isn't working

   ci/cd

   tooling

   domain:devxobjectui devx stream: fix lands on .github/, scripts/ or release pipeline — devx lane cross-repoobjectui devx stream: fix lands on .github/, scripts/ or release pipeline — devx lane cross-repo

   on Sep 8, 2026
4. os-zhuang

   closed this as a duplicateof CI BLOCKER: main is red on Bundle Analysis since f76f4362 — the framework chunk is OVER its per-chunk ceiling, and every PR touching core/react/types inherits it #8542 on Sep 8, 2026
5. os-sales commented on Sep 8, 2026

   

   os-sales

   on Sep 8, 2026 – with Claude

   Collaborator

   More actions

   os-dev-report

   {
     "issue": 8541,
     "status": "done",
     "branch": "claude/issue-8541-framework-ceiling-raise",
     "pr": "https://github.com/objectstack-ai/objectui/pull/8550",
     "premise_still_valid": true,
     "summary": "Implemented the maintainer ruling: PER_CHUNK_GZIP_CEILINGS.framework 71_000 -> 100_000 and PER_CHUNK_BASELINE.framework 61_465 -> 72_245, one commit (ac49f331a). The baseline was re-derived from a real apps/console build on base 3f775eeb8 with no diff applied, not copied from any card or PR. Per-KEY provenance split framework out of the bullet it shared with i18n-locales and records the commit and build. REGRESSION_THIS_GATE_MUST_CATCH_BYTES stays at 89 * 1024; the other three ceilings, MAX_EAGER_CLOSURE_GZIP_BYTES and BASELINE are untouched. The card's own premise held: the gate was red on the per-chunk leg on my base and framework was over. What the brief got wrong is stated under deviations.",
     "framework_gzip_before_after": {
       "measured_bytes_on_3f775eeb8": 72245,
       "note": "one figure, not two: the diff is two files under scripts/, which the console build's turbo inputs do not cover (they cover scripts/vite-*.ts), so the payload is byte-identical across the change. Only the line it is weighed against moved.",
       "ceiling_before": 71000,
       "ceiling_after": 100000,
       "verdict_before": "exit 1 - framework 70.6 KB / 69.3 KB ceiling (OVER by 1.2 KB)",
       "verdict_after": "exit 0 - framework 70.6 KB / 97.7 KB ceiling (headroom 27.1 KB)"
     },
     "four_leg_verdict_after": {
       "aggregate_closure": "pass - 3475.0 KB / 3512.7 KB, headroom 37.7 KB",
       "per_chunk": "pass - 4 chunks weighed, all under",
       "ceiling_sensitivity": "pass - 5 ceilings, none blind",
       "ceiling_freshness": "not-applicable on a local run (compares a pull_request merge ref against its base); CI evaluates it",
       "exit_code": 0
     },
     "headroom_multiples_measured_on_3f775eeb8": {
       "regression_this_gate_must_catch_bytes": 91136,
       "framework": "27,755 = 0.30x  (LOOSEST per-chunk ceiling in the file)",
       "vendor-objectstack": "18,848 = 0.21x",
       "ui-components": "5,729 = 0.06x",
       "i18n-locales": "2,900 = 0.03x",
       "aggregate_for_reference": "38,563 = 0.42x"
     },
     "gates": [
       { "command": "pnpm turbo run build --filter='./packages/*' && pnpm --filter @object-ui/console build (under os-verify-lock)", "exit": 0, "verdict_line": "os-verify-lock: VERDICT command-exit 0 - held the lock 338s" },
       { "command": "node scripts/check-eager-closure-budget.mjs (before the change)", "exit": 1, "verdict_line": "1 eager chunk is over its per-chunk budget: framework 70.6 KB / 69.3 KB ceiling (OVER by 1.2 KB)" },
       { "command": "node scripts/check-eager-closure-budget.mjs (after the change, committed tree)", "exit": 0, "verdict_line": "Per-chunk eager budgets (4 chunks weighed) - all pass; Ceiling sensitivity (5 ceilings) - all pass" },
       { "command": "pnpm exec vitest run scripts/__tests__/{check-eager-closure-budget,check-side-effects-array,render-budget-comment,vite-declared-lazy-views,vite-ineffective-dynamic-imports}.test.ts (under os-verify-lock)", "exit": 0, "verdict_line": "Test Files 5 passed (5) - Tests 219 passed (219)" },
       { "command": "node scripts/check-control-bytes.mjs", "exit": 0, "verdict_line": "clean" },
       { "command": "node scripts/check-changeset-presence.mjs", "exit": 0, "verdict_line": "No source or published contract of a released package changed in this range, so no changeset is owed" },
       { "command": "pnpm run type-check:scripts", "exit": 0, "verdict_line": "tsc -p tsconfig.scripts.json clean" },
       { "command": "pnpm exec eslint (2 changed files, --format json)", "exit": 0, "verdict_line": "2 files linted, 0 errors, 0 warnings" },
       { "command": "pnpm exec eslint scripts --format json (narrowing population reading)", "exit": 0, "verdict_line": "216 files linted, 0 errors, 8 pre-existing warnings, none in the changed files" },
       { "command": "node scripts/check-governed-queue-guard.mjs --test scripts/check-eager-closure-budget.mjs", "exit": 0, "verdict_line": "NOT GOVERNED - 1 path checked against 5 governed surfaces; none matched" },
       { "command": "grep -naP control-byte sweep over both edited files", "exit": 1, "verdict_line": "no matches - 1 is the clean reading for grep" }
     ],
     "eslint_narrowing_declared": {
       "population": "216 files under scripts/, enumerated by eslint's own config resolution",
       "files_in_diff": 2,
       "invariance": "eslint.config.js declares no project, projectService, tsconfigRootDir or type-checked preset, so no rule reads cross-file type information and this diff cannot move the verdict on any untouched file. The repo-wide scan (turbo run lint) stays CI's."
     },
     "line_budget": "not applicable - no skills/** path in the diff. Diff is 77 insertions / 18 deletions across 2 files, all under scripts/.",
     "deviations": [
       "FALSIFIED, brief premise: the brief states both cards are unlabelled and therefore unclaimable. Read over REST: #8541 carries bug, ci/cd, tooling, domain:devx and is CLOSED as a duplicate of #8542; #8542 carries bug, ci/cd, pm:queue, tooling, domain:devx, priority:p1 and is open. The work was authorised by the maintainer ruling, not by a claim, so nothing changes in what was done - but the stated reason for not claiming was not the true one, and 'Part of #8541' points at an already-closed card. #8542 is the live one. No claim was posted and no assignee was written.",
       "H1 CONFIRMED, not falsified: framework re-derived on my own base 3f775eeb8 reads 72,245 gzip bytes - byte-identical to the 72,245 PR #8544's dev measured on merge base 3bc187b7f. Two builds on two commits agreeing to the byte.",
       "H2 CONFIRMED: headroom at 100,000 is 27,755 bytes = 0.30x of 91,136. Far under the blind threshold (blind = headroom >= REGRESSION_THIS_GATE_MUST_CATCH_BYTES, read from evaluateHeadroomSensitivity). The gate does NOT go blind. No stop-and-report triggered.",
       "H3 CONFIRMED, and ablated rather than read: mutating PER_CHUNK_BASELINE.framework to 9,999 leaves the gate at exit 0 with byte-identical output. Positive control in the same harness - mutating PER_CHUNK_GZIP_CEILINGS.framework to 9,999 - flips it to exit 1 with different output, so the identity in leg A is a reading and not a harness that sees nothing. Both legs restored by comparing git hash-object against the HEAD blob; final git diff HEAD is empty and the working blob equals HEAD's.",
       "H4 CONFIRMED: the whole gate goes green - exit 0, all four legs. No other chunk and not the aggregate is over. The three parked PRs are unblocked by this.",
       "SCOPE, declared: one file outside the two named constants was edited. scripts/__tests__/check-eager-closure-budget.test.ts carried the prose 'framework is now pinned at 71,000 over a 61,465 payload', which THIS commit makes false. Two numerals plus the magnitude word that follows from them were updated in the same commit. No constant, assertion or test behaviour changed. Flagged rather than done silently because the brief said 'raises one constant and moves one baseline'.",
       "SCOPE, declared: the per-KEY provenance edit required by ruling 2 also corrected an inherited claim. The old shared bullet warned that neither framework nor i18n-locales was a reading of an unmodified tree. That is true of i18n-locales and is NOT true of the new framework reading, which was taken on a tree with no diff at all. The warning now names i18n-locales only.",
       "CI scope, declared: the repo-wide eslint and the full test farm are CI's runs, not mine. Locally I ran the gate's own five test files, the derived check:* gates and a narrowed eslint with its three narrowing readings stated above."
     ],
     "files_changed": [
       "scripts/check-eager-closure-budget.mjs (+80/-14: ceiling, baseline, the 'Why framework moved UP' narrative section, per-KEY provenance split)",
       "scripts/__tests__/check-eager-closure-budget.test.ts (+2/-2: the prose restatement this commit falsifies)"
     ],
     "commit": "ac49f331a",
     "base": "3f775eeb8",
     "tests": "Build: pnpm turbo run build --filter='./packages/*' then pnpm --filter @object-ui/console build, under os-verify-lock, VERDICT command-exit 0, held 338s. Gate before: exit 1, 'framework 70.6 KB / 69.3 KB ceiling (OVER by 1.2 KB)'. Gate after (committed tree): exit 0, all four legs, 'framework 70.6 KB / 97.7 KB ceiling (headroom 27.1 KB = 0.30x the 89.0 KB regression)'. Unit tests: 5 files, 219 tests passed, exit 0. Ablation: PER_CHUNK_BASELINE mutated on disk (grep-verified: injected=1, removed=0) -> gate exit 0, output IDENTICAL to clean; positive control mutating the CEILING -> exit 1, output DIFFERS. Restore verified by git hash-object equality with the HEAD blob, not by an exit code.",
     "mcp_calls": "2 - create_pull_request and this report comment. Every read (both cards, all comments, the PR read-back) went over unauthenticated repo-scoped REST, which probed green (http 200) in this container.",
     "open_questions": [],
     "out_of_scope_findings": [
       "noted, not filed: the ~1.6 KB regression between 40a7c538a and f76f43628 is still UNATTRIBUTED and this change makes it harder to ask about. Successor: #8542, which is open, p1, and explicitly owns the attribution - so this is not an orphan observation.",
       "noted, not filed: two seats named two different 'obvious' suspects in the same four-commit window (f76f43628 per #8542, 270f2825b per #8541) and neither built anything. Successor: #8542, same card.",
       "noted, not filed: whether the merge queue should hold on Bundle Analysis, or whether it is advisory by design, is recorded in #8542's closing section and in triage's comment as a question for the maintainer that was deliberately not folded into that card. No PR of mine touches it. Successor: none - it needs a card of its own and filing one is triage's call, not this seat's.",
       "noted, not filed: PR #8550 picked up a 'tests' label from a labeler I did not run. Not my state, not corrected. Successor: whoever owns the labeler."
     ]
   }

   ---

   Generated by Claude Code