moflo 4.9.8 → 4.9.9

package/bin/hooks.mjs

@@ -289,7 +289,20 @@ async function main() {
         // chain and producing the sqlite_autoindex corruption that #743 had
         // to repair on subsequent sessions. The launcher's foreground §3e
         // migration is the canonical migration site. See #744.
-        spawnWindowless('node', [resolve(__dirname, 'index-all.mjs')], 'sequential indexing chain');
+        // Prefer the npm-bin copy of index-all.mjs over __dirname resolution
+        // (#866). When this hooks.mjs is loaded from `.claude/scripts/` (e.g.
+        // an older launcher spawned us before the launcher's bin-anchor fix
+        // landed), `resolve(__dirname, 'index-all.mjs')` would point at the
+        // synced mirror — which the launcher's section 3 may still be in the
+        // middle of overwriting during an upgrade. resolveBinOrLocal's
+        // bin-first ordering guarantees the spawned chain matches the
+        // installed package even when the mirror is mid-sync.
+        const indexAllScript = resolveBinOrLocal('flo-index-all', 'index-all.mjs');
+        if (indexAllScript) {
+          spawnWindowless('node', [indexAllScript], 'sequential indexing chain');
+        } else {
+          log('warn', 'index-all.mjs not found (checked npm bin + .claude/scripts/)');
+        }
         // Neural patterns now loaded by moflo core routing — no external patching.
         break;
       }

package/bin/prompt-hook.mjs

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 import { execFileSync } from 'child_process';
+import { readFileSync } from 'fs';
 import { resolve } from 'path';
 
 // Read stdin JSON from Claude Code
@@ -70,6 +71,19 @@ if (TEST_HINTS.test(lower)) {
   }
 }
 
-var parts = [output.trim(), nsHint].filter(Boolean);
-if (parts.length) process.stdout.write(parts.join('\n') + '\n');
+// #867 — surface post-install restart notice. File is written by
+// scripts/post-install-notice.mjs and cleared by the SessionStart launcher
+// once the running moflo matches the file's version, so the file's
+// existence at prompt-time is itself the "still needs restart" signal.
+var restartNotice = '';
+try {
+  var pendingPath = resolve(projectDir, '.moflo', 'restart-pending.json');
+  var pending = JSON.parse(readFileSync(pendingPath, 'utf-8'));
+  if (pending && typeof pending.message === 'string' && pending.message.length > 0) {
+    restartNotice = pending.message;
+  }
+} catch (e) { /* ENOENT or malformed — silent fast-path */ }
+
+var parts = [restartNotice, output.trim(), nsHint].filter(Boolean);
+if (parts.length) process.stdout.write(parts.join('\n\n') + '\n');
 process.exit(0);

package/bin/session-start-launcher.mjs

@@ -217,6 +217,23 @@ try {
   // own errors if the DB is still broken.
 }
 
+// ── 0d. Clear post-install restart notice when version is current (#867) ───
+// scripts/post-install-notice.mjs drops `.moflo/restart-pending.json` on every
+// `npm install moflo`. The UserPromptSubmit hook surfaces it on every prompt
+// until cleared, so this session only sees the message between install and
+// the FIRST restart that actually picks up the new bits.
+try {
+  const pendingPath = join(mofloDir(projectRoot), 'restart-pending.json');
+  const pkgPath = resolve(projectRoot, 'node_modules/moflo/package.json');
+  const pending = JSON.parse(readFileSync(pendingPath, 'utf-8'));
+  const installedVersion = JSON.parse(readFileSync(pkgPath, 'utf-8')).version;
+  if (pending && typeof pending.version === 'string' && pending.version === installedVersion) {
+    unlinkSync(pendingPath);
+    try { unlinkSync(join(mofloDir(projectRoot), 'last-install-banner.json')); } catch { /* tracker may not exist */ }
+    emitMutation('cleared post-install restart notice', `${installedVersion} now running`);
+  }
+} catch { /* file missing or malformed — silent fast-path */ }
+
 // ── 1. Helper: fire-and-forget a background process ─────────────────────────
 function fireAndForget(cmd, args, label) {
   try {
@@ -1106,11 +1123,20 @@ if (mutationCount > 0) {
 }
 
 // ── 4. Spawn background tasks ───────────────────────────────────────────────
-const localCli = resolve(projectRoot, 'node_modules/moflo/bin/cli.js');
-const hasLocalCli = existsSync(localCli);
 
-// hooks.mjs session-start (daemon, indexer, pretrain, HNSW, neural patterns)
-const hooksScript = resolve(projectRoot, '.claude/scripts/hooks.mjs');
+// hooks.mjs session-start (daemon, indexer, pretrain, HNSW, neural patterns).
+// Prefer the npm-bin copy over the `.claude/scripts/` mirror (#866). The mirror
+// is a derived sync that races the launcher's section-3 file copies during the
+// very upgrade session — spawning the still-stale `.claude/scripts/hooks.mjs`
+// then chaining `__dirname/index-all.mjs` produces an orphan running pre-
+// upgrade argv (e.g. `rebuild-index --force` after #859 had already dropped
+// it). The bin/ copy is updated atomically by `npm install moflo` (single-
+// step), so spawning from there guarantees the running hook code matches the
+// installed package. Falls back to the mirror only when the package copy is
+// unresolvable (development / symlinked installs).
+const hooksPkg = resolve(projectRoot, 'node_modules/moflo/bin/hooks.mjs');
+const hooksMirror = resolve(projectRoot, '.claude/scripts/hooks.mjs');
+const hooksScript = existsSync(hooksPkg) ? hooksPkg : hooksMirror;
 if (existsSync(hooksScript)) {
   fireAndForget('node', [hooksScript, 'session-start'], 'hooks session-start');
 }

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.8';
+export const VERSION = '4.9.9';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moflo",
-  "version": "4.9.8",
+  "version": "4.9.9",
   "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.7",
+    "moflo": "^4.9.8",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",
     "vitest": "^4.0.0"