safeword 0.2.4 → 0.2.6

package/packages/cli/tests/helpers.ts

@@ -1,243 +0,0 @@
-import { execSync, exec } from 'node:child_process';
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync, readFileSync, existsSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join, dirname } from 'node:path';
-import { promisify } from 'node:util';
-import { fileURLToPath } from 'node:url';
-
-const execAsync = promisify(exec);
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
-/**
- * Path to the CLI entry point (built)
- */
-export const CLI_PATH = join(__dirname, '../dist/cli.js');
-
-/**
- * Path to the CLI source (for ts-node execution during development)
- */
-export const CLI_SRC_PATH = join(__dirname, '../src/cli.ts');
-
-/**
- * Creates a temporary directory for test isolation
- */
-export function createTempDir(): string {
-  return mkdtempSync(join(tmpdir(), 'safeword-test-'));
-}
-
-/**
- * Removes a temporary directory and all contents
- */
-export function removeTempDir(dir: string): void {
-  rmSync(dir, { recursive: true, force: true });
-}
-
-/**
- * Creates a minimal package.json in the given directory
- */
-export function createPackageJson(dir: string, overrides: Record<string, unknown> = {}): void {
-  const pkg = {
-    name: 'test-project',
-    version: '1.0.0',
-    ...overrides,
-  };
-  writeFileSync(join(dir, 'package.json'), JSON.stringify(pkg, null, 2));
-}
-
-/**
- * Creates a TypeScript package.json (with typescript in devDependencies)
- */
-export function createTypeScriptPackageJson(
-  dir: string,
-  overrides: Record<string, unknown> = {},
-): void {
-  createPackageJson(dir, {
-    devDependencies: {
-      typescript: '^5.0.0',
-    },
-    ...overrides,
-  });
-}
-
-/**
- * Creates a React package.json
- */
-export function createReactPackageJson(dir: string, overrides: Record<string, unknown> = {}): void {
-  createPackageJson(dir, {
-    dependencies: {
-      react: '^18.0.0',
-      'react-dom': '^18.0.0',
-    },
-    ...overrides,
-  });
-}
-
-/**
- * Creates a Next.js package.json
- */
-export function createNextJsPackageJson(
-  dir: string,
-  overrides: Record<string, unknown> = {},
-): void {
-  createPackageJson(dir, {
-    dependencies: {
-      next: '^14.0.0',
-      react: '^18.0.0',
-      'react-dom': '^18.0.0',
-    },
-    ...overrides,
-  });
-}
-
-/**
- * Result from running the CLI
- */
-export interface CliResult {
-  stdout: string;
-  stderr: string;
-  exitCode: number;
-}
-
-/**
- * Runs the CLI with the given arguments in the specified directory
- * Uses built CLI (dist/cli.js)
- */
-export async function runCli(
-  args: string[],
-  options: {
-    cwd?: string;
-    input?: string;
-    env?: Record<string, string>;
-    timeout?: number;
-  } = {},
-): Promise<CliResult> {
-  const { cwd = process.cwd(), input, env = {}, timeout = 30000 } = options;
-
-  const command = `node ${CLI_PATH} ${args.join(' ')}`;
-
-  try {
-    const { stdout, stderr } = await execAsync(command, {
-      cwd,
-      env: { ...process.env, ...env },
-      timeout,
-      ...(input ? { input } : {}),
-    });
-    return { stdout, stderr, exitCode: 0 };
-  } catch (error: unknown) {
-    const execError = error as {
-      stdout?: string;
-      stderr?: string;
-      code?: number;
-      status?: number;
-    };
-    return {
-      stdout: execError.stdout ?? '',
-      stderr: execError.stderr ?? '',
-      exitCode: execError.code ?? execError.status ?? 1,
-    };
-  }
-}
-
-/**
- * Runs the CLI synchronously (for simple tests)
- */
-export function runCliSync(
-  args: string[],
-  options: {
-    cwd?: string;
-    env?: Record<string, string>;
-    timeout?: number;
-  } = {},
-): CliResult {
-  const { cwd = process.cwd(), env = {}, timeout = 30000 } = options;
-
-  const command = `node ${CLI_PATH} ${args.join(' ')}`;
-
-  try {
-    const stdout = execSync(command, {
-      cwd,
-      env: { ...process.env, ...env },
-      timeout,
-      encoding: 'utf-8',
-      stdio: ['pipe', 'pipe', 'pipe'],
-    });
-    return { stdout, stderr: '', exitCode: 0 };
-  } catch (error: unknown) {
-    const execError = error as {
-      stdout?: string | Buffer;
-      stderr?: string | Buffer;
-      status?: number;
-    };
-    return {
-      stdout: execError.stdout?.toString() ?? '',
-      stderr: execError.stderr?.toString() ?? '',
-      exitCode: execError.status ?? 1,
-    };
-  }
-}
-
-/**
- * Reads a file from the test directory
- */
-export function readTestFile(dir: string, relativePath: string): string {
-  return readFileSync(join(dir, relativePath), 'utf-8');
-}
-
-/**
- * Writes a file to the test directory
- */
-export function writeTestFile(dir: string, relativePath: string, content: string): void {
-  const fullPath = join(dir, relativePath);
-  const parentDir = join(fullPath, '..');
-  if (!existsSync(parentDir)) {
-    mkdirSync(parentDir, { recursive: true });
-  }
-  writeFileSync(fullPath, content);
-}
-
-/**
- * Checks if a file exists in the test directory
- */
-export function fileExists(dir: string, relativePath: string): boolean {
-  return existsSync(join(dir, relativePath));
-}
-
-/**
- * Initializes a git repository in the given directory
- */
-export function initGitRepo(dir: string): void {
-  execSync('git init', { cwd: dir, stdio: 'pipe' });
-  execSync('git config user.email "test@test.com"', { cwd: dir, stdio: 'pipe' });
-  execSync('git config user.name "Test User"', { cwd: dir, stdio: 'pipe' });
-}
-
-/**
- * Creates a configured project (runs setup) for tests that need pre-configured state
- */
-export async function createConfiguredProject(dir: string): Promise<void> {
-  createTypeScriptPackageJson(dir);
-  initGitRepo(dir);
-  await runCli(['setup', '--yes'], { cwd: dir });
-}
-
-/**
- * Measures execution time of a function in milliseconds
- */
-export async function measureTime<T>(fn: () => Promise<T>): Promise<{ result: T; timeMs: number }> {
-  const start = performance.now();
-  const result = await fn();
-  const timeMs = performance.now() - start;
-  return { result, timeMs };
-}
-
-/**
- * Measures execution time of a sync function in milliseconds
- */
-export function measureTimeSync<T>(fn: () => T): { result: T; timeMs: number } {
-  const start = performance.now();
-  const result = fn();
-  const timeMs = performance.now() - start;
-  return { result, timeMs };
-}

package/packages/cli/tests/npm-package.test.ts

@@ -1,83 +0,0 @@
-/**
- * Test Suite: NPM Package Distribution
- *
- * Tests that the npm package is correctly structured and would work
- * when installed via `npm install` or `npx`.
- */
-
-import { describe, it, expect } from 'vitest';
-import { existsSync, readdirSync, readFileSync } from 'node:fs';
-import { join, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const cliRoot = join(__dirname, '..');
-
-describe('NPM Package Structure', () => {
-  it('should have package.json with correct files array', () => {
-    const packageJson = JSON.parse(readFileSync(join(cliRoot, 'package.json'), 'utf-8'));
-
-    expect(packageJson.files).toBeDefined();
-    expect(packageJson.files).toContain('dist');
-    expect(packageJson.files).toContain('templates');
-  });
-
-  it('should have dist directory with CLI entry point', () => {
-    const distPath = join(cliRoot, 'dist');
-    expect(existsSync(distPath)).toBe(true);
-    expect(existsSync(join(distPath, 'cli.js'))).toBe(true);
-  });
-
-  it('should have templates directory with all required subdirectories', () => {
-    const templatesPath = join(cliRoot, 'templates');
-    expect(existsSync(templatesPath)).toBe(true);
-
-    const required = ['SAFEWORD.md', 'guides', 'doc-templates', 'hooks', 'prompts', 'skills', 'commands', 'lib'];
-    for (const item of required) {
-      expect(existsSync(join(templatesPath, item))).toBe(true);
-    }
-  });
-
-  it('should have templates/hooks with all hook scripts', () => {
-    const hooksPath = join(cliRoot, 'templates', 'hooks');
-    const files = readdirSync(hooksPath);
-
-    expect(files).toContain('agents-md-check.sh');
-    expect(files).toContain('pre-commit.sh');
-    expect(files).toContain('post-tool.sh');
-    expect(files).toContain('inject-timestamp.sh');
-  });
-
-  it('should have templates/guides with methodology files', () => {
-    const guidesPath = join(cliRoot, 'templates', 'guides');
-    const files = readdirSync(guidesPath);
-
-    // Should have multiple guide files
-    const mdFiles = files.filter((f) => f.endsWith('.md'));
-    expect(mdFiles.length).toBeGreaterThan(5);
-  });
-
-  it('should have templates/skills with quality reviewer', () => {
-    const skillPath = join(cliRoot, 'templates', 'skills', 'safeword-quality-reviewer');
-    expect(existsSync(skillPath)).toBe(true);
-    expect(existsSync(join(skillPath, 'SKILL.md'))).toBe(true);
-  });
-
-  it('should have templates/commands with slash commands', () => {
-    const commandsPath = join(cliRoot, 'templates', 'commands');
-    const files = readdirSync(commandsPath);
-
-    expect(files).toContain('quality-review.md');
-    expect(files).toContain('arch-review.md');
-    expect(files).toContain('lint.md');
-  });
-
-  it('should resolve templates from dist context', () => {
-    // Simulate the path resolution that getTemplatesDir() does
-    const distDir = join(cliRoot, 'dist');
-    const templatesFromDist = join(distDir, '..', 'templates');
-
-    expect(existsSync(templatesFromDist)).toBe(true);
-    expect(existsSync(join(templatesFromDist, 'SAFEWORD.md'))).toBe(true);
-  });
-});

package/packages/cli/tests/technical-constraints.test.ts

@@ -1,96 +0,0 @@
-/**
- * Test Suite 0: Technical Constraints
- *
- * Tests for non-functional requirements that apply across all commands.
- * These tests verify performance, compatibility, and quality requirements.
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import {
-  createTempDir,
-  removeTempDir,
-  createTypeScriptPackageJson,
-  runCli,
-  runCliSync,
-  measureTime,
-  initGitRepo,
-} from './helpers';
-
-describe('Test Suite 0: Technical Constraints', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir();
-  });
-
-  afterEach(() => {
-    removeTempDir(tempDir);
-  });
-
-  describe('Test 0.1: CLI startup time under 500ms', () => {
-    it('should start quickly with average under 500ms', async () => {
-      const runs = 10;
-      const times: number[] = [];
-
-      for (let i = 0; i < runs; i++) {
-        const { timeMs } = await measureTime(async () => {
-          return runCliSync(['--version']);
-        });
-        times.push(timeMs);
-      }
-
-      const averageTime = times.reduce((a, b) => a + b, 0) / times.length;
-      const maxTime = Math.max(...times);
-
-      expect(averageTime).toBeLessThan(500);
-      expect(maxTime).toBeLessThan(750);
-    });
-  });
-
-  describe('Test 0.2: Setup completes under 30s', () => {
-    it('should complete setup in under 30 seconds', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      const { result, timeMs } = await measureTime(async () => {
-        return runCli(['setup', '--yes'], { cwd: tempDir, timeout: 60000 });
-      });
-
-      expect(result.exitCode).toBe(0);
-      expect(timeMs).toBeLessThan(30000);
-    });
-  });
-
-  describe('Test 0.3: Node.js version check', () => {
-    it.skip('should exit with error on Node.js < 18 (requires CI container)', async () => {
-      // This test requires running with Node.js < 18, which needs:
-      // - A Docker container with older Node, OR
-      // - nvm switching in CI
-      //
-      // The implementation should check process.version at startup
-      // and exit with code 1 if version < 18.
-      //
-      // Expected behavior:
-      // - Exit code 1
-      // - stderr contains "Node.js version" and "18"
-      expect(true).toBe(true); // Placeholder
-    });
-  });
-
-  describe('Test 0.4: Works with different package managers', () => {
-    it('should work with npm', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      const result = await runCli(['setup', '--yes'], {
-        cwd: tempDir,
-        timeout: 60000,
-      });
-
-      expect(result.exitCode).toBe(0);
-    });
-
-    // Note: pnpm and yarn tests should be in separate CI jobs
-    // to ensure proper package manager isolation
-  });
-});

package/packages/cli/tsconfig.json

@@ -1,25 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "lib": ["ES2022"],
-    "outDir": "./dist",
-    "rootDir": "./src",
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "resolveJsonModule": true,
-    "isolatedModules": true,
-    "noUnusedLocals": true,
-    "noUnusedParameters": true,
-    "noImplicitReturns": true,
-    "noFallthroughCasesInSwitch": true
-  },
-  "include": ["src/**/*"],
-  "exclude": ["node_modules", "dist", "tests"]
-}

package/packages/cli/tsup.config.ts

@@ -1,11 +0,0 @@
-import { defineConfig } from 'tsup';
-
-export default defineConfig({
-  entry: ['src/cli.ts', 'src/index.ts'],
-  format: ['esm'],
-  dts: true,
-  sourcemap: true,
-  clean: true,
-  target: 'node18',
-  shims: false,
-});

package/packages/cli/vitest.config.ts

@@ -1,23 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-export default defineConfig({
-  test: {
-    globals: true,
-    environment: 'node',
-    include: ['tests/**/*.test.ts', 'src/**/*.test.ts'],
-    coverage: {
-      provider: 'v8',
-      include: ['src/**/*.ts'],
-      exclude: ['src/**/*.test.ts', 'src/index.ts'],
-    },
-    // Increase timeout for integration tests that spawn processes
-    testTimeout: 30000,
-    // Run tests sequentially to avoid temp directory conflicts
-    pool: 'forks',
-    poolOptions: {
-      forks: {
-        singleFork: true,
-      },
-    },
-  },
-});