@akanjs/devkit 3.0.0-beta.19 → 3.0.0-beta.20

package/artifact/routeSeedIndex.ts

@@ -1,29 +1,13 @@
 import path from "node:path";
 import { assertUniqueRoutePatterns, compareRouteSpecificity, parseRouteModuleKey } from "akanjs/common";
+import {
+  ROUTE_SEED_INDEX_JSON,
+  type RouteSeedEntry,
+  type RouteSeedIndex,
+} from "akanjs/server/artifact/routeSeedIndexStore";
 import type { PageEntry } from "./implicitRootLayout";
 
-/**
- * Build-time utilities for the route seed index. `computeRouteSeedIndex`
- * takes resolved `PageEntry` rows (`key` plus absolute module path — files
- * under `app/` or the generated implicit root layout) and produces — for every
- * route — the list of source files that seed the `"use client"` graph walk.
- *
- * The result is serialized to `route-seed-index.json` by
- * `saveRouteSeedIndex` so the runtime server can call
- * `loadRouteSeedIndex` to restore it without re-importing `pages.ts`
- * (which no longer exists) or reparsing loader sources.
- */
-
-export interface RouteSeedEntry {
-  routeId: string;
-  pattern: string;
-  seeds: string[];
-}
-
-export interface RouteSeedIndex {
-  entries: RouteSeedEntry[];
-  globalLayoutFiles: string[];
-}
+export { ROUTE_SEED_INDEX_JSON, type RouteSeedEntry, type RouteSeedIndex };
 
 export type SerializedRouteSeedEntry = Pick<RouteSeedEntry, "routeId"> &
   Partial<Pick<RouteSeedEntry, "pattern" | "seeds">>;
@@ -33,10 +17,6 @@ export interface SerializedRouteSeedIndex {
   globalLayoutFiles?: string[];
 }
 
-/**
- * Compute the route seed index from `PageEntry`s (disk paths or generated
- * implicit root layout absolute paths).
- */
 export function computeRouteSeedIndex(pageEntries: PageEntry[]): RouteSeedIndex {
   const layoutsByPrefix = new Map<string, string[]>();
   const pagesBySegments: Array<{
@@ -51,11 +31,8 @@ export function computeRouteSeedIndex(pageEntries: PageEntry[]): RouteSeedIndex
     const parsed = parseRouteModuleKey(key);
     const files = [path.resolve(moduleAbsPath), ...(seedAbsPaths ?? []).map((seed) => path.resolve(seed))];
     if (parsed.kind === "layout" || parsed.kind === "overrides") {
-      // Overrides seed the client graph like layouts: every route under the prefix must pull the generated
-      // `"use client"` override wrapper (and its slot components) into the client bundle / RSC client manifest.
-      const prefix = parsed.routeSegments.join("/");
-      const prev = layoutsByPrefix.get(prefix) ?? [];
-      layoutsByPrefix.set(prefix, [...prev, ...files]);
+      //? Every route under an overrides prefix needs its "use client" wrapper in the client graph, as with a layout.
+      layoutsByPrefix.getOrInsert(parsed.routeSegments.join("/"), []).push(...files);
     } else if (parsed.kind === "page") {
       pagesBySegments.push({
         key,
@@ -68,9 +45,7 @@ export function computeRouteSeedIndex(pageEntries: PageEntry[]): RouteSeedIndex
   }
   assertUniqueRoutePatterns(pagesBySegments);
 
-  const rootLayouts = layoutsByPrefix.get("") ?? [];
-  const globalLayoutFiles = rootLayouts;
-
+  const globalLayoutFiles = layoutsByPrefix.get("") ?? [];
   const seedEntries: RouteSeedEntry[] = [];
   for (const { pattern, segments, files, includeOwnLayout } of pagesBySegments) {
     const layouts: string[] = [];
@@ -84,26 +59,18 @@ export function computeRouteSeedIndex(pageEntries: PageEntry[]): RouteSeedIndex
     const routeId = pattern || "/";
     seedEntries.push({ routeId, pattern: routeId, seeds });
   }
-  // Static segments must beat `:param` ones at match time, otherwise
-  // `/persona/new` resolves to `[personaId]=new`. Sort once at build time
-  // so the runtime matcher can stay first-match-wins.
+  // Static segments must beat `:param` ones (`/persona/new` is not `[personaId]=new`); the runtime is first-match-wins.
   seedEntries.sort((a, b) => compareRouteSpecificity(a.pattern, b.pattern));
   return { entries: seedEntries, globalLayoutFiles };
 }
 
-export const ROUTE_SEED_INDEX_JSON = "route-seed-index.json";
-
 export function serializeRouteSeedIndexForArtifact(
   index: RouteSeedIndex,
   artifactDir: string,
   options: { production?: boolean } = {},
 ): SerializedRouteSeedIndex {
   const normalizedArtifactDir = path.resolve(artifactDir);
-  if (options.production) {
-    return {
-      entries: index.entries.map((entry) => ({ routeId: entry.routeId })),
-    };
-  }
+  if (options.production) return { entries: index.entries.map((entry) => ({ routeId: entry.routeId })) };
   return {
     entries: index.entries.map((entry) => ({
       ...entry,

package/biomeBase.ts

@@ -1,9 +1,6 @@
 /** `extends` target for a workspace `biome.json`; Biome resolves it through node_modules. */
 export const biomeBaseConfig = "@akanjs/devkit/biome.base.json";
 
-// Biome moves rules between groups across minors — `noUnnecessaryConditions` sat in `nursery` at 2.4 and moved to
-// `suspicious` at 2.5 — and the stale position is a hard "unknown key" error, not a warning. A workspace whose
-// Biome disagrees with the shipped base config therefore fails to load it at all, which is why the version is
-// pinned here instead of resolved to latest at create time. Bump this and `biome.base.json` in one commit, and run
-// `biome migrate --write` in the workspace root and in `pkgs/@akanjs/devkit` so both configs move together.
+// Pinned: Biome moves rules between groups across minors and a stale position is a hard config error. Bump it with
+// `biome.base.json` in one commit, running `biome migrate --write` in the workspace root and in this package.
 export const biomeVersion = "2.5.12";

package/capacitorApp.test.ts

@@ -1,6 +1,5 @@
-import { afterEach, describe, expect, test } from "bun:test";
-import { mkdtemp, rm, writeFile } from "node:fs/promises";
-import os from "node:os";
+import { describe, expect, test } from "bun:test";
+import { writeFile } from "node:fs/promises";
 import path from "node:path";
 import type { AkanMobileTargetConfig } from "./akanConfig";
 import {
@@ -27,14 +26,9 @@ import {
   toIosInfoPlistUsageDescriptions,
   writeRootCapacitorConfig,
 } from "./capacitorApp";
+import { tempDirs } from "./testHelpers";
 
-const tempRoots: string[] = [];
-
-const makeTempRoot = async () => {
-  const root = await mkdtemp(path.join(os.tmpdir(), "akan-capacitor-app-"));
-  tempRoots.push(root);
-  return root;
-};
+const makeTempRoot = tempDirs("akan-capacitor-app-");
 
 const baseTarget: AkanMobileTargetConfig = {
   name: "default",
@@ -50,10 +44,6 @@ const baseTarget: AkanMobileTargetConfig = {
   files: { android: { "app/google-services.json": "private/google-services.json" } },
 };
 
-afterEach(async () => {
-  await Promise.all(tempRoots.splice(0).map((root) => rm(root, { recursive: true, force: true })));
-});
-
 describe("materializeCapacitorConfig", () => {
   test("writes only Capacitor fields for release config", () => {
     const config = materializeCapacitorConfig(