moflo 4.9.10 → 4.9.12

package/dist/src/cli/services/hook-wiring.js

@@ -38,7 +38,10 @@ export const HOOK_ENTRY_MAP = {
     'check-dangerous-command': { event: 'PreToolUse', matcher: '^Bash$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" check-dangerous-command', timeout: 2000 } },
     'check-before-pr': { event: 'PreToolUse', matcher: '^Bash$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" check-before-pr', timeout: 2000 } },
     'record-task-created': { event: 'PostToolUse', matcher: '^TaskCreate$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" record-task-created', timeout: 2000 } },
-    'record-memory-searched': { event: 'PostToolUse', matcher: 'mcp__moflo__memory_', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" record-memory-searched', timeout: 3000 } },
+    // record-memory-searched MUST go through gate-hook.mjs (not gate.cjs directly)
+    // — the wrapper forwards Claude Code's session_id as HOOK_SESSION_ID, which
+    // markMemorySearched needs to stamp the per-actor map (#879).
+    'record-memory-searched': { event: 'PostToolUse', matcher: 'mcp__moflo__memory_', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" record-memory-searched', timeout: 3000 } },
     'check-task-transition': { event: 'PostToolUse', matcher: '^TaskUpdate$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" check-task-transition', timeout: 2000 } },
     'record-learnings-stored': { event: 'PostToolUse', matcher: '^mcp__moflo__memory_store$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" record-learnings-stored', timeout: 2000 } },
     'check-bash-memory': { event: 'PostToolUse', matcher: '^Bash$', hook: { type: 'command', command: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" check-bash-memory', timeout: 2000 } },
@@ -84,4 +87,54 @@ export function repairHookWiring(settings) {
     settings.hooks = hooks;
     return { settings, repaired };
 }
+export const HOOK_REWRITE_RULES = [
+    // Issue #879 — record-memory-searched MUST use gate-hook.mjs so Claude Code's
+    // session_id is forwarded as HOOK_SESSION_ID. Without it, the per-actor map
+    // stays empty and the gate blocks every Read forever within a turn.
+    {
+        name: '#879: record-memory-searched → gate-hook.mjs',
+        from: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" record-memory-searched',
+        to: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" record-memory-searched',
+    },
+    // Symmetry hardening — same fix shape for check-bash-memory. The shipped
+    // settings.json already wires this through gate-hook.mjs in current versions,
+    // but a stale consumer settings.json may have it wrong.
+    {
+        name: '#879: check-bash-memory → gate-hook.mjs',
+        from: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate.cjs" check-bash-memory',
+        to: 'node "$CLAUDE_PROJECT_DIR/.claude/helpers/gate-hook.mjs" check-bash-memory',
+    },
+];
+/**
+ * Apply HOOK_REWRITE_RULES to every hook command in `settings.hooks.*`.
+ * Idempotent: a hook already at the `to` form won't match `from`.
+ *
+ * @returns The (potentially mutated) settings and a list of rewrites that fired.
+ */
+export function rewriteIncorrectHookWiring(settings) {
+    const hooks = (settings.hooks ?? {});
+    const rewrites = [];
+    for (const rule of HOOK_REWRITE_RULES) {
+        let count = 0;
+        for (const eventName of Object.keys(hooks)) {
+            const eventArray = hooks[eventName];
+            if (!Array.isArray(eventArray))
+                continue;
+            for (const block of eventArray) {
+                const blockHooks = block.hooks;
+                if (!Array.isArray(blockHooks))
+                    continue;
+                for (const h of blockHooks) {
+                    if (typeof h.command === 'string' && h.command.includes(rule.from)) {
+                        h.command = h.command.split(rule.from).join(rule.to);
+                        count++;
+                    }
+                }
+            }
+        }
+        if (count > 0)
+            rewrites.push({ name: rule.name, count });
+    }
+    return { settings, rewrites };
+}
 //# sourceMappingURL=hook-wiring.js.map

package/dist/src/cli/services/index.js

@@ -15,6 +15,8 @@ export { AgentRouter, getAgentRouter, routeTask, AGENT_CAPABILITIES, } from './a
 export { startDashboard, createDashboardMemoryAccessor, DEFAULT_DASHBOARD_PORT, } from './daemon-dashboard.js';
 // Hook Wiring (shared between doctor, upgrade, and session-start)
 export { repairHookWiring, HOOK_ENTRY_MAP, REQUIRED_HOOK_WIRING, } from './hook-wiring.js';
+// Hook Block Drift Detection (#881 — hash-based reconciliation)
+export { computeHookBlockHash, computeHookBlockDrift, formatDriftReport, getReferenceHookBlock, normaliseHooks, isHookBlockLocked, applyAdditiveRegeneration, DRIFT_MODES, } from './hook-block-hash.js';
 // Subagent Bootstrap Directive (single-source for SubagentStart + agent_spawn surfaces)
 export { SUBAGENT_BOOTSTRAP_DIRECTIVE } from './subagent-bootstrap.js';
 //# sourceMappingURL=index.js.map

package/dist/src/cli/services/process-registry.js

@@ -0,0 +1,58 @@
+/**
+ * Shared write side of the moflo background-process registry.
+ *
+ * Mirrors `bin/lib/process-manager.mjs`'s atomic write logic so TS spawn
+ * sites (auto-start daemon in src/cli/index.ts and daemon-readiness.ts) can
+ * register their PIDs alongside the ones written by bin/hooks.mjs's
+ * spawnWindowless helper. Without registry parity, doctor's zombie scan
+ * (src/cli/commands/doctor.ts) flags every TS-spawned background process as
+ * an orphan, because they are detached:true so their immediate parent dies.
+ *
+ * The .mjs module remains the canonical reader/writer with full read+killAll
+ * semantics; this TS helper only covers the registration we need from compiled
+ * paths. Both write to the same JSON file (`<projectRoot>/.moflo/background-pids.json`),
+ * so a process registered here is reapable via pm.killAll() at session-end.
+ */
+import { join } from 'path';
+import { existsSync, readFileSync, writeFileSync, renameSync, mkdirSync } from 'fs';
+const REGISTRY_FILENAME = 'background-pids.json';
+function registryPath(projectRoot) {
+    return join(projectRoot, '.moflo', REGISTRY_FILENAME);
+}
+function readRegistry(projectRoot) {
+    const path = registryPath(projectRoot);
+    if (!existsSync(path))
+        return [];
+    try {
+        const parsed = JSON.parse(readFileSync(path, 'utf-8'));
+        return Array.isArray(parsed) ? parsed : [];
+    }
+    catch {
+        return [];
+    }
+}
+function writeRegistry(projectRoot, entries) {
+    const path = registryPath(projectRoot);
+    mkdirSync(join(projectRoot, '.moflo'), { recursive: true });
+    const tmp = `${path}.tmp.${process.pid}`;
+    writeFileSync(tmp, JSON.stringify(entries, null, 2));
+    renameSync(tmp, path);
+}
+/**
+ * Register a spawned background PID under the given label. Replaces any
+ * pre-existing entry with the same label so a stale dead-PID row doesn't
+ * accumulate when a daemon crashes and is restarted.
+ */
+export function registerBackgroundPid(projectRoot, pid, label, cmd) {
+    if (!Number.isInteger(pid) || pid <= 0)
+        return;
+    const fresh = readRegistry(projectRoot).filter(e => e && e.label !== label);
+    fresh.push({
+        pid,
+        label,
+        cmd: cmd.slice(0, 200),
+        startedAt: new Date().toISOString(),
+    });
+    writeRegistry(projectRoot, fresh);
+}
+//# sourceMappingURL=process-registry.js.map

package/dist/src/cli/version.js

@@ -2,5 +2,5 @@
  * Auto-generated by build. Do not edit manually.
  * Source of truth: root package.json → scripts/sync-version.mjs
  */
-export const VERSION = '4.9.10';
+export const VERSION = '4.9.12';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moflo",
-  "version": "4.9.10",
+  "version": "4.9.12",
   "description": "MoFlo — AI agent orchestration for Claude Code. A standalone, opinionated toolkit with semantic memory, learned routing, gates, spells, and the /flo issue-execution skill.",
   "main": "dist/src/cli/index.js",
   "type": "module",
@@ -81,7 +81,7 @@
     "@typescript-eslint/eslint-plugin": "^7.18.0",
     "@typescript-eslint/parser": "^7.18.0",
     "eslint": "^8.0.0",
-    "moflo": "^4.9.9",
+    "moflo": "^4.9.11",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",
     "vitest": "^4.0.0"