@jango-blockchained/hoox-cli 0.5.2 → 0.5.6

package/src/services/config/config-service.test.ts

@@ -1,395 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from "bun:test";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { ConfigService } from "./config-service.js";
-
-/**
- * Helper: write a wrangler.jsonc file to a test directory using Bun.
- */
-async function writeConfig(dir: string, content: string): Promise<void> {
-  await Bun.write(join(dir, "wrangler.jsonc"), content);
-}
-
-/**
- * Helper: create a minimal valid config JSON string.
- */
-function validConfigJson(): string {
-  return JSON.stringify(
-    {
-      global: {
-        cloudflare_account_id: "abc123",
-      },
-      workers: {
-        "test-worker": {
-          enabled: true,
-          path: "workers/test-worker",
-        },
-      },
-    },
-    null,
-    2
-  );
-}
-
-/**
- * Helper: create a valid config with JSONC comments.
- */
-function validConfigWithComments(): string {
-  return `{
-  // This is a line comment
-  "global": {
-    /* Block comment explaining account_id */
-    "cloudflare_account_id": "abc123"
-  },
-  "workers": {
-    "test-worker": {
-      "enabled": true,
-      "path": "workers/test-worker" // inline comment
-    }
-  }
-}`;
-}
-
-describe("ConfigService", () => {
-  let tmpDir: string;
-
-  beforeEach(() => {
-    tmpDir = mkdtempSync(join(tmpdir(), "hoox-config-test-"));
-  });
-
-  afterEach(() => {
-    rmSync(tmpDir, { recursive: true, force: true });
-  });
-
-  // ── Constructor defaults ──────────────────────────────────────────
-
-  it("defaults config path to wrangler.jsonc in cwd", () => {
-    const service = new ConfigService();
-    // Constructor stores the default path — verified via load() behavior
-    expect(service).toBeInstanceOf(ConfigService);
-  });
-
-  it("accepts a custom config path in constructor", () => {
-    const customPath = "/tmp/custom-wrangler.jsonc";
-    const service = new ConfigService(customPath);
-    expect(service).toBeInstanceOf(ConfigService);
-  });
-
-  // ── load() — happy path ───────────────────────────────────────────
-
-  it("loads and parses a valid wrangler.jsonc", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    const config = await service.load();
-    expect(config.global.cloudflare_account_id).toBe("abc123");
-    expect(config.workers["test-worker"]).toBeDefined();
-  });
-
-  it("loads a config with enabled/disabled workers", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        "worker-a": { enabled: true, path: "workers/a" },
-        "worker-b": { enabled: false, path: "workers/b" },
-        "worker-c": { enabled: true, path: "workers/c" },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    expect(service.listWorkers()).toEqual(["worker-a", "worker-b", "worker-c"]);
-    expect(service.listEnabledWorkers()).toEqual(["worker-a", "worker-c"]);
-  });
-
-  // ── load() — error cases ──────────────────────────────────────────
-
-  it("throws when config file does not exist", async () => {
-    const service = new ConfigService(join(tmpDir, "nonexistent.jsonc"));
-    await expect(service.load()).rejects.toThrow("not found");
-  });
-
-  it("throws for invalid JSONC syntax", async () => {
-    await writeConfig(tmpDir, "{ not valid json @@@ }");
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await expect(service.load()).rejects.toThrow("Invalid JSONC");
-  });
-
-  it("throws when root is not an object (e.g. an array)", async () => {
-    await writeConfig(tmpDir, "[1, 2, 3]");
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await expect(service.load()).rejects.toThrow("must contain a JSON object");
-  });
-
-  it("throws when root is a primitive (e.g. string)", async () => {
-    await writeConfig(tmpDir, '"just a string"');
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await expect(service.load()).rejects.toThrow("must contain a JSON object");
-  });
-
-  // ── Comment stripping (JSONC support) ─────────────────────────────
-
-  it("strips // line comments via jsonc-parser", async () => {
-    await writeConfig(tmpDir, validConfigWithComments());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    const config = await service.load();
-    expect(config.global.cloudflare_account_id).toBe("abc123");
-  });
-
-  it("strips /* */ block comments via jsonc-parser", async () => {
-    const content = `{
-  /* multi
-     line
-     block */
-  "global": { "cloudflare_account_id": "xyz" },
-  "workers": { "w": { "enabled": true, "path": "p" } }
-}`;
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    const config = await service.load();
-    expect(config.global.cloudflare_account_id).toBe("xyz");
-  });
-
-  it("handles pure JSON (no comments) correctly", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    const config = await service.load();
-    expect(config.global.cloudflare_account_id).toBe("abc123");
-  });
-
-  // ── Accessor methods (after load) ─────────────────────────────────
-
-  it("getWorker() returns a worker config by name", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const worker = service.getWorker("test-worker");
-    expect(worker).toBeDefined();
-    expect(worker?.enabled).toBe(true);
-    expect(worker?.path).toBe("workers/test-worker");
-  });
-
-  it("getWorker() returns undefined for unknown worker", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    expect(service.getWorker("nonexistent")).toBeUndefined();
-  });
-
-  it("getGlobal() returns the global config section", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const global = service.getGlobal();
-    expect(global.cloudflare_account_id).toBe("abc123");
-  });
-
-  it("listWorkers() returns all worker names", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        w1: { enabled: true, path: "workers/w1" },
-        w2: { enabled: false, path: "workers/w2" },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    expect(service.listWorkers().sort()).toEqual(["w1", "w2"]);
-  });
-
-  it("listEnabledWorkers() filters to only enabled workers", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        w1: { enabled: true, path: "workers/w1" },
-        w2: { enabled: false, path: "workers/w2" },
-        w3: { enabled: true, path: "workers/w3" },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const enabled = service.listEnabledWorkers().sort();
-    expect(enabled).toEqual(["w1", "w3"]);
-    expect(enabled).not.toContain("w2");
-  });
-
-  it("listEnabledWorkers() returns empty array when all disabled", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        w1: { enabled: false, path: "workers/w1" },
-        w2: { enabled: false, path: "workers/w2" },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    expect(service.listEnabledWorkers()).toEqual([]);
-  });
-
-  // ── Accessor guards (before load) ─────────────────────────────────
-
-  it("getWorker() throws if called before load()", () => {
-    const service = new ConfigService();
-    expect(() => service.getWorker("any")).toThrow("not loaded");
-  });
-
-  it("getGlobal() throws if called before load()", () => {
-    const service = new ConfigService();
-    expect(() => service.getGlobal()).toThrow("not loaded");
-  });
-
-  it("listWorkers() throws if called before load()", () => {
-    const service = new ConfigService();
-    expect(() => service.listWorkers()).toThrow("not loaded");
-  });
-
-  it("listEnabledWorkers() throws if called before load()", () => {
-    const service = new ConfigService();
-    expect(() => service.listEnabledWorkers()).toThrow("not loaded");
-  });
-
-  // ── validate() ────────────────────────────────────────────────────
-
-  it("validate() returns valid: true for a valid config", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(true);
-    expect(result.errors).toEqual([]);
-  });
-
-  it("validate() returns errors when config not loaded", () => {
-    const service = new ConfigService();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    expect(result.errors[0]).toContain("not loaded");
-  });
-
-  it("validate() catches missing global.cloudflare_account_id", async () => {
-    const content = JSON.stringify({
-      global: {},
-      workers: { w: { enabled: true, path: "workers/w" } },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    expect(result.errors.some((e) => e.includes("cloudflare_account_id"))).toBe(
-      true
-    );
-  });
-
-  it("validate() catches missing global section entirely", async () => {
-    const content = JSON.stringify({
-      workers: { w: { enabled: true, path: "workers/w" } },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    expect(result.errors.some((e) => e.includes("cloudflare_account_id"))).toBe(
-      true
-    );
-  });
-
-  it("validate() catches missing worker path", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        "bad-worker": { enabled: true },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    expect(result.errors.some((e) => e.includes("path"))).toBe(true);
-  });
-
-  it("validate() catches empty workers section", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {},
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    expect(result.errors.some((e) => e.includes("No workers"))).toBe(true);
-  });
-
-  it("validate() reports multiple errors at once", async () => {
-    const content = JSON.stringify({
-      global: {},
-      workers: {
-        w1: { enabled: true },
-        w2: { enabled: false },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const result = service.validate();
-    expect(result.valid).toBe(false);
-    // Should have at least 3 errors: account_id missing, w1 missing path, w2 missing path
-    expect(result.errors.length).toBeGreaterThanOrEqual(3);
-  });
-
-  // ── Worker config with secrets and vars ───────────────────────────
-
-  it("parses worker secrets array", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        "secrets-worker": {
-          enabled: true,
-          path: "workers/secrets-worker",
-          secrets: ["API_KEY", "DB_PASSWORD"],
-        },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const worker = service.getWorker("secrets-worker");
-    expect(worker?.secrets).toEqual(["API_KEY", "DB_PASSWORD"]);
-  });
-
-  it("parses worker vars", async () => {
-    const content = JSON.stringify({
-      global: { cloudflare_account_id: "abc123" },
-      workers: {
-        "vars-worker": {
-          enabled: true,
-          path: "workers/vars-worker",
-          vars: { DATABASE_NAME: "my-db", LOG_LEVEL: "debug" },
-        },
-      },
-    });
-    await writeConfig(tmpDir, content);
-    const service = new ConfigService(join(tmpDir, "wrangler.jsonc"));
-    await service.load();
-    const worker = service.getWorker("vars-worker");
-    expect(worker?.vars).toEqual({
-      DATABASE_NAME: "my-db",
-      LOG_LEVEL: "debug",
-    });
-  });
-
-  // ── load() called with explicit path overrides constructor ─────────
-
-  it("load() with explicit path overrides the constructor path", async () => {
-    await writeConfig(tmpDir, validConfigJson());
-    const service = new ConfigService("/some/wrong/path.jsonc");
-    const config = await service.load(join(tmpDir, "wrangler.jsonc"));
-    expect(config.global.cloudflare_account_id).toBe("abc123");
-  });
-});

package/src/services/config/config-service.ts

@@ -1,202 +0,0 @@
-import { parse, printParseErrorCode } from "jsonc-parser";
-import type { ParseError } from "jsonc-parser";
-import type { HooxConfig, WorkerConfig, GlobalConfig } from "./types";
-
-/**
- * Loads, parses, and validates the central wrangler.jsonc configuration.
- *
- * Uses jsonc-parser for fault-tolerant JSONC parsing with native comment
- * support (both line and block comments). Provides typed accessors for the
- * global config and per-worker entries.
- *
- * @example
- * ```ts
- * const config = new ConfigService();
- * await config.load();
- * console.log(config.listEnabledWorkers()); // ["d1-worker", "hoox", ...]
- * ```
- */
-export class ConfigService {
-  private config: HooxConfig | null = null;
-  private configPath: string;
-
-  /**
-   * @param configPath - Absolute or relative path to wrangler.jsonc.
-   *   Defaults to `wrangler.jsonc` in the current working directory.
-   */
-  constructor(configPath?: string) {
-    this.configPath = configPath ?? "wrangler.jsonc";
-  }
-
-  /**
-   * Read and parse wrangler.jsonc from disk.
-   *
-   * Strips JSONC comments via jsonc-parser's native `parse()`.
-   * Throws on file-not-found, invalid JSONC, or missing root object.
-   *
-   * @param configPath - Override the path set in the constructor.
-   * @returns The parsed HooxConfig object.
-   */
-  async load(configPath?: string): Promise<HooxConfig> {
-    const filePath = configPath ?? this.configPath;
-    const file = Bun.file(filePath);
-
-    if (!(await file.exists())) {
-      throw new Error(
-        `Config file not found: ${filePath}. Run 'hoox config init' to create one.`
-      );
-    }
-
-    const content = await file.text();
-    const errors: ParseError[] = [];
-    const raw: unknown = parse(content, errors);
-
-    if (errors.length > 0) {
-      const messages = errors.map(
-        (e) =>
-          `  - ${printParseErrorCode(e.error)} at offset ${e.offset} (length ${e.length})`
-      );
-      throw new Error(`Invalid JSONC in ${filePath}:\n${messages.join("\n")}`);
-    }
-
-    if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
-      throw new Error(`${filePath} must contain a JSON object at the root`);
-    }
-
-    this.config = raw as HooxConfig;
-    return this.config;
-  }
-
-  /**
-   * Get the configuration for a specific worker by name.
-   *
-   * @param name - The worker key as defined in wrangler.jsonc (e.g. "d1-worker").
-   * @returns The worker config, or undefined if not found.
-   */
-  getWorker(name: string): WorkerConfig | undefined {
-    const config = this.ensureLoaded();
-    return config.workers[name];
-  }
-
-  /**
-   * Get the global configuration section.
-   */
-  getGlobal(): GlobalConfig {
-    const config = this.ensureLoaded();
-    return config.global;
-  }
-
-  /**
-   * List all worker names defined in wrangler.jsonc (enabled + disabled).
-   */
-  listWorkers(): string[] {
-    const config = this.ensureLoaded();
-    return Object.keys(config.workers);
-  }
-
-  /**
-   * List only enabled worker names.
-   */
-  listEnabledWorkers(): string[] {
-    const config = this.ensureLoaded();
-    return Object.entries(config.workers)
-      .filter(([, w]) => w.enabled)
-      .map(([name]) => name);
-  }
-
-  /**
-   * Validate the loaded configuration against required fields.
-   *
-   * Required fields:
-   *   - global.cloudflare_account_id
-   *   - Each worker must have a `path` property
-   *
-   * @returns An object with `valid` and an `errors` array.
-   */
-  validate(): { valid: boolean; errors: string[] } {
-    const errors: string[] = [];
-
-    if (!this.config) {
-      return {
-        valid: false,
-        errors: ["Config not loaded. Call load() first."],
-      };
-    }
-
-    // Global required fields
-    if (!this.config.global?.cloudflare_account_id) {
-      errors.push(
-        "Required field missing: global.cloudflare_account_id — must be set in wrangler.jsonc"
-      );
-    }
-
-    // Worker required fields
-    if (!this.config.workers || Object.keys(this.config.workers).length === 0) {
-      errors.push("No workers defined in wrangler.jsonc");
-    } else {
-      for (const [name, worker] of Object.entries(this.config.workers)) {
-        if (!worker.path) {
-          errors.push(`Worker "${name}" is missing required field: path`);
-        }
-      }
-    }
-
-    return { valid: errors.length === 0, errors };
-  }
-
-  /**
-   * Get the preferred dev runtime from config ("native" or "docker").
-   */
-  getDevRuntime(): "native" | "docker" | null {
-    return this.ensureLoaded().dev?.runtime ?? null;
-  }
-
-  /**
-   * Persist the dev runtime preference to wrangler.jsonc.
-   *
-   * Reads the current file, updates the dev.runtime field, and writes it
-   * back. Preserves JSONC comments where possible.
-   */
-  async setDevRuntime(runtime: "native" | "docker"): Promise<void> {
-    const filePath = this.configPath;
-    const file = Bun.file(filePath);
-
-    if (!(await file.exists())) {
-      throw new Error(`Config file not found: ${filePath}`);
-    }
-
-    const raw = await file.text();
-    const errors: ParseError[] = [];
-    const parsed = parse(raw, errors);
-
-    if (
-      errors.length > 0 ||
-      parsed === null ||
-      typeof parsed !== "object" ||
-      Array.isArray(parsed)
-    ) {
-      throw new Error(`Invalid JSONC in ${filePath}`);
-    }
-
-    const config = parsed as Record<string, unknown>;
-    if (!config.dev) {
-      config.dev = {};
-    }
-    (config.dev as Record<string, unknown>).runtime = runtime;
-
-    // Write back as regular JSON (jsonc-parser doesn't support round-trip comment preservation)
-    await Bun.write(filePath, JSON.stringify(config, null, 2) + "\n");
-  }
-
-  /**
-   * Guard: throws if config hasn't been loaded yet; returns the loaded config.
-   */
-  private ensureLoaded(): HooxConfig {
-    if (!this.config) {
-      throw new Error(
-        "Config not loaded. Call load() before accessing configuration."
-      );
-    }
-    return this.config;
-  }
-}

package/src/services/config/index.ts

@@ -1,9 +0,0 @@
-export { ConfigService } from "./config-service";
-export type {
-  HooxConfig,
-  GlobalConfig,
-  WorkerConfig,
-  WorkerSecret,
-  WorkerVars,
-  DevConfig,
-} from "./types";

package/src/services/config/types.ts

@@ -1,66 +0,0 @@
-/**
- * ConfigService types — strongly-typed interfaces for wrangler.jsonc.
- *
- * The shape mirrors the actual wrangler.jsonc at the project root:
- *   global: { cloudflare_api_token, cloudflare_account_id, cloudflare_secret_store_id, subdomain_prefix }
- *   workers: { name: { enabled, path, vars?, secrets? } }
- */
-
-/**
- * A secret referenced by a worker. Each entry is the Cloudflare secret
- * name that the worker expects at runtime.
- */
-export type WorkerSecret = string;
-
-/**
- * Environment variables bound to a worker at deploy time.
- */
-export interface WorkerVars {
-  [key: string]: string;
-}
-
-/**
- * Per-worker configuration as defined in wrangler.jsonc.
- */
-export interface WorkerConfig {
-  /** Whether the worker is deployed and managed by the CLI. */
-  enabled: boolean;
-  /** Relative path from project root to the worker directory. */
-  path: string;
-  /** Plain-text environment variables (non-secret build-time vars). */
-  vars?: WorkerVars;
-  /** Cloudflare secret names this worker needs at runtime. */
-  secrets?: WorkerSecret[];
-}
-
-/**
- * Development configuration for local runtime selection.
- */
-export interface DevConfig {
-  /** Preferred runtime: "native" (wrangler dev) or "docker" (compose). */
-  runtime?: "native" | "docker";
-}
-
-/**
- * Global configuration shared across all workers in the project.
- */
-export interface GlobalConfig {
-  /** Cloudflare API token for authenticated operations. */
-  cloudflare_api_token?: string;
-  /** Cloudflare account ID (required — must be set before deploy). */
-  cloudflare_account_id?: string;
-  /** Secret Store ID for centralized secret management. */
-  cloudflare_secret_store_id?: string;
-  /** Subdomain prefix used for worker URL routing. */
-  subdomain_prefix?: string;
-}
-
-/**
- * Top-level Hoox configuration — the root shape of wrangler.jsonc.
- */
-export interface HooxConfig {
-  global: GlobalConfig;
-  workers: Record<string, WorkerConfig>;
-  /** Development settings (runtime preference, etc.). */
-  dev?: DevConfig;
-}