specweave 0.23.0 → 0.23.1

package/plugins/specweave/hooks/post-edit-spec.sh

@@ -7,6 +7,11 @@
 #
 # This ensures status line stays in sync when ACs are marked complete via Edit tool
 # (not just TodoWrite, which only tracks internal todo lists)
+#
+# CRITICAL FIX (v0.24.1): Enhanced file detection to handle increment completion
+# - Detects edits via TOOL_USE_CONTENT, TOOL_RESULT, and argument parsing
+# - Always updates status line for ANY spec.md/tasks.md edit in increments folder
+# - Fixes bug where status line wasn't updating on increment close
 
 set -e
 
@@ -24,17 +29,65 @@ find_project_root() {
 }
 
 PROJECT_ROOT=$(find_project_root)
+LOGS_DIR="$PROJECT_ROOT/.specweave/logs"
+DEBUG_LOG="$LOGS_DIR/hooks-debug.log"
+
+# Ensure logs directory exists
+mkdir -p "$LOGS_DIR" 2>/dev/null || true
+
+# Extract edited file from multiple sources (Claude Code provides this in various ways)
+EDITED_FILE=""
+
+# Method 1: TOOL_USE_CONTENT environment variable (primary)
+if [[ -n "${TOOL_USE_CONTENT:-}" ]]; then
+  EDITED_FILE="$TOOL_USE_CONTENT"
+fi
+
+# Method 2: TOOL_RESULT environment variable (fallback)
+if [[ -z "$EDITED_FILE" ]] && [[ -n "${TOOL_RESULT:-}" ]]; then
+  # Extract file_path from tool result JSON
+  EDITED_FILE=$(echo "$TOOL_RESULT" | grep -o '"file_path"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"\([^"]*\)".*/\1/' || echo "")
+fi
+
+# Method 3: Parse tool use arguments (last resort)
+if [[ -z "$EDITED_FILE" ]] && [[ -n "${TOOL_USE_ARGS:-}" ]]; then
+  # Extract file_path from tool arguments
+  EDITED_FILE=$(echo "$TOOL_USE_ARGS" | grep -o '"file_path"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"\([^"]*\)".*/\1/' || echo "")
+fi
+
+# Log detection attempt
+echo "[$(date)] post-edit-spec: Detected file: ${EDITED_FILE:-<none>}" >> "$DEBUG_LOG" 2>/dev/null || true
+
+# Check if we detected a spec.md or tasks.md edit in increments folder
+SHOULD_UPDATE=false
+
+if [[ -n "$EDITED_FILE" ]]; then
+  # Check if the file is spec.md or tasks.md
+  if [[ "$EDITED_FILE" == *"/spec.md" ]] || [[ "$EDITED_FILE" == *"/tasks.md" ]]; then
+    # Check if it's in an increment folder
+    if [[ "$EDITED_FILE" == *"/.specweave/increments/"* ]]; then
+      SHOULD_UPDATE=true
+      echo "[$(date)] post-edit-spec: Increment file edited - will update status line" >> "$DEBUG_LOG" 2>/dev/null || true
+    fi
+  fi
+fi
+
+# If we couldn't detect the file via environment variables, always update status line
+# This ensures we don't miss updates during increment closure
+if [[ -z "$EDITED_FILE" ]]; then
+  echo "[$(date)] post-edit-spec: No file detected - updating status line as safety measure" >> "$DEBUG_LOG" 2>/dev/null || true
+  SHOULD_UPDATE=true
+fi
 
-# Get the file that was edited from TOOL_USE_CONTENT environment variable
-# Claude Code provides this in the hook context
-EDITED_FILE="${TOOL_USE_CONTENT:-}"
+# Update status line if needed
+if [[ "$SHOULD_UPDATE" == "true" ]]; then
+  echo "[$(date)] post-edit-spec: Running update-status-line.sh" >> "$DEBUG_LOG" 2>/dev/null || true
 
-# Only update status line if spec.md or tasks.md was edited
-if [[ "$EDITED_FILE" == *"/spec.md"* ]] || [[ "$EDITED_FILE" == *"/tasks.md"* ]]; then
-  # Check if the edit is in an increment folder
-  if [[ "$EDITED_FILE" == *"/.specweave/increments/"* ]]; then
-    # Run status line update
-    "$PROJECT_ROOT/plugins/specweave/hooks/lib/update-status-line.sh" &>/dev/null || true
+  # Run status line update (capture errors for debugging)
+  if "$PROJECT_ROOT/plugins/specweave/hooks/lib/update-status-line.sh" 2>&1 | tee -a "$DEBUG_LOG" >/dev/null; then
+    echo "[$(date)] post-edit-spec: Status line updated successfully" >> "$DEBUG_LOG" 2>/dev/null || true
+  else
+    echo "[$(date)] post-edit-spec: Warning - status line update failed (non-blocking)" >> "$DEBUG_LOG" 2>/dev/null || true
   fi
 fi
 

package/plugins/specweave/hooks/post-write-spec.sh

@@ -4,6 +4,11 @@
 #
 # Triggers: After Write tool creates/replaces spec.md or tasks.md
 # Action: Updates status line cache to reflect latest AC/task progress
+#
+# CRITICAL FIX (v0.24.1): Enhanced file detection to handle increment completion
+# - Detects writes via TOOL_USE_CONTENT, TOOL_RESULT, and argument parsing
+# - Always updates status line for ANY spec.md/tasks.md write in increments folder
+# - Matches post-edit-spec.sh robustness improvements
 
 set -e
 
@@ -21,16 +26,65 @@ find_project_root() {
 }
 
 PROJECT_ROOT=$(find_project_root)
+LOGS_DIR="$PROJECT_ROOT/.specweave/logs"
+DEBUG_LOG="$LOGS_DIR/hooks-debug.log"
+
+# Ensure logs directory exists
+mkdir -p "$LOGS_DIR" 2>/dev/null || true
+
+# Extract written file from multiple sources (Claude Code provides this in various ways)
+WRITTEN_FILE=""
+
+# Method 1: TOOL_USE_CONTENT environment variable (primary)
+if [[ -n "${TOOL_USE_CONTENT:-}" ]]; then
+  WRITTEN_FILE="$TOOL_USE_CONTENT"
+fi
+
+# Method 2: TOOL_RESULT environment variable (fallback)
+if [[ -z "$WRITTEN_FILE" ]] && [[ -n "${TOOL_RESULT:-}" ]]; then
+  # Extract file_path from tool result JSON
+  WRITTEN_FILE=$(echo "$TOOL_RESULT" | grep -o '"file_path"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"\([^"]*\)".*/\1/' || echo "")
+fi
+
+# Method 3: Parse tool use arguments (last resort)
+if [[ -z "$WRITTEN_FILE" ]] && [[ -n "${TOOL_USE_ARGS:-}" ]]; then
+  # Extract file_path from tool arguments
+  WRITTEN_FILE=$(echo "$TOOL_USE_ARGS" | grep -o '"file_path"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"\([^"]*\)".*/\1/' || echo "")
+fi
+
+# Log detection attempt
+echo "[$(date)] post-write-spec: Detected file: ${WRITTEN_FILE:-<none>}" >> "$DEBUG_LOG" 2>/dev/null || true
+
+# Check if we detected a spec.md or tasks.md write in increments folder
+SHOULD_UPDATE=false
+
+if [[ -n "$WRITTEN_FILE" ]]; then
+  # Check if the file is spec.md or tasks.md
+  if [[ "$WRITTEN_FILE" == *"/spec.md" ]] || [[ "$WRITTEN_FILE" == *"/tasks.md" ]]; then
+    # Check if it's in an increment folder
+    if [[ "$WRITTEN_FILE" == *"/.specweave/increments/"* ]]; then
+      SHOULD_UPDATE=true
+      echo "[$(date)] post-write-spec: Increment file written - will update status line" >> "$DEBUG_LOG" 2>/dev/null || true
+    fi
+  fi
+fi
+
+# If we couldn't detect the file via environment variables, always update status line
+# This ensures we don't miss updates during increment closure
+if [[ -z "$WRITTEN_FILE" ]]; then
+  echo "[$(date)] post-write-spec: No file detected - updating status line as safety measure" >> "$DEBUG_LOG" 2>/dev/null || true
+  SHOULD_UPDATE=true
+fi
 
-# Get the file that was written from TOOL_USE_CONTENT environment variable
-WRITTEN_FILE="${TOOL_USE_CONTENT:-}"
+# Update status line if needed
+if [[ "$SHOULD_UPDATE" == "true" ]]; then
+  echo "[$(date)] post-write-spec: Running update-status-line.sh" >> "$DEBUG_LOG" 2>/dev/null || true
 
-# Only update status line if spec.md or tasks.md was written
-if [[ "$WRITTEN_FILE" == *"/spec.md"* ]] || [[ "$WRITTEN_FILE" == *"/tasks.md"* ]]; then
-  # Check if the write is in an increment folder
-  if [[ "$WRITTEN_FILE" == *"/.specweave/increments/"* ]]; then
-    # Run status line update
-    "$PROJECT_ROOT/plugins/specweave/hooks/lib/update-status-line.sh" &>/dev/null || true
+  # Run status line update (capture errors for debugging)
+  if "$PROJECT_ROOT/plugins/specweave/hooks/lib/update-status-line.sh" 2>&1 | tee -a "$DEBUG_LOG" >/dev/null; then
+    echo "[$(date)] post-write-spec: Status line updated successfully" >> "$DEBUG_LOG" 2>/dev/null || true
+  else
+    echo "[$(date)] post-write-spec: Warning - status line update failed (non-blocking)" >> "$DEBUG_LOG" 2>/dev/null || true
   fi
 fi
 

package/plugins/specweave/lib/hooks/auto-transition.js.bak

@@ -0,0 +1,50 @@
+#!/usr/bin/env node
+import { AutoTransitionManager } from "../../../../dist/src/core/increment/auto-transition-manager.js";
+async function main() {
+  const event = process.argv[2];
+  const incrementId = process.argv[3];
+  const force = process.argv.includes("--force");
+  if (!event || !incrementId) {
+    console.error("Usage: auto-transition.ts <event> <increment-id> [--force]");
+    console.error("Events: spec-created, tasks-created, task-started, auto-correct");
+    console.error("Example: node auto-transition.ts spec-created 0039-ultra-smart-next-command");
+    process.exit(1);
+  }
+  const projectRoot = process.cwd();
+  const manager = new AutoTransitionManager(projectRoot);
+  let result;
+  try {
+    switch (event) {
+      case "spec-created":
+        result = await manager.handleSpecCreated(incrementId);
+        break;
+      case "tasks-created":
+        result = await manager.handleTasksCreated(incrementId);
+        break;
+      case "task-started":
+        result = await manager.handleTaskStarted(incrementId);
+        break;
+      case "auto-correct":
+        result = await manager.autoCorrect(incrementId, force);
+        break;
+      default:
+        console.error(`\u274C Unknown event: ${event}`);
+        console.error("Valid events: spec-created, tasks-created, task-started, auto-correct");
+        process.exit(1);
+    }
+    if (result.transitioned) {
+      console.log(`\u2705 Auto-transition: ${result.from} \u2192 ${result.to}`);
+      console.log(`   Reason: ${result.reason}`);
+    } else {
+      console.log(`\u2139\uFE0F  No transition: ${result.reason}`);
+    }
+    process.exit(0);
+  } catch (error) {
+    console.error("\u274C Auto-transition error:", error);
+    process.exit(1);
+  }
+}
+const isMainModule = import.meta.url === `file://${process.argv[1]}`;
+if (isMainModule) {
+  main();
+}

package/plugins/specweave/lib/hooks/auto-transition.ts.bak

@@ -0,0 +1,84 @@
+#!/usr/bin/env node
+
+/**
+ * Auto-Transition Hook Handler
+ *
+ * CLI wrapper for AutoTransitionManager
+ * Called by bash hooks to trigger status transitions
+ *
+ * Events:
+ * - spec-created: When spec.md is created (BACKLOG → PLANNING)
+ * - tasks-created: When tasks.md is created (PLANNING/BACKLOG → ACTIVE)
+ * - task-started: When first task is started (PLANNING → ACTIVE)
+ * - auto-correct: Fix status based on artifacts
+ *
+ * Usage:
+ *   node auto-transition.ts spec-created 0039-ultra-smart-next-command
+ *   node auto-transition.ts tasks-created 0039-ultra-smart-next-command
+ *   node auto-transition.ts task-started 0039-ultra-smart-next-command
+ *   node auto-transition.ts auto-correct 0039-ultra-smart-next-command
+ */
+
+import { AutoTransitionManager } from '../../../../dist/src/core/increment/auto-transition-manager.js';
+
+async function main() {
+  const event = process.argv[2];
+  const incrementId = process.argv[3];
+  const force = process.argv.includes('--force');
+
+  if (!event || !incrementId) {
+    console.error('Usage: auto-transition.ts <event> <increment-id> [--force]');
+    console.error('Events: spec-created, tasks-created, task-started, auto-correct');
+    console.error('Example: node auto-transition.ts spec-created 0039-ultra-smart-next-command');
+    process.exit(1);
+  }
+
+  const projectRoot = process.cwd();
+  const manager = new AutoTransitionManager(projectRoot);
+
+  let result;
+
+  try {
+    switch (event) {
+      case 'spec-created':
+        result = await manager.handleSpecCreated(incrementId);
+        break;
+
+      case 'tasks-created':
+        result = await manager.handleTasksCreated(incrementId);
+        break;
+
+      case 'task-started':
+        result = await manager.handleTaskStarted(incrementId);
+        break;
+
+      case 'auto-correct':
+        result = await manager.autoCorrect(incrementId, force);
+        break;
+
+      default:
+        console.error(`❌ Unknown event: ${event}`);
+        console.error('Valid events: spec-created, tasks-created, task-started, auto-correct');
+        process.exit(1);
+    }
+
+    // Display result
+    if (result.transitioned) {
+      console.log(`✅ Auto-transition: ${result.from} → ${result.to}`);
+      console.log(`   Reason: ${result.reason}`);
+    } else {
+      console.log(`ℹ️  No transition: ${result.reason}`);
+    }
+
+    process.exit(0);
+  } catch (error) {
+    console.error('❌ Auto-transition error:', error);
+    process.exit(1);
+  }
+}
+
+// Run if executed directly
+const isMainModule = import.meta.url === `file://${process.argv[1]}`;
+if (isMainModule) {
+  main();
+}

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
+};