specweave 0.23.0 → 0.23.1

package/plugins/specweave/lib/hooks/reflection-config-loader.js.bak

@@ -0,0 +1,92 @@
+import fs from "fs-extra";
+import path from "path";
+import { DEFAULT_REFLECTION_CONFIG } from "./types/reflection-types";
+function findSpecweaveRoot(startDir = process.cwd()) {
+  let currentDir = startDir;
+  const root = path.parse(currentDir).root;
+  while (currentDir !== root) {
+    const specweavePath = path.join(currentDir, ".specweave");
+    if (fs.existsSync(specweavePath)) {
+      return currentDir;
+    }
+    currentDir = path.dirname(currentDir);
+  }
+  return null;
+}
+function loadReflectionConfig(projectRoot) {
+  const rootDir = projectRoot || findSpecweaveRoot();
+  if (!rootDir) {
+    return { ...DEFAULT_REFLECTION_CONFIG };
+  }
+  const configPath = path.join(rootDir, ".specweave", "config.json");
+  if (!fs.existsSync(configPath)) {
+    return { ...DEFAULT_REFLECTION_CONFIG };
+  }
+  try {
+    const configContent = fs.readFileSync(configPath, "utf-8");
+    const config = JSON.parse(configContent);
+    const userReflectionConfig = config.reflection || {};
+    const mergedConfig = {
+      enabled: userReflectionConfig.enabled ?? DEFAULT_REFLECTION_CONFIG.enabled,
+      mode: userReflectionConfig.mode ?? DEFAULT_REFLECTION_CONFIG.mode,
+      depth: userReflectionConfig.depth ?? DEFAULT_REFLECTION_CONFIG.depth,
+      model: userReflectionConfig.model ?? DEFAULT_REFLECTION_CONFIG.model,
+      categories: {
+        security: userReflectionConfig.categories?.security ?? DEFAULT_REFLECTION_CONFIG.categories.security,
+        quality: userReflectionConfig.categories?.quality ?? DEFAULT_REFLECTION_CONFIG.categories.quality,
+        testing: userReflectionConfig.categories?.testing ?? DEFAULT_REFLECTION_CONFIG.categories.testing,
+        performance: userReflectionConfig.categories?.performance ?? DEFAULT_REFLECTION_CONFIG.categories.performance,
+        technicalDebt: userReflectionConfig.categories?.technicalDebt ?? DEFAULT_REFLECTION_CONFIG.categories.technicalDebt
+      },
+      criticalThreshold: userReflectionConfig.criticalThreshold ?? DEFAULT_REFLECTION_CONFIG.criticalThreshold,
+      storeReflections: userReflectionConfig.storeReflections ?? DEFAULT_REFLECTION_CONFIG.storeReflections,
+      autoCreateFollowUpTasks: userReflectionConfig.autoCreateFollowUpTasks ?? DEFAULT_REFLECTION_CONFIG.autoCreateFollowUpTasks,
+      soundNotifications: userReflectionConfig.soundNotifications ?? DEFAULT_REFLECTION_CONFIG.soundNotifications
+    };
+    return mergedConfig;
+  } catch (error) {
+    if (error instanceof SyntaxError) {
+      throw new Error(`Invalid JSON in config file: ${configPath}. ${error.message}`);
+    }
+    throw error;
+  }
+}
+function validateReflectionConfig(config) {
+  const errors = [];
+  if (config.enabled && config.mode !== "disabled") {
+    const hasEnabledCategory = Object.values(config.categories).some((enabled) => enabled);
+    if (!hasEnabledCategory) {
+      errors.push("At least one analysis category must be enabled when reflection is active");
+    }
+  }
+  if (!config.enabled && config.mode === "auto") {
+    errors.push('Reflection mode cannot be "auto" when reflection is disabled');
+  }
+  if (config.soundNotifications && !config.enabled) {
+    errors.push("Sound notifications require reflection to be enabled");
+  }
+  if (config.autoCreateFollowUpTasks && !config.storeReflections) {
+    errors.push("Auto-create follow-up tasks requires storeReflections to be enabled");
+  }
+  return {
+    valid: errors.length === 0,
+    errors
+  };
+}
+function loadAndValidateReflectionConfig(projectRoot) {
+  const config = loadReflectionConfig(projectRoot);
+  const validation = validateReflectionConfig(config);
+  if (!validation.valid) {
+    throw new Error(
+      `Invalid reflection configuration:
+${validation.errors.map((e) => `  - ${e}`).join("\n")}`
+    );
+  }
+  return config;
+}
+export {
+  findSpecweaveRoot,
+  loadAndValidateReflectionConfig,
+  loadReflectionConfig,
+  validateReflectionConfig
+};

package/plugins/specweave/lib/hooks/reflection-config-loader.ts.bak

@@ -0,0 +1,156 @@
+/**
+ * Reflection Configuration Loader
+ *
+ * Loads and validates reflection configuration from .specweave/config.json
+ * Merges user config with defaults, validates against schema
+ *
+ * @module reflection-config-loader
+ */
+
+import fs from 'fs-extra';
+import path from 'path';
+import { DEFAULT_REFLECTION_CONFIG, ReflectionConfig } from './types/reflection-types';
+
+/**
+ * Find .specweave directory by traversing up from current directory
+ * @param startDir Starting directory (defaults to cwd)
+ * @returns Path to .specweave directory or null if not found
+ */
+export function findSpecweaveRoot(startDir: string = process.cwd()): string | null {
+  let currentDir = startDir;
+  const root = path.parse(currentDir).root;
+
+  while (currentDir !== root) {
+    const specweavePath = path.join(currentDir, '.specweave');
+    if (fs.existsSync(specweavePath)) {
+      return currentDir;
+    }
+    currentDir = path.dirname(currentDir);
+  }
+
+  return null;
+}
+
+/**
+ * Load reflection configuration from .specweave/config.json
+ * Falls back to defaults if config file doesn't exist or reflection section is missing
+ *
+ * @param projectRoot Path to project root (optional, auto-detected if not provided)
+ * @returns Merged reflection configuration
+ * @throws Error if config file exists but has invalid JSON
+ */
+export function loadReflectionConfig(projectRoot?: string): ReflectionConfig {
+  // Auto-detect project root if not provided
+  const rootDir = projectRoot || findSpecweaveRoot();
+
+  if (!rootDir) {
+    // No .specweave directory found, return defaults
+    return { ...DEFAULT_REFLECTION_CONFIG };
+  }
+
+  const configPath = path.join(rootDir, '.specweave', 'config.json');
+
+  // Config file doesn't exist, return defaults
+  if (!fs.existsSync(configPath)) {
+    return { ...DEFAULT_REFLECTION_CONFIG };
+  }
+
+  try {
+    // Read and parse config file
+    const configContent = fs.readFileSync(configPath, 'utf-8');
+    const config = JSON.parse(configContent);
+
+    // Extract reflection section (may be undefined)
+    const userReflectionConfig = config.reflection || {};
+
+    // Deep merge with defaults (user config overrides defaults)
+    const mergedConfig: ReflectionConfig = {
+      enabled: userReflectionConfig.enabled ?? DEFAULT_REFLECTION_CONFIG.enabled,
+      mode: userReflectionConfig.mode ?? DEFAULT_REFLECTION_CONFIG.mode,
+      depth: userReflectionConfig.depth ?? DEFAULT_REFLECTION_CONFIG.depth,
+      model: userReflectionConfig.model ?? DEFAULT_REFLECTION_CONFIG.model,
+      categories: {
+        security: userReflectionConfig.categories?.security ?? DEFAULT_REFLECTION_CONFIG.categories.security,
+        quality: userReflectionConfig.categories?.quality ?? DEFAULT_REFLECTION_CONFIG.categories.quality,
+        testing: userReflectionConfig.categories?.testing ?? DEFAULT_REFLECTION_CONFIG.categories.testing,
+        performance: userReflectionConfig.categories?.performance ?? DEFAULT_REFLECTION_CONFIG.categories.performance,
+        technicalDebt: userReflectionConfig.categories?.technicalDebt ?? DEFAULT_REFLECTION_CONFIG.categories.technicalDebt,
+      },
+      criticalThreshold: userReflectionConfig.criticalThreshold ?? DEFAULT_REFLECTION_CONFIG.criticalThreshold,
+      storeReflections: userReflectionConfig.storeReflections ?? DEFAULT_REFLECTION_CONFIG.storeReflections,
+      autoCreateFollowUpTasks: userReflectionConfig.autoCreateFollowUpTasks ?? DEFAULT_REFLECTION_CONFIG.autoCreateFollowUpTasks,
+      soundNotifications: userReflectionConfig.soundNotifications ?? DEFAULT_REFLECTION_CONFIG.soundNotifications,
+    };
+
+    return mergedConfig;
+  } catch (error) {
+    // Invalid JSON or other error
+    if (error instanceof SyntaxError) {
+      throw new Error(`Invalid JSON in config file: ${configPath}. ${error.message}`);
+    }
+    throw error;
+  }
+}
+
+/**
+ * Validate reflection configuration against constraints
+ * Checks for logical inconsistencies (e.g., all categories disabled)
+ *
+ * @param config Reflection configuration to validate
+ * @returns Validation result with errors array
+ */
+export function validateReflectionConfig(config: ReflectionConfig): {
+  valid: boolean;
+  errors: string[];
+} {
+  const errors: string[] = [];
+
+  // Check: If reflection enabled, at least one category must be enabled
+  if (config.enabled && config.mode !== 'disabled') {
+    const hasEnabledCategory = Object.values(config.categories).some(enabled => enabled);
+    if (!hasEnabledCategory) {
+      errors.push('At least one analysis category must be enabled when reflection is active');
+    }
+  }
+
+  // Check: Mode cannot be 'auto' if reflection is disabled
+  if (!config.enabled && config.mode === 'auto') {
+    errors.push('Reflection mode cannot be "auto" when reflection is disabled');
+  }
+
+  // Check: Sound notifications require reflection to be enabled
+  if (config.soundNotifications && !config.enabled) {
+    errors.push('Sound notifications require reflection to be enabled');
+  }
+
+  // Check: Auto-create follow-up tasks requires storeReflections
+  if (config.autoCreateFollowUpTasks && !config.storeReflections) {
+    errors.push('Auto-create follow-up tasks requires storeReflections to be enabled');
+  }
+
+  return {
+    valid: errors.length === 0,
+    errors
+  };
+}
+
+/**
+ * Load and validate reflection configuration
+ * Throws error if configuration is invalid
+ *
+ * @param projectRoot Path to project root (optional, auto-detected if not provided)
+ * @returns Valid reflection configuration
+ * @throws Error if configuration is invalid
+ */
+export function loadAndValidateReflectionConfig(projectRoot?: string): ReflectionConfig {
+  const config = loadReflectionConfig(projectRoot);
+  const validation = validateReflectionConfig(config);
+
+  if (!validation.valid) {
+    throw new Error(
+      `Invalid reflection configuration:\n${validation.errors.map(e => `  - ${e}`).join('\n')}`
+    );
+  }
+
+  return config;
+}

package/plugins/specweave/lib/hooks/reflection-parser.d.ts.bak

@@ -0,0 +1,33 @@
+/**
+ * Reflection Parser
+ *
+ * Parses markdown output from reflective-reviewer agent
+ * Extracts structured data (issues, metrics, lessons learned, etc.)
+ *
+ * @module reflection-parser
+ */
+import { ReflectionResult, ReflectionModel } from './types/reflection-types';
+/**
+ * Parse complete reflection result from markdown
+ * Main parsing function
+ *
+ * @param markdown Reflection markdown from reflective-reviewer agent
+ * @param taskName Task name for result
+ * @param model Model used for reflection
+ * @param reflectionTime Time taken in seconds
+ * @param estimatedCost Estimated cost in USD
+ * @returns Complete ReflectionResult object
+ */
+export declare function parseReflectionMarkdown(markdown: string, taskName: string, model?: ReflectionModel, reflectionTime?: number, estimatedCost?: number): ReflectionResult;
+/**
+ * Validate parsed reflection result
+ * Checks for required fields and data quality
+ *
+ * @param result Parsed reflection result
+ * @returns Validation result with errors
+ */
+export declare function validateReflectionResult(result: ReflectionResult): {
+    valid: boolean;
+    errors: string[];
+};
+//# sourceMappingURL=reflection-parser.d.ts.map

package/plugins/specweave/lib/hooks/reflection-parser.js.bak

@@ -0,0 +1,301 @@
+import {
+  IssueSeverity,
+  IssueCategory,
+  ReflectionModel
+} from "./types/reflection-types";
+function extractSection(markdown, heading) {
+  const headingRegex = new RegExp(`^${heading}\\s*$`, "mi");
+  const match = markdown.match(headingRegex);
+  if (!match || match.index === void 0) {
+    return "";
+  }
+  const startIndex = match.index + match[0].length;
+  const afterHeading = markdown.slice(startIndex);
+  const nextHeadingMatch = afterHeading.match(/^#{1,3}\s+/m);
+  const endIndex = nextHeadingMatch?.index ?? afterHeading.length;
+  return afterHeading.slice(0, endIndex).trim();
+}
+function parseAccomplishments(markdown) {
+  const section = extractSection(markdown, "## \u2705 What Was Accomplished");
+  if (!section) return [];
+  const lines = section.split("\n").filter((line) => line.trim());
+  const accomplishments = [];
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (/^[-*+]\s+/.test(trimmed) || /^\d+\.\s+/.test(trimmed)) {
+      accomplishments.push(trimmed.replace(/^[-*+\d.]\s+/, "").trim());
+    } else if (trimmed.length > 10 && !trimmed.startsWith("#")) {
+      accomplishments.push(trimmed);
+    }
+  }
+  return accomplishments;
+}
+function parseStrengths(markdown) {
+  const section = extractSection(markdown, "### \u2705 Strengths");
+  if (!section) return [];
+  const lines = section.split("\n").filter((line) => line.trim());
+  const strengths = [];
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (trimmed.startsWith("- \u2705") || trimmed.startsWith("\u2705")) {
+      strengths.push(trimmed.replace(/^[-*+]?\s*✅\s*/, "").trim());
+    }
+  }
+  return strengths;
+}
+function parseSeverity(text) {
+  if (/CRITICAL/i.test(text)) return IssueSeverity.CRITICAL;
+  if (/HIGH/i.test(text)) return IssueSeverity.HIGH;
+  if (/MEDIUM/i.test(text)) return IssueSeverity.MEDIUM;
+  if (/LOW/i.test(text)) return IssueSeverity.LOW;
+  return void 0;
+}
+function parseCategory(text) {
+  if (/SECURITY/i.test(text)) return IssueCategory.SECURITY;
+  if (/QUALITY/i.test(text)) return IssueCategory.QUALITY;
+  if (/TESTING/i.test(text)) return IssueCategory.TESTING;
+  if (/PERFORMANCE/i.test(text)) return IssueCategory.PERFORMANCE;
+  if (/TECHNICAL[_\s]DEBT/i.test(text)) return IssueCategory.TECHNICAL_DEBT;
+  return void 0;
+}
+function parseLocation(text) {
+  const locationMatch = text.match(/\*\*Location\*\*:\s*`([^`]+)`/i);
+  if (!locationMatch) return void 0;
+  const locationStr = locationMatch[1];
+  const [file, lineStr] = locationStr.split(":");
+  const line = lineStr ? parseInt(lineStr, 10) : void 0;
+  return {
+    file: file.trim(),
+    line
+  };
+}
+function parseIssues(markdown) {
+  const section = extractSection(markdown, "### \u26A0\uFE0F Issues Identified");
+  if (!section) return [];
+  const issues = [];
+  const lines = section.split("\n");
+  let currentIssue = null;
+  let currentField = null;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    const headerMatch = trimmed.match(/^\*\*(CRITICAL|HIGH|MEDIUM|LOW)\s*\(([^)]+)\)\*\*/i);
+    if (headerMatch) {
+      if (currentIssue && currentIssue.severity && currentIssue.category && currentIssue.description) {
+        issues.push(currentIssue);
+      }
+      currentIssue = {
+        severity: parseSeverity(headerMatch[1]),
+        category: parseCategory(headerMatch[2]),
+        description: "",
+        impact: "",
+        recommendation: ""
+      };
+      currentField = null;
+      continue;
+    }
+    if ((trimmed.startsWith("- \u274C") || trimmed.startsWith("\u274C") || trimmed.startsWith("- \u26A0\uFE0F")) && currentIssue) {
+      currentField = "description";
+      const descText = trimmed.replace(/^[-*+]?\s*[❌⚠️]\s*/, "").trim();
+      currentIssue.description = descText;
+      continue;
+    }
+    if (trimmed.startsWith("- **Impact**:") || trimmed.startsWith("**Impact**:")) {
+      currentField = "impact";
+      const impactText = trimmed.replace(/^[-*+]?\s*\*\*Impact\*\*:\s*/, "").trim();
+      if (currentIssue) currentIssue.impact = impactText;
+      continue;
+    }
+    if (trimmed.startsWith("- **Recommendation**:") || trimmed.startsWith("**Recommendation**:")) {
+      currentField = "recommendation";
+      const recText = trimmed.replace(/^[-*+]?\s*\*\*Recommendation\*\*:\s*/, "").trim();
+      if (currentIssue) currentIssue.recommendation = recText;
+      continue;
+    }
+    if (trimmed.startsWith("- **Location**:") || trimmed.startsWith("**Location**:")) {
+      if (currentIssue) {
+        const location = parseLocation(trimmed);
+        if (location) currentIssue.location = location;
+      }
+      currentField = null;
+      continue;
+    }
+    if (currentField && currentIssue && trimmed && !trimmed.startsWith("#")) {
+      const fieldValue = currentIssue[currentField] || "";
+      currentIssue[currentField] = fieldValue + " " + trimmed;
+    }
+  }
+  if (currentIssue && currentIssue.severity && currentIssue.category && currentIssue.description) {
+    issues.push(currentIssue);
+  }
+  return issues;
+}
+function parseRecommendedActions(markdown) {
+  const section = extractSection(markdown, "## \u{1F527} Recommended Follow-Up Actions");
+  const actions = {
+    priority1: [],
+    priority2: [],
+    priority3: []
+  };
+  if (!section) return actions;
+  const lines = section.split("\n");
+  let currentPriority = null;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (/Priority 1|MUST FIX/i.test(trimmed)) {
+      currentPriority = "priority1";
+      continue;
+    }
+    if (/Priority 2|SHOULD FIX/i.test(trimmed)) {
+      currentPriority = "priority2";
+      continue;
+    }
+    if (/Priority 3|NICE TO HAVE/i.test(trimmed)) {
+      currentPriority = "priority3";
+      continue;
+    }
+    if (currentPriority && (/^\d+\.\s/.test(trimmed) || /^[-*+]\s/.test(trimmed))) {
+      const actionText = trimmed.replace(/^(\d+\.|-|\*|\+)\s+/, "").trim();
+      if (actionText.length > 0) {
+        actions[currentPriority].push(actionText);
+      }
+    }
+  }
+  return actions;
+}
+function parseLessonsLearned(markdown) {
+  const section = extractSection(markdown, "## \u{1F4DA} Lessons Learned");
+  const lessons = {
+    whatWentWell: [],
+    whatCouldImprove: [],
+    forNextTime: []
+  };
+  if (!section) return lessons;
+  const lines = section.split("\n");
+  let currentCategory = null;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (/What went well/i.test(trimmed)) {
+      currentCategory = "whatWentWell";
+      continue;
+    }
+    if (/What could improve/i.test(trimmed)) {
+      currentCategory = "whatCouldImprove";
+      continue;
+    }
+    if (/For next time/i.test(trimmed)) {
+      currentCategory = "forNextTime";
+      continue;
+    }
+    if (currentCategory && /^[-*+]\s/.test(trimmed)) {
+      const text = trimmed.replace(/^[-*+]\s+/, "").trim();
+      if (text.length > 0) {
+        lessons[currentCategory].push(text);
+      }
+    }
+  }
+  return lessons;
+}
+function parseMetrics(markdown) {
+  const section = extractSection(markdown, "## \u{1F4CA} Metrics");
+  const metrics = {
+    codeQuality: 5,
+    security: 5,
+    testCoverage: void 0,
+    technicalDebt: "MEDIUM",
+    performance: "ACCEPTABLE"
+  };
+  if (!section) return metrics;
+  const qualityMatch = section.match(/Code Quality.*?(\d+)/i);
+  if (qualityMatch) {
+    metrics.codeQuality = parseInt(qualityMatch[1], 10);
+  }
+  const securityMatch = section.match(/Security.*?(\d+)/i);
+  if (securityMatch) {
+    metrics.security = parseInt(securityMatch[1], 10);
+  }
+  const coverageMatch = section.match(/Test Coverage.*?(\d+)%/i);
+  if (coverageMatch) {
+    metrics.testCoverage = parseInt(coverageMatch[1], 10);
+  }
+  const debtMatch = section.match(/Technical Debt.*?(LOW|MEDIUM|HIGH)/i);
+  if (debtMatch) {
+    metrics.technicalDebt = debtMatch[1].toUpperCase();
+  }
+  const perfMatch = section.match(/Performance.*?(GOOD|ACCEPTABLE|NEEDS[\s_]WORK)/i);
+  if (perfMatch) {
+    metrics.performance = perfMatch[1].toUpperCase().replace(/[\s_]/g, "_");
+  }
+  return metrics;
+}
+function parseReflectionMarkdown(markdown, taskName, model = ReflectionModel.HAIKU, reflectionTime = 0, estimatedCost = 0) {
+  const result = {
+    taskName,
+    completed: (/* @__PURE__ */ new Date()).toISOString(),
+    duration: reflectionTime > 0 ? `${reflectionTime}s` : void 0,
+    filesModified: {
+      count: 0,
+      linesAdded: 0,
+      linesRemoved: 0
+    },
+    accomplishments: parseAccomplishments(markdown),
+    strengths: parseStrengths(markdown),
+    issues: parseIssues(markdown),
+    recommendedActions: parseRecommendedActions(markdown),
+    lessonsLearned: parseLessonsLearned(markdown),
+    metrics: parseMetrics(markdown),
+    metadata: {
+      model,
+      reflectionTime,
+      estimatedCost
+    }
+  };
+  const filesChangedMatch = markdown.match(/\*\*Files Modified\*\*:\s*(\d+)\s*files?,\s*\+(\d+)\s*-(\d+)/i);
+  if (filesChangedMatch) {
+    result.filesModified = {
+      count: parseInt(filesChangedMatch[1], 10),
+      linesAdded: parseInt(filesChangedMatch[2], 10),
+      linesRemoved: parseInt(filesChangedMatch[3], 10)
+    };
+  }
+  return result;
+}
+function validateReflectionResult(result) {
+  const errors = [];
+  if (!result.taskName) {
+    errors.push("Missing task name");
+  }
+  if (!result.completed) {
+    errors.push("Missing completion timestamp");
+  }
+  if (result.accomplishments.length === 0 && result.strengths.length === 0) {
+    errors.push("No accomplishments or strengths identified (reflection too sparse)");
+  }
+  if (result.metrics.codeQuality < 1 || result.metrics.codeQuality > 10) {
+    errors.push("Code quality must be 1-10");
+  }
+  if (result.metrics.security < 1 || result.metrics.security > 10) {
+    errors.push("Security must be 1-10");
+  }
+  if (result.metrics.testCoverage !== void 0 && (result.metrics.testCoverage < 0 || result.metrics.testCoverage > 100)) {
+    errors.push("Test coverage must be 0-100%");
+  }
+  for (const issue of result.issues) {
+    if (!issue.description) {
+      errors.push("Issue missing description");
+    }
+    if (!issue.impact) {
+      errors.push("Issue missing impact explanation");
+    }
+    if (!issue.recommendation) {
+      errors.push("Issue missing recommendation");
+    }
+  }
+  return {
+    valid: errors.length === 0,
+    errors
+  };
+}
+export {
+  parseReflectionMarkdown,
+  validateReflectionResult
+};