fix: Detect raw-to-parameterized route upgrades in cross-usage scenarios

- Add isParameterizedRoute() helper to detect if route has parameters (:id) or wildcards (*)
- Update shouldSkipNavigation() to detect raw path to parameterized route upgrades
- Enables proper updateName() calls in cross-usage scenarios
- Fixes 3 failing cross-usage navigation tests

In cross-usage scenarios (e.g., wrapCreateBrowserRouter + wrapUseRoutes), the first wrapper
may create a span with the raw URL path (e.g., /second-level/321/third-level/123) if nested
routes aren't fully resolved yet. The second wrapper should then upgrade it to the parameterized
form (e.g., /second-level/:id/third-level/:id).

Previously, this upgrade wasn't detected because we only checked for wildcard-to-parameterized
and length-based specificity. Now we also detect raw-to-parameterized upgrades.

Author: onurtemizkan

packages/react/src/reactrouter-compat-utils/instrumentation.tsx

@@ -96,6 +96,14 @@ export function computeLocationKey(location: Location): string {
   return `${location.pathname}${location.search || ''}${location.hash || ''}`;
 }
 
+/**
+ * Checks if a route name is parameterized (contains route parameters like :id or wildcards like *)
+ * vs a raw URL path.
+ */
+function isParameterizedRoute(routeName: string): boolean {
+  return routeName.includes(':') || routeName.includes('*');
+}
+
 /**
  * Determines if a navigation should be skipped as a duplicate, and if an existing span should be updated.
  * Exported for testing.
@@ -122,14 +130,17 @@ export function shouldSkipNavigation(
     // This allows cross-usage scenarios where the second wrapper has more complete route info
     const currentHasWildcard = !!trackedNav.routeName && transactionNameHasWildcard(trackedNav.routeName);
     const proposedHasWildcard = transactionNameHasWildcard(proposedName);
+    const currentIsParameterized = !!trackedNav.routeName && isParameterizedRoute(trackedNav.routeName);
+    const proposedIsParameterized = isParameterizedRoute(proposedName);
 
     const isWildcardUpgrade = currentHasWildcard && !proposedHasWildcard;
+    const isRawToParameterized = !currentIsParameterized && proposedIsParameterized;
     const isMoreSpecific =
       proposedName !== trackedNav.routeName &&
       proposedName.length > (trackedNav.routeName?.length || 0) &&
       !proposedHasWildcard;
 
-    const shouldUpdate = !!(trackedNav.routeName && (isWildcardUpgrade || isMoreSpecific));
+    const shouldUpdate = !!(trackedNav.routeName && (isWildcardUpgrade || isRawToParameterized || isMoreSpecific));
 
     return { skip: true, shouldUpdate };
   }
@@ -141,17 +152,21 @@ export function shouldSkipNavigation(
       // Check if we should update the span name with a better route
       // Allow updates if:
       // 1. Current has wildcard and new doesn't (wildcard → parameterized upgrade)
-      // 2. New name is different and more specific (longer, indicating nested routes resolved)
+      // 2. Current is raw path and new is parameterized (raw → parameterized upgrade)
+      // 3. New name is different and more specific (longer, indicating nested routes resolved)
       const currentHasWildcard = !!trackedNav.routeName && transactionNameHasWildcard(trackedNav.routeName);
       const proposedHasWildcard = transactionNameHasWildcard(proposedName);
+      const currentIsParameterized = !!trackedNav.routeName && isParameterizedRoute(trackedNav.routeName);
+      const proposedIsParameterized = isParameterizedRoute(proposedName);
 
       const isWildcardUpgrade = currentHasWildcard && !proposedHasWildcard;
+      const isRawToParameterized = !currentIsParameterized && proposedIsParameterized;
       const isMoreSpecific =
         proposedName !== trackedNav.routeName &&
         proposedName.length > (trackedNav.routeName?.length || 0) &&
         !proposedHasWildcard;
 
-      const shouldUpdate = !!(trackedNav.routeName && (isWildcardUpgrade || isMoreSpecific));
+      const shouldUpdate = !!(trackedNav.routeName && (isWildcardUpgrade || isRawToParameterized || isMoreSpecific));
 
       return { skip: true, shouldUpdate };
     }
@@ -802,7 +817,9 @@ export function handleNavigation(opts: {
           // Update placeholder's route name - the real span will be created with this name
           trackedNav.routeName = name;
           DEBUG_BUILD &&
-            debug.log(`[Tracing] Updated placeholder navigation name from "${oldName}" to "${name}" (will apply to real span)`);
+            debug.log(
+              `[Tracing] Updated placeholder navigation name from "${oldName}" to "${name}" (will apply to real span)`,
+            );
         } else {
           // Update existing real span from wildcard to parameterized route name
           trackedNav.span.updateName(name);