bundle-binary (#7)

* bundle-binary

* knip

Author: cramforce

.github/workflows/unit-tests.yml

@@ -28,3 +28,6 @@ jobs:
 
       - name: Run unit tests
         run: pnpm test:unit
+
+      - name: Run unit tests
+        run: pnpm test:dist

knip.json

@@ -1,5 +1,6 @@
 {
   "$schema": "https://unpkg.com/knip@5/schema.json",
+  "entry": ["src/cli/bash-env.ts"],
   "project": ["src/**/*.ts"],
   "ignore": ["src/**/*.test.ts", "src/spec-tests/**"],
   "ignoreBinaries": ["tsx"],

package.json

@@ -32,18 +32,20 @@
     "README.md"
   ],
   "bin": {
-    "bash-env": "./dist/cli/bash-env.js",
+    "bash-env": "./dist/bin/bash-env.js",
     "bash-env-shell": "./dist/cli/shell.js"
   },
   "scripts": {
-    "build": "tsc",
-    "validate": "pnpm build && pnpm typecheck && pnpm lint && pnpm knip && pnpm test:run",
+    "build": "tsc && pnpm build:cli",
+    "build:cli": "esbuild dist/cli/bash-env.js --bundle --splitting --platform=node --format=esm --minify --outdir=dist/bin --entry-names=[name] --chunk-names=chunks/[name]-[hash] --banner:js='#!/usr/bin/env node'",
+    "validate": "pnpm build && pnpm typecheck && pnpm lint && pnpm knip && pnpm test:run && pnpm test:dist",
     "typecheck": "tsc --noEmit",
     "lint": "biome check .",
     "lint:fix": "biome check --write .",
     "knip": "knip",
     "test": "vitest",
     "test:run": "vitest run",
+    "test:dist": "vitest run src/cli/bash-env.bundle.test.ts",
     "test:unit": "vitest run --config vitest.unit.config.ts",
     "test:comparison": "vitest run --config vitest.comparison.config.ts",
     "shell": "npx tsx src/cli/shell.ts",
@@ -58,6 +60,7 @@
     "@types/sprintf-js": "^1.1.4",
     "@types/turndown": "^5.0.6",
     "ai": "^6.0.3",
+    "esbuild": "^0.27.2",
     "knip": "^5.41.1",
     "typescript": "^5.9.3",
     "vitest": "^4.0.16",

pnpm-lock.yaml

@@ -0,0 +1,96 @@
+import { execFile } from "node:child_process";
+import { resolve } from "node:path";
+import { promisify } from "node:util";
+import { describe, expect, it } from "vitest";
+
+const execFileAsync = promisify(execFile);
+const binPath = resolve(__dirname, "../../dist/bin/bash-env.js");
+
+async function runBin(
+  args: string[],
+): Promise<{ stdout: string; stderr: string; exitCode: number }> {
+  try {
+    const { stdout, stderr } = await execFileAsync("node", [binPath, ...args]);
+    return { stdout, stderr, exitCode: 0 };
+  } catch (error: unknown) {
+    const e = error as { stdout?: string; stderr?: string; code?: number };
+    return {
+      stdout: e.stdout ?? "",
+      stderr: e.stderr ?? "",
+      exitCode: e.code ?? 1,
+    };
+  }
+}
+
+describe("bash-env bundled binary", () => {
+  it("should show version", async () => {
+    const result = await runBin(["--version"]);
+    expect(result.stdout).toContain("bash-env");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should show help", async () => {
+    const result = await runBin(["--help"]);
+    expect(result.stdout).toContain("Usage:");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should execute echo command", async () => {
+    const result = await runBin(["-c", "echo hello world"]);
+    expect(result.stdout).toBe("hello world\n");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should execute pipes", async () => {
+    const result = await runBin(["-c", 'echo "line1\nline2\nline3" | wc -l']);
+    expect(result.stdout.trim()).toBe("3");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should handle file operations with --allow-write", async () => {
+    const result = await runBin([
+      "-c",
+      'echo "test" > /tmp/test.txt && cat /tmp/test.txt',
+      "--allow-write",
+    ]);
+    expect(result.stdout).toBe("test\n");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should support JSON output", async () => {
+    const result = await runBin(["-c", "echo hello", "--json"]);
+    const json = JSON.parse(result.stdout);
+    expect(json.stdout).toBe("hello\n");
+    expect(json.stderr).toBe("");
+    expect(json.exitCode).toBe(0);
+  });
+
+  it("should lazy-load commands (grep)", async () => {
+    const result = await runBin([
+      "-c",
+      'echo -e "foo\\nbar\\nbaz" | grep ba',
+      "--allow-write",
+    ]);
+    expect(result.stdout).toContain("bar");
+    expect(result.stdout).toContain("baz");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should lazy-load commands (sed)", async () => {
+    const result = await runBin(["-c", "echo hello | sed 's/hello/world/'"]);
+    expect(result.stdout).toBe("world\n");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should lazy-load commands (awk)", async () => {
+    const result = await runBin(["-c", "echo 'a b c' | awk '{print $2}'"]);
+    expect(result.stdout).toBe("b\n");
+    expect(result.exitCode).toBe(0);
+  });
+
+  it("should handle errexit mode", async () => {
+    const result = await runBin(["-e", "-c", "false; echo should not print"]);
+    expect(result.stdout).not.toContain("should not print");
+    expect(result.exitCode).toBe(1);
+  });
+});

src/cli/bash-env.bundle.test.ts

@@ -1,4 +1,3 @@
-#!/usr/bin/env node
 /**
  * bash-env CLI - A secure alternative to bash for AI agents
  *