@invisibleloop/pulse 0.21.1 → 0.21.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invisibleloop/pulse",
-  "version": "0.21.1",
+  "version": "0.21.3",
   "type": "module",
   "description": "AI-first frontend framework. The spec is the source of truth.",
   "license": "MIT",
@@ -76,7 +76,7 @@
   "scripts": {
     "dev": "node --watch src/cli/dev.js",
     "build": "node scripts/build.js",
-    "test": "node src/spec/schema.test.js && node src/runtime/runtime.test.js && node src/runtime/navigate.test.js && node src/runtime/morph.test.js && node src/runtime/ssr.test.js && node src/runtime/image.test.js && node src/server/server.test.js && node src/cli/cli.test.js && node src/cli/validate.test.js && node src/cli/diagnose.test.js && node src/cli/check-bundles.test.js && node src/cli/review.test.js && node src/cli/process-utils.test.js && node src/cli/stamp.test.js && node src/cli/feedback.test.js && node src/cli/update.test.js && node src/cli/dev-host.test.js && node src/cli/dev-reload.test.js && node src/ui/ui.test.js && node src/testing/testing.test.js && node src/store/store.test.js && node src/md/md.test.js && node scripts/strip-server.test.js && node scripts/css-purge.test.js && node scripts/release-version.test.js && node src/mcp/mcp.test.js && node src/mcp/guards.test.js",
+    "test": "node src/spec/schema.test.js && node src/runtime/runtime.test.js && node src/runtime/navigate.test.js && node src/runtime/morph.test.js && node src/runtime/ssr.test.js && node src/runtime/image.test.js && node src/server/server.test.js && node src/cli/cli.test.js && node src/cli/validate.test.js && node src/cli/diagnose.test.js && node src/cli/check-bundles.test.js && node src/cli/review.test.js && node src/cli/process-utils.test.js && node src/cli/stamp.test.js && node src/cli/feedback.test.js && node src/cli/update.test.js && node src/cli/dev-host.test.js && node src/cli/dev-reload.test.js && node src/ui/ui.test.js && node src/testing/testing.test.js && node src/store/store.test.js && node src/md/md.test.js && node scripts/strip-server.test.js && node scripts/css-purge.test.js && node scripts/bootstrap.test.js && node scripts/release-version.test.js && node src/mcp/mcp.test.js && node src/mcp/guards.test.js",
     "typecheck": "tsc --noEmit",
     "docs": "node --watch src/cli/dev.js --root ../../docs --port 4000",
     "docs:start": "node ../../docs/server.js"

package/public/.pulse-ui-version

@@ -1 +1 @@
-0.21.1
+0.21.3

package/scripts/bootstrap.js

@@ -0,0 +1,25 @@
+/**
+ * Source of the production boot bundle entry generated for each hydrated page by
+ * scripts/build.js. Its own module (build.js runs the whole build when imported) so the
+ * generated code can be unit-tested.
+ *
+ * The first page's mount() instance is handed to initNavigation so the first client
+ * navigation destroys it; without that its listeners outlive the page (#168).
+ */
+export function buildBootstrapSource({ relSpec, relRuntime, relNavigate, storeImport = '', mountOpts = '{ ssr: true }' }) {
+  return `\
+import spec from '${relSpec}'
+import { mount } from '${relRuntime}'
+import { initNavigation } from '${relNavigate}'${storeImport}
+
+const root = document.getElementById('pulse-root')
+if (root && !root.dataset.pulseMounted) {
+  root.dataset.pulseMounted = '1'
+  const instance = mount(spec, root, window.__PULSE_SERVER__ || {}, ${mountOpts})
+  // Register the first page's instance so the first client navigation destroys it
+  initNavigation(root, mount).setMount(instance)
+}
+
+export default spec
+`
+}

package/scripts/build.js

@@ -21,6 +21,7 @@ import { discoverPages } from '../src/cli/discover.js'
 import { renderToString } from '../src/runtime/ssr.js'
 import { c, header, table, ok, fail, info, warn, elapsed, icon } from '../src/cli/fmt.js'
 import { extractUsedClasses, purgeCss, minifyCss } from './css-purge.js'
+import { buildBootstrapSource } from './bootstrap.js'
 
 // ---------------------------------------------------------------------------
 // Server-only property stripping
@@ -499,20 +500,7 @@ const bootstrapFiles = pages.flatMap(({ filePath }) => {
   const storeImport = STORE_PATH ? `\nimport store from '${path.relative(TMP_DIR, STORE_PATH)}'` : ''
   const mountOpts   = STORE_PATH ? `{ ssr: true, store }` : `{ ssr: true }`
 
-  fs.writeFileSync(bootstrapPath, `\
-import spec from '${relSpec}'
-import { mount } from '${relRuntime}'
-import { initNavigation } from '${relNavigate}'${storeImport}
-
-const root = document.getElementById('pulse-root')
-if (root && !root.dataset.pulseMounted) {
-  root.dataset.pulseMounted = '1'
-  mount(spec, root, window.__PULSE_SERVER__ || {}, ${mountOpts})
-  initNavigation(root, mount)
-}
-
-export default spec
-`)
+  fs.writeFileSync(bootstrapPath, buildBootstrapSource({ relSpec, relRuntime, relNavigate, storeImport, mountOpts }))
 
   return [{ filePath, bootstrapPath, name }]
 })

package/src/runtime/ssr.js

@@ -298,8 +298,9 @@ export function buildHydrateScript(spec, storeDef, nonce) {
   import { mount } from '/@pulse/runtime/index.js'
   import { initNavigation } from '/@pulse/runtime/navigate.js'${storeImport}
   const root = document.getElementById('pulse-root')
-  mount(spec, root, window.__PULSE_SERVER__ || {}${storeArg})
-  initNavigation(root, mount)
+  const instance = mount(spec, root, window.__PULSE_SERVER__ || {}${storeArg})
+  // Register the first page's instance so the first client navigation destroys it
+  initNavigation(root, mount).setMount(instance)
 </script>`
 }
 

package/src/spec/schema.js

@@ -124,17 +124,18 @@
 
 /**
  * Validate a spec object against the schema.
- * Returns { valid: true } or { valid: false, errors: string[] }
+ * Returns { valid, errors, warnings } — `warnings` is always an array, including on the early
+ * returns (non-object spec, raw-response spec), because callers spread it.
  *
  * @param {unknown} spec
- * @returns {{ valid: boolean, errors: string[] }}
+ * @returns {{ valid: boolean, errors: string[], warnings: string[] }}
  */
 export function validateSpec(spec) {
   const errors   = []
   const warnings = []
 
   if (!spec || typeof spec !== 'object') {
-    return { valid: false, errors: ['Spec must be a plain object'] }
+    return { valid: false, errors: ['Spec must be a plain object'], warnings }
   }
 
   // route — '*' is the custom not-found page, rendered with status 404 when no route matches
@@ -191,7 +192,7 @@ export function validateSpec(spec) {
       errors.push('spec.render is required when spec.contentType is set — (ctx, server) => string')
     }
     // Raw specs don't need state or view — skip those checks
-    return { valid: errors.length === 0, errors }
+    return { valid: errors.length === 0, errors, warnings }
   }
 
   // state

package/types/navigate.d.ts

@@ -14,7 +14,9 @@ import type { PulseSpec } from './schema.js'
  * swaps #pulse-root, and re-mounts the new spec without a full page reload.
  * Falls back to location.href on any fetch or import error.
  *
- * Call once after the initial mount().
+ * Call once after the initial mount(), and pass that instance to the returned
+ * `setMount` so the first navigation destroys it. Without it the first page's
+ * listeners stay alive and every handler present on both pages runs twice.
  *
  * @param root    The #pulse-root element
  * @param mountFn The mount() function from @invisibleloop/pulse/runtime
@@ -24,8 +26,8 @@ import type { PulseSpec } from './schema.js'
  * import { initNavigation } from '@invisibleloop/pulse/navigate'
  *
  * const root = document.getElementById('pulse-root')
- * const m = mount(spec, root, window.__PULSE_SERVER__ ?? {}, { ssr: true })
- * initNavigation(root, mount)
+ * const instance = mount(spec, root, window.__PULSE_SERVER__ ?? {}, { ssr: true })
+ * initNavigation(root, mount).setMount(instance)
  */
 export function initNavigation(
   root:    HTMLElement,
@@ -35,4 +37,7 @@ export function initNavigation(
     serverState?: Record<string, unknown>,
     options?:    { ssr?: boolean; store?: unknown }
   ) => MountResult
-): void
+): {
+  /** Register the first page's mount so the first navigation destroys it. */
+  setMount(instance: MountResult | null | undefined): void
+}