stx-router 0.2.288 → 0.2.290

package/dist/client.js

@@ -1190,6 +1190,13 @@ else {
         log('[router] full body swap for layout change');
         // Replace entire body content \u2014 layout chrome and all
         var bodyHTML=newBody.innerHTML;
+        // The outgoing chrome's components are destroyed with it (#1958).
+        // They are no longer re-run on every navigation, so they live as long
+        // as their layout does, and _cleanupContainer above only covered the
+        // container: their onDestroy and their scopes waited for the next
+        // navigation's orphan sweep, and their effects were never disposed.
+        // Before the swap, while the old markup is still here to walk.
+        if(window.stx&&window.stx._cleanupContainer)window.stx._cleanupContainer(document.body);
         document.body.innerHTML=bodyHTML;
         // Copy body attributes (class, data-stx, etc.)
         Array.from(newBody.attributes).forEach(function(attr){document.body.setAttribute(attr.name,attr.value)});
@@ -1275,6 +1282,18 @@ else {
           if(isSignalsRuntimeScript(s,text))return;
           if(text.indexOf('__stxRouter')!==-1)return;
           if(newContent.contains(s))return;
+          // Code only, as prepareRoutedBodyScripts takes it. A layout island's
+          // type="stx/island" script is hydrated by its own trigger, and one
+          // collected here was set up on every navigation, the trigger or not.
+          var scriptType=(s.getAttribute('type')||'').trim().toLowerCase();
+          if(scriptType&&scriptType!=='text/javascript'&&scriptType!=='application/javascript'&&scriptType!=='module')return;
+          // A component with no scope root has no root for the owner check,
+          // so its script is marked data-stx-instance instead. Out here it
+          // belongs to the layout chrome, which this swap leaves in place with
+          // its instance running; running it again set up a second one, and a
+          // stx.mount wrapper re-run from here mounted onto the container, the
+          // incoming page's content (#1958). Fragments leave these out.
+          if(s.hasAttribute('data-stx-instance'))return;
           if(text.indexOf('__stx_setup_')===-1&&!s.hasAttribute('data-stx-scoped')&&!s.hasAttribute('data-stx-page')&&!s.hasAttribute('data-stx-route-params'))return;
           var key=text.substring(0,80);
           if(seenSetups[key])return;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "stx-router",
   "type": "module",
-  "version": "0.2.288",
+  "version": "0.2.290",
   "description": "File-based router for STX — .stx template discovery, nested layouts, typed route params, middleware, and client-side SPA navigation.",
   "author": "Chris Breuer <chris@stacksjs.org>",
   "license": "MIT",