@jango-blockchained/hoox-cli 0.5.2 → 0.5.6

package/src/commands/schema/schema-command.ts

@@ -1,137 +0,0 @@
-import { resolve } from "path";
-import { Command } from "commander";
-import { SchemaService } from "../../services/schema/schema-service.js";
-import {
-  formatSuccess,
-  formatError,
-  formatKeyValue,
-  formatTable,
-  getFormatOptions,
-} from "../../utils/formatters.js";
-import { CLIError, ExitCode } from "../../utils/errors.js";
-import { withErrorHandling } from "../../utils/error-handler.js";
-import {
-  generateWranglerJsonc,
-  generateDevVars,
-} from "@jango-blockchained/hoox-shared";
-
-export function registerSchemaCommand(program: Command): void {
-  const schemaCmd = program
-    .command("schema")
-    .summary("Validate and manage worker configuration")
-    .description(
-      "Validate, generate, and repair worker wrangler.jsonc against canonical manifests."
-    );
-
-  schemaCmd
-    .command("validate [worker]")
-    .description(
-      "Validate worker(s) against manifest (omit worker to validate all)"
-    )
-    .action(
-      withErrorHandling(async (worker?: string) => {
-        const fmt = getFormatOptions(schemaCmd);
-        const svc = new SchemaService();
-        const results = worker
-          ? [svc.validateWorker(worker)]
-          : svc.validateAll();
-        let totalErrors = 0;
-        for (const r of results) {
-          if (r.passed) {
-            formatSuccess(`${r.worker}: ✅ passed`, fmt);
-          } else {
-            formatError(
-              new CLIError(
-                `${r.worker}: ❌ failed (${r.errors.length} issues)`,
-                ExitCode.ERROR
-              ),
-              fmt
-            );
-            for (const e of r.errors.filter((e) => e.severity === "error")) {
-              process.stderr.write(`  ✗ ${e.message}\n`);
-            }
-            totalErrors += r.errors.filter(
-              (e) => e.severity === "error"
-            ).length;
-          }
-        }
-        if (totalErrors > 0) process.exitCode = ExitCode.ERROR;
-      })
-    );
-
-  schemaCmd
-    .command("list")
-    .description("List all workers with their binding counts")
-    .action(
-      withErrorHandling(async () => {
-        const fmt = getFormatOptions(schemaCmd);
-        const svc = new SchemaService();
-        const rows: Record<string, string>[] = [];
-        for (const name of svc.getWorkerNames()) {
-          const m = svc.getManifest(name)!;
-          const secretCount = Object.values(m.vars).filter(
-            (v) => v.type === "secret"
-          ).length;
-          const svcCount = m.services.length;
-          const infraCount = Object.keys(m.infrastructure).length;
-          const middleware = m.middleware.length
-            ? m.middleware.join(", ")
-            : "\u2014";
-          rows.push({
-            worker: name,
-            secrets: String(secretCount),
-            services: String(svcCount),
-            infra: String(infraCount),
-            middleware,
-          });
-        }
-        formatTable(rows, fmt);
-      })
-    );
-
-  schemaCmd
-    .command("generate <worker>")
-    .description("Generate wrangler.jsonc and .dev.vars from manifest")
-    .option("--dry-run", "Print generated content without writing")
-    .action(
-      withErrorHandling(async (worker: string, opts: { dryRun?: boolean }) => {
-        const fmt = getFormatOptions(schemaCmd);
-        const svc = new SchemaService();
-        const manifest = svc.getManifest(worker);
-        if (!manifest) {
-          formatError(
-            new CLIError(`Unknown worker "${worker}"`, ExitCode.INVALID_USAGE),
-            fmt
-          );
-          return;
-        }
-
-        const wranglerContent = generateWranglerJsonc(manifest);
-        const devVarsContent = generateDevVars(manifest);
-
-        if (opts.dryRun) {
-          process.stdout.write(`--- wrangler.jsonc (${worker}) ---\n`);
-          process.stdout.write(wranglerContent + "\n");
-          process.stdout.write(`--- .dev.vars (${worker}) ---\n`);
-          process.stdout.write(devVarsContent + "\n");
-          return;
-        }
-
-        const workersDir = resolve(process.cwd(), "workers", worker);
-        const wranglerPath = resolve(workersDir, "wrangler.jsonc");
-        const devVarsPath = resolve(workersDir, ".dev.vars");
-
-        await Bun.write(wranglerPath, wranglerContent);
-        await Bun.write(devVarsPath, devVarsContent);
-
-        formatKeyValue(
-          {
-            "wrangler.jsonc": wranglerPath,
-            ".dev.vars": devVarsPath,
-          },
-          { ...fmt, json: false }
-        );
-        formatSuccess(`Generated files for ${worker}`, fmt);
-      })
-    );
-}

package/src/commands/test/index.ts

@@ -1,2 +0,0 @@
-export { registerTestCommand } from "./test-command.js";
-export type { TestStepResult, TestSummary } from "./test-command.js";

package/src/commands/test/test-command.test.ts

@@ -1,318 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, mock } from "bun:test";
-import { Command } from "commander";
-import { registerTestCommand, runStep, printSummary } from "./test-command.js";
-import type { TestSummary } from "./test-command.js";
-import { ExitCode } from "../../utils/errors.js";
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-const originalSpawn = Bun.spawn;
-
-/** Create a mock Bun.spawn that returns the given exit code and output. */
-function mockSpawn(exitCode: number, stdout = "", stderr = "") {
-  return mock(() => ({
-    exited: Promise.resolve(exitCode),
-    stdout: new Blob([stdout]),
-    stderr: new Blob([stderr]),
-  }));
-}
-
-/** Capture stdout for assertion. */
-function captureStdout(): { output: () => string; restore: () => void } {
-  const chunks: string[] = [];
-  const originalWrite = process.stdout.write.bind(process.stdout);
-  const writeMock = mock((chunk: string | Buffer) => {
-    chunks.push(typeof chunk === "string" ? chunk : chunk.toString());
-    return true;
-  });
-  process.stdout.write = writeMock as unknown as typeof process.stdout.write;
-  return {
-    output: () => chunks.join(""),
-    restore: () => {
-      process.stdout.write = originalWrite;
-    },
-  };
-}
-
-/** Create a fresh Commander program with global --json / --quiet options. */
-function makeProgram(): Command {
-  const program = new Command();
-  program.name("hoox");
-  program.option("--json", "");
-  program.option("--quiet", "");
-  return program;
-}
-
-// ---------------------------------------------------------------------------
-// runStep
-// ---------------------------------------------------------------------------
-
-describe("runStep", () => {
-  afterEach(() => {
-    (Bun as any).spawn = originalSpawn;
-  });
-
-  it("returns success when exit code is 0", async () => {
-    (Bun as any).spawn = mockSpawn(0, "all tests passed");
-    const result = await runStep(["bun", "test"]);
-    expect(result.success).toBe(true);
-    expect(result.exitCode).toBe(0);
-    expect(result.output).toBe("all tests passed");
-  });
-
-  it("returns failure when exit code is non-zero", async () => {
-    (Bun as any).spawn = mockSpawn(1, "", "3 tests failed");
-    const result = await runStep(["bun", "test"]);
-    expect(result.success).toBe(false);
-    expect(result.exitCode).toBe(1);
-    expect(result.error).toBe("3 tests failed");
-  });
-
-  it("captures both stdout and stderr", async () => {
-    (Bun as any).spawn = mockSpawn(0, "stdout text", "stderr text");
-    const result = await runStep(["echo", "hello"]);
-    expect(result.output).toBe("stdout text");
-    expect(result.error).toBe("stderr text");
-  });
-
-  it("records duration", async () => {
-    (Bun as any).spawn = mockSpawn(0);
-    const result = await runStep(["bun", "test"]);
-    expect(result.duration).toBeGreaterThanOrEqual(0);
-  });
-
-  it("returns the command string", async () => {
-    (Bun as any).spawn = mockSpawn(0);
-    const result = await runStep([
-      "vitest",
-      "run",
-      "--config",
-      "vite.config.ts",
-    ]);
-    expect(result.command).toBe("vitest run --config vite.config.ts");
-  });
-
-  it("handles spawn errors gracefully", async () => {
-    (Bun as any).spawn = mock(() => {
-      throw new Error("ENOENT: no such file");
-    });
-    const result = await runStep(["nonexistent"]);
-    expect(result.success).toBe(false);
-    expect(result.exitCode).toBeNull();
-    expect(result.error).toContain("ENOENT");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// printSummary
-// ---------------------------------------------------------------------------
-
-describe("printSummary", () => {
-  let capture: ReturnType<typeof captureStdout>;
-
-  beforeEach(() => {
-    capture = captureStdout();
-  });
-
-  afterEach(() => {
-    capture.restore();
-  });
-
-  const passingSummary: TestSummary = {
-    total: 4,
-    passed: 4,
-    failed: 0,
-    results: [
-      {
-        step: "Lint",
-        command: "bun run lint",
-        success: true,
-        exitCode: 0,
-        duration: 1200,
-      },
-      {
-        step: "TypeCheck",
-        command: "bun run typecheck",
-        success: true,
-        exitCode: 0,
-        duration: 3400,
-      },
-      {
-        step: "Unit Tests",
-        command: "bun test --coverage",
-        success: true,
-        exitCode: 0,
-        duration: 8900,
-      },
-      {
-        step: "Integration Tests",
-        command: "vitest run...",
-        success: true,
-        exitCode: 0,
-        duration: 15000,
-      },
-    ],
-  };
-
-  const failingSummary: TestSummary = {
-    total: 2,
-    passed: 1,
-    failed: 1,
-    results: [
-      {
-        step: "Lint",
-        command: "bun run lint",
-        success: true,
-        exitCode: 0,
-        duration: 500,
-      },
-      {
-        step: "TypeCheck",
-        command: "bun run typecheck",
-        success: false,
-        exitCode: 2,
-        duration: 800,
-        error: "TS2322: type error",
-      },
-    ],
-  };
-
-  it("prints a table for human output", () => {
-    printSummary(passingSummary, {});
-    const out = capture.output();
-    expect(out).toContain("All 4 steps passed");
-    expect(out).toContain("Lint");
-    expect(out).toContain("passed");
-  });
-
-  it("shows failure count when some steps fail", () => {
-    printSummary(failingSummary, {});
-    const out = capture.output();
-    expect(out).toContain("1/2 passed");
-    expect(out).toContain("TypeCheck");
-    expect(out).toContain("failed");
-  });
-
-  it("outputs JSON when json=true", () => {
-    printSummary(passingSummary, { json: true });
-    const out = capture.output();
-    const parsed = JSON.parse(out);
-    expect(parsed.total).toBe(4);
-    expect(parsed.passed).toBe(4);
-    expect(parsed.failed).toBe(0);
-    expect(parsed.results).toHaveLength(4);
-  });
-
-  it("outputs nothing when quiet=true", () => {
-    printSummary(passingSummary, { quiet: true });
-    expect(capture.output()).toBe("");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// registerTestCommand — structure verification
-// ---------------------------------------------------------------------------
-
-describe("registerTestCommand", () => {
-  it("registers test command with all subcommands", () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    const testCmd = program.commands.find((c) => c.name() === "test");
-    expect(testCmd).toBeDefined();
-    expect(testCmd?.description()).toContain("Run tests");
-
-    const subNames = testCmd?.commands.map((c) => c.name()) ?? [];
-    expect(subNames).toContain("all");
-    expect(subNames).toContain("unit");
-    expect(subNames).toContain("integration");
-    expect(subNames).toContain("worker");
-  });
-
-  it("all subcommand has --json option", () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    const testCmd = program.commands.find((c) => c.name() === "test");
-    const allCmd = testCmd?.commands.find((c) => c.name() === "all");
-    expect(allCmd).toBeDefined();
-
-    // Commander stores options; verify --json is registered
-    const optionNames = allCmd?.options.map((o) => o.long) ?? [];
-    expect(optionNames).toContain("--json");
-  });
-
-  it("unit subcommand has --coverage option", () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    const testCmd = program.commands.find((c) => c.name() === "test");
-    const unitCmd = testCmd?.commands.find((c) => c.name() === "unit");
-    expect(unitCmd).toBeDefined();
-
-    const optionNames = unitCmd?.options.map((o) => o.long) ?? [];
-    expect(optionNames).toContain("--coverage");
-  });
-
-  it("integration subcommand has --coverage option", () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    const testCmd = program.commands.find((c) => c.name() === "test");
-    const intCmd = testCmd?.commands.find((c) => c.name() === "integration");
-    expect(intCmd).toBeDefined();
-
-    const optionNames = intCmd?.options.map((o) => o.long) ?? [];
-    expect(optionNames).toContain("--coverage");
-  });
-
-  it("worker subcommand requires a name argument and has --coverage option", () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    const testCmd = program.commands.find((c) => c.name() === "test");
-    const workerCmd = testCmd?.commands.find((c) => c.name() === "worker");
-    expect(workerCmd).toBeDefined();
-
-    // Verify it expects a required argument
-    const args = workerCmd?.registeredArguments ?? [];
-    expect(args.length).toBeGreaterThan(0);
-
-    const optionNames = workerCmd?.options.map((o) => o.long) ?? [];
-    expect(optionNames).toContain("--coverage");
-  });
-
-  it("all subcommand runs pipeline steps and stops on first failure", async () => {
-    const program = makeProgram();
-    registerTestCommand(program);
-
-    // First lint passes, typecheck fails → should stop after 2 steps
-    let callCount = 0;
-    const mockSpawnFn = mock((args: string[]) => {
-      callCount++;
-      const cmd = args.join(" ");
-      // Lint passes (call 1), TypeCheck fails (call 2)
-      const exitCode = cmd.includes("lint") ? 0 : 1;
-      const stderr = cmd.includes("lint") ? "" : "TS2322: type error";
-      return {
-        exited: Promise.resolve(exitCode),
-        stdout: new Blob([]),
-        stderr: new Blob([stderr]),
-      };
-    });
-    (Bun as any).spawn = mockSpawnFn;
-
-    // Suppress spinner output by running without tty-like env
-    await program.parseAsync(["test", "all"], {
-      from: "user",
-    });
-
-    // Should only run 2 steps (lint + typecheck) before stopping
-    expect(callCount).toBe(2);
-    expect(process.exitCode).toBe(ExitCode.ERROR);
-
-    (Bun as any).spawn = originalSpawn;
-  });
-});