add case if no pathname set

Author: nicohrubec

dev-packages/e2e-tests/test-applications/react-router-7-framework/app/routes/performance/index.tsx

@@ -8,6 +8,7 @@ export default function PerformancePage() {
         <Link to="/performance/ssr">SSR Page</Link>
         <Link to="/performance/with/sentry">With Param Page</Link>
         <Link to={{ pathname: '/performance/with/object-nav', search: '?foo=bar' }}>Object Navigate</Link>
+        <Link to={{ search: '?query=test' }}>Search Only Navigate</Link>
         <Link to="/performance/server-loader">Server Loader</Link>
       </nav>
     </div>

dev-packages/e2e-tests/test-applications/react-router-7-framework/tests/performance/navigation.client.test.ts

@@ -81,6 +81,29 @@ test.describe('client - navigation performance', () => {
     });
   });
 
+  test('should create navigation transaction when navigating with search-only object `to` prop', async ({ page }) => {
+    const txPromise = waitForTransaction(APP_NAME, async transactionEvent => {
+      return transactionEvent.transaction === '/performance' && transactionEvent.contexts?.trace?.op === 'navigation';
+    });
+
+    await page.goto(`/performance`); // pageload
+    await page.waitForTimeout(1000); // give it a sec before navigation
+    await page.getByRole('link', { name: 'Search Only Navigate' }).click(); // navigation with search-only object to
+
+    const transaction = await txPromise;
+
+    expect(transaction).toMatchObject({
+      contexts: {
+        trace: {
+          op: 'navigation',
+          origin: 'auto.navigation.react_router',
+        },
+      },
+      transaction: '/performance',
+      type: 'transaction',
+    });
+  });
+
   test('should update navigation transaction for dynamic routes', async ({ page }) => {
     const txPromise = waitForTransaction(APP_NAME, async transactionEvent => {
       return transactionEvent.transaction === '/performance/with/:param';

packages/react-router/src/client/utils.ts

@@ -1,11 +1,22 @@
 /**
- * Resolves a navigate argument to a string path.
+ * Resolves a navigate argument to a pathname string.
  *
- * React Router's navigate() accepts a string, number, or To object ({ pathname, search, hash }).
+ * React Router's navigate() accepts a string, number, or a To object ({ pathname, search, hash }).
+ * All fields in the To object are optional (Partial<Path>), so we need to detect object args
+ * to avoid "[object Object]" transaction names.
  */
 export function resolveNavigateArg(target: unknown): string {
-  if (typeof target === 'object' && target !== null && 'pathname' in target) {
-    return (target as { pathname?: string }).pathname || '/';
+  if (typeof target !== 'object' || target === null) {
+    // string or number
+    return String(target);
   }
-  return String(target);
+
+  // Object `to` with pathname
+  const pathname = (target as Record<string, unknown>).pathname;
+  if (typeof pathname === 'string') {
+    return pathname || '/';
+  }
+
+  // Object `to` without pathname - navigation stays on current path
+  return (typeof window !== 'undefined' && window.location?.pathname) || '/';
 }