specweave 0.24.8 → 0.24.9

package/plugins/specweave/lib/hooks/consolidated-sync.js

@@ -0,0 +1,183 @@
+#!/usr/bin/env node
+/**
+ * Consolidated Sync Hook
+ *
+ * PERFORMANCE OPTIMIZATION (v0.24.4):
+ * =====================================
+ * This script consolidates ALL post-task-completion operations into a SINGLE
+ * Node.js process instead of spawning 5-6 separate processes.
+ *
+ * BEFORE (v0.24.3):
+ * - Spawn 1: update-tasks-md.js       (10KB)
+ * - Spawn 2: sync-living-docs.js      (18KB)
+ * - Spawn 3: update-ac-status.js      (2KB)
+ * - Spawn 4: translate-living-docs.js (4.4KB)
+ * - Spawn 5: run-self-reflection.js   (4.1KB - conditional)
+ * Total: 5-6 Node.js processes per TodoWrite
+ *
+ * AFTER (v0.24.4):
+ * - Spawn 1: consolidated-sync.js (runs all operations sequentially)
+ * Total: 1 Node.js process per TodoWrite
+ *
+ * IMPACT:
+ * - 83% reduction in process spawning overhead
+ * - Faster execution (shared Node.js context)
+ * - Better error handling (single error boundary)
+ * - Reduced system resource usage
+ *
+ * ROOT CAUSE ADDRESSED:
+ * Multiple rapid TodoWrite calls were spawning 12+ concurrent Node.js processes,
+ * causing process exhaustion and Claude Code crashes. This consolidation prevents
+ * that by limiting to 1 process per hook fire.
+ *
+ * See: .specweave/increments/0051-automatic-github-sync/reports/HOOK-CRASH-ANALYSIS.md
+ * ADR: (pending - will be created after testing)
+ */
+
+// Import REAL implementations from existing modules
+import { updateTasksMd } from './update-tasks-md.js';
+import { syncLivingDocs } from './sync-living-docs.js';
+import { translateLivingDocs } from './translate-living-docs.js';
+
+// Import for AC status (uses ACStatusManager directly)
+import { ACStatusManager } from '../vendor/core/increment/ac-status-manager.js';
+
+// ============================================================================
+// WRAPPER: UPDATE AC STATUS
+// ============================================================================
+// Note: update-ac-status.js doesn't export a function, it only has CLI logic
+// So we replicate the core logic here using ACStatusManager
+async function updateACStatus(incrementId) {
+  try {
+    console.log(`\n🔄 [3/5] Syncing AC status for increment ${incrementId}...`);
+
+    const projectRoot = process.cwd();
+
+    if (process.env.SKIP_AC_SYNC === 'true') {
+      console.log('ℹ️  AC sync skipped (SKIP_AC_SYNC=true)');
+      return { success: true, message: 'Sync skipped' };
+    }
+
+    const manager = new ACStatusManager(projectRoot);
+    const result = await manager.syncACStatus(incrementId);
+
+    if (result.warnings && result.warnings.length > 0) {
+      console.log('\n⚠️  Warnings:');
+      result.warnings.forEach((warning) => console.log(`   ${warning}`));
+    }
+
+    if (result.conflicts && result.conflicts.length > 0) {
+      console.log('\n⚠️  Conflicts detected:');
+      result.conflicts.forEach((conflict) => console.log(`   ${conflict}`));
+    }
+
+    if (result.updated && result.updated.length > 0) {
+      console.log('\n✅ Updated AC checkboxes:');
+      result.updated.forEach((acId) => console.log(`   ${acId} → [x]`));
+    } else if (result.synced) {
+      console.log('✅ All ACs already in sync (no changes needed)');
+    } else {
+      console.log('ℹ️  No AC updates needed');
+    }
+
+    return { success: true, result };
+  } catch (error) {
+    console.error('❌ Error updating AC status:', error.message);
+    return { success: false, error: error.message };
+  }
+}
+
+// ============================================================================
+// MAIN EXECUTION
+// ============================================================================
+async function runConsolidatedSync(incrementId) {
+  console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
+  console.log(`🚀 CONSOLIDATED SYNC: ${incrementId}`);
+  console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
+
+  const startTime = Date.now();
+  const results = {};
+
+  try {
+    // OPERATION 1: Update tasks.md (uses imported function)
+    console.log('\n🔄 [1/4] Updating tasks.md...');
+    try {
+      await updateTasksMd(incrementId);
+      results.updateTasks = { success: true };
+    } catch (error) {
+      console.error('❌ Error updating tasks.md:', error.message);
+      results.updateTasks = { success: false, error: error.message };
+    }
+
+    // OPERATION 2: Sync living docs (uses imported function)
+    console.log('\n📚 [2/4] Syncing living docs...');
+    try {
+      await syncLivingDocs(incrementId);
+      results.syncDocs = { success: true };
+    } catch (error) {
+      console.error('❌ Error syncing living docs:', error.message);
+      results.syncDocs = { success: false, error: error.message };
+    }
+
+    // OPERATION 3: Update AC status (uses local wrapper)
+    try {
+      results.updateAC = await updateACStatus(incrementId);
+    } catch (error) {
+      console.error('❌ Error updating AC status:', error.message);
+      results.updateAC = { success: false, error: error.message };
+    }
+
+    // OPERATION 4: Translate living docs (uses imported function)
+    console.log('\n🌐 [4/4] Checking translation needs...');
+    try {
+      await translateLivingDocs(incrementId);
+      results.translate = { success: true };
+    } catch (error) {
+      console.error('❌ Error translating docs:', error.message);
+      results.translate = { success: false, error: error.message };
+    }
+
+    const duration = Date.now() - startTime;
+
+    console.log('\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
+    console.log(`✅ CONSOLIDATED SYNC COMPLETED in ${duration}ms`);
+    console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n');
+
+    // Summary of results
+    const successCount = Object.values(results).filter(r => r.success).length;
+    const totalCount = Object.keys(results).length;
+    console.log(`📊 Results: ${successCount}/${totalCount} operations successful`);
+
+    // Check for any failures (non-blocking - we tolerate partial failures)
+    const failures = Object.entries(results).filter(([_, result]) => !result.success);
+    if (failures.length > 0) {
+      console.warn('\n⚠️  Some operations had issues (non-blocking):');
+      failures.forEach(([op, result]) => {
+        console.warn(`   ${op}: ${result.error || 'Unknown error'}`);
+      });
+    }
+
+    // Always exit 0 to prevent hook failures from blocking Claude Code
+    process.exit(0);
+  } catch (error) {
+    console.error('\n❌ FATAL ERROR in consolidated sync:', error);
+    // Even on fatal error, exit 0 to prevent blocking Claude Code
+    process.exit(0);
+  }
+}
+
+// CLI Interface
+const isMainModule = import.meta.url === `file://${process.argv[1]}`;
+if (isMainModule) {
+  const incrementId = process.argv[2];
+
+  if (!incrementId) {
+    console.error('Usage: node consolidated-sync.js <increment-id>');
+    console.error('Example: node consolidated-sync.js 0051-automatic-github-sync');
+    process.exit(1);
+  }
+
+  runConsolidatedSync(incrementId);
+}
+
+export { runConsolidatedSync };

package/plugins/specweave/lib/hooks/git-diff-analyzer.d.ts.bak

@@ -0,0 +1,89 @@
+/**
+ * Git Diff Analyzer
+ *
+ * Extracts modified files from git diff for reflection analysis
+ * Parses git diff output to get file changes, line counts, and content
+ *
+ * @module git-diff-analyzer
+ */
+import { GitDiffInfo } from './types/reflection-types';
+/**
+ * Check if directory is a git repository
+ * @param dir Directory to check (defaults to cwd)
+ * @returns True if directory is in a git repository
+ */
+export declare function isGitRepository(dir?: string): boolean;
+/**
+ * Get list of modified files in the working directory
+ * Includes both staged and unstaged changes
+ *
+ * @param cwd Working directory (optional, defaults to process.cwd())
+ * @returns Array of file paths relative to git root
+ */
+export declare function getModifiedFilesList(cwd?: string): string[];
+/**
+ * Parse git diff numstat output to get line counts
+ * Format: <added>\t<removed>\t<filename>
+ *
+ * @param numstatOutput Output from git diff --numstat
+ * @returns Map of filename to {added, removed} counts
+ */
+export declare function parseNumstat(numstatOutput: string): Map<string, {
+    added: number;
+    removed: number;
+}>;
+/**
+ * Get diff content for a specific file
+ * @param file File path relative to git root
+ * @param cwd Working directory (optional)
+ * @returns Diff content as string
+ */
+export declare function getFileDiff(file: string, cwd?: string): string;
+/**
+ * Get current file content
+ * @param file File path (can be absolute or relative to cwd)
+ * @param cwd Working directory (optional)
+ * @returns File content as string, or empty string if file doesn't exist
+ */
+export declare function getFileContent(file: string, cwd?: string): string;
+/**
+ * Get modified files with diff information
+ * Main function for reflection analysis
+ *
+ * @param cwd Working directory (optional, defaults to process.cwd())
+ * @param maxFiles Maximum number of files to return (optional, defaults to 100)
+ * @returns Array of GitDiffInfo objects with file changes
+ */
+export declare function getModifiedFiles(cwd?: string, maxFiles?: number): GitDiffInfo[];
+/**
+ * Get summary statistics for modified files
+ * Useful for reflection metadata
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @returns Summary with file count, total lines added/removed
+ */
+export declare function getModifiedFilesSummary(modifiedFiles: GitDiffInfo[]): {
+    count: number;
+    linesAdded: number;
+    linesRemoved: number;
+    totalChanges: number;
+};
+/**
+ * Filter files by extension
+ * Useful for focusing reflection on specific file types
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @param extensions Array of file extensions (e.g., ['.ts', '.js'])
+ * @returns Filtered array of GitDiffInfo objects
+ */
+export declare function filterFilesByExtension(modifiedFiles: GitDiffInfo[], extensions: string[]): GitDiffInfo[];
+/**
+ * Exclude files matching patterns
+ * Useful for excluding generated files, test files, etc.
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @param patterns Array of glob patterns to exclude
+ * @returns Filtered array of GitDiffInfo objects
+ */
+export declare function excludeFilesByPattern(modifiedFiles: GitDiffInfo[], patterns: string[]): GitDiffInfo[];
+//# sourceMappingURL=git-diff-analyzer.d.ts.map

package/plugins/specweave/lib/hooks/git-diff-analyzer.js.bak

@@ -0,0 +1,142 @@
+import { execSync } from "child_process";
+import fs from "fs-extra";
+import path from "path";
+function executeGitCommand(command, cwd) {
+  try {
+    return execSync(command, {
+      cwd: cwd || process.cwd(),
+      encoding: "utf-8",
+      stdio: ["pipe", "pipe", "pipe"]
+    });
+  } catch (error) {
+    throw new Error(`Git command failed: ${command}. ${error.message}`);
+  }
+}
+function isGitRepository(dir = process.cwd()) {
+  try {
+    executeGitCommand("git rev-parse --git-dir", dir);
+    return true;
+  } catch {
+    return false;
+  }
+}
+function getModifiedFilesList(cwd) {
+  if (!isGitRepository(cwd)) {
+    return [];
+  }
+  try {
+    const output = executeGitCommand("git diff --name-only HEAD", cwd);
+    if (!output.trim()) {
+      return [];
+    }
+    return output.trim().split("\n").filter((file) => file.length > 0).filter((file) => !file.startsWith(".git/"));
+  } catch {
+    return [];
+  }
+}
+function parseNumstat(numstatOutput) {
+  const stats = /* @__PURE__ */ new Map();
+  if (!numstatOutput.trim()) {
+    return stats;
+  }
+  const lines = numstatOutput.trim().split("\n");
+  for (const line of lines) {
+    const parts = line.split("	");
+    if (parts.length < 3) continue;
+    const added = parts[0] === "-" ? 0 : parseInt(parts[0], 10);
+    const removed = parts[1] === "-" ? 0 : parseInt(parts[1], 10);
+    const filename = parts[2];
+    stats.set(filename, { added, removed });
+  }
+  return stats;
+}
+function getFileDiff(file, cwd) {
+  if (!isGitRepository(cwd)) {
+    return "";
+  }
+  try {
+    const output = executeGitCommand(`git diff HEAD -- "${file}"`, cwd);
+    return output;
+  } catch {
+    return "";
+  }
+}
+function getFileContent(file, cwd) {
+  try {
+    const workingDir = cwd || process.cwd();
+    const absolutePath = path.isAbsolute(file) ? file : path.join(workingDir, file);
+    if (!fs.existsSync(absolutePath)) {
+      return "";
+    }
+    return fs.readFileSync(absolutePath, "utf-8");
+  } catch {
+    return "";
+  }
+}
+function getModifiedFiles(cwd, maxFiles = 100) {
+  if (!isGitRepository(cwd)) {
+    return [];
+  }
+  const workingDir = cwd || process.cwd();
+  const modifiedFiles = getModifiedFilesList(workingDir);
+  if (modifiedFiles.length === 0) {
+    return [];
+  }
+  const filesToAnalyze = modifiedFiles.slice(0, maxFiles);
+  let numstatOutput = "";
+  try {
+    numstatOutput = executeGitCommand("git diff --numstat HEAD", workingDir);
+  } catch {
+  }
+  const stats = parseNumstat(numstatOutput);
+  const result = [];
+  for (const file of filesToAnalyze) {
+    const fileStat = stats.get(file) || { added: 0, removed: 0 };
+    const diffContent = getFileDiff(file, workingDir);
+    if (fileStat.added > 0 || fileStat.removed > 0 || diffContent.length > 0) {
+      result.push({
+        file,
+        linesAdded: fileStat.added,
+        linesRemoved: fileStat.removed,
+        content: diffContent
+      });
+    }
+  }
+  return result;
+}
+function getModifiedFilesSummary(modifiedFiles) {
+  return {
+    count: modifiedFiles.length,
+    linesAdded: modifiedFiles.reduce((sum, file) => sum + file.linesAdded, 0),
+    linesRemoved: modifiedFiles.reduce((sum, file) => sum + file.linesRemoved, 0),
+    totalChanges: modifiedFiles.reduce(
+      (sum, file) => sum + file.linesAdded + file.linesRemoved,
+      0
+    )
+  };
+}
+function filterFilesByExtension(modifiedFiles, extensions) {
+  return modifiedFiles.filter((file) => {
+    const ext = path.extname(file.file).toLowerCase();
+    return extensions.some((allowedExt) => ext === allowedExt.toLowerCase());
+  });
+}
+function excludeFilesByPattern(modifiedFiles, patterns) {
+  return modifiedFiles.filter((file) => {
+    return !patterns.some((pattern) => {
+      const regex = new RegExp("^" + pattern.replace(/\*/g, ".*") + "$");
+      return regex.test(file.file);
+    });
+  });
+}
+export {
+  excludeFilesByPattern,
+  filterFilesByExtension,
+  getFileContent,
+  getFileDiff,
+  getModifiedFiles,
+  getModifiedFilesList,
+  getModifiedFilesSummary,
+  isGitRepository,
+  parseNumstat
+};

package/plugins/specweave/lib/hooks/git-diff-analyzer.ts.bak

@@ -0,0 +1,269 @@
+/**
+ * Git Diff Analyzer
+ *
+ * Extracts modified files from git diff for reflection analysis
+ * Parses git diff output to get file changes, line counts, and content
+ *
+ * @module git-diff-analyzer
+ */
+
+import { execSync } from 'child_process';
+import fs from 'fs-extra';
+import path from 'path';
+import { GitDiffInfo } from './types/reflection-types';
+
+/**
+ * Execute git command safely
+ * @param command Git command to execute
+ * @param cwd Working directory (optional)
+ * @returns Command output as string
+ * @throws Error if command fails
+ */
+function executeGitCommand(command: string, cwd?: string): string {
+  try {
+    return execSync(command, {
+      cwd: cwd || process.cwd(),
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe']
+    });
+  } catch (error: any) {
+    throw new Error(`Git command failed: ${command}. ${error.message}`);
+  }
+}
+
+/**
+ * Check if directory is a git repository
+ * @param dir Directory to check (defaults to cwd)
+ * @returns True if directory is in a git repository
+ */
+export function isGitRepository(dir: string = process.cwd()): boolean {
+  try {
+    executeGitCommand('git rev-parse --git-dir', dir);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Get list of modified files in the working directory
+ * Includes both staged and unstaged changes
+ *
+ * @param cwd Working directory (optional, defaults to process.cwd())
+ * @returns Array of file paths relative to git root
+ */
+export function getModifiedFilesList(cwd?: string): string[] {
+  if (!isGitRepository(cwd)) {
+    return [];
+  }
+
+  try {
+    // Get both staged and unstaged changes
+    const output = executeGitCommand('git diff --name-only HEAD', cwd);
+
+    if (!output.trim()) {
+      return [];
+    }
+
+    return output
+      .trim()
+      .split('\n')
+      .filter(file => file.length > 0)
+      .filter(file => !file.startsWith('.git/')); // Exclude .git directory
+  } catch {
+    return [];
+  }
+}
+
+/**
+ * Parse git diff numstat output to get line counts
+ * Format: <added>\t<removed>\t<filename>
+ *
+ * @param numstatOutput Output from git diff --numstat
+ * @returns Map of filename to {added, removed} counts
+ */
+export function parseNumstat(numstatOutput: string): Map<string, { added: number; removed: number }> {
+  const stats = new Map<string, { added: number; removed: number }>();
+
+  if (!numstatOutput.trim()) {
+    return stats;
+  }
+
+  const lines = numstatOutput.trim().split('\n');
+
+  for (const line of lines) {
+    const parts = line.split('\t');
+    if (parts.length < 3) continue;
+
+    const added = parts[0] === '-' ? 0 : parseInt(parts[0], 10);
+    const removed = parts[1] === '-' ? 0 : parseInt(parts[1], 10);
+    const filename = parts[2];
+
+    stats.set(filename, { added, removed });
+  }
+
+  return stats;
+}
+
+/**
+ * Get diff content for a specific file
+ * @param file File path relative to git root
+ * @param cwd Working directory (optional)
+ * @returns Diff content as string
+ */
+export function getFileDiff(file: string, cwd?: string): string {
+  if (!isGitRepository(cwd)) {
+    return '';
+  }
+
+  try {
+    // Get unified diff for the file
+    const output = executeGitCommand(`git diff HEAD -- "${file}"`, cwd);
+    return output;
+  } catch {
+    return '';
+  }
+}
+
+/**
+ * Get current file content
+ * @param file File path (can be absolute or relative to cwd)
+ * @param cwd Working directory (optional)
+ * @returns File content as string, or empty string if file doesn't exist
+ */
+export function getFileContent(file: string, cwd?: string): string {
+  try {
+    const workingDir = cwd || process.cwd();
+    const absolutePath = path.isAbsolute(file) ? file : path.join(workingDir, file);
+
+    if (!fs.existsSync(absolutePath)) {
+      return '';
+    }
+
+    return fs.readFileSync(absolutePath, 'utf-8');
+  } catch {
+    return '';
+  }
+}
+
+/**
+ * Get modified files with diff information
+ * Main function for reflection analysis
+ *
+ * @param cwd Working directory (optional, defaults to process.cwd())
+ * @param maxFiles Maximum number of files to return (optional, defaults to 100)
+ * @returns Array of GitDiffInfo objects with file changes
+ */
+export function getModifiedFiles(
+  cwd?: string,
+  maxFiles: number = 100
+): GitDiffInfo[] {
+  if (!isGitRepository(cwd)) {
+    return [];
+  }
+
+  const workingDir = cwd || process.cwd();
+
+  // Get list of modified files
+  const modifiedFiles = getModifiedFilesList(workingDir);
+
+  if (modifiedFiles.length === 0) {
+    return [];
+  }
+
+  // Limit number of files to prevent overwhelming the analysis
+  const filesToAnalyze = modifiedFiles.slice(0, maxFiles);
+
+  // Get line count statistics
+  let numstatOutput = '';
+  try {
+    numstatOutput = executeGitCommand('git diff --numstat HEAD', workingDir);
+  } catch {
+    // If numstat fails, continue with empty stats
+  }
+
+  const stats = parseNumstat(numstatOutput);
+
+  // Build GitDiffInfo array
+  const result: GitDiffInfo[] = [];
+
+  for (const file of filesToAnalyze) {
+    const fileStat = stats.get(file) || { added: 0, removed: 0 };
+    const diffContent = getFileDiff(file, workingDir);
+
+    // Only include files with actual changes
+    if (fileStat.added > 0 || fileStat.removed > 0 || diffContent.length > 0) {
+      result.push({
+        file,
+        linesAdded: fileStat.added,
+        linesRemoved: fileStat.removed,
+        content: diffContent
+      });
+    }
+  }
+
+  return result;
+}
+
+/**
+ * Get summary statistics for modified files
+ * Useful for reflection metadata
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @returns Summary with file count, total lines added/removed
+ */
+export function getModifiedFilesSummary(modifiedFiles: GitDiffInfo[]): {
+  count: number;
+  linesAdded: number;
+  linesRemoved: number;
+  totalChanges: number;
+} {
+  return {
+    count: modifiedFiles.length,
+    linesAdded: modifiedFiles.reduce((sum, file) => sum + file.linesAdded, 0),
+    linesRemoved: modifiedFiles.reduce((sum, file) => sum + file.linesRemoved, 0),
+    totalChanges: modifiedFiles.reduce(
+      (sum, file) => sum + file.linesAdded + file.linesRemoved,
+      0
+    )
+  };
+}
+
+/**
+ * Filter files by extension
+ * Useful for focusing reflection on specific file types
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @param extensions Array of file extensions (e.g., ['.ts', '.js'])
+ * @returns Filtered array of GitDiffInfo objects
+ */
+export function filterFilesByExtension(
+  modifiedFiles: GitDiffInfo[],
+  extensions: string[]
+): GitDiffInfo[] {
+  return modifiedFiles.filter(file => {
+    const ext = path.extname(file.file).toLowerCase();
+    return extensions.some(allowedExt => ext === allowedExt.toLowerCase());
+  });
+}
+
+/**
+ * Exclude files matching patterns
+ * Useful for excluding generated files, test files, etc.
+ *
+ * @param modifiedFiles Array of GitDiffInfo objects
+ * @param patterns Array of glob patterns to exclude
+ * @returns Filtered array of GitDiffInfo objects
+ */
+export function excludeFilesByPattern(
+  modifiedFiles: GitDiffInfo[],
+  patterns: string[]
+): GitDiffInfo[] {
+  return modifiedFiles.filter(file => {
+    return !patterns.some(pattern => {
+      // Simple pattern matching (supports * wildcard)
+      const regex = new RegExp('^' + pattern.replace(/\*/g, '.*') + '$');
+      return regex.test(file.file);
+    });
+  });
+}