@cosmicdrift/kumiko-framework 0.387.0 → 0.388.0

package/dist/search/meilisearch-adapter.js

@@ -34,7 +34,8 @@ function isSubsetOf(stored, wanted) {
 }
 // A stored list that lies inside the current default and misses attributes is an
 // older default that gained fields since; anything with a foreign attribute is
-// an explicit configure() and stays untouched.
+// an explicit configure() and stays untouched. Known limit: an explicit configure() that
+// narrowed to a strict subset of the default is indistinguishable and gets widened again.
 function isOutdatedDefault(stored, wanted) {
     const pairs = [
         [stored.searchable, wanted.searchable],

package/dist/upgrade-cli.js

@@ -659,27 +659,27 @@ function matchManualRefs(open, refs) {
     }
     return { resolvedIds, errors };
 }
+function failUpgrade(out, ...lines) {
+    out.err("");
+    for (const line of lines)
+        out.err(`  ${line}`);
+    out.err("");
+    return 1;
+}
 function resolveManualMigrations(out, args, targetDir) {
-    const fail = (...lines) => {
-        out.err("");
-        for (const line of lines)
-            out.err(`  ${line}`);
-        out.err("");
-        return 1;
-    };
     const input = parseResolveInput(args);
     if ("error" in input)
-        return fail(input.error);
+        return failUpgrade(out, input.error);
     const read = readMarker(targetDir);
     if (read.kind === "missing") {
-        return fail(`No .kumiko/upgrade-state.json under ${targetDir}, nothing to resolve.`);
+        return failUpgrade(out, `No .kumiko/upgrade-state.json under ${targetDir}, nothing to resolve.`);
     }
     if (read.kind === "invalid")
-        return fail("invalid .kumiko/upgrade-state.json, fix it first.");
+        return failUpgrade(out, "invalid .kumiko/upgrade-state.json, fix it first.");
     const open = read.marker.pendingManual;
     const { resolvedIds, errors } = matchManualRefs(open, input.refs);
     if (errors.length > 0)
-        return fail(...errors, "Nothing was written.");
+        return failUpgrade(out, ...errors, "Nothing was written.");
     const { reason } = input;
     const resolution = getFlag(args, "not-applicable")
         ? "not-applicable"
@@ -721,32 +721,7 @@ kumiko-upgrade --resolve <id|version>[,<id|version>...] --reason "<text>" [--not
   --not-applicable   with --resolve, record the entries as not applicable instead of migrated
   --dry-run          with --apply, report what would run without writing anything
   --help, -h         print this usage and exit`;
-// kumiko-lint-ignore complexity-budget CLI orchestration moved from bin/commands/upgrade.ts — same branching surface, shared by kumiko upgrade + published kumiko-upgrade bin
-export async function runUpgradeCli(argv, cwd, out, options) {
-    if (argv.includes("--help") || argv.includes("-h")) {
-        out.log(UPGRADE_USAGE);
-        return 0;
-    }
-    // Standalone CLI entry, not booted via runProdApp/runDevApp — Temporal needs an explicit polyfill here.
-    await ensureTemporalPolyfill();
-    const repoRoot = options?.repoRoot ?? cwd;
-    const args = parseArgs(argv);
-    const jsonMode = getFlag(args, "json");
-    const verbose = getFlag(args, "verbose");
-    const fromFlag = getStringFlag(args, "from");
-    // Resolved once here (not just inside --apply) so both the filter
-    // baseline below and the --apply codemod run target the same directory.
-    const dirFlag = getStringFlag(args, "dir");
-    const targetDir = dirFlag ? resolve(dirFlag) : cwd;
-    if (!existsSync(targetDir) || !statSync(targetDir).isDirectory()) {
-        out.err("");
-        out.err(`  --dir path is not a directory: ${targetDir}`);
-        out.err("");
-        return 1;
-    }
-    if (args.flags.has("resolve")) {
-        return resolveManualMigrations(out, args, targetDir);
-    }
+function resolveUpgradeBaseline(cwd, targetDir, fromFlag) {
     // Always resolve the actually installed version, even when --from is set,
     // so --json can report both the filter baseline and what's really there.
     const installedVersion = readCurrentVersion(cwd);
@@ -761,33 +736,40 @@ export async function runUpgradeCli(argv, cwd, out, options) {
     const markerRead = readMarker(targetDir);
     const marker = fromFlag === undefined ? markerRead : { kind: "missing" };
     if (marker.kind === "invalid") {
-        out.err("");
-        out.err("  invalid .kumiko/upgrade-state.json, fix it or pass --from <version>");
-        out.err("");
-        return 1;
+        return {
+            kind: "error",
+            lines: ["invalid .kumiko/upgrade-state.json, fix it or pass --from <version>"],
+        };
     }
     const currentVersion = fromFlag ?? (marker.kind === "ok" ? marker.version : undefined) ?? installedVersion;
     if (!currentVersion) {
-        out.err("");
-        out.err("  Could not detect Kumiko version.");
-        out.err("  Run from an app directory with node_modules, or use --from <version>.");
-        out.err("");
-        return 1;
+        return {
+            kind: "error",
+            lines: [
+                "Could not detect Kumiko version.",
+                "Run from an app directory with node_modules, or use --from <version>.",
+            ],
+        };
     }
     if (!SEMVER_RE.test(currentVersion)) {
-        out.err("");
-        out.err(`  Invalid version format: "${currentVersion}" — expected x.y.z`);
-        out.err("");
-        return 1;
+        return {
+            kind: "error",
+            lines: [`Invalid version format: "${currentVersion}" — expected x.y.z`],
+        };
     }
+    return { kind: "ok", value: { currentVersion, installedVersion, markerRead } };
+}
+function collectAllChangelogEntries(cwd) {
     const featuresDirs = findFeaturesDirs(cwd);
     const packageChangelogFiles = findPackageChangelogFiles(cwd);
     if (featuresDirs.length === 0 && packageChangelogFiles.length === 0) {
-        out.err("");
-        out.err("  Could not find bundled-features directory.");
-        out.err("  Run from framework/enterprise repo or an app with node_modules.");
-        out.err("");
-        return 1;
+        return {
+            kind: "error",
+            lines: [
+                "Could not find bundled-features directory.",
+                "Run from framework/enterprise repo or an app with node_modules.",
+            ],
+        };
     }
     const allEntries = [];
     for (const dir of featuresDirs) {
@@ -796,6 +778,18 @@ export async function runUpgradeCli(argv, cwd, out, options) {
     for (const changelogFile of packageChangelogFiles) {
         allEntries.push(...readChangelogFile(changelogFile));
     }
+    return { kind: "ok", value: allEntries };
+}
+function planUpgrade(args, cwd, targetDir) {
+    const dirFlag = getStringFlag(args, "dir");
+    const baseline = resolveUpgradeBaseline(cwd, targetDir, getStringFlag(args, "from"));
+    if (baseline.kind === "error")
+        return baseline;
+    const entries = collectAllChangelogEntries(cwd);
+    if (entries.kind === "error")
+        return entries;
+    const { currentVersion, installedVersion } = baseline.value;
+    const allEntries = entries.value;
     // Installed version of the target the marker belongs to (cwd unless --dir).
     const targetInstalledVersion = dirFlag ? readCurrentVersion(targetDir) : installedVersion;
     // Changelogs can be found in a parent workspace's newer install. A change
@@ -804,87 +798,121 @@ export async function runUpgradeCli(argv, cwd, out, options) {
         ? allEntries
         : allEntries.filter((entry) => compareVersions(entry.version, targetInstalledVersion) <= 0);
     const pending = sortEntries(filterEntriesAfter(installedEntries, currentVersion));
-    if (getFlag(args, "apply")) {
-        // Marker must reflect what is actually installed under the target (or
-        // cwd), not the filter baseline above — otherwise CI stays green forever.
-        const markerVersion = targetInstalledVersion;
-        const dryRun = getFlag(args, "dry-run");
-        if (markerVersion === null && !dryRun) {
-            out.err("");
-            out.err(`  Could not detect the installed Kumiko version under ${targetDir}.`);
-            out.err("  Refusing to write an upgrade marker without it (--from is only a filter).");
-            out.err("");
-            return 1;
+    return {
+        kind: "ok",
+        value: { ...baseline.value, targetInstalledVersion, allEntries, pending },
+    };
+}
+async function applyPlannedUpgrade(out, args, plan, repoRoot, targetDir) {
+    // Marker must reflect what is actually installed under the target (or
+    // cwd), not the filter baseline above — otherwise CI stays green forever.
+    const markerVersion = plan.targetInstalledVersion;
+    const dryRun = getFlag(args, "dry-run");
+    if (markerVersion === null && !dryRun) {
+        return failUpgrade(out, `Could not detect the installed Kumiko version under ${targetDir}.`, "Refusing to write an upgrade marker without it (--from is only a filter).");
+    }
+    out.log("");
+    const code = await applyCodemods(out, plan.pending, repoRoot, targetDir, dryRun, markerVersion ?? plan.currentVersion, carryOf(plan.markerRead), plan.currentVersion, getFlag(args, "verbose") ? manualDetailsOf(plan.allEntries) : undefined);
+    out.log("");
+    return code;
+}
+function reportUpgradeJson(out, plan) {
+    const { currentVersion, installedVersion } = plan;
+    const { pendingManual } = carryOf(plan.markerRead);
+    const pendingWithManualId = plan.pending.map((entry) => {
+        const manualId = manualIdOf(entry);
+        return manualId === undefined ? entry : { ...entry, manualId };
+    });
+    out.log(JSON.stringify({ currentVersion, installedVersion, pending: pendingWithManualId, pendingManual }, null, 2));
+    return 0;
+}
+function logEntryDetail(out, entry, verbose) {
+    if (verbose && entry.detail) {
+        out.log(`      ${entry.detail}`);
+    }
+}
+function logBreakingSection(out, breaking, verbose) {
+    // skip: no breaking entries, the section is omitted
+    if (breaking.length === 0)
+        return;
+    out.log(`  ⚠ BREAKING (${breaking.length})`);
+    out.log("");
+    for (const e of breaking) {
+        const manualId = manualIdOf(e);
+        out.log(`    ${e.version} · ${e.title}${manualId === undefined ? "" : `  [${manualId}]`}`);
+        logEntryDetail(out, e, verbose);
+        if (e.migration) {
+            out.log(`      → Migration: ${e.migration}`);
         }
         out.log("");
-        const code = await applyCodemods(out, pending, repoRoot, targetDir, dryRun, markerVersion ?? currentVersion, carryOf(markerRead), currentVersion, verbose ? manualDetailsOf(allEntries) : undefined);
-        out.log("");
-        return code;
-    }
-    if (jsonMode) {
-        const { pendingManual } = carryOf(markerRead);
-        const pendingWithManualId = pending.map((entry) => {
-            const manualId = manualIdOf(entry);
-            return manualId === undefined ? entry : { ...entry, manualId };
-        });
-        out.log(JSON.stringify({ currentVersion, installedVersion, pending: pendingWithManualId, pendingManual }, null, 2));
-        return 0;
     }
+}
+function logNonBreakingSection(out, heading, entries, verbose, showCodemod) {
+    // skip: no entries of this type, the section is omitted
+    if (entries.length === 0)
+        return;
+    out.log(`  ✓ ${heading} (${entries.length})`);
+    out.log("");
+    for (const e of entries) {
+        out.log(`    ${e.version} · ${e.title}`);
+        logEntryDetail(out, e, verbose);
+        if (showCodemod && hasCodemod(e)) {
+            out.log(`      → codemod: ${e.codemod} (runs on --apply)`);
+        }
+    }
+    out.log("");
+}
+function reportUpgradeText(out, plan, verbose) {
+    const { pending } = plan;
     const breaking = pending.filter((e) => e.type === "breaking");
-    const improvements = pending.filter((e) => e.type === "improvement");
-    const fixes = pending.filter((e) => e.type === "fix");
-    const openManual = carryOf(markerRead).pendingManual;
+    const openManual = carryOf(plan.markerRead).pendingManual;
+    const manualDetails = verbose ? manualDetailsOf(plan.allEntries) : undefined;
     out.log("");
-    out.log(`  Upgrade: ${currentVersion} → latest`);
+    out.log(`  Upgrade: ${plan.currentVersion} → latest`);
     out.log("");
     if (pending.length === 0) {
         out.log("  ✓ Nothing new since your version.");
         out.log("");
-        logOpenManual(out, openManual, verbose ? manualDetailsOf(allEntries) : undefined);
+        logOpenManual(out, openManual, manualDetails);
         return 0;
     }
-    if (breaking.length > 0) {
-        out.log(`  ⚠ BREAKING (${breaking.length})`);
-        out.log("");
-        for (const e of breaking) {
-            const manualId = manualIdOf(e);
-            out.log(`    ${e.version} · ${e.title}${manualId === undefined ? "" : `  [${manualId}]`}`);
-            if (verbose && e.detail) {
-                out.log(`      ${e.detail}`);
-            }
-            if (e.migration) {
-                out.log(`      → Migration: ${e.migration}`);
-            }
-            out.log("");
-        }
-    }
-    if (improvements.length > 0) {
-        out.log(`  ✓ IMPROVEMENTS (${improvements.length})`);
-        out.log("");
-        for (const e of improvements) {
-            out.log(`    ${e.version} · ${e.title}`);
-            if (verbose && e.detail) {
-                out.log(`      ${e.detail}`);
-            }
-        }
-        out.log("");
-    }
-    if (fixes.length > 0) {
-        out.log(`  ✓ FIXES (${fixes.length})`);
-        out.log("");
-        for (const e of fixes) {
-            out.log(`    ${e.version} · ${e.title}`);
-            if (verbose && e.detail) {
-                out.log(`      ${e.detail}`);
-            }
-        }
-        out.log("");
-    }
+    logBreakingSection(out, breaking, verbose);
+    logNonBreakingSection(out, "IMPROVEMENTS", pending.filter((e) => e.type === "improvement"), verbose, true);
+    logNonBreakingSection(out, "FIXES", pending.filter((e) => e.type === "fix"), verbose, false);
     if (breaking.length > 0) {
         out.log("  ⚠ Review breaking changes above before upgrading.");
         out.log("  Run with --verbose for full migration details.");
         out.log("");
     }
-    logOpenManual(out, openManual, verbose ? manualDetailsOf(allEntries) : undefined);
+    logOpenManual(out, openManual, manualDetails);
     return 0;
 }
+// Shared by `kumiko upgrade` and the published kumiko-upgrade bin.
+export async function runUpgradeCli(argv, cwd, out, options) {
+    if (argv.includes("--help") || argv.includes("-h")) {
+        out.log(UPGRADE_USAGE);
+        return 0;
+    }
+    // Standalone CLI entry, not booted via runProdApp/runDevApp — Temporal needs an explicit polyfill here.
+    await ensureTemporalPolyfill();
+    const args = parseArgs(argv);
+    // Resolved once here (not just inside --apply) so both the filter
+    // baseline and the --apply codemod run target the same directory.
+    const dirFlag = getStringFlag(args, "dir");
+    const targetDir = dirFlag ? resolve(dirFlag) : cwd;
+    if (!existsSync(targetDir) || !statSync(targetDir).isDirectory()) {
+        return failUpgrade(out, `--dir path is not a directory: ${targetDir}`);
+    }
+    if (args.flags.has("resolve")) {
+        return resolveManualMigrations(out, args, targetDir);
+    }
+    const planned = planUpgrade(args, cwd, targetDir);
+    if (planned.kind === "error")
+        return failUpgrade(out, ...planned.lines);
+    if (getFlag(args, "apply")) {
+        return applyPlannedUpgrade(out, args, planned.value, options?.repoRoot ?? cwd, targetDir);
+    }
+    if (getFlag(args, "json"))
+        return reportUpgradeJson(out, planned.value);
+    return reportUpgradeText(out, planned.value, getFlag(args, "verbose"));
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.387.0",
+  "version": "0.388.0",
   "description": "Framework core — engine, pipeline, API, DB, and every other bit that makes Kumiko go.",
   "license": "MIT",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -199,7 +199,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-types": "0.387.0",
+    "@cosmicdrift/kumiko-types": "0.388.0",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.13.1",
@@ -208,14 +208,14 @@
     "jose": "^6.2.3",
     "meilisearch": "^0.58.0",
     "pino": "^10.3.1",
-    "postgres": "npm:@bender0oo0/postgres@3.4.9-kumiko.1",
+    "postgres": "npm:@bender0oo0/postgres@3.4.9-kumiko.2",
     "temporal-polyfill": "^0.3.2",
     "ts-morph": "^28.0.0",
     "uuid": "^14.0.0",
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-testing": "0.387.0",
+    "@cosmicdrift/kumiko-testing": "0.388.0",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },