specweave 0.13.5 → 0.13.6

package/plugins/specweave/lib/hooks/run-self-reflection.js

@@ -1,203 +1,132 @@
-/**
- * Core Reflection Engine
- *
- * Orchestrates the self-reflection process:
- * 1. Load configuration
- * 2. Get modified files
- * 3. Build prompt
- * 4. Invoke reflective-reviewer agent
- * 5. Parse response
- * 6. Store reflection
- *
- * @module run-self-reflection
- */
-import { loadAndValidateReflectionConfig } from './reflection-config-loader';
-import { getModifiedFiles, getModifiedFilesSummary } from './git-diff-analyzer';
-import { buildReflectionPrompt, estimatePromptTokens } from './reflection-prompt-builder';
-import { parseReflectionMarkdown, validateReflectionResult } from './reflection-parser';
-import { saveReflection } from './reflection-storage';
-import { ReflectionMode, ReflectionModel } from './types/reflection-types';
-/**
- * Check if reflection should run for the current context
- *
- * @param context Reflection context with config and modified files
- * @returns True if reflection should run
- */
+import { loadAndValidateReflectionConfig } from "./reflection-config-loader";
+import { getModifiedFiles, getModifiedFilesSummary } from "./git-diff-analyzer";
+import { buildReflectionPrompt, estimatePromptTokens } from "./reflection-prompt-builder";
+import { parseReflectionMarkdown, validateReflectionResult } from "./reflection-parser";
+import { saveReflection } from "./reflection-storage";
+import {
+  ReflectionMode,
+  ReflectionModel
+} from "./types/reflection-types";
 function shouldRunReflection(context) {
-    const { config, modifiedFiles } = context;
-    // Check if reflection is enabled
-    if (!config.enabled) {
-        return false;
-    }
-    // Check mode
-    if (config.mode === ReflectionMode.DISABLED) {
-        return false;
-    }
-    if (config.mode === ReflectionMode.MANUAL) {
-        // Manual mode requires explicit invocation
-        // This function is only called in auto mode, so skip
-        return false;
-    }
-    // Check if there are modified files to analyze
-    if (modifiedFiles.length === 0) {
-        return false;
-    }
-    return true;
+  const { config, modifiedFiles } = context;
+  if (!config.enabled) {
+    return false;
+  }
+  if (config.mode === ReflectionMode.DISABLED) {
+    return false;
+  }
+  if (config.mode === ReflectionMode.MANUAL) {
+    return false;
+  }
+  if (modifiedFiles.length === 0) {
+    return false;
+  }
+  return true;
 }
-/**
- * Estimate reflection cost based on model and prompt size
- *
- * @param model Reflection model (haiku, sonnet, opus)
- * @param promptTokens Estimated prompt token count
- * @param responseTokens Estimated response token count (default: 2000)
- * @returns Estimated cost in USD
- */
-function estimateReflectionCost(model, promptTokens, responseTokens = 2000) {
-    // Pricing per 1M tokens (as of Nov 2025)
-    const pricing = {
-        [ReflectionModel.HAIKU]: { input: 0.80, output: 4.00 },
-        [ReflectionModel.SONNET]: { input: 3.00, output: 15.00 },
-        [ReflectionModel.OPUS]: { input: 15.00, output: 75.00 }
-    };
-    const modelPricing = pricing[model];
-    const inputCost = (promptTokens / 1000000) * modelPricing.input;
-    const outputCost = (responseTokens / 1000000) * modelPricing.output;
-    return inputCost + outputCost;
+function estimateReflectionCost(model, promptTokens, responseTokens = 2e3) {
+  const pricing = {
+    [ReflectionModel.HAIKU]: { input: 0.8, output: 4 },
+    [ReflectionModel.SONNET]: { input: 3, output: 15 },
+    [ReflectionModel.OPUS]: { input: 15, output: 75 }
+  };
+  const modelPricing = pricing[model];
+  const inputCost = promptTokens / 1e6 * modelPricing.input;
+  const outputCost = responseTokens / 1e6 * modelPricing.output;
+  return inputCost + outputCost;
 }
-/**
- * Invoke reflective-reviewer agent to analyze code
- *
- * @param prompt Reflection prompt
- * @param model Model to use (haiku, sonnet, opus)
- * @returns Agent response (markdown reflection)
- */
 async function invokeReflectiveReviewerAgent(prompt, model) {
-    // NOTE: This is a placeholder for agent invocation
-    // In the real implementation, this would use Claude Code's Task tool
-    // to invoke the specweave:reflective-reviewer:reflective-reviewer agent
-    //
-    // For now, we'll return a mock response to enable testing
-    // The actual implementation will be in the hook integration (T-008)
-    throw new Error('Agent invocation not implemented yet. ' +
-        'This function will be implemented in hook integration (T-008). ' +
-        'Call this from the post-task-completion hook using the Task tool.');
+  throw new Error(
+    "Agent invocation not implemented yet. This function will be implemented in hook integration (T-008). Call this from the post-task-completion hook using the Task tool."
+  );
 }
-/**
- * Run self-reflection analysis
- * Main entry point for the reflection system
- *
- * @param context Reflection context with increment, task, and files
- * @returns Reflection execution result (success or error)
- */
-export async function runSelfReflection(context) {
-    const { incrementId, taskId, modifiedFiles, config } = context;
-    const startTime = Date.now();
-    try {
-        // Step 1: Check if reflection should run
-        if (!shouldRunReflection(context)) {
-            return {
-                success: true,
-                result: undefined,
-                error: undefined
-            };
-        }
-        // Step 2: Build reflection prompt
-        const prompt = buildReflectionPrompt({
-            taskId,
-            taskName: `Task ${taskId}`, // Will be enhanced with actual task name in hook
-            modifiedFiles,
-            config,
-            incrementId,
-            includeFullDiff: false // Use simplified prompt for token efficiency
-        });
-        // Step 3: Estimate cost
-        const promptTokens = estimatePromptTokens(prompt);
-        const estimatedCost = estimateReflectionCost(config.model, promptTokens);
-        // Step 4: Invoke reflective-reviewer agent
-        const agentResponse = await invokeReflectiveReviewerAgent(prompt, config.model);
-        // Step 5: Parse response
-        const endTime = Date.now();
-        const reflectionTime = Math.round((endTime - startTime) / 1000); // seconds
-        const reflectionResult = parseReflectionMarkdown(agentResponse, `Task ${taskId}`, config.model, reflectionTime, estimatedCost);
-        // Step 6: Add file modification stats
-        const fileStats = getModifiedFilesSummary(modifiedFiles);
-        reflectionResult.filesModified = {
-            count: fileStats.count,
-            linesAdded: fileStats.linesAdded,
-            linesRemoved: fileStats.linesRemoved
-        };
-        // Step 7: Validate reflection result
-        const validation = validateReflectionResult(reflectionResult);
-        if (!validation.valid) {
-            return {
-                success: false,
-                error: {
-                    message: 'Reflection validation failed',
-                    code: 'VALIDATION_ERROR',
-                    details: validation.errors
-                }
-            };
-        }
-        // Step 8: Store reflection (if enabled)
-        if (config.storeReflections) {
-            try {
-                saveReflection(reflectionResult, incrementId, taskId);
-            }
-            catch (storageError) {
-                // Storage failure shouldn't fail the entire reflection
-                console.warn(`Failed to store reflection: ${storageError.message}`);
-            }
+async function runSelfReflection(context) {
+  const { incrementId, taskId, modifiedFiles, config } = context;
+  const startTime = Date.now();
+  try {
+    if (!shouldRunReflection(context)) {
+      return {
+        success: true,
+        result: void 0,
+        error: void 0
+      };
+    }
+    const prompt = buildReflectionPrompt({
+      taskId,
+      taskName: `Task ${taskId}`,
+      // Will be enhanced with actual task name in hook
+      modifiedFiles,
+      config,
+      incrementId,
+      includeFullDiff: false
+      // Use simplified prompt for token efficiency
+    });
+    const promptTokens = estimatePromptTokens(prompt);
+    const estimatedCost = estimateReflectionCost(config.model, promptTokens);
+    const agentResponse = await invokeReflectiveReviewerAgent(prompt, config.model);
+    const endTime = Date.now();
+    const reflectionTime = Math.round((endTime - startTime) / 1e3);
+    const reflectionResult = parseReflectionMarkdown(
+      agentResponse,
+      `Task ${taskId}`,
+      config.model,
+      reflectionTime,
+      estimatedCost
+    );
+    const fileStats = getModifiedFilesSummary(modifiedFiles);
+    reflectionResult.filesModified = {
+      count: fileStats.count,
+      linesAdded: fileStats.linesAdded,
+      linesRemoved: fileStats.linesRemoved
+    };
+    const validation = validateReflectionResult(reflectionResult);
+    if (!validation.valid) {
+      return {
+        success: false,
+        error: {
+          message: "Reflection validation failed",
+          code: "VALIDATION_ERROR",
+          details: validation.errors
         }
-        // Step 9: Return success
-        return {
-            success: true,
-            result: reflectionResult
-        };
+      };
     }
-    catch (error) {
-        // Graceful degradation: Return error but don't throw
-        return {
-            success: false,
-            error: {
-                message: error.message || 'Unknown error during reflection',
-                code: 'REFLECTION_ERROR',
-                details: error.stack
-            }
-        };
+    if (config.storeReflections) {
+      try {
+        saveReflection(reflectionResult, incrementId, taskId);
+      } catch (storageError) {
+        console.warn(`Failed to store reflection: ${storageError.message}`);
+      }
     }
-}
-/**
- * Create reflection context from hook environment
- * Helper function for hook integration
- *
- * @param incrementId Increment identifier
- * @param taskId Task identifier
- * @param projectRoot Project root directory (optional, auto-detected)
- * @returns Reflection context ready for runSelfReflection
- */
-export function createReflectionContext(incrementId, taskId, projectRoot) {
-    // Load and validate configuration
-    const config = loadAndValidateReflectionConfig(projectRoot);
-    // Get modified files from git
-    const modifiedFiles = getModifiedFiles(projectRoot);
     return {
-        incrementId,
-        taskId,
-        modifiedFiles,
-        config
+      success: true,
+      result: reflectionResult
     };
+  } catch (error) {
+    return {
+      success: false,
+      error: {
+        message: error.message || "Unknown error during reflection",
+        code: "REFLECTION_ERROR",
+        details: error.stack
+      }
+    };
+  }
+}
+function createReflectionContext(incrementId, taskId, projectRoot) {
+  const config = loadAndValidateReflectionConfig(projectRoot);
+  const modifiedFiles = getModifiedFiles(projectRoot);
+  return {
+    incrementId,
+    taskId,
+    modifiedFiles,
+    config
+  };
 }
-/**
- * Run reflection with automatic context creation
- * Convenience function for hook integration
- *
- * @param incrementId Increment identifier
- * @param taskId Task identifier
- * @param projectRoot Project root directory (optional)
- * @returns Reflection execution result
- */
-export async function runReflectionAuto(incrementId, taskId, projectRoot) {
-    const context = createReflectionContext(incrementId, taskId, projectRoot);
-    return runSelfReflection(context);
+async function runReflectionAuto(incrementId, taskId, projectRoot) {
+  const context = createReflectionContext(incrementId, taskId, projectRoot);
+  return runSelfReflection(context);
 }
-//# sourceMappingURL=run-self-reflection.js.map
+export {
+  createReflectionContext,
+  runReflectionAuto,
+  runSelfReflection
+};

package/plugins/specweave/lib/hooks/sync-living-docs.js

@@ -1,144 +1,102 @@
 #!/usr/bin/env node
-/**
- * SpecWeave Living Docs Auto-Sync
- *
- * Automatically syncs living documentation after task completion.
- *
- * Usage:
- *   node dist/hooks/lib/sync-living-docs.js <incrementId>
- *
- * Example:
- *   node dist/hooks/lib/sync-living-docs.js 0006-llm-native-i18n
- *
- * What it does:
- * 1. Checks if sync_living_docs enabled in config
- * 2. Detects changed docs via git diff
- * 3. Invokes /sync-docs update command (future implementation)
- * 4. Logs sync actions
- *
- * @author SpecWeave Team
- * @version 1.0.0
- */
-import fs from 'fs-extra';
-import path from 'path';
-import { execSync } from 'child_process';
-/**
- * Main function - sync living docs for given increment
- */
+import fs from "fs-extra";
+import path from "path";
+import { execSync } from "child_process";
 async function syncLivingDocs(incrementId) {
-    try {
-        console.log(`\n📚 Checking living docs sync for increment: ${incrementId}`);
-        // 1. Load config
-        const configPath = path.join(process.cwd(), '.specweave', 'config.json');
-        let config = {};
-        if (fs.existsSync(configPath)) {
-            config = JSON.parse(await fs.readFile(configPath, 'utf-8'));
-        }
-        // 2. Check if sync enabled
-        const syncEnabled = config.hooks?.post_task_completion?.sync_living_docs ?? false;
-        if (!syncEnabled) {
-            console.log('ℹ️  Living docs sync disabled in config');
-            console.log('   To enable: Set hooks.post_task_completion.sync_living_docs = true');
-            return;
-        }
-        console.log('✅ Living docs sync enabled');
-        // 3. Detect changed docs via git diff
-        const changedDocs = detectChangedDocs();
-        if (changedDocs.length === 0) {
-            console.log('ℹ️  No living docs changed (no git diff in .specweave/docs/)');
-            return;
-        }
-        console.log(`📄 Detected ${changedDocs.length} changed doc(s):`);
-        changedDocs.forEach((doc) => console.log(`   - ${doc}`));
-        // 4. Sync to GitHub if configured
-        await syncToGitHub(incrementId, changedDocs);
-        console.log('✅ Living docs sync complete\n');
+  try {
+    console.log(`
+\u{1F4DA} Checking living docs sync for increment: ${incrementId}`);
+    const configPath = path.join(process.cwd(), ".specweave", "config.json");
+    let config = {};
+    if (fs.existsSync(configPath)) {
+      config = JSON.parse(await fs.readFile(configPath, "utf-8"));
     }
-    catch (error) {
-        console.error('❌ Error syncing living docs:', error);
-        // Don't exit with error - this is best-effort
+    const syncEnabled = config.hooks?.post_task_completion?.sync_living_docs ?? false;
+    if (!syncEnabled) {
+      console.log("\u2139\uFE0F  Living docs sync disabled in config");
+      console.log("   To enable: Set hooks.post_task_completion.sync_living_docs = true");
+      return;
     }
+    console.log("\u2705 Living docs sync enabled");
+    const changedDocs = detectChangedDocs();
+    if (changedDocs.length === 0) {
+      console.log("\u2139\uFE0F  No living docs changed (no git diff in .specweave/docs/)");
+      return;
+    }
+    console.log(`\u{1F4C4} Detected ${changedDocs.length} changed doc(s):`);
+    changedDocs.forEach((doc) => console.log(`   - ${doc}`));
+    await syncToGitHub(incrementId, changedDocs);
+    console.log("\u2705 Living docs sync complete\n");
+  } catch (error) {
+    console.error("\u274C Error syncing living docs:", error);
+  }
 }
-/**
- * Detect changed documentation files via git diff
- */
 function detectChangedDocs() {
-    try {
-        // Get changed files in .specweave/docs/
-        const output = execSync('git diff --name-only .specweave/docs/ 2>/dev/null || true', {
-            encoding: 'utf-8',
-            cwd: process.cwd(),
-        }).trim();
-        if (!output) {
-            return [];
-        }
-        // Filter to only .md files
-        const files = output
-            .split('\n')
-            .filter((f) => f.endsWith('.md'))
-            .filter((f) => f.length > 0);
-        return files;
-    }
-    catch (error) {
-        console.warn('⚠️  Could not detect git changes:', error);
-        return [];
+  try {
+    const output = execSync("git diff --name-only .specweave/docs/ 2>/dev/null || true", {
+      encoding: "utf-8",
+      cwd: process.cwd()
+    }).trim();
+    if (!output) {
+      return [];
     }
+    const files = output.split("\n").filter((f) => f.endsWith(".md")).filter((f) => f.length > 0);
+    return files;
+  } catch (error) {
+    console.warn("\u26A0\uFE0F  Could not detect git changes:", error);
+    return [];
+  }
 }
-/**
- * Sync changed docs to GitHub
- */
 async function syncToGitHub(incrementId, changedDocs) {
-    try {
-        console.log('\n🔄 Syncing to GitHub...');
-        // Dynamic import to avoid circular dependencies
-        const { loadIncrementMetadata, detectRepo, collectLivingDocs, updateIssueLivingDocs, postArchitectureComment } = await import('../../../specweave-github/lib/github-issue-updater.js');
-        // 1. Load metadata
-        const metadata = await loadIncrementMetadata(incrementId);
-        if (!metadata?.github?.issue) {
-            console.log('ℹ️  No GitHub issue linked, skipping GitHub sync');
-            return;
-        }
-        // 2. Detect repository
-        const repoInfo = await detectRepo();
-        if (!repoInfo) {
-            console.log('⚠️  Could not detect GitHub repository, skipping GitHub sync');
-            return;
-        }
-        const { owner, repo } = repoInfo;
-        const issueNumber = metadata.github.issue;
-        console.log(`   Syncing to ${owner}/${repo}#${issueNumber}`);
-        // 3. Collect all living docs
-        const livingDocs = await collectLivingDocs(incrementId);
-        // 4. Update issue with living docs section
-        if (livingDocs.specs.length > 0 || livingDocs.architecture.length > 0 || livingDocs.diagrams.length > 0) {
-            await updateIssueLivingDocs(issueNumber, livingDocs, owner, repo);
-        }
-        // 5. Post comments for newly created architecture docs
-        for (const docPath of changedDocs) {
-            if (docPath.includes('/architecture/')) {
-                await postArchitectureComment(issueNumber, docPath, owner, repo);
-            }
-        }
-        console.log('✅ GitHub sync complete');
+  try {
+    console.log("\n\u{1F504} Syncing to GitHub...");
+    const {
+      loadIncrementMetadata,
+      detectRepo,
+      collectLivingDocs,
+      updateIssueLivingDocs,
+      postArchitectureComment
+    } = await import("../../../specweave-github/lib/github-issue-updater.js");
+    const metadata = await loadIncrementMetadata(incrementId);
+    if (!metadata?.github?.issue) {
+      console.log("\u2139\uFE0F  No GitHub issue linked, skipping GitHub sync");
+      return;
+    }
+    const repoInfo = await detectRepo();
+    if (!repoInfo) {
+      console.log("\u26A0\uFE0F  Could not detect GitHub repository, skipping GitHub sync");
+      return;
     }
-    catch (error) {
-        console.error('❌ Error syncing to GitHub:', error);
-        console.error('   (Non-blocking - continuing...)');
+    const { owner, repo } = repoInfo;
+    const issueNumber = metadata.github.issue;
+    console.log(`   Syncing to ${owner}/${repo}#${issueNumber}`);
+    const livingDocs = await collectLivingDocs(incrementId);
+    if (livingDocs.specs.length > 0 || livingDocs.architecture.length > 0 || livingDocs.diagrams.length > 0) {
+      await updateIssueLivingDocs(issueNumber, livingDocs, owner, repo);
     }
+    for (const docPath of changedDocs) {
+      if (docPath.includes("/architecture/")) {
+        await postArchitectureComment(issueNumber, docPath, owner, repo);
+      }
+    }
+    console.log("\u2705 GitHub sync complete");
+  } catch (error) {
+    console.error("\u274C Error syncing to GitHub:", error);
+    console.error("   (Non-blocking - continuing...)");
+  }
 }
-// CLI entry point
 const isMainModule = import.meta.url === `file://${process.argv[1]}`;
 if (isMainModule) {
-    const incrementId = process.argv[2];
-    if (!incrementId) {
-        console.error('❌ Usage: sync-living-docs <incrementId>');
-        console.error('   Example: sync-living-docs 0006-llm-native-i18n');
-        process.exit(1);
-    }
-    syncLivingDocs(incrementId).catch((error) => {
-        console.error('❌ Fatal error:', error);
-        // Don't exit with error code - best-effort sync
-    });
+  const incrementId = process.argv[2];
+  if (!incrementId) {
+    console.error("\u274C Usage: sync-living-docs <incrementId>");
+    console.error("   Example: sync-living-docs 0006-llm-native-i18n");
+    process.exit(1);
+  }
+  syncLivingDocs(incrementId).catch((error) => {
+    console.error("\u274C Fatal error:", error);
+  });
 }
-export { syncLivingDocs };
-//# sourceMappingURL=sync-living-docs.js.map
+export {
+  syncLivingDocs
+};