@cosmicdrift/kumiko-dev-server 0.112.0 → 0.113.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-dev-server",
-  "version": "0.112.0",
+  "version": "0.113.0",
   "description": "Development server bootstrap for Kumiko apps. Bundles the client, mints dev-JWTs, injects the resolved AppSchema, and seeds an admin. Not for production.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -50,8 +50,8 @@
     "kumiko-schema-check": "./bin/kumiko-schema-check.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.112.0",
-    "@cosmicdrift/kumiko-framework": "0.112.0",
+    "@cosmicdrift/kumiko-bundled-features": "0.113.0",
+    "@cosmicdrift/kumiko-framework": "0.113.0",
     "ts-morph": "^28.0.0"
   },
   "publishConfig": {

package/src/__tests__/compose-features.test.ts

@@ -98,12 +98,11 @@ describe("composeFeatures", () => {
   });
 
   test("app feature duplicating a bundled name is dropped (no createRegistry crash)", () => {
-    // Bug discovered during Phase 3 recording: create-kumiko-app's picker
-    // hands back createAuthEmailPasswordFeature() because the user ticked
-    // it in the recommended set; runDevApp then adds its OWN bundled
-    // copy via includeBundled:true, and createRegistry throws "Duplicate
-    // feature: auth-email-password". The dedupe path keeps the bundled
-    // instance (it carries authOptions wiring) and drops the app stub.
+    // create-kumiko-app's picker hands back createAuthEmailPasswordFeature()
+    // because the user ticked it in the recommended set; runDevApp then adds
+    // its OWN bundled copy via includeBundled:true, and createRegistry throws
+    // "Duplicate feature: auth-email-password". The dedupe path keeps the
+    // bundled instance (it carries authOptions wiring) and drops the app stub.
     const warnSpy = spyOn(console, "warn").mockImplementation(() => {});
     const features = composeFeatures([pickerAuthDupe, noopFeature], {
       includeBundled: true,

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

@@ -6,7 +6,7 @@
 // `relation "kumiko_events" does not exist`. Dieser Test fährt den echten
 // Pfad gegen eine leere DB + den idempotenten Re-Run.
 
-import { afterAll, beforeAll, describe, expect, test } from "bun:test";
+import { afterAll, beforeAll, describe, expect, spyOn, test } from "bun:test";
 import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -76,7 +76,7 @@ describe("runSchemaApply", () => {
     expect(await tableExists(conn.db, "public.read_thing")).toBe(true);
   });
 
-  test("rebuild-Marker für nicht-registrierte Tabelle → kein Crash, 0", async () => {
+  test("rebuild-Marker für nicht-registrierte Tabelle → kein Crash, 0, aber laut warnen (522/3)", async () => {
     writeFileSync(
       join(migDir, "0002_more.sql"),
       `CREATE TABLE "read_more" ("id" text PRIMARY KEY);`,
@@ -86,7 +86,10 @@ describe("runSchemaApply", () => {
       JSON.stringify({ version: 1, tables: ["read_more"] }),
     );
 
+    const warn = spyOn(console, "warn").mockImplementation(() => {});
     expect(await runSchemaApply({ ...APPLY, appCwd })).toBe(0);
     expect(await tableExists(conn.db, "public.read_more")).toBe(true);
+    expect(warn).toHaveBeenCalledWith(expect.stringContaining('Table "read_more"'));
+    warn.mockRestore();
   });
 });

package/src/run-dev-app.ts

@@ -238,8 +238,13 @@ export async function runDevApp(options: RunDevAppOptions): Promise<KumikoServer
   });
 
   // An explicitly wired file provider (options.files) satisfies the
-  // FILE_STORAGE_PROVIDER boot gate — set it before validateBoot runs.
-  if (options.files !== undefined && process.env["FILE_STORAGE_PROVIDER"] === undefined) {
+  // FILE_STORAGE_PROVIDER boot gate — set it before validateBoot runs. Only
+  // if WE set it (532/2): deleting it after use instead of leaving a
+  // permanent process.env mutation, so a second runDevApp call in the same
+  // process (no files this time) doesn't fall through the gate falsely.
+  const setFileStorageProviderEnv =
+    options.files !== undefined && process.env["FILE_STORAGE_PROVIDER"] === undefined;
+  if (setFileStorageProviderEnv) {
     process.env["FILE_STORAGE_PROVIDER"] = "configured";
   }
 
@@ -248,7 +253,11 @@ export async function runDevApp(options: RunDevAppOptions): Promise<KumikoServer
   // die früher nur runProdApp fing und sonst erst den Prod-Pod im
   // CrashLoopBackOff sterben ließ (#359). Wirft synchron, bevor ein
   // Socket oder Watcher (codegen-Write) aufgeht.
-  validateBoot(features);
+  try {
+    validateBoot(features);
+  } finally {
+    if (setFileStorageProviderEnv) delete process.env["FILE_STORAGE_PROVIDER"];
+  }
   warnIfNonUtcServerTimeZone();
   validateAppCustomScreenWriteQns(process.cwd(), collectWriteHandlerQns(features));
 

package/src/schema-apply.ts

@@ -100,7 +100,16 @@ async function rebuildAffectedProjections(
   const projections = new Set<string>();
   for (const table of changedTables) {
     const name = tableToProjection.get(table);
-    if (name) projections.add(name);
+    if (name) {
+      projections.add(name);
+    } else {
+      // 522/3: a table in a .rebuild.json marker that no longer matches any
+      // registered projection would otherwise rebuild nothing and exit 0 —
+      // indistinguishable from "nothing needed a rebuild".
+      console.warn(
+        `  ⚠ Table "${table}" is in a rebuild marker but matches no registered projection — skipped.`,
+      );
+    }
   }
   if (projections.size === 0) return;
 

package/src/welcome-banner.ts

@@ -37,10 +37,7 @@ export function renderWelcomeBanner(input: WelcomeBannerInput): string {
   return [top, ...padded, bottom].join("\n");
 }
 
-// Plain monospace-cell width — counts each codepoint as one cell. Good
-// enough for ASCII + the small set of arrows/checkmarks used above; if a
-// real wide-char ever lands in the banner the row alignment will drift,
-// caught visually by the snapshot test.
+// ponytail: codepoint width; ok for ASCII + arrows, add a width-lib if CJK ever lands here.
 function stringWidth(s: string): number {
   return [...s].length;
 }