@ollie-shop/cli 0.3.4 → 1.0.1

package/src/utils/validation-helpers.ts

@@ -1,325 +0,0 @@
-import {
-  ComponentNameSchema,
-  FunctionNameSchema,
-  PrioritySchema,
-  SemverSchema,
-  StoreIdSchema,
-  UrlSchema,
-  VersionNameSchema,
-} from "../schemas/command.schema";
-import type { CliConsole } from "./console";
-
-/**
- * Result of a validation operation
- * @interface ValidationResult
- * @property {boolean} valid - Whether the validation passed
- * @property {Array<{message?: string} | string>} [errors] - List of validation errors
- * @property {Array<{message?: string} | string>} [warnings] - List of validation warnings
- */
-export interface ValidationResult {
-  valid: boolean;
-  errors?: Array<{ message?: string } | string>;
-  warnings?: Array<{ message?: string } | string>;
-}
-
-/**
- * Interface for managing spinner states
- * @interface SpinnerManager
- * @property {Function} succeed - Mark spinner as successful with message
- * @property {Function} fail - Mark spinner as failed with message
- */
-export interface SpinnerManager {
-  succeed: (text: string) => void;
-  fail: (text: string) => void;
-}
-
-/**
- * Validates a component name against naming conventions
- * Component names must be lowercase, use hyphens, and follow kebab-case
- *
- * @param {string} name - The component name to validate
- * @returns {true | string} Returns true if valid, or an error message string if invalid
- * @example
- * ```typescript
- * validateComponentName("my-header") // returns true
- * validateComponentName("MyHeader") // returns "Component name must be lowercase"
- * validateComponentName("my_header") // returns "Use hyphens instead of underscores"
- * ```
- */
-export function validateComponentName(name: string): true | string {
-  const result = ComponentNameSchema.safeParse(name);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid component name";
-}
-
-/**
- * Validates a function name against naming conventions
- * Function names must be lowercase, use hyphens, and follow kebab-case
- *
- * @param {string} name - The function name to validate
- * @returns {true | string} Returns true if valid, or an error message string if invalid
- * @example
- * ```typescript
- * validateFunctionName("validate-cart") // returns true
- * validateFunctionName("validateCart") // returns "Function name must be lowercase"
- * validateFunctionName("validate cart") // returns "No spaces allowed in function names"
- * ```
- */
-export function validateFunctionName(name: string): true | string {
-  const result = FunctionNameSchema.safeParse(name);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid function name";
-}
-
-/**
- * Validates a store ID is in proper UUID v4 format
- *
- * @param {string} storeId - The store ID to validate
- * @returns {true | string} Returns true if valid UUID, or an error message if invalid
- * @example
- * ```typescript
- * validateStoreId("123e4567-e89b-12d3-a456-426614174000") // returns true
- * validateStoreId("invalid-id") // returns "Invalid store ID format"
- * ```
- */
-export function validateStoreId(storeId: string): true | string {
-  const result = StoreIdSchema.safeParse(storeId);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid store ID";
-}
-
-/**
- * Validates a version name meets requirements
- * Version names must be 1-100 characters long
- *
- * @param {string} name - The version name to validate
- * @returns {true | string} Returns true if valid, or an error message if invalid
- * @example
- * ```typescript
- * validateVersionName("Production v2.0") // returns true
- * validateVersionName("") // returns "Version name is required"
- * validateVersionName("a".repeat(101)) // returns "Version name is too long (max 100 characters)"
- * ```
- */
-export function validateVersionName(name: string): true | string {
-  const result = VersionNameSchema.safeParse(name);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid version name";
-}
-
-/**
- * Validates a priority value is within acceptable range
- * Priority must be an integer between 0 and 100 (inclusive)
- *
- * @param {number} priority - The priority value to validate
- * @returns {true | string} Returns true if valid, or an error message if invalid
- * @example
- * ```typescript
- * validatePriority(50) // returns true
- * validatePriority(0) // returns true (minimum)
- * validatePriority(100) // returns true (maximum)
- * validatePriority(150) // returns "Priority must be between 0 and 100"
- * validatePriority(50.5) // returns "Priority must be a whole number"
- * ```
- */
-export function validatePriority(priority: number): true | string {
-  const result = PrioritySchema.safeParse(priority);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid priority";
-}
-
-/**
- * Validates a URL or URL pattern for functions
- * Supports both full URLs (http/https) and path patterns (e.g., /api/*)
- *
- * @param {string} url - The URL or pattern to validate
- * @returns {true | string} Returns true if valid, or an error message if invalid
- * @example
- * ```typescript
- * validateUrl("https://api.example.com/webhook") // returns true
- * validateUrl("/api/cart/*") // returns true (path pattern)
- * validateUrl("/*") // returns true (wildcard pattern)
- * validateUrl("ftp://example.com") // returns "URL must be a valid URL or path pattern"
- * validateUrl("") // returns "URL is required"
- * ```
- */
-export function validateUrl(url: string): true | string {
-  const result = UrlSchema.safeParse(url);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid URL";
-}
-
-/**
- * Validates a semantic version string
- * Must follow semver format: MAJOR.MINOR.PATCH (e.g., 1.0.0)
- *
- * @param {string} version - The version string to validate
- * @returns {true | string} Returns true if valid semver, or an error message if invalid
- * @example
- * ```typescript
- * validateSemver("1.0.0") // returns true
- * validateSemver("2.4.6") // returns true
- * validateSemver("1.0") // returns "Version must follow semantic versioning format (e.g., 1.0.0)"
- * validateSemver("v1.0.0") // returns "Version must follow semantic versioning format (e.g., 1.0.0)"
- * ```
- */
-export function validateSemver(version: string): true | string {
-  const result = SemverSchema.safeParse(version);
-  return result.success
-    ? true
-    : result.error.errors[0]?.message || "Invalid semantic version";
-}
-
-/**
- * Processes and displays validation errors to the user
- * Formats errors into a readable list with bullet points
- *
- * @param {Array<{message?: string} | string>} errors - Array of error objects or strings
- * @param {CliConsole} cliConsole - Console instance for output
- * @example
- * ```typescript
- * processValidationErrors(["Invalid name", {message: "Path not found"}], console);
- * // Output:
- * // Errors:
- * // • Invalid name
- * // • Path not found
- * ```
- */
-export function processValidationErrors(
-  errors: Array<{ message?: string } | string>,
-  cliConsole: CliConsole,
-): void {
-  cliConsole.error("Errors:");
-  for (const error of errors) {
-    const message = typeof error === "string" ? error : error.message || error;
-    cliConsole.error(`• ${message}`);
-  }
-}
-
-/**
- * Processes and displays validation warnings to the user
- * Formats warnings into a readable list with bullet points
- *
- * @param {Array<{message?: string} | string>} warnings - Array of warning objects or strings
- * @param {CliConsole} cliConsole - Console instance for output
- * @example
- * ```typescript
- * processValidationWarnings(["Deprecated feature used"], console);
- * // Output:
- * // Warnings:
- * // • Deprecated feature used
- * ```
- */
-export function processValidationWarnings(
-  warnings: Array<{ message?: string } | string>,
-  cliConsole: CliConsole,
-): void {
-  cliConsole.warn("Warnings:");
-  for (const warning of warnings) {
-    const message =
-      typeof warning === "string" ? warning : warning.message || warning;
-    cliConsole.warn(`• ${message}`);
-  }
-}
-
-/**
- * Handles validation results by updating spinner state and displaying errors/warnings
- * Central function for processing validation outcomes in CLI commands
- *
- * @param {ValidationResult | unknown} result - Validation result object or unknown type
- * @param {SpinnerManager} spinner - Spinner instance to update visual feedback
- * @param {CliConsole} cliConsole - Console instance for output
- * @param {string} entityType - Type of entity being validated (e.g., "Component", "Function")
- * @example
- * ```typescript
- * const result = await validateComponent(path);
- * handleValidationResult(result, spinner, console, "Component");
- * // Updates spinner to success/fail and displays any errors or warnings
- * ```
- */
-export function handleValidationResult(
-  result: ValidationResult | unknown,
-  spinner: SpinnerManager,
-  cliConsole: CliConsole,
-  entityType: string,
-): void {
-  if (typeof result === "object" && result !== null && "valid" in result) {
-    const validationResult = result as ValidationResult;
-
-    handleValidResult(validationResult, spinner, cliConsole, entityType);
-    handleValidationWarnings(validationResult, cliConsole);
-    handleValidationErrors(validationResult);
-  } else {
-    spinner.succeed(`${entityType} validation completed`);
-  }
-}
-
-/**
- * Updates spinner state based on validation success/failure
- * Internal helper for handleValidationResult
- *
- * @param {ValidationResult} result - Validation result with valid status
- * @param {SpinnerManager} spinner - Spinner to update
- * @param {CliConsole} cliConsole - Console for error output
- * @param {string} entityType - Type of entity for messaging
- * @private
- */
-function handleValidResult(
-  result: ValidationResult,
-  spinner: SpinnerManager,
-  cliConsole: CliConsole,
-  entityType: string,
-): void {
-  if (result.valid) {
-    spinner.succeed(`${entityType} is valid!`);
-  } else {
-    spinner.fail(`${entityType} validation failed`);
-    if (result.errors?.length) {
-      processValidationErrors(result.errors, cliConsole);
-    }
-  }
-}
-
-/**
- * Displays validation warnings if present
- * Internal helper for handleValidationResult
- *
- * @param {ValidationResult} result - Validation result potentially containing warnings
- * @param {CliConsole} cliConsole - Console for warning output
- * @private
- */
-function handleValidationWarnings(
-  result: ValidationResult,
-  cliConsole: CliConsole,
-): void {
-  if (result.warnings?.length) {
-    processValidationWarnings(result.warnings, cliConsole);
-  }
-}
-
-/**
- * Handle validation errors and throw if invalid
- */
-function handleValidationErrors(result: ValidationResult): void {
-  if (!result.valid) {
-    throw new Error("Validation failed");
-  }
-}
-
-// Test helpers for mocking
-export function createMockSpinner(): SpinnerManager {
-  return {
-    succeed: () => {
-      // Mock spinner success - intentionally empty
-    },
-    fail: () => {
-      // Mock spinner failure - intentionally empty
-    },
-  };
-}

package/vitest.config.ts

@@ -1,35 +0,0 @@
-import path from "node:path";
-import { defineConfig } from "vitest/config";
-
-export default defineConfig({
-  test: {
-    globals: true,
-    environment: "node",
-    exclude: ["**/node_modules/**", "**/dist/**", "**/components/**"],
-    include: ["src/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"],
-    coverage: {
-      provider: "v8",
-      reporter: ["text", "json", "html"],
-      exclude: [
-        "node_modules/",
-        "dist/",
-        "**/*.d.ts",
-        "**/*.config.ts",
-        "**/__tests__/**",
-      ],
-      thresholds: {
-        lines: 80,
-        functions: 80,
-        branches: 80,
-        statements: 80,
-      },
-    },
-    setupFiles: ["./vitest.setup.ts"],
-  },
-  resolve: {
-    alias: {
-      "@": path.resolve(__dirname, "./src"),
-      "@tests": path.resolve(__dirname, "./src/__tests__"),
-    },
-  },
-});

package/vitest.setup.ts

@@ -1,29 +0,0 @@
-import { vi } from "vitest";
-
-// Mock console methods
-global.console = {
-  ...console,
-  log: vi.fn(),
-  debug: vi.fn(),
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-};
-
-// Mock process.exit
-vi.spyOn(process, "exit").mockImplementation((code?: string | number) => {
-  throw new Error(`process.exit called with code ${code}`);
-});
-
-// Set test environment
-process.env.NODE_ENV = "test";
-process.env.OLLIE_SHOP_CONFIG_DIR = "/tmp/ollie-shop-test";
-
-// Global test utilities
-beforeEach(() => {
-  vi.clearAllMocks();
-});
-
-afterEach(() => {
-  vi.restoreAllMocks();
-});