@sparkleideas/claude-flow-patch 3.1.0-alpha.44.patch.3

package/patch/200-NS-002-targeted-require-namespace/fix.py

@@ -0,0 +1,158 @@
+# NS-002: Targeted ops (store/delete/retrieve) accept wrong namespace
+# GitHub: #581
+# Rule: Require explicit namespace, block 'all'
+# 14 ops: 20a-i (store/delete) + 21a,b,e,f,h (retrieve)
+
+# ── Store ops (from old Patch 20) ──
+
+# 20a: MCP store — add 'namespace' to required fields in schema
+patch("20a: MCP store require namespace",
+    MCP_MEMORY,
+    "required: ['key', 'value'],",
+    "required: ['key', 'value', 'namespace'],")
+
+# 20b: MCP store — remove || 'default' fallback + add runtime throw
+patch("20b: MCP store namespace no fallback",
+    MCP_MEMORY,
+    "const namespace = input.namespace || 'default';\n            const value = typeof",
+    "const namespace = input.namespace;\n            if (!namespace || namespace === 'all') {\n                throw new Error('Namespace is required (cannot be \"all\"). Use namespace: \"patterns\", \"solutions\", or \"tasks\"');\n            }\n            const value = typeof")
+
+# 20c: MCP delete — add 'namespace' to required + update description
+patch("20c: MCP delete require namespace",
+    MCP_MEMORY,
+    """        description: 'Delete a memory entry by key',
+        category: 'memory',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                key: { type: 'string', description: 'Memory key' },
+                namespace: { type: 'string', description: 'Namespace (default: "default")' },
+            },
+            required: ['key'],""",
+    """        description: 'Delete a memory entry by key',
+        category: 'memory',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                key: { type: 'string', description: 'Memory key' },
+                namespace: { type: 'string', description: 'Namespace (e.g. "patterns", "solutions", "tasks")' },
+            },
+            required: ['key', 'namespace'],""")
+
+# 20d: MCP delete — remove || 'default' fallback + add runtime throw
+patch("20d: MCP delete namespace no fallback",
+    MCP_MEMORY,
+    "const { deleteEntry } = await getMemoryFunctions();\n            const key = input.key;\n            const namespace = input.namespace || 'default';",
+    "const { deleteEntry } = await getMemoryFunctions();\n            const key = input.key;\n            const namespace = input.namespace;\n            if (!namespace || namespace === 'all') {\n                throw new Error('Namespace is required (cannot be \"all\"). Use namespace: \"patterns\", \"solutions\", or \"tasks\"');\n            }")
+
+# 20e: CLI store — add namespace-required check after key check
+patch("20e: CLI store require namespace",
+    CLI_MEMORY,
+    """        if (!key) {
+            output.printError('Key is required. Use --key or -k');
+            return { success: false, exitCode: 1 };
+        }
+        if (!value && ctx.interactive) {""",
+    """        if (!key) {
+            output.printError('Key is required. Use --key or -k');
+            return { success: false, exitCode: 1 };
+        }
+        if (!namespace || namespace === 'all') {
+            output.printError('Namespace is required (cannot be "all"). Use --namespace or -n (e.g. "patterns", "solutions", "tasks")');
+            return { success: false, exitCode: 1 };
+        }
+        if (!value && ctx.interactive) {""")
+
+# 20f: CLI delete — remove || 'default' fallback and fix error message
+patch("20f: CLI delete namespace no fallback",
+    CLI_MEMORY,
+    "const namespace = ctx.flags.namespace || 'default';\n        const force = ctx.flags.force;\n        if (!key) {\n            output.printError('Key is required. Use: memory delete -k \"key\" [-n \"namespace\"]');",
+    "const namespace = ctx.flags.namespace;\n        const force = ctx.flags.force;\n        if (!key) {\n            output.printError('Key is required. Use: memory delete -k \"key\" -n \"namespace\"');")
+
+# 20g: CLI delete — add namespace-required check
+patch("20g: CLI delete namespace check",
+    CLI_MEMORY,
+    """            output.printError('Key is required. Use: memory delete -k "key" -n "namespace"');
+            return { success: false, exitCode: 1 };
+        }
+        if (!force && ctx.interactive) {""",
+    """            output.printError('Key is required. Use: memory delete -k "key" -n "namespace"');
+            return { success: false, exitCode: 1 };
+        }
+        if (!namespace || namespace === 'all') {
+            output.printError('Namespace is required (cannot be "all"). Use: memory delete -k "key" -n "namespace" (e.g. "patterns", "solutions", "tasks")');
+            return { success: false, exitCode: 1 };
+        }
+        if (!force && ctx.interactive) {""")
+
+# 20h: Core storeEntry() — remove dead 'default' parameter default + throw
+patch("20h: storeEntry no default namespace",
+    MI,
+    "const { key, value, namespace = 'default', generateEmbeddingFlag = true, tags = [], ttl, dbPath: customPath, upsert = false } = options;",
+    "const { key, value, namespace, generateEmbeddingFlag = true, tags = [], ttl, dbPath: customPath, upsert = false } = options;\n    if (!namespace || namespace === 'all') throw new Error('storeEntry: namespace is required (cannot be \"all\")');")
+
+# 20i: Core deleteEntry() — remove dead 'default' parameter default + throw
+patch("20i: deleteEntry no default namespace",
+    MI,
+    "export async function deleteEntry(options) {\n    const { key, namespace = 'default', dbPath: customPath } = options;\n    const swarmDir = path.join(process.cwd(), '.swarm');\n    const dbPath = customPath || path.join(swarmDir, 'memory.db');",
+    "export async function deleteEntry(options) {\n    const { key, namespace, dbPath: customPath } = options;\n    if (!namespace || namespace === 'all') throw new Error('deleteEntry: namespace is required (cannot be \"all\")');\n    const swarmDir = path.join(process.cwd(), '.swarm');\n    const dbPath = customPath || path.join(swarmDir, 'memory.db');")
+
+# ── Retrieve ops (from old Patch 21) ──
+
+# 21a: MCP retrieve — add 'namespace' to required + update description
+patch("21a: MCP retrieve require namespace",
+    MCP_MEMORY,
+    """        description: 'Retrieve a value from memory by key',
+        category: 'memory',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                key: { type: 'string', description: 'Memory key' },
+                namespace: { type: 'string', description: 'Namespace (default: "default")' },
+            },
+            required: ['key'],""",
+    """        description: 'Retrieve a value from memory by key',
+        category: 'memory',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                key: { type: 'string', description: 'Memory key' },
+                namespace: { type: 'string', description: 'Namespace (e.g. "patterns", "solutions", "tasks")' },
+            },
+            required: ['key', 'namespace'],""")
+
+# 21b: MCP retrieve — remove || 'default' fallback + add runtime throw
+patch("21b: MCP retrieve namespace no fallback",
+    MCP_MEMORY,
+    "const { getEntry } = await getMemoryFunctions();\n            const key = input.key;\n            const namespace = input.namespace || 'default';",
+    "const { getEntry } = await getMemoryFunctions();\n            const key = input.key;\n            const namespace = input.namespace;\n            if (!namespace || namespace === 'all') {\n                throw new Error('Namespace is required (cannot be \"all\"). Use namespace: \"patterns\", \"solutions\", or \"tasks\"');\n            }")
+
+# 21e: CLI retrieve — remove default: 'default' from flag definition
+patch("21e: CLI retrieve remove default",
+    CLI_MEMORY,
+    "            type: 'string',\n            default: 'default'\n        }\n    ],\n    action: async (ctx) => {\n        const key = ctx.flags.key || ctx.args[0];\n        const namespace = ctx.flags.namespace;\n        if (!key) {\n            output.printError('Key is required');",
+    "            type: 'string'\n        }\n    ],\n    action: async (ctx) => {\n        const key = ctx.flags.key || ctx.args[0];\n        const namespace = ctx.flags.namespace;\n        if (!key) {\n            output.printError('Key is required');")
+
+# 21f: CLI retrieve — add namespace-required check
+patch("21f: CLI retrieve namespace check",
+    CLI_MEMORY,
+    """        if (!key) {
+            output.printError('Key is required');
+            return { success: false, exitCode: 1 };
+        }
+        // Use sql.js directly for consistent data access""",
+    """        if (!key) {
+            output.printError('Key is required');
+            return { success: false, exitCode: 1 };
+        }
+        if (!namespace || namespace === 'all') {
+            output.printError('Namespace is required (cannot be "all"). Use --namespace or -n (e.g. "patterns", "solutions", "tasks")');
+            return { success: false, exitCode: 1 };
+        }
+        // Use sql.js directly for consistent data access""")
+
+# 21h: Core getEntry() — remove dead 'default' parameter default + throw
+patch("21h: getEntry no default namespace",
+    MI,
+    "const { key, namespace = 'default', dbPath: customPath } = options;\n    const swarmDir = path.join(process.cwd(), '.swarm');\n    const dbPath = customPath || path.join(swarmDir, 'memory.db');",
+    "const { key, namespace, dbPath: customPath } = options;\n    if (!namespace || namespace === 'all') throw new Error('getEntry: namespace is required (cannot be \"all\")');\n    const swarmDir = path.join(process.cwd(), '.swarm');\n    const dbPath = customPath || path.join(swarmDir, 'memory.db');")

package/patch/200-NS-002-targeted-require-namespace/sentinel

@@ -0,0 +1,2 @@
+grep "Namespace is required" mcp-tools/memory-tools.js
+grep "cannot be .all." mcp-tools/memory-tools.js

package/patch/210-NS-003-namespace-typo-pattern/README.md

@@ -0,0 +1,15 @@
+# NS-003: Namespace typo 'pattern' vs 'patterns'
+**Severity**: Medium
+**GitHub**: [#1136](https://github.com/ruvnet/claude-flow/issues/1136)
+## Root Cause
+hooks-tools.js used singular 'pattern' in 4 locations while every other file (memory-tools.js, CLI, README) uses 'patterns' (plural). Patterns stored via hooks were invisible to search.
+## Fix
+Replace all 4 occurrences of 'pattern' with 'patterns' in hooks-tools.js.
+## Files Patched
+- mcp-tools/hooks-tools.js
+## Dependencies
+
+Depends on NS-002 (200) — NS-003 fixes typos in code paths that NS-002 modifies.
+
+## Ops
+4 ops in fix.py (22a-d)

package/patch/210-NS-003-namespace-typo-pattern/fix.py

@@ -0,0 +1,23 @@
+# NS-003: Namespace typo 'pattern' vs 'patterns'
+# GitHub: #1136
+# 4 ops: 22a-d
+
+patch("22a: hooks pattern-store namespace",
+    MCP_HOOKS,
+    "namespace: 'pattern',",
+    "namespace: 'patterns',")
+
+patch("22b: hooks pattern-search default",
+    MCP_HOOKS,
+    "const namespace = params.namespace || 'pattern';",
+    "const namespace = params.namespace || 'patterns';")
+
+patch("22c: hooks pattern-search description",
+    MCP_HOOKS,
+    "description: 'Namespace to search (default: pattern)'",
+    "description: 'Namespace to search (default: patterns)'")
+
+patch("22d: hooks pattern-search note",
+    MCP_HOOKS,
+    'namespace "pattern".',
+    'namespace "patterns".')

package/patch/210-NS-003-namespace-typo-pattern/sentinel

@@ -0,0 +1 @@
+grep "|| 'patterns'" mcp-tools/hooks-tools.js

package/patch/220-RS-001-better-sqlite3-node24/README.md

@@ -0,0 +1,54 @@
+# RS-001: ruv-swarm MCP fails on Node 24 — better-sqlite3 missing native bindings
+
+**Severity**: Critical
+**Package**: `ruv-swarm@1.0.20`
+**GitHub**: [ruv-FANN#185](https://github.com/ruvnet/ruv-FANN/issues/185)
+
+## Symptoms
+
+`ruv-swarm mcp start` crashes immediately with:
+
+```
+Could not locate the bindings file. Tried:
+ → .../better-sqlite3/lib/binding/node-v137-linux-x64/better_sqlite3.node
+```
+
+The MCP server never starts. All ruv-swarm MCP tools are unavailable.
+
+## Root Cause
+
+`ruv-swarm@1.0.20` declares `"better-sqlite3": "^11.6.0"` in `package.json`.
+When installed via `npx`, npm resolves this to `better-sqlite3@11.10.0`.
+
+`better-sqlite3` 11.x does **not** ship prebuilt binaries for Node 24
+(ABI version `node-v137`). The native addon compilation fails silently
+during `npx` install — `build/Release/` contains intermediate `obj/`
+directories but no `better_sqlite3.node` binary.
+
+`better-sqlite3` 12.x (`^12.0.0`) added Node 24 to its supported engines
+and ships prebuilt binaries for `node-v137-linux-x64`.
+
+## Why @claude-flow/cli is unaffected
+
+`@claude-flow/cli` uses `sql.js` (pure WASM SQLite) instead of `better-sqlite3`.
+WASM has no native bindings and works on any Node version.
+
+## Fix
+
+Bump the dependency from `^11.6.0` to `^12.0.0` in `ruv-swarm/package.json`.
+
+## Files Patched
+
+- `ruv-swarm` `package.json` (in npx cache)
+
+## Ops
+
+1 op in fix.py (JSON string replacement in package.json)
+1 op: `npm rebuild better-sqlite3` in the npx cache directory
+
+## Verification
+
+```bash
+npx -y ruv-swarm --version
+# Should print version without crashing
+```

package/patch/220-RS-001-better-sqlite3-node24/fix.py

@@ -0,0 +1,22 @@
+# RS-001: ruv-swarm better-sqlite3 Node 24 compatibility
+# GitHub: ruvnet/ruv-FANN#185
+#
+# ruv-swarm@1.0.20 pins better-sqlite3@^11.6.0 which lacks Node 24
+# prebuilt binaries. Bump to ^12.0.0 which supports Node 24 (v137 ABI).
+#
+# This patch targets the ruv-swarm package.json in the npx cache,
+# NOT @claude-flow/cli. It uses the same patch() infrastructure but
+# finds its own target path.
+
+import glob
+
+# Find ruv-swarm package.json in npx cache
+rs_pkg_candidates = glob.glob(os.path.expanduser(
+    "~/.npm/_npx/*/node_modules/ruv-swarm/package.json"
+))
+rs_pkg = rs_pkg_candidates[0] if rs_pkg_candidates else ""
+
+patch("RS-001: better-sqlite3 ^11 -> ^12 (Node 24)",
+    rs_pkg,
+    '"better-sqlite3": "^11.6.0"',
+    '"better-sqlite3": "^12.0.0"')

package/patch/220-RS-001-better-sqlite3-node24/rebuild.sh

@@ -0,0 +1,31 @@
+#!/bin/bash
+# RS-001: After patching package.json, reinstall better-sqlite3 with
+# correct prebuilt binaries for Node 24.
+#
+# Usage: bash rebuild.sh
+
+set -euo pipefail
+
+RS_DIR=$(find ~/.npm/_npx -path "*/node_modules/ruv-swarm" -type d 2>/dev/null | head -1)
+
+if [ -z "$RS_DIR" ]; then
+  echo "[RS-001] ruv-swarm not found in npx cache, skipping rebuild"
+  exit 0
+fi
+
+# Install at the npx cache root (where better-sqlite3 is hoisted),
+# not inside ruv-swarm's own node_modules
+NPX_ROOT=$(echo "$RS_DIR" | sed 's|/node_modules/ruv-swarm$||')
+
+echo "[RS-001] Reinstalling better-sqlite3@^12.0.0 in: $NPX_ROOT"
+
+cd "$NPX_ROOT"
+npm install better-sqlite3@"^12.0.0" --no-save 2>&1
+
+echo "[RS-001] Verifying..."
+if npx -y ruv-swarm --version 2>/dev/null; then
+  echo "[RS-001] OK: ruv-swarm starts successfully"
+else
+  echo "[RS-001] WARN: ruv-swarm still failing — may need full cache clear"
+  echo "  Try: rm -rf ~/.npm/_npx/$(basename $(dirname $RS_DIR)) && npx -y ruv-swarm --version"
+fi

package/patch/220-RS-001-better-sqlite3-node24/sentinel

@@ -0,0 +1,2 @@
+package: ruv-swarm
+grep "better-sqlite3.*\^12" package.json

package/patch/230-RV-001-force-learn-tick/README.md

@@ -0,0 +1,31 @@
+# RV-001: force-learn command calls intel.tick() which doesn't exist
+
+**Severity**: Medium
+**GitHub**: [#1156](https://github.com/ruvnet/claude-flow/issues/1156)
+
+## Root Cause
+
+The `hooks force-learn` command (line 4224) creates an Intelligence instance with
+`skipEngine: true` (fast mode), then calls `intel.tick()`. The `tick()` method
+exists on the engine object (`eng.tick()`), not on the Intelligence class itself.
+With `skipEngine: true`, no engine is created, so the call crashes.
+
+```javascript
+// Line 4224-4226
+const intel = new Intelligence({ skipEngine: true });  // Fast mode
+intel.tick();  // BUG: tick() doesn't exist on Intelligence
+```
+
+## Fix
+
+1. Remove `skipEngine: true` so the engine can initialize
+2. Call `intel.engine.tick()` instead of `intel.tick()`
+3. Add null check for when engine fails to load
+
+## Files Patched
+
+- `ruvector/bin/cli.js`
+
+## Ops
+
+2 ops in fix.py

package/patch/230-RV-001-force-learn-tick/fix.py

@@ -0,0 +1,14 @@
+# RV-001: force-learn calls intel.tick() which doesn't exist
+# The tick() method is on the engine, not the Intelligence class
+
+# Fix 1: Remove skipEngine: true so engine can initialize
+patch("RV-001a: remove skipEngine from force-learn",
+    ruvector_cli,
+    "const intel = new Intelligence({ skipEngine: true });  // Fast mode\n    intel.tick();",
+    "const intel = new Intelligence();  // Need engine for tick()\n    const eng = intel.engine;\n    if (eng) { eng.tick(); }")
+
+# Fix 2: Update the success message to handle no-engine case
+patch("RV-001b: handle no-engine case in force-learn",
+    ruvector_cli,
+    "if (eng) { eng.tick(); }\n    console.log(JSON.stringify({ success: true, result: 'Learning cycle triggered', stats: intel.stats() }));",
+    "if (eng) { eng.tick(); }\n    console.log(JSON.stringify({ success: true, result: eng ? 'Learning cycle triggered' : 'Engine not available', stats: intel.stats() }));")

package/patch/230-RV-001-force-learn-tick/sentinel

@@ -0,0 +1,2 @@
+package: ruvector
+grep "Need engine for tick" bin/cli.js

package/patch/240-RV-002-trajectory-load/README.md

@@ -0,0 +1,28 @@
+# RV-002: activeTrajectories not loaded from saved file
+
+**Severity**: High
+**GitHub**: [#1157](https://github.com/ruvnet/claude-flow/issues/1157)
+
+## Root Cause
+
+The `load()` method in the Intelligence class (line ~2721) doesn't include
+`activeTrajectories` in the returned object. When `trajectory-begin` saves an
+active trajectory, it's written to the file, but the next command
+(`trajectory-step`) creates a new Intelligence instance that doesn't load
+`activeTrajectories` back from the file.
+
+This causes `trajectory-step` and `trajectory-end` to fail with
+"No active trajectory" error.
+
+## Fix
+
+1. Add `activeTrajectories: {}` to the defaults object
+2. Add `activeTrajectories: data.activeTrajectories || {}` to the loaded data return
+
+## Files Patched
+
+- `ruvector/bin/cli.js`
+
+## Ops
+
+2 ops in fix.py

package/patch/240-RV-002-trajectory-load/fix.py

@@ -0,0 +1,14 @@
+# RV-002: activeTrajectories not loaded from saved file
+# The load() method doesn't include activeTrajectories in the return object
+
+# Fix 1: Add activeTrajectories to defaults
+patch("RV-002a: add activeTrajectories to defaults",
+    ruvector_cli,
+    "stats: { total_patterns: 0, total_memories: 0, total_trajectories: 0, total_errors: 0, session_count: 0, last_session: 0 }\n    };",
+    "stats: { total_patterns: 0, total_memories: 0, total_trajectories: 0, total_errors: 0, session_count: 0, last_session: 0 },\n      activeTrajectories: {}\n    };")
+
+# Fix 2: Add activeTrajectories to loaded data return
+patch("RV-002b: load activeTrajectories from file",
+    ruvector_cli,
+    "// Preserve learning data if present\n          learning: data.learning || undefined",
+    "// Preserve learning data if present\n          learning: data.learning || undefined,\n          // Preserve active trajectories for cross-command persistence\n          activeTrajectories: data.activeTrajectories || {}")

package/patch/240-RV-002-trajectory-load/sentinel

@@ -0,0 +1,2 @@
+package: ruvector
+grep "activeTrajectories: data.activeTrajectories" bin/cli.js

package/patch/250-RV-003-trajectory-stats-sync/README.md

@@ -0,0 +1,31 @@
+# RV-003: trajectory-end does not update stats counters
+
+**Severity**: Medium
+**GitHub**: [ruv-FANN#186](https://github.com/ruvnet/ruv-FANN/issues/186)
+
+## Root Cause
+
+The `hooks trajectory-end` command pushes completed trajectories to
+`data.trajectories[]` and calls `intel.save()`, but never syncs the
+`data.stats.total_trajectories` counter. The `hooks stats` command reads
+the stale counter and reports 0 trajectories even when the array has entries.
+
+Compare with `learn()` which correctly does:
+```javascript
+this.data.stats.total_trajectories = this.data.trajectories.length;
+```
+
+## Fix
+
+Add stats counter sync before `intel.save()` in the `trajectory-end` handler:
+- `total_trajectories` from `trajectories.length`
+- `total_patterns` from `Object.keys(patterns).length`
+- `total_memories` from `memories.length`
+
+## Files Patched
+
+- `bin/cli.js` (ruvector)
+
+## Ops
+
+1 op in fix.py

package/patch/250-RV-003-trajectory-stats-sync/fix.py

@@ -0,0 +1,18 @@
+# RV-003: trajectory-end does not update stats counters
+# GitHub: #186 (ruv-FANN)
+
+patch("RV-003a: sync stats counters before save in trajectory-end",
+    ruvector_cli,
+    """    if (!intel.data.trajectories) intel.data.trajectories = [];
+    intel.data.trajectories.push(traj);
+    delete trajectories[latestTrajId];
+    intel.save();""",
+    """    if (!intel.data.trajectories) intel.data.trajectories = [];
+    intel.data.trajectories.push(traj);
+    delete trajectories[latestTrajId];
+    // RV-003: sync stats counters from actual data before saving
+    if (!intel.data.stats) intel.data.stats = {};
+    intel.data.stats.total_trajectories = intel.data.trajectories.length;
+    intel.data.stats.total_patterns = Object.keys(intel.data.patterns || {}).length;
+    intel.data.stats.total_memories = (intel.data.memories || []).length;
+    intel.save();""")

package/patch/250-RV-003-trajectory-stats-sync/sentinel

@@ -0,0 +1,2 @@
+package: ruvector
+grep "RV-003: sync stats" bin/cli.js

package/patch/260-SG-001-init-settings/README.md

@@ -0,0 +1,29 @@
+# SG-001: Init generates invalid settings
+
+**Severity**: High
+**GitHub**: [#1150](https://github.com/ruvnet/claude-flow/issues/1150)
+
+## Root Cause
+
+1. **Invalid hook events**: `TeammateIdle` and `TaskCompleted` are not valid Claude Code hooks
+2. **Overly broad permissions**: Glob patterns like `Bash(npx @claude-flow*)` match unintended commands
+3. **Relative hook paths**: `node .claude/helpers/` breaks when running from subdirectories
+4. **Phantom statusLine config**: `--minimal` and `init hooks` emit `statusLine` config
+   referencing `statusline.cjs` but the file is never generated (`components.statusline: false`)
+5. **MINIMAL statusline inconsistency**: `MINIMAL_INIT_OPTIONS` inherits
+   `statusline.enabled: true` from DEFAULT but sets `components.statusline: false`
+
+## Fix
+
+Replace invalid hooks with `SubagentStop` (valid event), fix permission patterns, use
+`$CLAUDE_PROJECT_DIR` for all hook paths, gate statusLine config emission on
+`components.statusline`, and override `statusline.enabled` to `false` in MINIMAL preset.
+
+## Files Patched
+
+- `init/settings-generator.js`
+- `init/types.js`
+
+## Ops
+
+5 ops in fix.py (SG-001a: replace invalid hooks, SG-001b: fix permissions, SG-001c: CLAUDE_PROJECT_DIR paths, SG-001d: gate statusLine on component flag, SG-001e: MINIMAL statusline.enabled=false)

package/patch/260-SG-001-init-settings/fix.py

@@ -0,0 +1,143 @@
+# SG-001: Init generates invalid settings
+# GitHub: #1150
+#
+# Problems:
+# 1. TeammateIdle and TaskCompleted are not valid Claude Code hook events
+# 2. Permission patterns are too broad (glob wildcards)
+# 3. Hook paths use relative .claude/helpers/ which breaks from subdirectories
+#
+# Fix: Replace invalid hooks with SubagentStop, fix permissions, use $CLAUDE_PROJECT_DIR
+
+# SG-001a: Replace invalid hooks with SubagentStop (with correct $CLAUDE_PROJECT_DIR paths)
+patch("SG-001a: replace invalid hooks with SubagentStop",
+    SETTINGS_GEN,
+    """    // SubagentStart — status update
+    hooks.SubagentStart = [
+        {
+            hooks: [
+                {
+                    type: 'command',
+                    command: 'node .claude/helpers/hook-handler.cjs status',
+                    timeout: 3000,
+                },
+            ],
+        },
+    ];
+    // TeammateIdle — auto-assign pending tasks to idle teammates
+    hooks.TeammateIdle = [
+        {
+            hooks: [
+                {
+                    type: 'command',
+                    command: 'node .claude/helpers/hook-handler.cjs post-task',
+                    timeout: 5000,
+                },
+            ],
+        },
+    ];
+    // TaskCompleted — train patterns and record completion
+    hooks.TaskCompleted = [
+        {
+            hooks: [
+                {
+                    type: 'command',
+                    command: 'node .claude/helpers/hook-handler.cjs post-task',
+                    timeout: 5000,
+                },
+            ],
+        },
+    ];
+    return hooks;""",
+    """    // SubagentStart — status update
+    hooks.SubagentStart = [
+        {
+            hooks: [
+                {
+                    type: 'command',
+                    command: 'node "$CLAUDE_PROJECT_DIR"/.claude/helpers/hook-handler.cjs status',
+                    timeout: 3000,
+                },
+            ],
+        },
+    ];
+    // SubagentStop — task completion + teammate idle (replaces invalid TeammateIdle/TaskCompleted)
+    hooks.SubagentStop = [
+        {
+            hooks: [
+                {
+                    type: 'command',
+                    command: 'node "$CLAUDE_PROJECT_DIR"/.claude/helpers/hook-handler.cjs post-task',
+                    timeout: 5000,
+                },
+                {
+                    type: 'command',
+                    command: 'npx @claude-flow/cli@latest hooks teammate-idle',
+                    timeout: 5000,
+                },
+            ],
+        },
+    ];
+    // TeammateIdle removed — not a valid Claude Code hook event (see #1150)
+    // TaskCompleted removed — not a valid Claude Code hook event (see #1150)
+    return hooks;""")
+
+# SG-001b: Fix permission patterns (too broad) and add $CLAUDE_PROJECT_DIR
+patch("SG-001b: fix permissions",
+    SETTINGS_GEN,
+    """    settings.permissions = {
+        allow: [
+            'Bash(npx @claude-flow*)',
+            'Bash(npx claude-flow*)',
+            'Bash(node .claude/*)',
+            'mcp__claude-flow__:*',
+        ],""",
+    """    settings.permissions = {
+        allow: [
+            'Bash(npx @claude-flow/cli:*)',
+            'Bash(npx claude-flow:*)',
+            'Bash(node "$CLAUDE_PROJECT_DIR"/.claude/*)',
+            'mcp__claude-flow__:*',
+        ],""")
+
+# SG-001c: Fix all other hook paths to use $CLAUDE_PROJECT_DIR
+# (catches statusline, pre-bash, post-edit, route, session hooks, etc.)
+patch_all("SG-001c: use CLAUDE_PROJECT_DIR for all hook paths",
+    SETTINGS_GEN,
+    "command: 'node .claude/helpers/",
+    'command: \'node "$CLAUDE_PROJECT_DIR"/.claude/helpers/')
+
+# SG-001d: statusLine config emitted even when statusline component is off
+# MINIMAL_INIT_OPTIONS has components.statusline=false but statusline.enabled=true.
+# Settings-generator checks statusline.enabled, emitting statusLine config for a
+# file that won't exist. Gate on components.statusline too.
+patch("SG-001d: only emit statusLine when statusline component is generated",
+    SETTINGS_GEN,
+    """    if (options.statusline.enabled) {""",
+    """    // SG-001: Only emit statusLine config if the component will actually be generated
+    if (options.components.statusline && options.statusline.enabled) {""")
+
+# SG-001e: MINIMAL_INIT_OPTIONS inherits statusline.enabled=true from DEFAULT
+# but sets components.statusline=false. Override to prevent inconsistency.
+# (SG-001d guards at the consumer; this fixes the producer for MINIMAL)
+patch("SG-001e: MINIMAL statusline.enabled matches components.statusline",
+    TYPES,
+    """    hooks: {
+        ...DEFAULT_INIT_OPTIONS.hooks,
+        userPromptSubmit: false,
+        stop: false,
+        notification: false,
+    },
+    skills: {""",
+    """    hooks: {
+        ...DEFAULT_INIT_OPTIONS.hooks,
+        userPromptSubmit: false,
+        stop: false,
+        notification: false,
+    },
+    // SG-001: statusline file not generated (components.statusline: false)
+    // so disable the feature flag to prevent dangling settings.json references
+    statusline: {
+        ...DEFAULT_INIT_OPTIONS.statusline,
+        enabled: false,
+    },
+    skills: {""")

package/patch/260-SG-001-init-settings/sentinel

@@ -0,0 +1,4 @@
+grep "hooks.SubagentStop" init/settings-generator.js
+grep "CLAUDE_PROJECT_DIR" init/settings-generator.js
+grep "components.statusline && options.statusline.enabled" init/settings-generator.js
+grep "SG-001" init/types.js