Skip to content

Switch active testnet to Summit + fix jsxDEV compile crash#421

Merged
UtkarshBhardwaj007 merged 2 commits into
mainfrom
feat/summit-switch-and-jsx-crash-fix
Jun 18, 2026
Merged

Switch active testnet to Summit + fix jsxDEV compile crash#421
UtkarshBhardwaj007 merged 2 commits into
mainfrom
feat/summit-switch-and-jsx-crash-fix

Conversation

@UtkarshBhardwaj007

Copy link
Copy Markdown
Member

Summary

Two changes, each with its own changeset:

1. Switch active testnet to Web3 Summit (minor)

Flips ACTIVE_TESTNET_ENV to "summit" in src/config.ts — the single constant that threads through DEFAULT_ENV, the legacy testnet alias, the token symbol (SUM), descriptor/endpoint selection, and the CDM meta-registry resolution (w3s).

  • The config.test.ts divergence guard (summit endpoints must match polkadot-app-deploy's bundled environments.json) and the non-empty-registry guard both pass.
  • The summit playground-registry leaf (0x49091Eb6…) is not stored in the repo — it resolves live at runtime from the w3s CDM meta-registry (0xa5747e60…, shipped in @parity/cdm-env@2.0.6). Verified live against summit Asset Hub.
  • @parity/polkadot-app-deploy@0.11.0 (our current pin) already ships the full summit env (endpoints + DotNS contracts), all confirmed on-chain.

2. Fix jsxDEV compile crash (patch)

Every UI-rendering command crashed with jsx_dev_runtime.jsxDEV is not a function. bun 1.3's bundler emits the development JSX automatic-runtime (jsxDEV) unconditionally, but --define process.env.NODE_ENV='"production"' flips React's inlined jsx-dev-runtime onto its production branch, which never assigns jsxDEV → undefined → crash at first Ink render. This shipped in v0.43.5/v0.43.6.

  • Removes the define from all bun build --compile invocations (build, cli:install, and the three release workflows).
  • No production-preserving alternative exists in bun 1.3.13 (bun issue #23959; fix pending in bun PR #31651). The define can return once that lands — documented as a tripwire in CLAUDE.md.
  • Trade-off: React dev-mode warnings return for end users; strictly better than a 100%-crash binary.
  • --version/--help were unaffected (they don't render Ink).

Verification

  • pnpm typecheck, pnpm test (1005/1005), pnpm format:check, pnpm lint:license — all green.
  • Crash fix reproduced and validated by building both binaries: WITH the define crashes, WITHOUT renders cleanly (and shows · summit, confirming both changes together).

Follow-ups (not in this PR)

  • A CI smoke test that runs the compiled binary against one Ink command — unit tests run under Node and cannot catch this class of bug.
  • Refresh a stale CLAUDE.md sentence claiming the direct-chain layer is hardwired to paseo_* descriptors (it is now env-gated).

Flip ACTIVE_TESTNET_ENV to "summit". This single constant threads
through DEFAULT_ENV, the legacy testnet alias, token symbol (SUM),
descriptor/endpoint selection, and the CDM meta-registry resolution
(w3s). The config.test.ts divergence guard + non-empty-registry guard
gate the flip and pass.
The compiled binary crashed on every UI-rendering command with
"jsx_dev_runtime.jsxDEV is not a function". bun 1.3's bundler emits
the development JSX automatic-runtime (jsxDEV) unconditionally, but
--define process.env.NODE_ENV='"production"' flips React's inlined
jsx-dev-runtime onto its production branch, which never assigns jsxDEV
-> undefined -> crash at first Ink render. Shipped in v0.43.5/v0.43.6.

Remove the define from all bun build --compile invocations (build,
cli:install, and the three release workflows). Document the tripwire in
CLAUDE.md (bun issue #23959, fix pending in bun PR #31651); restore the
define once that lands. --version/--help were unaffected.
@UtkarshBhardwaj007 UtkarshBhardwaj007 merged commit aa07095 into main Jun 18, 2026
15 of 20 checks passed
@UtkarshBhardwaj007 UtkarshBhardwaj007 deleted the feat/summit-switch-and-jsx-crash-fix branch June 18, 2026 09:05
@github-actions

Copy link
Copy Markdown
Contributor

E2E Test Pass · ❌ FAIL

Tag: e2e-ci-pr · Branch: feat/summit-switch-and-jsx-crash-fix · Commit: 764704f · Run logs

Cell Result Time
pr-deploy-cdm ✅ PASS 3m48s
pr-login-session ❌ FAIL 4m24s
pr-deploy-foundry ✅ PASS 0m51s
pr-deploy-frontend ❌ FAIL 3m23s
pr-preflight ❌ FAIL 2m39s
pr-mod ❌ FAIL 2m37s
pr-install ✅ PASS 0m50s
${{ matrix.cell }} ⏭️ SKIP 0m00s
${{ matrix.cell }} ⏭️ SKIP 0m00s

Sentry traces: view spans for this run

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