@coworker-jp/aidr 0.1.293 → 0.1.294

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coworker-jp/aidr",
-  "version": "0.1.293",
+  "version": "0.1.294",
   "description": "AIDR setup CLI - installs ai-scanner hooks for 19+ AI coding agents",
   "type": "module",
   "bin": {

package/src/agents/claude.mjs

@@ -13,11 +13,15 @@ const SCRIPT_FILES = [
   "scan_webfetch.sh", "scan_websearch.sh",
   "log_web_urls.sh",
 ];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "claude",

package/src/agents/codex.mjs

@@ -27,11 +27,15 @@ const SCRIPT_FILES = [
   "pre_shell.sh",
   "post_shell.sh",
 ];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 const ROOT_FILES = ["AGENTS.md"];
 
 export const meta = {

package/src/agents/cursor.mjs

@@ -28,11 +28,15 @@ const SCRIPT_FILES = [
   "post_read.sh",
   "post_write.sh",
 ];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "cursor",

package/src/agents/gemini.mjs

@@ -5,11 +5,15 @@ import * as T from "../templates.mjs";
 import { buildGeminiSettings, mergeGeminiSettings, unmergeGeminiSettings } from "../merge.mjs";
 
 const HOOK_FILES = ["aidr-pre.sh"];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "gemini",

package/src/agents/kiro.mjs

@@ -18,11 +18,15 @@ const SCRIPT_FILES = [
   "post_read.sh",
   "post_write.sh",
 ];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "kiro",

package/src/agents/standalone.mjs

@@ -2,11 +2,15 @@ import path from "node:path";
 import fsp from "node:fs/promises";
 import { pathExists, rmdirIfEmpty } from "../fs-utils.mjs";
 
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "standalone",

package/src/agents/windsurf.mjs

@@ -25,11 +25,15 @@ const SCRIPT_FILES = [
   "post_read.sh",
   "post_write.sh",
 ];
+// bin ディレクトリ直下に生まれる、バイナリ以外の生成物。
 // `.ai-scanner-unscanned` は hook が「無検査でツールコールが通った」ことを
-// 記録するファイル (#1774)。**消す対象に入れておくこと** —— 残ると bin
-// ディレクトリが空にならず `rmdirIfEmpty` が効かないので、アンインストール後に
-// ディレクトリと記録が居座る。7 エージェントすべて同じ表を持つ。
-const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned"];
+// 記録するファイル (#1774)、`.ai-scanner-last-pull` は自己更新の間引きスタンプ
+// (#1795。エージェントごとに 1 個で、以前は `~/.claude/` に 1 個だけだった)。
+// **どちらも消す対象に入れておくこと** —— 残ると bin ディレクトリが空にならず
+// `rmdirIfEmpty` が効かないので、アンインストール後にディレクトリと記録が居座る。
+// 7 エージェントすべて同じ表を持ち、`tests/bin-dir-artifacts.test.mjs` が
+// 「表 ⇔ テンプレートが実際に書く名前」を両方向で照合する。
+const BIN_FILES = ["ai-scanner", "opengrep", ".ai-scanner-unscanned", ".ai-scanner-last-pull"];
 
 export const meta = {
   name: "windsurf",

package/src/cli.mjs

@@ -624,7 +624,16 @@ export async function readUnscannedRecord(binDir) {
   }
   const parts = raw.split(/\r?\n/)[0].trim().split(/\s+/);
   if (parts.length < 3) return null;
-  const [first, last, count] = parts.map((n) => (/^\d+$/.test(n) ? Number(n) : null));
+  // 10 進の非負整数だけを受け、**`Number` が正確に持てる範囲を超えたら壊れている
+  // ものとして扱う**。超えた値をそのまま返すと `9007199254740993` が
+  // `9007199254740992` として表示され、しかも誰も気付けない。Rust 側
+  // (`docker/scanner/src/agent_hooks.rs::parse_unscanned_record`) が同じ上限を
+  // 持ち、`tests/fixtures/unscanned-record-cases.json` が両者の一致を固定する。
+  const [first, last, count] = parts.map((n) => {
+    if (!/^\d+$/.test(n)) return null;
+    const v = Number(n);
+    return Number.isSafeInteger(v) ? v : null;
+  });
   if (first === null || last === null || count === null || count === 0) return null;
   return { first: new Date(first * 1000), last: new Date(last * 1000), count };
 }

package/src/templates.mjs

@@ -32,11 +32,26 @@ export function getStartScannerSh(downloadBase = "https://pytd2jd5p7.execute-api
 # - Fail-open: never block Claude Code startup
 BIN="\${AI_SCANNER_BIN:-$HOME/.claude/bin/ai-scanner}"
 BIN_DIR="$(dirname "$BIN")"
-PULL_STAMP="$HOME/.claude/.ai-scanner-last-pull"
+# The self-update throttle belongs to the binary it throttles, so it is derived
+# from \$BIN — the one source this script already trusts for "where is my copy"
+# (\`.ai-scanner-unscanned\` is placed the same way). It used to be pinned to
+# \`\$HOME/.claude/\`, which made ONE stamp for ALL agents: every agent installs
+# its own copy under \`\$HOME/.<agent>/bin/\`, so a Claude Code session starting
+# suppressed the update check of the Cursor copy for PULL_INTERVAL seconds and
+# vice versa (#1795). docs/scanner/SCANNER.md already documented it as per-agent.
+PULL_STAMP="$BIN_DIR/.ai-scanner-last-pull"
 PULL_INTERVAL=300
 DOWNLOAD_BASE="${downloadBase}"
 
-mkdir -p "$BIN_DIR" "$HOME/.claude/quarantine" "$HOME/.claude/ai-scanner"
+# Only the directory this script itself writes into (the download lands here).
+# The scanner's own state directories are created by the binary at startup
+# (\`main()\` -> \`paths::process_paths().create_dirs()\`, docker/scanner/src/paths.rs),
+# which is the single source of truth for that layout and honours
+# AI_SCANNER_DATA_DIR / AI_SCANNER_QUARANTINE_DIR. Creating them here as well
+# was a second, hardcoded copy of that layout that ignored both overrides, and
+# on a machine where the binary never arrives it left empty \`~/.claude/\`
+# directories behind for agents that are not Claude Code (#1795).
+mkdir -p "$BIN_DIR"
 
 # macOS has no base \`timeout\` (it's in GNU coreutils). Shim it so hooks work
 # on fresh Macs without brew install coreutils.