Skip to content

Fix panel chunk re-split after lazy service clients#3946

Closed
koala73 wants to merge 1 commit into
mainfrom
codex/re-split-panel-chunks
Closed

Fix panel chunk re-split after lazy service clients#3946
koala73 wants to merge 1 commit into
mainfrom
codex/re-split-panel-chunks

Conversation

@koala73

@koala73 koala73 commented May 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • lazy-initialize generated service clients in panel modules so panel evaluation no longer creates TDZ-prone cross-chunk singletons
  • split panel components into documented domain chunks with lazy panel-support, while routing static-entry shared code to preloadable app-shared
  • keep app-shell panel imports type-only/dynamic and strengthen chunk guard tests

Tradeoff

Validation

  • node --test tests/chunk-assignment.test.mjs
  • npm run typecheck
  • npm run build:full
  • production preview smoke at http://127.0.0.1:4173/: HTTP 200, no page errors, hasTdzReferenceError: false
  • verified dist/index.html no longer modulepreloads panels-* or panel-support, and the built main chunk has no static from "./panel-support..." import

Fixes #3944

@vercel

vercel Bot commented May 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
worldmonitor Ready Ready Preview, Comment May 28, 2026 6:21am

Request Review

@koala73 koala73 added High Value Meaningful contribution to the project Not Ready to Merge PR has conflicts, failing checks, or needs work labels Jun 23, 2026
@koala73 koala73 closed this Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Value Meaningful contribution to the project Not Ready to Merge PR has conflicts, failing checks, or needs work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Follow-up: re-split panel chunks after lazy service-client initialization

1 participant