wogiflow 2.20.1 → 2.22.0

package/scripts/flow-step-regression.js

@@ -9,7 +9,7 @@
 
 const { execSync } = require('node:child_process');
 const path = require('node:path');
-const { getProjectRoot, PATHS } = require('./flow-utils');
+const { PATHS } = require('./flow-utils');
 
 /**
  * Run regression tests as a workflow step
@@ -37,7 +37,7 @@ async function run(options = {}) {
     let parsed;
     try {
       parsed = JSON.parse(result);
-    } catch (err) {
+    } catch (_err) {
       // Non-JSON output means success with no issues
       return { passed: true, message: 'Regression tests passed' };
     }
@@ -67,7 +67,7 @@ async function run(options = {}) {
             details: parsed.failures,
           };
         }
-      } catch (err) {
+      } catch (_err) {
         // Not JSON, treat as error
       }
     }

package/scripts/flow-step-review.js

@@ -142,7 +142,7 @@ function printIssue(issue) {
 /**
  * Run multi-agent review (3 perspectives)
  */
-async function runMultiAgentReview(files, config) {
+async function runMultiAgentReview(files, _config) {
   const allIssues = [];
 
   for (const file of files) {
@@ -165,7 +165,7 @@ async function runMultiAgentReview(files, config) {
       ]);
 
       allIssues.push(...fileIssues);
-    } catch (err) {
+    } catch (_err) {
       // Skip unreadable files
     }
   }
@@ -176,7 +176,7 @@ async function runMultiAgentReview(files, config) {
 /**
  * Run simple review (single pass)
  */
-async function runSimpleReview(files, config) {
+async function runSimpleReview(files, _config) {
   const allIssues = [];
 
   for (const file of files) {
@@ -187,7 +187,7 @@ async function runSimpleReview(files, config) {
       const content = fs.readFileSync(filePath, 'utf8');
       const fileIssues = runBasicChecks(content, file);
       allIssues.push(...fileIssues);
-    } catch (err) {
+    } catch (_err) {
       // Skip unreadable files
     }
   }
@@ -200,7 +200,7 @@ async function runSimpleReview(files, config) {
  */
 function reviewArchitecture(content, fileName) {
   const issues = [];
-  const lines = content.split('\n');
+  const _lines = content.split('\n');
 
   // Check for god objects (too many methods/properties)
   const methodCount = (content.match(/(?:function\s+\w+|(?:const|let|var)\s+\w+\s*=\s*(?:async\s*)?\()/g) || []).length;

package/scripts/flow-story.js

@@ -15,7 +15,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const {
-  getProjectRoot,
+  getProjectRoot: _getProjectRoot,
   colors,
   getConfig,
   getConfigValue,
@@ -26,7 +26,7 @@ const {
   safeJsonParse,
   isPathWithinProject, PATHS
 } = require('./flow-utils');
-const { success, warn, error, info, print, printHeader } = require('./flow-output');
+const { success, warn, error, info } = require('./flow-output');
 
 // Import context orchestrator for product context
 let contextOrchestrator = null;

package/scripts/flow-strict-adherence.js

@@ -20,7 +20,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const {
-  getProjectRoot,
+  getProjectRoot: _getProjectRoot,
   safeJsonParse,
   getConfig,
   colors,
@@ -500,7 +500,7 @@ function reviewCode(options = {}) {
 
   if (!isEnabled()) return { warnings };
 
-  const standards = loadStandards();
+  const _standards = loadStandards();
   const config = loadConfig();
 
   // Only run in user review mode

package/scripts/flow-structure-sensor.js

@@ -0,0 +1,283 @@
+#!/usr/bin/env node
+
+/**
+ * Wogi Flow — Structure-Change Sensor
+ *
+ * Detects folder-restructure patterns in a git diff range so /wogi-finalize
+ * and cross-repo audit workflows can flag merges that will produce
+ * modify/delete conflicts rather than routine content conflicts.
+ *
+ * Patterns detected (all evaluated against `git diff --name-status <range>`):
+ *
+ *   1. FOLDER_PER_COMPONENT   — `X.tsx` deleted  +  `X/X.tsx` added
+ *   2. SPLIT_INTO_SUBMODULE   — `X.ts`  deleted  +  one or more `<dir>/X-*.ts`
+ *                               or `<dir>/<other>.ts` added at deeper depth
+ *   3. BARREL_INTRODUCTION    — `X.ts`  deleted  +  `X/index.ts` added
+ *   4. RENAME_NEW_HOME        — single file deletion + single file addition,
+ *                               same basename, different directory
+ *
+ * The sensor is intentionally pattern-based and bounded: no full-diff parsing,
+ * no AST walks. Downstream callers (wogi-finalize merge-plan gate) use the
+ * output to drive a percentage threshold — when restructure-matched files
+ * exceed `restructureThreshold` (default 20%) of the changed set, a structural
+ * warning is surfaced.
+ *
+ * Usage (CLI):
+ *   flow-structure-sensor.js <base>..<branch>
+ *     → emits one JSON object on stdout
+ *
+ * Usage (module):
+ *   const { detectStructureChanges } = require('./flow-structure-sensor');
+ *   const result = detectStructureChanges({ base: 'master', branch: 'feature/x' });
+ *
+ * Exit codes:
+ *   0 — success (patterns may or may not be present; see result.warn)
+ *   1 — git invocation failed or invalid range
+ */
+
+'use strict';
+
+const { execFileSync } = require('node:child_process');
+const path = require('node:path');
+
+const DEFAULT_RESTRUCTURE_THRESHOLD = 0.20;
+
+/**
+ * Run `git diff --name-status` and return the parsed entries.
+ * Each entry is { status: 'A'|'D'|'M'|'R100'|..., path: string, origPath?: string }.
+ */
+function getDiffEntries({ range, cwd }) {
+  const args = ['diff', '--name-status', range];
+  let output;
+  try {
+    output = execFileSync('git', args, {
+      cwd: cwd || process.cwd(),
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    });
+  } catch (err) {
+    throw new Error(`git diff failed: ${err.message || err}`);
+  }
+  const entries = [];
+  for (const line of output.split('\n')) {
+    if (!line.trim()) continue;
+    const parts = line.split('\t');
+    const status = parts[0];
+    if (!status) continue;
+    // R/C statuses carry two paths: origPath, newPath
+    if (/^R/.test(status) || /^C/.test(status)) {
+      if (parts.length >= 3) {
+        entries.push({ status, origPath: parts[1], path: parts[2] });
+      }
+    } else {
+      entries.push({ status, path: parts[1] });
+    }
+  }
+  return entries;
+}
+
+/**
+ * Analyze entries for restructure patterns.
+ * Returns:
+ *   {
+ *     range, totalChanged, restructureCount, ratio, warn,
+ *     patterns: [{ type, deleted, added, basename }]
+ *   }
+ */
+function classify(entries) {
+  const deletions = entries.filter((e) => e.status === 'D').map((e) => e.path);
+  const additions = entries.filter((e) => e.status === 'A').map((e) => e.path);
+  const renames = entries.filter((e) => /^R/.test(e.status));
+
+  const patterns = [];
+  const restructureFiles = new Set();
+
+  const deletedByBasename = new Map();
+  for (const p of deletions) {
+    const base = path.basename(p, path.extname(p));
+    if (!deletedByBasename.has(base)) deletedByBasename.set(base, []);
+    deletedByBasename.get(base).push(p);
+  }
+  const addedByBasename = new Map();
+  for (const p of additions) {
+    const base = path.basename(p, path.extname(p));
+    if (!addedByBasename.has(base)) addedByBasename.set(base, []);
+    addedByBasename.get(base).push(p);
+  }
+
+  // 1. FOLDER_PER_COMPONENT: X.tsx deleted AND X/X.tsx (same basename/ext) added
+  for (const [base, dels] of deletedByBasename) {
+    const adds = addedByBasename.get(base) || [];
+    for (const del of dels) {
+      for (const add of adds) {
+        const delDir = path.dirname(del);
+        const addDir = path.dirname(add);
+        const delExt = path.extname(del);
+        const addExt = path.extname(add);
+        if (delExt !== addExt) continue;
+        // Folder-per-component: addDir = delDir + '/' + base
+        if (addDir === path.join(delDir, base)) {
+          patterns.push({ type: 'FOLDER_PER_COMPONENT', deleted: del, added: add, basename: base });
+          restructureFiles.add(del);
+          restructureFiles.add(add);
+        }
+      }
+    }
+  }
+
+  // 2. SPLIT_INTO_SUBMODULE: X.ts deleted AND 2+ files added under a dir
+  //    named X/ or at deeper depth carrying X's name as dir.
+  for (const [base, dels] of deletedByBasename) {
+    for (const del of dels) {
+      const delDir = path.dirname(del);
+      const delExt = path.extname(del);
+      const submoduleDir = path.join(delDir, base);
+      const adds = additions.filter(
+        (a) =>
+          path.extname(a) === delExt &&
+          (a.startsWith(submoduleDir + path.sep) || a.startsWith(submoduleDir + '/'))
+      );
+      // Need to see this as a split only if 2+ files land there — single file
+      // was already picked up as FOLDER_PER_COMPONENT above.
+      if (adds.length >= 2) {
+        patterns.push({ type: 'SPLIT_INTO_SUBMODULE', deleted: del, added: adds, basename: base });
+        restructureFiles.add(del);
+        for (const a of adds) restructureFiles.add(a);
+      }
+    }
+  }
+
+  // 3. BARREL_INTRODUCTION: X.ts deleted AND X/index.ts added (any extension match)
+  for (const [base, dels] of deletedByBasename) {
+    for (const del of dels) {
+      const delDir = path.dirname(del);
+      const delExt = path.extname(del);
+      const barrelPath = path.join(delDir, base, `index${delExt}`);
+      if (additions.includes(barrelPath)) {
+        patterns.push({ type: 'BARREL_INTRODUCTION', deleted: del, added: barrelPath, basename: base });
+        restructureFiles.add(del);
+        restructureFiles.add(barrelPath);
+      }
+    }
+  }
+
+  // 4. RENAME_NEW_HOME: git-reported renames across different directories
+  for (const r of renames) {
+    if (!r.origPath || !r.path) continue;
+    if (path.dirname(r.origPath) !== path.dirname(r.path)) {
+      patterns.push({
+        type: 'RENAME_NEW_HOME',
+        deleted: r.origPath,
+        added: r.path,
+        basename: path.basename(r.path, path.extname(r.path)),
+      });
+      restructureFiles.add(r.origPath);
+      restructureFiles.add(r.path);
+    }
+  }
+
+  const totalChanged = entries.length;
+  const ratio = totalChanged > 0 ? restructureFiles.size / totalChanged : 0;
+
+  return {
+    totalChanged,
+    restructureCount: restructureFiles.size,
+    ratio,
+    patterns,
+  };
+}
+
+/**
+ * Public API — detect structure changes for a git range.
+ *
+ * @param {Object} opts
+ * @param {string} opts.range - Git range (e.g., "master..feature/foo" or a single SHA range).
+ *                              If opts.base+opts.branch given, they combine as "base..branch".
+ * @param {string} [opts.base]
+ * @param {string} [opts.branch]
+ * @param {string} [opts.cwd]
+ * @param {number} [opts.threshold=0.20] - Ratio above which warn=true.
+ * @returns {Object}
+ */
+function detectStructureChanges(opts = {}) {
+  const range = opts.range || (opts.base && opts.branch ? `${opts.base}..${opts.branch}` : null);
+  if (!range || typeof range !== 'string') {
+    return {
+      ok: false,
+      reason: 'missing-range',
+      totalChanged: 0,
+      restructureCount: 0,
+      ratio: 0,
+      warn: false,
+      patterns: [],
+    };
+  }
+  const threshold = typeof opts.threshold === 'number' ? opts.threshold : DEFAULT_RESTRUCTURE_THRESHOLD;
+
+  let entries;
+  try {
+    entries = getDiffEntries({ range, cwd: opts.cwd });
+  } catch (err) {
+    return {
+      ok: false,
+      reason: `git-failed: ${err.message}`,
+      range,
+      totalChanged: 0,
+      restructureCount: 0,
+      ratio: 0,
+      warn: false,
+      patterns: [],
+    };
+  }
+
+  const classified = classify(entries);
+  const warn = classified.ratio >= threshold && classified.restructureCount >= 2;
+
+  return {
+    ok: true,
+    range,
+    threshold,
+    totalChanged: classified.totalChanged,
+    restructureCount: classified.restructureCount,
+    ratio: Number(classified.ratio.toFixed(4)),
+    warn,
+    patterns: classified.patterns,
+  };
+}
+
+// ============================================================
+// CLI
+// ============================================================
+
+function main() {
+  const argv = process.argv.slice(2);
+  if (argv.length === 0 || argv[0] === '--help' || argv[0] === '-h') {
+    console.log('Usage: flow-structure-sensor.js <range> [--threshold=N]');
+    console.log('  range      — git diff range, e.g. "master..feature/x" or "<sha1>..<sha2>"');
+    console.log('  --threshold=N   float 0–1, default 0.20');
+    process.exit(argv.length === 0 ? 1 : 0);
+  }
+  const range = argv[0];
+  let threshold = DEFAULT_RESTRUCTURE_THRESHOLD;
+  for (const a of argv.slice(1)) {
+    const m = a.match(/^--threshold=(.+)$/);
+    if (m) {
+      const n = Number(m[1]);
+      if (Number.isFinite(n)) threshold = n;
+    }
+  }
+  const result = detectStructureChanges({ range, threshold });
+  process.stdout.write(JSON.stringify(result, null, 2) + '\n');
+  process.exit(result.ok ? 0 : 1);
+}
+
+if (require.main === module) {
+  main();
+}
+
+module.exports = {
+  detectStructureChanges,
+  classify,
+  getDiffEntries,
+  DEFAULT_RESTRUCTURE_THRESHOLD,
+};

package/scripts/flow-sync-anonymizer.js

@@ -103,14 +103,14 @@ function anonymizeBatch(records) {
  */
 function createUploadPayload(params) {
   const { records, capabilityScores, routingEffectiveness } = params;
-  const config = getConfig();
+  const _config = getConfig();
 
   // Get WogiFlow version from package.json
   let version = 'unknown';
   try {
     const pkg = require(path.join(PATHS.root, 'package.json'));
     version = pkg.version || 'unknown';
-  } catch (err) {
+  } catch (_err) {
     // Non-critical
   }
 
@@ -209,7 +209,7 @@ function main() {
       const sample = [
         {
           taskId: 'wf-abc123',
-          model: 'claude-opus-4-6',
+          model: 'claude-opus-4-7',
           taskType: 'feature',
           iterations: 2,
           firstAttemptPass: true,

package/scripts/flow-task-checkpoint.js

@@ -14,10 +14,10 @@
  * Part of S1: Smart Context Compaction
  */
 
-const fs = require('node:fs');
+const _fs = require('node:fs');
 const path = require('node:path');
 const {
-  getConfig,
+  getConfig: _getConfig,
   STATE_DIR,
   readJson,
   writeJson,

package/scripts/flow-task-classifier.js

@@ -25,7 +25,7 @@ const {
   PROJECT_ROOT,
   readFile,
   fileExists,
-  info,
+  info: _info,
   warn,
   parseFlags,
   outputJson
@@ -113,7 +113,7 @@ const TASK_TYPE_PATTERNS = {
  * @param {Object} options - Classification options
  * @returns {Object} - Classification result with type and confidence
  */
-function classifyTask(taskDescription, affectedFiles = [], options = {}) {
+function classifyTask(taskDescription, affectedFiles = [], _options = {}) {
   const descLower = taskDescription.toLowerCase();
   const scores = {};
 

package/scripts/flow-task-completion-summary.js

@@ -12,7 +12,7 @@
 const path = require('node:path');
 const fs = require('node:fs');
 const { execFileSync } = require('node:child_process');
-const { getConfig, PATHS, safeJsonParse, readFile, writeFile, ensureDir, isPathWithinProject, validateTaskId } = require('./flow-utils');
+const { getConfig, PATHS, safeJsonParse, writeFile, ensureDir, isPathWithinProject, validateTaskId } = require('./flow-utils');
 
 /**
  * Check if completion summaries are enabled in config

package/scripts/flow-task-enforcer.js

@@ -14,7 +14,7 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
-const { getConfig, getProjectRoot, writeJson, readJson, safeJsonParse, PATHS } = require('./flow-utils');
+const { getConfig, getProjectRoot, writeJson, readJson, safeJsonParse } = require('./flow-utils');
 const { getCommand } = require('./flow-script-resolver');
 const { sanitizeShellArg } = require('./flow-security');
 
@@ -84,7 +84,7 @@ function isRecheckEnabled() {
  * Returns { allowed: boolean, message: string }
  */
 function canSkipCriterion(criterionId, approvalGiven = false) {
-  const config = getConfig();
+  const _config = getConfig();
   const session = getActiveLoop();
 
   if (!session) {
@@ -150,7 +150,7 @@ function startLoop(taskId, acceptanceCriteria) {
 
   // v2.0: Use durable session if enabled
   if (config.durableSteps?.enabled !== false) {
-    const session = durableSession.createDurableSession(taskId, 'task', acceptanceCriteria);
+    const _session = durableSession.createDurableSession(taskId, 'task', acceptanceCriteria);
     // Return backward-compatible format
     return durableSession.getActiveLoop();
   }
@@ -753,7 +753,7 @@ function getLoopStats() {
 function verifyCriterion(criterion, context = {}) {
   const { execSync, execFileSync } = require('node:child_process');
   const { changedFiles = [], testResults = null, lintResults = null } = context;
-  const config = getConfig();
+  const _config = getConfig();
   const taskConfig = getTaskConfig();
   const desc = criterion.description;
   const descLower = desc.toLowerCase();
@@ -848,7 +848,7 @@ function verifyCriterion(criterion, context = {}) {
             }
           }
         }
-      } catch (err) { /* ignore permission errors */ }
+      } catch (_err) { /* ignore permission errors */ }
       return results;
     }
 

package/scripts/flow-tech-debt.js

@@ -32,13 +32,13 @@ const {
   colors: c,
   safeJsonParse,
   writeJson,
-  fileExists,
+  fileExists: _fileExists,
   getProjectRoot,
   ensureDir,
-  generateTaskId, PATHS,
+  generateTaskId, PATHS: _PATHS,
   getTodayDate
 } = require('./flow-utils');
-const { success: printSuccess, warn: printWarn } = require('./flow-output');
+const { warn: printWarn } = require('./flow-output');
 
 // ============================================================================
 // Constants

package/scripts/flow-template-extractor.js

@@ -289,7 +289,7 @@ function generateTemplate(file, type) {
   const content = file.content;
   const lines = content.split('\n');
   const templateLines = [];
-  let insideBody = false;
+  let _insideBody = false;
   let braceDepth = 0;
   let bodyStartDepth = 0;
   let skipUntilCloseBrace = false;
@@ -379,10 +379,10 @@ function generateTemplate(file, type) {
 /**
  * Check if a line is structural (should always be kept)
  */
-function isStructuralLine(trimmed, type) {
+function isStructuralLine(trimmed, _type) {
   // Import statements
   if (trimmed.startsWith('import ') || trimmed.startsWith('const ') && trimmed.includes('require(')) return true;
-const { PATHS } = require('./flow-utils');
+const { } = require('./flow-utils');
   if (trimmed.startsWith('from ')) return true;
 
   // Export statements

package/scripts/flow-templates.js

@@ -13,7 +13,7 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
-const { getProjectRoot, readJson, info, PATHS } = require('./flow-utils');
+const { readJson, info, PATHS } = require('./flow-utils');
 
 const TEMPLATES_DIR = path.join(PATHS.root, 'templates', 'hybrid');
 

package/scripts/flow-test-api.js

@@ -19,14 +19,14 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { spawn } = require('node:child_process');
-const { getProjectRoot, PATHS, ensureDir, safeJsonParse, safeJsonParseString } = require('./flow-utils');
+const { PATHS, ensureDir, safeJsonParse, safeJsonParseString } = require('./flow-utils');
 const { getConfig } = require('./flow-config-loader');
 const { loadProfile } = require('./flow-verification-profile');
 
 let scenarioEngine;
 try {
   scenarioEngine = require('./flow-scenario-engine');
-} catch (err) {
+} catch (_err) {
   scenarioEngine = null;
 }
 
@@ -58,7 +58,7 @@ function parseAPIMap(apiMapPath) {
   let content;
   try {
     content = fs.readFileSync(apiMapPath, 'utf-8');
-  } catch (err) {
+  } catch (_err) {
     return [];
   }
 
@@ -137,7 +137,7 @@ function parseOpenAPISpec(specPath) {
   let content;
   try {
     content = fs.readFileSync(specPath, 'utf-8');
-  } catch (err) {
+  } catch (_err) {
     return { endpoints: [], raw: null };
   }
 
@@ -524,13 +524,13 @@ async function executeAPITest(testCase, baseUrl, options = {}) {
     if (contentType.includes('application/json')) {
       try {
         body = await response.json();
-      } catch (err) {
+      } catch (_err) {
         body = null;
       }
     } else {
       try {
         body = await response.text();
-      } catch (err) {
+      } catch (_err) {
         body = null;
       }
     }
@@ -665,7 +665,7 @@ async function startAPIServer(command, baseUrl, timeout = SERVER_READY_TIMEOUT)
         if (response.ok || response.status < 500) {
           return { process: serverProcess, ready: true, error: null };
         }
-      } catch (err) {
+      } catch (_err) {
         // Server not ready yet — continue polling
       }
     }
@@ -693,7 +693,7 @@ function stopAPIServer(serverProcess) {
     if (serverProcess.pid) {
       try {
         process.kill(-serverProcess.pid, 'SIGTERM');
-      } catch (err) {
+      } catch (_err) {
         // Process group kill failed — try direct kill
         try {
           serverProcess.kill('SIGTERM');
@@ -708,12 +708,12 @@ function stopAPIServer(serverProcess) {
           if (!serverProcess.killed) {
             process.kill(-serverProcess.pid, 'SIGKILL');
           }
-        } catch (err) {
+        } catch (_err) {
           // Already dead
         }
       }, 5000);
     }
-  } catch (err) {
+  } catch (_err) {
     // Process already terminated
   }
 }
@@ -1049,7 +1049,7 @@ function loadOrGenerateScenarios(taskId, options = {}) {
   let apiMapContent = null;
   try {
     apiMapContent = fs.readFileSync(apiMapPath, 'utf-8');
-  } catch (err) {
+  } catch (_err) {
     // No API map available
   }
 
@@ -1105,7 +1105,7 @@ async function runScenarioTests(taskId, scenarios, options = {}) {
     try {
       fs.writeFileSync(reportPath, JSON.stringify(report, null, 2));
       report.reportPath = reportPath;
-    } catch (err) {
+    } catch (_err) {
       // Non-fatal
     }
   }