safeword 0.2.4 → 0.2.6

package/packages/cli/tests/commands/reset.test.ts

@@ -1,310 +0,0 @@
-/**
- * Test Suite 11: Reset
- *
- * Tests for `safeword reset` command.
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import {
-  createTempDir,
-  removeTempDir,
-  createTypeScriptPackageJson,
-  createConfiguredProject,
-  runCli,
-  readTestFile,
-  writeTestFile,
-  fileExists,
-  initGitRepo,
-} from '../helpers';
-
-describe('Test Suite 11: Reset', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir();
-  });
-
-  afterEach(() => {
-    removeTempDir(tempDir);
-  });
-
-  describe('Test 11.1: Prompts for confirmation', () => {
-    // In TTY mode, should prompt. We test the prompt message content.
-    it('should mention removing safeword', async () => {
-      await createConfiguredProject(tempDir);
-
-      // With --yes, it shouldn't prompt but we can check the output
-      const result = await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // Should mention what it's removing
-      const output = result.stdout + result.stderr;
-      expect(output.toLowerCase()).toMatch(/remov|reset|safeword/i);
-    });
-  });
-
-  describe('Test 11.2: --yes auto-confirms', () => {
-    it('should skip confirmation and remove .safeword', async () => {
-      await createConfiguredProject(tempDir);
-
-      expect(fileExists(tempDir, '.safeword')).toBe(true);
-
-      const result = await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(0);
-      expect(fileExists(tempDir, '.safeword')).toBe(false);
-    });
-  });
-
-  describe('Test 11.3: No TTY auto-confirms', () => {
-    it('should auto-confirm in non-TTY mode', async () => {
-      await createConfiguredProject(tempDir);
-
-      const result = await runCli(['reset'], {
-        cwd: tempDir,
-        env: { CI: 'true' },
-      });
-
-      // Should complete without hanging
-      expect(result.exitCode).toBeDefined();
-    });
-  });
-
-  describe('Test 11.4: Removes .safeword directory', () => {
-    it('should remove .safeword/ completely', async () => {
-      await createConfiguredProject(tempDir);
-
-      expect(fileExists(tempDir, '.safeword')).toBe(true);
-      expect(fileExists(tempDir, '.safeword/SAFEWORD.md')).toBe(true);
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      expect(fileExists(tempDir, '.safeword')).toBe(false);
-    });
-  });
-
-  describe('Test 11.5: Removes hooks from settings.json', () => {
-    it('should remove safeword hooks but preserve custom hooks', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Add a custom hook
-      const settings = JSON.parse(readTestFile(tempDir, '.claude/settings.json'));
-      settings.hooks.SessionStart = settings.hooks.SessionStart || [];
-      settings.hooks.SessionStart.push({
-        command: 'echo "Custom preserved hook"',
-        description: 'Custom',
-      });
-      writeTestFile(tempDir, '.claude/settings.json', JSON.stringify(settings, null, 2));
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      expect(fileExists(tempDir, '.claude/settings.json')).toBe(true);
-
-      const updatedSettings = JSON.parse(readTestFile(tempDir, '.claude/settings.json'));
-
-      // Custom hook preserved
-      const hasCustom = updatedSettings.hooks?.SessionStart?.some(
-        (h: { command?: string }) => h.command === 'echo "Custom preserved hook"',
-      );
-      expect(hasCustom).toBe(true);
-
-      // Safeword hooks removed (no references to .safeword)
-      const settingsStr = JSON.stringify(updatedSettings);
-      expect(settingsStr).not.toContain('.safeword/hooks');
-    });
-  });
-
-  describe('Test 11.6: Removes safeword skills', () => {
-    it('should remove safeword-* skill directories', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Verify skills exist after setup
-      const skillsExist = fileExists(tempDir, '.claude/skills');
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // After reset, safeword skills should be gone
-      // but .claude/skills directory may remain if empty or have other skills
-      if (skillsExist) {
-        // Check no safeword-* directories remain
-        // This would require listing directory contents
-        // For now, verify the command completed
-      }
-    });
-  });
-
-  describe('Test 11.7: Removes git hook markers', () => {
-    it('should remove safeword markers from pre-commit', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      // Create custom pre-commit content
-      writeTestFile(
-        tempDir,
-        '.git/hooks/pre-commit',
-        `#!/bin/bash
-# Custom content before
-echo "Before safeword"
-
-# SAFEWORD_ARCH_CHECK_START
-echo "Safeword check"
-# SAFEWORD_ARCH_CHECK_END
-
-# Custom content after
-echo "After safeword"
-`,
-      );
-
-      await runCli(['setup', '--yes'], { cwd: tempDir });
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      const content = readTestFile(tempDir, '.git/hooks/pre-commit');
-
-      // Markers and content between them removed
-      expect(content).not.toContain('SAFEWORD_ARCH_CHECK_START');
-      expect(content).not.toContain('SAFEWORD_ARCH_CHECK_END');
-      expect(content).not.toContain('Safeword check');
-
-      // Custom content preserved
-      expect(content).toContain('Before safeword');
-      expect(content).toContain('After safeword');
-    });
-  });
-
-  describe('Test 11.8: Removes link from AGENTS.md', () => {
-    it('should remove safeword link but preserve other content', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Add custom content to AGENTS.md
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      writeTestFile(tempDir, 'AGENTS.md', content + '\n## My Custom Section\n\nCustom content.\n');
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      expect(fileExists(tempDir, 'AGENTS.md')).toBe(true);
-
-      const updatedContent = readTestFile(tempDir, 'AGENTS.md');
-
-      // Link removed
-      expect(updatedContent).not.toContain('@./.safeword/SAFEWORD.md');
-
-      // Custom content preserved
-      expect(updatedContent).toContain('My Custom Section');
-      expect(updatedContent).toContain('Custom content');
-    });
-  });
-
-  describe('Test 11.9: Preserves linting config', () => {
-    it('should keep eslint and prettier config', async () => {
-      await createConfiguredProject(tempDir);
-
-      expect(fileExists(tempDir, 'eslint.config.mjs')).toBe(true);
-      expect(fileExists(tempDir, '.prettierrc')).toBe(true);
-
-      const pkgBefore = JSON.parse(readTestFile(tempDir, 'package.json'));
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // Linting files preserved
-      expect(fileExists(tempDir, 'eslint.config.mjs')).toBe(true);
-      expect(fileExists(tempDir, '.prettierrc')).toBe(true);
-
-      // Scripts preserved
-      const pkgAfter = JSON.parse(readTestFile(tempDir, 'package.json'));
-      expect(pkgAfter.scripts?.lint).toBe(pkgBefore.scripts?.lint);
-      expect(pkgAfter.scripts?.format).toBe(pkgBefore.scripts?.format);
-    });
-  });
-
-  describe('Test 11.10: Unconfigured project message', () => {
-    it('should show nothing to remove message', async () => {
-      createTypeScriptPackageJson(tempDir);
-      // No setup
-
-      const result = await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(0);
-      expect(result.stdout.toLowerCase()).toMatch(/nothing|already|not configured/i);
-    });
-  });
-
-  describe('Test 11.11: Removes safeword slash commands', () => {
-    it('should remove safeword commands but preserve custom ones', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Verify commands directory exists after setup
-      expect(fileExists(tempDir, '.claude/commands')).toBe(true);
-
-      // Add a custom command that should be preserved
-      writeTestFile(
-        tempDir,
-        '.claude/commands/my-custom-command.md',
-        '# My Custom Command\n\nDo something custom.',
-      );
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // Custom command should be preserved
-      expect(fileExists(tempDir, '.claude/commands/my-custom-command.md')).toBe(true);
-      const customContent = readTestFile(tempDir, '.claude/commands/my-custom-command.md');
-      expect(customContent).toContain('My Custom Command');
-
-      // Safeword commands should be removed (quality-review, arch-review, lint)
-      expect(fileExists(tempDir, '.claude/commands/quality-review.md')).toBe(false);
-    });
-  });
-
-  describe('Test 11.12: Removes MCP servers from .mcp.json', () => {
-    it('should remove context7 and playwright servers', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Verify MCP config exists
-      expect(fileExists(tempDir, '.mcp.json')).toBe(true);
-
-      // Add a custom MCP server
-      const mcpConfig = JSON.parse(readTestFile(tempDir, '.mcp.json'));
-      mcpConfig.mcpServers['my-custom-server'] = {
-        command: 'my-server',
-        args: ['--port', '3000'],
-      };
-      writeTestFile(tempDir, '.mcp.json', JSON.stringify(mcpConfig, null, 2));
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // .mcp.json should still exist with custom server
-      expect(fileExists(tempDir, '.mcp.json')).toBe(true);
-
-      const updatedConfig = JSON.parse(readTestFile(tempDir, '.mcp.json'));
-
-      // Custom server preserved
-      expect(updatedConfig.mcpServers['my-custom-server']).toBeDefined();
-      expect(updatedConfig.mcpServers['my-custom-server'].command).toBe('my-server');
-
-      // Safeword servers removed
-      expect(updatedConfig.mcpServers.context7).toBeUndefined();
-      expect(updatedConfig.mcpServers.playwright).toBeUndefined();
-    });
-
-    it('should delete or empty .mcp.json if only safeword servers remain', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Overwrite with only safeword servers
-      const mcpConfig = {
-        mcpServers: {
-          context7: { command: 'npx', args: ['@context7/mcp'] },
-          playwright: { command: 'npx', args: ['@playwright/mcp'] },
-        },
-      };
-      writeTestFile(tempDir, '.mcp.json', JSON.stringify(mcpConfig, null, 2));
-
-      await runCli(['reset', '--yes'], { cwd: tempDir });
-
-      // Either .mcp.json is deleted OR mcpServers is empty
-      if (fileExists(tempDir, '.mcp.json')) {
-        const updatedConfig = JSON.parse(readTestFile(tempDir, '.mcp.json'));
-        const serverCount = Object.keys(updatedConfig.mcpServers || {}).length;
-        expect(serverCount).toBe(0);
-      }
-      // If file doesn't exist, test passes implicitly
-    });
-  });
-});

package/packages/cli/tests/commands/self-healing.test.ts

@@ -1,170 +0,0 @@
-/**
- * Test Suite 12: AGENTS.md Self-Healing
- *
- * Tests for SessionStart hook that maintains AGENTS.md link.
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { execSync } from 'node:child_process';
-import { join } from 'node:path';
-import {
-  createTempDir,
-  removeTempDir,
-  createConfiguredProject,
-  runCli,
-  readTestFile,
-  writeTestFile,
-  fileExists,
-  measureTimeSync,
-} from '../helpers';
-import { unlinkSync } from 'node:fs';
-
-describe('Test Suite 12: AGENTS.md Self-Healing', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir();
-  });
-
-  afterEach(() => {
-    removeTempDir(tempDir);
-  });
-
-  /**
-   * Helper to run the self-healing hook script
-   */
-  function runSelfHealingHook(dir: string): { stdout: string; exitCode: number } {
-    // The hook script location depends on implementation
-    // Typically: .safeword/hooks/agents-md-check.sh
-    const hookPath = join(dir, '.safeword/hooks/agents-md-check.sh');
-
-    if (!fileExists(dir, '.safeword/hooks/agents-md-check.sh')) {
-      // Hook may have different name - check for any agents-related hook
-      // For now, return a placeholder
-      return { stdout: '', exitCode: 0 };
-    }
-
-    try {
-      const stdout = execSync(`bash "${hookPath}"`, {
-        cwd: dir,
-        encoding: 'utf-8',
-        stdio: ['pipe', 'pipe', 'pipe'],
-      });
-      return { stdout, exitCode: 0 };
-    } catch (error: unknown) {
-      const execError = error as { stdout?: string; status?: number };
-      return {
-        stdout: execError.stdout ?? '',
-        exitCode: execError.status ?? 1,
-      };
-    }
-  }
-
-  describe('Test 12.1: Hook detects missing link', () => {
-    it('should detect when safeword link is missing', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Remove the safeword link from AGENTS.md
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      const withoutLink = content
-        .split('\n')
-        .filter(line => !line.includes('@./.safeword/SAFEWORD.md'))
-        .join('\n');
-      writeTestFile(tempDir, 'AGENTS.md', withoutLink);
-
-      const result = runSelfHealingHook(tempDir);
-
-      // Hook should indicate repair is needed or perform repair
-      // The exact output depends on implementation
-      expect(result.exitCode).toBeDefined();
-    });
-  });
-
-  describe('Test 12.2: Hook re-adds missing link', () => {
-    it('should restore removed link', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Remove the safeword link
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      const withoutLink = '# My Project\n\nSome content without the link.\n';
-      writeTestFile(tempDir, 'AGENTS.md', withoutLink);
-
-      // Run hook
-      runSelfHealingHook(tempDir);
-
-      // Link should be restored
-      const updatedContent = readTestFile(tempDir, 'AGENTS.md');
-      expect(updatedContent).toContain('@./.safeword/SAFEWORD.md');
-
-      // Original content preserved
-      expect(updatedContent).toContain('My Project');
-      expect(updatedContent).toContain('Some content');
-    });
-  });
-
-  describe('Test 12.3: Hook shows warning on restoration', () => {
-    it('should output message when restoring', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Remove link
-      writeTestFile(tempDir, 'AGENTS.md', '# No link\n');
-
-      const result = runSelfHealingHook(tempDir);
-
-      // Should mention restoration
-      const output = result.stdout.toLowerCase();
-      expect(output).toMatch(/restor|repair|add|fix|agents/i);
-    });
-  });
-
-  describe('Test 12.4: Hook recreates deleted AGENTS.md', () => {
-    it('should recreate AGENTS.md if deleted', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Delete AGENTS.md entirely
-      unlinkSync(join(tempDir, 'AGENTS.md'));
-      expect(fileExists(tempDir, 'AGENTS.md')).toBe(false);
-
-      // Run hook
-      runSelfHealingHook(tempDir);
-
-      // AGENTS.md should be recreated
-      expect(fileExists(tempDir, 'AGENTS.md')).toBe(true);
-
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      expect(content).toContain('@./.safeword/SAFEWORD.md');
-    });
-  });
-
-  describe('Test 12.5: Hook prevents duplicates', () => {
-    it('should not add duplicate links', async () => {
-      await createConfiguredProject(tempDir);
-
-      // Verify link exists
-      const contentBefore = readTestFile(tempDir, 'AGENTS.md');
-      expect(contentBefore).toContain('@./.safeword/SAFEWORD.md');
-
-      // Run hook multiple times
-      runSelfHealingHook(tempDir);
-      runSelfHealingHook(tempDir);
-      runSelfHealingHook(tempDir);
-
-      // Count links
-      const contentAfter = readTestFile(tempDir, 'AGENTS.md');
-      const linkCount = (contentAfter.match(/@\.\/\.safeword\/SAFEWORD\.md/g) || []).length;
-
-      expect(linkCount).toBe(1);
-    });
-  });
-
-  describe('Test 12.6: Hook exits cleanly', () => {
-    it('should exit with code 0 and complete quickly', async () => {
-      await createConfiguredProject(tempDir);
-
-      const { result, timeMs } = measureTimeSync(() => runSelfHealingHook(tempDir));
-
-      expect(result.exitCode).toBe(0);
-      expect(timeMs).toBeLessThan(1000); // Should complete in under 1 second
-    });
-  });
-});

package/packages/cli/tests/commands/setup-blocking.test.ts

@@ -1,71 +0,0 @@
-/**
- * Test Suite 5: Setup Blocks on Existing
- *
- * Tests for setup error when already configured.
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { mkdirSync } from 'node:fs';
-import { join } from 'node:path';
-import {
-  createTempDir,
-  removeTempDir,
-  createTypeScriptPackageJson,
-  runCli,
-  readTestFile,
-  writeTestFile,
-  fileExists,
-} from '../helpers';
-
-describe('Test Suite 5: Setup Blocks on Existing', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir();
-  });
-
-  afterEach(() => {
-    removeTempDir(tempDir);
-  });
-
-  describe('Test 5.1: Error when .safeword exists', () => {
-    it('should error with exit 1 when .safeword/ already exists', async () => {
-      createTypeScriptPackageJson(tempDir);
-
-      // Create existing .safeword directory
-      mkdirSync(join(tempDir, '.safeword'));
-
-      const result = await runCli(['setup'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(1);
-      expect(result.stderr.toLowerCase()).toContain('already configured');
-      expect(result.stderr.toLowerCase()).toContain('upgrade');
-    });
-  });
-
-  describe('Test 5.2: No files modified on error', () => {
-    it('should not modify files when erroring', async () => {
-      createTypeScriptPackageJson(tempDir);
-
-      // Create existing .safeword directory
-      mkdirSync(join(tempDir, '.safeword'));
-
-      // Create AGENTS.md with known content
-      const originalContent = '# Original AGENTS.md\n\nThis should not change.\n';
-      writeTestFile(tempDir, 'AGENTS.md', originalContent);
-
-      const result = await runCli(['setup'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(1);
-
-      // AGENTS.md should be unchanged
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      expect(content).toBe(originalContent);
-
-      // No new files should be created
-      expect(fileExists(tempDir, '.claude')).toBe(false);
-      expect(fileExists(tempDir, 'eslint.config.mjs')).toBe(false);
-      expect(fileExists(tempDir, '.prettierrc')).toBe(false);
-    });
-  });
-});

package/packages/cli/tests/commands/setup-core.test.ts

@@ -1,135 +0,0 @@
-/**
- * Test Suite 2: Setup - Core Files
- *
- * Tests for .safeword/ directory creation and AGENTS.md handling.
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import {
-  createTempDir,
-  removeTempDir,
-  createTypeScriptPackageJson,
-  runCli,
-  readTestFile,
-  writeTestFile,
-  fileExists,
-  initGitRepo,
-  createConfiguredProject,
-} from '../helpers';
-
-describe('Test Suite 2: Setup - Core Files', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = createTempDir();
-  });
-
-  afterEach(() => {
-    removeTempDir(tempDir);
-  });
-
-  describe('Test 2.1: Creates .safeword directory structure', () => {
-    it('should create complete .safeword/ directory', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      const result = await runCli(['setup', '--yes'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(0);
-
-      // Core structure
-      expect(fileExists(tempDir, '.safeword')).toBe(true);
-      expect(fileExists(tempDir, '.safeword/SAFEWORD.md')).toBe(true);
-      expect(fileExists(tempDir, '.safeword/version')).toBe(true);
-
-      // Subdirectories
-      expect(fileExists(tempDir, '.safeword/guides')).toBe(true);
-      expect(fileExists(tempDir, '.safeword/templates')).toBe(true);
-      expect(fileExists(tempDir, '.safeword/hooks')).toBe(true);
-    });
-
-    it('should write CLI version to .safeword/version', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      await runCli(['setup', '--yes'], { cwd: tempDir });
-
-      const version = readTestFile(tempDir, '.safeword/version').trim();
-      // Should be semver format
-      expect(version).toMatch(/^\d+\.\d+\.\d+(-[\w.]+)?$/);
-    });
-  });
-
-  describe('Test 2.2: Creates AGENTS.md if missing', () => {
-    it('should create AGENTS.md with safeword link', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      await runCli(['setup', '--yes'], { cwd: tempDir });
-
-      expect(fileExists(tempDir, 'AGENTS.md')).toBe(true);
-
-      const content = readTestFile(tempDir, 'AGENTS.md');
-      expect(content).toContain('**⚠️ ALWAYS READ FIRST: @./.safeword/SAFEWORD.md**');
-    });
-  });
-
-  describe('Test 2.3: Prepends link to existing AGENTS.md', () => {
-    it('should prepend link without losing content', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      // Create existing AGENTS.md
-      const existingContent = '# My Project\n\nExisting content here.\n';
-      writeTestFile(tempDir, 'AGENTS.md', existingContent);
-
-      await runCli(['setup', '--yes'], { cwd: tempDir });
-
-      const content = readTestFile(tempDir, 'AGENTS.md');
-
-      // Link should be first
-      const lines = content.split('\n');
-      expect(lines[0]).toContain('**⚠️ ALWAYS READ FIRST: @./.safeword/SAFEWORD.md**');
-
-      // Original content preserved
-      expect(content).toContain('# My Project');
-      expect(content).toContain('Existing content here.');
-    });
-  });
-
-  describe('Test 2.4: No duplicate links in AGENTS.md on upgrade', () => {
-    it('should not add duplicate link on upgrade', async () => {
-      // First setup
-      await createConfiguredProject(tempDir);
-
-      // Verify link exists
-      const contentBefore = readTestFile(tempDir, 'AGENTS.md');
-      const linkCount = (contentBefore.match(/@\.\/\.safeword\/SAFEWORD\.md/g) || []).length;
-      expect(linkCount).toBe(1);
-
-      // Run upgrade
-      await runCli(['upgrade'], { cwd: tempDir });
-
-      // Count links after
-      const contentAfter = readTestFile(tempDir, 'AGENTS.md');
-      const linkCountAfter = (contentAfter.match(/@\.\/\.safeword\/SAFEWORD\.md/g) || []).length;
-
-      expect(linkCountAfter).toBe(1);
-    });
-  });
-
-  describe('Test 2.5: Prints summary of created files', () => {
-    it('should output summary of created files', async () => {
-      createTypeScriptPackageJson(tempDir);
-      initGitRepo(tempDir);
-
-      const result = await runCli(['setup', '--yes'], { cwd: tempDir });
-
-      expect(result.exitCode).toBe(0);
-
-      // Should mention what was created
-      expect(result.stdout).toMatch(/created|Created/i);
-      expect(result.stdout).toMatch(/\.safeword|safeword/i);
-    });
-  });
-});