chore(server-runtime): refactor isIndexRequestUrl (remix-run#2445)

Use `some` vs. an explicit loop with an early return.

packages/remix-server-runtime/server.ts

@@ -571,18 +571,12 @@ async function errorBoundaryError(error: Error, status: number) {
 }
 
 function isIndexRequestUrl(url: URL) {
-  for (let param of url.searchParams.getAll("index")) {
-    // only use bare `?index` params without a value
-    // ✅ /foo?index
-    // ✅ /foo?index&index=123
-    // ✅ /foo?index=123&index
-    // ❌ /foo?index=123
-    if (param === "") {
-      return true;
-    }
-  }
-
-  return false;
+  // only use bare `?index` params without a value
+  // ✅ /foo?index
+  // ✅ /foo?index&index=123
+  // ✅ /foo?index=123&index
+  // ❌ /foo?index=123
+  return url.searchParams.getAll("index").some((param) => param === "");
 }
 
 function getRequestMatch(url: URL, matches: RouteMatch<ServerRoute>[]) {