@cosmicdrift/kumiko-cli 0.334.0 → 0.335.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-cli",
-  "version": "0.334.0",
+  "version": "0.335.0",
   "description": "CLI for Kumiko apps. `bunx @cosmicdrift/kumiko-cli new app <name>` scaffolds a runnable workspace; `add feature <name>` mounts a feature; `check` runs boot validation plus the guard suites with the step list derived from kumiko.json; `agent lint`, `project` and `consumer` operate on an existing app.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -35,12 +35,12 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.334.0",
-    "@cosmicdrift/kumiko-dev-server": "0.334.0",
-    "@cosmicdrift/kumiko-framework": "0.334.0",
-    "@cosmicdrift/kumiko-guards": "0.334.0",
-    "@cosmicdrift/kumiko-repo-manifest": "0.334.0",
-    "@cosmicdrift/kumiko-testing": "0.334.0",
+    "@cosmicdrift/kumiko-bundled-features": "0.335.0",
+    "@cosmicdrift/kumiko-dev-server": "0.335.0",
+    "@cosmicdrift/kumiko-framework": "0.335.0",
+    "@cosmicdrift/kumiko-guards": "0.335.0",
+    "@cosmicdrift/kumiko-repo-manifest": "0.335.0",
+    "@cosmicdrift/kumiko-testing": "0.335.0",
     "zod": "^4.4.3"
   },
   "publishConfig": {

package/src/changes.json

@@ -1,4 +1,16 @@
 [
+  {
+    "version": "0.335.0",
+    "type": "fix",
+    "title": "kumiko check runs boot validation against the repo root when started from a subdirectory",
+    "detail": "`kumiko check` now runs the boot validation against the repo root instead of the current directory, so running it from a subdirectory no longer reports a missing `kumiko/schema.ts`."
+  },
+  {
+    "version": "0.335.0",
+    "type": "fix",
+    "title": "kumiko check survives throwing steps, config loading validates its shape",
+    "detail": "`kumiko check` counts a step that throws as a failure and still runs the remaining steps, and `--explain` now returns the guards' exit code. The `agent`, `consumer` and `project` commands report a readable message when `kumiko.config.ts` has no default export with a `features` array instead of crashing. Invalid-manifest errors name the root as `<root>` instead of an empty path."
+  },
   {
     "version": "0.292.0",
     "type": "improvement",

package/src/commands/agent.ts

@@ -1,4 +1,5 @@
 import { join } from "node:path";
+import { loadAppConfig } from "./load-app-config";
 import type { CliCommand } from "./types";
 
 const LINT_SUBCOMMAND = "lint";
@@ -29,9 +30,8 @@ export const agentCommand: CliCommand = {
       return 1;
     }
 
-    const config = (await import(configPath)).default as {
-      features: readonly import("@cosmicdrift/kumiko-framework/engine").FeatureDefinition[];
-    };
+    const config = await loadAppConfig(ctx, configPath);
+    if (config === null) return 1;
     const { findAgentDocGaps, formatAgentDocGap } = await import(
       "@cosmicdrift/kumiko-bundled-features/agent-tools"
     );

package/src/commands/check.ts

@@ -144,28 +144,44 @@ export async function runCheck(
 
   if (ctx.argv.includes("--explain")) {
     // The guard suite is the only one with its own --explain; pass it through.
-    if (steps.some((step) => step.id === "guards")) guards.runGuardsCli(["--explain"]);
-    return 0;
+    return steps.some((step) => step.id === "guards") ? guards.runGuardsCli(["--explain"]) : 0;
   }
 
   let failed = 0;
   for (const step of steps) {
     ctx.out.log(`  ▸ ${step.label}`);
-    failed += await runStep(step.id, ctx, guards, deps);
+    failed += await runStepReportingThrow(step, ctx, repo.absPath, guards, deps);
   }
   return failed > 0 ? 1 : 0;
 }
 
+async function runStepReportingThrow(
+  step: { readonly id: CheckStepId; readonly label: string },
+  ctx: CliCommandContext,
+  repoRoot: string,
+  guards: GuardsCli,
+  deps: CheckDeps,
+): Promise<number> {
+  try {
+    return await runStep(step.id, ctx, repoRoot, guards, deps);
+  } catch (e) {
+    ctx.out.err(`  ✗ ${step.label}: ${e instanceof Error ? e.message : String(e)}`);
+    return 1;
+  }
+}
+
 async function runStep(
   id: CheckStepId,
   ctx: CliCommandContext,
+  repoRoot: string,
   guards: GuardsCli,
   deps: CheckDeps,
 ): Promise<number> {
   switch (id) {
     case "boot": {
       const { runSchemaCli } = await deps.loadSchemaCli();
-      return await runSchemaCli(["validate"], ctx.cwd, ctx.out);
+      // findLocalRepo walks upwards, so ctx.cwd may be a subdirectory without kumiko/schema.ts.
+      return await runSchemaCli(["validate"], repoRoot, ctx.out);
     }
     case "guards":
       return guards.runGuardsCli([]);

package/src/commands/consumer.ts

@@ -1,4 +1,5 @@
 import { join } from "node:path";
+import { loadAppConfig } from "./load-app-config";
 import type { CliCommand, CliCommandContext } from "./types";
 
 export const consumerCommand: CliCommand = {
@@ -25,9 +26,8 @@ export const consumerCommand: CliCommand = {
       return 1;
     }
 
-    const config = (await import(configPath)).default as {
-      features: readonly import("@cosmicdrift/kumiko-framework/engine").FeatureDefinition[];
-    };
+    const config = await loadAppConfig(ctx, configPath);
+    if (config === null) return 1;
     const { createRegistry } = await import("@cosmicdrift/kumiko-framework/engine");
     const { createDbConnection } = await import("@cosmicdrift/kumiko-framework/db");
     const {

package/src/commands/load-app-config.ts

@@ -0,0 +1,32 @@
+import type { FeatureDefinition } from "@cosmicdrift/kumiko-framework/engine";
+import type { CliCommandContext } from "./types";
+
+export type AppConfig = { readonly features: readonly FeatureDefinition[] };
+
+function isAppConfig(value: unknown): value is AppConfig {
+  return (
+    typeof value === "object" &&
+    value !== null &&
+    "features" in value &&
+    Array.isArray(value.features)
+  );
+}
+
+// kumiko.config.ts is user-authored: a missing default export or `features`
+// array must produce a readable message, not a TypeError further down.
+export async function loadAppConfig(
+  ctx: CliCommandContext,
+  configPath: string,
+): Promise<AppConfig | null> {
+  const mod: unknown = await import(configPath);
+  const config =
+    typeof mod === "object" && mod !== null && "default" in mod ? mod.default : undefined;
+  if (!isAppConfig(config)) {
+    ctx.out.err("");
+    ctx.out.err(`  ${configPath} must default-export an object with a "features" array:`);
+    ctx.out.err("    export default { features: [myFeature] };");
+    ctx.out.err("");
+    return null;
+  }
+  return config;
+}

package/src/commands/project.ts

@@ -1,4 +1,5 @@
 import { join } from "node:path";
+import { loadAppConfig } from "./load-app-config";
 import type { CliCommand, CliCommandContext } from "./types";
 
 export const projectCommand: CliCommand = {
@@ -30,9 +31,8 @@ export const projectCommand: CliCommand = {
       return 1;
     }
 
-    const config = (await import(configPath)).default as {
-      features: readonly import("@cosmicdrift/kumiko-framework/engine").FeatureDefinition[];
-    };
+    const config = await loadAppConfig(ctx, configPath);
+    if (config === null) return 1;
     const { createRegistry } = await import("@cosmicdrift/kumiko-framework/engine");
     const { createDbConnection } = await import("@cosmicdrift/kumiko-framework/db");
     const {