@escape-game-over/atlas 0.1.28 → 0.1.30

package/docs/NOT-BUILT.md

@@ -500,27 +500,22 @@ looking at" better than any link can.
 
 ### Should `/en-US/about-us` redirect when the default locale is unprefixed?
 
-**No. Only `/en-US` itself does.**
+**Yes, by one wildcard — not by a rule per page.**
 
 With `prefixDefaultLocale: false` the default locale is served at `/about-us`
-and nothing is built under `/en-US/`. Claiming the prefixed root is one rule and
-answers the URL people actually type, having seen every other language wear a
-prefix. Claiming the pages under it is a different proposition: a rule per page,
-and slugs are translated per locale, so the URL a reader lands on by editing
-`/el-GR/sxetika-me-emas` down to English keeps the *Greek* slug —
+and nothing is built under `/en-US/`. Claiming the pages under it one rule at a
+time does not work: slugs are translated per locale, so the URL a reader lands
+on by editing `/el-GR/sxetika-me-emas` down to English keeps the *Greek* slug —
 `/en-US/sxetika-me-emas`. Covering that means the cross product of the prefix
-with every locale's spelling of every slug, which is a `_redirects` of hundreds
-against the 2000 a host reads, all of it for URLs nothing has ever linked.
-
-This was built first, in the one-rule-per-default-locale-page form, which is
-worth naming because it looks complete and is not: it answers `/en-US/about-us`
-and misses `/en-US/sxetika-me-emas`, so the case that motivated it is the case
-it does not cover.
-
-A wildcard would sidestep the count — `/en-US/* /:splat 301` is one line — but
-not the translated slug, and it is Cloudflare's syntax rather than something
-`ResolvedRedirect` can carry to another host. The pages get the 404 page, which
-is what it is for.
+with every locale's spelling of every slug, a `_redirects` of hundreds against
+the 2000 a host reads.
+
+`/en-US/* /:splat 301` is one line instead. It still does not cover the
+translated slug — `/en-US/sxetika-me-emas` goes to `/sxetika-me-emas`, which
+404s — but that URL 404ed without the rule too, so it is never worse and it
+answers every English slug. It is Cloudflare's syntax, which is what every site
+deploys to. It is written last, so a stated `/en-US/old` rule still fires, and
+left out if anything is built under the prefix.
 
 ---
 

package/docs/toolchain.md

@@ -93,14 +93,16 @@ slug or a switched-off page moves the rule with the page:
 | -------------- | ------------- | ---------------------------------------------- |
 | `/`            | `/en-US`      | `prefixDefaultLocale: true` leaves `/` unowned |
 | `/en-US`       | `/`           | `prefixDefaultLocale: false` leaves it unbuilt |
+| `/en-US/*`     | `/:splat`     | the same, for every page under the prefix      |
 | `/news/page/1` | `/news`       | page one is the bare slug, never `page/1`      |
 
 The first two are the same rule in its two directions: whichever root the
 routing mode does not serve points at the one it does. Both together cost one
-rule; the third costs one per paginated list per language. Nothing here scales
-with the number of pages a site has — see [`NOT-BUILT.md`](NOT-BUILT.md) for why
-the prefixed form of every *page* is left to the 404 instead. A project that
-states its own rule for one of these paths keeps it: the inferred one steps
+rule. The wildcard costs one more, and it is written last in the file, because
+Cloudflare fires the first rule that matches and a stated `/en-US/old` rule has
+to be reached before it. The page-one rule costs one per paginated list per
+language. Nothing here scales with the number of pages a site has. A project
+that states its own rule for one of these paths keeps it: the inferred one steps
 aside.
 
 The root points at whichever route has an empty slug, so nothing has to name a

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@escape-game-over/atlas",
-    "version": "0.1.28",
+    "version": "0.1.30",
     "type": "module",
     "description": "Typed, data-driven machinery for static multi-locale, multi-deployment Astro sites.",
     "private": false,
@@ -54,7 +54,7 @@
         "@biomejs/biome": "2.5.14",
         "@types/node": "26.6.2",
         "@vitest/coverage-istanbul": "5.0.1",
-        "astro": "7.3.3",
+        "astro": "7.3.4",
         "typescript": "6.0.3",
         "vitest": "5.0.1"
     }

package/src/redirects.ts

@@ -101,11 +101,12 @@ export type ValidateRedirectTargets<Rules> = {
  * against what this project builds and its URL is derived — a redirect cannot
  * outlive the page it points at, or miss a slug that was retranslated. The
  * locale is optional and defaults to the site's own, since an old URL usually
- * predates translation.
+ * predates translation. `page` names a page of a paginated list, and is refused
+ * the same way `pathFor` refuses it when the list has no such page.
  */
 export type RedirectTarget<Id extends string, L extends string> =
     | ExternalUrl
-    | { readonly route: Id; readonly locale?: L }
+    | { readonly route: Id; readonly locale?: L; readonly page?: number }
     /**
      * A file served verbatim from `public/` — a PDF, a spreadsheet.
      *

package/src/site/api.ts

@@ -309,14 +309,11 @@ export interface Site<
      * **What comes back is more than what went in.** lib adds the rules the
      * route table implies, for the URLs its own design leaves unpublished but
      * reachable: whichever site root the routing mode does not serve — `/` when
-     * every locale is prefixed, `/en-US` when the default locale is not — and
-     * `/news/page/1` for a list whose page one is the bare path. A rule you
-     * state for one of those paths replaces the inferred one, so pass `[]` and
-     * you still get a file worth writing.
-     *
-     * Pages under an unused locale prefix are *not* claimed: that is a rule per
-     * page per language, and it still misses the reader who edits a translated
-     * slug's prefix. See `NOT-BUILT.md`.
+     * every locale is prefixed, `/en-US` when the default locale is not — the
+     * pages under that unused `/en-US/`, as one `/en-US/* /:splat` wildcard
+     * written last, and `/news/page/1` for a list whose page one is the bare
+     * path. A rule you state for one of those paths replaces the inferred one,
+     * so pass `[]` and you still get a file worth writing.
      *
      * Returns the rules as data. Rendering is a separate step —
      * `buildCloudflareRedirects` writes the `_redirects` that Cloudflare and

package/src/site/create.ts

@@ -513,10 +513,8 @@ export function createSite<
      * switched-off page moves the rule with it instead of leaving one pointing
      * at a 404.
      *
-     * Two, and deliberately not a third per *page*: mirroring every URL a
-     * project builds is a rule per page, and with slugs translated per locale
-     * it is a rule per page per language — a file of hundreds against the 2000
-     * a host reads, for URLs nothing ever linked. See `NOT-BUILT.md`.
+     * The pages under an unused prefix are `prefixSplat`'s, as one wildcard
+     * rather than a rule per page per language. See `NOT-BUILT.md`.
      *
      * Both permanent. Neither is a state that changes while the config stays as
      * it is: page one will never live at `/page/1`, and the root will not move
@@ -570,6 +568,32 @@ export function createSite<
         return [...root, ...pageOne];
     }
 
+    /**
+     * `/en-US/about` → `/about` when the default locale is unprefixed: the
+     * pages under the prefix `inferredRedirects` points at the root, in one
+     * Cloudflare wildcard. A translated slug under the wrong prefix still ends
+     * on the 404, one hop later — no worse than without it.
+     *
+     * Kept apart because it has to be written last: Cloudflare fires the first
+     * rule that matches, and a wildcard ahead of a stated `/en-US/old` rule
+     * would swallow it. Skipped when anything is built under the prefix, which
+     * the wildcard would make unreachable.
+     */
+    function prefixSplat(): readonly ResolvedRedirect[] {
+        if (prefixDefaultLocale) return [];
+        const prefix = `/${defaultLocale}/`;
+        for (const path of builtPaths) {
+            if (path.startsWith(prefix)) return [];
+        }
+        return [
+            {
+                from: `${prefix}*`,
+                to: "/:splat",
+                status: statusFor("permanent"),
+            },
+        ];
+    }
+
     function redirects(
         rules: readonly RedirectRule<RouteId, L>[]
     ): readonly ResolvedRedirect[] {
@@ -577,17 +601,20 @@ export function createSite<
             from: rule.from,
             // Three kinds of target, told apart by shape: a string is external
             // and passes through, `file` is served verbatim from `public/`, and
-            // a route is resolved to whatever URL it has in the locale asked
-            // for.
+            // a route is resolved to whatever URL it has in the locale and on
+            // the page asked for.
             to: ((): string => {
                 if (typeof rule.to === "string") return rule.to;
                 if ("file" in rule.to) return rule.to.file;
-                return pathFor(rule.to.route, rule.to.locale ?? defaultLocale);
+                return pathFor(rule.to.route, rule.to.locale ?? defaultLocale, {
+                    page: rule.to.page,
+                });
             })(),
             status: statusFor(rule.kind),
         }));
 
         const claimed = new Set(stated.map((rule) => rule.from));
+        const splat = prefixSplat().filter((rule) => !claimed.has(rule.from));
         const inferred = inferredRedirects().filter(
             // Dropped rather than reported, both times, because neither is a
             // mistake anyone made: a rule the project wrote for one of these
@@ -602,7 +629,7 @@ export function createSite<
             // So a rule that shadows a real page is rejected rather than
             // sitting dead in the file.
             builtPaths,
-            rules: [...inferred, ...stated],
+            rules: [...inferred, ...stated, ...splat],
         });
     }