brainclaw 1.11.0 → 1.11.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -417,6 +417,14 @@ npm run test:coverage # with coverage report
417
417
 
418
418
  For older releases (v0.x and the early v1.0 launch series), `git log` on `master` is the source of truth — every release commit follows the `chore(release): bump version to <semver>` convention, and the matching feature/fix commits reference their plan id (e.g. `feat(mcp): self-heal ... (pln#478)`).
419
419
 
420
+ ### v1.11.1
421
+
422
+ Agent-identity & session-hook resilience, from a fresh-CLI dogfood on a monorepo:
423
+
424
+ - **Session hooks no longer spam `UserPromptSubmit hook error` on every prompt.** Root cause was an agent-identity error whose own remediation (`register-agent --set-current`) the resolver ignored, swallowed by `2>/dev/null`. Now: the error hint points at what actually works (`--agent` / `$BRAINCLAW_AGENT_NAME`), a **single registered agent auto-resolves** with no env signal, and `session-start`/`context-diff`/`session-end` run with `--hook` so they degrade to exit 0 + `~/.brainclaw/hook.log` instead of erroring the prompt loop. Multi-agent safety (pln#562) is unchanged.
425
+ - **`brainclaw doctor --fix-hooks`** purges stale/broken/duplicate brainclaw hooks across all Claude Code settings scopes (user + cwd) and rewrites the canonical ones — for the broken hooks that accumulate where `setup`'s git-repo discovery never reached.
426
+ - **`setup` repo discovery recurses** (bounded, skipping `node_modules`/build dirs) so repos nested deep in a workspace are found instead of silently missed, and it registers the detected agent so the hooks it installs can resolve an identity.
427
+
420
428
  ### v1.11.0
421
429
 
422
430
  Monorepo project-resolution + Code Map, cross-project relocation, and dispatch-worktree hygiene — from a multi-project (monorepo) dogfood:
Binary file
package/dist/cli.js CHANGED
@@ -774,6 +774,7 @@ program
774
774
  .option('--json', 'Output as JSON dashboard')
775
775
  .option('--migration-check', 'Report versioned documents that need schema migration')
776
776
  .option('--fix-agent-ignore', 'Add missing .gitignore entries for generated local Brainclaw agent files')
777
+ .option('--fix-hooks', 'Purge stale/broken/duplicate brainclaw session hooks across all Claude Code settings scopes (user + cwd) and rewrite the canonical ones')
777
778
  .option('--fix', 'Fix auto-resolvable issues (e.g. drifting MCP configs)')
778
779
  .option('--repair', 'Rebuild dist/ when the MCP runtime is missing or stale')
779
780
  .option('--after-migration', 'Run the v1.0 post-migration health check only (exits non-zero on any failure)')
@@ -1435,6 +1436,7 @@ program
1435
1436
  .option('--model <id>', 'Model identifier (e.g. claude-sonnet-4-6)')
1436
1437
  .option('--maintenance-mode <mode>', 'Maintenance mode: full (default) or fast')
1437
1438
  .option('--include-context', 'Output full project context after starting session (replaces separate context call)')
1439
+ .option('--hook', 'Hook mode: degrade to exit 0 + ~/.brainclaw/hook.log on failure (advisory session hooks)')
1438
1440
  .option('--json', 'Output as JSON')
1439
1441
  .action(async (options) => {
1440
1442
  await runSessionStart(options);
@@ -1452,6 +1454,7 @@ program
1452
1454
  .option('--dispatch-review', 'When used with --reflect-handoff, auto-dispatch a code review if the handoff is reviewable')
1453
1455
  .option('--reviewer <name>', 'Explicit reviewer to route the reflected handoff review to')
1454
1456
  .option('--no-reflect', 'Suppress the dogfooding reflection prompt (project + your surfaces/skills/tools), shown by default')
1457
+ .option('--hook', 'Hook mode: degrade to exit 0 + ~/.brainclaw/hook.log on failure (advisory Stop hook)')
1455
1458
  .option('--json', 'Output as JSON')
1456
1459
  .action(async (options) => {
1457
1460
  await runSessionEnd({
@@ -1462,6 +1465,7 @@ program
1462
1465
  dispatchReview: options.dispatchReview,
1463
1466
  reviewer: options.reviewer,
1464
1467
  reflect: options.reflect,
1468
+ hook: options.hook,
1465
1469
  });
1466
1470
  });
1467
1471
  // --- whoami ---
@@ -1753,9 +1757,10 @@ program
1753
1757
  .description('Show what changed in memory since last context read, a session start, or a given timestamp')
1754
1758
  .option('--since <date>', 'Show changes since this ISO date')
1755
1759
  .option('--session <id>', 'Show changes since the start of this session')
1760
+ .option('--hook', 'Hook mode: exit 0 silently when there is no diff baseline (advisory session hooks)')
1756
1761
  .option('--json', 'Output as JSON')
1757
1762
  .action((options) => {
1758
- runContextDiff({ since: options.since, session: options.session, json: options.json });
1763
+ runContextDiff({ since: options.since, session: options.session, json: options.json, hook: options.hook });
1759
1764
  });
1760
1765
  program
1761
1766
  .command('capability <subcommand> [args...]')
@@ -4,18 +4,45 @@ import { loadInstructions, resolveInstructions } from '../core/instructions.js';
4
4
  import { memoryExists } from '../core/io.js';
5
5
  import { loadState } from '../core/state.js';
6
6
  import { isTrapActive } from '../core/traps.js';
7
+ import { logHookDiagnostic } from '../core/hook-log.js';
7
8
  /**
8
9
  * Hybrid context-diff: always includes critical anchors (active claims,
9
10
  * top traps, instructions) so the agent stays grounded, plus the memory
10
11
  * delta since last context read.
11
12
  */
12
13
  export function runContextDiff(options = {}) {
14
+ try {
15
+ contextDiffBody(options);
16
+ }
17
+ catch (e) {
18
+ const message = e instanceof Error ? e.message : String(e);
19
+ if (options.hook) {
20
+ // Advisory hook (trp#917): any failure past the early guards — e.g. a
21
+ // corrupt session snapshot / claim / instruction file feeding buildContextDiff
22
+ // or buildCriticalAnchors — must NOT error the prompt loop. Log + exit 0.
23
+ logHookDiagnostic(`context-diff skipped: ${message}`);
24
+ return;
25
+ }
26
+ console.error(`Error: ${message}`);
27
+ process.exit(1);
28
+ }
29
+ }
30
+ function contextDiffBody(options) {
13
31
  if (!memoryExists(options.cwd)) {
32
+ if (options.hook) {
33
+ logHookDiagnostic('context-diff skipped: .brainclaw/ not found');
34
+ return;
35
+ }
14
36
  console.error('Error: .brainclaw/ not found. Run `brainclaw init` first.');
15
37
  process.exit(1);
16
38
  }
17
39
  const resolved = resolveContextDiffSince(options);
18
40
  if (!resolved.since) {
41
+ if (options.hook) {
42
+ // No diff baseline yet (e.g. first prompt before a session marker exists).
43
+ // Nothing to surface — exit 0 silently so the hook never errors (trp#917).
44
+ return;
45
+ }
19
46
  if (options.session) {
20
47
  console.error(`Error: session '${options.session}' not found in session snapshots or audit log.`);
21
48
  process.exit(1);
@@ -25,6 +52,10 @@ export function runContextDiff(options = {}) {
25
52
  }
26
53
  const diff = buildContextDiff({ ...options, includeItems: true });
27
54
  if (!diff) {
55
+ if (options.hook) {
56
+ logHookDiagnostic('context-diff skipped: unable to build diff');
57
+ return;
58
+ }
28
59
  console.error('Error: unable to build context diff.');
29
60
  process.exit(1);
30
61
  }
@@ -1,5 +1,6 @@
1
1
  import crypto from 'node:crypto';
2
2
  import fs from 'node:fs';
3
+ import os from 'node:os';
3
4
  import path from 'node:path';
4
5
  import * as childProcess from 'node:child_process';
5
6
  import { reconcileAllOpenRuns } from '../core/agentrun-reconciler.js';
@@ -35,7 +36,7 @@ import { assessBrainclawVersion, detectConcurrentInstallations } from '../core/b
35
36
  import { resolveStoreChain } from '../core/store-resolution.js';
36
37
  import { listWorktrees, detectSharedCheckoutRisk } from '../core/worktree.js';
37
38
  import { resolveCrossProjectLinks, detectCrossProjectCycles } from '../core/cross-project.js';
38
- import { auditLocalAgentWorkspaceFiles, ensureGitignoreEntries, patchAllMcpConfigs } from '../core/agent-files.js';
39
+ import { auditLocalAgentWorkspaceFiles, ensureGitignoreEntries, patchAllMcpConfigs, fixClaudeCodeHooksAllScopes } from '../core/agent-files.js';
39
40
  import { summarizeWorkspaceProjects } from '../core/workspace-projects.js';
40
41
  import { detectStaleness, staleSummary } from '../core/staleness.js';
41
42
  import { InboxMessageSchema } from '../core/schema.js';
@@ -654,6 +655,27 @@ export function runDoctor(options = {}) {
654
655
  process.exit(report.exit_code);
655
656
  return;
656
657
  }
658
+ if (options.fixHooks) {
659
+ const cwd = options.cwd ?? process.cwd();
660
+ const results = fixClaudeCodeHooksAllScopes(cwd, os.homedir());
661
+ const fixed = results.filter((r) => r.changed);
662
+ if (options.json) {
663
+ console.log(JSON.stringify({ ok: true, action: 'fix-hooks', results }, null, 2));
664
+ }
665
+ else if (fixed.length === 0) {
666
+ console.log('✔ No stale/broken brainclaw session hooks found across Claude Code settings scopes.');
667
+ }
668
+ else {
669
+ for (const r of fixed) {
670
+ const suffix = r.collapsed > 0
671
+ ? ` (collapsed ${r.collapsed} stale/duplicate entr${r.collapsed === 1 ? 'y' : 'ies'})`
672
+ : '';
673
+ console.log(`✔ Sanitized brainclaw session hooks in ${r.filePath}${suffix}`);
674
+ }
675
+ console.log(' → Restart your Claude Code session to pick up the canonical hooks.');
676
+ }
677
+ return;
678
+ }
657
679
  if (options.repair) {
658
680
  try {
659
681
  const result = repairDistRuntime(options);
@@ -39,6 +39,7 @@ import { extractFilesFromDiff } from '../commands/handoff.js';
39
39
  import { capHandoffDiff } from '../core/handoff-snapshot.js';
40
40
  import { suggestCompaction } from '../core/memory-compactor.js';
41
41
  import { dispatchReview } from '../core/dispatcher.js';
42
+ import { logHookDiagnostic } from '../core/hook-log.js';
42
43
  export const REFLECTION_QUESTIONS = [
43
44
  'Dogfooding the project — using brainclaw to do real work this session, what friction did you hit (slow reads, confusing surfaces, missing affordances, awkward workflows)? What concrete change to the project would have removed it?',
44
45
  'Your surfaces, skills & tools — did your generated surface files (CLAUDE.md / agent surface), skills (SKILL.md), or tools (MCP / CLI) help or get in the way? Name at least one concrete edit that would make them serve you better next time.',
@@ -113,7 +114,13 @@ export async function runSessionEnd(options = {}) {
113
114
  }
114
115
  }
115
116
  catch (e) {
116
- console.error(`Error: ${e instanceof Error ? e.message : String(e)}`);
117
+ const message = e instanceof Error ? e.message : String(e);
118
+ if (options.hook) {
119
+ // Advisory Stop hook (trp#917): never fail the prompt loop. Log + exit 0.
120
+ logHookDiagnostic(`session-end skipped: ${message}`);
121
+ return;
122
+ }
123
+ console.error(`Error: ${message}`);
117
124
  process.exit(1);
118
125
  }
119
126
  }
@@ -11,6 +11,7 @@ import { writeContextMarker } from '../core/freshness.js';
11
11
  import { saveRuntimeNote, generateRuntimeNoteId } from '../core/runtime.js';
12
12
  import { nowISO, generateId } from '../core/ids.js';
13
13
  import { appendAuditEntry } from '../core/audit.js';
14
+ import { logHookDiagnostic } from '../core/hook-log.js';
14
15
  import { releaseStaleClaimsFromOtherAgents } from '../core/claims.js';
15
16
  import { SessionSnapshotSchema } from '../core/schema.js';
16
17
  import { auditLocalAgentWorkspaceFiles } from '../core/agent-files.js';
@@ -93,7 +94,13 @@ export async function runSessionStart(options = {}) {
93
94
  }
94
95
  }
95
96
  catch (e) {
96
- console.error(`Error: ${e instanceof Error ? e.message : String(e)}`);
97
+ const message = e instanceof Error ? e.message : String(e);
98
+ if (options.hook) {
99
+ // Advisory hook (trp#917): never fail the prompt loop. Log + exit 0.
100
+ logHookDiagnostic(`session-start skipped: ${message}`);
101
+ return;
102
+ }
103
+ console.error(`Error: ${message}`);
97
104
  process.exit(1);
98
105
  }
99
106
  }
@@ -12,6 +12,7 @@ import { ensureClaudeCodeUserSettings, ensureClaudeCodeUserCommand, ensureCursor
12
12
  import { MEMORY_DIR, memoryExists } from '../core/io.js';
13
13
  import { ensureUserStore, readSetupState, resolveHomeDir, writeSetupState } from '../core/setup-state.js';
14
14
  import { writeDetectedAgentHooks } from './hooks.js';
15
+ import { normalizeAgentName, findAgentIdentityByName, registerAgentIdentity } from '../core/agent-registry.js';
15
16
  export { readSetupState } from '../core/setup-state.js';
16
17
  // ─── Constants ────────────────────────────────────────────────────────────────
17
18
  export const ALL_KNOWN_AGENTS = [
@@ -83,37 +84,59 @@ export function parseRoots(input, env = process.env) {
83
84
  return results;
84
85
  }
85
86
  // ─── Step 2: Scan repos ───────────────────────────────────────────────────────
86
- export function scanGitRepos(roots) {
87
+ /** Heavy / generated directories never worth descending into when hunting repos. */
88
+ const SCAN_SKIP_DIRS = new Set([
89
+ 'node_modules', '.git', 'dist', 'dist-test', 'build', 'out', '.next',
90
+ 'coverage', 'vendor', '.venv', 'venv', '__pycache__', '.cache', 'target', '.gradle',
91
+ ]);
92
+ /**
93
+ * Discover git repositories under each root, recursing up to `maxDepth` levels.
94
+ *
95
+ * trp#918: the previous scan was depth-1 only (root + immediate children), so in
96
+ * a workspace like /srv with repos nested at /srv/dev/repos/global/<svc> it found
97
+ * just the shallow ones and silently missed the rest. This walk descends (skipping
98
+ * heavy/build/hidden dirs, bounded by `maxDepth`) and surfaces every directory
99
+ * that contains a `.git`. It keeps descending past a found repo because a
100
+ * workspace repo can legitimately contain independent child repos (the prior
101
+ * depth-1 scan already surfaced depth-1 children); the user selects which to init.
102
+ */
103
+ export function scanGitRepos(roots, maxDepth = 6) {
87
104
  const repos = [];
88
105
  const seen = new Set();
89
- for (const root of roots) {
90
- const candidates = [root];
91
- try {
92
- const entries = fs.readdirSync(root, { withFileTypes: true });
93
- for (const entry of entries) {
94
- if (!entry.isDirectory())
95
- continue;
96
- candidates.push(path.join(root, entry.name));
97
- }
98
- }
99
- catch {
100
- // skip unreadable dirs
101
- }
102
- for (const candidate of candidates) {
103
- const repoPath = path.resolve(candidate);
104
- if (seen.has(repoPath))
105
- continue;
106
- if (isBrainclawInternalPath(repoPath))
107
- continue;
108
- if (!fs.existsSync(path.join(repoPath, '.git')))
109
- continue;
106
+ const walk = (dir, depth) => {
107
+ const repoPath = path.resolve(dir);
108
+ if (seen.has(repoPath))
109
+ return;
110
+ if (isBrainclawInternalPath(repoPath))
111
+ return;
112
+ if (fs.existsSync(path.join(repoPath, '.git'))) {
110
113
  seen.add(repoPath);
111
114
  repos.push({
112
115
  path: repoPath,
113
116
  name: path.basename(repoPath) || repoPath,
114
117
  alreadyInitialised: memoryExists(repoPath),
115
118
  });
119
+ // Keep descending — a workspace repo may contain independent child repos.
120
+ }
121
+ if (depth >= maxDepth)
122
+ return;
123
+ let entries;
124
+ try {
125
+ entries = fs.readdirSync(repoPath, { withFileTypes: true });
126
+ }
127
+ catch {
128
+ return; // unreadable dir — skip
116
129
  }
130
+ for (const entry of entries) {
131
+ if (!entry.isDirectory())
132
+ continue;
133
+ if (SCAN_SKIP_DIRS.has(entry.name) || entry.name.startsWith('.'))
134
+ continue;
135
+ walk(path.join(repoPath, entry.name), depth + 1);
136
+ }
137
+ };
138
+ for (const root of roots) {
139
+ walk(root, 0);
117
140
  }
118
141
  return repos;
119
142
  }
@@ -309,6 +332,39 @@ export async function initReposAndConfigureAgents(selectedRepos, selectedAgents,
309
332
  }
310
333
  return { initialisedRepos, configActions };
311
334
  }
335
+ /**
336
+ * Ensure a resolvable session identity exists (pln#596 / trp#917). setup installs
337
+ * session hooks that need an agent identity; a hook with no identity now degrades
338
+ * silently (exit 0, logs to ~/.brainclaw/hook.log) but won't inject context. If we
339
+ * detected the running agent, register it in each selected repo so the hook resolves
340
+ * immediately. Otherwise emit a clear, actionable note — we don't guess an agent (a
341
+ * wrong identity is worse than none, and the single-registered-agent fallback wants
342
+ * exactly one).
343
+ */
344
+ export function ensureSessionIdentityForRepos(repoPaths, detectedName) {
345
+ if (detectedName) {
346
+ const normalized = normalizeAgentName(detectedName);
347
+ let registered = 0;
348
+ for (const repoPath of repoPaths) {
349
+ try {
350
+ if (!findAgentIdentityByName(normalized, repoPath)) {
351
+ registerAgentIdentity({ agentName: normalized, kind: 'agent', trustLevel: 'contributor', cwd: repoPath });
352
+ registered++;
353
+ }
354
+ }
355
+ catch {
356
+ /* best-effort — identity registration must not abort setup */
357
+ }
358
+ }
359
+ if (registered > 0) {
360
+ console.log(` ✔ Registered session identity '${normalized}' in ${registered} repo(s) so session hooks resolve immediately.`);
361
+ }
362
+ return;
363
+ }
364
+ console.log('\n⚠ No agent identity registered yet — session hooks will no-op (exit 0) until one resolves.');
365
+ console.log(' Set `export BRAINCLAW_AGENT_NAME=<agent>` in your shell, or run `brainclaw register-agent <name>`.');
366
+ console.log(' (A single registered agent then resolves automatically; running inside a detected agent auto-registers on first session.)');
367
+ }
312
368
  // ─── Step 7: Reload reminder ──────────────────────────────────────────────────
313
369
  export function printReloadReminder(detectedAgent) {
314
370
  console.log('');
@@ -540,6 +596,8 @@ export async function runSetup(options = {}) {
540
596
  // Step 6: Init repos + configure agents
541
597
  console.log('\n→ Initialising repositories and configuring agents...');
542
598
  const { initialisedRepos, configActions } = await initReposAndConfigureAgents(selectedRepos, selectedAgents, env);
599
+ // Step 6.5: ensure a resolvable session identity for the hooks we just installed.
600
+ ensureSessionIdentityForRepos(selectedRepos.map((r) => r.path), detectedName);
543
601
  // Step 7: VS Code extension
544
602
  installVscodeExtension();
545
603
  // Save state
@@ -1095,6 +1095,98 @@ function replaceBrainclawHooks(entries, canonical) {
1095
1095
  kept.push(canonical);
1096
1096
  return kept;
1097
1097
  }
1098
+ /**
1099
+ * Canonical Claude Code session-hook commands.
1100
+ *
1101
+ * `--hook` (pln#596) makes session-start / context-diff / session-end degrade to
1102
+ * exit 0 + ~/.brainclaw/hook.log on failure instead of hard-erroring the prompt
1103
+ * loop (trp#917). `2>/dev/null` is kept to suppress incidental stderr noise —
1104
+ * the actionable diagnostic now goes to hook.log, not stderr, so it survives the
1105
+ * redirect (the bug was the non-zero EXIT, not the stream).
1106
+ */
1107
+ function buildClaudeCodeHookCommands(bclawBin) {
1108
+ return {
1109
+ session: `f=.claude/.bclaw-session; if [ ! -f "$f" ]; then touch "$f"; ${bclawBin} session-start --include-context --hook 2>/dev/null; else ${bclawBin} context-diff --hook 2>/dev/null; fi`,
1110
+ stop: `rm -f .claude/.bclaw-session; ${bclawBin} session-end --auto-release --reflect --reflect-handoff --dispatch-review --hook 2>/dev/null`,
1111
+ checkEvents: `${bclawBin} check-events 2>/dev/null`,
1112
+ };
1113
+ }
1114
+ /**
1115
+ * Sanitize the brainclaw session hooks in ONE Claude Code settings file: collapse
1116
+ * every recognized brainclaw hook (across UserPromptSubmit / Stop / PostToolUse)
1117
+ * to a single canonical entry, repairing stale/broken forms (e.g. the legacy
1118
+ * `node session-start` with the cli.js arg dropped, or dead install paths).
1119
+ *
1120
+ * Only touches events that ALREADY contain a brainclaw hook — it never injects
1121
+ * hooks into a file (or event) that lacked them, so user-scope settings without
1122
+ * brainclaw hooks stay untouched. This is the cross-scope counterpart to
1123
+ * `ensureClaudeCodeSettings`, which only rewrites the cwd project file (pln#596 /
1124
+ * trp#918: setup's git-repo discovery never reaches the launch dir or user scope,
1125
+ * so broken hooks accumulate exactly where the agent executes them).
1126
+ */
1127
+ export function fixClaudeCodeHooksInFile(filePath) {
1128
+ if (!fs.existsSync(filePath))
1129
+ return { filePath, existed: false, changed: false, collapsed: 0 };
1130
+ const existing = readJsonObject(filePath); // returns {} for missing, undefined for unparseable
1131
+ if (existing === undefined)
1132
+ return { filePath, existed: true, changed: false, collapsed: 0 };
1133
+ const hooksObj = isJsonObject(existing.hooks) ? existing.hooks : undefined;
1134
+ if (!hooksObj)
1135
+ return { filePath, existed: true, changed: false, collapsed: 0 };
1136
+ const bclawBin = getBclawCliParts().map(quoteShellArg).join(' ');
1137
+ const cmds = buildClaudeCodeHookCommands(bclawBin);
1138
+ const countBrainclawHooks = (arr) => Array.isArray(arr)
1139
+ ? arr.reduce((n, e) => n + (isJsonObject(e) && Array.isArray(e.hooks)
1140
+ ? e.hooks.filter((h) => isJsonObject(h) && typeof h.command === 'string' && isBrainclawHookCommand(h.command)).length
1141
+ : 0), 0)
1142
+ : 0;
1143
+ const events = [
1144
+ ['UserPromptSubmit', buildCommandHookEntry(cmds.session)],
1145
+ ['Stop', buildCommandHookEntry(cmds.stop)],
1146
+ ['PostToolUse', buildMatchedCommandHookEntry('mcp__brainclaw__', cmds.checkEvents)],
1147
+ ];
1148
+ const nextHooks = { ...hooksObj };
1149
+ let collapsed = 0;
1150
+ let touched = false;
1151
+ for (const [event, canonical] of events) {
1152
+ const arr = Array.isArray(hooksObj[event]) ? hooksObj[event] : undefined;
1153
+ if (!arr)
1154
+ continue;
1155
+ const before = countBrainclawHooks(arr);
1156
+ if (before === 0)
1157
+ continue; // never add hooks to an event that had none
1158
+ nextHooks[event] = replaceBrainclawHooks(arr, canonical);
1159
+ collapsed += Math.max(0, before - 1); // N brainclaw entries → 1 canonical
1160
+ touched = true;
1161
+ }
1162
+ if (!touched)
1163
+ return { filePath, existed: true, changed: false, collapsed: 0 };
1164
+ const { updated } = writeJsonFileIfChanged(filePath, { ...existing, hooks: nextHooks });
1165
+ return { filePath, existed: true, changed: updated, collapsed };
1166
+ }
1167
+ /**
1168
+ * Run `fixClaudeCodeHooksInFile` across every Claude Code settings scope that can
1169
+ * carry brainclaw hooks: user-scope (`~/.claude/settings*.json`) and the cwd
1170
+ * project (`<cwd>/.claude/settings*.json`). Independent of git-repo discovery.
1171
+ */
1172
+ export function fixClaudeCodeHooksAllScopes(cwd, homeDir) {
1173
+ const candidates = [
1174
+ path.join(homeDir, '.claude', 'settings.json'),
1175
+ path.join(homeDir, '.claude', 'settings.local.json'),
1176
+ path.join(cwd, '.claude', 'settings.json'),
1177
+ path.join(cwd, '.claude', 'settings.local.json'),
1178
+ ];
1179
+ const seen = new Set();
1180
+ const results = [];
1181
+ for (const candidate of candidates) {
1182
+ const resolved = path.resolve(candidate);
1183
+ if (seen.has(resolved))
1184
+ continue;
1185
+ seen.add(resolved);
1186
+ results.push(fixClaudeCodeHooksInFile(resolved));
1187
+ }
1188
+ return results;
1189
+ }
1098
1190
  export function ensureProjectDevDependency(cwd) {
1099
1191
  const filePath = path.join(cwd, 'package.json');
1100
1192
  if (!fs.existsSync(filePath))
@@ -1242,10 +1334,8 @@ export function ensureClaudeCodeSettings(cwd) {
1242
1334
  // binary resolution succeeded (hidden by 2>/dev/null).
1243
1335
  const hooks = isJsonObject(existing.hooks) ? { ...existing.hooks } : {};
1244
1336
  const bclawBin = getBclawCliParts().map(quoteShellArg).join(' ');
1245
- const sessionCommand = `f=.claude/.bclaw-session; if [ ! -f "$f" ]; then touch "$f"; ${bclawBin} session-start --include-context 2>/dev/null; else ${bclawBin} context-diff 2>/dev/null; fi`;
1246
- const stopCommand = `rm -f .claude/.bclaw-session; ${bclawBin} session-end --auto-release --reflect --reflect-handoff --dispatch-review 2>/dev/null`;
1247
1337
  // PostToolUse — check for unseen events after any brainclaw MCP tool call
1248
- const checkEventsCommand = `${bclawBin} check-events 2>/dev/null`;
1338
+ const { session: sessionCommand, stop: stopCommand, checkEvents: checkEventsCommand } = buildClaudeCodeHookCommands(bclawBin);
1249
1339
  hooks.UserPromptSubmit = replaceBrainclawHooks(Array.isArray(hooks.UserPromptSubmit) ? hooks.UserPromptSubmit : [], buildCommandHookEntry(sessionCommand));
1250
1340
  hooks.Stop = replaceBrainclawHooks(Array.isArray(hooks.Stop) ? hooks.Stop : [], buildCommandHookEntry(stopCommand));
1251
1341
  hooks.PostToolUse = replaceBrainclawHooks(Array.isArray(hooks.PostToolUse) ? hooks.PostToolUse : [], buildMatchedCommandHookEntry('mcp__brainclaw__', checkEventsCommand));
@@ -314,6 +314,18 @@ export function resolveCurrentAgentIdentity(cwd, preferredDirName, _homeDir) {
314
314
  // In multi-agent setups this always resolves to the wrong agent.
315
315
  // The field remains in config for display (status, doctor) and for resolveExistingCurrentAgent
316
316
  // which is used during setup/init only.
317
+ // Single-registered-agent fallback (pln#596). When there is NO identity signal
318
+ // at all — no env id/name AND no detected native agent — but exactly one agent
319
+ // is registered in this scope, resolving it is unambiguous. This is the
320
+ // solo-dev / single-CLI case (a fresh hook with no BRAINCLAW_AGENT_NAME). The
321
+ // pln#562 guard against config.current_agent only matters at ≥2 agents, so this
322
+ // preserves multi-agent safety: ≥2 registered → still falls through to undefined.
323
+ if (!envAgentId && !envAgentName && !detected) {
324
+ const registered = listAgentIdentities(cwd, preferredDirName);
325
+ if (registered.length === 1) {
326
+ return registered[0];
327
+ }
328
+ }
317
329
  return undefined;
318
330
  }
319
331
  export function resolveRegisteredAgentIdentity(options = {}) {
@@ -416,7 +428,10 @@ export function requireRegisteredAgentIdentity(options = {}) {
416
428
  throw new AgentIdentityResolutionError(`Environment agent '${normalizedEnv}' is not registered.`, { agent_name: normalizedEnv });
417
429
  }
418
430
  }
419
- throw new AgentIdentityResolutionError('No registered agent identity resolved. Use --agent/--agent-id or configure a current agent with `brainclaw register-agent <name> --set-current`.');
431
+ throw new AgentIdentityResolutionError('No registered agent identity resolved. Pass `--agent <name>` or `--agent-id <id>`, '
432
+ + 'set $BRAINCLAW_AGENT_NAME, or register an agent with `brainclaw register-agent <name>` '
433
+ + '— a single registered agent is then resolved automatically. '
434
+ + '(`--set-current` alone does NOT affect resolution.)');
420
435
  }
421
436
  /**
422
437
  * Resolve agent identity for session start, returning both the resolved identity and whether
@@ -0,0 +1,43 @@
1
+ import fs from 'node:fs';
2
+ import os from 'node:os';
3
+ import path from 'node:path';
4
+ /**
5
+ * Append a one-line, timestamped diagnostic to ~/.brainclaw/hook.log.
6
+ *
7
+ * brainclaw session hooks (UserPromptSubmit / Stop) historically wrapped every
8
+ * CLI call in `2>/dev/null`, which turned an actionable failure (e.g. "no
9
+ * registered agent identity resolved") into a contentless "hook error: No
10
+ * stderr output" on every prompt (trp#917). Hook-mode commands now degrade to
11
+ * exit 0 and drop a line here instead, so the failure is silent to the agent's
12
+ * prompt loop but still debuggable.
13
+ *
14
+ * Best-effort and never throws — a logging failure must not break a hook.
15
+ */
16
+ const MAX_LOG_BYTES = 256 * 1024;
17
+ export function hookLogPath(homeDir = os.homedir()) {
18
+ return path.join(homeDir, '.brainclaw', 'hook.log');
19
+ }
20
+ export function logHookDiagnostic(message, homeDir = os.homedir()) {
21
+ try {
22
+ const file = hookLogPath(homeDir);
23
+ fs.mkdirSync(path.dirname(file), { recursive: true });
24
+ // Size-cap: when the log grows past the cap, keep only the tail so it never
25
+ // balloons unbounded across thousands of prompts.
26
+ try {
27
+ const stat = fs.statSync(file);
28
+ if (stat.size > MAX_LOG_BYTES) {
29
+ const tail = fs.readFileSync(file, 'utf-8').slice(-Math.floor(MAX_LOG_BYTES / 2));
30
+ fs.writeFileSync(file, tail, 'utf-8');
31
+ }
32
+ }
33
+ catch {
34
+ /* no existing file — nothing to truncate */
35
+ }
36
+ const line = `${new Date().toISOString()} ${message.replace(/\s+/g, ' ').trim()}\n`;
37
+ fs.appendFileSync(file, line, 'utf-8');
38
+ }
39
+ catch {
40
+ /* best-effort — never break a hook over logging */
41
+ }
42
+ }
43
+ //# sourceMappingURL=hook-log.js.map
package/dist/facts.js CHANGED
@@ -1,8 +1,8 @@
1
1
  // Generated by scripts/emit-site-facts.mjs at build time. Do not edit manually.
2
- // Source: brainclaw v1.11.0 on 2026-06-24T07:26:01.488Z
2
+ // Source: brainclaw v1.11.1 on 2026-06-24T13:31:04.040Z
3
3
  export const FACTS = {
4
- "version": "1.11.0",
5
- "generated_at": "2026-06-24T07:26:01.488Z",
4
+ "version": "1.11.1",
5
+ "generated_at": "2026-06-24T13:31:04.040Z",
6
6
  "tools": {
7
7
  "count": 67,
8
8
  "published_count": 66,
package/dist/facts.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
- "version": "1.11.0",
3
- "generated_at": "2026-06-24T07:26:01.488Z",
2
+ "version": "1.11.1",
3
+ "generated_at": "2026-06-24T13:31:04.040Z",
4
4
  "tools": {
5
5
  "count": 67,
6
6
  "published_count": 66,
package/docs/cli.md CHANGED
@@ -264,14 +264,18 @@ Run health checks on config, state, and generated views.
264
264
  | `--json` | Output as JSON |
265
265
  | `--migration-check` | Check for pending migrations |
266
266
  | `--fix-agent-ignore` | Add missing `.gitignore` entries for generated local Brainclaw agent files |
267
+ | `--fix-hooks` | Purge stale/broken/duplicate brainclaw session hooks across all Claude Code settings scopes (user + cwd) and rewrite the canonical ones |
267
268
 
268
269
  ```bash
269
270
  brainclaw doctor
270
271
  brainclaw doctor --json
271
272
  brainclaw doctor --migration-check
272
273
  brainclaw doctor --fix-agent-ignore
274
+ brainclaw doctor --fix-hooks
273
275
  ```
274
276
 
277
+ `--fix-hooks` collapses every recognized brainclaw session hook (across `UserPromptSubmit` / `Stop` / `PostToolUse`) in `~/.claude/settings.json`, `~/.claude/settings.local.json`, `<cwd>/.claude/settings.json`, and `<cwd>/.claude/settings.local.json` down to a single canonical entry, repairing legacy/broken forms. It only touches files (and events) that already contain a brainclaw hook, so user-authored hooks and hook-less scopes are left untouched. Use it when you see repeated `UserPromptSubmit hook error` warnings.
278
+
275
279
  When Brainclaw detects generated local agent files such as `.mcp.json` or `.claude/settings.local.json` inside a Git repo, `doctor` warns if they are not ignored or are still tracked. `--fix-agent-ignore` only updates `.gitignore`; if a file is already tracked you still need to untrack it with `git rm --cached <path>`.
276
280
 
277
281
  In `multi-project` mode with `projects.strategy: folder`, `doctor` now checks the effective workspace project set, not just `config.projects.known`. That avoids false positives on workspaces that resolve child stores from the filesystem or global project registry.
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "brainclaw",
3
- "version": "1.11.0",
3
+ "version": "1.11.1",
4
4
  "description": "Shared project memory for humans and coding agents.",
5
5
  "type": "module",
6
6
  "repository": {