@rangojs/router 0.0.0-experimental.d20dd405 → 0.0.0-experimental.dacec167

package/src/response-utils.ts

@@ -26,3 +26,12 @@ export function isWebSocketUpgradeResponse(response: Response): boolean {
     (response as unknown as { webSocket?: unknown }).webSocket != null
   );
 }
+
+// Location truthiness (not presence) so an empty `Location: ""` is not a redirect.
+export function isRedirectResponse(response: Response): boolean {
+  return (
+    response.status >= 300 &&
+    response.status < 400 &&
+    Boolean(response.headers.get("Location"))
+  );
+}

package/src/reverse.ts

@@ -231,47 +231,50 @@ export type LocalReverseParams<TPattern extends string> =
   };
 
 /**
- * Type-safe local reverse function with dot-prefixed names only.
+ * Type-safe local reverse function.
  *
  * Returned by `useReverse(routes)` on the client. The route map is the
  * exposure boundary (a generated `routes` from a `urls()` module) and the
- * scope is implicit from that import — there is no global namespace, so
- * names must be dot-prefixed to mirror `ctx.reverse(".name")`.
+ * scope is implicit from that import. Names may be written with or without a
+ * leading dot — `reverse("post")` and `reverse(".post")` are identical. The dot
+ * is a cosmetic readability convention (and parity with `ctx.reverse(".name")`);
+ * there is no separate global namespace here, so it carries no meaning.
  *
  * @example
  * ```typescript
  * const reverse = useReverse(blogRoutes);
- * reverse(".index");                         // ✓ no params
- * reverse(".post", { postId: "hello" });     // ✓ with params
- * reverse(".search", {}, { q: "hi" });       // ✓ with search schema
- * reverse(".typo");                          // ✗ compile error
+ * reverse("index");                          // ✓ no params (dot optional)
+ * reverse(".index");                         // ✓ identical to the above
+ * reverse("post", { postId: "hello" });      // ✓ with params
+ * reverse("search", {}, { q: "hi" });        // ✓ with search schema
+ * reverse("typo");                           // ✗ compile error
  * ```
  */
 export type LocalReverseFunction<TLocalRoutes> = {
   /**
-   * Dot-prefixed local route without params
+   * Route without params (leading dot optional)
    */
   <TName extends keyof TLocalRoutes & string>(
     name: IsEmptyObject<
       ExtractParams<RoutePatternFor<TLocalRoutes, TName>>
     > extends true
-      ? `.${TName}`
+      ? TName | `.${TName}`
       : never,
   ): string;
 
   /**
-   * Dot-prefixed local route with params
+   * Route with params (leading dot optional)
    */
   <TName extends keyof TLocalRoutes & string>(
-    name: `.${TName}`,
+    name: TName | `.${TName}`,
     params: LocalReverseParams<RoutePatternFor<TLocalRoutes, TName>>,
   ): string;
 
   /**
-   * Dot-prefixed local route with params and search
+   * Route with params and search (leading dot optional)
    */
   <TName extends keyof TLocalRoutes & string>(
-    name: `.${TName}`,
+    name: TName | `.${TName}`,
     params: LocalReverseParams<RoutePatternFor<TLocalRoutes, TName>>,
     search: ResolveSearchSchema<ExtractSearchSchema<TLocalRoutes, TName>>,
   ): string;

package/src/route-content-wrapper.tsx

@@ -4,7 +4,7 @@ import { Suspense, use, useId } from "react";
 import { invariant } from "./errors";
 import { OutletProvider } from "./outlet-provider.js";
 import type { ResolvedSegment } from "./types.js";
-import { isLoaderDataResult } from "./types.js";
+import { decodeLoaderResults } from "./decode-loader-results.js";
 
 /**
  * Stable async wrapper component for route content
@@ -26,10 +26,6 @@ export function RouteContentWrapper({
   fallback?: ReactNode;
   segmentId?: string;
 }): ReactNode {
-  if (!content) {
-    // Already resolved
-    return content as ReactNode;
-  }
   return (
     <Suspense
       fallback={fallback ?? null}
@@ -159,28 +155,10 @@ function LoaderResolver({
       ? use(loaderDataPromise)
       : loaderDataPromise;
 
-  // Build loaderData record from resolved values
-  const loaderData: Record<string, any> = {};
-  let loaderErrorFallback: ReactNode = null;
-
-  loaderIds.forEach((id, i) => {
-    const result = resolvedData[i];
-
-    if (isLoaderDataResult(result)) {
-      if (result.ok) {
-        loaderData[id] = result.data;
-      } else {
-        if (result.fallback) {
-          loaderErrorFallback = result.fallback;
-        } else {
-          throw new Error(result.error.message);
-        }
-      }
-    } else {
-      // Legacy format - direct data
-      loaderData[id] = result;
-    }
-  });
+  const { loaderData, errorFallback } = decodeLoaderResults(
+    resolvedData,
+    loaderIds,
+  );
 
   return (
     <OutletProvider
@@ -190,7 +168,7 @@ function LoaderResolver({
       parallel={parallel}
       loaderData={Object.keys(loaderData).length > 0 ? loaderData : undefined}
     >
-      {loaderErrorFallback ?? children}
+      {errorFallback ?? children}
     </OutletProvider>
   );
 }