claude-spotter 1.5.8 → 1.5.10

package/CHANGELOG.md

@@ -3,6 +3,28 @@
 各節はそのversion公開時点の変更記録であり、後続versionにより置換された仕様を含む。
 現行runtime契約は[`docs/00_overview.md`](docs/00_overview.md)から辿る。
 
+## 1.5.10 — 2026-08-14
+
+- **非ClaudeのHook入力を副作用前に無視する。** 非空の`sessionId`と`hookEventName`を持ち、
+  Claudeの`session_id`を持たないcamelCase envelopeだけをunsupportedとして識別する。
+  SessionStart、UserPromptSubmit、PreToolUse、Stop、SessionEndはproject探索、daemon、評価store、
+  tool記録、Stop監査、Hook eventより前に無出力・exit 0で終了する。
+- **正式host範囲を広げない。** camelCase payloadをClaude形へ変換せず、Grok向けtool DB、auditor、
+  installer、diagnosticsを追加しない。Claude/Codexの既存入力と処理は変更しない。
+
+## 1.5.9 — 2026-08-14
+
+- **Codex side conversationのStopを正常終了させる。** Codex公式Hook契約でnullableな
+  `transcript_path`が欠落または`null`の場合と、互換payloadが空文字の場合、未処理例外とexit 2を出さず、
+  `transcript_unavailable`の構造Hook eventを記録してexit 0で終了する。
+- **不完全な利用観測を監査成功へ偽装しない。** transcriptが無いturnはauditorを呼ばず、
+  対応する評価turnを`usageStatus: incomplete`で閉じる。非空pathのENOENT・transcript anomaly・
+  通常taskの監査経路は変更せず、非string値はmalformed envelopeとしてexit 2を維持する。
+- **4環境CIを工場共通workflowへ統合。** macOS native、Linux native、Windows native、WSL2の
+  reusable workflowへ移行し、依存導入、文書検証、full testの時間を分離する。
+- **Windows CLI試験を端末PATHから分離。** Codex CLI検出・診断のWindows shim解決へ明示envを渡し、
+  開発端末の実PATHに左右されない回帰testへ固定する。
+
 ## 1.5.8 — 2026-08-05
 
 - **コードから全ドキュメントを再照合。** 47 Markdownを現行contract、ADR、完了計画、

package/docs/11_dashboard-operations.md

@@ -1,6 +1,6 @@
 # Spotter評価dashboard運用
 
-現行npm配布版: **v1.5.8**（2026-08-05）。v1.5.8は文書整合releaseであり、
+現行npm配布版: **v1.5.10**（2026-08-14）。v1.5.10は非Claude Hook入力の副作用前no-op releaseであり、
 dashboard routing構成はv1.5.3から変更していない。
 
 この文書はservice設定の正本であり、各端末に現在installされているnpm versionの台帳ではない。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-spotter",
-  "version": "1.5.8",
+  "version": "1.5.10",
   "description": "Audit agent running alongside Claude Code that catches missed tool calls — 気づく役と実行する役の分離",
   "type": "module",
   "bin": {

package/src/cli/codex-hook-cmd.mjs

@@ -15,6 +15,7 @@ import {
   die,
   findSpotterMarker,
   isChildCall,
+  optionalString,
   readStdinJson,
   requireString,
 } from '../hooks/lib.mjs';
@@ -286,7 +287,27 @@ export async function runCodexStopHook({
     });
     return;
   }
-  const transcriptPath = requireString(input, 'transcript_path');
+  const transcriptPath = optionalString(input, 'transcript_path');
+  if (!transcriptPath) {
+    await closeCodexEvaluationTurn({
+      createEvaluationStoreFn,
+      sessionId: codexSessionId(input),
+      usageStatus: 'incomplete',
+      completedAtMs: now(),
+      writeError: reportError,
+    });
+    await recordCodexHookEventSafe(recordHookEventFn, {
+      projectRoot,
+      event: {
+        hook: 'Stop',
+        status: 'skipped',
+        reason: 'transcript_unavailable',
+        usedToolCount: 0,
+        durationMs: Date.now() - startedAt,
+      },
+    }, reportError);
+    return;
+  }
   const finalResponse = codexLastAssistantMessage(input) ?? '(no final response available)';
   let toolUsage;
   try {
@@ -561,6 +582,7 @@ export async function codexHookDiagnostics({
     command: 'codex',
     args: ['features', 'list'],
     platform,
+    env,
   });
   const features = spawnSyncFn(invocation.command, invocation.args, {
     encoding: 'utf8',

package/src/cli/doctor.mjs

@@ -123,12 +123,14 @@ export function isSupportedNodeVersion(value) {
 export async function inspectCodexCliVersion({
   codexBin = 'codex',
   platform = process.platform,
+  env = process.env,
   execFileFn = execFileP,
 } = {}) {
   const invocation = buildWindowsCompatibleInvocation({
     command: codexBin,
     args: ['--version'],
     platform,
+    env,
   });
   const { stdout } = await execFileFn(invocation.command, invocation.args, {
     timeout: 5_000,

package/src/cli/install.mjs

@@ -281,6 +281,7 @@ async function exists(path) {
 
 export function isCodexCliPresent({
   platform = process.platform,
+  env = process.env,
   spawnSyncFn = spawnSync,
   codexBin = 'codex',
 } = {}) {
@@ -288,6 +289,7 @@ export function isCodexCliPresent({
     command: codexBin,
     args: ['--version'],
     platform,
+    env,
   });
   const result = spawnSyncFn(invocation.command, invocation.args, {
     encoding: 'utf8',

package/src/hooks/lib.mjs

@@ -46,6 +46,19 @@ export function isSubagentCall(input) {
       && input.agent_id.length > 0;
 }
 
+// Grok invokes Claude-compatible hook commands with a camelCase wire envelope.
+// Spotter does not support Grok as a host: ignore that envelope before any
+// project lookup, daemon, evaluation, or hook-event side effect.
+export function isUnsupportedNonClaudeEnvelope(input) {
+  return input !== null
+      && typeof input === 'object'
+      && typeof input.sessionId === 'string'
+      && input.sessionId.length > 0
+      && typeof input.hookEventName === 'string'
+      && input.hookEventName.length > 0
+      && !Object.hasOwn(input, 'session_id');
+}
+
 // Walk up from startCwd looking for .spotter/marker.json. Returns the project
 // root path containing the marker, or null if none was found before reaching
 // the filesystem root.

package/src/hooks/pre-tool-use.mjs

@@ -10,6 +10,7 @@ import {
   die,
   isChildCall,
   isSubagentCall,
+  isUnsupportedNonClaudeEnvelope,
   isOutsideSpotterProject,
   findSpotterMarker,
   recordClaudeHookEvent,
@@ -29,6 +30,7 @@ export async function runPreToolUse({
 } = {}) {
   if (isChildCall()) return;
   const input = await readInput();
+  if (isUnsupportedNonClaudeEnvelope(input)) return;
   if (isSubagentCall(input)) return;
   if (isOutsideSpotterProject(input)) return;
 

package/src/hooks/session-end.mjs

@@ -8,6 +8,7 @@ import {
   requireString,
   isChildCall,
   isSubagentCall,
+  isUnsupportedNonClaudeEnvelope,
   isOutsideSpotterProject,
   findSpotterMarker,
   recordClaudeHookEvent,
@@ -24,6 +25,7 @@ export async function runSessionEnd({
 } = {}) {
   if (isChildCall()) return;
   const input = await readInput();
+  if (isUnsupportedNonClaudeEnvelope(input)) return;
   if (isSubagentCall(input)) return;
   if (isOutsideSpotterProject(input)) return;
 

package/src/hooks/session-start.mjs

@@ -22,6 +22,7 @@ import {
   die,
   isChildCall,
   isSubagentCall,
+  isUnsupportedNonClaudeEnvelope,
   isOutsideSpotterProject,
   findSpotterMarker,
   recordClaudeHookEvent,
@@ -39,6 +40,7 @@ export async function runSessionStart({
 
   const input = await readInput();
 
+  if (isUnsupportedNonClaudeEnvelope(input)) return;
   if (isSubagentCall(input)) return;
   if (input.source !== 'startup') return;
   if (isOutsideSpotterProject(input)) return;

package/src/hooks/stop.mjs

@@ -8,6 +8,7 @@ import {
   findSpotterMarker,
   isChildCall,
   isSubagentCall,
+  isUnsupportedNonClaudeEnvelope,
   recordClaudeHookEvent,
 } from './lib.mjs';
 import { getLastAssistantText } from './transcript-reader.mjs';
@@ -26,6 +27,7 @@ export async function runStop({
 } = {}) {
   if (isChildCall()) return;
   const input = await readInput();
+  if (isUnsupportedNonClaudeEnvelope(input)) return;
   if (isSubagentCall(input)) return;
   // Resolve the installed project once. Keeping this proven root lets the failure event remain
   // attributable even if the marker disappears while the daemon request is in flight.

package/src/hooks/user-prompt.mjs

@@ -17,6 +17,7 @@ import {
   die,
   isChildCall,
   isSubagentCall,
+  isUnsupportedNonClaudeEnvelope,
   isOutsideSpotterProject,
   findSpotterMarker,
   recordClaudeHookEvent,
@@ -48,6 +49,7 @@ export async function runUserPrompt({
 } = {}) {
   if (isChildCall()) return;
   const input = await readInput();
+  if (isUnsupportedNonClaudeEnvelope(input)) return;
   if (isSubagentCall(input)) return;
   if (isOutsideSpotterProject(input)) return;