@descent-vtt/spec-harness 0.17.0 → 0.19.0

package/dist/configure.js

@@ -11,8 +11,9 @@ import { isAbsolute, join } from 'node:path';
 import { runInNewContext } from 'node:vm';
 import { CALLABLE_TOOLS } from './hooks.js';
 import { literal } from './literal.js';
+import { namedTree } from './usage.js';
 import { parseGlobList } from './vendor/spec-core/pattern/index.js';
-import { CLAUDE_CODE_MINIMUM } from './versions.js';
+import { CLAUDE_CODE_MINIMUM, meets, parseVersion } from './versions.js';
 function isObject(value) {
     return typeof value === 'object' && value !== null;
 }
@@ -51,6 +52,12 @@ const BEFORE = 'PreToolUse';
 /** The event after a call that succeeded, where a hook can only tell the model what it wrote. */
 const AFTER = 'PostToolUse';
 const GUARDED_EVENTS = [BEFORE, AFTER];
+/**
+ * The event Claude Code runs as a session starts, under which the hook says
+ * what would hold every write of it (ADR-0005). Its entry has no matcher:
+ * left out, it is every way a session starts.
+ */
+const AT_START = 'SessionStart';
 const GUARD_ARGS = Object.freeze([`${PROJECT_DIR}/${BIN}`, 'hook', 'claude']);
 /**
  * The guard as a Claude Code hook: exec form, `args` given, so the project's
@@ -86,19 +93,109 @@ function upgraded(group) {
  * `_`, `-`, spaces, `,` and `|` alone, the tools it names, one between each
  * `|` or `,`; with any other character in it, a JavaScript regular
  * expression that may match anywhere in a tool's name. `null` where the
- * reference does not say, or it cannot be told: a matcher that is not text,
- * is no pattern, or is a pattern whose reading runs out its budget.
+ * reference does not say, or it cannot be told: a matcher that is neither
+ * text nor an array, is no pattern, or is a pattern whose reading runs out
+ * its budget.
+ *
+ * Two things its page on debugging a configuration adds
+ * (https://code.claude.com/docs/en/debug-your-config, Check hooks and Check
+ * common causes, read 2026-10-14). A matcher that is an array is an entry
+ * Claude Code lists as an invalid setting, among the reasons a hook is not
+ * loaded and never fires: no tool. And a comma separates names only from
+ * {@link COMMA_SINCE} on; its changelog adds that a hyphenated name is
+ * matched exactly only from {@link HYPHEN_SINCE} on. So a matcher of names
+ * with either in it is read as `matching` has the Claude Code here read it
+ * ({@link namesReadings}), and where the readings it may have differ, it
+ * cannot be told.
  */
-function matchedTools(matcher, pattern) {
+function matchedTools(matcher, pattern, matching) {
     if (matcher === undefined || matcher === '' || matcher === '*')
         return CALLABLE_TOOLS;
+    if (Array.isArray(matcher))
+        return [];
     if (typeof matcher !== 'string')
         return null;
-    if (/^[A-Za-z0-9_\- ,|]*$/.test(matcher)) {
-        const names = matcher.split(/[|,]/).map((name) => name.trim());
-        return CALLABLE_TOOLS.filter((tool) => names.includes(tool));
-    }
-    return pattern(matcher);
+    if (!NAMES.test(matcher))
+        return pattern(matcher);
+    // A reading that cannot be told, or two that differ, leave the matcher one
+    // whose tools cannot be told; where every reading comes to the same tools,
+    // nothing turns on which the release has.
+    const readings = namesReadings(matcher, pattern, matching);
+    return readings.every((tools) => sameStrings(tools, readings[0])) ? readings[0] : null;
+}
+/** A matcher that is a name, or names between `|` and `,`: what the reference compares with a tool's name exactly. */
+const NAMES = /^[A-Za-z0-9_\- ,|]*$/;
+/** The writing tools a matcher of names lists, one between each `|` or `,`. */
+function listedTools(matcher) {
+    const names = matcher.split(/[|,]/).map((name) => name.trim());
+    return CALLABLE_TOOLS.filter((tool) => names.includes(tool));
+}
+/**
+ * The release of Claude Code from which a comma in a matcher separates
+ * names as a `|` does. Its page on debugging a configuration (Check hooks,
+ * read 2026-10-14) has a comma before it fall through to the reading of a
+ * pattern, under which the matcher never matched, and its changelog has the
+ * fix under this release: hooks with comma-separated matchers no longer
+ * silently never firing.
+ */
+export const COMMA_SINCE = '2.1.191';
+/**
+ * The release of Claude Code from which a hyphenated name in a matcher of
+ * names is matched exactly. Its changelog
+ * (https://code.claude.com/docs/en/changelog, 2.1.195, read 2026-10-18) has,
+ * among its fixes, hook matchers with hyphenated identifiers that matched as
+ * substrings, and match exactly from this release on; it does not say how
+ * the rest of such a matcher was read.
+ */
+export const HYPHEN_SINCE = '2.1.195';
+/** The reading every release from {@link HYPHEN_SINCE} on has, and what a caller that asks no release is given. */
+const TODAY = { as: 'today' };
+/**
+ * How the Claude Code doctor asked its release reads a matcher of names. A
+ * release that was not asked, or could not be told, leaves the reading
+ * untold: asked or not, nothing says which it is.
+ */
+export function matchingUnder(release) {
+    if (release === null || release.state === 'unknown')
+        return { as: 'untold' };
+    // A release that was told is one whose version was read as one, and the
+    // constant is a version, so both parse.
+    return meets(parseVersion(release.version), parseVersion(HYPHEN_SINCE)) ? TODAY : { as: 'older', release: release.version };
+}
+/** Whether `release` reads a comma in `matcher` as part of a pattern: one before {@link COMMA_SINCE}, where the matcher holds one. */
+function commaAsPattern(matcher, release) {
+    return matcher.includes(',') && !meets(parseVersion(release), parseVersion(COMMA_SINCE));
+}
+/** A matcher of names that holds a comma, as a release before {@link COMMA_SINCE} reads it. */
+function commaEarlier(matcher, pattern) {
+    // No tool, where the matcher read as a pattern matches none of the writing
+    // tools, which is what the page has such a matcher do; and where it matches
+    // one, as `Edit|Write,Bash` matches `Edit`, the page's two sentences, that
+    // a comma is read as part of a pattern and that the matcher never matched,
+    // disagree, and it cannot be told.
+    const read = pattern(matcher);
+    return read !== null && read.length === 0 ? read : null;
+}
+/**
+ * Each reading the Claude Code here may have of a matcher of names: the
+ * writing tools it lists, or `null` where a reading cannot be told. From
+ * {@link HYPHEN_SINCE} on there is one, the names matched exactly. Before
+ * {@link COMMA_SINCE} a comma in it makes it a pattern ({@link commaEarlier}).
+ * Before {@link HYPHEN_SINCE} a hyphen in it made it match as a substring,
+ * and the changelog does not say which way: each name matched exactly but
+ * the one with the hyphen, the matcher read as a pattern, which matches
+ * anywhere in a tool's name, or its whole text found in a tool's name, which
+ * no writing tool's holds, since none has a hyphen. A release that cannot be
+ * told may read it any way a release does.
+ */
+function namesReadings(matcher, pattern, matching) {
+    const listed = listedTools(matcher);
+    if (matching.as === 'today')
+        return [listed];
+    const substring = matcher.includes('-') ? [pattern(matcher), []] : [];
+    if (matching.as === 'older')
+        return commaAsPattern(matcher, matching.release) ? [commaEarlier(matcher, pattern)] : [listed, ...substring];
+    return [listed, ...(matcher.includes(',') ? [commaEarlier(matcher, pattern)] : []), ...substring];
 }
 /**
  * How long the reading of one pattern may take, in milliseconds of the
@@ -169,12 +266,12 @@ function inBackground(hook) {
  * entry started only for the calls its rule matches, in the syntax of a
  * permission rule, which is not read here.
  */
-function answered(event, matcher, hook, pattern) {
+function answered(event, matcher, hook, pattern, matching) {
     if (event === BEFORE && inBackground(hook))
         return [];
     if (hook['if'] !== undefined)
         return null;
-    return matchedTools(matcher, pattern);
+    return matchedTools(matcher, pattern, matching);
 }
 /**
  * `.claude/settings.json`, or the plugin's hooks, with the guard hooks merged
@@ -191,10 +288,14 @@ function answered(event, matcher, hook, pattern) {
  *
  * `pattern` reads a matcher that is a pattern: the run's own reader, where
  * the run has read the file's matchers already ({@link patternReader}).
+ * `matching` is how the Claude Code here reads a matcher of names, where
+ * the run asked its release ({@link matchingUnder}): a line that
+ * release starts for no write is no guard, and init's own is written beside
+ * it.
  */
-export function mergeClaudeSettings(current, isHook = () => false, pattern = patternReader()) {
+export function mergeClaudeSettings(current, isHook = () => false, pattern = patternReader(), matching = TODAY) {
     const hooks = isObject(current['hooks']) ? current['hooks'] : {};
-    const theirs = harnessEntries(current).some((entry) => !isGuard(entry.hook) && !isLegacyGuard(entry.hook) && isHook(entry.words) && isStarted(toolsOf(entry, pattern)));
+    const theirs = harnessEntries(current).some((entry) => !isGuard(entry.hook) && !isLegacyGuard(entry.hook) && isHook(entry.words) && isStarted(toolsOf(entry, pattern, matching)));
     if (theirs)
         return null;
     let changed = false;
@@ -204,7 +305,7 @@ export function mergeClaudeSettings(current, isHook = () => false, pattern = pat
         const before = Array.isArray(listed) ? listed : [];
         const groups = before.map(upgraded);
         const replaced = groups.some((group, index) => group !== before[index]);
-        const present = harnessEntries({ hooks: { [event]: groups } }).some((entry) => isGuard(entry.hook) && isStarted(toolsOf(entry, pattern)));
+        const present = harnessEntries({ hooks: { [event]: groups } }).some((entry) => isGuard(entry.hook) && isStarted(toolsOf(entry, pattern, matching)));
         if (!present)
             groups.push({ matcher: HOOK_MATCHER, hooks: [{ ...GUARD_HOOK, args: [...GUARD_ARGS] }] });
         if (replaced || !present) {
@@ -214,6 +315,45 @@ export function mergeClaudeSettings(current, isHook = () => false, pattern = pat
     }
     return changed ? { ...current, hooks: next } : null;
 }
+/**
+ * Settings with init's entry under `SessionStart` added beside its two, or
+ * `null` where it is not to be: where init's own line does not stand
+ * under both `PreToolUse` and `PostToolUse`, a file init writes no hook
+ * into or one it has yet to complete; and where a line of the hook stands
+ * under `SessionStart` already, init's or one `isHook` knows, which would
+ * say the same twice (ADR-0012).
+ */
+export function withSessionStart(settings, isHook) {
+    const hooks = isObject(settings['hooks']) ? settings['hooks'] : {};
+    const own = GUARDED_EVENTS.every((event) => harnessEntries({ hooks: { [event]: hooks[event] } }).some((entry) => isGuard(entry.hook)));
+    if (!own || startsSessions(settings, isHook))
+        return null;
+    const listed = hooks[AT_START];
+    return { ...settings, hooks: { ...hooks, [AT_START]: [...(Array.isArray(listed) ? listed : []), { hooks: [{ ...GUARD_HOOK, args: [...GUARD_ARGS] }] }] } };
+}
+/**
+ * Whether settings have Claude Code start the hook as a session starts: an
+ * entry under `SessionStart` whose line is init's, or one `isHook` knows
+ * as the hook's. Its matcher is not read: one that names some of the ways a
+ * session starts has the check made for those.
+ */
+export function startsSessions(settings, isHook) {
+    const groups = field(field(settings, 'hooks'), AT_START);
+    if (!Array.isArray(groups))
+        return false;
+    return groups.some((group) => hooksOf(group).some((hook) => isObject(hook) && (isGuard(hook) || knownAs(hook, isHook))));
+}
+/** Whether `isHook` knows an entry's line as the hook's. */
+function knownAs(hook, isHook) {
+    const written = writtenHook(hook);
+    return written !== null && isHook(written.words);
+}
+/**
+ * What doctor notes where settings hold the guard hooks and none under
+ * `SessionStart`: what would hold every write is said at the first write,
+ * which it holds, and not as the session starts.
+ */
+export const NO_SESSION_START = 'note: no settings file has Claude Code start the hook as a session starts, so a configuration that does not load, or a brief named that spec-brief does not list, is said only at the first write, which it holds: spec-harness init --write adds the entry beside the two it wrote';
 /** The server's entry, reading the project at `projectDir`: {@link PROJECT_DIR} for the plugin, {@link PROJECT_DIR_OR_HERE} for a project. */
 export function mcpServer(projectDir) {
     return { command: 'node', args: [`${projectDir}/${BIN}`, 'mcp', '--root', projectDir] };
@@ -309,8 +449,8 @@ function sameLine(one, other) {
  * `null` where that cannot be told. Read where it is asked for and no
  * sooner: a matcher that is a pattern can cost its whole budget.
  */
-function toolsOf(entry, pattern) {
-    return answered(entry.event, entry.matcher, entry.hook, pattern);
+function toolsOf(entry, pattern, matching) {
+    return answered(entry.event, entry.matcher, entry.hook, pattern, matching);
 }
 /** Every entry that runs the harness under the two events the guard answers, in the order the settings hold them. */
 function harnessEntries(settings) {
@@ -355,18 +495,47 @@ function named(names) {
     return names.length === 1 ? last : `${names.slice(0, -1).join(', ')} and ${last}`;
 }
 /** The writing tools a line's entries under `event` are waited on for between them, or `null` where an entry that cannot be told might add one. */
-function toolsUnder(event, entries, pattern) {
-    const read = entries.filter((entry) => entry.event === event).map((entry) => toolsOf(entry, pattern));
+function toolsUnder(event, entries, reading) {
+    const read = entries.filter((entry) => entry.event === event).map((entry) => toolsOf(entry, reading.pattern, reading.matching));
     const sure = CALLABLE_TOOLS.filter((tool) => read.some((tools) => tools !== null && tools.includes(tool)));
     return sure.length < CALLABLE_TOOLS.length && read.some((tools) => tools === null) ? null : sure;
 }
+/**
+ * What is said after a matcher that Claude Code does not read as its text
+ * suggests: an array, which it loads no hook under, and names with a comma or
+ * a hyphen where the release here reads them another way than the newest
+ * does, or cannot be told. Nothing for any other matcher, and nothing where
+ * every reading the release may have is the names as listed.
+ */
+function saidOfMatcher(matcher, { pattern, matching }) {
+    if (Array.isArray(matcher))
+        return ', an array, which Claude Code lists as an invalid setting and loads no hook under';
+    if (typeof matcher !== 'string' || !NAMES.test(matcher))
+        return '';
+    const tools = listedTools(matcher);
+    if (namesReadings(matcher, pattern, matching).every((read) => sameStrings(read, tools)))
+        return '';
+    const listed = tools.length === 0 ? 'no tool that writes' : named(tools);
+    const earlier = commaEarlier(matcher, pattern) === null ? 'a pattern whose matches its documentation does not give' : 'a pattern that matches no tool';
+    const substring = "a matcher with a hyphenated name in it matched as a substring, and Claude Code's changelog does not say how the rest of it was read";
+    if (matching.as === 'older') {
+        if (commaAsPattern(matcher, matching.release))
+            return `, which Claude Code ${matching.release} reads as ${earlier}: a comma separates names from ${COMMA_SINCE} on, where the matcher is ${listed}`;
+        return `, which Claude Code ${matching.release} may read otherwise: before ${HYPHEN_SINCE} ${substring}; from ${HYPHEN_SINCE} on it is ${listed}`;
+    }
+    // The release cannot be told: a comma alone turns on 2.1.191, and a hyphen on 2.1.195.
+    if (!matcher.includes('-'))
+        return `, which is ${listed} from Claude Code ${COMMA_SINCE} on, where a comma separates names, and ${earlier} before it: the release here cannot be told`;
+    const comma = matcher.includes(',') ? `, and before ${COMMA_SINCE} ${earlier}, where a comma separated no names` : '';
+    return `, which is ${listed} from Claude Code ${HYPHEN_SINCE} on, and may be read otherwise before it, where ${substring}${comma}: the release here cannot be told`;
+}
 /** What a line's entries under `event` hold that decides when Claude Code starts them, as written. */
-function entriesUnder(event, entries) {
+function entriesUnder(event, entries, reading) {
     const under = entries.filter((entry) => entry.event === event);
     if (under.length === 0)
         return `it stands under no ${event}`;
     const said = under.map(({ matcher, hook }) => {
-        const pattern = matcher === undefined ? 'no matcher' : `the matcher ${JSON.stringify(matcher)}`;
+        const pattern = matcher === undefined ? 'no matcher' : `the matcher ${JSON.stringify(matcher)}${saidOfMatcher(matcher, reading)}`;
         const background = inBackground(hook) ? ' in the background' : '';
         const rule = hook['if'] === undefined ? '' : ` with "if" ${JSON.stringify(hook['if'])}`;
         return `${pattern}${background}${rule}`;
@@ -381,24 +550,58 @@ function entriesUnder(event, entries) {
  * hook still reads a path from, is named as unchecked for no line, since no
  * such Claude Code has it to call (hooks.ts).
  */
-function describeCover(where, before, after, entries) {
+function describeCover(where, before, after, entries, reading) {
     const parts = [];
     const left = (tools) => CALLABLE_TOOLS.filter((tool) => !tools.includes(tool));
     if (before === null)
-        parts.push(`which writes Claude Code waits on it before cannot be told here (${entriesUnder(BEFORE, entries)})`);
+        parts.push(`which writes Claude Code waits on it before cannot be told here (${entriesUnder(BEFORE, entries, reading)})`);
     else if (before.length === 0)
-        parts.push(`Claude Code waits on it before no write, so it is not the guard and refuses none (${entriesUnder(BEFORE, entries)})`);
+        parts.push(`Claude Code waits on it before no write, so it is not the guard and refuses none (${entriesUnder(BEFORE, entries, reading)})`);
     else if (left(before).length > 0) {
-        parts.push(`Claude Code waits on it before a write by ${named(before)}, and before none by ${named(left(before))}, which goes unchecked (${entriesUnder(BEFORE, entries)})`);
+        parts.push(`Claude Code waits on it before a write by ${named(before)}, and before none by ${named(left(before))}, which goes unchecked (${entriesUnder(BEFORE, entries, reading)})`);
     }
     if (after === null)
-        parts.push(`which writes it is started after cannot be told here (${entriesUnder(AFTER, entries)})`);
+        parts.push(`which writes it is started after cannot be told here (${entriesUnder(AFTER, entries, reading)})`);
     else if (after.length === 0)
-        parts.push(`it is started after no write, so it warns of none (${entriesUnder(AFTER, entries)})`);
+        parts.push(`it is started after no write, so it warns of none (${entriesUnder(AFTER, entries, reading)})`);
     else if (left(after).length > 0)
-        parts.push(`it is started after a write by ${named(after)} alone (${entriesUnder(AFTER, entries)})`);
+        parts.push(`it is started after a write by ${named(after)} alone (${entriesUnder(AFTER, entries, reading)})`);
     return parts.length === 0 ? null : `${where}: ${parts.join('; ')}`;
 }
+/** The events under which an array for a matcher keeps Claude Code from loading the rest of a file's hooks. */
+const UNLOADING = [BEFORE, 'PermissionRequest'];
+/**
+ * The first array that stands for a matcher under `PreToolUse` or
+ * `PermissionRequest` in a settings file, or `null`.
+ *
+ * Claude Code's page on debugging a configuration
+ * (https://code.claude.com/docs/en/debug-your-config, Check hooks, read
+ * 2026-10-14) has a matcher that is an array, where it must be one string,
+ * as an entry Claude Code lists as an invalid setting, at the start of an
+ * interactive session and in `claude doctor`; and where the array is under
+ * one of these two events, none of that file's other hooks load either. It
+ * names no release. So a file that holds one runs no hook at all, whoever
+ * wrote the entry and whatever it runs: the array's own is not loaded, and
+ * neither is any other.
+ */
+export function arrayMatcher(settings) {
+    const hooks = field(settings, 'hooks');
+    for (const event of UNLOADING) {
+        const groups = field(hooks, event);
+        if (!Array.isArray(groups))
+            continue;
+        const group = groups.find((candidate) => Array.isArray(field(candidate, 'matcher')));
+        if (group !== undefined)
+            return { event, matcher: JSON.stringify(field(group, 'matcher')) };
+    }
+    return null;
+}
+/** What is said of a settings file that an array for a matcher keeps Claude Code from loading the hooks of. */
+export function describeArrayMatcher(file, { event, matcher }) {
+    return `${file} has an array for a matcher under ${event}, ${matcher}, which Claude Code lists as an invalid setting, and beside one under PreToolUse or PermissionRequest it loads none of the file's other hooks`;
+}
+/** How a matcher that is an array is mended. */
+export const ONE_STRING = 'write the matcher as one string, its names between "|"';
 /** The words Claude Code hands the hook for a line: what follows the harness's command, each `${CLAUDE_PROJECT_DIR}` filled with `project`. */
 function handedWords(words, project) {
     return words.map((word) => word.split(PROJECT_DIR).join(project));
@@ -417,18 +620,29 @@ export function handedLines(sources, project) {
  * A line that guards and holds no more than a release wrote is reported
  * with nothing to say, as it always was. One with more on it is said as it
  * is, and fails nothing. One the hook cannot read, or whose root is no
- * directory, is a guard that is off, and one that names a brief which is not
+ * directory, or is in no work tree that holds the project, is a guard that
+ * is off, and one that names a brief which is not
  * one holds every write: each a problem, and doctor exits 1 for it. A line
  * Claude Code waits on before no write is not the guard, whatever the hook
  * would do with it.
  *
  * `pattern` reads a matcher that is a pattern, once for every entry under
  * it in every file: the run's own reader, where something else in the run
- * reads the same files ({@link patternReader}).
+ * reads the same files ({@link patternReader}). `matching` is how the
+ * Claude Code here reads a matcher of names, where its release was asked
+ * ({@link matchingUnder}); a caller that asks none reads one as every
+ * release from {@link HYPHEN_SINCE} on does.
+ *
+ * A file that holds an array for a matcher under `PreToolUse` or
+ * `PermissionRequest` has none of its hooks loaded ({@link arrayMatcher}):
+ * each line of the hook there is started for no write, and one that stood
+ * to be started for a write is a guard that is off.
  */
-export function reportHooks(sources, project, read, pattern = patternReader()) {
+export function reportHooks(sources, project, read, pattern = patternReader(), matching = TODAY) {
+    const reading = { pattern, matching };
     return sources.flatMap(({ file, settings }) => {
         const entries = harnessEntries(settings);
+        const array = arrayMatcher(settings);
         return writtenHooks(settings).flatMap((hook) => {
             const handed = read(handedWords(hook.words, project));
             if (handed === null)
@@ -438,9 +652,23 @@ export function reportHooks(sources, project, read, pattern = patternReader()) {
             // A line that is not under both events is under one.
             const where = `"${hook.words.map((word) => (word === '' ? "''" : word)).join(' ')}" in ${file}${both ? '' : ` (under ${hook.events[0]} alone)`}`;
             const mine = entries.filter((entry) => sameLine(entry, hook));
-            const before = toolsUnder(BEFORE, mine, pattern);
-            const after = toolsUnder(AFTER, mine, pattern);
-            const written = { file, events: hook.events, line: hook.words, onFailure: hook.blocks ? 'block' : null, before, after, cover: describeCover(where, before, after, mine) };
+            const before = toolsUnder(BEFORE, mine, reading);
+            const after = toolsUnder(AFTER, mine, reading);
+            if (array !== null) {
+                // Nothing of the file is loaded, whatever the line holds and
+                // whatever the hook would do with it. A line that stood to be
+                // started for a write, or stands under an array itself, where
+                // someone meant a matcher, is a guard set up that checks nothing;
+                // one that stood to be started for none leaves no write unchecked
+                // that it was ever asked about.
+                const unloaded = { file, events: hook.events, line: hook.words, onFailure: hook.blocks ? 'block' : null, before: [], after: [], cover: null, note: null };
+                const said = `${where}: ${describeArrayMatcher(file, array)}`;
+                if (isStarted(before) || isStarted(after) || mine.some((entry) => Array.isArray(entry.matcher))) {
+                    return [{ ...unloaded, state: 'off', detail: `${said}, so the guard is off and no write is checked; ${ONE_STRING}` }];
+                }
+                return [{ ...unloaded, state: 'unstarted', detail: `${said}: the line is not started, and stood to be started for no write as it is, so no guard is switched off` }];
+            }
+            const written = { file, events: hook.events, line: hook.words, onFailure: hook.blocks ? 'block' : null, before, after, cover: describeCover(where, before, after, mine, reading) };
             const note = describeOnFailure(where, hook.blocks, false);
             if (hook.untyped !== null) {
                 const detail = `${where} stands on an entry with ${hook.untyped}, where Claude Code's hooks reference has "type": "command" on an entry it runs as a command, so whether Claude Code runs it cannot be told here: give the entry "type": "command"`;
@@ -454,15 +682,15 @@ export function reportHooks(sources, project, read, pattern = patternReader()) {
             const unstarted = { ...written, state: 'unstarted', detail: null, note };
             const waited = before === null || before.length > 0;
             if (handed.state === 'off') {
-                const detail = `${where}: ${handed.fault.wrong}, so the guard is off and no write is checked; ${handed.fault.mend}`;
+                const detail = `${where}: ${handed.fault.wrong}, so the guard is off and ${handed.unchecked ?? 'no write is checked'}; ${handed.fault.mend}`;
                 // A fault the hook says, it says with exit 1 after every write it is
                 // started for as well, so it is said wherever the line stands.
                 if (handed.silent !== true)
                     return [{ ...written, state: 'off', detail, note: describeOnFailure(where, hook.blocks, true) }];
-                // A root outside any work tree, of which the hook says nothing: no
-                // run of it fails there, so onFailure holds no write for it, and
-                // where the line is waited on before no write there is none it
-                // leaves unchecked.
+                // A root outside any work tree, or in one that does not hold the
+                // project, of which the hook says nothing: no run of it fails
+                // there, so onFailure holds no write for it, and where the line is
+                // waited on before no write there is none it leaves unchecked.
                 return [waited ? { ...written, state: 'off', detail, note } : unstarted];
             }
             if (!waited)
@@ -477,7 +705,9 @@ export function reportHooks(sources, project, read, pattern = patternReader()) {
             if (sameStrings(hook.words, ['hook', 'claude']))
                 return [{ ...written, state: 'guards', detail: null, note }];
             const root = handed.root === undefined ? '' : `, and its root, ${handed.root}, is a directory`;
-            return [{ ...written, state: 'guards', detail: `${where} is more than init writes, and guards: the hook reads the line${root}`, note }];
+            // Whose briefs a write is checked by, where they are not the project's own.
+            const holder = handed.holder === undefined ? '' : ` in ${namedTree(handed.holder)}, which holds this project: the hook checks a write here by that work tree's briefs and its branch, as a file under ${handed.holder.path}/ there`;
+            return [{ ...written, state: 'guards', detail: `${where} is more than init writes, and guards: the hook reads the line${root}${holder}`, note }];
         });
     });
 }
@@ -672,6 +902,18 @@ export function describeSkipped(plugin, part, present, elsewhere = []) {
     }
     return `${isOn(plugin)} and brings ${words.brings}, so ${words.skipped}: with both, ${words.twice}. To have ${words.them} here instead, for every clone of the repository, ${turnOff(plugin)} and run init again`;
 }
+/**
+ * How init comes to write the guard hooks. Where an array unloads the
+ * project's settings file, init writes none there and says the array, so
+ * the advice is the array's, and init after it: advice that led to more
+ * advice is the advice at the end.
+ */
+function runInit(wiring) {
+    const unloaded = wiring.unloaded ?? null;
+    if (unloaded === null)
+        return 'run spec-harness init --write';
+    return `${describeArrayMatcher(PROJECT_SETTINGS, unloaded)}, those init writes among them: ${ONE_STRING}, then run spec-harness init --write`;
+}
 /** Why the plugin settings turn on is not on this machine, or `null` where it is, may be, or none is on. */
 function absence(wiring) {
     return wiring.presence?.state === 'absent' ? wiring.presence.detail : null;
@@ -722,11 +964,11 @@ export function describePresence(wiring) {
         return `whether ${plugin.id} is on this machine cannot be told: ${presence.detail}. A plugin that is not there brings no hook: Claude Code's /plugin lists it, and its errors`;
     if (wiring.hooks.length > 0)
         return `${isAbsent(plugin, presence.detail, 'no hook')}. With the plugin installed as well, ${PARTS.hooks.twice}: ${turnOff(plugin)}`;
-    return wiring.server ? unguarded(plugin, presence.detail) : null;
+    return wiring.server ? unguarded(plugin, presence.detail, wiring) : null;
 }
 /** That the plugin settings turn on is not on this machine and no hook of init's stands in for it, with the two ways to a guard. */
-function unguarded(plugin, detail) {
-    return `${isAbsent(plugin, detail, 'no hook')}, and Claude Code guards no write. To have the guard, ${bring(plugin)}, or ${turnOff(plugin)} and run spec-harness init --write`;
+function unguarded(plugin, detail, wiring) {
+    return `${isAbsent(plugin, detail, 'no hook')}, and Claude Code guards no write. To have the guard, ${bring(plugin)}, or ${turnOff(plugin)} and ${runInit(wiring)}`;
 }
 /** What `doctor` says of the wiring, and what to do when it is doubled or missing. */
 export function describeClaudeCode(wiring) {
@@ -735,12 +977,12 @@ export function describeClaudeCode(wiring) {
     const own = `init's entries: ${entries}`;
     if (plugin === null) {
         if (entries === '')
-            return 'neither the spec-harness plugin nor the hooks init writes, so Claude Code guards no write: run spec-harness init --write, or install the plugin';
+            return `neither the spec-harness plugin nor the hooks init writes, so Claude Code guards no write: ${runInit(wiring)}, or install the plugin`;
         return own;
     }
     const absent = absence(wiring);
     if (absent !== null)
-        return entries === '' ? unguarded(plugin, absent) : own;
+        return entries === '' ? unguarded(plugin, absent, wiring) : own;
     if (entries === '')
         return `${isOn(plugin)} and brings the guard hooks and the server`;
     const twice = [...(wiring.hooks.length > 0 ? [PARTS.hooks.twice] : []), ...(wiring.server ? [PARTS.server.twice] : [])].join(' and ');
@@ -748,8 +990,7 @@ export function describeClaudeCode(wiring) {
 }
 /** The spec-brief plugin this package ships, as spec-brief's `plugins` names it. */
 export const PLUGIN = '@descent-vtt/spec-harness/spec-brief-plugin';
-/** spec-brief's configuration files, in the order spec-brief looks for them in a directory. */
-export const SPEC_BRIEF_CONFIGS = ['.spec-brief.json', 'spec-brief.json'];
+export { SPEC_BRIEF_CONFIGS } from './config.js';
 /** The conditions an import meets in Node, its `defaultConditions`, and `default`, which every one meets. */
 const IMPORT_CONDITIONS = ['node', 'import', 'default'];
 /** A target Node refuses as invalid, which an array passes over and anything else stops at. */
@@ -859,9 +1100,13 @@ export function chooseBase(facts) {
     const unrecorded = 'no remote records a default branch (refs/remotes/origin/HEAD)';
     if (branch !== null) {
         const conventional = branch === 'main' || branch === 'master';
-        if (conventional || facts.branches.every((name) => name === branch)) {
+        if (conventional || facts.branches.every((local) => local.name === branch)) {
             const which = conventional ? 'the branch init runs on' : 'the only branch';
-            return { base: branch, detail: `rounds are measured from ${branch}, ${which}: ${unrecorded}; change "base" if rounds merge into another` };
+            // The base is a ref git and spec-brief read, so it is the name git
+            // reads as the branch and nothing else: `heads/main` beside a tag
+            // `main`. A branch with no commit yet is listed by no ref.
+            const base = facts.branches.find((local) => local.name === branch)?.ref ?? branch;
+            return { base, detail: `rounds are measured from ${base}, ${which}: ${unrecorded}; change "base" if rounds merge into another` };
         }
     }
     const where = branch === null ? 'HEAD is detached' : `${branch} is not main, master or the only branch`;
@@ -943,20 +1188,30 @@ export const PROTECT_RULE_FILES = "Have the forge protect this file, the ADRs, t
 export const GIT_HOOK_LINE = `node ${BIN} hook git`;
 /**
  * The hook init writes. It is a POSIX sh script on every platform, since Git
- * for Windows runs a hook with the sh it ships. Where the project's install
- * is not in the work tree - a linked worktree nobody installed into, a
+ * for Windows runs a hook with the sh it ships. git starts it at the top of
+ * the work tree it commits in, so the project's install is read from there,
+ * and where a linked worktree has none, from the repository's main work
+ * tree: the work tree whose `.git` the common directory is, where `git
+ * worktree add` was run and the project installed (ADR-0005). Where neither
+ * has the install - a worktree of a repository nobody installed into, a
  * repository that shares its hooks with one that uses the harness - node
- * would end on a stack trace, so the hook says what is missing and where, and
- * the commit waits, as it did when npx found nothing to run.
+ * would end on a stack trace, so the hook says what is missing and where,
+ * and the commit waits, as it did when npx found nothing to run.
  */
 export const PRE_COMMIT = `#!/bin/sh
 # spec-harness: refuse a commit that changes what the active brief protects.
 # git runs this at the top of the work tree it commits in, a linked worktree's
-# own, wherever the hook is kept: the project's install is read from there.
+# own, wherever the hook is kept: the project's install is read from there,
+# or, where a linked worktree has none, from the repository's main work tree.
 bin=${BIN}
 if [ ! -f "$bin" ]; then
-  printf '%s\\n' "spec-harness: $bin is not in $PWD, the work tree git commits in: install the project's dependencies there, as with npm ci, and commit again" >&2
-  exit 2
+  common=$(git rev-parse --path-format=absolute --git-common-dir 2>/dev/null)
+  if [ "\${common##*/}" = .git ] && [ -f "\${common%/.git}/$bin" ]; then
+    bin=\${common%/.git}/$bin
+  else
+    printf '%s\\n' "spec-harness: $bin is not in $PWD, the work tree git commits in, nor in its repository's main work tree: install the project's dependencies there, as with npm ci, and commit again" >&2
+    exit 2
+  fi
 fi
 exec node "$bin" hook git
 `;
@@ -983,6 +1238,14 @@ export function describeGitHook(hook) {
             return `${hook.file} runs spec-harness, but is not executable, so git skips it: chmod +x ${hook.file}`;
         case 'other':
             return `${hook.file} does not run spec-harness: add the line "${GIT_HOOK_LINE}" to it`;
+        case 'stops':
+            return `${hook.file} is ${hook.what}, not a hook, and git stops every commit at it, since it cannot run it ("cannot spawn" on Windows, "cannot exec" elsewhere): nothing is checked there, and nothing is committed. Take it away, then run spec-harness init --git-hook --write`;
+        case 'ignored':
+            return `${hook.file} is ${hook.what}, not a hook, and is not executable, so git skips it and runs no hook: take it away, then run spec-harness init --git-hook --write`;
+        case 'unreadable':
+            if (!hook.runs)
+                return `${hook.file} cannot be read (${hook.reason}), and is not executable, so git skips it and runs no hook: make it readable and executable, or take it away and run spec-harness init --git-hook --write`;
+            return `${hook.file} cannot be read (${hook.reason}), so whether it runs spec-harness cannot be told: git runs it at a commit, and a script there that cannot be read stops every commit, where a program runs; make it readable`;
         case 'absent':
             return missingGitHook(hook.file);
         case 'unknown':