@jsenv/navi 0.16.35 → 0.16.36

package/dist/jsenv_navi.js

@@ -10291,6 +10291,9 @@ const resolveRouteUrl = (relativeUrl) => {
  */
 
 
+// Flag to prevent signal-to-URL synchronization during URL-to-signal synchronization
+let isUpdatingRoutesFromUrl = false;
+
 // Controls what happens to actions when their route stops matching:
 // 'abort' - Cancel the action immediately when route stops matching
 // 'keep-loading' - Allow action to continue running after route stops matching
@@ -10386,6 +10389,9 @@ const updateRoutes = (
     }
 
     // URL -> Signal synchronization (moved from individual route effects to eliminate circular dependency)
+    // Prevent signal-to-URL synchronization during URL-to-signal synchronization
+    isUpdatingRoutesFromUrl = true;
+
     for (const {
       route,
       routePrivateProperties,
@@ -10539,6 +10545,9 @@ const updateRoutes = (
     }
   });
 
+  // Reset flag after URL -> Signal synchronization is complete
+  isUpdatingRoutesFromUrl = false;
+
   // must be after paramsSignal.value update to ensure the proxy target is set
   // (so after the batch call)
   const toLoadSet = new Set();
@@ -10739,12 +10748,19 @@ const registerRoute = (routePattern) => {
     }
     // Signal -> URL sync: When signal changes, update URL to reflect meaningful state
     // Only sync non-default values to keep URLs clean (static fallbacks stay invisible)
+    // eslint-disable-next-line no-loop-func
     effect(() => {
       const value = paramSignal.value;
       const params = rawParamsSignal.value;
       const urlParamValue = params[paramName];
       const matching = matchingSignal.value;
 
+      // Signal returned to default - clean up URL by removing the parameter
+      // Skip cleanup during URL-to-signal synchronization to prevent recursion
+      if (isUpdatingRoutesFromUrl) {
+        return;
+      }
+
       if (!matching) {
         // Route not matching, no URL sync needed
         return;
@@ -10764,6 +10780,19 @@ const registerRoute = (routePattern) => {
         route.replaceParams({ [paramName]: value });
         return;
       }
+
+      // URL parameter exists - check if we need to update or clean it up
+      const defaultValue = connection.getDefaultValue();
+      if (value === defaultValue) {
+        if (debug) {
+          console.debug(
+            `[route] Signal->URL: ${paramName} cleaning URL (removing default value ${value})`,
+          );
+        }
+        route.replaceParams({ [paramName]: undefined });
+        return;
+      }
+
       if (value === urlParamValue) {
         // Values already match, no sync needed
         return;
@@ -10778,17 +10807,17 @@ const registerRoute = (routePattern) => {
   }
 
   route.navTo = (params) => {
-    if (!browserIntegration$1) {
+    if (!integration) {
       return Promise.resolve();
     }
     const routeUrl = route.buildUrl(params);
-    return browserIntegration$1.navTo(routeUrl);
+    return integration.navTo(routeUrl);
   };
   route.redirectTo = (params) => {
-    if (!browserIntegration$1) {
+    if (!integration) {
       return Promise.resolve();
     }
-    return browserIntegration$1.navTo(route.buildUrl(params), {
+    return integration.navTo(route.buildUrl(params), {
       replace: true,
     });
   };
@@ -10952,9 +10981,9 @@ const useRouteStatus = (route) => {
   };
 };
 
-let browserIntegration$1;
-const setBrowserIntegration = (integration) => {
-  browserIntegration$1 = integration;
+let integration;
+const setRouteIntegration = (integrationInterface) => {
+  integration = integrationInterface;
 };
 let onRouteDefined = () => {};
 const setOnRouteDefined = (v) => {
@@ -11539,7 +11568,7 @@ const browserIntegration = setupBrowserIntegrationViaHistory({
 setOnRouteDefined(() => {
   browserIntegration.init();
 });
-setBrowserIntegration(browserIntegration);
+setRouteIntegration(browserIntegration);
 
 const actionIntegratedVia = browserIntegration.integration;
 const navTo = (target, options) => {