@fuzeelogik/myflo 1.0.0 → 1.0.2

package/lib/memory-backend-agentdb.js

@@ -39,18 +39,18 @@ async function getBackend() {
   if (_backend) return _backend;
   if (!existsSync(FLO_HOME)) await mkdir(FLO_HOME, { recursive: true });
   // Dynamic import keeps the heavy module out of the load path for the JSONL
-  // codepath. @myflo/memory is an optionalDependency — when missing (e.g. a
-  // bare `npm install myflo` without the optional extras), throw a clear error
-  // so the caller knows to either install the optional dep or stick with the
-  // jsonl backend.
+  // codepath. @myflo/memory is not yet published to npm — the agentdb backend
+  // only works when myflo is run from the monorepo (pnpm workspace resolves
+  // the package locally). For standalone `npx @fuzeelogik/myflo` installs,
+  // stick with the default jsonl backend.
   let SqlJsBackend;
   try {
     ({ SqlJsBackend } = await import('@myflo/memory'));
   } catch (err) {
     throw new Error(
-      `agentdb backend requires @myflo/memory (optional dependency). ` +
-      `Install it: npm i @myflo/memory. ` +
-      `Or stick with the default jsonl backend (unset FLO_MEMORY_BACKEND). ` +
+      `agentdb backend requires @myflo/memory, which is not yet published to npm. ` +
+      `It currently resolves only inside the myflo monorepo. ` +
+      `Stick with the default jsonl backend (unset FLO_MEMORY_BACKEND). ` +
       `Underlying error: ${err.message}`
     );
   }

package/lib/replace-ruflo.js

@@ -1,12 +1,15 @@
 // `flo replace ruflo` — cutover script.
 // Removes ruflo / claude-flow entries from ~/.claude/mcp.json and project
-// .claude/settings.json mcpServers blocks, leaving only flo. Idempotent.
-// Backs up both files before writing.
+// .claude/settings.json mcpServers blocks, AND shells out to
+// `claude mcp remove <name>` for entries the Claude CLI itself manages
+// (e.g. ones registered via `claude mcp add` that don't live in those files).
+// Idempotent. Backs up json files before writing.
 
 import { readFile, writeFile, copyFile, mkdir } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { join } from 'node:path';
+import { spawn } from 'node:child_process';
 
 const USER_MCP = join(homedir(), '.claude', 'mcp.json');
 const PROJECT_SETTINGS = join(process.cwd(), '.claude', 'settings.json');
@@ -57,11 +60,72 @@ export async function replaceCommand(args) {
     console.log(`✓ ${label}: removed [${result.removed.join(', ')}] from ${path} (backup: ${backup})`);
   }
 
+  // Phase 2: ask Claude CLI itself to remove any ruflo/claude-flow servers it
+  // tracks. This catches entries registered via `claude mcp add` that don't
+  // live in mcp.json or settings.json (e.g. user/local scope state the CLI
+  // keeps elsewhere). The flo replace ruflo command was previously a no-op
+  // for users in this state — see #1 on the v1.0.1 premortem.
+  const claudePath = await whichClaudeCli();
+  if (!claudePath) {
+    console.log('');
+    console.log('- claude CLI not on PATH — skipping `claude mcp remove` step.');
+    console.log('  (Files were cleaned above; that is usually enough.)');
+  } else {
+    console.log('');
+    console.log(`Running \`claude mcp remove\` for each known ruflo name...`);
+    for (const name of RUFLO_KEYS) {
+      if (opts.dryRun) {
+        console.log(`# would run: claude mcp remove "${name}"`);
+        continue;
+      }
+      const res = await runClaudeMcpRemove(claudePath, name);
+      if (res.removed) {
+        console.log(`✓ claude mcp remove "${name}" — removed from ${res.scope || 'config'}`);
+      } else if (res.notFound) {
+        console.log(`= claude mcp remove "${name}" — not registered, skipping`);
+      } else {
+        console.log(`× claude mcp remove "${name}" — failed: ${res.error}`);
+      }
+    }
+  }
+
   console.log('');
   console.log('Done. Restart Claude Code to pick up the change.');
   console.log("If you need to roll back, the .flo-bak.* file is your snapshot.");
 }
 
+async function whichClaudeCli() {
+  return new Promise((resolve) => {
+    const p = spawn('which', ['claude'], { stdio: ['ignore', 'pipe', 'ignore'] });
+    let out = '';
+    p.stdout.on('data', (b) => { out += b.toString(); });
+    p.on('close', (code) => resolve(code === 0 ? out.trim() : null));
+    p.on('error', () => resolve(null));
+  });
+}
+
+async function runClaudeMcpRemove(claudePath, name) {
+  return new Promise((resolve) => {
+    const p = spawn(claudePath, ['mcp', 'remove', name], { stdio: ['ignore', 'pipe', 'pipe'] });
+    let stdout = '';
+    let stderr = '';
+    p.stdout.on('data', (b) => { stdout += b.toString(); });
+    p.stderr.on('data', (b) => { stderr += b.toString(); });
+    p.on('close', (code) => {
+      const combined = (stdout + stderr).toLowerCase();
+      if (code === 0) {
+        const scopeMatch = stdout.match(/from\s+(\w+)\s+config/i);
+        resolve({ removed: true, scope: scopeMatch?.[1] || null });
+      } else if (/no.*server.*found|not.*found|does not exist/.test(combined)) {
+        resolve({ notFound: true });
+      } else {
+        resolve({ error: (stderr || stdout).trim().slice(0, 200) || `exit ${code}` });
+      }
+    });
+    p.on('error', (err) => resolve({ error: err.message }));
+  });
+}
+
 function removeRufloFrom(obj) {
   const removed = [];
   // mcpServers block
@@ -115,6 +179,10 @@ Specifically removes:
 
 Both files are backed up to <path>.flo-bak.<ts> before being rewritten.
 
+Also runs \`claude mcp remove ruflo\` and \`claude mcp remove claude-flow\`
+to catch entries the Claude CLI tracks outside those files (registered via
+\`claude mcp add\`). Silently skipped if the claude CLI isn't on PATH.
+
 Idempotent. Re-running on a clean config is a no-op.
 
 This complements 'flo migrate', which only ADDED the flo entry. Run that first

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fuzeelogik/myflo",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "myflo — local-first developer workbench. CLI + MCP server + Next.js dashboard. Forked from ruflo for the runtime substrate; flo CLI on top is ours.",
   "type": "module",
   "bin": {
@@ -14,7 +14,7 @@
     "README.md"
   ],
   "dependencies": {},
-  "optionalDependencies": {
+  "devDependencies": {
     "@myflo/memory": "workspace:*"
   },
   "engines": {

package/tests/smoke.sh

@@ -387,6 +387,32 @@ for line in sys.stdin:
 if [ "$TOOLCOUNT" = "22" ]; then echo "  PASS  mcp tools/list count (22)"; PASS=$((PASS+1))
 else echo "  FAIL  mcp tools/list count (got $TOOLCOUNT)"; FAIL=$((FAIL+1)); fi
 
+# `flo replace ruflo` against a fixture project — file rewrites + graceful
+# skip when claude CLI is unavailable (PATH hidden to avoid mutating real config).
+REPLACE_DIR="$TMP/replace-ruflo-test"
+mkdir -p "$REPLACE_DIR/.claude"
+cat > "$REPLACE_DIR/.claude/settings.json" <<'JSON'
+{
+  "mcpServers": {
+    "ruflo": {"command": "npx", "args": ["-y", "ruflo@latest", "mcp", "start"]},
+    "flo": {"command": "node", "args": ["/x.js"]}
+  },
+  "enabledMcpjsonServers": ["ruflo", "flo"],
+  "permissions": {"allow": ["mcp__ruflo__foo", "Bash(ls:*)", "mcp__claude-flow__bar"]}
+}
+JSON
+NODE_BIN=$(which node)
+REPLACE_OUT=$(cd "$REPLACE_DIR" && PATH="/usr/bin:/bin" $NODE_BIN "$REPO_ROOT/apps/cli/bin/flo.js" replace ruflo 2>&1)
+if echo "$REPLACE_OUT" | grep -q "claude CLI not on PATH"; then
+  if python3 -c "import json; d=json.load(open('$REPLACE_DIR/.claude/settings.json')); assert list(d['mcpServers'].keys())==['flo'] and d['enabledMcpjsonServers']==['flo'] and d['permissions']['allow']==['Bash(ls:*)']" 2>/dev/null; then
+    echo "  PASS  replace ruflo (file rewrite + graceful claude-cli skip)"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  replace ruflo: file content unexpected"; FAIL=$((FAIL+1))
+  fi
+else
+  echo "  FAIL  replace ruflo: expected 'claude CLI not on PATH' message"; FAIL=$((FAIL+1))
+fi
+
 echo "--------------"
 echo "$PASS passed, $FAIL failed."
 if [ "$FAIL" -gt 0 ]; then exit 1; fi