novaui-cli 1.1.2 → 1.1.3

package/src/__tests__/fuzzy.test.js

@@ -0,0 +1,150 @@
+import { describe, it, expect } from 'vitest'
+
+import { findSimilarComponents, formatComponentNotFoundError } from '../utils/fuzzy.js'
+
+describe('findSimilarComponents', () => {
+  const availableComponents = [
+    'button',
+    'card',
+    'dialog',
+    'drawer',
+    'dropdown-menu',
+    'alert',
+    'alert-dialog',
+    'badge',
+    'breadcrumb',
+    'calendar',
+  ]
+
+  it('finds exact typo matches', () => {
+    const result = findSimilarComponents('buton', availableComponents)
+    expect(result).toContain('button')
+  })
+
+  it('finds similar names with single character difference', () => {
+    const result = findSimilarComponents('crad', availableComponents)
+    expect(result).toContain('card')
+  })
+
+  it('finds names starting with same letter', () => {
+    const result = findSimilarComponents('but', availableComponents)
+    expect(result[0]).toBe('button')
+  })
+
+  it('returns empty array for very different inputs', () => {
+    const result = findSimilarComponents('xyz123', availableComponents)
+    expect(result).toEqual([])
+  })
+
+  it('limits results to maxResults parameter', () => {
+    const result = findSimilarComponents('a', availableComponents, 2)
+    expect(result.length).toBeLessThanOrEqual(2)
+  })
+
+  it('sorts by similarity (closest match first)', () => {
+    const result = findSimilarComponents('dilog', availableComponents)
+    expect(result[0]).toBe('dialog')
+  })
+
+  it('finds close matches with reasonable threshold', () => {
+    const result = findSimilarComponents('buton', availableComponents)
+    // Should find button with 1 character difference
+    expect(result).toContain('button')
+  })
+
+  it('handles case insensitivity', () => {
+    const result = findSimilarComponents('BUTTON', availableComponents)
+    expect(result).toContain('button')
+  })
+
+  it('handles whitespace in input', () => {
+    const result = findSimilarComponents('  button  ', availableComponents)
+    expect(result).toContain('button')
+  })
+
+  it('suggests exact matches and similar names', () => {
+    const result = findSimilarComponents('alert', availableComponents)
+    expect(result).toContain('alert')
+    // May also include alert-dialog if distance is close enough
+    expect(result.length).toBeGreaterThan(0)
+  })
+
+  it('returns up to 5 results by default', () => {
+    const result = findSimilarComponents('a', availableComponents)
+    expect(result.length).toBeLessThanOrEqual(5)
+  })
+
+  it('prefers similar matches', () => {
+    const result = findSimilarComponents('butn', availableComponents)
+    // Should find button as closest match
+    expect(result[0]).toBe('button')
+  })
+
+  it('handles component with multiple word separators', () => {
+    const result = findSimilarComponents('alertdialog', availableComponents)
+    expect(result).toContain('alert-dialog')
+  })
+})
+
+describe('formatComponentNotFoundError', () => {
+  const availableComponents = ['button', 'card', 'dialog', 'alert', 'badge']
+
+  it('includes the component name in error message', () => {
+    const error = formatComponentNotFoundError('buton', availableComponents)
+    expect(error).toContain('buton')
+    expect(error).toContain('not found in registry')
+  })
+
+  it('suggests similar component names', () => {
+    const error = formatComponentNotFoundError('buton', availableComponents)
+    expect(error).toContain('Did you mean one of these?')
+    expect(error).toContain('button')
+  })
+
+  it('lists all available components', () => {
+    const error = formatComponentNotFoundError('xyz', availableComponents)
+    expect(error).toContain('Available components:')
+    expect(error).toContain('button')
+    expect(error).toContain('card')
+    expect(error).toContain('dialog')
+  })
+
+  it('formats available components in columns', () => {
+    const manyComponents = Array.from({ length: 20 }, (_, i) => `component-${i}`)
+    const error = formatComponentNotFoundError('xyz', manyComponents)
+    
+    // Should have multiple lines
+    const lines = error.split('\n')
+    expect(lines.length).toBeGreaterThan(5)
+  })
+
+  it('does not suggest anything for very different names', () => {
+    const error = formatComponentNotFoundError('xyz123abc', availableComponents)
+    expect(error).not.toContain('Did you mean')
+    expect(error).toContain('Available components:')
+  })
+
+  it('handles empty component list gracefully', () => {
+    const error = formatComponentNotFoundError('button', [])
+    expect(error).toContain('not found in registry')
+    expect(error).toContain('Available components:')
+  })
+
+  it('suggests multiple matches when applicable', () => {
+    const components = ['alert', 'alert-dialog', 'alert-box']
+    const error = formatComponentNotFoundError('aler', components)
+    expect(error).toContain('alert')
+    expect(error).toContain('Did you mean one of these?')
+  })
+
+  it('includes formatting with bullet points for suggestions', () => {
+    const error = formatComponentNotFoundError('buton', availableComponents)
+    expect(error).toContain('•')
+  })
+
+  it('handles long component names', () => {
+    const components = ['very-long-component-name-that-exceeds-twenty-chars']
+    const error = formatComponentNotFoundError('verylong', components)
+    expect(error).toContain('very-long-component-name-that-exceeds-twenty-chars')
+  })
+})

package/src/__tests__/helpers.test.js

@@ -0,0 +1,520 @@
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest"
+import fs from "node:fs"
+import path from "node:path"
+import os from "node:os"
+
+import {
+  DEFAULT_CONFIG,
+  CONFIG_FILENAME,
+  GLOBAL_CSS_CONTENT,
+  UTILS_CONTENT,
+  getTailwindConfigContent,
+  detectPackageManager,
+  ensureDir,
+  writeIfNotExists,
+  getMissingDeps,
+  loadConfig,
+  writeConfig,
+  assertValidComponentConfig,
+  fetchWithTimeout,
+  formatError,
+  getCliVersion,
+  getInstallHint,
+  init,
+  add,
+} from "../bin.js"
+
+// ─── Test helpers ───────────────────────────────────────────────────────────
+
+function makeTmpDir() {
+  return fs.mkdtempSync(path.join(os.tmpdir(), "novaui-test-"))
+}
+
+function cleanTmpDir(dir) {
+  fs.rmSync(dir, { recursive: true, force: true })
+}
+
+// ─── DEFAULT_CONFIG ─────────────────────────────────────────────────────────
+
+describe("DEFAULT_CONFIG", () => {
+  it("has the expected default paths", () => {
+    expect(DEFAULT_CONFIG).toEqual({
+      globalCss: "global.css",
+      componentsUi: "components/ui",
+      lib: "lib",
+      theme: "default",
+    })
+  })
+})
+
+// ─── detectPackageManager ───────────────────────────────────────────────────
+
+describe("detectPackageManager", () => {
+  const originalAgent = process.env.npm_config_user_agent
+
+  afterEach(() => {
+    if (originalAgent !== undefined) {
+      process.env.npm_config_user_agent = originalAgent
+    } else {
+      delete process.env.npm_config_user_agent
+    }
+  })
+
+  it("returns npm by default", () => {
+    process.env.npm_config_user_agent = ""
+    const result = detectPackageManager()
+    expect(result.command).toBe("npm")
+    expect(result.baseArgs).toEqual(["install"])
+  })
+
+  it("detects yarn", () => {
+    process.env.npm_config_user_agent = "yarn/1.22.0 npm/? node/v18.0.0"
+    const result = detectPackageManager()
+    expect(result.command).toBe("yarn")
+    expect(result.baseArgs).toEqual(["add"])
+  })
+
+  it("detects pnpm", () => {
+    process.env.npm_config_user_agent = "pnpm/8.0.0 npm/? node/v18.0.0"
+    const result = detectPackageManager()
+    expect(result.command).toBe("pnpm")
+    expect(result.baseArgs).toEqual(["add"])
+  })
+
+  it("detects bun", () => {
+    process.env.npm_config_user_agent = "bun/1.0.0"
+    const result = detectPackageManager()
+    expect(result.command).toBe("bun")
+    expect(result.baseArgs).toEqual(["add"])
+  })
+})
+
+// ─── ensureDir ──────────────────────────────────────────────────────────────
+
+describe("ensureDir", () => {
+  let tmp
+
+  beforeEach(() => {
+    tmp = makeTmpDir()
+  })
+
+  afterEach(() => {
+    cleanTmpDir(tmp)
+  })
+
+  it("creates a directory that does not exist", () => {
+    const dir = path.join(tmp, "a", "b", "c")
+    expect(fs.existsSync(dir)).toBe(false)
+    ensureDir(dir)
+    expect(fs.existsSync(dir)).toBe(true)
+    expect(fs.statSync(dir).isDirectory()).toBe(true)
+  })
+
+  it("does nothing if directory already exists", () => {
+    const dir = path.join(tmp, "existing")
+    fs.mkdirSync(dir)
+    ensureDir(dir)
+    expect(fs.existsSync(dir)).toBe(true)
+  })
+})
+
+// ─── writeIfNotExists ───────────────────────────────────────────────────────
+
+describe("writeIfNotExists", () => {
+  let tmp
+
+  beforeEach(() => {
+    tmp = makeTmpDir()
+  })
+
+  afterEach(() => {
+    cleanTmpDir(tmp)
+  })
+
+  it("creates a file when it does not exist", () => {
+    const filePath = path.join(tmp, "test.txt")
+    const result = writeIfNotExists(filePath, "hello", "test.txt")
+    expect(result).toBe(true)
+    expect(fs.readFileSync(filePath, "utf8")).toBe("hello")
+  })
+
+  it("does not overwrite an existing file", () => {
+    const filePath = path.join(tmp, "test.txt")
+    fs.writeFileSync(filePath, "original")
+    const result = writeIfNotExists(filePath, "new content", "test.txt")
+    expect(result).toBe(false)
+    expect(fs.readFileSync(filePath, "utf8")).toBe("original")
+  })
+})
+
+// ─── getMissingDeps ─────────────────────────────────────────────────────────
+
+describe("getMissingDeps", () => {
+  let tmp
+
+  beforeEach(() => {
+    tmp = makeTmpDir()
+  })
+
+  afterEach(() => {
+    cleanTmpDir(tmp)
+  })
+
+  it("returns all deps when no package.json exists", () => {
+    const result = getMissingDeps(tmp, ["react", "react-native"])
+    expect(result).toEqual(["react", "react-native"])
+  })
+
+  it("returns only missing deps", () => {
+    const pkgJson = {
+      dependencies: { react: "^18.0.0", clsx: "^2.0.0" },
+      devDependencies: { typescript: "^5.0.0" },
+    }
+    fs.writeFileSync(path.join(tmp, "package.json"), JSON.stringify(pkgJson))
+
+    const result = getMissingDeps(tmp, ["react", "nativewind", "typescript", "tailwindcss"])
+    expect(result).toEqual(["nativewind", "tailwindcss"])
+  })
+
+  it("returns empty array when all deps are present", () => {
+    const pkgJson = {
+      dependencies: { react: "^18.0.0", nativewind: "^4.0.0" },
+    }
+    fs.writeFileSync(path.join(tmp, "package.json"), JSON.stringify(pkgJson))
+
+    const result = getMissingDeps(tmp, ["react", "nativewind"])
+    expect(result).toEqual([])
+  })
+
+  it("checks devDependencies too", () => {
+    const pkgJson = {
+      devDependencies: { vitest: "^1.0.0" },
+    }
+    fs.writeFileSync(path.join(tmp, "package.json"), JSON.stringify(pkgJson))
+
+    const result = getMissingDeps(tmp, ["vitest"])
+    expect(result).toEqual([])
+  })
+})
+
+// ─── loadConfig / writeConfig ───────────────────────────────────────────────
+
+describe("loadConfig / writeConfig", () => {
+  let tmp
+
+  beforeEach(() => {
+    tmp = makeTmpDir()
+  })
+
+  afterEach(() => {
+    cleanTmpDir(tmp)
+  })
+
+  it("returns null when no config file exists", () => {
+    expect(loadConfig(tmp)).toBeNull()
+  })
+
+  it("writes and reads config correctly", () => {
+    const config = {
+      globalCss: "app/global.css",
+      componentsUi: "app/components/ui",
+      lib: "app/lib",
+    }
+    writeConfig(tmp, config)
+
+    const configPath = path.join(tmp, CONFIG_FILENAME)
+    expect(fs.existsSync(configPath)).toBe(true)
+
+    const loaded = loadConfig(tmp)
+    expect(loaded).toEqual({ ...DEFAULT_CONFIG, ...config })
+  })
+
+  it("merges with DEFAULT_CONFIG on partial config", () => {
+    const partial = { globalCss: "custom/global.css" }
+    fs.writeFileSync(
+      path.join(tmp, CONFIG_FILENAME),
+      JSON.stringify(partial),
+      "utf8"
+    )
+
+    const loaded = loadConfig(tmp)
+    expect(loaded.globalCss).toBe("custom/global.css")
+    expect(loaded.componentsUi).toBe(DEFAULT_CONFIG.componentsUi)
+    expect(loaded.lib).toBe(DEFAULT_CONFIG.lib)
+  })
+
+  it("returns null for invalid JSON", () => {
+    fs.writeFileSync(path.join(tmp, CONFIG_FILENAME), "not json!", "utf8")
+    expect(loadConfig(tmp)).toBeNull()
+  })
+})
+
+// ─── getTailwindConfigContent ───────────────────────────────────────────────
+
+describe("getTailwindConfigContent", () => {
+  it("generates valid tailwind config with default paths", () => {
+    const content = getTailwindConfigContent(DEFAULT_CONFIG)
+    expect(content).toContain("module.exports")
+    expect(content).toContain('require("nativewind/preset")')
+    expect(content).toContain("hsl(var(--primary))")
+    expect(content).toContain("hsl(var(--background))")
+    expect(content).toContain("var(--radius)")
+    expect(content).toContain("./src/**/*.{js,jsx,ts,tsx}")
+  })
+
+  it("does not duplicate paths when componentsUi is under src/", () => {
+    const content = getTailwindConfigContent(DEFAULT_CONFIG)
+    const matches = content.match(/\.\/src\//g)
+    expect(matches).toHaveLength(1)
+    expect(content).not.toContain("./src/components/ui/**/*.{js,jsx,ts,tsx}")
+  })
+
+  it("includes custom componentsUi path when outside src/", () => {
+    const config = { ...DEFAULT_CONFIG, componentsUi: "app/ui" }
+    const content = getTailwindConfigContent(config)
+    expect(content).toContain("./app/ui/**/*.{js,jsx,ts,tsx}")
+    expect(content).toContain("./src/**/*.{js,jsx,ts,tsx}")
+  })
+})
+
+// ─── assertValidComponentConfig ─────────────────────────────────────────────
+
+describe("assertValidComponentConfig", () => {
+  it("accepts a valid config with files", () => {
+    expect(() =>
+      assertValidComponentConfig("button", {
+        files: ["src/components/ui/button.tsx"],
+      })
+    ).not.toThrow()
+  })
+
+  it("accepts a valid config with files and dependencies", () => {
+    expect(() =>
+      assertValidComponentConfig("button", {
+        files: ["src/components/ui/button.tsx"],
+        dependencies: ["class-variance-authority"],
+      })
+    ).not.toThrow()
+  })
+
+  it("throws for null config", () => {
+    expect(() => assertValidComponentConfig("button", null)).toThrow(
+      /invalid/i
+    )
+  })
+
+  it("throws for missing files array", () => {
+    expect(() =>
+      assertValidComponentConfig("button", { dependencies: [] })
+    ).toThrow(/files/i)
+  })
+
+  it("throws for empty string in files array", () => {
+    expect(() =>
+      assertValidComponentConfig("button", { files: ["valid.tsx", ""] })
+    ).toThrow(/files/i)
+  })
+
+  it("throws for non-string in files array", () => {
+    expect(() =>
+      assertValidComponentConfig("button", { files: [123] })
+    ).toThrow(/files/i)
+  })
+
+  it("throws for invalid dependencies", () => {
+    expect(() =>
+      assertValidComponentConfig("button", {
+        files: ["button.tsx"],
+        dependencies: [123],
+      })
+    ).toThrow(/dependencies/i)
+  })
+
+  it("throws for empty string in dependencies", () => {
+    expect(() =>
+      assertValidComponentConfig("button", {
+        files: ["button.tsx"],
+        dependencies: ["valid", ""],
+      })
+    ).toThrow(/dependencies/i)
+  })
+})
+
+// ─── GLOBAL_CSS_CONTENT ─────────────────────────────────────────────────────
+
+describe("GLOBAL_CSS_CONTENT", () => {
+  it("contains tailwind directives", () => {
+    expect(GLOBAL_CSS_CONTENT).toContain("@tailwind base")
+    expect(GLOBAL_CSS_CONTENT).toContain("@tailwind components")
+    expect(GLOBAL_CSS_CONTENT).toContain("@tailwind utilities")
+  })
+
+  it("contains light theme variables", () => {
+    expect(GLOBAL_CSS_CONTENT).toContain(":root")
+    expect(GLOBAL_CSS_CONTENT).toContain("--background:")
+    expect(GLOBAL_CSS_CONTENT).toContain("--primary:")
+    expect(GLOBAL_CSS_CONTENT).toContain("--radius:")
+  })
+
+  it("contains dark theme variables", () => {
+    expect(GLOBAL_CSS_CONTENT).toContain(".dark")
+  })
+})
+
+// ─── UTILS_CONTENT ──────────────────────────────────────────────────────────
+
+describe("UTILS_CONTENT", () => {
+  it("exports the cn function", () => {
+    expect(UTILS_CONTENT).toContain("export function cn")
+    expect(UTILS_CONTENT).toContain("twMerge")
+    expect(UTILS_CONTENT).toContain("clsx")
+  })
+})
+
+// ─── formatError ────────────────────────────────────────────────────────────
+
+describe("formatError", () => {
+  it("extracts message from Error objects", () => {
+    expect(formatError(new Error("test error"))).toBe("test error")
+  })
+
+  it("converts non-Error values to string", () => {
+    expect(formatError("string error")).toBe("string error")
+    expect(formatError(42)).toBe("42")
+    expect(formatError(null)).toBe("null")
+  })
+})
+
+// ─── getCliVersion ──────────────────────────────────────────────────────────
+
+describe("getCliVersion", () => {
+  it("returns a valid semver-like version string", () => {
+    const version = getCliVersion()
+    expect(version).not.toBe("unknown")
+    expect(version).toMatch(/^\d+\.\d+\.\d+/)
+  })
+})
+
+// ─── getInstallHint ─────────────────────────────────────────────────────────
+
+describe("getInstallHint", () => {
+  const originalAgent = process.env.npm_config_user_agent
+
+  afterEach(() => {
+    if (originalAgent !== undefined) {
+      process.env.npm_config_user_agent = originalAgent
+    } else {
+      delete process.env.npm_config_user_agent
+    }
+  })
+
+  it("generates npm install hint", () => {
+    process.env.npm_config_user_agent = ""
+    const hint = getInstallHint(["react", "react-native"])
+    expect(hint).toBe("npm install react react-native")
+  })
+
+  it("generates yarn add hint", () => {
+    process.env.npm_config_user_agent = "yarn/1.22.0"
+    const hint = getInstallHint(["react"])
+    expect(hint).toBe("yarn add react")
+  })
+
+  it("generates pnpm add hint", () => {
+    process.env.npm_config_user_agent = "pnpm/8.0.0"
+    const hint = getInstallHint(["clsx", "tailwind-merge"])
+    expect(hint).toBe("pnpm add clsx tailwind-merge")
+  })
+
+  it("generates bun add hint", () => {
+    process.env.npm_config_user_agent = "bun/1.0.0"
+    const hint = getInstallHint(["react"])
+    expect(hint).toBe("bun add react")
+  })
+})
+
+// ─── fetchWithTimeout ───────────────────────────────────────────────────────
+
+describe("fetchWithTimeout", () => {
+  let originalFetch
+
+  beforeEach(() => {
+    originalFetch = globalThis.fetch
+  })
+
+  afterEach(() => {
+    globalThis.fetch = originalFetch
+  })
+
+  it("returns response on successful fetch", async () => {
+    globalThis.fetch = vi.fn(async () => ({
+      ok: true,
+      json: async () => ({ data: "test" }),
+    }))
+
+    const response = await fetchWithTimeout("https://example.com/data.json")
+    expect(response.ok).toBe(true)
+    expect(await response.json()).toEqual({ data: "test" })
+  })
+
+  it("passes the abort signal to fetch", async () => {
+    globalThis.fetch = vi.fn(async (url, options) => {
+      expect(options).toHaveProperty("signal")
+      expect(options.signal).toBeInstanceOf(AbortSignal)
+      return { ok: true }
+    })
+
+    await fetchWithTimeout("https://example.com/data.json")
+    expect(globalThis.fetch).toHaveBeenCalledTimes(1)
+  })
+
+  it("throws descriptive error on abort/timeout", async () => {
+    globalThis.fetch = vi.fn(async (url, options) => {
+      const err = new Error("The operation was aborted")
+      err.name = "AbortError"
+      throw err
+    })
+
+    await expect(fetchWithTimeout("https://example.com/slow")).rejects.toThrow(
+      /timed out/i
+    )
+  })
+
+  it("re-throws non-abort errors as-is", async () => {
+    globalThis.fetch = vi.fn(async () => {
+      throw new TypeError("fetch failed")
+    })
+
+    await expect(
+      fetchWithTimeout("https://example.com/fail")
+    ).rejects.toThrow("fetch failed")
+  })
+})
+
+// ─── Exports smoke test ─────────────────────────────────────────────────────
+
+describe("exports", () => {
+  it("exports all expected constants", () => {
+    expect(typeof DEFAULT_CONFIG).toBe("object")
+    expect(typeof CONFIG_FILENAME).toBe("string")
+    expect(typeof GLOBAL_CSS_CONTENT).toBe("string")
+    expect(typeof UTILS_CONTENT).toBe("string")
+  })
+
+  it("exports all expected functions", () => {
+    expect(typeof getTailwindConfigContent).toBe("function")
+    expect(typeof detectPackageManager).toBe("function")
+    expect(typeof ensureDir).toBe("function")
+    expect(typeof writeIfNotExists).toBe("function")
+    expect(typeof getMissingDeps).toBe("function")
+    expect(typeof loadConfig).toBe("function")
+    expect(typeof writeConfig).toBe("function")
+    expect(typeof assertValidComponentConfig).toBe("function")
+    expect(typeof fetchWithTimeout).toBe("function")
+    expect(typeof formatError).toBe("function")
+    expect(typeof getCliVersion).toBe("function")
+    expect(typeof getInstallHint).toBe("function")
+    expect(typeof init).toBe("function")
+    expect(typeof add).toBe("function")
+  })
+})