@jsenv/navi 0.16.13 → 0.16.15

package/dist/jsenv_navi.js

@@ -8709,11 +8709,17 @@ const createRoutePattern = (pattern) => {
       }
       // Include parameters that target pattern specifically needs
       if (targetQueryParamNames.has(paramName)) {
-        ancestorParams[paramName] = value;
-        if (DEBUG$2) {
-          console.debug(
-            `[${pattern}] tryDirectOptimization: Added target param ${paramName}=${value}`,
-          );
+        // Only include if the value is not the default value
+        const connection = targetAncestor.connections.find(
+          (conn) => conn.paramName === paramName,
+        );
+        if (connection && connection.options.defaultValue !== value) {
+          ancestorParams[paramName] = value;
+          if (DEBUG$2) {
+            console.debug(
+              `[${pattern}] tryDirectOptimization: Added target param ${paramName}=${value}`,
+            );
+          }
         }
       }
       // Include source query parameters (these should be inherited during ancestor optimization)
@@ -8745,6 +8751,39 @@ const createRoutePattern = (pattern) => {
       }
     }
 
+    // Also check target ancestor's own signal values for parameters not in resolvedParams
+    for (const connection of targetAncestor.connections) {
+      const { paramName, signal, options } = connection;
+      const defaultValue = options.defaultValue;
+
+      // Only include if not already processed and has non-default value
+      if (
+        !(paramName in ancestorParams) &&
+        signal?.value !== undefined &&
+        signal.value !== defaultValue
+      ) {
+        // Don't include path parameters that correspond to literal segments we're optimizing away
+        const targetParam = targetParams.find((p) => p.name === paramName);
+        const isPathParam = targetParam !== undefined; // Any param in segments is a path param
+        if (isPathParam) {
+          // Skip path parameters - we want them to use default values for optimization
+          if (DEBUG$2) {
+            console.debug(
+              `[${pattern}] tryDirectOptimization: Skipping path param ${paramName}=${signal.value} (will use default)`,
+            );
+          }
+          continue;
+        }
+
+        ancestorParams[paramName] = signal.value;
+        if (DEBUG$2) {
+          console.debug(
+            `[${pattern}] tryDirectOptimization: Added target signal param ${paramName}=${signal.value}`,
+          );
+        }
+      }
+    }
+
     // Then, get all ancestors starting from the target ancestor's parent (skip the target itself)
     const targetAncestorRelationships = patternRelationships.get(
       targetAncestor.originalPattern,
@@ -10807,9 +10846,6 @@ const navTo = (target, options) => {
   }
   return browserIntegration.navTo(url, options);
 };
-const replaceUrl = (target, options = {}) => {
-  return navTo(target, { ...options, replace: true });
-};
 const stopLoad = (reason = "stopLoad() called") => {
   const windowIsLoading = windowIsLoadingSignal.value;
   if (windowIsLoading) {
@@ -11195,10 +11231,6 @@ const initRouteObserver = ({
     // ensure we re-render on document url change (useful when navigating from /users/list to /users)
     // so that we re-replace urls back to /users/list when /users/list is an index
     useDocumentUrl();
-    if (matchingRouteInfo && matchingRouteInfo.index && !matchingRouteInfo.route.matching) {
-      const routeUrl = matchingRouteInfo.route.routeFromProps.buildUrl();
-      replaceUrl(routeUrl);
-    }
     return jsx(RouteInfoContext.Provider, {
       value: matchingRouteInfo,
       children: jsx(SlotContext.Provider, {