@cosmicdrift/kumiko-framework 0.211.0 → 0.213.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.211.0",
+  "version": "0.213.0",
   "description": "Framework core — engine, pipeline, API, DB, and every other bit that makes Kumiko go.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -159,6 +159,10 @@
       "types": "./src/consumer-cli.ts",
       "default": "./src/consumer-cli.ts"
     },
+    "./upgrade-cli": {
+      "types": "./src/upgrade-cli.ts",
+      "default": "./src/upgrade-cli.ts"
+    },
     "./stack": {
       "types": "./src/stack/index.ts",
       "default": "./src/stack/index.ts"
@@ -190,7 +194,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-types": "0.211.0",
+    "@cosmicdrift/kumiko-types": "0.213.0",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.13.1",
@@ -206,7 +210,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.211.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.213.0",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },

package/src/__tests__/schema-cli.integration.test.ts

@@ -69,6 +69,51 @@ describe("runSchemaCli — no-DB paths", () => {
     expect(cap.err.join("\n")).not.toContain("kumiko-schema");
   });
 
+  test("generate --help prints usage, exits 0, writes no migration (framework#2191)", async () => {
+    writeSchemaFile(appCwd, "tbl_a");
+    const cap = captureOut();
+    const code = await runSchemaCli(["generate", "--help"], appCwd, cap.out);
+    expect(code).toBe(0);
+    expect(cap.log.join("\n")).toContain("Usage: schema generate <name>");
+    expect(cap.err).toHaveLength(0);
+    expect(existsSync(join(appCwd, "kumiko/migrations"))).toBe(false);
+  });
+
+  test("generate -h prints usage, exits 0, writes no migration", async () => {
+    writeSchemaFile(appCwd, "tbl_a");
+    const cap = captureOut();
+    const code = await runSchemaCli(["generate", "-h"], appCwd, cap.out);
+    expect(code).toBe(0);
+    expect(cap.log.join("\n")).toContain("Usage: schema generate <name>");
+    expect(existsSync(join(appCwd, "kumiko/migrations"))).toBe(false);
+  });
+
+  test("generate name starting with -- is rejected, exits 1, writes no migration", async () => {
+    writeSchemaFile(appCwd, "tbl_a");
+    const cap = captureOut();
+    const code = await runSchemaCli(["generate", "--evil"], appCwd, cap.out);
+    expect(code).toBe(1);
+    expect(cap.err.join("\n")).toContain('Invalid migration name "--evil"');
+    expect(existsSync(join(appCwd, "kumiko/migrations"))).toBe(false);
+  });
+
+  test("generate name with path separators is rejected (path traversal)", async () => {
+    writeSchemaFile(appCwd, "tbl_a");
+    const cap = captureOut();
+    const code = await runSchemaCli(["generate", "../../evil"], appCwd, cap.out);
+    expect(code).toBe(1);
+    expect(cap.err.join("\n")).toContain("Invalid migration name");
+    expect(existsSync(join(appCwd, "kumiko/migrations"))).toBe(false);
+  });
+
+  test("generate with a hyphenated name still works", async () => {
+    writeSchemaFile(appCwd, "tbl_a");
+    const cap = captureOut();
+    const code = await runSchemaCli(["generate", "add-user-table"], appCwd, cap.out);
+    expect(code).toBe(0);
+    expect(existsSync(join(appCwd, "kumiko/migrations/0001_add-user-table.sql"))).toBe(true);
+  });
+
   test("generate with missing schema.ts exits 1", async () => {
     const cap = captureOut();
     const code = await runSchemaCli(["generate", "init"], appCwd, cap.out);

package/src/__tests__/upgrade-cli.test.ts

@@ -0,0 +1,370 @@
+import { afterEach, describe, expect, test } from "bun:test";
+import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { resolveCodemodScript, runUpgradeCli, type UpgradeCliOut } from "../upgrade-cli";
+
+function makeSpyOutput(): {
+  readonly out: UpgradeCliOut;
+  readonly logs: string[];
+  readonly errs: string[];
+} {
+  const logs: string[] = [];
+  const errs: string[] = [];
+  return {
+    logs,
+    errs,
+    out: {
+      log: (m: string) => logs.push(m),
+      err: (m: string) => errs.push(m),
+    },
+  };
+}
+
+function makeTempCwd(files?: Record<string, string>): {
+  readonly cwd: string;
+  readonly cleanup: () => void;
+} {
+  const cwd = mkdtempSync(join(tmpdir(), "kumiko-upgrade-cli-"));
+  if (files) {
+    for (const [relPath, content] of Object.entries(files)) {
+      const full = join(cwd, relPath);
+      mkdirSync(join(full, ".."), { recursive: true });
+      writeFileSync(full, content, "utf-8");
+    }
+  }
+  return {
+    cwd,
+    cleanup: () => {
+      try {
+        rmSync(cwd, { recursive: true, force: true });
+      } catch {
+        // ignore — best-effort
+      }
+    },
+  };
+}
+
+const cleanups: Array<() => void> = [];
+afterEach(() => {
+  for (const c of cleanups) c();
+  cleanups.length = 0;
+});
+
+const CORE_ENTRY = JSON.stringify([
+  {
+    version: "0.167.0",
+    type: "breaking",
+    title: "core helper moved",
+    migration: "import from /testing",
+  },
+]);
+
+const FEATURE_ENTRY = JSON.stringify([{ version: "0.166.0", type: "fix", title: "feature fix" }]);
+
+function tmp(files: Record<string, string>): string {
+  const t = makeTempCwd(files);
+  cleanups.push(t.cleanup);
+  return t.cwd;
+}
+
+async function runJson(cwd: string, from: string): Promise<{ pending: Array<{ title: string }> }> {
+  const spy = makeSpyOutput();
+  const exit = await runUpgradeCli(["--from", from, "--json"], cwd, spy.out);
+  expect(exit).toBe(0);
+  return JSON.parse(spy.logs.join("\n"));
+}
+
+describe("upgrade command — framework core changelog", () => {
+  test("collects core changes.json from the framework repo layout", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": CORE_ENTRY,
+      "packages/bundled-features/src/user/changes.json": FEATURE_ENTRY,
+    });
+
+    const result = await runJson(cwd, "0.165.0");
+
+    expect(result.pending.map((e) => e.title)).toEqual(["core helper moved", "feature fix"]);
+  });
+
+  test("finds core changes.json in hoisted node_modules from an app subdir", async () => {
+    const cwd = tmp({
+      "node_modules/@cosmicdrift/kumiko-framework/src/changes.json": CORE_ENTRY,
+      "apps/web/package.json": "{}",
+    });
+
+    const result = await runJson(`${cwd}/apps/web`, "0.165.0");
+
+    expect(result.pending.map((e) => e.title)).toEqual(["core helper moved"]);
+  });
+
+  test("repo file wins over node_modules — no duplicate entries", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": CORE_ENTRY,
+      "node_modules/@cosmicdrift/kumiko-framework/src/changes.json": CORE_ENTRY,
+    });
+
+    const result = await runJson(cwd, "0.165.0");
+
+    expect(result.pending).toHaveLength(1);
+  });
+
+  test("feature entries are not duplicated by the workspace symlink", async () => {
+    const cwd = tmp({
+      "packages/bundled-features/src/user/changes.json": FEATURE_ENTRY,
+      "node_modules/@cosmicdrift/kumiko-bundled-features/src/user/changes.json": FEATURE_ENTRY,
+    });
+
+    const result = await runJson(cwd, "0.165.0");
+
+    expect(result.pending).toHaveLength(1);
+  });
+
+  test("core entries older than the current version are filtered out", async () => {
+    const cwd = tmp({ "packages/framework/src/changes.json": CORE_ENTRY });
+
+    const result = await runJson(cwd, "0.167.0");
+
+    expect(result.pending).toEqual([]);
+  });
+
+  test("--from is rejected when it isn't a valid semver — no silent 'nothing new'", async () => {
+    const cwd = tmp({ "packages/framework/src/changes.json": CORE_ENTRY });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "latest", "--json"], cwd, spy.out);
+
+    expect(exit).toBe(1);
+    expect(spy.errs.join("\n")).toContain("Invalid version format");
+    expect(spy.logs).toEqual([]);
+  });
+});
+
+describe("upgrade command — enterprise package layout", () => {
+  // Layout is detected by presence of changes.json, not an "ai-" name
+  // prefix — the old heuristic silently dropped every enterprise package
+  // whose name didn't start with "ai-" (fw#1605).
+  test("collects changes.json from a package without an 'ai-' prefix", async () => {
+    const cwd = tmp({
+      "packages/billing-designer/src/changes.json": FEATURE_ENTRY,
+    });
+
+    const result = await runJson(cwd, "0.165.0");
+
+    expect(result.pending.map((e) => e.title)).toEqual(["feature fix"]);
+  });
+
+  test("flat layout (no src/ subdir) is also collected", async () => {
+    const cwd = tmp({
+      "packages/billing-designer/changes.json": FEATURE_ENTRY,
+    });
+
+    const result = await runJson(cwd, "0.165.0");
+
+    expect(result.pending.map((e) => e.title)).toEqual(["feature fix"]);
+  });
+});
+
+// The repo actually checked out on disk — scripts/codemod/ isn't published,
+// so --apply only ever works against a real local framework checkout.
+const REAL_REPO_ROOT = join(import.meta.dir, "../../../..");
+const REAL_CODEMOD = "scripts/codemod/crypto-shredding-testing-move.ts";
+
+function breakingEntryWithCodemod(codemod: string | undefined): string {
+  const entry: Record<string, unknown> = {
+    version: "0.167.0",
+    type: "breaking",
+    title: "helper moved",
+    migration: "import from /testing",
+  };
+  if (codemod !== undefined) entry["codemod"] = codemod;
+  return JSON.stringify([entry]);
+}
+
+const LEGACY_IMPORT_FIXTURE = [
+  'import { resetPiiSubjectKmsForTests } from "@cosmicdrift/kumiko-framework/crypto";',
+  "",
+  "resetPiiSubjectKmsForTests();",
+  "",
+].join("\n");
+
+describe("resolveCodemodScript", () => {
+  test("resolves a real script under scripts/codemod/", () => {
+    const resolved = resolveCodemodScript(REAL_REPO_ROOT, REAL_CODEMOD);
+    expect(resolved).toBe(join(REAL_REPO_ROOT, REAL_CODEMOD));
+  });
+
+  test("rejects an absolute path", () => {
+    expect(resolveCodemodScript(REAL_REPO_ROOT, "/etc/passwd.ts")).toBeNull();
+  });
+
+  test("rejects path traversal that escapes scripts/codemod/", () => {
+    expect(
+      resolveCodemodScript(REAL_REPO_ROOT, "scripts/codemod/../../package.json.ts"),
+    ).toBeNull();
+    expect(resolveCodemodScript(REAL_REPO_ROOT, "../outside/x.ts")).toBeNull();
+  });
+
+  test("rejects a non-.ts file", () => {
+    expect(resolveCodemodScript(REAL_REPO_ROOT, "scripts/codemod/README.md")).toBeNull();
+  });
+
+  test("rejects a script that doesn't exist", () => {
+    expect(resolveCodemodScript(REAL_REPO_ROOT, "scripts/codemod/does-not-exist.ts")).toBeNull();
+  });
+
+  test("rejects an undefined codemod field", () => {
+    expect(resolveCodemodScript(REAL_REPO_ROOT, undefined)).toBeNull();
+  });
+});
+
+describe("upgrade command — --apply", () => {
+  test("runs the real codemod against a fixture file and writes the marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(REAL_CODEMOD),
+      "legacy-test-helper.ts": LEGACY_IMPORT_FIXTURE,
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.165.0", "--apply"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(0);
+
+    const rewritten = readFileSync(join(cwd, "legacy-test-helper.ts"), "utf-8");
+    expect(rewritten).toContain('from "@cosmicdrift/kumiko-framework/testing"');
+    expect(rewritten).not.toContain('from "@cosmicdrift/kumiko-framework/crypto"');
+
+    const markerPath = join(cwd, ".kumiko/upgrade-state.json");
+    expect(existsSync(markerPath)).toBe(true);
+    const marker = JSON.parse(readFileSync(markerPath, "utf-8"));
+    expect(marker.version).toBe("0.167.0");
+    expect(typeof marker.appliedAt).toBe("string");
+    expect(marker.codemods).toEqual([
+      { version: "0.167.0", codemod: REAL_CODEMOD, title: "helper moved" },
+    ]);
+  });
+
+  test("--dry-run runs the codemod but changes nothing and writes no marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(REAL_CODEMOD),
+      "legacy-test-helper.ts": LEGACY_IMPORT_FIXTURE,
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.165.0", "--apply", "--dry-run"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(0);
+    expect(readFileSync(join(cwd, "legacy-test-helper.ts"), "utf-8")).toBe(LEGACY_IMPORT_FIXTURE);
+    expect(existsSync(join(cwd, ".kumiko/upgrade-state.json"))).toBe(false);
+    expect(spy.logs.join("\n")).toContain("Touched 1 files, moved 1 import(s)");
+  });
+
+  test("rejects a path-traversal codemod field and writes no marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod("../../../etc/passwd.ts"),
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.165.0", "--apply"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(1);
+    expect(spy.errs.join("\n")).toContain("invalid codemod path");
+    expect(existsSync(join(cwd, ".kumiko/upgrade-state.json"))).toBe(false);
+  });
+
+  test("fails when the codemod script doesn't exist, writes no marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(
+        "scripts/codemod/does-not-exist.ts",
+      ),
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.165.0", "--apply"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(1);
+    expect(spy.errs.join("\n")).toContain("invalid codemod path");
+    expect(existsSync(join(cwd, ".kumiko/upgrade-state.json"))).toBe(false);
+  });
+
+  test("breaking changes without a codemod field are reported as manual, no marker written", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(undefined),
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.165.0", "--apply"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(0);
+    expect(spy.logs.join("\n")).toContain("no codemod, manual migration required");
+    expect(existsSync(join(cwd, ".kumiko/upgrade-state.json"))).toBe(false);
+  });
+
+  test("nothing pending: reports up to date, still bootstraps the marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(REAL_CODEMOD),
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.170.0", "--apply"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(0);
+    expect(spy.logs.join("\n")).toContain("Nothing new since your version");
+
+    const markerPath = join(cwd, ".kumiko/upgrade-state.json");
+    expect(existsSync(markerPath)).toBe(true);
+    const marker = JSON.parse(readFileSync(markerPath, "utf-8"));
+    expect(marker.version).toBe("0.170.0");
+    expect(marker.codemods).toEqual([]);
+    expect(typeof marker.appliedAt).toBe("string");
+  });
+
+  test("nothing pending + --dry-run: reports up to date, writes no marker", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(REAL_CODEMOD),
+    });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(["--from", "0.170.0", "--apply", "--dry-run"], cwd, spy.out, {
+      repoRoot: REAL_REPO_ROOT,
+    });
+
+    expect(exit).toBe(0);
+    expect(spy.logs.join("\n")).toContain("Nothing new since your version");
+    expect(existsSync(join(cwd, ".kumiko/upgrade-state.json"))).toBe(false);
+  });
+
+  test("--dir targets a different directory than cwd", async () => {
+    const cwd = tmp({
+      "packages/framework/src/changes.json": breakingEntryWithCodemod(REAL_CODEMOD),
+    });
+    const target = tmp({ "legacy-test-helper.ts": LEGACY_IMPORT_FIXTURE });
+    const spy = makeSpyOutput();
+
+    const exit = await runUpgradeCli(
+      ["--from", "0.165.0", "--apply", "--dir", target],
+      cwd,
+      spy.out,
+      {
+        repoRoot: REAL_REPO_ROOT,
+      },
+    );
+
+    expect(exit).toBe(0);
+    const rewritten = readFileSync(join(target, "legacy-test-helper.ts"), "utf-8");
+    expect(rewritten).toContain('from "@cosmicdrift/kumiko-framework/testing"');
+    expect(existsSync(join(target, ".kumiko/upgrade-state.json"))).toBe(true);
+  });
+});

package/src/api/__tests__/request-locale.integration.test.ts

@@ -0,0 +1,96 @@
+// Regression guard for the request-locale wiring: X-Locale header (or, when
+// absent, Accept-Language) must reach ctx.locale on real HTTP calls — the
+// AsyncLocalStorage plumbing in request-id-middleware.ts / dispatch-shared.ts
+// can't be exercised via createTestDispatcher, which skips the HTTP layer
+// entirely.
+import { afterAll, beforeAll, describe, expect, test } from "bun:test";
+import { defineFeature } from "@cosmicdrift/kumiko-framework/engine";
+import {
+  createTestUser,
+  setupTestStack,
+  type TestStack,
+} from "@cosmicdrift/kumiko-framework/stack";
+import { z } from "zod";
+import { LOCALE_HEADER_NAME } from "../api-constants";
+
+const localeProbe = defineFeature("locale-probe", (r) => {
+  r.writeHandler(
+    "read-locale",
+    z.object({}),
+    async (_event, ctx) => ({ isSuccess: true, data: { locale: ctx.locale } }),
+    { access: { openToAll: true } },
+  );
+});
+
+async function readLocale(stack: TestStack, headers: Record<string, string>): Promise<string> {
+  const res = await stack.http.writeWithHeaders(
+    "locale-probe:write:read-locale",
+    {},
+    createTestUser({ id: 1 }),
+    headers,
+  );
+  const body = (await res.json()) as { data: { locale: string } };
+  return body.data.locale;
+}
+
+describe("ctx.locale resolution over real HTTP", () => {
+  let stack: TestStack;
+
+  beforeAll(async () => {
+    stack = await setupTestStack({ features: [localeProbe] });
+  });
+
+  afterAll(async () => {
+    await stack.cleanup();
+  });
+
+  test("X-Locale header wins", async () => {
+    const locale = await readLocale(stack, { [LOCALE_HEADER_NAME]: "de-AT" });
+    expect(locale).toBe("de-AT");
+  });
+
+  test("falls back to Accept-Language when X-Locale is absent", async () => {
+    const locale = await readLocale(stack, { "accept-language": "fr-FR,fr;q=0.9,en;q=0.8" });
+    expect(locale).toBe("fr-FR");
+  });
+
+  test("Accept-Language with only invalid tags falls back to the boot default", async () => {
+    const locale = await readLocale(stack, { "accept-language": "*, ;q=0.1" });
+    expect(locale).toBe("en");
+  });
+
+  test("a malformed/manipulated X-Locale header is rejected, not passed through", async () => {
+    const locale = await readLocale(stack, {
+      [LOCALE_HEADER_NAME]: "<script>alert(1)</script>",
+    });
+    expect(locale).toBe("en");
+  });
+
+  test("an invalid X-Locale header falls through to a valid Accept-Language instead of the default", async () => {
+    const locale = await readLocale(stack, {
+      [LOCALE_HEADER_NAME]: "not-a-real-locale-tag-way-too-long-to-be-valid",
+      "accept-language": "es-ES",
+    });
+    expect(locale).toBe("es-ES");
+  });
+});
+
+describe("ctx.locale falls back to the app's boot-configured defaultLocale", () => {
+  let stack: TestStack;
+
+  beforeAll(async () => {
+    stack = await setupTestStack({
+      features: [localeProbe],
+      extraContext: { defaultLocale: "ja" },
+    });
+  });
+
+  afterAll(async () => {
+    await stack.cleanup();
+  });
+
+  test("no header signal at all uses the app's defaultLocale, not the hardcoded default", async () => {
+    const locale = await readLocale(stack, {});
+    expect(locale).toBe("ja");
+  });
+});

package/src/api/api-constants.ts

@@ -118,4 +118,11 @@ export const STATE_CHANGING_METHODS: ReadonlySet<string> = new Set([
 export const TENANT_HEADER_NAME = "X-Tenant";
 export const TENANT_COOKIE_NAME = "kumiko_tenant";
 
+// Client-declared active UI locale (BCP-47). Read once per request in
+// request-id-middleware.ts, before auth, so it reaches public routes
+// (e.g. signup-request) too. Falls back to Accept-Language, then the app's
+// boot-configured defaultLocale, when absent or malformed — see
+// request-locale.ts and dispatch-shared.ts's ctx.locale resolution.
+export const LOCALE_HEADER_NAME = "X-Locale";
+
 export type Route = (typeof Routes)[keyof typeof Routes];

package/src/api/request-context.ts

@@ -33,6 +33,11 @@ export type RequestContextData = {
   // Raw User-Agent header — audit trails (download tokens, GDPR export
   // access) want it alongside `ip`. Undefined for non-HTTP entry points.
   readonly userAgent?: string;
+  // Client-declared active UI locale (BCP-47), resolved once from
+  // X-Locale / Accept-Language by requestIdMiddleware — see
+  // request-locale.ts. Undefined when neither header carried a valid tag;
+  // callers fall back further (dispatch-shared.ts's ctx.locale chain).
+  readonly locale?: string;
 };
 
 const storage = new AsyncLocalStorage<RequestContextData>();

package/src/api/request-id-middleware.ts

@@ -1,4 +1,6 @@
 import type { Context, Next } from "hono";
+import { resolveHeaderLocale } from "../i18n/request-locale";
+import { LOCALE_HEADER_NAME } from "./api-constants";
 import { type RequestContextData, requestContext } from "./request-context";
 
 const REQUEST_ID_HEADER = "X-Request-ID";
@@ -45,6 +47,13 @@ export function buildRequestContextData(c: Context): RequestContextData {
   const xff = c.req.header("x-forwarded-for");
   const ip = xff?.split(",")[0]?.trim();
   const userAgent = c.req.header("user-agent");
+  // Runs before auth-middleware, so this reaches public routes too (e.g.
+  // signup-request) — that's the whole point: the active UI locale must
+  // survive to anonymous callers, not just authenticated ones.
+  const locale = resolveHeaderLocale({
+    headerLocale: c.req.header(LOCALE_HEADER_NAME),
+    acceptLanguage: c.req.header("accept-language"),
+  });
 
   return {
     requestId,
@@ -52,6 +61,7 @@ export function buildRequestContextData(c: Context): RequestContextData {
     ...(signal ? { signal } : {}),
     ...(ip && ip.length > 0 ? { ip } : {}),
     ...(userAgent !== undefined ? { userAgent } : {}),
+    ...(locale !== undefined ? { locale } : {}),
   };
 }
 

package/src/arg-parser.ts

@@ -0,0 +1,66 @@
+// Small arg parser for CLI commands. Not as feature-rich as commander or
+// yargs — just:
+//   - Positional args:  cmd "value1" "value2"
+//   - Flags:            --flag (boolean) / --key value
+//   - Negatable:        --no-flag
+//
+// Copy of bin/commands/arg-parser.ts — registry-free, but upgrade-cli.ts
+// lives in the published @cosmicdrift/kumiko-framework package and can't
+// import from bin/commands/ (outside the package's export surface).
+
+export type ParsedArgs = {
+  readonly positional: ReadonlyArray<string>;
+  readonly flags: ReadonlyMap<string, string | boolean>;
+};
+
+export function parseArgs(argv: ReadonlyArray<string>): ParsedArgs {
+  const positional: string[] = [];
+  const flags = new Map<string, string | boolean>();
+
+  for (let i = 0; i < argv.length; i++) {
+    const a = argv[i];
+    if (!a) continue;
+    if (a.startsWith("--")) {
+      const key = a.slice(2);
+      // `--no-foo` form
+      if (key.startsWith("no-")) {
+        flags.set(key.slice(3), false);
+        continue;
+      }
+      // `--key=value` inline form
+      const eq = key.indexOf("=");
+      if (eq !== -1) {
+        flags.set(key.slice(0, eq), key.slice(eq + 1));
+        continue;
+      }
+      // `--key value` if next isn't a flag, else boolean
+      const next = argv[i + 1];
+      if (next !== undefined && !next.startsWith("--")) {
+        flags.set(key, next);
+        i++;
+      } else {
+        flags.set(key, true);
+      }
+    } else {
+      positional.push(a);
+    }
+  }
+
+  return { positional, flags };
+}
+
+export function getFlag(args: ParsedArgs, name: string): boolean {
+  return args.flags.get(name) === true;
+}
+
+export function getStringFlag(args: ParsedArgs, name: string): string | undefined {
+  const v = args.flags.get(name);
+  return typeof v === "string" ? v : undefined;
+}
+
+export function getNumberFlag(args: ParsedArgs, name: string): number | undefined {
+  const v = args.flags.get(name);
+  if (typeof v !== "string") return undefined;
+  const n = Number.parseInt(v, 10);
+  return Number.isNaN(n) ? undefined : n;
+}

package/src/bun-db/index.ts

@@ -28,6 +28,7 @@ export {
   selectMany,
   transaction,
   type UpsertOnConflictOptions,
+  unsafeReadRetrying,
   updateMany,
   upsertByPk,
   upsertOnConflict,