wogiflow 1.8.5 → 1.8.7

package/.claude/settings.json

@@ -102,6 +102,6 @@
     ]
   },
   "_wogiFlowManaged": true,
-  "_wogiFlowVersion": "1.5.9",
+  "_wogiFlowVersion": "1.8.6",
   "_comment": "Shared WogiFlow hook configuration. Committed to repo for team use. User-specific overrides go in settings.local.json."
 }

package/.workflow/bridges/claude-bridge.js

@@ -327,16 +327,23 @@ Last synced: ${new Date().toISOString()}
   // Do not override - see security-patterns.md rule #2
 
   /**
-   * Get installed WogiFlow version from settings.json (set by postinstall)
+   * Get installed WogiFlow version from package.json (canonical source)
    * @returns {string} Version string or 'unknown'
    */
   _getInstalledVersion() {
     try {
-      const settingsPath = path.join(this.projectDir, this.cliFolder, 'settings.json');
-      const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf-8'));
-      return settings._wogiFlowVersion || 'unknown';
+      const pkgPath = path.join(this.projectDir, 'node_modules', 'wogiflow', 'package.json');
+      const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+      return pkg.version || 'unknown';
     } catch (err) {
-      return 'unknown';
+      // Fallback: try settings.json (legacy)
+      try {
+        const settingsPath = path.join(this.projectDir, this.cliFolder, 'settings.json');
+        const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf-8'));
+        return settings._wogiFlowVersion || 'unknown';
+      } catch (err) {
+        return 'unknown';
+      }
     }
   }
 

package/lib/installer.js

@@ -40,7 +40,6 @@ const DEFAULT_CONFIG = {
   version: PACKAGE_VERSION,
   projectName: '',
   cliKey: 'claude',
-  strictMode: true,
   projectType: 'unknown'
 };
 
@@ -77,7 +76,17 @@ function getDefaultConfig(overrides = {}) {
         small: { maxFiles: 3, maxHours: 1 },
         medium: { maxFiles: 10, maxHours: 4 },
         large: { minFiles: 10, minHours: 4 }
-      }
+      },
+      taskGating: { enabled: true, blockWithoutTask: true, autoCreateTask: false },
+      scopeGating: {
+        enabled: true,
+        mode: 'warn',
+        exemptPatterns: ['.workflow/state/**', '.workflow/specs/**', '.workflow/plans/**', 'package.json', 'package-lock.json', 'tsconfig.json']
+      },
+      implementationGate: { enabled: true },
+      todoWriteGate: { enabled: true, blockImplementationWithoutTask: true },
+      routingGate: { enabled: true },
+      loopEnforcement: { enabled: true }
     },
     execution: {
       maxIterations: 20,
@@ -104,6 +113,12 @@ function getDefaultConfig(overrides = {}) {
         simpleMode: { enabled: false },
         recheckAllAfterFix: true,
         regressionOnRecheck: 'warn'
+      },
+      tdd: {
+        enforced: true,
+        defaultForTypes: ['bugfix'],
+        requireFailingTestFirst: true,
+        testFrameworkDetection: true
       }
     },
     errorRecovery: {
@@ -114,8 +129,7 @@ function getDefaultConfig(overrides = {}) {
       maxAttemptsPerLevel: 3,
       architecturalReassessment: { enabled: false },
       recursive: { enabled: false },
-      hypothesisGeneration: { usePatterns: true, useAI: false, aiModel: 'haiku' },
-      learning: { recordSuccessfulFixes: true, recordFailedHypotheses: true }
+      hypothesisGeneration: { usePatterns: true, useAI: false, aiModel: 'haiku' }
     },
     workflow: { planningStyle: 'feature-based', agentStructure: 'unified' },
     parallelExecution: {
@@ -195,30 +209,24 @@ function getDefaultConfig(overrides = {}) {
       scopeByTaskType: true,
       alwaysCheck: ['naming', 'security'],
       similarityThreshold: 0.8,
-      similarityWarningThreshold: 0.6,
-      learning: { enabled: false }
-    },
-    validation: {
-      afterFileEdit: { enabled: false },
-      afterTaskComplete: { enabled: false },
-      beforeCommit: { enabled: false }
+      similarityWarningThreshold: 0.6
     },
     checkpoint: { enabled: false },
     regressionTesting: { enabled: false },
-    tdd: {
-      enforced: true,
-      defaultForTypes: ['bugfix'],
-      requireFailingTestFirst: true,
-      testFrameworkDetection: true
-    },
-    componentRules: {
+    componentReuse: {
+      enabled: true,
       preferVariants: true,
       requireAppMapEntry: true,
       requireDetailDoc: false,
       autoGenerateStorybook: false,
-      storybookPath: 'src/stories'
+      storybookPath: 'src/stories',
+      threshold: 30,
+      allRegistries: true,
+      aiAsJudge: true,
+      blockOnSimilar: false,
+      injectContext: true
     },
-    strictMode: {
+    reporting: {
       verificationChecklist: false,
       correctionReportsOnFail: false,
       featureReportsOnComplete: false
@@ -249,7 +257,10 @@ function getDefaultConfig(overrides = {}) {
         modelSpecificLearning: true
       },
       modelAdapters: { enabled: false },
-      skill: { enabled: false }
+      skill: { enabled: false },
+      standardsLearning: { enabled: false },
+      errorRecoveryLearning: { enabled: false },
+      bugFlowLearning: { enabled: false }
     },
     bugFlow: {
       investigationAgents: {
@@ -258,7 +269,6 @@ function getDefaultConfig(overrides = {}) {
         dependencyAnalyzer: { enabled: false }
       },
       autoRoute: true,
-      learningEnforcement: { enabled: false },
       inlineDiscovery: {
         maxSearchOperations: 3,
         maxFileReads: 2,
@@ -299,7 +309,7 @@ function getDefaultConfig(overrides = {}) {
       promotion: { enabled: false }
     },
     prd: { enabled: false },
-    context: {
+    contextManagement: {
       compaction: {
         enabled: false,
         thresholds: { warnAt: 50000, compactAt: 80000, maxExpanded: 20000 },
@@ -328,8 +338,9 @@ function getDefaultConfig(overrides = {}) {
         useHaiku: true,
         phases: ['exploring', 'spec_review', 'scenario', 'criteria_check', 'validating']
       },
-      scoring: { enabled: false },
-      monitor: { enabled: false },
+      monitor: { enabled: false }
+    },
+    taskContext: {
       auto: {
         enabled: false,
         strategy: 'dynamic',
@@ -346,6 +357,7 @@ function getDefaultConfig(overrides = {}) {
         fallbackLimits: { maxFilesHard: 50, maxTokensHard: 150000 },
         lspEnrichment: { enabled: false }
       },
+      scoring: { enabled: false },
       session: { enabled: false }
     },
     eval: {
@@ -525,34 +537,23 @@ function getDefaultConfig(overrides = {}) {
     },
     multiApproach: { enabled: false },
     hooks: {
-      enabled: false,
+      enabled: true,
       targets: ['claude-code'],
       gracefulDegradation: true,
       timeout: 600000,
       rules: {
-        enforcement: {
-          taskGating: { enabled: false, blockWithoutTask: true, autoCreateTask: false },
-          scopeGating: {
-            enabled: false,
-            mode: 'warn',
-            exemptPatterns: ['.workflow/state/**', '.workflow/specs/**', '.workflow/plans/**', 'package.json', 'package-lock.json', 'tsconfig.json']
-          },
-          implementationGate: { enabled: false },
-          todoWriteGate: { enabled: false, blockImplementationWithoutTask: true },
-          routingGate: { enabled: false },
-          loopEnforcement: { enabled: false }
-        },
         intelligence: {
-          componentReuse: { enabled: false, threshold: 30, allRegistries: true, aiAsJudge: true, blockOnSimilar: false, injectContext: true },
-          sessionContext: { enabled: false, loadSuspendedTasks: true, loadDecisions: true, loadRecentActivity: true },
-          validation: { enabled: false, runAfterEdit: true }
+          sessionContext: { enabled: true, loadSuspendedTasks: true, loadDecisions: true, loadRecentActivity: true },
+          promptCapture: { enabled: true },
+          correctionDetection: { enabled: true },
+          validation: { enabled: true, runAfterEdit: true, afterTaskComplete: false, beforeCommit: false }
         },
         lifecycle: {
-          taskCompleted: { enabled: false },
+          taskCompleted: { enabled: true },
           completionSummaries: { enabled: true },
           autoLogging: { enabled: false },
           configChange: { enabled: false },
-          setup: { enabled: false, autoOnboard: false, maintenanceTasks: ['healthCheck', 'cleanupLocks'] },
+          setup: { enabled: true, autoOnboard: false, maintenanceTasks: ['healthCheck', 'cleanupLocks'] },
           sessionCleanup: { enabled: true },
           phaseGate: { enabled: true }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wogiflow",
-  "version": "1.8.5",
+  "version": "1.8.7",
   "description": "AI-powered development workflow management system with multi-model support",
   "main": "lib/index.js",
   "bin": {

package/scripts/flow-auto-context.js

@@ -356,7 +356,8 @@ function searchComponentIndex(keywords, config = null) {
 
   // Use config values if available
   const cfg = config || getConfig();
-  const maxComponentMatches = cfg.context?.auto?.maxComponentMatches || 15;
+  const autoCfg = cfg.taskContext?.auto || cfg.context?.auto;
+  const maxComponentMatches = autoCfg?.maxComponentMatches || 15;
 
   try {
     // Use safeJsonParse for prototype pollution protection
@@ -419,8 +420,9 @@ function grepCodebase(keywords, maxResults = 10, config = null) {
 
   // Use config values if available
   const cfg = config || getConfig();
-  const effectiveMaxResults = cfg.context?.auto?.maxGrepResults || maxResults;
-  const maxContentLines = cfg.context?.auto?.maxContentLines || 50;
+  const grepAutoCfg = cfg.taskContext?.auto || cfg.context?.auto;
+  const effectiveMaxResults = grepAutoCfg?.maxGrepResults || maxResults;
+  const maxContentLines = grepAutoCfg?.maxContentLines || 50;
 
   // Only grep for high-value keywords to avoid noise
   const searchKeywords = keywords.high.slice(0, 5);
@@ -454,7 +456,7 @@ function grepCodebase(keywords, maxResults = 10, config = null) {
         if (!results.some(r => r.path === relPath)) {
           // Optionally read file content with truncation
           let content = null;
-          if (cfg.context?.auto?.includeContent && isPathWithinProject(file)) {
+          if (grepAutoCfg?.includeContent && isPathWithinProject(file)) {
             try {
               const fullContent = fs.readFileSync(file, 'utf-8');
               const lines = fullContent.split('\n');
@@ -558,7 +560,8 @@ async function searchSemanticMemory(keywords, config = null) {
   if (!cfg.memory?.enabled) return [];
 
   const results = [];
-  const maxFacts = cfg.context?.auto?.maxSemanticFacts || 5;
+  const semAutoCfg = cfg.taskContext?.auto || cfg.context?.auto;
+  const maxFacts = semAutoCfg?.maxSemanticFacts || 5;
 
   try {
     // Build query from high-value keywords
@@ -604,7 +607,8 @@ async function searchSemanticMemory(keywords, config = null) {
  */
 async function enrichWithLSP(fileResults, config) {
   // Skip if disabled
-  if (!config.context?.auto?.lspEnrichment?.enabled) return fileResults;
+  const autoConfig = config.taskContext?.auto || config.context?.auto;
+  if (!autoConfig?.lspEnrichment?.enabled) return fileResults;
 
   // Lazy load LSP module to avoid circular dependencies
   let getLSP, isLSPEnabled;
@@ -621,8 +625,8 @@ async function enrichWithLSP(fileResults, config) {
   const lsp = await getLSP();
   if (!lsp) return fileResults;
 
-  const timeout = config.context?.auto?.lspEnrichment?.timeoutMs || 2000;
-  const maxFiles = config.context?.auto?.lspEnrichment?.maxFiles || 5;
+  const timeout = autoConfig?.lspEnrichment?.timeoutMs || 2000;
+  const maxFiles = autoConfig?.lspEnrichment?.maxFiles || 5;
 
   // Only enrich top N JS/TS files to limit latency
   const filesToEnrich = fileResults
@@ -688,12 +692,13 @@ function searchWithAstGrep(keywords, taskType = null, config = null) {
   const cfg = config || getConfig();
 
   // Skip if disabled or ast-grep not available
-  if (!cfg.context?.auto?.useAstGrep || !isAstGrepAvailable()) {
+  const astAutoCfg = cfg.taskContext?.auto || cfg.context?.auto;
+  if (!astAutoCfg?.useAstGrep || !isAstGrepAvailable()) {
     return [];
   }
 
   const results = [];
-  const maxResults = cfg.context?.auto?.maxAstGrepResults || 5;
+  const maxResults = astAutoCfg?.maxAstGrepResults || 5;
 
   try {
     // Determine search strategy based on task type
@@ -855,12 +860,13 @@ async function getAutoContext(description, options = {}) {
   const config = getConfig();
 
   // Check if auto-context is enabled
-  if (config.context?.auto?.enabled === false) {
+  const autoCtx = config.taskContext?.auto || config.context?.auto;
+  if (autoCtx?.enabled === false) {
     return { enabled: false, files: [], context: [] };
   }
 
   // v3.0: Use Smart Context System if strategy is 'dynamic'
-  const strategy = config.context?.auto?.strategy || 'fixed';
+  const strategy = autoCtx?.strategy || 'fixed';
   if (strategy === 'dynamic' && smartContextGatherer) {
     const smartResult = await getSmartContext(description, options);
     if (smartResult) {
@@ -897,8 +903,9 @@ async function getLegacyContext(description, options = {}, config = null) {
     checkAndRefreshIndex(config);
   }
 
-  const maxFiles = options.maxFiles || config.context?.auto?.maxFilesToLoad || 10;
-  const showFiles = options.showFiles ?? config.context?.auto?.showLoadedFiles ?? true;
+  const autoConf = config.taskContext?.auto || config.context?.auto;
+  const maxFiles = options.maxFiles || autoConf?.maxFilesToLoad || 10;
+  const showFiles = options.showFiles ?? autoConf?.showLoadedFiles ?? true;
 
   // Extract keywords
   const keywords = extractKeywords(description);
@@ -955,7 +962,8 @@ async function getLegacyContext(description, options = {}, config = null) {
   const enrichedUnique = await enrichWithLSP(unique, config);
 
   // Re-sort: prioritize files without errors (if LSP enrichment added data)
-  if (config.context?.auto?.lspEnrichment?.prioritizeHealthyFiles !== false) {
+  const autoLsp = config.taskContext?.auto || config.context?.auto;
+  if (autoLsp?.lspEnrichment?.prioritizeHealthyFiles !== false) {
     enrichedUnique.sort((a, b) => {
       // Files with LSP errors go to bottom
       const aErrors = a.lsp?.errorCount || 0;

package/scripts/flow-auto-learn.js

@@ -607,6 +607,7 @@ module.exports = {
   normalizeIssueToPattern,
   getAutoLearnConfig,
   loadAutoPatterns,
+  saveAutoPatterns,
   showStatus
 };
 

package/scripts/flow-config-loader.js

@@ -50,20 +50,20 @@ const KNOWN_CONFIG_KEYS = [
   // Core
   'version', 'projectName', 'cli', 'scripts', 'requireApproval',
   // Feature toggles
-  'autoLog', 'autoUpdateAppMap', 'strictMode',
+  'autoLog', 'autoUpdateAppMap', 'strictMode', 'reporting',
   // Execution
   'hybrid', 'parallel', 'worktree', 'enforcement', 'tasks', 'workflow',
   'loops', 'taskQueue', 'durableSteps', 'suspension', 'phases',
   // Quality & validation
   'qualityGates', 'testing', 'validation', 'specificationMode', 'tdd',
   // Components & registries
-  'componentRules', 'componentIndex', 'registries', 'functionRegistry', 'apiRegistry',
+  'componentRules', 'componentReuse', 'componentIndex', 'registries', 'functionRegistry', 'apiRegistry',
   // Learning & memory
   'learning', 'corrections', 'automaticMemory', 'automaticPromotion',
   'crossSessionLearning', 'sessionLearning', 'skillLearning', 'memory',
   'codebaseInsights', 'knowledgeRouting',
   // Skills & context
-  'skills', 'autoContext', 'context', 'contextMonitor', 'contextScoring',
+  'skills', 'autoContext', 'context', 'contextManagement', 'taskContext', 'contextMonitor', 'contextScoring',
   // Review & analysis
   'review', 'reviewFix', 'originTaskTracing', 'standardsCompliance',
   'semanticMatching', 'peerReview', 'triage', 'consistency',
@@ -162,18 +162,18 @@ function validateConfig(config, warnOnUnknown = true) {
 function applyConfigCompatShim(config) {
   if (!config || typeof config !== 'object') return config;
 
-  // execution <-> tasks + loops (bidirectional)
+  // execution <-> tasks + loops (bidirectional, shallow copy to avoid shared references)
   if (config.execution && !config.tasks) {
-    config.tasks = config.execution;
+    config.tasks = { ...config.execution };
   }
   if (config.tasks && !config.execution) {
-    config.execution = config.tasks;
+    config.execution = { ...config.tasks };
   }
   if (config.execution && config.execution.loops && !config.loops) {
-    config.loops = config.execution.loops;
+    config.loops = { ...config.execution.loops };
   }
   if (config.loops && config.execution && !config.execution.loops) {
-    config.execution.loops = config.loops;
+    config.execution.loops = { ...config.loops };
   }
 
   // memory <- memory.automatic, memory.promotion
@@ -276,6 +276,87 @@ function applyConfigCompatShim(config) {
     config.communitySync = config.community.sync;
   }
 
+  // v1.8.6: Config restructuring compat shims
+  // AC1: hooks.rules.enforcement -> enforcement (moved to top-level enforcement)
+  if (config.hooks?.rules?.enforcement) {
+    const hooksEnforcement = config.hooks.rules.enforcement;
+    if (!config.enforcement) config.enforcement = {};
+    if (hooksEnforcement.taskGating && !config.enforcement.taskGating) config.enforcement.taskGating = hooksEnforcement.taskGating;
+    if (hooksEnforcement.scopeGating && !config.enforcement.scopeGating) config.enforcement.scopeGating = hooksEnforcement.scopeGating;
+    if (hooksEnforcement.implementationGate && !config.enforcement.implementationGate) config.enforcement.implementationGate = hooksEnforcement.implementationGate;
+    if (hooksEnforcement.todoWriteGate && !config.enforcement.todoWriteGate) config.enforcement.todoWriteGate = hooksEnforcement.todoWriteGate;
+    if (hooksEnforcement.routingGate && !config.enforcement.routingGate) config.enforcement.routingGate = hooksEnforcement.routingGate;
+    if (hooksEnforcement.loopEnforcement && !config.enforcement.loopEnforcement) config.enforcement.loopEnforcement = hooksEnforcement.loopEnforcement;
+  }
+
+  // AC3: strictMode object -> reporting
+  if (config.strictMode && typeof config.strictMode === 'object' && !config.reporting) {
+    config.reporting = config.strictMode;
+  }
+
+  // AC4: componentRules + hooks.rules.intelligence.componentReuse -> componentReuse
+  if (config.componentRules && !config.componentReuse) {
+    config.componentReuse = config.componentRules;
+  }
+  if (config.hooks?.rules?.intelligence?.componentReuse) {
+    if (!config.componentReuse) config.componentReuse = {};
+    const hooksComponentReuse = config.hooks.rules.intelligence.componentReuse;
+    for (const key of Object.keys(hooksComponentReuse)) {
+      if (config.componentReuse[key] === undefined) {
+        config.componentReuse[key] = hooksComponentReuse[key];
+      }
+    }
+  }
+  // Also expose back as hooks.rules.componentReuse for code that reads that path
+  if (config.componentReuse && config.hooks?.rules) {
+    if (!config.hooks.rules.componentReuse) config.hooks.rules.componentReuse = { ...config.componentReuse };
+  }
+
+  // AC5: scattered learning sub-keys -> learning.*
+  if (config.standardsCompliance?.learning && config.learning && !config.learning.standardsLearning) {
+    config.learning.standardsLearning = config.standardsCompliance.learning;
+  }
+  if (config.errorRecovery?.learning && config.learning && !config.learning.errorRecoveryLearning) {
+    config.learning.errorRecoveryLearning = config.errorRecovery.learning;
+  }
+  if (config.bugFlow?.learningEnforcement && config.learning && !config.learning.bugFlowLearning) {
+    config.learning.bugFlowLearning = config.bugFlow.learningEnforcement;
+  }
+
+  // AC6: context -> contextManagement + taskContext
+  // Note: mergeWithDefaults may have already created these keys, so check sub-keys not top-level
+  if (config.context) {
+    if (!config.contextManagement) config.contextManagement = {};
+    if (config.context.compaction && !config.contextManagement.compaction) config.contextManagement.compaction = config.context.compaction;
+    if (config.context.smart && !config.contextManagement.smart) config.contextManagement.smart = config.context.smart;
+    if (config.context.proactive && !config.contextManagement.proactive) config.contextManagement.proactive = config.context.proactive;
+    if (config.context.monitor && !config.contextManagement.monitor) config.contextManagement.monitor = config.context.monitor;
+
+    if (!config.taskContext) config.taskContext = {};
+    if (config.context.auto && !config.taskContext.auto) config.taskContext.auto = config.context.auto;
+    if (config.context.scoring && !config.taskContext.scoring) config.taskContext.scoring = config.context.scoring;
+    if (config.context.session && !config.taskContext.session) config.taskContext.session = config.context.session;
+  }
+
+  // AC7: tdd -> execution.tdd
+  if (config.tdd && config.execution && !config.execution.tdd) {
+    config.execution.tdd = config.tdd;
+  }
+  if (config.execution?.tdd && !config.tdd) {
+    config.tdd = config.execution.tdd;
+  }
+
+  // AC2: top-level validation -> hooks.rules.intelligence.validation
+  if (config.validation && config.hooks?.rules?.intelligence?.validation) {
+    const v = config.hooks.rules.intelligence.validation;
+    if (config.validation.afterTaskComplete !== undefined && v.afterTaskComplete === undefined) {
+      v.afterTaskComplete = config.validation.afterTaskComplete;
+    }
+    if (config.validation.beforeCommit !== undefined && v.beforeCommit === undefined) {
+      v.beforeCommit = config.validation.beforeCommit;
+    }
+  }
+
   return config;
 }
 

package/scripts/flow-constants.js

@@ -114,7 +114,7 @@ const KNOWN_CONFIG_KEYS = [
   'worktree',
   'qualityGates',
   'testing',
-  'componentRules',
+  'componentReuse',
   'mandatorySteps',
   'phases',
   'corrections',
@@ -136,6 +136,9 @@ const KNOWN_CONFIG_KEYS = [
   'damageControl',
   'storyDecomposition',
   'specificationMode',
+  'contextManagement',
+  'taskContext',
+  'reporting',
 ];
 
 module.exports = {

package/scripts/flow-context-compact/expander.js

@@ -276,7 +276,7 @@ function collapseAll() {
  */
 function checkContextPressure() {
   const config = getConfig();
-  const compactionConfig = config.context?.compaction || {};
+  const compactionConfig = config.contextManagement?.compaction || config.context?.compaction || {};
   const thresholds = compactionConfig.thresholds || {
     warnAt: 50000,
     compactAt: 80000,

package/scripts/flow-context-compact/section-extractor.js

@@ -125,7 +125,7 @@ function selectNodes(tree, options = {}) {
   } = options;
 
   const config = getConfig();
-  const compactionConfig = config.context?.compaction || {};
+  const compactionConfig = config.contextManagement?.compaction || config.context?.compaction || {};
 
   const queryKeywords = extractKeywords(query);
   const selected = [];

package/scripts/flow-context-compact/summary-tree.js

@@ -259,7 +259,7 @@ function mergeIntoTree(existingTree, newData) {
 
   // Apply relevance decay to existing nodes
   const config = getConfig();
-  const compactionConfig = config.context?.compaction || DEFAULT_COMPACTION_CONFIG;
+  const compactionConfig = config.contextManagement?.compaction || config.context?.compaction || DEFAULT_COMPACTION_CONFIG;
 
   if (compactionConfig.relevanceDecay?.enabled) {
     const decayRate = compactionConfig.relevanceDecay.decayPerTurn || 0.05;