wogiflow 2.20.1 → 2.22.0

package/scripts/hooks/core/session-end.js

@@ -38,7 +38,7 @@ function getUncommittedCount() {
       stdio: ['pipe', 'pipe', 'pipe']
     });
     return output.trim().split('\n').filter(line => line.trim()).length;
-  } catch (err) {
+  } catch (_err) {
     return 0;
   }
 }
@@ -131,7 +131,7 @@ function handleSessionEnd(input) {
           console.error(`[Session End] Community sync-up failed: ${err.message}`);
         }
       });
-    } catch (err) {
+    } catch (_err) {
       // Non-critical — community sync module may not be available
     }
 
@@ -144,7 +144,7 @@ function handleSessionEnd(input) {
           // Non-critical - memory pipeline may not be available
         });
       }
-    } catch (err) {
+    } catch (_err) {
       // Non-critical — memory DB may not be available
     }
 

package/scripts/hooks/core/session-history.js

@@ -31,7 +31,7 @@
  * `.workflow/archive/session-history-<date>.json`.
  */
 
-const fs = require('node:fs');
+const _fs = require('node:fs');
 const path = require('node:path');
 
 const { PATHS, safeJsonParse, writeJson } = require('../../flow-utils');

package/scripts/hooks/core/setup-handler.js

@@ -93,7 +93,7 @@ function handleSetup(options = {}) {
  * @param {Object} options
  * @returns {Object} Result: { tasks, message, results }
  */
-function handleMaintenance(options = {}) {
+function handleMaintenance(_options = {}) {
   if (!isSetupEnabled()) {
     return {
       tasks: [],

package/scripts/hooks/core/task-boundary-reset.js

@@ -47,6 +47,7 @@ const fs = require('node:fs');
 const path = require('node:path');
 
 const { getConfig, PATHS } = require('../../flow-utils');
+const { safeJsonParse } = require('../../flow-io');
 
 const PENDING_MARKER_FILE = 'task-just-completed';
 
@@ -157,10 +158,7 @@ function consumeAndTriggerRestart() {
   // Consume the marker. Do this BEFORE signaling so we never double-fire if
   // for some reason the signal delivery is delayed and a second Stop-hook
   // invocation races through.
-  let markerPayload = null;
-  try {
-    markerPayload = JSON.parse(fs.readFileSync(markerPath, 'utf-8'));
-  } catch (_err) { /* payload read is optional */ }
+  const markerPayload = safeJsonParse(markerPath, null);
   try {
     fs.unlinkSync(markerPath);
   } catch (err) {

package/scripts/hooks/core/task-completed.js

@@ -168,7 +168,7 @@ async function handleTaskCompleted(input) {
         writeJson(readyPath, ready);
         result.completed = true;
         result.message = `Task ${completedTask.id} (${completedTask.title}) moved to completed`;
-      } catch (err) {
+      } catch (_err) {
         result.message = 'Failed to update ready.json';
       }
     });
@@ -401,12 +401,18 @@ async function handleTaskCompleted(input) {
             { mode: 0o644 }
           );
         }
-      } catch (_err) {
-        // Non-critical — workspace message is defense-in-depth.
-        // The Stop hook curl remains as fallback.
-        if (process.env.DEBUG) {
-          console.error(`[Task Completed] Workspace message write failed: ${_err.message}`);
-        }
+      } catch (err) {
+        // Workspace message is the VERIFIED completion signal. A silent failure
+        // produces "workers stopped writing since <date>" incidents that are
+        // indistinguishable from "no tasks completed recently" — surface on
+        // stderr unconditionally so regressions are visible in worker logs.
+        // (diagnostic D1, 2026-04-16 honesty-infrastructure review.)
+        const reason = err && err.message ? err.message : 'unknown';
+        const root = process.env.WOGI_WORKSPACE_ROOT || '(unset)';
+        const repo = process.env.WOGI_REPO_NAME || '(unset)';
+        console.error(
+          `[Task Completed] workspace message write FAILED for ${completedTask.id}: ${reason} (root=${root}, repo=${repo})`
+        );
       }
     }
 

package/scripts/hooks/core/task-created.js

@@ -31,7 +31,7 @@ function isTaskCreatedEnabled() {
  * @param {Object} input - Parsed hook input from Claude Code
  * @returns {Object} Core result
  */
-async function handleTaskCreated(input) {
+async function handleTaskCreated(_input) {
   if (!isTaskCreatedEnabled()) {
     return { enabled: false, message: null };
   }

package/scripts/hooks/core/worktree-lifecycle.js

@@ -25,7 +25,7 @@ const CORE_STATE_FILES = ['ready.json', 'decisions.md'];
 
 function getEssentialStateFiles() {
   try {
-    const { getRegistryMapFiles, PATHS } = require('../../flow-utils');
+    const { getRegistryMapFiles } = require('../../flow-utils');
     return [...CORE_STATE_FILES, ...getRegistryMapFiles()];
   } catch (_err) {
     return [...CORE_STATE_FILES, 'app-map.md', 'function-map.md', 'api-map.md'];

package/scripts/hooks/entry/claude-code/permission-denied.js

@@ -31,13 +31,15 @@ runHook('PermissionDenied', async ({ input, parsedInput }) => {
   try {
     const fs = require('node:fs');
     const { PATHS } = require('../../../flow-utils');
+    const { safeJsonParseString } = require('../../../flow-io');
     const logPath = path.join(PATHS.state, 'permission-denials.json');
 
     let denials = [];
     try {
       if (fs.existsSync(logPath)) {
-        denials = JSON.parse(fs.readFileSync(logPath, 'utf-8'));
-        if (!Array.isArray(denials)) denials = [];
+        const raw = fs.readFileSync(logPath, 'utf-8');
+        const parsed = safeJsonParseString(raw, []);
+        denials = Array.isArray(parsed) ? parsed : [];
       }
     } catch (_err) {
       denials = [];

package/scripts/hooks/entry/claude-code/stop.js

@@ -242,6 +242,66 @@ runHook('Stop', async ({ parsedInput }) => {
     }
   }
 
+  // G3 (v2.21.0) — AI-based worker-question classifier.
+  //
+  // If the worker ends a turn with a question to the user in free text (no tool
+  // call, just hedging), Gap B above won't fire when the queue is empty.
+  // Regex-based detection was rejected as brittle. Instead: a single Haiku call
+  // classifies the final assistant message. If it IS an open question to the
+  // user → block with escalation instructions.
+  //
+  // Fail-open throughout: missing API key, missing transcript, model errors,
+  // malformed responses all skip cleanly. Silent-stall false-negatives recover;
+  // blocking legitimate stops on classifier bugs does not.
+  try {
+    const isWorker = process.env.WOGI_WORKSPACE_ROOT &&
+                     process.env.WOGI_REPO_NAME &&
+                     process.env.WOGI_REPO_NAME !== 'manager';
+    if (isWorker) {
+      const { getConfig } = require('../../../flow-utils');
+      const config = getConfig();
+      const clf = config.workspace?.aiWorkerQuestionClassifier;
+      const enabled = clf?.enabled !== false;  // default true
+      if (enabled && parsedInput?.transcriptPath) {
+        const { classifyWorkerQuestion } = require('../../../flow-worker-question-classifier');
+        const result = await classifyWorkerQuestion({
+          transcriptPath: parsedInput.transcriptPath,
+          minConfidence: Number.isFinite(clf?.minConfidence) ? clf.minConfidence : 70,
+          model: typeof clf?.model === 'string' ? clf.model : undefined
+        });
+        if (result?.blocked) {
+          const port = process.env.WOGI_MANAGER_PORT || '8800';
+          const repo = process.env.WOGI_REPO_NAME;
+          const msg = [
+            `WORKER→USER QUESTION DETECTED (confidence ${result.confidence}%, threshold ${result.minConfidence}%):`,
+            `  "${String(result.reason || '').slice(0, 200)}"`,
+            '',
+            'In workspace mode, workers CANNOT ask the user directly — the user only sees',
+            'the manager terminal. Your question will stall silently.',
+            '',
+            'Channel-dispatch to the manager instead, THEN end the turn:',
+            '',
+            `  curl -s -X POST http://127.0.0.1:${port} \\`,
+            `    -H "X-Wogi-From: ${repo}" \\`,
+            `    --data-binary "## QUESTION: <your question>"`,
+            '',
+            'The manager will relay to the user, capture the answer, and dispatch a',
+            'follow-up task to you with the resolved context.',
+            '',
+            'If you don\'t actually need the user — make a reasonable autonomous decision',
+            'and note it in your ## Results reply to the manager. Then end the turn.'
+          ].join('\n');
+          return { __raw: true, continue: true, stopReason: msg };
+        }
+      }
+    }
+  } catch (err) {
+    // Fail-OPEN — classifier errors must not block legitimate stops.
+    if (process.env.DEBUG) {
+      console.error(`[Stop] Worker question classifier error (fail-open): ${err.message}`);
+    }
+  }
+
   // Check if loop can exit
   return await checkLoopExit();
 }, { failMode: 'warn', failOutput: { continue: false } });

package/scripts/hooks/entry/claude-code/user-prompt-submit.js

@@ -55,7 +55,7 @@ runHook('UserPromptSubmit', async ({ input, parsedInput }) => {
   let hookConfig;
   try {
     hookConfig = getConfig();
-  } catch (err) {
+  } catch (_err) {
     hookConfig = {};
   }
 

package/scripts/hooks/git/post-commit.js

@@ -15,7 +15,7 @@ const path = require('node:path');
 
 // Resolve paths relative to this script's location
 const scriptsDir = path.resolve(__dirname, '../..');
-const { getReadyData, saveReadyData, info, color } = require(path.join(scriptsDir, 'flow-utils'));
+const { getReadyData, saveReadyData, info } = require(path.join(scriptsDir, 'flow-utils'));
 const { success } = require('../../flow-output');
 
 /**

package/scripts/postinstall.js

@@ -62,7 +62,7 @@ function safeJsonParseString(jsonString, defaultValue = null) {
       return defaultValue;
     }
     return parsed;
-  } catch (err) {
+  } catch (_err) {
     return defaultValue;
   }
 }
@@ -93,7 +93,7 @@ function hasDangerousKeys(obj) {
  */
 function safeClose(fd) {
   if (fd !== null) {
-    try { fs.closeSync(fd); } catch (err) { /* intentionally ignored */ }
+    try { fs.closeSync(fd); } catch (_err) { /* intentionally ignored */ }
   }
 }
 
@@ -520,7 +520,7 @@ function copyClaudeResources() {
         }
         try {
           fs.chmodSync(projectSettings, FILE_MODE);
-        } catch (err) { /* non-critical */ }
+        } catch (_err) { /* non-critical */ }
       } catch (err) {
         if (process.env.DEBUG) {
           console.error(`[postinstall] settings.json initial copy failed: ${err.message}`);
@@ -608,7 +608,7 @@ function getPackageVersion() {
     const content = fs.readFileSync(path.join(PACKAGE_ROOT, 'package.json'), 'utf-8');
     const pkg = safeJsonParseString(content, null);
     return (pkg && pkg.version) || 'unknown';
-  } catch (err) {
+  } catch (_err) {
     return 'unknown';
   }
 }
@@ -644,7 +644,7 @@ function regenerateClaudeMd() {
       }
       return;
     }
-  } catch (err) {
+  } catch (_err) {
     // No version file yet — proceed with regen
   }
 
@@ -688,7 +688,7 @@ function regenerateClaudeMd() {
     // Record successful regen version for skip gate
     try {
       fs.writeFileSync(versionFile, pkgVersion, { mode: FILE_MODE });
-    } catch (err) {
+    } catch (_err) {
       // Non-critical — next install will just regen again
     }
 
@@ -1015,7 +1015,7 @@ function main() {
       // Combine access check and open into single try-catch to avoid TOCTOU
       ttyFd = fs.openSync('/dev/tty', 'w');
       output = { write: (msg) => fs.writeSync(ttyFd, msg) };
-    } catch (err) {
+    } catch (_err) {
       // /dev/tty not available (no terminal, CI, etc.) - fallback to stderr
       ttyFd = null;
     }

package/scripts/preuninstall.js

@@ -61,7 +61,7 @@ function readManifest() {
       return manifest;
     }
     return null;
-  } catch (err) {
+  } catch (_err) {
     return null;
   }
 }
@@ -112,7 +112,7 @@ function cleanupEmptyDirs(baseDir) {
           if (remaining.length === 0) {
             fs.rmdirSync(subDir);
           }
-        } catch (err) {
+        } catch (_err) {
           // Dir already gone or inaccessible
         }
       }
@@ -156,7 +156,7 @@ function manifestBasedCleanup(manifest) {
           preserved.push(path.join(dir, f));
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Dir doesn't exist or already removed
     }
   }
@@ -231,7 +231,7 @@ function removeClaudeMd() {
       return true;
     }
     return false;
-  } catch (err) {
+  } catch (_err) {
     return false;
   }
 }
@@ -254,7 +254,7 @@ function cleanupClaudeDir() {
       return { removed: true, preserved: [] };
     }
     return { removed: false, preserved: userFiles };
-  } catch (err) {
+  } catch (_err) {
     return { removed: false, preserved: [] };
   }
 }

package/scripts/registries/component-registry.js

@@ -14,7 +14,7 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
-const { getProjectRoot, getConfig, color, success, warn, PATHS } = require('../flow-utils');
+const { getConfig, color, success, warn, PATHS } = require('../flow-utils');
 const { RegistryPlugin } = require('../flow-registry-manager');
 const { BaseScanner, PROJECT_ROOT } = require('../flow-scanner-base');
 const { info } = require('../flow-output');
@@ -92,7 +92,7 @@ class ComponentScanner extends BaseScanner {
       } else {
         this._scanJSXFile(content, relativePath, category);
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip files that can't be read
     }
   }

package/scripts/registries/contract-scanner.js

@@ -57,7 +57,7 @@ function walkSourceFiles(dir, options = {}) {
     let entries;
     try {
       entries = fs.readdirSync(currentDir, { withFileTypes: true });
-    } catch (err) {
+    } catch (_err) {
       return; // Skip unreadable directories
     }
 
@@ -120,12 +120,12 @@ function scanHttpClients(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(filePath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
     const relPath = path.relative(projectRoot, filePath);
-    const lines = content.split('\n');
+    const _lines = content.split('\n');
 
     for (const pattern of patterns) {
       // Reset lastIndex for global regex
@@ -192,7 +192,7 @@ function scanRouteDefinitions(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(filePath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
@@ -286,7 +286,7 @@ function scanNextjsApiRoutes(projectRoot) {
       let content;
       try {
         content = fs.readFileSync(filePath, 'utf-8');
-      } catch (err) {
+      } catch (_err) {
         continue;
       }
 
@@ -352,7 +352,7 @@ function scanEventBus(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(filePath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
@@ -422,7 +422,7 @@ function scanSharedTypes(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(filePath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
@@ -495,7 +495,7 @@ function scanEnvVars(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(filePath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
@@ -531,7 +531,7 @@ function scanEnvVars(projectRoot, options = {}) {
     let content;
     try {
       content = fs.readFileSync(envPath, 'utf-8');
-    } catch (err) {
+    } catch (_err) {
       continue;
     }
 
@@ -588,7 +588,7 @@ function detectProjectType(projectRoot) {
         try {
           const entries = fs.readdirSync(dir, { withFileTypes: true });
           packageCount += entries.filter(e => e.isDirectory()).length;
-        } catch (err) {
+        } catch (_err) {
           // skip
         }
       }
@@ -651,7 +651,7 @@ function getCommitSha(projectRoot) {
       encoding: 'utf-8',
       stdio: ['pipe', 'pipe', 'pipe']
     }).trim();
-  } catch (err) {
+  } catch (_err) {
     return null;
   }
 }

package/scripts/registries/schema-registry.js

@@ -12,7 +12,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { RegistryPlugin } = require('../flow-registry-manager');
-const { getProjectRoot, safeJsonParse: safeJsonParseFile, PATHS, getTodayDate } = require('../flow-utils');
+const { safeJsonParse: safeJsonParseFile, PATHS, getTodayDate } = require('../flow-utils');
 
 const INDEX_PATH = path.join(PATHS.state, 'schema-index.json');
 const MAP_PATH = path.join(PATHS.state, 'schema-map.md');
@@ -47,7 +47,7 @@ class SchemaRegistry extends RegistryPlugin {
         const allDeps = { ...pkg.dependencies, ...pkg.devDependencies };
         const ormPackages = ['prisma', '@prisma/client', 'typeorm', 'drizzle-orm', 'sequelize', 'mongoose'];
         if (ormPackages.some(p => allDeps[p])) return true;
-      } catch (err) {
+      } catch (_err) {
         // ignore
       }
     }
@@ -197,7 +197,7 @@ class SchemaRegistry extends RegistryPlugin {
         const content = fs.readFileSync(file, 'utf-8');
         const relPath = path.relative(PATHS.root, file);
         this._parsePrismaFile(content, relPath);
-      } catch (err) {
+      } catch (_err) {
         // Skip unreadable files
       }
     }
@@ -217,7 +217,7 @@ class SchemaRegistry extends RegistryPlugin {
           results.push(fullPath);
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip unreadable directories
     }
     return results;
@@ -368,7 +368,7 @@ class SchemaRegistry extends RegistryPlugin {
           }
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip unreadable directories
     }
   }

package/scripts/registries/service-registry.js

@@ -13,7 +13,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { RegistryPlugin } = require('../flow-registry-manager');
-const { getProjectRoot, safeJsonParse: safeJsonParseFile, PATHS, getTodayDate } = require('../flow-utils');
+const { safeJsonParse: safeJsonParseFile, PATHS, getTodayDate } = require('../flow-utils');
 
 const INDEX_PATH = path.join(PATHS.state, 'service-index.json');
 const MAP_PATH = path.join(PATHS.state, 'service-map.md');
@@ -314,12 +314,12 @@ class ServiceRegistry extends RegistryPlugin {
             if (content.includes('NestMiddleware')) {
               this._parseNestJSMiddleware(content, relPath, 'middleware');
             }
-          } catch (err) {
+          } catch (_err) {
             // Skip unreadable files
           }
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip unreadable directories
     }
   }
@@ -466,12 +466,12 @@ class ServiceRegistry extends RegistryPlugin {
                 file: relPath
               });
             }
-          } catch (err) {
+          } catch (_err) {
             // Skip
           }
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip
     }
   }
@@ -523,12 +523,12 @@ class ServiceRegistry extends RegistryPlugin {
             const content = fs.readFileSync(fullPath, 'utf-8');
             const relPath = path.relative(PATHS.root, fullPath);
             this._parseDjangoFile(content, relPath);
-          } catch (err) {
+          } catch (_err) {
             // Skip
           }
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip
     }
   }
@@ -589,12 +589,12 @@ class ServiceRegistry extends RegistryPlugin {
               const relPath = path.relative(PATHS.root, fullPath);
               this._parseGoFile(content, relPath);
             }
-          } catch (err) {
+          } catch (_err) {
             // Skip
           }
         }
       }
-    } catch (err) {
+    } catch (_err) {
       // Skip
     }
   }