@ollie-shop/cli 0.3.3 → 1.0.0

package/src/actions/version.actions.ts

@@ -1,233 +0,0 @@
-import * as core from "@ollie-shop/core";
-import type { VersionListItem } from "../schemas/command.schema";
-import { console as cliConsole } from "../utils/console";
-import { handleError } from "../utils/error-handler";
-
-interface CreateOptions {
-  store: string;
-  name: string;
-  template?: string;
-  active?: boolean;
-}
-
-interface ListOptions {
-  store: string;
-}
-
-export async function create(options: CreateOptions): Promise<void> {
-  const spinner = cliConsole.spinner("Creating version...");
-
-  try {
-    const version = await core.createVersion({
-      storeId: options.store,
-      name: options.name,
-      template: options.template || "default",
-      active: options.active ?? true,
-    });
-
-    spinner.succeed(`Version "${version.name}" created successfully`);
-
-    cliConsole.info("\nVersion details:");
-    cliConsole.table([
-      {
-        ID: version.id,
-        Name: version.name,
-        Template: version.template || "default",
-        Active: version.active ? "✓" : "✗",
-        Default: version.default ? "✓" : "✗",
-      },
-    ]);
-
-    if (!version.active) {
-      cliConsole.dim("\nNote: This version is inactive. Activate it with:");
-      cliConsole.dim(`  ollieshop version activate ${version.id}`);
-    }
-  } catch (error) {
-    spinner.fail("Failed to create version");
-    handleError(error);
-  }
-}
-
-export async function list(options: ListOptions): Promise<void> {
-  const spinner = cliConsole.spinner("Loading versions...");
-
-  try {
-    const versions: VersionListItem[] = await core.listVersions(options.store);
-
-    spinner.succeed();
-
-    if (versions.length === 0) {
-      cliConsole.warn("No versions found for this store");
-      cliConsole.dim("\nCreate a version with:");
-      cliConsole.dim(
-        `  ollieshop version create --store ${options.store} --name "My Version"`,
-      );
-      return;
-    }
-
-    cliConsole.info(
-      `Found ${versions.length} version${versions.length === 1 ? "" : "s"}:\n`,
-    );
-
-    cliConsole.table(
-      versions.map((v) => ({
-        ID: v.id,
-        Name: v.name,
-        Template: v.template || "default",
-        Active: v.active ? "✓" : "✗",
-        Default: v.default ? "✓" : "✗",
-        Created: new Date(v.createdAt).toLocaleDateString(),
-      })),
-    );
-  } catch (error) {
-    spinner.fail("Failed to list versions");
-    handleError(error);
-  }
-}
-
-export async function get(versionId: string): Promise<void> {
-  const spinner = cliConsole.spinner("Loading version...");
-
-  try {
-    const version = await core.getVersion(versionId);
-
-    spinner.succeed();
-
-    cliConsole.info("Version details:\n");
-
-    const details = {
-      ID: version.id,
-      Name: version.name,
-      "Store ID": version.storeId,
-      Template: version.template || "default",
-      Active: version.active ? "Yes" : "No",
-      Default: version.default ? "Yes" : "No",
-      Created: new Date(version.createdAt).toLocaleString(),
-      Updated: new Date(version.updatedAt).toLocaleString(),
-    };
-
-    for (const [key, value] of Object.entries(details)) {
-      cliConsole.log(`${cliConsole.bold(`${key}:`)} ${value}`);
-    }
-
-    if (version.theme) {
-      cliConsole.info("\nTheme configuration:");
-      cliConsole.json(version.theme);
-    }
-
-    if (version.props) {
-      cliConsole.info("\nProps configuration:");
-      cliConsole.json(version.props);
-    }
-  } catch (error) {
-    spinner.fail("Failed to get version");
-    handleError(error);
-  }
-}
-
-export async function setDefault(versionId: string): Promise<void> {
-  const spinner = cliConsole.spinner("Setting default version...");
-
-  try {
-    await core.setDefaultVersion(versionId);
-
-    spinner.succeed("Version set as default successfully");
-
-    cliConsole.dim(
-      "\nThis version will now be used for all new checkout sessions.",
-    );
-  } catch (error) {
-    spinner.fail("Failed to set default version");
-    handleError(error);
-  }
-}
-
-export async function activate(versionId: string): Promise<void> {
-  const spinner = cliConsole.spinner("Activating version...");
-
-  try {
-    const version = await core.activateVersion(versionId);
-
-    spinner.succeed(`Version "${version.name}" activated successfully`);
-
-    if (!version.default) {
-      cliConsole.dim("\nTo make this the default version, run:");
-      cliConsole.dim(`  ollieshop version set-default ${versionId}`);
-    }
-  } catch (error) {
-    spinner.fail("Failed to activate version");
-    handleError(error);
-  }
-}
-
-export async function deactivate(versionId: string): Promise<void> {
-  const spinner = cliConsole.spinner("Deactivating version...");
-
-  try {
-    const version = await core.deactivateVersion(versionId);
-
-    spinner.succeed(`Version "${version.name}" deactivated successfully`);
-
-    cliConsole.warn(
-      "\nWarning: This version can no longer be used for checkout sessions.",
-    );
-  } catch (error) {
-    spinner.fail("Failed to deactivate version");
-    handleError(error);
-  }
-}
-
-export async function clone(sourceId: string, name: string): Promise<void> {
-  const spinner = cliConsole.spinner("Cloning version...");
-
-  try {
-    const newVersion = await core.cloneVersion(sourceId, name);
-
-    spinner.succeed(`Version cloned successfully as "${newVersion.name}"`);
-
-    cliConsole.info("\nCloned version details:");
-    cliConsole.table([
-      {
-        ID: newVersion.id,
-        Name: newVersion.name,
-        Template: newVersion.template || "default",
-        Active: newVersion.active ? "✓" : "✗",
-        Default: newVersion.default ? "✓" : "✗",
-      },
-    ]);
-
-    cliConsole.dim("\nNote: The cloned version is inactive. To activate it:");
-    cliConsole.dim(`  ollieshop version activate ${newVersion.id}`);
-  } catch (error) {
-    spinner.fail("Failed to clone version");
-    handleError(error);
-  }
-}
-
-export async function deleteVersion(
-  versionId: string,
-  force?: boolean,
-): Promise<void> {
-  // Confirm deletion if not forced
-  if (!force) {
-    const confirmed = await cliConsole.confirm(
-      "Are you sure you want to delete this version? This action cannot be undone.",
-    );
-
-    if (!confirmed) {
-      cliConsole.info("Deletion cancelled");
-      return;
-    }
-  }
-
-  const spinner = cliConsole.spinner("Deleting version...");
-
-  try {
-    await core.deleteVersion(versionId);
-
-    spinner.succeed("Version deleted successfully");
-  } catch (error) {
-    spinner.fail("Failed to delete version");
-    handleError(error);
-  }
-}

package/src/commands/__tests__/component-validation.test.ts

@@ -1,250 +0,0 @@
-import type { Command } from "@commander-js/extra-typings";
-import {
-  createTestProgram,
-  executeCLI,
-  resetCLIMocks,
-} from "@tests/helpers/cli-test-helper";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-import { registerComponentCommands } from "../component";
-
-// Mock the component actions module
-vi.mock("../../actions/component.actions", () => ({
-  createComponent: vi.fn(),
-  listComponents: vi.fn(),
-  deployComponent: vi.fn(),
-  validateComponent: vi.fn(),
-}));
-
-import * as componentActions from "../../actions/component.actions";
-
-describe("Component Command Validation and DX", () => {
-  let program: Command;
-  let testSetup: ReturnType<typeof createTestProgram>;
-
-  beforeEach(() => {
-    testSetup = createTestProgram();
-    program = testSetup.program;
-    registerComponentCommands(program);
-    vi.clearAllMocks();
-  });
-
-  afterEach(() => {
-    resetCLIMocks();
-  });
-
-  describe("validation messages", () => {
-    it("should provide helpful error for uppercase component names", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "TestComponent",
-      ]);
-
-      // The validation errors are written through console.error which is captured by our mocks
-      const consoleErrorCalls = testSetup.mocks.console.error.mock.calls;
-      const consoleLogCalls = testSetup.mocks.console.log.mock.calls;
-
-      const consoleOutput = [
-        ...consoleErrorCalls.map((call) => call.join(" ")),
-        ...consoleLogCalls.map((call) => call.join(" ")),
-      ].join("\n");
-
-      expect(result.exitCode).toBe(1);
-      expect(consoleOutput).toContain("Validation failed");
-      expect(consoleOutput).toContain("name:");
-      expect(consoleOutput).toContain("Must be lowercase with hyphens only");
-      expect(consoleOutput).toContain("Examples:");
-      expect(componentActions.createComponent).not.toHaveBeenCalled();
-    });
-
-    it("should provide helpful error for component names with spaces", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "test component",
-      ]);
-
-      const consoleOutput = testSetup.mocks.console.error.mock.calls
-        .map((call) => call.join(" "))
-        .join("\n");
-
-      expect(result.exitCode).toBe(1);
-      expect(consoleOutput).toContain("Validation failed");
-      expect(consoleOutput).toContain("Must be lowercase with hyphens only");
-      expect(componentActions.createComponent).not.toHaveBeenCalled();
-    });
-
-    it("should provide helpful error for component names with underscores", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "test_component",
-      ]);
-
-      const consoleOutput = testSetup.mocks.console.error.mock.calls
-        .map((call) => call.join(" "))
-        .join("\n");
-
-      expect(result.exitCode).toBe(1);
-      expect(consoleOutput).toContain("Validation failed");
-      expect(consoleOutput).toContain("Must be lowercase with hyphens only");
-      expect(componentActions.createComponent).not.toHaveBeenCalled();
-    });
-
-    it("should fail when name is missing", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--slot",
-        "header",
-      ]);
-
-      expect(result.exitCode).toBe(1);
-      expect(componentActions.createComponent).not.toHaveBeenCalled();
-    });
-
-    it("should provide helpful error for invalid slot", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "test-component",
-        "--slot",
-        "invalid",
-      ]);
-
-      // The slot validation happens at the Commander.js level with a parser
-      // So we get a different error message format
-      const allOutput = [
-        ...result.stderr,
-        ...testSetup.mocks.console.error.mock.calls.map((call) =>
-          call.join(" "),
-        ),
-      ].join("\n");
-
-      expect(result.exitCode).toBe(1);
-      expect(allOutput).toContain("Invalid component slot");
-      expect(allOutput).toContain(
-        "Valid slots: header, footer, sidebar, main, checkout-summary, payment-method, shipping-method, customer-info, order-confirmation",
-      );
-      expect(componentActions.createComponent).not.toHaveBeenCalled();
-    });
-  });
-
-  describe("unknown options and commands", () => {
-    it("should provide helpful error for unknown options", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "test",
-        "--unknown",
-      ]);
-
-      expect(result.exitCode).toBe(1);
-      // Commander.js formats the error as "error: unknown option '--unknown'"
-      expect(result.stderr.join("")).toContain("unknown option '--unknown'");
-    });
-
-    it("should provide helpful error for unknown commands", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "craete",
-        "--name",
-        "test",
-      ]);
-
-      expect(result.exitCode).toBe(1);
-      // Commander.js formats the error as "error: unknown command 'craete'"
-      expect(result.stderr.join("")).toContain("unknown command 'craete'");
-      // Commander has built-in suggestions
-      expect(result.stderr.join("")).toContain("Did you mean");
-    });
-  });
-
-  describe("valid inputs", () => {
-    it("should accept valid kebab-case component names", async () => {
-      vi.mocked(componentActions.createComponent).mockResolvedValue();
-
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "header-navigation",
-      ]);
-
-      expect(result.exitCode).toBe(0);
-      expect(componentActions.createComponent).toHaveBeenCalledWith(
-        expect.objectContaining({
-          name: "header-navigation",
-          slot: "main",
-          tests: true,
-          interactive: false,
-        }),
-        expect.any(Object),
-      );
-    });
-
-    it("should accept component names with numbers", async () => {
-      vi.mocked(componentActions.createComponent).mockResolvedValue();
-
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "product-list-v2",
-      ]);
-
-      expect(result.exitCode).toBe(0);
-      expect(componentActions.createComponent).toHaveBeenCalledWith(
-        expect.objectContaining({
-          name: "product-list-v2",
-        }),
-        expect.any(Object),
-      );
-    });
-
-    it("should accept single word lowercase names", async () => {
-      vi.mocked(componentActions.createComponent).mockResolvedValue();
-
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--name",
-        "header",
-      ]);
-
-      expect(result.exitCode).toBe(0);
-      expect(componentActions.createComponent).toHaveBeenCalledWith(
-        expect.objectContaining({
-          name: "header",
-        }),
-        expect.any(Object),
-      );
-    });
-  });
-
-  describe("help command", () => {
-    it("should show examples in help output", async () => {
-      const result = await executeCLI(program, [
-        "component",
-        "create",
-        "--help",
-      ]);
-
-      const output = result.stdout.join("") + result.stderr.join("");
-      expect(output).toContain("Examples:");
-      expect(output).toContain("Create a header component:");
-      expect(output).toContain(
-        "$ ollieshop component create --name header --slot header",
-      );
-      expect(output).toContain("Create a component with TypeScript:");
-      expect(output).toContain(
-        "$ ollieshop component create --name product-list --typescript",
-      );
-    });
-  });
-});