@coworker-jp/aidr 0.1.303 → 0.1.306

package/package.json

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

package/src/red-team.mjs

@@ -272,6 +272,68 @@ export function buildNpxMcpAddArgs(meta, scope) {
   ];
 }
 
+/**
+ * `fs.mkdir(dir, { recursive: true })` と同じことをして、**このプロセスが
+ * 実際に作ったディレクトリだけ**を深い順に返す (issue #2012)。
+ *
+ * ## なぜ「作ったかどうか」を覚える必要があるか
+ *
+ * `--scope project` は「いま居るディレクトリに入れる」意味なので、`home` は
+ * 利用者のリポジトリそのものである。**403 で拒否された回** ——
+ * 何もダウンロードせず、MCP にも何も登録しない回 —— でも
+ * `<cwd>/.claude/bin/` が先に作られていたので、**点検を試して断られただけの
+ * 人のリポジトリにディレクトリが 2 つ生えて残った。**
+ *
+ * 失敗経路で畳むなら、**自分が作ったものと、既に在ったものを必ず区別する。**
+ * 利用者の既存の `.claude/` を失敗時に消す実装は、元の欠陥よりはるかに重い。
+ */
+export async function mkdirTracked(dir) {
+  const created = [];
+  let cur = path.resolve(dir);
+  for (;;) {
+    try {
+      await fs.stat(cur);
+      break; // ここから上は既に在る = 我々のものではない
+    } catch (e) {
+      // **ENOENT 以外は「無い」と読まない。** 権限で見えないだけのものを
+      // 「自分が作った」と記録すると、後で他人のディレクトリを畳みにいく。
+      if (e?.code !== "ENOENT") break;
+    }
+    created.push(cur); // 深い順に積まれる = そのまま削除の順序
+    const parent = path.dirname(cur);
+    if (parent === cur) break;
+    cur = parent;
+  }
+  await fs.mkdir(dir, { recursive: true });
+  return created;
+}
+
+/**
+ * `mkdirTracked` が返したディレクトリを、**空である間だけ**深い順に畳む。
+ *
+ * **`export` しているのは、この 2 つが単体で検査できないと安全保証が固定できない
+ * から**である。`fetchAsset` は fail-closed で、**検証を通ってから**一時ファイルを
+ * 書いて rename する (`binary-fetcher.mjs`) ので、失敗経路で `bin/` の中に物が
+ * 残ることは通常起きない —— つまり install を丸ごと駆動するテストでは、
+ * **非再帰であることが効く場面に到達できない**。守っているのは並行して書いた
+ * 別プロセス (もう 1 つの `aidr install`、利用者自身) との競合であり、そこは
+ * ここを直接呼ばないと測れない。
+ *
+ * 削除は `fs.rmdir` (非再帰) で行う。これが唯一の安全保証である ——
+ * **中身のあるディレクトリに対しては ENOTEMPTY で必ず失敗する**ので、
+ * この関数は利用者のファイルを 1 バイトも消せない。1 つでも残ったら親も
+ * 空ではないので、そこで止める。
+ */
+export async function pruneEmptyCreatedDirs(dirs) {
+  for (const dir of dirs) {
+    try {
+      await fs.rmdir(dir);
+    } catch {
+      break;
+    }
+  }
+}
+
 /**
  * Red Team バイナリを取得し、Claude Code に MCP サーバーとして登録する。
  *
@@ -372,8 +434,18 @@ export async function installRedTeam({
     return { installed: false, dryRun: true, path: binPath };
   }
 
-  await fs.mkdir(binDir, { recursive: true });
-  const res = await fetchRedTeamBinary(binPath, { env, accessKey, product });
+  // 失敗した回に痕跡を残さないため、作ったディレクトリを覚えておく (#2012)。
+  const createdDirs = await mkdirTracked(binDir);
+  let res;
+  try {
+    res = await fetchRedTeamBinary(binPath, { env, accessKey, product });
+  } catch (e) {
+    // ダウンロードが失敗した = 何も置いていない。**空のままなら畳む。**
+    // 部分書き込みが残っていれば `rmdir` が ENOTEMPTY で拒むので、
+    // 診断できるものを勝手に消すこともない。
+    await pruneEmptyCreatedDirs(createdDirs);
+    throw e;
+  }
   stdout(
     `binary: ${res.path} (${res.platform}, sha256 verified)`,
   );