@holmes-lab/holmes-kit 0.7.1 → 0.8.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/CHANGELOG.md CHANGED
@@ -5,6 +5,72 @@ All notable changes to this project will be documented in this file.
5
5
  The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
6
6
  and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
7
7
  <!-- @implements A-SPEC-209 -->
8
+ ## [0.8.1] - 2026-09-03
9
+
10
+ Codex hard-enforcement completed and verified on real codex-cli 0.152.1 (GOAL-codex-enforcement),
11
+ plus the README refreshed to the 0.8.x feature set.
12
+
13
+ ### Fixed
14
+
15
+ - **Codex marketplace manifest location** (REQ-533): `init --agent codex` now writes the manifest
16
+ where codex-cli 0.152.x actually reads it (`.claude-plugin/marketplace.json`), keeping the old
17
+ `.agents/plugins/` location for 0.151.x. Measured: the shipped wiring installs with no manual copy
18
+ — `codex plugin marketplace add` → `codex plugin add` → `installed, enabled`. `doctor` now flags a
19
+ wiring that has only the old location.
20
+
21
+ ### Verified
22
+
23
+ - **Codex hard gate, model-independently**: the codex PreToolUse payload was captured (Claude-shaped:
24
+ `tool_name` + `tool_input`) and replayed through the gate — an unauthorized shell code-write is
25
+ denied, a harmless read passes. A session cannot self-grant `HOLMES_AUTONOMOUS_APPROVAL` (blocked
26
+ like `HOLMES_ROLE`).
27
+
28
+ ## [0.8.0] - 2026-09-03
29
+
30
+ A hardening campaign (GOAL-hardening-2026-09) closing measured gate gaps, adding server-side CI
31
+ re-validation, a session banner with update notice, and — new this release — **autonomous spec
32
+ approval** so an agent in an explicitly-enabled autonomous mode can seal low/mid-risk specs
33
+ without the human elicitation TUI, while gate-behavior, architecture, taint, and upstream specs
34
+ still ask a human. Every slice was TDD'd (RED verified first) with two consecutive clean
35
+ adversarial rounds where a gate was changed.
36
+
37
+ ### Added
38
+
39
+ - **Autonomous spec approval** (REQ-532): with the out-of-band `HOLMES_AUTONOMOUS_APPROVAL`
40
+ switch set (an agent cannot set it — the pre-tool-use gate blocks that, like `HOLMES_ROLE`),
41
+ `spec_approve` seals a low/mid-risk spec itself, ledgered under an `autonomous:<client>` actor
42
+ so audits tell the channel apart from human (`elicitation:`) and operator (`env`) approvals.
43
+ The bound is conservative: `gate-behavior` breaking changes, architecture/gate/governance/taint
44
+ files, and every REQ/H-SPEC/C-SPEC stay on the human channel. Autonomy OFF is byte-identical to
45
+ before.
46
+ - **Session banner + update notice** (REQ-531): every session start emits an English intro line
47
+ (version + governance rule + npm page URL) to both the human transcript and the agent context
48
+ via a SessionStart hook, and the MCP server carries the same banner in its `instructions` for
49
+ harnesses without that hook. When the `~/.holmes` cache knows a newer published version, an
50
+ install-mode-aware update command is appended. The registry refresh is detached and fail-silent;
51
+ `HOLMES_NO_UPDATE_CHECK` or `CI` opts out.
52
+ - **Server-side CI re-validation** (REQ-530): a Gitea Actions workflow re-runs the pre-push
53
+ evidence check (npm ci → build → full suite → tarball install probe) so a `--no-verify` push or
54
+ a hook-less clone is still caught. The lockfile is now tracked (`npm ci` reproducible). NOTE:
55
+ wiring only until a runner is registered — an operator step.
56
+
57
+ ### Fixed
58
+
59
+ - **cd-relative shell writes are judged at the segment's effective directory** (REQ-528): a write
60
+ spelled from a subdirectory (`cd sub && cat > ../src/x.ts`) is judged at the real location, not
61
+ the project root — five bypasses (subshell, `sh -c`, heredoc-program, …) are sealed and three
62
+ legitimate out-of-tree writes are un-blocked. A quote-aware split closes a data-injection route
63
+ a transparent split would open.
64
+ - **A file's whole governing anchor SET is judged** (REQ-529): comma-list anchors past the first
65
+ id used to be absent from the stale gate and phaseCheck; every governing id now participates and
66
+ the refusal names the one that failed. String-value anchors in JSON/config files
67
+ (`"//": "@implements …"` in package.json) are honored; code-file fixture strings stay excluded.
68
+
69
+ ### Guardrail
70
+
71
+ - **`HOLMES_AUTONOMOUS_APPROVAL` joins the self-disarm family**: a session cannot read or set the
72
+ autonomy switch (same protection as `HOLMES_ROLE`/`HOLMES_GATE_BYPASS`).
73
+
8
74
  ## [0.7.1] - 2026-09-02
9
75
 
10
76
  An adversarial self-review of the seven-language work — 42 probes the test suite never pinned
package/README.md CHANGED
@@ -14,16 +14,19 @@
14
14
 
15
15
  ---
16
16
 
17
- ### 🛡️ Currently Supported Features (v0.3.x Production Features)
17
+ ### 🛡️ Currently Supported Features (v0.8.x Production Features)
18
18
 
19
- - 📋 **Requirements & Specification Governance**: Strict **"No Spec, No Code"** enforcement with 4-tier spec chain traceability (`REQ ➔ H-SPEC ➔ A-SPEC ➔ T-SPEC`) and `// @implements A-SPEC-XXX` line 1 code anchors.
19
+ - 📋 **Requirements & Specification Governance**: Strict **"No Spec, No Code"** enforcement with 4-tier spec chain traceability (`REQ ➔ H-SPEC ➔ A-SPEC ➔ T-SPEC`) and `// @implements A-SPEC-XXX` code anchors (comma-lists and every anchor in a file participate in the gate).
20
+ - 🤖 **Autonomous Approval** *(new in 0.8.0)*: with the out-of-band `HOLMES_AUTONOMOUS_APPROVAL` switch on, an agent seals **low/mid-risk** specs itself (ledgered under an `autonomous:<client>` actor) — while `gate-behavior` changes, architecture/gate/taint files, and every upstream `REQ`/`H-SPEC`/`C-SPEC` still ask a human through the in-session TUI. The switch is env-only; a session cannot set it (blocked like `HOLMES_ROLE`). Off = byte-identical to before.
21
+ - 🪧 **Session Banner + Update Notice** *(new in 0.8.0)*: every session start emits an English intro (version + governance rule + npm URL) to both the human transcript and the agent context (SessionStart hook + MCP `instructions`); when a newer published version is cached, an install-mode-aware update command is appended. Registry check is detached, fail-silent, and opts out via `HOLMES_NO_UPDATE_CHECK`/`CI`.
22
+ - 🧱 **Deterministic Gate, Hardened** *(new in 0.8.0)*: shell writes are judged at the segment's **effective working directory** (`cd sub && cat > ../src/x.ts` is sealed, legitimate out-of-tree scratch writes are freed); the governing anchor is the **whole set**, not the first match. Every gate change ships with two consecutive clean adversarial rounds.
20
23
  - 🧠 **3-Tier Semantic Layer** *(new in 0.3.0)*: knowledge-graph semantic search with an explicit consent ladder — `none` (default, **zero egress**), `local` (bge-m3, no egress, optional module), `cloud` (gemini-embedding-001, opt-in via `GEMINI_API_KEY`). Measured on 305 traceability cases: recall 0.486 (lexical) → 0.667 (local) → **0.887 (cloud)**; on lexical-zero requests: 0% → 52% → **92%**. Surfaced only additively — rerank, evidence (`semCos`), and `semanticAlternates` — never as a hard filter.
21
24
  - 🎯 **Graded Impact Surface** *(new in 0.3.0)*: `rankedImpact` (personalized-PageRank over the spec/code graph) beat its pre-registered naive baseline on **both recall and precision across 3 corpora (×1.6–×17)** — the necessary condition for any better-than-a-person phrasing, measured before claimed.
22
- - 🐞 **Causal Defect Localization & CPG**: AST Code Property Graph (CPG) & Dataflow Taint reachability analysis across 7 languages (TS/JS, Python, Go, Rust, Java, C/C++, C#).
25
+ - 🐞 **Causal Defect Localization & CPG** *(equalized in 0.5–0.7)*: AST Code Property Graph (CFG/DDG/CDG) & Dataflow Taint reachability across 7 languages (TS/JS, Python, Go, Rust, Java, C/C++, C#) — **42 language×layer cells graded on measured evidence** (11 corpora, 39,344 functions, zero invariant violations; C++ conditional on 67.9% parse coverage, disclosed in the matrix).
23
26
  - 📏 **Measured, Not Claimed** *(new in 0.3.x)*: performance is judged against a pre-registered modeled-human band (R 0.67–0.78 / P ≈0.9±). Current official grade: **band entry on recall; division-of-labor precision 0.727 = 81% of the modeled human — reproduced by an independent context-free judge on a fresh blind window.** No superhuman claims until both metrics exceed the band.
24
27
  - 🧪 **Self-Healing & Diagnostic Doctor**: Automated integrity checks and self-healing auto-fix remediation (`holmes-kit doctor --fix` & `spec_remediate`) — wiring-handshake checks run on Windows natively as of 0.3.2.
25
28
  - 🔔 **Approval UX** *(new in 0.3.1)*: in-session approval dialogs forewarn their 120s deadline and, on expiry, the refusal says exactly where the decision went (`npx holmes-kit approve` out-of-band queue) — no more silently dead dialogs.
26
- - 🚦 **CI/CD Governance Gate Runner**: Non-interactive headless CI/CD build gate (`holmes-kit ci`) for GitHub Actions and GitLab CI pipelines.
29
+ - 🚦 **Push & Server-Side Re-Validation** *(hardened in 0.8.0)*: a local `pre-push` evidence gate (test-run ledger head == push HEAD, green, executed > 0) plus a **server-side CI workflow** that re-runs `npm ci → build → full suite → tarball install probe`, so a `--no-verify` push or a hook-less clone is still caught.
27
30
  - 📊 **Automated RTM & Taint Heatmap**: Interactive standalone HTML/SVG report generation (`generateRtmHeatmap`) for spec coverage and security dataflow reachability.
28
31
  - 🤖 **CLI-First AI Harness Matrix**: Native process hook gating for Claude Code, Antigravity CLI (AGY), Codex CLI, and Google Antigravity SDK.
29
32
 
@@ -46,7 +49,7 @@ Holmes-Kit prioritizes **CLI-based AI Coding Agents** where OS-level process hoo
46
49
  | :--- | :--- | :--- |
47
50
  | 🤖 **Claude Code CLI** | 🥇 Tier 1 (Native) | OS PreToolUse & Stop hooks (`.claude/settings.local.json`), MCP server (`.mcp.json`) |
48
51
  | 🚀 **Antigravity CLI (AGY)** | 🥇 Tier 1 (Native) | AGY Hooks (`hooks.json`), MCP config (`.agents/mcp_config.json`), Governance Skills |
49
- | 💻 **Codex CLI / Agentic Shell** | 🥇 Tier 1 (Native) | Codex MCP integration (`.codex/config.toml`), plugin-packaged gate hooks (installed via Codex plugin marketplace) |
52
+ | 💻 **Codex CLI / Agentic Shell** | 🥇 Tier 1 (Native) | Codex MCP integration (`.codex/config.toml`), plugin-packaged gate hooks at the marketplace path Codex reads (`.claude-plugin/marketplace.json`, codex-cli 0.152.x). Hard-gate enforcement verified on real codex-cli: the captured PreToolUse payload is judged and an unauthorized code-write is denied. |
50
53
  | 🧩 **Google Antigravity SDK** | 🥇 Tier 1 (Native) | Autonomous Agent SDK bindings and cryptographic provenance verification |
51
54
 
52
55
  > **Note**: Holmes-Kit focuses strictly on CLI-based autonomous agents to guarantee 100% deterministic OS hook gating (`deny` enforcement) before file modifications occur.
package/dist/.build-id CHANGED
@@ -1 +1 @@
1
- 9196dd1-mtk1gito
1
+ a14dfa1-mtkqq7jw
@@ -204,19 +204,29 @@ function agentFiles(agent, opts) {
204
204
  // The MCP wiring stays in `.codex/config.toml` (merged by init.ts).
205
205
  // Wiring is not installation: the user still runs `codex plugin marketplace add` and
206
206
  // `codex plugin add`, which is why HARNESS_ENFORCES.codex is false.
207
+ // @implements A-SPEC-533.1 — the marketplace manifest, built ONCE and emitted at BOTH
208
+ // locations. codex-cli 0.152.x reads `<root>/.claude-plugin/marketplace.json` (measured
209
+ // 2026-09-03: the `.agents/plugins/` path A-SPEC-442 wrote for 0.151.0 fails with "does not
210
+ // contain a supported manifest"); the old path stays for 0.151.x backward compatibility.
211
+ // One source so the two files cannot drift in format or source.path.
212
+ const marketplaceJson = `${JSON.stringify({
213
+ name: exports.CODEX_MARKETPLACE,
214
+ interface: { displayName: 'Holmes-Kit (local)' },
215
+ plugins: [{
216
+ name: 'holmes-kit',
217
+ source: { source: 'local', path: `./${exports.CODEX_PLUGIN_DIR.split(path.sep).join('/')}` },
218
+ policy: { installation: 'AVAILABLE', authentication: 'ON_INSTALL' },
219
+ category: 'Developer Tools',
220
+ }],
221
+ }, null, 2)}\n`;
207
222
  return [
208
223
  {
209
- path: path.join(target, '.agents', 'plugins', 'marketplace.json'),
210
- content: `${JSON.stringify({
211
- name: exports.CODEX_MARKETPLACE,
212
- interface: { displayName: 'Holmes-Kit (local)' },
213
- plugins: [{
214
- name: 'holmes-kit',
215
- source: { source: 'local', path: `./${exports.CODEX_PLUGIN_DIR.split(path.sep).join('/')}` },
216
- policy: { installation: 'AVAILABLE', authentication: 'ON_INSTALL' },
217
- category: 'Developer Tools',
218
- }],
219
- }, null, 2)}\n`,
224
+ path: path.join(target, '.claude-plugin', 'marketplace.json'), // codex 0.152.x reads here
225
+ content: marketplaceJson,
226
+ },
227
+ {
228
+ path: path.join(target, '.agents', 'plugins', 'marketplace.json'), // 0.151.x (backward compat)
229
+ content: marketplaceJson,
220
230
  },
221
231
  {
222
232
  path: path.join(target, exports.CODEX_PLUGIN_DIR, '.codex-plugin', 'plugin.json'),
@@ -876,9 +876,14 @@ async function runDoctor(packageRoot, target, opts, extraChecks) {
876
876
  // it made the tool agree with the mistake.
877
877
  const cdxPluginJson = path.join(target, agents_1.CODEX_PLUGIN_DIR, '.codex-plugin', 'plugin.json');
878
878
  const cdxHooksJson = path.join(target, agents_1.CODEX_PLUGIN_DIR, 'hooks.json');
879
- const cdxMarketplace = path.join(target, '.agents', 'plugins', 'marketplace.json');
879
+ // @implements A-SPEC-533.3 — codex-cli 0.152.x reads the marketplace manifest at
880
+ // `.claude-plugin/marketplace.json` (measured 2026-09-03); 0.151.x read `.agents/plugins/`.
881
+ // Judge the gate against the location codex ACTUALLY reads, so a pre-A-SPEC-533.1 wiring (old
882
+ // location only) is caught instead of passing as "wiring is correct" when it cannot install.
883
+ const cdxMarketplaceNew = path.join(target, '.claude-plugin', 'marketplace.json');
884
+ const cdxMarketplaceOld = path.join(target, '.agents', 'plugins', 'marketplace.json');
880
885
  const installCmds = `codex plugin marketplace add "${target}" 후 codex plugin add holmes-kit@${agents_1.CODEX_MARKETPLACE}`;
881
- if (fs.existsSync(cdxPluginJson) && fs.existsSync(cdxHooksJson) && fs.existsSync(cdxMarketplace)) {
886
+ if (fs.existsSync(cdxPluginJson) && fs.existsSync(cdxHooksJson) && (fs.existsSync(cdxMarketplaceNew) || fs.existsSync(cdxMarketplaceOld))) {
882
887
  try {
883
888
  const h = JSON.parse(fs.readFileSync(cdxHooksJson, 'utf8'));
884
889
  const manifest = JSON.parse(fs.readFileSync(cdxPluginJson, 'utf8'));
@@ -895,6 +900,12 @@ async function runDoctor(packageRoot, target, opts, extraChecks) {
895
900
  // The one path measured to load. `./hooks/hooks.json` parses fine and never fires.
896
901
  add('codex gate', 'FAIL', `플러그인 매니페스트의 hooks 가 './hooks.json' 이 아닙니다: ${String(manifest.hooks)} — Codex 가 적재하지 않습니다`, 'holmes-kit init --target <dir> --agent codex --force 로 다시 배선하십시오.');
897
902
  }
903
+ else if (!fs.existsSync(cdxMarketplaceNew)) {
904
+ // @implements A-SPEC-533.3 — hooks are correct, but the marketplace manifest is only at
905
+ // the old `.agents/plugins/` location; codex 0.152.x reads `.claude-plugin/` and cannot
906
+ // install this. Judged AFTER the stale-hook / manifest FAILs so those still win.
907
+ add('codex gate', 'WARN', `codex 0.152.x 가 읽는 .claude-plugin/marketplace.json 이 없습니다 — 구 위치(.agents/plugins/marketplace.json)만 있어 codex plugin marketplace add 가 "manifest 없음" 으로 실패합니다 (0.151.x 는 .agents/plugins/, 0.152.x 는 .claude-plugin/)`, 'holmes-kit init --target <dir> --agent codex --force 로 다시 배선하면 두 위치에 매니페스트가 놓입니다.');
908
+ }
898
909
  else {
899
910
  add('codex gate', 'WARN', '플러그인 배선은 옳습니다 — 그러나 설치되기 전까지 게이트는 집행되지 않습니다(Codex 는 설치된 마켓플레이스에서만 플러그인을 적재하며, doctor 는 설치 여부를 오프라인에서 확인할 수 없습니다)', `${installCmds} 를 실행한 뒤 새 세션에서 사용하십시오.`);
900
911
  }
@@ -97,6 +97,9 @@ function buildHookPlan(packageRoot, matcher, specsDir = '.ax/specs') {
97
97
  preToolUseMatcher: matcher,
98
98
  preToolUseCommand: q(path.join(packageRoot, 'dist', 'holmes', 'hooks', 'pre-tool-use.js')),
99
99
  stopCommand: q(path.join(packageRoot, 'dist', 'holmes', 'hooks', 'stop.js')),
100
+ // @implements A-SPEC-531.2 — the SessionStart banner runs the same node entry; it takes no
101
+ // `--specs` arg (the banner reads only the package version and the ~/.holmes cache).
102
+ sessionStartCommand: `node "${path.join(packageRoot, 'dist', 'holmes', 'hooks', 'session-start.js')}"`,
100
103
  };
101
104
  }
102
105
  /**
@@ -32,6 +32,7 @@ export interface HookPlan {
32
32
  preToolUseMatcher: string;
33
33
  preToolUseCommand: string;
34
34
  stopCommand: string;
35
+ sessionStartCommand?: string;
35
36
  }
36
37
  /** A hook entry belongs to holmes-kit iff its command references our hook scripts. */
37
38
  export declare function isHolmesCommand(command: string): boolean;
@@ -12,7 +12,7 @@ exports.mergeMcpServers = mergeMcpServers;
12
12
  exports.removeMcpServer = removeMcpServer;
13
13
  /** A hook entry belongs to holmes-kit iff its command references our hook scripts. */
14
14
  function isHolmesCommand(command) {
15
- return /dist[/\\]holmes[/\\]hooks[/\\](pre-tool-use|stop)\.js/.test(command) || /\bholmes-(kit|mcp|pre-tool-use|stop)\b/.test(command);
15
+ return /dist[/\\]holmes[/\\]hooks[/\\](pre-tool-use|stop|session-start)\.js/.test(command) || /\bholmes-(kit|mcp|pre-tool-use|stop)\b/.test(command);
16
16
  }
17
17
  /**
18
18
  * The script path out of a hook command line, e.g. `node "/pkg/dist/.../stop.js"` -> `/pkg/dist/.../stop.js`.
@@ -43,6 +43,11 @@ function mergeHooks(existing, plan) {
43
43
  };
44
44
  upsert('PreToolUse', { matcher: plan.preToolUseMatcher, hooks: [{ type: 'command', command: plan.preToolUseCommand }] });
45
45
  upsert('Stop', { hooks: [{ type: 'command', command: plan.stopCommand }] }); // Stop takes NO matcher
46
+ // @implements A-SPEC-531.2 — the banner hook, wired like Stop (no matcher). Guarded so an older
47
+ // caller that builds a plan without it is unchanged (no empty SessionStart group appears).
48
+ if (plan.sessionStartCommand) {
49
+ upsert('SessionStart', { hooks: [{ type: 'command', command: plan.sessionStartCommand }] });
50
+ }
46
51
  out.hooks = hooks;
47
52
  return out;
48
53
  }
@@ -0,0 +1,14 @@
1
+ import { Spec } from '../spec/spec-parser';
2
+ /** The out-of-band switch. An agent cannot read or set it — the pre-tool-use gate blocks that
3
+ * (A-SPEC-532.2), the same self-disarm protection HOLMES_APPROVAL and HOLMES_GATE_BYPASS have. */
4
+ export declare const AUTONOMY_ENV = "HOLMES_AUTONOMOUS_APPROVAL";
5
+ export type ApprovalAutonomy = 'auto' | 'hitl';
6
+ export declare function isHighRiskPath(p: string): boolean;
7
+ /**
8
+ * The autonomy verdict for approving THIS spec. `resolveParent` is accepted for future
9
+ * parent-aware rules (kept in the signature so callers wire it once); the current bound is decided
10
+ * from the spec itself.
11
+ */
12
+ export declare function specApprovalAutonomy(spec: Spec, _resolveParent: (id: string) => Spec | null): ApprovalAutonomy;
13
+ /** Whether autonomous approval is enabled at all — the out-of-band switch, read at the wiring layer. */
14
+ export declare function autonomousApprovalEnabled(env: NodeJS.ProcessEnv): boolean;
@@ -0,0 +1,75 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ exports.AUTONOMY_ENV = void 0;
4
+ exports.isHighRiskPath = isHighRiskPath;
5
+ exports.specApprovalAutonomy = specApprovalAutonomy;
6
+ exports.autonomousApprovalEnabled = autonomousApprovalEnabled;
7
+ const scope_judgment_1 = require("../guardrail/scope-judgment");
8
+ /** The out-of-band switch. An agent cannot read or set it — the pre-tool-use gate blocks that
9
+ * (A-SPEC-532.2), the same self-disarm protection HOLMES_APPROVAL and HOLMES_GATE_BYPASS have. */
10
+ exports.AUTONOMY_ENV = 'HOLMES_AUTONOMOUS_APPROVAL';
11
+ /**
12
+ * Path prefixes whose files govern the gate, the ledger, taint boundaries, or this session's own
13
+ * wiring. A spec touching one of these must not self-approve even at a mid grade — changing them is
14
+ * exactly the "structural / irreversible" class the owner drew the line at.
15
+ */
16
+ const HIGH_RISK_PREFIXES = [
17
+ 'src/holmes/hooks/',
18
+ 'src/holmes/guardrail/',
19
+ 'src/holmes/governance/',
20
+ '.ax/roles',
21
+ '.claude',
22
+ ];
23
+ const TAINT_MARKERS = ['taint', 'dataflow-taint', 'flow-sensitive'];
24
+ function isHighRiskPath(p) {
25
+ const s = p.replace(/^\.\//, '').replace(/^["'`]|["'`]$/g, '');
26
+ if (HIGH_RISK_PREFIXES.some((pre) => s.startsWith(pre)))
27
+ return true;
28
+ if (s === '.mcp.json' || s.endsWith('/.mcp.json'))
29
+ return true;
30
+ // A taint/security boundary file anywhere under rtm/ — the flow engine and its vocabulary.
31
+ if (s.startsWith('src/holmes/rtm/') && TAINT_MARKERS.some((m) => s.includes(m)))
32
+ return true;
33
+ return false;
34
+ }
35
+ /** The declared breaking-change grade of an A-SPEC, or null when absent/blank. */
36
+ function breakingGrade(spec) {
37
+ const raw = spec.frontmatter?.breaking_change;
38
+ if (typeof raw !== 'string' || raw.trim() === '')
39
+ return null;
40
+ // `<grade>: <reason>` or a bare `none` — the grade is the token before the first colon.
41
+ return raw.trim().split(':')[0].trim();
42
+ }
43
+ // Grades at or below code-interface are auto; gate-behavior (and anything unrecognised) is not.
44
+ const AUTO_GRADES = new Set(['none', 'persisted-artifact', 'derived-artifact', 'code-interface']);
45
+ /**
46
+ * The autonomy verdict for approving THIS spec. `resolveParent` is accepted for future
47
+ * parent-aware rules (kept in the signature so callers wire it once); the current bound is decided
48
+ * from the spec itself.
49
+ */
50
+ function specApprovalAutonomy(spec, _resolveParent) {
51
+ switch (spec.type) {
52
+ case 'T-SPEC':
53
+ return 'auto'; // tests are low risk
54
+ case 'REQ':
55
+ case 'H-SPEC':
56
+ case 'C-SPEC':
57
+ return 'hitl'; // wide blast radius / structural constraint
58
+ case 'A-SPEC': {
59
+ const grade = breakingGrade(spec);
60
+ if (grade === null || !AUTO_GRADES.has(grade))
61
+ return 'hitl'; // undeclared or gate-behavior
62
+ const paths = (0, scope_judgment_1.fttPathTokens)(spec.sections['Files to Touch'] ?? '');
63
+ if (paths.some(isHighRiskPath))
64
+ return 'hitl'; // gate/governance/taint file
65
+ return 'auto';
66
+ }
67
+ default:
68
+ return 'hitl'; // unknown kind: fail-safe to human
69
+ }
70
+ }
71
+ /** Whether autonomous approval is enabled at all — the out-of-band switch, read at the wiring layer. */
72
+ function autonomousApprovalEnabled(env) {
73
+ const v = env[exports.AUTONOMY_ENV];
74
+ return typeof v === 'string' && v !== '';
75
+ }
@@ -153,4 +153,29 @@ export interface PathImpl {
153
153
  resolve: (...parts: string[]) => string;
154
154
  sep: string;
155
155
  }
156
+ /**
157
+ * @implements A-SPEC-528.1
158
+ * One shell command, cut at its separators, with the EFFECTIVE working directory each piece runs
159
+ * in. The shell-write rules used to hand every relative candidate to `resolvesInside` with the
160
+ * project root as the base; a `cd` earlier in the command moves that base, and the mismatch was
161
+ * measured both ways (2026-09-03 probe): five in-project writes spelled from a subdirectory were
162
+ * allowed, three legitimate out-of-tree writes were denied.
163
+ *
164
+ * `base: null` means the gate cannot know where the piece runs (dynamic cd argument, `cd -`,
165
+ * `popd`, ambiguous multi-token target). The consumer fails CLOSED on a governed relative write
166
+ * candidate in such a piece — and stays silent when no candidate follows.
167
+ *
168
+ * The split is QUOTE-AWARE on purpose: a separator inside quotes is data, and a transparent split
169
+ * would let `printf 'cd /\n'` place a fake `cd` at a segment start and push the base out of the
170
+ * project (injection found while designing this — the honest-revision note in H-SPEC-528).
171
+ * Command strings are re-opened explicitly instead: `sh -c '…'`/`eval '…'` programs and
172
+ * shell-stdin heredoc bodies recurse (their cds stay LOCAL to the child shell, matching real
173
+ * semantics); data heredoc bodies keep their text for candidate matching but never move the base.
174
+ * Pure: no fs, no env reads beyond `os.homedir()` for `~`, no `process.platform`.
175
+ */
176
+ export interface ShellSegment {
177
+ text: string;
178
+ base: string | null;
179
+ }
180
+ export declare function shellSegments(command: string, startBase: string, depth?: number): ShellSegment[];
156
181
  export declare function resolvesInside(raw: string, roots: readonly string[], impl?: PathImpl): boolean;
@@ -43,6 +43,7 @@ exports.isProtectedTarget = isProtectedTarget;
43
43
  exports.specTargetOf = specTargetOf;
44
44
  exports.protectedFileKindOf = protectedFileKindOf;
45
45
  exports.protectedKindOf = protectedKindOf;
46
+ exports.shellSegments = shellSegments;
46
47
  exports.resolvesInside = resolvesInside;
47
48
  // @implements A-SPEC-163
48
49
  const fs = __importStar(require("node:fs"));
@@ -300,6 +301,148 @@ function protectedKindOf(root, raw) {
300
301
  }
301
302
  return null;
302
303
  }
304
+ const HEREDOC_RE = /<<-?\s*(['"]?)([A-Za-z_]\w*)\1/;
305
+ const SHELL_STDIN_RE = /(?:^|[\s;|&(])(?:sh|bash|zsh|dash|ksh)\b[^<\n]*<</;
306
+ const PROG_STRING_RE = /\b(?:(?:sh|bash|zsh|dash|ksh)\b[^\n;|&]*?-c|eval)\s+(?:'([^']*)'|"([^"]*)")/g;
307
+ function shellSegments(command, startBase, depth = 0) {
308
+ if (depth > 3)
309
+ return [{ text: command, base: null }];
310
+ const out = [];
311
+ let base = startBase;
312
+ const pieces = [];
313
+ {
314
+ const lines = command.split('\n');
315
+ let cur = [];
316
+ const flush = () => { if (cur.length > 0) {
317
+ pieces.push({ text: cur.join('\n'), kind: 'cmd' });
318
+ cur = [];
319
+ } };
320
+ for (let i = 0; i < lines.length; i++) {
321
+ const line = lines[i];
322
+ const hd = HEREDOC_RE.exec(line);
323
+ if (!hd) {
324
+ cur.push(line);
325
+ continue;
326
+ }
327
+ cur.push(line);
328
+ flush();
329
+ const body = [];
330
+ for (i++; i < lines.length && lines[i].replace(/^\t+/, '') !== hd[2]; i++)
331
+ body.push(lines[i]);
332
+ pieces.push({ text: body.join('\n'), kind: SHELL_STDIN_RE.test(line) ? 'prog' : 'data' });
333
+ }
334
+ flush();
335
+ }
336
+ // Quote-aware split at ; & | and newlines — separators inside quotes are data.
337
+ const splitTop = (text) => {
338
+ const parts = [];
339
+ let acc = '';
340
+ let quote = null;
341
+ for (let k = 0; k < text.length; k++) {
342
+ const ch = text[k];
343
+ if (quote !== null) {
344
+ acc += ch;
345
+ if (ch === quote)
346
+ quote = null;
347
+ continue;
348
+ }
349
+ if (ch === "'" || ch === '"') {
350
+ quote = ch;
351
+ acc += ch;
352
+ continue;
353
+ }
354
+ if (ch === '\\' && k + 1 < text.length) {
355
+ acc += ch + text[++k];
356
+ continue;
357
+ }
358
+ if (ch === ';' || ch === '&' || ch === '|' || ch === '\n') {
359
+ if (acc.trim() !== '')
360
+ parts.push(acc);
361
+ acc = '';
362
+ continue;
363
+ }
364
+ acc += ch;
365
+ }
366
+ if (acc.trim() !== '')
367
+ parts.push(acc);
368
+ return parts;
369
+ };
370
+ const applyCd = (part) => {
371
+ // `(`/`{` open groups whose leading cd still runs (a brace group shares the CURRENT shell; a
372
+ // subshell's cd is over-approximated by design — REQ-528 Out). `builtin cd` and `command cd`
373
+ // ARE the real cd — round-1 adversarial harvest: the unprefixed matcher left the base behind
374
+ // while the shell moved.
375
+ const lead = part.replace(/^[\s({]+/, '').replace(/^(?:builtin|command(?:\s+-p)?)\s+/, '');
376
+ if (/^popd\b/.test(lead)) {
377
+ base = null;
378
+ return;
379
+ }
380
+ const m = /^(?:cd|chdir|pushd)(?:\s+([\s\S]*))?$/.exec(lead);
381
+ if (!m)
382
+ return;
383
+ const rawArg = (m[1] ?? '').trim();
384
+ if (rawArg === '') {
385
+ base = os.homedir();
386
+ return;
387
+ }
388
+ if (/[$`]/.test(rawArg)) {
389
+ base = null;
390
+ return;
391
+ }
392
+ const words = rawArg.match(/(?:[^\s'"]+|'[^']*'|"[^"]*")+/g) ?? [];
393
+ if (words.length !== 1) {
394
+ base = null;
395
+ return;
396
+ }
397
+ const arg = words[0].replace(/^"([^"]*)"$/, '$1').replace(/^'([^']*)'$/, '$1');
398
+ if (arg === '' || arg === '-') {
399
+ base = null;
400
+ return;
401
+ }
402
+ if (arg === '~') {
403
+ base = os.homedir();
404
+ return;
405
+ }
406
+ if (arg.startsWith('~/')) {
407
+ base = path.join(os.homedir(), arg.slice(2));
408
+ return;
409
+ }
410
+ if (base === null) {
411
+ const kind = absoluteKindOf(arg);
412
+ base = kind === null ? null : pathFlavorFor(arg, arg).resolve(arg);
413
+ return;
414
+ }
415
+ base = pathFlavorFor(base, arg).resolve(base, arg);
416
+ };
417
+ for (const piece of pieces) {
418
+ if (piece.kind === 'data') {
419
+ out.push({ text: piece.text, base });
420
+ continue;
421
+ }
422
+ if (piece.kind === 'prog') {
423
+ // A shell reading its program from stdin is a CHILD — its cds do not move the parent base.
424
+ out.push(...shellSegments(piece.text, base ?? startBase, depth + 1)
425
+ .map((s) => (base === null ? { ...s, base: null } : s)));
426
+ continue;
427
+ }
428
+ for (const part of splitTop(piece.text)) {
429
+ let rem = part;
430
+ for (const m of part.matchAll(PROG_STRING_RE)) {
431
+ const prog = m[1] ?? m[2];
432
+ if (prog === undefined || prog === '')
433
+ continue;
434
+ // Same child-shell rule as stdin programs; blank the program out of the parent text so its
435
+ // candidates are judged once, at the child's own bases.
436
+ out.push(...shellSegments(prog, base ?? startBase, depth + 1)
437
+ .map((s) => (base === null ? { ...s, base: null } : s)));
438
+ rem = rem.replace(`'${prog}'`, "''").replace(`"${prog}"`, '""');
439
+ }
440
+ applyCd(rem);
441
+ out.push({ text: rem, base });
442
+ }
443
+ }
444
+ return out;
445
+ }
303
446
  function resolvesInside(raw, roots, impl = path) {
304
447
  if (typeof raw !== 'string' || raw.length === 0 || roots.length === 0)
305
448
  return false;
@@ -535,11 +535,14 @@ function evaluateHook(input, specsDir, opts) {
535
535
  // class as HOLMES_ROLE. The GEMINI_API_KEY spelling is included because that is the
536
536
  // ecosystem-compatible name the resolver honors; the narrowed usage forms below keep
537
537
  // `grep GEMINI_API_KEY src/…` free exactly as they do for the HOLMES names.
538
- const SECRET = String.raw `(?:HOLMES_(?:LEDGER_KEY|APPROVAL|ROLE|GATE_BYPASS|SEMANTIC_API_KEY)|GEMINI_API_KEY|GOOGLE_API_KEY)`;
538
+ // @implements A-SPEC-532.2 — HOLMES_AUTONOMOUS_APPROVAL is the out-of-band autonomy switch; a
539
+ // session that could set it would self-grant autonomous approval, the same self-disarm the
540
+ // ROLE/GATE_BYPASS names are blocked for.
541
+ const SECRET = String.raw `(?:HOLMES_(?:LEDGER_KEY|APPROVAL|ROLE|GATE_BYPASS|SEMANTIC_API_KEY|AUTONOMOUS_APPROVAL)|GEMINI_API_KEY|GOOGLE_API_KEY)`;
539
542
  // Which of the two harms this is. Setting a role or a bypass is not reading a secret, it is
540
543
  // self-granting authority — reporting both as "reads the environment" sends an operator to
541
544
  // hunt a leak that never happened.
542
- const GRANTS_SELF = new RegExp(String.raw `\b(?:HOLMES_(?:ROLE|GATE_BYPASS)|HOLMES_SEMANTIC_API_KEY|GEMINI_API_KEY|GOOGLE_API_KEY)\b`).test(command);
545
+ const GRANTS_SELF = new RegExp(String.raw `\b(?:HOLMES_(?:ROLE|GATE_BYPASS|AUTONOMOUS_APPROVAL)|HOLMES_SEMANTIC_API_KEY|GEMINI_API_KEY|GOOGLE_API_KEY)\b`).test(command);
543
546
  const usesSecret =
544
547
  // @implements A-SPEC-477 — the credential file is the same secret at rest; reading,
545
548
  // copying or redirecting it is the harvest in file form. Path-usage only, so mentioning
@@ -655,22 +658,53 @@ function evaluateHook(input, specsDir, opts) {
655
658
  return m[i];
656
659
  return undefined;
657
660
  };
658
- const cfgCandidates = [];
659
- const cfgCollect = (re) => {
660
- for (const m of command.matchAll(re)) {
661
+ // @implements A-SPEC-528.1 — every shell-write candidate carries the EFFECTIVE base of the
662
+ // segment it appears in. The rules below used to hand relative candidates to `resolvesInside`
663
+ // with the project root as the base; a `cd` earlier in the command moves that base, and the
664
+ // mismatch was measured both ways (2026-09-03 probe C1~C11): five in-project writes spelled
665
+ // from a subdirectory walked through, three legitimate out-of-tree writes were denied.
666
+ const segments = (0, write_target_1.shellSegments)(command, opts.projectRoot);
667
+ const isTilde = (p) => p === '~' || p.startsWith('~/');
668
+ // Relative candidate in a segment whose base is unknowable (dynamic cd argument): 'unknown',
669
+ // and the caller fails CLOSED — but only when such a candidate exists, so `cd "$X" && npm
670
+ // test` stays free.
671
+ const judgeCand = (c, roots) => {
672
+ if ((0, write_target_1.absoluteKindOf)(c.raw) !== null || isTilde(c.raw))
673
+ return (0, write_target_1.resolvesInside)(c.raw, roots) ? 'in' : 'out';
674
+ if (c.base === null)
675
+ return 'unknown';
676
+ const pp = (0, write_target_1.pathFlavorFor)(c.base, c.raw);
677
+ const abs = pp.resolve(c.base, c.raw);
678
+ const kind = (0, write_target_1.absoluteKindOf)(abs);
679
+ return (0, write_target_1.resolvesInside)(abs, roots, kind === 'drive' || kind === 'unc' ? path.win32 : path.posix) ? 'in' : 'out';
680
+ };
681
+ const collectSeg = (seg, re, list) => {
682
+ for (const m of seg.text.matchAll(re)) {
661
683
  const g = firstGroup(m);
662
684
  if (g !== undefined)
663
- cfgCandidates.push(g);
685
+ list.push({ raw: g, base: seg.base });
664
686
  }
665
687
  };
666
- cfgCollect(new RegExp(`(?:>{1,2}|\\btee\\b(?:\\s+-a)?\\s)\\s*${pathAlt(CONFIG_FILE)}`, 'g'));
667
- cfgCollect(new RegExp(`\\b(?:cp|mv|install|rsync|ln|dd|truncate|chmod|chown)\\b[^;|&]*?${pathAlt(CONFIG_FILE)}`, 'g'));
668
- cfgCollect(new RegExp(`\\bsed\\b[^;|&]*\\s-i\\b[^;|&]*?${pathAlt(CONFIG_FILE)}`, 'g'));
688
+ const cfgCandidates = [];
689
+ for (const seg of segments) {
690
+ collectSeg(seg, new RegExp(`(?:>{1,2}|\\btee\\b(?:\\s+-a)?\\s)\\s*${pathAlt(CONFIG_FILE)}`, 'g'), cfgCandidates);
691
+ collectSeg(seg, new RegExp(`\\b(?:cp|mv|install|rsync|ln|dd|truncate|chmod|chown)\\b[^;|&]*?${pathAlt(CONFIG_FILE)}`, 'g'), cfgCandidates);
692
+ collectSeg(seg, new RegExp(`\\bsed\\b[^;|&]*\\s-i\\b[^;|&]*?${pathAlt(CONFIG_FILE)}`, 'g'), cfgCandidates);
693
+ }
694
+ // Interpreter one-liner paths keep their whole-text matching (A-SPEC-448/450); a RELATIVE one
695
+ // is judged as a may-analysis over every determinate base plus the start base — deny-only
696
+ // over-approximation, sealed in A-SPEC-528.1 §3.
697
+ const detBases = [...new Set(segments.map((s) => s.base).filter((b) => b !== null)), opts.projectRoot];
698
+ const hasUnknownBase = segments.some((s) => s.base === null);
699
+ const expandInterp = (raw) => (0, write_target_1.absoluteKindOf)(raw) !== null || isTilde(raw)
700
+ ? [{ raw, base: opts.projectRoot }]
701
+ : [...detBases.map((b) => ({ raw, base: b })), ...(hasUnknownBase ? [{ raw, base: null }] : [])];
669
702
  // @implements A-SPEC-447 — one containment predicate, shared with the code rule below.
670
703
  const SESSION_ROOTS = [opts.projectRoot, path.join(os.homedir(), '.claude')];
671
704
  const CFG_RE = new RegExp(`${CONFIG_FILE}$`);
672
- const configWrite = [...cfgCandidates, ...interpreterWrites.filter((p) => CFG_RE.test(p))]
673
- .some((raw) => (0, write_target_1.resolvesInside)(raw, SESSION_ROOTS));
705
+ const cfgVerdicts = [...cfgCandidates, ...interpreterWrites.filter((p) => CFG_RE.test(p)).flatMap(expandInterp)]
706
+ .map((c) => judgeCand(c, SESSION_ROOTS));
707
+ const configWrite = cfgVerdicts.includes('in');
674
708
  // Residual, stated rather than papered over: an INTERPRETER can write these files too. A first
675
709
  // draft matched `node|python|perl|ruby` anywhere near the pattern and denied
676
710
  // `node -e "console.log(cfg.env.name)"` — reading a property named `env`. A rule that cannot
@@ -694,46 +728,55 @@ function evaluateHook(input, specsDir, opts) {
694
728
  // extension part and `pathAlt` supplies the prefix and the quoting alternatives.
695
729
  const EXT_TAIL = `\\.${CODE_EXT}`;
696
730
  const candidates = [];
697
- const collect = (re) => {
698
- for (const m of command.matchAll(re)) {
699
- const g = firstGroup(m);
700
- if (g !== undefined)
701
- candidates.push(g);
702
- }
703
- };
704
- collect(new RegExp(`(?:>{1,2}|\\btee\\b(?:\\s+-a)?\\s)\\s*${pathAlt(EXT_TAIL)}`, 'g'));
705
- collect(new RegExp(`\\btouch\\s+[^;|&]*?${pathAlt(EXT_TAIL)}`, 'g'));
706
- collect(new RegExp(`\\bsed\\b[^;|&]*\\s-i\\b[^;|&]*?${pathAlt(EXT_TAIL)}`, 'g'));
731
+ for (const seg of segments) {
732
+ collectSeg(seg, new RegExp(`(?:>{1,2}|\\btee\\b(?:\\s+-a)?\\s)\\s*${pathAlt(EXT_TAIL)}`, 'g'), candidates);
733
+ collectSeg(seg, new RegExp(`\\btouch\\s+[^;|&]*?${pathAlt(EXT_TAIL)}`, 'g'), candidates);
734
+ collectSeg(seg, new RegExp(`\\bsed\\b[^;|&]*\\s-i\\b[^;|&]*?${pathAlt(EXT_TAIL)}`, 'g'), candidates);
735
+ }
707
736
  // @implements A-SPEC-449 — the copy/move family. The CONFIG rule has carried it since
708
737
  // A-SPEC-191 §28 ("a gate that names one syntax for an act is a gate over that syntax, not
709
738
  // over the act"); the code rule never got it, and `cp /tmp/e.ts src/a.ts` walked through.
710
739
  // Only the TARGET counts: `cp src/a.ts /tmp/backup.ts` reads the project and writes outside,
711
740
  // which is an ordinary backup and not this gate's business.
712
- for (const seg of command.split(/[;|&]+/)) {
713
- if (!/\b(?:cp|mv|install|rsync|ln|dd)\b/.test(seg))
714
- continue;
715
- const of = seg.match(new RegExp(`\\bof=${pathAlt(EXT_TAIL)}`));
716
- if (of) {
717
- const g = firstGroup(of);
718
- if (g !== undefined) {
719
- candidates.push(g);
741
+ // (A data heredoc body may still carry separators inside one segment, so the per-segment
742
+ // text is split the way the whole command used to be — the base is the segment's either way.)
743
+ for (const seg of segments) {
744
+ for (const sub of seg.text.split(/[;|&]+/)) {
745
+ if (!/\b(?:cp|mv|install|rsync|ln|dd)\b/.test(sub))
720
746
  continue;
747
+ const of = sub.match(new RegExp(`\\bof=${pathAlt(EXT_TAIL)}`));
748
+ if (of) {
749
+ const g = firstGroup(of);
750
+ if (g !== undefined) {
751
+ candidates.push({ raw: g, base: seg.base });
752
+ continue;
753
+ }
721
754
  }
755
+ const all = [...sub.matchAll(new RegExp(pathAlt(EXT_TAIL), 'g'))]
756
+ .map(firstGroup).filter((x) => x !== undefined);
757
+ if (all.length >= 2)
758
+ candidates.push({ raw: all[all.length - 1], base: seg.base }); // last path is the destination
722
759
  }
723
- const all = [...seg.matchAll(new RegExp(pathAlt(EXT_TAIL), 'g'))]
724
- .map(firstGroup).filter((x) => x !== undefined);
725
- if (all.length >= 2)
726
- candidates.push(all[all.length - 1]); // last path is the destination
727
760
  }
728
761
  // @implements A-SPEC-447 — same predicate as the config rule; it lived in two copies, which
729
762
  // is how one of two gets fixed.
730
763
  const CODE_RE = new RegExp(`\\.${CODE_EXT}$`);
731
- const shellCodeWrite = [...candidates, ...interpreterWrites.filter((p) => CODE_RE.test(p))]
732
- .some((raw) => (0, write_target_1.resolvesInside)(raw, [opts.projectRoot]));
733
- if (shellCodeWrite) {
764
+ const codeVerdicts = [...candidates, ...interpreterWrites.filter((p) => CODE_RE.test(p)).flatMap(expandInterp)]
765
+ .map((c) => judgeCand(c, [opts.projectRoot]));
766
+ // @implements A-SPEC-528.1 — an unresolvable base under a governed relative write candidate
767
+ // fails CLOSED: the gate cannot know where the file lands, and guessing is the bypass this
768
+ // slice closes. Config candidates get the same fail-closed unconditionally (the config rule
769
+ // has never depended on governance); code candidates stay dormant on a spec-less repo.
770
+ const UNRESOLVED_BASE_DENY = '[Holmes-Kit] shell write to a relative path cannot be located: an earlier cd has a target the gate cannot resolve statically (dynamic argument) — use an absolute path, the gated Write/Edit tools, or out-of-band approval';
771
+ if (cfgVerdicts.includes('unknown')) {
772
+ return { permissionDecision: 'deny', permissionDecisionReason: UNRESOLVED_BASE_DENY };
773
+ }
774
+ if (codeVerdicts.includes('in') || codeVerdicts.includes('unknown')) {
734
775
  const governed = readSpecsSync(specsDir).some((s) => s.status === 'approved');
735
776
  if (governed) {
736
- return { permissionDecision: 'deny', permissionDecisionReason: '[Holmes-Kit] shell write to a project code file bypasses the No-Spec-No-Code gate — use the Write/Edit tools (gated) or supply out-of-band approval' };
777
+ return codeVerdicts.includes('in')
778
+ ? { permissionDecision: 'deny', permissionDecisionReason: '[Holmes-Kit] shell write to a project code file bypasses the No-Spec-No-Code gate — use the Write/Edit tools (gated) or supply out-of-band approval' }
779
+ : { permissionDecision: 'deny', permissionDecisionReason: UNRESOLVED_BASE_DENY };
737
780
  }
738
781
  }
739
782
  }
@@ -1231,10 +1274,32 @@ function evaluateHook(input, specsDir, opts) {
1231
1274
  // edit to that id. Only when the file carries no on-disk anchor (new file, or first anchoring) does
1232
1275
  // the incoming payload supply it. Found live: editing a test file whose fixture named an unapproved
1233
1276
  // spec was denied even though the file's own anchor is approved.
1234
- const ANCHOR_RE = /@implements\s+(A-SPEC-\d{3,}(?:\.\d+)?)/;
1235
- const payloadAnchor = ANCHOR_RE.exec([input.tool_input.content ?? '', input.tool_input.new_string ?? ''].join('\n'));
1236
- const diskAnchor = ANCHOR_RE.exec(onDiskContent);
1237
- const m = diskAnchor ?? payloadAnchor;
1277
+ // @implements A-SPEC-529.1
1278
+ // The GOVERNING anchor is a SET, not the first regex match. Comma lists are legal (S-503.1), so
1279
+ // `@implements A-SPEC-100, A-SPEC-200` claims both — and every id after the first used to be
1280
+ // absent from the stale gate and from phaseCheck (measured 2026-09-03: a broken seal on the
1281
+ // second id passed). The shared parser is consumed through `stripStringLiterals`, the same
1282
+ // preprocessing the claim gate got in S-504.1, so a fixture string anchor no longer governs.
1283
+ // Disk-anchor precedence is preserved as a set: a non-empty disk set governs, else the payload.
1284
+ const { anchorSpecIds: govAnchorIds, stripStringLiterals: govStrip } = require('../rtm/anchor-ids');
1285
+ const uniq = (xs) => [...new Set(xs)];
1286
+ // @implements A-SPEC-529.2 — the preprocessing branches on FILE FORMAT. A-SPEC-529.1 stripped
1287
+ // string literals so a code fixture like `const s = "@implements A-SPEC-999"` could not become
1288
+ // the governing anchor; but a JSON file's ONLY way to carry an anchor is a string value
1289
+ // (`"//": "@implements A-SPEC-209"` in package.json), and stripping over-dropped it — measured
1290
+ // 2026-09-03, a release package.json edit was refused with "A-SPEC(unspecified)". Code formats
1291
+ // (line-comment anchors) still strip; JSON/YAML/TOML/config and unknown extensions read raw
1292
+ // (over-inclusion is safe — an anchored-but-unapproved id is still refused downstream).
1293
+ const GOV_CODE_EXT = /\.(?:ts|tsx|mts|cts|js|jsx|mjs|cjs|py|go|rs|java|kt|cs|cc|cxx|cpp|hh|hpp|rb|php|swift)$/i;
1294
+ const govPrep = (s) => (GOV_CODE_EXT.test(relPath) ? govStrip(s) : s);
1295
+ const diskGovIds = uniq(govAnchorIds(govPrep(onDiskContent)));
1296
+ const payloadGovIds = uniq(govAnchorIds(govPrep([input.tool_input.content ?? '', input.tool_input.new_string ?? ''].join('\n'))));
1297
+ const governingIds = diskGovIds.length > 0 ? diskGovIds : payloadGovIds;
1298
+ // The single "primary" id kept for message templates and phaseCheck's targetAspecId default: the
1299
+ // first governing id, which is exactly what `m?.[1]` used to be for a single-anchor file.
1300
+ const m = governingIds.length > 0
1301
+ ? Object.assign([`@implements ${governingIds[0]}`, governingIds[0]], { index: 0, input: '', groups: undefined })
1302
+ : null;
1238
1303
  // RE-ANCHORING scope (review C6) moved into judgeScope below (S-508.1): every payload anchor the
1239
1304
  // disk does not carry is an ADMISSION, judged against the incoming spec's gate — the C6 property
1240
1305
  // (a rewrite of the anchor line is judged under the NEW spec) is preserved there, now for the
@@ -1268,11 +1333,15 @@ function evaluateHook(input, specsDir, opts) {
1268
1333
  }
1269
1334
  return null;
1270
1335
  };
1271
- // The qualifying T-SPEC matters for WRITE_CODE too a stale test spec must not bless code.
1272
- const staleTarget = sealProblem(m?.[1]) ?? (action === 'WRITE_CODE'
1273
- ? (specs.filter((s) => s.type === 'T-SPEC' && s.status === 'approved' && m?.[1] && s.dependsOn.includes(m[1]))
1274
- .map((s) => sealProblem(s.id)).find(Boolean) ?? null)
1275
- : null);
1336
+ // @implements A-SPEC-529.1 the stale gate runs over the WHOLE governing set: any anchored spec
1337
+ // whose seal is broken, and (for WRITE_CODE) any of that spec's qualifying T-SPECs, blocks — and
1338
+ // the message names the one that failed, not the first anchor. The single-anchor path is a set
1339
+ // of one, byte-identical to the previous `m?.[1]` behaviour.
1340
+ const staleTarget = (governingIds.map((id) => sealProblem(id)).find(Boolean) ?? null)
1341
+ ?? (action === 'WRITE_CODE'
1342
+ ? (specs.filter((s) => s.type === 'T-SPEC' && s.status === 'approved' && governingIds.some((id) => s.dependsOn.includes(id)))
1343
+ .map((s) => sealProblem(s.id)).find(Boolean) ?? null)
1344
+ : null);
1276
1345
  // @implements A-SPEC-133 — the override is authorized by a token covering the code write.
1277
1346
  const codeWriteCovered = (0, risk_gate_1.approvalCovers)(weApproval, { kind: 'code-write', target: relPath }, nowTs);
1278
1347
  if (staleTarget && !codeWriteCovered) {
@@ -1306,7 +1375,21 @@ function evaluateHook(input, specsDir, opts) {
1306
1375
  };
1307
1376
  }
1308
1377
  }
1309
- const res = (0, phase_1.phaseCheck)(action, { specs, targetAspecId: m?.[1] });
1378
+ // @implements A-SPEC-529.1 phaseCheck runs for EVERY governing id; the first failure is the
1379
+ // verdict, and its id flows into the refusal template (so the author sees the anchor that is
1380
+ // actually unsatisfied, not merely the first one on the file). An empty set keeps the prior
1381
+ // single call with an undefined target. Deterministic: governing-set order is document order.
1382
+ const phaseTargets = governingIds.length > 0 ? governingIds : [undefined];
1383
+ let failedTarget;
1384
+ let res = (0, phase_1.phaseCheck)(action, { specs, targetAspecId: phaseTargets[0] });
1385
+ for (const tid of phaseTargets) {
1386
+ const r = (0, phase_1.phaseCheck)(action, { specs, targetAspecId: tid });
1387
+ if (r.decision === 'deny') {
1388
+ res = r;
1389
+ failedTarget = tid;
1390
+ break;
1391
+ }
1392
+ }
1310
1393
  if (res.decision === 'deny') {
1311
1394
  // @implements A-SPEC-175
1312
1395
  // When the spec tree is gone from a project the ledger remembers approving, the generic
@@ -1315,7 +1398,7 @@ function evaluateHook(input, specsDir, opts) {
1315
1398
  // the phase gate's own verdict is still what denied this.
1316
1399
  const lost = specs.length === 0 && !fs.existsSync(specsDir)
1317
1400
  && (0, governance_history_1.hasGovernanceHistory)(path.resolve(specsDir, '..', '..'));
1318
- const targetSpecId = m?.[1] ?? 'A-SPEC-XXX';
1401
+ const targetSpecId = failedTarget ?? m?.[1] ?? 'A-SPEC-XXX';
1319
1402
  const prescriptiveGuide = ` — Next Action (DO NOT write workaround scripts in /tmp): Step 1: Call 'spec_next({})' to verify slice state. Step 2: Call 'spec_approve({ id: "${targetSpecId}" })' to seal spec. Step 3: Ensure '// @implements ${targetSpecId}' is on line 1 of target file.`;
1320
1403
  return {
1321
1404
  permissionDecision: 'deny',
@@ -0,0 +1,23 @@
1
+ import { InstallMode } from '../update/update-notice';
2
+ export interface SessionStartInput {
3
+ version: string;
4
+ home: string;
5
+ now: number;
6
+ env: NodeJS.ProcessEnv;
7
+ readFile: (p: string) => string;
8
+ mode: InstallMode;
9
+ }
10
+ export interface SessionStartOutput {
11
+ hookSpecificOutput: {
12
+ hookEventName: 'SessionStart';
13
+ additionalContext: string;
14
+ };
15
+ shouldRefresh: boolean;
16
+ }
17
+ /** node_modules → local dependency; anything else is treated as a source checkout (conservative:
18
+ * a source install shows no update command, which is safer than a wrong one). A global npx pin is
19
+ * detected by the caller from argv/exec path; here the package root alone distinguishes local-dep. */
20
+ export declare function detectInstallMode(packageRoot: string): InstallMode;
21
+ /** Pure: build the banner output and decide whether a refresh should fire. No I/O beyond the
22
+ * injected readFile; the caller performs the detached spawn when shouldRefresh is true. */
23
+ export declare function buildSessionStartOutput(input: SessionStartInput): SessionStartOutput;
@@ -0,0 +1,111 @@
1
+ "use strict";
2
+ var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
3
+ if (k2 === undefined) k2 = k;
4
+ var desc = Object.getOwnPropertyDescriptor(m, k);
5
+ if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
6
+ desc = { enumerable: true, get: function() { return m[k]; } };
7
+ }
8
+ Object.defineProperty(o, k2, desc);
9
+ }) : (function(o, m, k, k2) {
10
+ if (k2 === undefined) k2 = k;
11
+ o[k2] = m[k];
12
+ }));
13
+ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
14
+ Object.defineProperty(o, "default", { enumerable: true, value: v });
15
+ }) : function(o, v) {
16
+ o["default"] = v;
17
+ });
18
+ var __importStar = (this && this.__importStar) || (function () {
19
+ var ownKeys = function(o) {
20
+ ownKeys = Object.getOwnPropertyNames || function (o) {
21
+ var ar = [];
22
+ for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
23
+ return ar;
24
+ };
25
+ return ownKeys(o);
26
+ };
27
+ return function (mod) {
28
+ if (mod && mod.__esModule) return mod;
29
+ var result = {};
30
+ if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
31
+ __setModuleDefault(result, mod);
32
+ return result;
33
+ };
34
+ })();
35
+ Object.defineProperty(exports, "__esModule", { value: true });
36
+ exports.detectInstallMode = detectInstallMode;
37
+ exports.buildSessionStartOutput = buildSessionStartOutput;
38
+ // @implements A-SPEC-531.2
39
+ // The SessionStart banner hook. It emits the intro (+ optional update notice) as additionalContext,
40
+ // which Claude Code delivers to BOTH the human transcript and the agent context — the "human and
41
+ // agent" delivery REQ-531 asks for. The update refresh is a DETACHED, fire-and-forget child so the
42
+ // session start is never delayed; its result shows up on the NEXT session.
43
+ //
44
+ // FAIL-OPEN: this is a banner, not a gate. Any error yields an empty, harmless output and exit 0 —
45
+ // the session must always start.
46
+ const fs = __importStar(require("node:fs"));
47
+ const os = __importStar(require("node:os"));
48
+ const path = __importStar(require("node:path"));
49
+ const node_child_process_1 = require("node:child_process");
50
+ const update_notice_1 = require("../update/update-notice");
51
+ /** node_modules → local dependency; anything else is treated as a source checkout (conservative:
52
+ * a source install shows no update command, which is safer than a wrong one). A global npx pin is
53
+ * detected by the caller from argv/exec path; here the package root alone distinguishes local-dep. */
54
+ function detectInstallMode(packageRoot) {
55
+ const folded = packageRoot.replace(/\\/g, '/');
56
+ return /(?:^|\/)node_modules\//.test(folded) ? 'local-dep' : 'source';
57
+ }
58
+ /** Pure: build the banner output and decide whether a refresh should fire. No I/O beyond the
59
+ * injected readFile; the caller performs the detached spawn when shouldRefresh is true. */
60
+ function buildSessionStartOutput(input) {
61
+ const cached = (0, update_notice_1.readCache)(input.home, input.readFile);
62
+ const additionalContext = (0, update_notice_1.composeBanner)({ current: input.version, cached, mode: input.mode, npmUrl: update_notice_1.NPM_URL });
63
+ const shouldRefresh = (0, update_notice_1.shouldQuery)(input.env) && (0, update_notice_1.cacheIsStale)(cached, input.now);
64
+ return { hookSpecificOutput: { hookEventName: 'SessionStart', additionalContext }, shouldRefresh };
65
+ }
66
+ // The package root, derived from this file's location (dist/holmes/hooks/session-start.js → 3 up).
67
+ function pkgRootFromEntry() {
68
+ return path.resolve(__dirname, '..', '..', '..');
69
+ }
70
+ function pkgVersion() {
71
+ try {
72
+ const raw = fs.readFileSync(path.join(pkgRootFromEntry(), 'package.json'), 'utf8');
73
+ return JSON.parse(raw).version ?? '0.0.0';
74
+ }
75
+ catch {
76
+ return '0.0.0';
77
+ }
78
+ }
79
+ // CLI entry: read the hook JSON on stdin (unused beyond triggering), emit the banner, and — when
80
+ // allowed — fire the detached refresh. Wrapped so any failure is an empty output + exit 0.
81
+ if (require.main === module) {
82
+ let buf = '';
83
+ process.stdin.on('data', (c) => (buf += c));
84
+ process.stdin.on('end', () => {
85
+ try {
86
+ const out = buildSessionStartOutput({
87
+ version: pkgVersion(),
88
+ home: os.homedir(),
89
+ now: Date.now(),
90
+ env: process.env,
91
+ readFile: (p) => fs.readFileSync(p, 'utf8'),
92
+ mode: detectInstallMode(pkgRootFromEntry()),
93
+ });
94
+ process.stdout.write(JSON.stringify({ hookSpecificOutput: out.hookSpecificOutput }));
95
+ if (out.shouldRefresh) {
96
+ // Detached, unref'd child so the session start does not wait on the network. The refresh
97
+ // subcommand queries the registry (dist-tags.latest), times out fast, and writes the cache;
98
+ // any failure there is silent. Reads only public metadata — no spec text, no egress.
99
+ try {
100
+ const child = (0, node_child_process_1.spawn)(process.execPath, [__filename, '--refresh'], { detached: true, stdio: 'ignore' });
101
+ child.unref();
102
+ }
103
+ catch { /* refresh is best-effort */ }
104
+ }
105
+ }
106
+ catch {
107
+ // Fail-open: emit nothing and let the session start.
108
+ process.stdout.write('{}');
109
+ }
110
+ });
111
+ }
@@ -130,6 +130,7 @@ const package_1 = require("../review/package");
130
130
  const risk_classifier_1 = require("../guardrail/risk-classifier");
131
131
  const risk_gate_1 = require("../guardrail/risk-gate");
132
132
  const elicit_approval_1 = require("./elicit-approval");
133
+ const autonomy_1 = require("../governance/autonomy");
133
134
  const anchor_comment_1 = require("../rtm/anchor-comment");
134
135
  const consistency_lints_1 = require("../cpg/consistency-lints");
135
136
  const approval_queue_1 = require("../governance/approval-queue");
@@ -669,6 +670,15 @@ function makeRawHandlers(store, opts) {
669
670
  token: crypto.randomUUID(),
670
671
  rationale: reason ?? 'elicitation grant',
671
672
  });
673
+ // @implements A-SPEC-532.2 — the autonomous channel: the SAME synthesized-Approval shape that
674
+ // rides the existing seal path, but its actor names `autonomous:<client>` so an audit can tell a
675
+ // self-approved seal from a human-approved (elicitation) or operator (env/grant) one. Single-use
676
+ // by construction — it exists only inside this call, persisted nowhere.
677
+ const autonomousApproval = () => ({
678
+ actor: `autonomous:${opts?.clientName?.() ?? 'unknown'}`,
679
+ token: crypto.randomUUID(),
680
+ rationale: `autonomous grant (${'HOLMES_AUTONOMOUS_APPROVAL'} enabled, spec grade auto)`,
681
+ });
672
682
  /**
673
683
  * Where the audit record for a governance act belongs — resolved BEFORE the act writes anything.
674
684
  *
@@ -1134,26 +1144,38 @@ function makeRawHandlers(store, opts) {
1134
1144
  if (approveResolved === undefined) {
1135
1145
  const target = await store.read(a.id).catch(() => null);
1136
1146
  if (target) {
1137
- const resealing = typeof target.spec.frontmatter.approved_digest === 'string';
1138
- // The MODEL text is capped BEFORE the server markers are appended (round-2): a ~185+ char
1139
- // title pushed '(재봉인)' past the dialog's 200-char summary cap, dressing a re-seal (the
1140
- // more consequential act) as a first approval. The cap cuts the title, never the marker.
1141
- const out = await tryElicit('spec-approve', a.id, `${a.id} ${target.spec.title.slice(0, 120)}${resealing ? ' (재봉인)' : ''}`);
1142
- if (out.kind === 'answered' && out.decision.granted) {
1143
- approveResolved = { approval: elicitApproval(out.decision.reason), source: 'elicitation' };
1147
+ // @implements A-SPEC-532.2 the autonomous gate sits BEFORE the human ask: when the
1148
+ // out-of-band autonomy switch is on AND the spec is low/mid-risk (never gate-behavior, an
1149
+ // architecture/taint file, or an upstream REQ/H/C those stay human), the agent seals it
1150
+ // itself. The switch is env-only and an agent cannot set it (pre-tool-use blocks that,
1151
+ // A-SPEC-532.2). Off, or a hitl-classed spec, falls straight through to the elicitor
1152
+ // unchanged the autonomous-OFF path is byte-identical to before.
1153
+ if ((0, autonomy_1.autonomousApprovalEnabled)(process.env)
1154
+ && (0, autonomy_1.specApprovalAutonomy)(target.spec, resolver([target.spec])) === 'auto') {
1155
+ approveResolved = { approval: autonomousApproval(), source: 'autonomous' };
1144
1156
  }
1145
- else if (out.kind === 'answered') {
1146
- // The human ANSWERED (deny/question/decline): the answer is the message, and no queue
1147
- // entry is filed a decided request is not a pending one (REQ-246 visibility).
1148
- return { ok: false, reason: `spec_approve: 세션에서 거부됨 — ${out.decision.reason ?? '(사유 없음)'}. 사유를 해소한 다시 시도하십시오.` };
1149
- }
1150
- else if (out.kind === 'expired') {
1151
- // @implements A-SPEC-497.1 only the expiry earns a name: the notice LEADS the same
1152
- // fail-closed refusal + queue path, so the semantics stay refusal+queue and only the
1153
- // message learned to say what happened.
1154
- elicitExpiredMs = out.waitedMs;
1157
+ else {
1158
+ const resealing = typeof target.spec.frontmatter.approved_digest === 'string';
1159
+ // The MODEL text is capped BEFORE the server markers are appended (round-2): a ~185+ char
1160
+ // title pushed '(재봉인)' past the dialog's 200-char summary cap, dressing a re-seal (the
1161
+ // more consequential act) as a first approval. The cap cuts the title, never the marker.
1162
+ const out = await tryElicit('spec-approve', a.id, `${a.id} — ${target.spec.title.slice(0, 120)}${resealing ? ' (재봉인)' : ''}`);
1163
+ if (out.kind === 'answered' && out.decision.granted) {
1164
+ approveResolved = { approval: elicitApproval(out.decision.reason), source: 'elicitation' };
1165
+ }
1166
+ else if (out.kind === 'answered') {
1167
+ // The human ANSWERED (deny/question/decline): the answer is the message, and no queue
1168
+ // entry is filed — a decided request is not a pending one (REQ-246 visibility).
1169
+ return { ok: false, reason: `spec_approve: 세션에서 거부됨 — ${out.decision.reason ?? '(사유 없음)'}. 사유를 해소한 뒤 다시 시도하십시오.` };
1170
+ }
1171
+ else if (out.kind === 'expired') {
1172
+ // @implements A-SPEC-497.1 — only the expiry earns a name: the notice LEADS the same
1173
+ // fail-closed refusal + queue path, so the semantics stay refusal+queue and only the
1174
+ // message learned to say what happened.
1175
+ elicitExpiredMs = out.waitedMs;
1176
+ }
1177
+ // silent: the channel gave no answer — fall through to the byte-identical refusal.
1155
1178
  }
1156
- // silent: the channel gave no answer — fall through to the byte-identical refusal.
1157
1179
  }
1158
1180
  }
1159
1181
  if (approveResolved === undefined) {
@@ -0,0 +1,8 @@
1
+ import { InstallMode } from '../update/update-notice';
2
+ export interface ServerInstructionsInput {
3
+ version: string;
4
+ home: string;
5
+ mode: InstallMode;
6
+ readFile: (p: string) => string;
7
+ }
8
+ export declare function buildServerInstructions(input: ServerInstructionsInput): string;
@@ -0,0 +1,13 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ exports.buildServerInstructions = buildServerInstructions;
4
+ // @implements A-SPEC-531.2
5
+ // The MCP server's `instructions` banner. A harness with no SessionStart hook (codex, antigravity)
6
+ // still gets the intro (+ optional update notice) delivered to the agent through the server's
7
+ // initialize response. PURE and fail-soft — a cache read failure yields the intro only, and never
8
+ // throws, so server construction is never blocked by the banner.
9
+ const update_notice_1 = require("../update/update-notice");
10
+ function buildServerInstructions(input) {
11
+ const cached = (0, update_notice_1.readCache)(input.home, input.readFile); // readCache already swallows a throw → null
12
+ return (0, update_notice_1.composeBanner)({ current: input.version, cached, mode: input.mode, npmUrl: update_notice_1.NPM_URL });
13
+ }
@@ -54,7 +54,27 @@ const PKG_VERSION = (() => {
54
54
  return '0.0.0';
55
55
  }
56
56
  })();
57
- const server = new index_js_1.Server({ name: 'holmes-kit', version: PKG_VERSION }, { capabilities: { tools: {} } });
57
+ // @implements A-SPEC-531.2 the banner rides `instructions` so a harness without a SessionStart
58
+ // hook still delivers the intro (+ update notice) to the agent. Fail-soft: any failure omits it and
59
+ // the server starts normally.
60
+ const SERVER_INSTRUCTIONS = (() => {
61
+ try {
62
+ const { buildServerInstructions } = require('./server-instructions');
63
+ const { detectInstallMode } = require('../hooks/session-start');
64
+ const os = require('node:os');
65
+ const fs = require('node:fs');
66
+ return buildServerInstructions({
67
+ version: PKG_VERSION,
68
+ home: os.homedir(),
69
+ mode: detectInstallMode(require('node:path').resolve(__dirname, '..', '..', '..')),
70
+ readFile: (p) => fs.readFileSync(p, 'utf8'),
71
+ });
72
+ }
73
+ catch {
74
+ return undefined;
75
+ }
76
+ })();
77
+ const server = new index_js_1.Server({ name: 'holmes-kit', version: PKG_VERSION }, { capabilities: { tools: {} }, instructions: SERVER_INSTRUCTIONS });
58
78
  const fullProfile = process.env.HOLMES_MCP_PROFILE === 'full';
59
79
  // Register each handler as a tool with its real typed inputSchema so MCP
60
80
  // clients can marshal complex (array/object) arguments; fall back to a
@@ -0,0 +1,28 @@
1
+ export declare const NPM_URL = "https://www.npmjs.com/package/@holmes-lab/holmes-kit";
2
+ /** How this install was wired — decides how (or whether) to phrase the update command. */
3
+ export type InstallMode = 'global-npx' | 'local-dep' | 'source';
4
+ export interface UpdateCache {
5
+ latest: string;
6
+ checkedAt: number;
7
+ }
8
+ export interface BannerInput {
9
+ current: string;
10
+ cached: UpdateCache | null;
11
+ mode: InstallMode;
12
+ npmUrl: string;
13
+ }
14
+ /** -1 if a<b, 0 if equal, 1 if a>b — NUMERIC per field, so 0.9.0 < 0.10.0 (a string compare fails). */
15
+ export declare function compareSemver(a: string, b: string): -1 | 0 | 1;
16
+ /** The cache file, read only — no network, no write. Anything unreadable/malformed/mis-shaped → null. */
17
+ export declare function readCache(home: string, readFile: (p: string) => string): UpdateCache | null;
18
+ /** The one-line update guidance, branched by install mode. `source` returns null (git updates it). */
19
+ export declare function installModeGuide(mode: InstallMode, latest: string, current: string): string | null;
20
+ /**
21
+ * The banner: always an English intro line (version + governance rule + npm page). If the cache
22
+ * knows a newer version AND the install mode has an update path, a second guidance line follows.
23
+ */
24
+ export declare function composeBanner(input: BannerInput): string;
25
+ /** Whether a network refresh is allowed at all. Opt-out via HOLMES_NO_UPDATE_CHECK or CI. */
26
+ export declare function shouldQuery(env: NodeJS.ProcessEnv): boolean;
27
+ /** Whether the cache is old enough to refresh. A missing cache is stale. */
28
+ export declare function cacheIsStale(cached: UpdateCache | null, now: number, ttlMs?: number): boolean;
@@ -0,0 +1,131 @@
1
+ "use strict";
2
+ var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
3
+ if (k2 === undefined) k2 = k;
4
+ var desc = Object.getOwnPropertyDescriptor(m, k);
5
+ if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
6
+ desc = { enumerable: true, get: function() { return m[k]; } };
7
+ }
8
+ Object.defineProperty(o, k2, desc);
9
+ }) : (function(o, m, k, k2) {
10
+ if (k2 === undefined) k2 = k;
11
+ o[k2] = m[k];
12
+ }));
13
+ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
14
+ Object.defineProperty(o, "default", { enumerable: true, value: v });
15
+ }) : function(o, v) {
16
+ o["default"] = v;
17
+ });
18
+ var __importStar = (this && this.__importStar) || (function () {
19
+ var ownKeys = function(o) {
20
+ ownKeys = Object.getOwnPropertyNames || function (o) {
21
+ var ar = [];
22
+ for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
23
+ return ar;
24
+ };
25
+ return ownKeys(o);
26
+ };
27
+ return function (mod) {
28
+ if (mod && mod.__esModule) return mod;
29
+ var result = {};
30
+ if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
31
+ __setModuleDefault(result, mod);
32
+ return result;
33
+ };
34
+ })();
35
+ Object.defineProperty(exports, "__esModule", { value: true });
36
+ exports.NPM_URL = void 0;
37
+ exports.compareSemver = compareSemver;
38
+ exports.readCache = readCache;
39
+ exports.installModeGuide = installModeGuide;
40
+ exports.composeBanner = composeBanner;
41
+ exports.shouldQuery = shouldQuery;
42
+ exports.cacheIsStale = cacheIsStale;
43
+ // @implements A-SPEC-531.1
44
+ // The session banner and update notice, as a PURE core. Every side channel — the filesystem, the
45
+ // clock, the environment — is a parameter, so the whole module verifies offline. The delivery
46
+ // points (SessionStart hook, MCP instructions, init wiring) consume this in A-SPEC-531.2.
47
+ //
48
+ // No new dependency: semver comparison is a three-integer compare (our versions are plain
49
+ // major.minor.patch — a full semver library would carry prerelease/build code this never runs), and
50
+ // the registry query (A-SPEC-531.2) uses Node's built-in https.
51
+ const path = __importStar(require("node:path"));
52
+ exports.NPM_URL = 'https://www.npmjs.com/package/@holmes-lab/holmes-kit';
53
+ const DEFAULT_TTL_MS = 24 * 3600_000;
54
+ /** Parse `major.minor.patch` to a 3-tuple; a non-numeric field becomes 0 (never throws). */
55
+ function triple(v) {
56
+ const parts = String(v).split('.');
57
+ const n = (i) => { const x = Number.parseInt(parts[i] ?? '', 10); return Number.isFinite(x) ? x : 0; };
58
+ return [n(0), n(1), n(2)];
59
+ }
60
+ /** -1 if a<b, 0 if equal, 1 if a>b — NUMERIC per field, so 0.9.0 < 0.10.0 (a string compare fails). */
61
+ function compareSemver(a, b) {
62
+ const x = triple(a);
63
+ const y = triple(b);
64
+ for (let i = 0; i < 3; i++) {
65
+ if (x[i] < y[i])
66
+ return -1;
67
+ if (x[i] > y[i])
68
+ return 1;
69
+ }
70
+ return 0;
71
+ }
72
+ /** The cache file, read only — no network, no write. Anything unreadable/malformed/mis-shaped → null. */
73
+ function readCache(home, readFile) {
74
+ let raw;
75
+ try {
76
+ raw = readFile(path.join(home, '.holmes', 'update-check.json'));
77
+ }
78
+ catch {
79
+ return null;
80
+ }
81
+ if (!raw)
82
+ return null;
83
+ try {
84
+ const v = JSON.parse(raw);
85
+ if (v && typeof v === 'object'
86
+ && typeof v.latest === 'string'
87
+ && typeof v.checkedAt === 'number') {
88
+ return { latest: v.latest, checkedAt: v.checkedAt };
89
+ }
90
+ return null;
91
+ }
92
+ catch {
93
+ return null;
94
+ }
95
+ }
96
+ /** The one-line update guidance, branched by install mode. `source` returns null (git updates it). */
97
+ function installModeGuide(mode, latest, current) {
98
+ const head = `[Holmes-Kit] Update available: ${latest} (current ${current}).`;
99
+ switch (mode) {
100
+ case 'global-npx':
101
+ return `${head} Run: npm i -g @holmes-lab/holmes-kit@latest, then holmes-kit init --force (re-pins wiring; requires HOLMES_APPROVAL).`;
102
+ case 'local-dep':
103
+ return `${head} Run: npm i -D @holmes-lab/holmes-kit@latest`;
104
+ case 'source':
105
+ return null;
106
+ }
107
+ }
108
+ /**
109
+ * The banner: always an English intro line (version + governance rule + npm page). If the cache
110
+ * knows a newer version AND the install mode has an update path, a second guidance line follows.
111
+ */
112
+ function composeBanner(input) {
113
+ const intro = `[Holmes-Kit] This session is governed by Holmes-Kit v${input.current} (No Spec, No Code) — ${input.npmUrl}`;
114
+ const latest = input.cached?.latest;
115
+ if (latest && compareSemver(latest, input.current) > 0) {
116
+ const guide = installModeGuide(input.mode, latest, input.current);
117
+ if (guide)
118
+ return `${intro}\n${guide}`;
119
+ }
120
+ return intro;
121
+ }
122
+ /** Whether a network refresh is allowed at all. Opt-out via HOLMES_NO_UPDATE_CHECK or CI. */
123
+ function shouldQuery(env) {
124
+ return !env.HOLMES_NO_UPDATE_CHECK && !env.CI;
125
+ }
126
+ /** Whether the cache is old enough to refresh. A missing cache is stale. */
127
+ function cacheIsStale(cached, now, ttlMs = DEFAULT_TTL_MS) {
128
+ if (!cached)
129
+ return true;
130
+ return now - cached.checkedAt > ttlMs;
131
+ }
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "//": "@implements A-SPEC-209",
3
3
  "name": "@holmes-lab/holmes-kit",
4
- "version": "0.7.1",
4
+ "version": "0.8.1",
5
5
  "description": "Holmes-Kit — deterministic Agentic Software Engineering (ASE) harness with causal traceability (spec chain + D-CPG + RTM + phase guardrail)",
6
6
  "main": "dist/holmes/mcp/server.js",
7
7
  "types": "dist/holmes/mcp/server.d.ts",