beth-copilot 1.1.0 → 2.0.0

package/bin/cli.js

@@ -2,10 +2,9 @@
 
 import { fileURLToPath } from 'url';
 import { dirname, join, relative } from 'path';
-import { existsSync, mkdirSync, readdirSync, statSync, copyFileSync, readFileSync, writeFileSync, unlinkSync } from 'fs';
+import { existsSync, mkdirSync, readdirSync, statSync, copyFileSync, readFileSync, writeFileSync, unlinkSync, chmodSync } from 'fs';
 import { createRequire } from 'module';
 import { execSync, spawn } from 'child_process';
-import { validateBeadsPath, validateBinaryPath } from './lib/pathValidation.js';
 
 const require = createRequire(import.meta.url);
 const __filename = fileURLToPath(import.meta.url);
@@ -518,59 +517,6 @@ async function checkForUpdates() {
   }
 }
 
-function getBeadsPath() {
-  // Check if bd is available in PATH
-  try {
-    logDebug('Checking if bd is in PATH...');
-    execSync('bd --version', { stdio: 'ignore' });
-    logDebug('Found bd in PATH');
-    return 'bd';
-  } catch {
-    logDebug('bd not in PATH, checking common locations...');
-    // Check common installation paths based on platform
-    const homeDir = process.env.HOME || process.env.USERPROFILE || '';
-    const isWindows = process.platform === 'win32';
-    
-    const commonPaths = isWindows ? [
-      // Windows: npm global, Go bin, local apps
-      join(process.env.APPDATA || '', 'npm', 'bd.cmd'),
-      join(homeDir, 'AppData', 'Roaming', 'npm', 'bd.cmd'),
-      join(homeDir, 'AppData', 'Local', 'Microsoft', 'WindowsApps', 'bd.exe'),
-      join(homeDir, 'go', 'bin', 'bd.exe'),
-      join(process.env.GOPATH || join(homeDir, 'go'), 'bin', 'bd.exe'),
-    ] : [
-      // Unix: homebrew, npm global, go bin, local bin
-      '/opt/homebrew/bin/bd',
-      '/usr/local/bin/bd',
-      join(homeDir, '.local', 'bin', 'bd'),
-      join(homeDir, 'bin', 'bd'),
-      join(homeDir, '.npm-global', 'bin', 'bd'),
-      join(homeDir, 'go', 'bin', 'bd'),
-      join(process.env.GOPATH || join(homeDir, 'go'), 'bin', 'bd'),
-    ];
-    
-    for (const bdPath of commonPaths) {
-      logDebug(`Checking: ${bdPath}`);
-      if (existsSync(bdPath)) {
-        logDebug(`Found at: ${bdPath}`);
-        return bdPath;
-      }
-    }
-    
-    logDebug('bd not found in any common location');
-    return null;
-  }
-}
-
-function isBeadsInstalled() {
-  return getBeadsPath() !== null;
-}
-
-function isBeadsInitialized(cwd) {
-  // Check if .beads directory exists in the project
-  return existsSync(join(cwd, '.beads'));
-}
-
 async function promptYesNo(question) {
   const readline = await import('readline');
   const rl = readline.createInterface({
@@ -601,180 +547,6 @@ async function promptForInput(question) {
   });
 }
 
-/**
- * Installs the beads CLI globally via npm.
- * 
- * SECURITY NOTE - shell:true usage:
- * - Required for cross-platform npm execution (npm.cmd on Windows, npm on Unix)
- * - Arguments are HARDCODED - no user input is passed to the shell
- * - Command injection risk: NONE (no dynamic/user-supplied values)
- * 
- * Alternative considered: Using platform-specific binary names (npm.cmd vs npm)
- * would eliminate shell:true but adds complexity and edge cases for non-standard installs.
- * 
- * @returns {Promise<boolean>} True if installation succeeded and was verified
- */
-async function installBeads() {
-  const isWindows = process.platform === 'win32';
-  const isMac = process.platform === 'darwin';
-  
-  log('\nInstalling beads CLI via npm...', COLORS.cyan);
-  logInfo('npm install -g @beads/bd');
-  
-  // SECURITY: shell:true is required for cross-platform npm execution.
-  // All arguments are hardcoded constants - no user input reaches the shell.
-  return new Promise((resolve) => {
-    const child = spawn('npm', ['install', '-g', '@beads/bd'], {
-      stdio: 'inherit',
-      shell: true
-    });
-    
-    child.on('close', (code) => {
-      if (code === 0) {
-        // CRITICAL: Verify installation actually worked before claiming success
-        // npm can exit 0 even when the package isn't properly installed
-        const verifiedPath = getBeadsPath();
-        if (verifiedPath) {
-          logSuccess('beads CLI installed and verified!');
-          resolve(true);
-        } else {
-          logWarning('npm reported success but beads CLI not found in PATH.');
-          logInfo('This can happen if npm global bin is not in your PATH.');
-          if (globalThis.VERBOSE) {
-            showPathDiagnostics();
-          } else {
-            logInfo('Run with --verbose for PATH diagnostics.');
-          }
-          console.log('');
-          showBeadsAlternatives(isWindows, isMac);
-          resolve(false);
-        }
-      } else {
-        logError('npm install failed.');
-        console.log('');
-        showBeadsAlternatives(isWindows, isMac);
-        resolve(false);
-      }
-    });
-    
-    child.on('error', () => {
-      logError('Failed to run npm.');
-      logInfo('Make sure npm is installed and in your PATH.');
-      resolve(false);
-    });
-  });
-}
-
-function showBeadsAlternatives(isWindows, isMac) {
-  logInfo('Alternative installation methods:');
-  if (isWindows) {
-    logInfo('  PowerShell: irm https://raw.githubusercontent.com/steveyegge/beads/main/install.ps1 | iex');
-    logInfo('  Go:         go install github.com/steveyegge/beads/cmd/bd@latest');
-  } else {
-    if (isMac) {
-      logInfo('  Homebrew:   brew install beads');
-    }
-    logInfo('  Script:     curl -fsSL https://raw.githubusercontent.com/steveyegge/beads/main/scripts/install.sh | bash');
-    logInfo('  Go:         go install github.com/steveyegge/beads/cmd/bd@latest');
-  }
-  logInfo('');
-  logInfo('Learn more: https://github.com/steveyegge/beads');
-}
-
-/**
- * Initializes beads in the current project directory.
- * 
- * SECURITY NOTE - shell:true usage:
- * - bdPath is validated via getBeadsPath() which only returns paths that:
- *   1. Pass execSync('bd --version') verification, OR
- *   2. Exist on disk (verified via existsSync) from a HARDCODED list of paths
- * - Arguments are HARDCODED ('init') - no user input is passed to the shell
- * - Command injection risk: LOW (bdPath is validated, no user input in args)
- * 
- * The shell:true is used for PATH resolution consistency, though it could be
- * eliminated since we have an absolute path. Kept for consistency with other
- * spawn calls and to handle edge cases in shell script wrappers.
- * 
- * @param {string} cwd - Current working directory (validated by caller)
- * @returns {Promise<boolean>} True if initialization succeeded
- */
-async function initializeBeads(cwd) {
-  log('\nInitializing beads in project...', COLORS.cyan);
-  
-  const bdPath = getBeadsPath();
-  if (!bdPath) {
-    logWarning('Failed to initialize beads. Run manually: bd init');
-    return false;
-  }
-  
-  // SECURITY: bdPath is validated by getBeadsPath() (existsSync check).
-  // Only 'init' argument is passed - no user input reaches the shell.
-  return new Promise((resolve) => {
-    const child = spawn(bdPath, ['init'], {
-      stdio: 'inherit',
-      shell: true,
-      cwd
-    });
-    
-    child.on('close', (code) => {
-      if (code === 0) {
-        logSuccess('beads initialized successfully!');
-        resolve(true);
-      } else {
-        logWarning('Failed to initialize beads. Run manually: bd init');
-        resolve(false);
-      }
-    });
-    
-    child.on('error', () => {
-      logWarning('Failed to initialize beads. Run manually: bd init');
-      resolve(false);
-    });
-  });
-}
-
-/**
- * Runs `bd doctor` to verify beads configuration health.
- * 
- * SECURITY NOTE - shell:true usage:
- * - bdPath is validated via getBeadsPath() (same as initializeBeads)
- * - Arguments are HARDCODED ('doctor') - no user input is passed to the shell
- * - Command injection risk: LOW (bdPath is validated, no user input in args)
- * 
- * @returns {Promise<boolean>} True if bd doctor passed
- */
-async function runBeadsDoctor() {
-  log('\nRunning beads doctor to verify configuration...', COLORS.cyan);
-  
-  const bdPath = getBeadsPath();
-  if (!bdPath) {
-    logWarning('Cannot run beads doctor: bd not found.');
-    return false;
-  }
-  
-  return new Promise((resolve) => {
-    const child = spawn(bdPath, ['doctor'], {
-      stdio: 'inherit',
-      shell: true,
-    });
-    
-    child.on('close', (code) => {
-      if (code === 0) {
-        logSuccess('beads doctor passed!');
-        resolve(true);
-      } else {
-        logWarning('beads doctor reported issues. Run "bd doctor" manually to investigate.');
-        resolve(false);
-      }
-    });
-    
-    child.on('error', () => {
-      logWarning('Failed to run beads doctor. Run "bd doctor" manually.');
-      resolve(false);
-    });
-  });
-}
-
 const BETH_GUARD_BEGIN = '# --- BEGIN BETH GUARD ---';
 const BETH_GUARD_END = '# --- END BETH GUARD ---';
 
@@ -817,35 +589,6 @@ ${BETH_GUARD_END}
 `;
 }
 
-/**
- * Install the pre-push guard into .beads/hooks/pre-push.
- * Appends the guard section after the beads integration section.
- * Idempotent — skips if guard is already installed.
- *
- * @param {string} cwd - Project root directory
- */
-function installPrePushGuard(cwd) {
-  const hookPath = join(cwd, '.beads', 'hooks', 'pre-push');
-
-  if (!existsSync(hookPath)) {
-    logWarning('Pre-push hook not found (.beads/hooks/pre-push). Skipping guard installation.');
-    return;
-  }
-
-  const content = readFileSync(hookPath, 'utf-8');
-
-  // Already installed?
-  if (content.includes(BETH_GUARD_BEGIN)) {
-    logSuccess('Pre-push branch guard already installed');
-    return;
-  }
-
-  // Append guard after existing content
-  const guardScript = generateGuardScript();
-  writeFileSync(hookPath, content.trimEnd() + '\n' + guardScript, 'utf-8');
-  logSuccess('Installed pre-push branch guard (blocks direct pushes to main)');
-}
-
 function showHelp() {
   showBethBannerStatic({ showQuickHelp: false });
   console.log(`${COLORS.bright}Beth${COLORS.reset} - AI Orchestrator for GitHub Copilot
@@ -853,18 +596,18 @@ function showHelp() {
 ${COLORS.bright}Usage:${COLORS.reset}
   npx beth-copilot init [options]     Initialize Beth in current directory
   npx beth-copilot doctor             Check system health and dependencies
-  npx beth-copilot close <id> [opts]   Close issue with dependency enforcement
   npx beth-copilot land [opts]          Automated session completion (test, commit, push)
   npx beth-copilot pre-push-guard      Run branch discipline checks (used by git hook)
-  npx beth-copilot quickstart         Run init + doctor + beads setup
+  npx beth-copilot update [options]    Update project files to latest templates
+  npx beth-copilot quickstart         Run init + doctor
   npx beth-copilot help               Show this help message
 
 ${COLORS.bright}Options:${COLORS.reset}
   --force                             Overwrite existing files
   --skip-backlog                      Don't create Backlog.md
   --skip-mcp                          Don't create mcp.json.example
-  --skip-beads                        Skip beads check (not recommended)
   --verbose                           Show detailed diagnostics on errors
+  --check-only                        Check for updates without modifying files
 
 ${COLORS.bright}Examples:${COLORS.reset}
   npx beth-copilot init               Set up Beth in current project
@@ -939,7 +682,7 @@ function copyDirRecursive(src, dest, options = {}) {
 }
 
 async function init(options = {}) {
-  const { force = false, skipBacklog = false, skipMcp = false, skipBeads = false } = options;
+  const { force = false, skipBacklog = false, skipMcp = false } = options;
   const cwd = process.cwd();
   
   // Check for updates
@@ -1053,142 +796,10 @@ ${COLORS.yellow}╔════════════════════
     logWarning('No files were copied. Use --force to overwrite existing files.');
   }
 
-  // Check for beads CLI (REQUIRED for Beth)
-  if (!skipBeads) {
-    console.log('');
-    log('Checking beads (required for task tracking)...', COLORS.cyan);
-    
-    let bdPath = getBeadsPath();
-    
-    // Loop until beads is installed
-    while (!bdPath) {
-      logWarning('beads CLI is not installed.');
-      logInfo('Beth requires beads for task tracking. Agents use it to coordinate work.');
-      logInfo('Learn more: https://github.com/steveyegge/beads');
-      console.log('');
-      
-      const shouldInstallBeads = await promptYesNo('Install beads CLI now? (required)');
-      if (shouldInstallBeads) {
-        const installed = await installBeads();
-        if (installed) {
-          // Re-check for beads after installation
-          bdPath = getBeadsPath();
-          if (!bdPath) {
-            console.log('');
-            logWarning('beads installed but not found in common paths.');
-            logInfo('The installer may have placed it in a custom location.');
-            console.log('');
-            logInfo('Please try one of these options:');
-            logInfo('  1. Open a NEW terminal and run: npx beth-copilot init');
-            logInfo('  2. Add ~/.local/bin to your PATH and retry');
-            logInfo('  3. Run: source ~/.bashrc (or ~/.zshrc) then retry');
-            console.log('');
-            
-            const retryCheck = await promptYesNo('Retry detection? (select No to enter path manually)');
-            if (retryCheck) {
-              bdPath = getBeadsPath();
-              continue;
-            }
-            
-            // Allow manual path entry
-            const customPath = await promptForInput('Enter full path to bd binary (or press Enter to retry installation):');
-            if (customPath) {
-              const validation = validateBeadsPath(customPath);
-              if (validation.valid) {
-                bdPath = validation.normalizedPath;
-                logSuccess(`Found beads at: ${bdPath}`);
-              } else {
-                logError(`Invalid path: ${validation.error}`);
-              }
-            }
-          }
-        } else {
-          console.log('');
-          logError('Installation script failed.');
-          logInfo('You can try installing manually:');
-          logInfo('  curl -fsSL https://raw.githubusercontent.com/steveyegge/beads/main/scripts/install.sh | bash');
-          console.log('');
-        }
-      } else {
-        console.log('');
-        logError('beads is REQUIRED for Beth to function.');
-        logInfo('Beth agents use beads to track tasks, dependencies, and coordinate work.');
-        logInfo('Without beads, the multi-agent workflow will not work correctly.');
-        console.log('');
-        
-        const tryAgain = await promptYesNo('Would you like to try installing beads?');
-        if (!tryAgain) {
-          logError('Cannot continue without beads. Exiting.');
-          logInfo('Install beads manually and run "npx beth-copilot init" again:');
-          logInfo('  npm install -g @beads/bd');
-          process.exit(1);
-        }
-      }
-    }
-    
-    // Show path info if not in standard PATH
-    if (bdPath && bdPath !== 'bd') {
-      logSuccess(`beads CLI found at: ${bdPath}`);
-      const isWindows = process.platform === 'win32';
-      if (isWindows) {
-        logInfo('Tip: Ensure npm global bin is in your PATH to use "bd" directly.');
-      } else {
-        logInfo('Tip: Add ~/.local/bin or npm global bin to your PATH to use "bd" directly.');
-      }
-    } else {
-      logSuccess('beads CLI is installed');
-    }
-    
-    // Initialize beads in the project if not already done
-    if (!isBeadsInitialized(cwd)) {
-      logInfo('beads not initialized in this project.');
-      let initialized = false;
-      
-      while (!initialized) {
-        const shouldInitBeads = await promptYesNo('Initialize beads now? (required)');
-        if (shouldInitBeads) {
-          initialized = await initializeBeads(cwd);
-          if (!initialized) {
-            logWarning('Initialization failed. Let\'s try again.');
-          }
-        } else {
-          logError('beads must be initialized for Beth to work correctly.');
-          logInfo('The .beads directory stores task tracking data used by all agents.');
-          console.log('');
-        }
-      }
-    } else {
-      logSuccess('beads is initialized in this project');
-    }
-  } else {
-    logWarning('Skipped beads check (--skip-beads). Beth may not function correctly.');
-  }
-
-  // Run bd doctor to verify beads configuration
-  if (!skipBeads && getBeadsPath() && isBeadsInitialized(cwd)) {
-    await runBeadsDoctor();
-  }
-
-  // Install pre-push guard hook
-  if (!skipBeads && isBeadsInitialized(cwd)) {
-    installPrePushGuard(cwd);
-  }
-
   // Final verification
   console.log('');
   log('Verifying installation...', COLORS.cyan);
-  
-  const finalBeadsOk = skipBeads || getBeadsPath();
-  const finalBeadsInit = skipBeads || isBeadsInitialized(cwd);
-  
-  if (finalBeadsOk && finalBeadsInit) {
-    logSuccess('All dependencies installed and configured!');
-  } else {
-    if (!finalBeadsOk) logError('beads CLI not found');
-    if (!finalBeadsInit) logError('beads not initialized in project');
-    logError('Setup incomplete. Please resolve issues above and run init again.');
-    process.exit(1);
-  }
+  logSuccess('All files installed and configured!');
 
   // Next steps
   console.log(`
@@ -1207,15 +818,15 @@ ${COLORS.cyan}"They broke my wings and forgot I had claws."${COLORS.reset}
 }
 
 // Input validation constants
-const ALLOWED_COMMANDS = ['init', 'help', '--help', '-h', 'doctor', 'quickstart', 'close', 'pre-push-guard'];
-const ALLOWED_FLAGS = ['--force', '--skip-backlog', '--skip-mcp', '--skip-beads', '--verbose', '--reason', '-r', '-f', '--skip-tests', '--skip-backup', '--message', '-m', '--dry-run'];
+const ALLOWED_COMMANDS = ['init', 'help', '--help', '-h', 'doctor', 'quickstart', 'pre-push-guard', 'update', 'land'];
+const ALLOWED_FLAGS = ['--force', '--skip-backlog', '--skip-mcp', '--verbose', '--reason', '-r', '-f', '--skip-tests', '--message', '-m', '--dry-run', '--check-only'];
 const MAX_ARG_LENGTH = 50;
 
 // Validate and sanitize input
 function validateArgs(args) {
-  // The 'close' and 'land' commands handle their own arg validation
+  // The 'land' and 'update' commands handle their own arg validation
   const command = args[0]?.toLowerCase();
-  if (command === 'close' || command === 'land') return;
+  if (command === 'land' || command === 'update') return;
 
   for (const arg of args) {
     // Prevent excessively long arguments (log injection, DoS)
@@ -1241,7 +852,6 @@ const options = {
   force: args.includes('--force'),
   skipBacklog: args.includes('--skip-backlog'),
   skipMcp: args.includes('--skip-mcp'),
-  skipBeads: args.includes('--skip-beads'),
   verbose: args.includes('--verbose'),
 };
 
@@ -1249,8 +859,8 @@ const options = {
 globalThis.VERBOSE = options.verbose;
 
 // Validate unknown flags (exclude --help which is handled as a command)
-// Skip for 'close' and 'land' commands which handle their own arg parsing
-if (command !== 'close' && command !== 'land') {
+// Skip for 'land' and 'update' commands which handle their own arg parsing
+if (command !== 'land' && command !== 'update') {
   const unknownFlags = args.filter(arg => arg.startsWith('--') && !ALLOWED_FLAGS.includes(arg) && arg !== '--help');
   if (unknownFlags.length > 0) {
     logError(`Unknown flag: ${unknownFlags[0].slice(0, MAX_ARG_LENGTH)}`);
@@ -1283,14 +893,6 @@ switch (command) {
       await quickstart(options);
     }
     break;
-  case 'close':
-    {
-      const { close } = await loadTsCommand('close');
-      // Pass raw args after 'close' — the command handles its own parsing
-      const closeArgs = process.argv.slice(3);
-      await close(closeArgs);
-    }
-    break;
   case 'land':
     {
       const { land } = await loadTsCommand('land');
@@ -1299,6 +901,13 @@ switch (command) {
       await land(landArgs);
     }
     break;
+  case 'update':
+    {
+      const { update } = await loadTsCommand('update');
+      const updateArgs = process.argv.slice(3);
+      await update(updateArgs);
+    }
+    break;
   case 'pre-push-guard':
     {
       const { prePushGuard } = await loadTsCommand('pre-push-guard');

package/dist/__tests__/inject-skills.test.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Unit tests for .github/hooks/scripts/inject-skills.mjs
+ *
+ * Tests the SubagentStart hook that deterministically injects skill context
+ * for each agent type. Verifies correct skill mapping, output format,
+ * and graceful handling of edge cases.
+ */
+export {};
+//# sourceMappingURL=inject-skills.test.d.ts.map

package/dist/__tests__/inject-skills.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"inject-skills.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/inject-skills.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG"}

package/dist/__tests__/inject-skills.test.js

@@ -0,0 +1,143 @@
+/**
+ * Unit tests for .github/hooks/scripts/inject-skills.mjs
+ *
+ * Tests the SubagentStart hook that deterministically injects skill context
+ * for each agent type. Verifies correct skill mapping, output format,
+ * and graceful handling of edge cases.
+ */
+import { describe, it, expect } from 'vitest';
+import { execFileSync } from 'node:child_process';
+import { join } from 'node:path';
+const SCRIPT_PATH = join(process.cwd(), '.github/hooks/scripts/inject-skills.mjs');
+const PROJECT_ROOT = process.cwd();
+/** Helper: pipe JSON input to inject-skills.mjs and parse the JSON output */
+function runHook(input) {
+    const result = execFileSync('node', [SCRIPT_PATH], {
+        input: JSON.stringify(input),
+        encoding: 'utf8',
+        cwd: PROJECT_ROOT,
+        timeout: 10000,
+    });
+    return JSON.parse(result);
+}
+/** Helper: extract additionalContext string from hook output */
+function getContext(input) {
+    const output = runHook(input);
+    return output.hookSpecificOutput?.additionalContext ?? '';
+}
+// ─── Core behavior ─────────────────────────────────────────────────────────
+describe('inject-skills.mjs: output structure', () => {
+    it('should always set continue: true', () => {
+        const output = runHook({ agent_type: 'developer', cwd: PROJECT_ROOT });
+        expect(output.continue).toBe(true);
+    });
+    it('should include hookSpecificOutput with SubagentStart event name', () => {
+        const output = runHook({ agent_type: 'developer', cwd: PROJECT_ROOT });
+        expect(output.hookSpecificOutput).toBeDefined();
+        expect(output.hookSpecificOutput.hookEventName).toBe('SubagentStart');
+    });
+});
+// ─── UX Designer (includes ui-ux-pro-max) ──────────────────────────────────
+describe('inject-skills.mjs: ux-designer', () => {
+    const ctx = () => getContext({ agent_type: 'ux-designer', cwd: PROJECT_ROOT });
+    it('should inject web-design-guidelines into context', () => {
+        expect(ctx()).toContain('.github/skills/web-design-guidelines/SKILL.md');
+        expect(ctx()).toContain('Skills loaded into context');
+    });
+    it('should mandate readFile for framer-components', () => {
+        expect(ctx()).toContain('.github/skills/framer-components/SKILL.md');
+        expect(ctx()).toContain('Skills to load via readFile');
+    });
+    it('should mandate readFile for ui-ux-pro-max', () => {
+        expect(ctx()).toContain('.github/prompts/ui-ux-pro-max/PROMPT.md');
+    });
+    it('should include the NON-NEGOTIABLE header', () => {
+        expect(ctx()).toContain('SKILL ENFORCEMENT');
+        expect(ctx()).toContain('NON-NEGOTIABLE');
+    });
+    it('should identify the agent type in the context', () => {
+        expect(ctx()).toContain('You are `ux-designer`');
+    });
+});
+// ─── Developer ─────────────────────────────────────────────────────────────
+describe('inject-skills.mjs: developer', () => {
+    const ctx = () => getContext({ agent_type: 'developer', cwd: PROJECT_ROOT });
+    it('should inject vercel-react-best-practices SKILL.md into context', () => {
+        expect(ctx()).toContain('.github/skills/vercel-react-best-practices/SKILL.md');
+        expect(ctx()).toContain('Skills loaded into context');
+    });
+    it('should mandate readFile for shadcn-ui', () => {
+        expect(ctx()).toContain('.github/skills/shadcn-ui/SKILL.md');
+    });
+    it('should mandate readFile for vercel-react-best-practices AGENTS.md', () => {
+        expect(ctx()).toContain('.github/skills/vercel-react-best-practices/AGENTS.md');
+    });
+});
+// ─── Product Manager ───────────────────────────────────────────────────────
+describe('inject-skills.mjs: product-manager', () => {
+    const ctx = () => getContext({ agent_type: 'product-manager', cwd: PROJECT_ROOT });
+    it('should mandate readFile for prd skill', () => {
+        expect(ctx()).toContain('.github/skills/prd/SKILL.md');
+    });
+    it('should NOT have "Skills loaded into context" (no inject files)', () => {
+        expect(ctx()).not.toContain('Skills loaded into context');
+    });
+});
+// ─── Security Reviewer ─────────────────────────────────────────────────────
+describe('inject-skills.mjs: security-reviewer', () => {
+    const ctx = () => getContext({ agent_type: 'security-reviewer', cwd: PROJECT_ROOT });
+    it('should mandate readFile for security-analysis', () => {
+        expect(ctx()).toContain('.github/skills/security-analysis/SKILL.md');
+    });
+});
+// ─── Tester ────────────────────────────────────────────────────────────────
+describe('inject-skills.mjs: tester', () => {
+    const ctx = () => getContext({ agent_type: 'tester', cwd: PROJECT_ROOT });
+    it('should inject web-design-guidelines into context', () => {
+        expect(ctx()).toContain('.github/skills/web-design-guidelines/SKILL.md');
+        expect(ctx()).toContain('Skills loaded into context');
+    });
+    it('should NOT have readFile mandate (no readFile files)', () => {
+        expect(ctx()).not.toContain('Skills to load via readFile');
+    });
+});
+// ─── Researcher ────────────────────────────────────────────────────────────
+describe('inject-skills.mjs: researcher', () => {
+    const ctx = () => getContext({ agent_type: 'researcher', cwd: PROJECT_ROOT });
+    it('should inject web-search skill into context', () => {
+        expect(ctx()).toContain('.github/skills/web-search/SKILL.md');
+    });
+});
+// ─── Edge cases ────────────────────────────────────────────────────────────
+describe('inject-skills.mjs: edge cases', () => {
+    it('should pass through unknown agent types with continue: true', () => {
+        const output = runHook({ agent_type: 'unknown-agent', cwd: PROJECT_ROOT });
+        expect(output.continue).toBe(true);
+        expect(output).not.toHaveProperty('hookSpecificOutput');
+    });
+    it('should handle missing agent_type gracefully', () => {
+        const output = runHook({ cwd: PROJECT_ROOT });
+        expect(output.continue).toBe(true);
+    });
+    it('should handle malformed JSON input gracefully', () => {
+        const result = execFileSync('node', [SCRIPT_PATH], {
+            input: 'not json at all',
+            encoding: 'utf8',
+            cwd: PROJECT_ROOT,
+            timeout: 10000,
+        });
+        const output = JSON.parse(result);
+        expect(output.continue).toBe(true);
+    });
+    it('should handle empty input gracefully', () => {
+        const result = execFileSync('node', [SCRIPT_PATH], {
+            input: '',
+            encoding: 'utf8',
+            cwd: PROJECT_ROOT,
+            timeout: 10000,
+        });
+        const output = JSON.parse(result);
+        expect(output.continue).toBe(true);
+    });
+});
+//# sourceMappingURL=inject-skills.test.js.map