@invisibleloop/pulse 0.21.3 → 0.21.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invisibleloop/pulse",
-  "version": "0.21.3",
+  "version": "0.21.4",
   "type": "module",
   "description": "AI-first frontend framework. The spec is the source of truth.",
   "license": "MIT",

package/public/.pulse-ui-version

@@ -1 +1 @@
-0.21.3
+0.21.4

package/src/runtime/index.js

@@ -169,7 +169,9 @@ export function mount(spec, el, serverState = {}, options = {}) {
   // ---------------------------------------------------------------------------
 
   function persist() {
-    if (!persistKey) return
+    // The key is per route, not per instance: once destroyed, a late write (an action resolving
+    // after navigation) could overwrite what the next page, on the same route, has persisted.
+    if (!persistKey || _destroyed) return
     try {
       const snapshot = {}
       spec.persist.forEach(k => { snapshot[k] = state[k] })
@@ -184,6 +186,11 @@ export function mount(spec, el, serverState = {}, options = {}) {
   let lastHtml = null
 
   function render() {
+    // A destroyed instance never touches the DOM again. el now belongs to the page that replaced
+    // it, and an action that resolves late or a debounce timer that fires late would otherwise
+    // morph this page's view into that one (#172). State updates, toasts and store pushes from
+    // late work still go through; only the render (and persist(), below) is dropped.
+    if (_destroyed) return
     let html
     try {
       html = resolveView(spec, state, getEffectiveServerState())
@@ -303,8 +310,9 @@ export function mount(spec, el, serverState = {}, options = {}) {
     /** Force a re-render — useful after external state changes */
     refresh: render,
 
-    /** Tear down — remove event listeners, unsubscribe from store, clear element */
-    destroy: () => { _destroyed = true; _unsubStore?.(); _eventAbort.abort(); el.innerHTML = trustedHTML('') }
+    /** Tear down — remove event listeners, unsubscribe from store, clear element
+     *  (pass { keepDom: true } to leave the element's content in place) */
+    destroy: (opts) => { _destroyed = true; _unsubStore?.(); _eventAbort.abort(); if (!opts?.keepDom) el.innerHTML = trustedHTML('') }
   }
 }
 

package/src/runtime/navigate.js

@@ -26,6 +26,17 @@ export function initNavigation(root, mountFn) {
   let currentMount = null
   let currentNavController = null
 
+  // The target page has no client bundle (a purely server-rendered page), so nothing will mount
+  // over the previous page's instance. Release it, or its listeners keep firing on the new DOM
+  // and a store change re-renders the old view into it (#171). Call it immediately before the
+  // new HTML replaces the old (never earlier: if the response is cut off first, the visitor
+  // keeps a working page). keepDom: the caller swaps the root's content itself.
+  function releaseCurrentMount() {
+    const mount = currentMount
+    currentMount = null            // cleared first, so a throwing destroy() cannot leave a stale reference
+    mount?.destroy({ keepDom: true })
+  }
+
   async function navigate(path, push) {
     // Cancel any in-flight navigation before starting a new one
     currentNavController?.abort()
@@ -63,6 +74,10 @@ export function initNavigation(root, mountFn) {
             await applyScripts(msg.scripts)
 
           } else if (msg.type === 'html') {
+            if (!hydratePath) {
+              if (controller.signal.aborted) return   // superseded: leave the old page alone
+              releaseCurrentMount()
+            }
             root.innerHTML = trustedHTML(msg.html)
             // Index <pulse-deferred> placeholders so deferred chunks land in the right spot
             for (const id of (msg.deferred || [])) {
@@ -126,6 +141,7 @@ export function initNavigation(root, mountFn) {
           window.__updatePulseStore__(storeState)
         }
 
+        if (!hydrate) releaseCurrentMount()
         root.innerHTML = trustedHTML(html)
         document.title = title || document.title
         applyStyles(styles)

package/types/runtime.d.ts

@@ -27,8 +27,9 @@ export interface MountOptions {
 }
 
 export interface MountResult {
-  /** Destroy this mount, remove event listeners, and unsubscribe from the store. */
-  destroy(): void
+  /** Remove listeners and unsubscribe from the store, then clear the element.
+   *  Pass `{ keepDom: true }` to leave the element's content in place. */
+  destroy(options?: { keepDom?: boolean }): void
 }
 
 /**