wogiflow 2.29.3 → 2.29.4

package/lib/workspace-channel-server.js

@@ -450,6 +450,22 @@ function broadcastSSE(event) {
   }
 }
 
+// ============================================================
+// Dispatch tracking integration (silent-halt RCA fix, v2.29.4)
+// ============================================================
+// The channel server is the only place that sees EVERY inbound message,
+// regardless of whether the manager dispatched via the programmatic
+// `dispatchToChannel()` helper or a raw `curl POST`. Recording at this
+// layer guarantees `dispatched-tasks.json` exists for every dispatch,
+// closing the wogi-hub 2026-04-27 silent-halt failure shape.
+//
+// Helpers live in `workspace-channel-tracking.js` so they can be unit-
+// tested without spawning the channel-server process. Both fail-open;
+// idempotency lives at the call site (Fix A skips on existing record;
+// Fix B delegates to reconcileDispatch which is idempotent).
+
+const channelTracking = require('./workspace-channel-tracking');
+
 // ============================================================
 // HTTP Server
 // ============================================================
@@ -506,6 +522,11 @@ const server = http.createServer(async (req, res) => {
       : cleanBody;
     sendChannelNotification(notificationBody, meta);
 
+    // v2.29.4 silent-halt RCA fixes — both fail-open
+    const trackingCtx = { workspaceRoot: WORKSPACE_ROOT, repoName: REPO_NAME, from, body: cleanBody };
+    channelTracking.tryRecordInboundDispatch(trackingCtx);
+    channelTracking.tryReconcileInboundCompletion(trackingCtx);
+
     // Also broadcast to SSE subscribers
     if (sseClients.size > 0) {
       const crypto = require('node:crypto');

package/lib/workspace-channel-tracking.js

@@ -0,0 +1,125 @@
+'use strict';
+
+/**
+ * Wogi Workspace — Channel Server Dispatch Tracking Helpers (v2.29.4)
+ *
+ * Pure-function helpers that the channel server's HTTP POST handler calls
+ * to record inbound dispatches and reconcile inbound completions. Extracted
+ * from `workspace-channel-server.js` so they can be unit-tested without
+ * spawning the channel-server process.
+ *
+ * Why these exist (silent-halt RCA, 2026-04-27):
+ *   `recordDispatch` was only called from the programmatic dispatch helper
+ *   (`workspace-routing.js → dispatchToChannel`). Manager AI sessions that
+ *   used raw `curl POST http://localhost:8801` bypassed it entirely. With
+ *   no record, the overdue detector had nothing to detect — workers could
+ *   die silently with zero manager-side signal.
+ *
+ * Both helpers are best-effort and fail-open. Idempotency lives at the
+ * call site (Fix A skips when a pending record already exists; Fix B
+ * delegates idempotency to `reconcileDispatch` which returns `null` when
+ * no pending record matches).
+ */
+
+const TASK_ID_PATTERN = /\bwf-[0-9a-f]{8}\b/i;
+const DISPATCH_BODY_PATTERN = /^\s*\/wogi-start\s+(wf-[0-9a-f]{8})\b/i;
+const QUESTION_BODY_PATTERN = /^\s*##\s*QUESTION/im;
+const COMPLETION_BODY_PATTERN = /##\s*Results\b|task-complete\b/i;
+
+/**
+ * Record an inbound dispatch when the channel server (running in worker
+ * mode) receives a `/wogi-start <id>` POST from the manager.
+ *
+ * No-ops when:
+ *   - workspaceRoot is missing
+ *   - body is not a non-empty string
+ *   - this server is in manager mode (REPO_NAME === 'manager')
+ *   - the `from` header is not the manager
+ *   - the body does not match the dispatch pattern
+ *   - a pending record for (taskId, repoName) already exists (idempotency)
+ *
+ * @param {Object} ctx
+ * @param {string} ctx.workspaceRoot
+ * @param {string} ctx.repoName
+ * @param {string} ctx.from
+ * @param {string} ctx.body
+ * @param {Object} [tracking] — injectable for tests; defaults to the lib module
+ * @returns {{action: 'recorded'|'skip-existing'|'skip-not-worker'|'skip-bad-from'|'skip-no-match'|'skip-no-root'|'skip-empty-body'|'error', reason?: string, taskId?: string}}
+ */
+function tryRecordInboundDispatch(ctx, tracking) {
+  const { workspaceRoot, repoName, from, body } = ctx || {};
+  if (!workspaceRoot) return { action: 'skip-no-root' };
+  if (typeof body !== 'string' || !body) return { action: 'skip-empty-body' };
+  if (repoName === 'manager') return { action: 'skip-not-worker' };
+  if (from !== 'manager' && from !== 'workspace-manager') return { action: 'skip-bad-from' };
+  const m = body.match(DISPATCH_BODY_PATTERN);
+  if (!m) return { action: 'skip-no-match' };
+  const taskId = m[1].toLowerCase();
+  try {
+    const tr = tracking || require('./workspace-dispatch-tracking');
+    const existing = tr.readDispatches(workspaceRoot).find(r =>
+      r && r.taskId === taskId && r.repoName === repoName && r.status === 'pending'
+    );
+    if (existing) return { action: 'skip-existing', taskId };
+    tr.recordDispatch(workspaceRoot, {
+      taskId,
+      repoName,
+      dispatchedBy: from
+    });
+    return { action: 'recorded', taskId };
+  } catch (err) {
+    return { action: 'error', reason: err.message, taskId };
+  }
+}
+
+/**
+ * Reconcile an inbound completion when the channel server (running in
+ * manager mode) receives a worker-side POST that looks like a completion.
+ *
+ * No-ops when:
+ *   - workspaceRoot is missing
+ *   - body is not a non-empty string
+ *   - this server is in worker mode (REPO_NAME !== 'manager')
+ *   - the `from` header IS the manager (no self-completion)
+ *   - the body looks like a `## QUESTION:` (escalation, not completion)
+ *   - the body does not contain `## Results` or `task-complete`
+ *   - the body does not contain a `wf-XXXXXXXX` reference
+ *   - reconcileDispatch finds no pending record (idempotent)
+ *
+ * @param {Object} ctx
+ * @param {string} ctx.workspaceRoot
+ * @param {string} ctx.repoName
+ * @param {string} ctx.from
+ * @param {string} ctx.body
+ * @param {Object} [tracking] — injectable for tests; defaults to the lib module
+ * @returns {{action: 'reconciled'|'skip-not-manager'|'skip-self'|'skip-question'|'skip-not-completion'|'skip-no-id'|'skip-no-pending'|'skip-no-root'|'skip-empty-body'|'error', reason?: string, taskId?: string}}
+ */
+function tryReconcileInboundCompletion(ctx, tracking) {
+  const { workspaceRoot, repoName, from, body } = ctx || {};
+  if (!workspaceRoot) return { action: 'skip-no-root' };
+  if (typeof body !== 'string' || !body) return { action: 'skip-empty-body' };
+  if (repoName !== 'manager') return { action: 'skip-not-manager' };
+  if (from === 'manager' || from === 'workspace-manager') return { action: 'skip-self' };
+  if (QUESTION_BODY_PATTERN.test(body)) return { action: 'skip-question' };
+  if (!COMPLETION_BODY_PATTERN.test(body)) return { action: 'skip-not-completion' };
+  const m = body.match(TASK_ID_PATTERN);
+  if (!m) return { action: 'skip-no-id' };
+  const taskId = m[0].toLowerCase();
+  try {
+    const tr = tracking || require('./workspace-dispatch-tracking');
+    const result = tr.reconcileDispatch(workspaceRoot, taskId, 'completed', 'channel-server-completion');
+    if (!result) return { action: 'skip-no-pending', taskId };
+    return { action: 'reconciled', taskId };
+  } catch (err) {
+    return { action: 'error', reason: err.message, taskId };
+  }
+}
+
+module.exports = {
+  TASK_ID_PATTERN,
+  DISPATCH_BODY_PATTERN,
+  QUESTION_BODY_PATTERN,
+  COMPLETION_BODY_PATTERN,
+  tryRecordInboundDispatch,
+  tryReconcileInboundCompletion
+};

package/package.json

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