@cosmicdrift/kumiko-framework 0.370.0 → 0.371.0

package/agents/examples/chat-channels.md

@@ -184,7 +184,7 @@ describe("chat-channels recipe", () => {
     });
     expect(rows.map((r) => r["recipientAddress"])).toEqual(["ops"]);
     expect(JSON.stringify(rows)).not.toContain("XXXX");
-  }, 20000);
+  });
 
   test("announce rejects a caller without the Admin role", async () => {
     const attemptsBefore = await selectMany(db(), deliveryAttemptsTable, {

package/agents/examples/env-composition.md

@@ -135,7 +135,10 @@ describe("KUMIKO_DRY_RUN_ENV=boot", () => {
 
   test("exits 1 on an invalid value and never prints it", async () => {
     const endpoint = "http://s3.leak-check.example.com";
-    const { stdout, stderr, exitCode } = await runBootDryRun({ S3_ENDPOINT: endpoint });
+    const { stdout, stderr, exitCode } = await runBootDryRun({
+      NODE_ENV: "production",
+      S3_ENDPOINT: endpoint,
+    });
 
     expect(exitCode).toBe(1);
     expect(stdout).toMatch(/S3_ENDPOINT\s+file-provider-s3-env\s+optional\s+invalid/);
@@ -206,11 +209,18 @@ describe("env composition", () => {
     expect(report).not.toContain("S3_BUCKET");
   });
 
-  test("rejects a remote http S3 endpoint", () => {
+  test("rejects a remote http S3 endpoint in production", () => {
     const composed = composeAppEnv();
-    expect(() =>
-      parseEnv(composed.schema, { ...minimalValidValues, S3_ENDPOINT: "http://s3.example.com" }),
-    ).toThrow(KumikoBootError);
+    const originalNodeEnv = process.env["NODE_ENV"];
+    process.env["NODE_ENV"] = "production";
+    try {
+      expect(() =>
+        parseEnv(composed.schema, { ...minimalValidValues, S3_ENDPOINT: "http://s3.example.com" }),
+      ).toThrow(KumikoBootError);
+    } finally {
+      if (originalNodeEnv === undefined) delete process.env["NODE_ENV"];
+      else process.env["NODE_ENV"] = originalNodeEnv;
+    }
   });
 });
 ```

package/agents/examples/inbound-mail-triage.md

@@ -237,7 +237,7 @@ describe("inbound-mail triage sample", () => {
     } finally {
       await entry.stop();
     }
-  }, 30_000);
+  });
 });
 ```
 

package/dist/env/index.js

@@ -76,6 +76,18 @@ export function classifyField(field) {
     for (let i = 0; i < 8; i++) {
         if (current instanceof z.ZodDefault)
             return "withDefault";
+        // `.required()` on an object wraps optional fields in nonoptional(); the
+        // wrapper makes the field required, whatever the optional beneath it says.
+        if (current instanceof z.ZodNonOptional) {
+            const beneath = zodDef(current)?.innerType;
+            if (beneath instanceof z.ZodOptional) {
+                const unwrapped = zodDef(beneath)?.innerType;
+                if (unwrapped === undefined)
+                    return "required";
+                current = unwrapped;
+                continue;
+            }
+        }
         if (current instanceof z.ZodOptional || current instanceof z.ZodNullable) {
             return "optional";
         }
@@ -324,7 +336,10 @@ export function parseEnv(schema, env, options = {}) {
         const isMissing = issue.code === "invalid_type" && !(name in cleaned);
         const kind = isMissing ? "missing" : "invalid";
         const desc = field ? getFieldDescription(field) : undefined;
-        const message = desc ? `${issue.message} — ${desc}` : issue.message;
+        const issueText = issue.code === "invalid_type" && issue.expected === "nonoptional"
+            ? "Invalid input: required"
+            : issue.message;
+        const message = desc ? `${issueText} — ${desc}` : issueText;
         const suggestion = field ? buildPulumiSuggestion(name, field, options.pulumiPrefix) : undefined;
         const source = options.sources?.[name];
         return {

package/dist/schema-cli.js

@@ -20,12 +20,21 @@ import { join, resolve as resolvePath } from "node:path";
 import { configureBlindIndexKey, configurePiiSubjectKms, resolveKmsWiringAsync, } from "./crypto/index.js";
 import { assertUniqueTableNames, assertValidMigrationName, baselineMigrations, createDbConnection, diffReplayAgainstSnapshot, fetchAppliedMigrations, findCommentedDropTables, generateMigration, isRetiredFrameworkTable, loadMigrationsFromDir, loadSnapshotJson, readMigrationSqlTexts, rebuildTablesFromDiff, replayMigrationsDir, runMigrationsFromDir, tableExists, writeRebuildMarker, writeSnapshotJson, } from "./db/index.js";
 import { validateBoot } from "./engine/boot-validator.js";
+import { FILE_STORAGE_PROVIDER_BOOT_SENTINEL, FILE_STORAGE_PROVIDER_ENV, } from "./engine/extension-names.js";
 import { createRegistry } from "./engine/registry.js";
 import { createEventsTable } from "./event-store/index.js";
 import { queueRebuildsFromMarkers, reportPendingRebuildRun, runPendingRebuilds, } from "./migrations/index.js";
 import { createEventConsumerStateTable, createProjectionStateTable } from "./pipeline/index.js";
 import { ensureTemporalPolyfill } from "./time/index.js";
 const SNAPSHOT_FILENAME = ".snapshot.json";
+// `validate` only sees the composed FEATURES. The file provider normally comes from
+// runProdApp's `fileStorage` option, which defaults FILE_STORAGE_PROVIDER at boot, so
+// an unset variable here is no misconfiguration. The deploy env is checked by the real boot.
+function withFileStoragePresence(env) {
+    return env[FILE_STORAGE_PROVIDER_ENV] === undefined
+        ? { ...env, [FILE_STORAGE_PROVIDER_ENV]: FILE_STORAGE_PROVIDER_BOOT_SENTINEL }
+        : { ...env };
+}
 // Single load-path for kumiko/schema.ts (512/3) — `generate` and `validate`
 // both need ENTITY_METAS (and validate additionally wants FEATURES); this is
 // the one place that enforces "ENTITY_METAS must be an array" so the two
@@ -247,7 +256,9 @@ export async function runSchemaCli(argv, appCwd, out, options = {}) {
             // 2. Boot validity — needs the composed feature set.
             if (Array.isArray(mod.FEATURES)) {
                 try {
-                    validateBoot(mod.FEATURES);
+                    validateBoot(mod.FEATURES, {
+                        env: withFileStoragePresence(process.env),
+                    });
                     out.log("  ✓ boot: feature configuration is valid");
                 }
                 catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.370.0",
+  "version": "0.371.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>",
@@ -199,8 +199,8 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-http": "0.370.0",
-    "@cosmicdrift/kumiko-types": "0.370.0",
+    "@cosmicdrift/kumiko-http": "0.371.0",
+    "@cosmicdrift/kumiko-types": "0.371.0",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.13.1",
@@ -216,8 +216,8 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.370.0",
-    "@cosmicdrift/kumiko-testing": "0.370.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.371.0",
+    "@cosmicdrift/kumiko-testing": "0.371.0",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.371.0",
+    "type": "improvement",
+    "title": "Dry-run env table and boot check respect required overrides and the fileStorage default",
+    "detail": "The env table of KUMIKO_DRY_RUN_ENV=boot shows a variable as required when the app schema overlays required() on an optional field, and the error says required. The schema-cli boot check applies runProdApp's fileStorage default instead of demanding FILE_STORAGE_PROVIDER."
+  },
   {
     "version": "0.370.0",
     "type": "improvement",