Skip to content

Fix Nuxt cold-start runtime identity and verify native replay - #78

Merged
Narrator merged 2 commits into
mainfrom
codex/next-nuxt-portable-replay
Oct 25, 2026
Merged

Narrator merged 2 commits into
mainfrom
codex/next-nuxt-portable-replay

Conversation

@Narrator

@Narrator Narrator commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

On a cold Nuxt 3 start, the native plugin could initialize a raw ESM RuntimeManager while application queries imported a separate Vite-prebundled copy. The page and overlay rendered, but the queried runtime never became ready. Exclude the runtime from prebundling, preserving existing excludes, so both paths share the initialized adapter.

Extend portable replay regression coverage to native Next App Router (15/Webpack and 16/Turbopack) and Nuxt SSR (3/4). Add these four cases to existing CI, alongside React/Vue Vite cases. Check live committed state, stale handles after unmount, fresh-session replay and correct UI updates, console/page/network errors, and conditional production instrumentation removal. Build/server process groups now have bounded cleanup with retained failure logs.

Validation:

  • Meaningful RED→GREEN: two fresh Nuxt3/npm cold starts failed readiness; browser diagnostics showed raw vs optimized runtime imports. An isolated exclusion confirmed the cause, unit regression failed before the fix, then a fresh cold consumer passed.
  • 2,114 uncached tests passed across all 11 configured test projects. Nuxt configured lint/build and 41 tests passed; dist synchronization passed. No dependency or lockfile changes.
  • Eight fresh local-tarball consumer combinations passed eight real Chromium assertions each: React18/Vue3 with npm/Vite7 and pnpm/Vite5; Next15/npm, Next16/pnpm, Nuxt3/npm, Nuxt4/pnpm. Node24.22.1 / Chromium156. After the process helper review fix, Next16, Nuxt3 and Vue/pnpm were rerun successfully.
  • Real website verification (Next16.4.3 / React19.2.4) passed exact InstallTabs source/state among six instances, persisted replay in a fresh session, safe rejection after navigation, mobile state/replay after reload, and controlled HMR handle retirement/restoration. Production build and real UI passed without instrumentation. Local Vercel hosting script404s were classified separately; both deployed endpoints return200. Temporary website config/source changes were restored. HMR manifest propagation can briefly lag DOM IDs; documented bounded source-resolution retry. An exported real-site annotation was also imported into a fresh relay in a separate source snapshot on a different HTTP port after the original server stopped. Its unchanged trace completed, with correct live state and source mapping under the new directory; stable source IDs and stale old handles were verified. This does not establish an agent efficiency/adoption advantage.
  • Independent review found one build-timeout cleanup gap, reproduced and fixed. Two process regression tests passed, including a TERM-ignoring parent and worker; focused re-review found no material issues.
  • Formatting, Node syntax, workflow YAML parse and diff checks passed. One local install hit /tmp inode exhaustion; preserved evidence, cleaned completed temporary dependencies, and verified a fresh install successfully.

These are local-artifact installs; pnpm explicitly overrides Domscribe dependencies to those tarballs. This is not a published-package release or full router/version matrix. The APIs remain limited to main-document light DOM and explicit replay steps. No publication, deployment, settings changes or issue-to-PR GitHub Action product is included.

@nx-cloud

nx-cloud Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 0e7a87a

Command Status Duration Result
nx run domscribe-test-fixtures:install-fixture-... ✅ Succeeded 18s View ↗
nx run domscribe-test-fixtures:install-fixture-... ✅ Succeeded 15s View ↗
nx run domscribe-test-fixtures:integration--web... ✅ Succeeded 2m 11s View ↗
nx run domscribe-test-fixtures:integration--web... ✅ Succeeded 2m 3s View ↗
nx run domscribe-test-fixtures:integration--web... ✅ Succeeded 1m 31s View ↗
nx run domscribe-test-fixtures:integration--web... ✅ Succeeded 1m 31s View ↗
nx run domscribe-test-fixtures:install-fixture-... ✅ Succeeded 43s View ↗
nx run domscribe-test-fixtures:install-fixture-... ✅ Succeeded 42s View ↗
Additional runs (18) ✅ Succeeded ... View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-10-25 22:52:29 UTC

@Narrator
Narrator merged commit 92e939a into main Oct 25, 2026
40 checks passed
@Narrator
Narrator deleted the codex/next-nuxt-portable-replay branch October 25, 2026 22:58
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