wogiflow 2.31.1 → 2.31.2

package/package.json

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

package/scripts/flow-config-defaults.js

@@ -886,6 +886,33 @@ const CONFIG_DEFAULTS = {
     }
   },
 
+  // --- Deferral Gate (wf-f9912af6 + wf-b8839d99) ---
+  // wf-740f47e4 (DOCS-DRIFT): explicit defaults so config-schema.md isn't lying.
+  // The gate worked via inline fallbacks before, but config consumers couldn't
+  // discover the keys through the defaults loader.
+  deferralGate: {
+    enabled: true,
+    authTtlSeconds: 600,
+    classifyUserPrompts: true,
+    minClassifierConfidence: 75
+  },
+
+  // --- Self-Adversary Gate (wf-e399bd8d) ---
+  selfAdversaryGate: {
+    enabled: true,
+    targetConfidence: 95,
+    maxIterations: 8,
+    generatorModel: 'anthropic:claude-sonnet-4-6',
+    adversaryModel: 'anthropic:claude-3-5-haiku-latest'
+  },
+
+  // --- Research-Required Gate (wf-5cd71b1f) ---
+  researchRequiredGate: {
+    enabled: true,
+    requiredEvidence: 2,
+    maxAttempts: 3
+  },
+
   // --- Long Input Gate ---
   longInputGate: {
     enabled: true,

package/scripts/flow-self-adversary-loop.js

@@ -359,17 +359,22 @@ async function runSelfAdversaryLoop(opts = {}) {
       verdict
     });
 
-    // Termination checks
+    // Termination checks. wf-740f47e4 (L-1-RESIDUAL): adversary VERDICT is
+    // authoritative — confidence threshold alone cannot override 'revise'.
+    // Previously a second unconditional `if (adjustedConfidence >= target)`
+    // bypassed the verdict, accepting decisions the adversary explicitly
+    // wanted refined. The S-3 confidence-cap (+10 ceiling) limited damage
+    // but the verdict contract was still violated.
     if (verdict === 'needs-user') {
       return buildEscalate(candidate, iterationMemory, targetConfidence, 'adversary-says-needs-user');
     }
     if (verdict === 'accept' && adjustedConfidence >= targetConfidence) {
       return buildSuccess({ ...candidate, confidence: adjustedConfidence }, iterationMemory, targetConfidence);
     }
-    if (adjustedConfidence >= targetConfidence) {
-      return buildSuccess({ ...candidate, confidence: adjustedConfidence }, iterationMemory, targetConfidence);
-    }
-    // Otherwise loop again with the critique in memory
+    // verdict === 'revise' or any other value → continue iterating, even if
+    // adjustedConfidence is high. The adversary explicitly said "not yet";
+    // honor it. Only the loop-exhausted path (below) can ship a 'revise'
+    // decision — and even then it surfaces via buildEscalate, not Success.
   }
 
   // Max iterations exhausted without reaching threshold

package/scripts/hooks/core/deferral-gate.js

@@ -331,7 +331,9 @@ function stripQuotedContent(cmd) {
   // heredoc; longer than that, the gate fails open (no strip) which is safer than
   // ReDoS. Single unified terminator regex covers both EOL-anchored and word-
   // boundary cases; tolerates optional trailing whitespace/punctuation.
-  stripped = stripped.replace(/<<-?\s*['"]?(\w+)['"]?[\s\S]{0,8000}?\n\1(?:\s*[;)]?\s*$|\b)/gm, ' <<HEREDOC>> ');
+  // wf-740f47e4 (CRLF): accept both \n and \r\n terminators so Windows-style
+  // line endings in fixtures or test inputs don't bypass the strip.
+  stripped = stripped.replace(/<<-?\s*['"]?(\w+)['"]?[\s\S]{0,8000}?\r?\n\1(?:\s*[;)]?\s*$|\b)/gm, ' <<HEREDOC>> ');
   // Single-quoted strings
   stripped = stripped.replace(/'[^']*'/g, "''");
   // Backtick command substitution

package/scripts/hooks/core/gate-orchestrator.js

@@ -46,6 +46,22 @@ const REMEDIATION_LABELS = Object.freeze({
   'workspace-overdue': 'workspace-overdue (a worker dispatch is past its deadline)'
 });
 
+/**
+ * Internal: rank gate IDs by REMEDIATION_PRIORITY. Unknown gates sort to
+ * the bottom. Returns the IDs in priority order. wf-740f47e4 (DUAL API):
+ * extracted from pickTopRemediation + pickStopHookGate so the priority
+ * source-of-truth is single.
+ */
+function _rankByPriority(gateIds) {
+  return [...gateIds].sort((a, b) => {
+    const ia = REMEDIATION_PRIORITY.indexOf(a);
+    const ib = REMEDIATION_PRIORITY.indexOf(b);
+    const na = ia === -1 ? Number.POSITIVE_INFINITY : ia;
+    const nb = ib === -1 ? Number.POSITIVE_INFINITY : ib;
+    return na - nb;
+  });
+}
+
 /**
  * Pick the top-priority active remediation from a set of (gateId, message) pairs.
  *
@@ -58,16 +74,13 @@ function pickTopRemediation(active) {
   if (!Array.isArray(active) || active.length === 0) {
     return { top: null, queued: [] };
   }
-  // Filter to valid entries and sort by priority index.
   const valid = active.filter(g => g && typeof g.id === 'string' && typeof g.message === 'string' && g.message.trim().length > 0);
   if (valid.length === 0) return { top: null, queued: [] };
 
-  const indexed = valid.map(g => ({ ...g, idx: REMEDIATION_PRIORITY.indexOf(g.id) }))
-    .map(g => ({ ...g, idx: g.idx === -1 ? Number.POSITIVE_INFINITY : g.idx }));
-  indexed.sort((a, b) => a.idx - b.idx);
-
-  const top = { id: indexed[0].id, message: indexed[0].message };
-  const queued = indexed.slice(1).map(g => g.id);
+  const byId = new Map(valid.map(g => [g.id, g.message]));
+  const ranked = _rankByPriority(valid.map(g => g.id));
+  const top = { id: ranked[0], message: byId.get(ranked[0]) };
+  const queued = ranked.slice(1);
   return { top, queued };
 }
 
@@ -111,12 +124,12 @@ function selectAndRender(gateMap) {
  */
 function pickStopHookGate(activeFlags) {
   if (!activeFlags || typeof activeFlags !== 'object') return { topGateId: null, queued: [] };
-  const active = REMEDIATION_PRIORITY.filter(id => activeFlags[id] === true);
-  if (active.length === 0) return { topGateId: null, queued: [] };
-  return {
-    topGateId: active[0],
-    queued: active.slice(1)
-  };
+  // wf-740f47e4 (DUAL API): use the shared _rankByPriority helper so this
+  // and pickTopRemediation can never disagree on priority order.
+  const activeIds = Object.keys(activeFlags).filter(id => activeFlags[id] === true);
+  if (activeIds.length === 0) return { topGateId: null, queued: [] };
+  const ranked = _rankByPriority(activeIds);
+  return { topGateId: ranked[0], queued: ranked.slice(1) };
 }
 
 module.exports = {

package/scripts/hooks/core/stop-orchestrator.js

@@ -42,10 +42,13 @@ async function orchestrateStop({ parsedInput }) {
   let orchestratorTopGate = null;
   try {
     const { pickStopHookGate } = require('./gate-orchestrator');
-    const { topGateId } = pickStopHookGate({
+    // wf-740f47e4 (NULL-CHECK): guard against malformed return shape.
+    const result = pickStopHookGate({
       'long-input-pending': activeGates['long-input-pending'] === true
     });
-    orchestratorTopGate = topGateId;
+    orchestratorTopGate = (result && typeof result === 'object' && typeof result.topGateId === 'string')
+      ? result.topGateId
+      : null;
   } catch (_err) { /* fail-open */ }
   const longInputActive = orchestratorTopGate === 'long-input-pending';