@cosmicdrift/kumiko-feature-ast 0.390.1 → 0.391.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-feature-ast",
-  "version": "0.390.1",
+  "version": "0.391.0",
   "description": "Feature-file AST for Kumiko: parse, patch and render feature definitions, the pattern library, and the codemods kumiko-upgrade runs.",
   "license": "MIT",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -22,8 +22,8 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.390.1",
-    "@cosmicdrift/kumiko-types": "0.390.1",
+    "@cosmicdrift/kumiko-framework": "0.391.0",
+    "@cosmicdrift/kumiko-types": "0.391.0",
     "ts-morph": "^28.0.0",
     "zod": "^4.6.7"
   },

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.391.0",
+    "type": "fix",
+    "title": "Codemods no longer fail when git is not installed",
+    "detail": "collectSourceFiles asks git check-ignore which files the repository's .gitignore excludes. Without a git executable Bun.spawnSync threw ENOENT and kumiko-upgrade --apply aborted at the first codemod, for example ctx-namespaces in the customer-app builder image. A missing git is now treated like a directory outside a repository: no file is filtered."
+  },
   {
     "version": "0.390.1",
     "type": "fix",

package/src/scripts/codemod/collect-source-files.ts

@@ -1,5 +1,6 @@
 import { lstatSync, readdirSync, statSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
+import { errorCodeOf } from "@cosmicdrift/kumiko-types/type-guards";
 
 export const BUILD_AND_VENDOR_DIR_NAMES: readonly string[] = [
   "node_modules",
@@ -68,15 +69,26 @@ export function envWithoutGitRepositoryOverrides(): Record<string, string> {
 // a codemod must not rewrite them.
 function withoutGitIgnored(cwd: string, files: readonly string[]): readonly string[] {
   if (files.length === 0) return files;
-  const result = Bun.spawnSync({
-    cmd: ["git", "check-ignore", "--stdin", "-z"],
-    cwd,
-    env: envWithoutGitRepositoryOverrides(),
-    stdin: Buffer.from(`${files.join("\0")}\0`),
-    stdout: "pipe",
-    stderr: "pipe",
-  });
-  if (result.exitCode !== GIT_CHECK_IGNORE_SOME_IGNORED) return files;
+  const result = runGitCheckIgnore(cwd, files);
+  if (result === undefined || result.exitCode !== GIT_CHECK_IGNORE_SOME_IGNORED) return files;
   const ignored = new Set(result.stdout.toString().split("\0"));
   return files.filter((file) => !ignored.has(file));
 }
+
+function runGitCheckIgnore(cwd: string, files: readonly string[]) {
+  try {
+    return Bun.spawnSync({
+      cmd: ["git", "check-ignore", "--stdin", "-z"],
+      cwd,
+      env: envWithoutGitRepositoryOverrides(),
+      stdin: Buffer.from(`${files.join("\0")}\0`),
+      stdout: "pipe",
+      stderr: "pipe",
+    });
+  } catch (error) {
+    // Build images such as the customer-app builder ship without git; like outside a repository,
+    // there is no .gitignore to honour then.
+    if (errorCodeOf(error) === "ENOENT") return undefined;
+    throw error;
+  }
+}