@crewhaus/docker-images 0.1.1 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crewhaus/docker-images",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "type": "module",
   "description": "Per-target-shape Docker images (multi-stage, non-root, healthchecked) + crewhaus build-image CLI wrapper (Section 32)",
   "main": "src/index.ts",
@@ -12,13 +12,13 @@
     "test": "bun test src"
   },
   "dependencies": {
-    "@crewhaus/errors": "0.1.1"
+    "@crewhaus/errors": "0.1.3"
   },
   "license": "Apache-2.0",
   "author": {
     "name": "Max Meier",
-    "email": "max@studiomax.io",
-    "url": "https://studiomax.io"
+    "email": "max@crewhaus.ai",
+    "url": "https://crewhaus.ai"
   },
   "repository": {
     "type": "git",
@@ -30,12 +30,7 @@
     "url": "https://github.com/crewhaus/factory/issues"
   },
   "publishConfig": {
-    "access": "restricted"
+    "access": "public"
   },
-  "files": [
-    "src",
-    "README.md",
-    "LICENSE",
-    "NOTICE"
-  ]
+  "files": ["src", "README.md", "LICENSE", "NOTICE"]
 }

package/src/index.test.ts

@@ -8,6 +8,8 @@ import {
   DockerImagesError,
   TARGET_SHAPES,
   buildImage,
+  defaultRunner,
+  digestsPath,
   dockerRoot,
   dockerfilePath,
   dockerfileSize,
@@ -61,6 +63,11 @@ describe("Dockerfiles on disk", () => {
   test("dockerRoot points inside the package", () => {
     expect(dockerRoot()).toMatch(/packages\/docker-images\/docker$/);
   });
+
+  test("digestsPath resolves to docker/digests.json inside the package", () => {
+    expect(digestsPath()).toMatch(/packages\/docker-images\/docker\/digests\.json$/);
+    expect(digestsPath()).toBe(join(dockerRoot(), "digests.json"));
+  });
 });
 
 describe("Dockerfile structural contract (T2)", () => {
@@ -252,3 +259,49 @@ CMD ["--help"]
     expect(fp.exposedPorts).toEqual([]);
   });
 });
+
+describe("defaultRunner (spawn integration, deterministic)", () => {
+  // We drive the real spawn-based runner using the running JS binary
+  // (`process.execPath`) with `-e` — no docker, no network, no external
+  // service. argv[0] is the program; the rest are passed verbatim.
+  const exe = process.execPath;
+
+  test("captures stdout, stderr, and a zero exit code", async () => {
+    const result = await defaultRunner(
+      [exe, "-e", 'process.stdout.write("hello-out");process.stderr.write("hello-err")'],
+      process.cwd(),
+    );
+    expect(result.exitCode).toBe(0);
+    expect(result.stdout).toBe("hello-out");
+    expect(result.stderr).toBe("hello-err");
+  });
+
+  test("propagates a non-zero exit code", async () => {
+    const result = await defaultRunner(
+      [exe, "-e", 'process.stderr.write("boom");process.exit(7)'],
+      process.cwd(),
+    );
+    expect(result.exitCode).toBe(7);
+    expect(result.stderr).toBe("boom");
+  });
+
+  test("resolves with exitCode 1 when the program cannot be spawned (error event)", async () => {
+    const result = await defaultRunner(
+      ["./crewhaus-nonexistent-binary-9f3a2c", "--version"],
+      process.cwd(),
+    );
+    expect(result.exitCode).toBe(1);
+    expect(result.stdout).toBe("");
+    expect(result.stderr.length).toBeGreaterThan(0);
+  });
+
+  test("falls back to exitCode 1 when the child is killed by a signal (null code)", async () => {
+    // A child that SIGKILLs itself terminates via signal, so `close` fires with
+    // code === null — exercising the `code ?? 1` fallback deterministically.
+    const result = await defaultRunner(
+      [exe, "-e", "process.kill(process.pid, 'SIGKILL')"],
+      process.cwd(),
+    );
+    expect(result.exitCode).toBe(1);
+  });
+});

package/src/index.ts

@@ -152,7 +152,7 @@ export async function buildImage(opts: BuildImageOptions): Promise<BuildImageRes
   return { tag: opts.tag, target: opts.target, buildArgv: argv };
 }
 
-const defaultRunner: BuildRunner = async (argv, cwd) => {
+export const defaultRunner: BuildRunner = async (argv, cwd) => {
   const { spawn } = await import("node:child_process");
   return new Promise((resolve_) => {
     const head = argv[0] ?? "docker";