@coworker-jp/aidr 0.1.302 → 0.1.305

package/package.json

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

package/src/binary-fetcher.mjs

@@ -49,18 +49,54 @@ export function detectOpengrepPlatform() {
  * being handled as a release problem.
  */
 export class HttpStatusError extends Error {
-  constructor(url, status) {
+  constructor(url, status, payload = null) {
     super(`GET ${url} -> HTTP ${status}`);
     this.name = "HttpStatusError";
     this.status = status;
     this.url = url;
+    /**
+     * The endpoint's own JSON error body, or null when there wasn't one.
+     *
+     * Carried because a status code alone cannot say WHY. `/download/red-team`
+     * answers 403 for two unrelated reasons — a disabled access key and an
+     * account type that does not include the scanner — and until #2003 the CLI
+     * printed `GET <url> -> HTTP 403` for both, which names neither. The
+     * server writes one sentence for each (`aidr_common.red_team.
+     * red_team_refusal_message`); reading it here is what lets the caller show
+     * it instead of a bare number.
+     *
+     * Never parsed for a DECISION. It is display material, and it is
+     * sanitised at the point of display, not here — a body we could not parse
+     * must stay distinguishable from one that had no message.
+     */
+    this.payload = payload;
+  }
+}
+
+/** Cap on the error body we will read. Big enough for any refusal we emit. */
+const ERROR_BODY_LIMIT = 8192;
+
+/**
+ * Best-effort read of a non-2xx JSON body. Returns the parsed object or null.
+ *
+ * Never throws and never rejects: a failure to read the explanation must not
+ * replace the real failure (the HTTP status) with a parse error, which would
+ * turn "your key is not entitled" into "unexpected token < in JSON".
+ */
+async function readErrorPayload(res) {
+  try {
+    const text = (await res.text()).slice(0, ERROR_BODY_LIMIT);
+    const parsed = JSON.parse(text);
+    return parsed && typeof parsed === "object" && !Array.isArray(parsed) ? parsed : null;
+  } catch {
+    return null;
   }
 }
 
 async function fetchBuffer(url, accessKey) {
   const headers = accessKey ? { "x-access-key": accessKey } : {};
   const res = await fetch(url, { redirect: "follow", headers });
-  if (!res.ok) throw new HttpStatusError(url, res.status);
+  if (!res.ok) throw new HttpStatusError(url, res.status, await readErrorPayload(res));
   const ab = await res.arrayBuffer();
   return Buffer.from(ab);
 }

package/src/cli.mjs

@@ -784,10 +784,13 @@ async function cmdRedTeamInstall(opts) {
       );
     }
   } catch (e) {
-    // 403 red_team_requires_10_seats は「Pro/Trial かつ契約シート 10 以上」の
-    // エンタイトルメント不足。サーバー側が唯一の判断者なので、ここでは
-    // メッセージをそのまま見せる。
-    console.error(`red-team install failed: ${e.message}`);
+    // サーバー側が唯一の判断者であり、**理由を知っているのもサーバーだけ**なので、
+    // 403 の本文をそのまま見せる (issue #2003)。ここが `e.message` を出していた
+    // 間、利用者が受け取っていたのは `GET <url> -> HTTP 403` だけで、応答の本文は
+    // 1 バイトも読まれていなかった —— このコメント自身が「メッセージをそのまま
+    // 見せる」「シート 10 以上が要る」と、どちらも事実でないことを書いていた。
+    const { redTeamInstallFailureText } = await import("./red-team.mjs");
+    console.error(redTeamInstallFailureText(e));
     process.exit(1);
   }
 }

package/src/red-team.mjs

@@ -82,9 +82,10 @@ function binFileName(meta) {
  * Red Team の生バイナリを取得する。
  *
  * verify Lambda の `GET /download/red-team/[<product>/]<platform>` はアクセス
- * キー認証に加えて Red Team エンタイトルメント (pro/trial かつ契約シート >= 10)
- * を要求し、不適格なら 403 を返す。fetchAsset は非 2xx で throw するため、
- * ここでは特別扱いせず呼び出し側にメッセージを見せる。
+ * キー認証に加えて Red Team エンタイトルメント (`account_type == "red_team"`、
+ * issue #1954) を要求し、不適格なら 403 を返す。fetchAsset は非 2xx で throw
+ * するため、ここでは特別扱いせず呼び出し側にメッセージを見せる ——
+ * **見せるものは `redTeamInstallFailureText` が組み立てる** (issue #2003)。
  *
  * network-scanner は従来どおり 1 セグメント (`/red-team/<platform>`) を使う
  * (配信済みルートとバイト等価)。他 product は `/red-team/<product>/<platform>`。
@@ -107,6 +108,93 @@ export async function fetchRedTeamBinary(
   return { ...result, platform, product };
 }
 
+/**
+ * サーバーが Red Team のエンタイトルメント拒否に使う識別子。
+ *
+ * 単一情報源は `aidr_common.red_team.RED_TEAM_ACCOUNT_REQUIRED` で、これはその
+ * 写しである。**判定の写しではない** —— ここに在るのは「サーバーが返した拒否は
+ * どの拒否か」を読むための名前だけで、可否を計算する行は 1 つも無い。
+ */
+export const RED_TEAM_ACCOUNT_REQUIRED = "red_team_account_required";
+
+/**
+ * サーバー由来の文字列を端末に出す前の掃除。
+ *
+ * 制御文字 (改行とタブを除く) を落とし、長さを切る。**内容は書き換えない** ——
+ * 拒否の理由を我々が要約し直すと、サーバーが言ったことと違うものを見せる経路が
+ * できる。掃除するのは端末に対する形だけである。
+ */
+function sanitizeServerText(raw, limit = 1000) {
+  if (typeof raw !== "string") return "";
+  // eslint-disable-next-line no-control-regex
+  return raw.replace(/[\u0000-\u0008\u000b-\u001f\u007f]/g, "").trim().slice(0, limit);
+}
+
+/**
+ * `red-team install` が失敗したときに端末へ出す文面を組み立てる (issue #2003)。
+ *
+ * ## 何が壊れていたか
+ *
+ * 失敗経路は `console.error(\`red-team install failed: \${e.message}\`)` の 1 行
+ * だけで、`e.message` は `fetchBuffer` が投げる `GET <url> -> HTTP 403` だった。
+ * **応答の本文は 1 バイトも読んでいなかった。** つまり利用者が受け取っていたのは
+ * **理由の書かれていない番号**であり、その横のコメントは
+ * 「403 red_team_requires_10_seats は『Pro/Trial かつ契約シート 10 以上』の
+ * エンタイトルメント不足。…メッセージをそのまま見せる」と書いていた ——
+ * **条件は #1954 で消えており、「そのまま見せる」も事実ではなかった。**
+ *
+ * ## なぜ `status` ではなく `error` で分岐するか
+ *
+ * このルートの 403 は 2 つある: **鍵が無効 / 停止中** と
+ * **Red Team アカウントではない**。番号だけで分岐すると、停止された鍵を持つ人に
+ * 「Red Team アカウントを発行してもらってください」と言うことになる ——
+ * 発行されても、その鍵は動かない。
+ *
+ * ## なぜサーバーの文面を優先するか
+ *
+ * 可否を決めるのはサーバーだけで、**理由を知っているのもサーバーだけ**である
+ * (解決済みの `account_type` を見て、通常アカウントと「解釈できない種別」を
+ * 書き分けている)。ここで書き直すと 2 つ目の説明ができて必ず食い違う。
+ * 手元の文面は**本文が無かったとき**にだけ使う保険で、そちらも
+ * プランやシート数を解決策として名指ししない (docs/product/UI_COPY.md)。
+ */
+export function redTeamInstallFailureText(err) {
+  const status = err?.status;
+  const payload = err?.payload && typeof err.payload === "object" ? err.payload : null;
+  const code = typeof payload?.error === "string" ? payload.error : "";
+  const serverMessage = sanitizeServerText(payload?.message);
+
+  if (code === RED_TEAM_ACCOUNT_REQUIRED) {
+    return [
+      "red-team install failed: nothing was installed — the download endpoint " +
+      "refused this access key.",
+      serverMessage ||
+        // 本文の無い古い配備向け。**シートもプランも解決策として出さない。**
+        "The scanner is only available on a Red Team account — its own account " +
+        "type, issued from the CoWorker admin console. Ask your CoWorker " +
+        "administrator to issue one, then re-run this command with that " +
+        "account's access key.",
+    ].join("\n");
+  }
+
+  if (status === 401 || status === 403) {
+    // 同じ番号の別の拒否 (無効・停止中の鍵など)。**Red Team の説明を貼らない。**
+    return [
+      `red-team install failed: the download endpoint refused this access key ` +
+      `(HTTP ${status}), so nothing was installed.`,
+      serverMessage ||
+        (typeof payload?.error === "string" && payload.error) ||
+        "Check that the key is the one issued for this environment and is still " +
+        "active; ask your CoWorker administrator if it is not.",
+    ].join("\n");
+  }
+
+  // それ以外は従来どおり。**握りつぶさない** —— 分類できなかったことと、
+  // 分類した結果として何も言うことが無いことは別である。
+  const detail = sanitizeServerText(err?.message) || String(err);
+  return `red-team install failed: ${detail}`;
+}
+
 /**
  * aidr の scope を `claude mcp add` の scope にマップする。
  *
@@ -184,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 サーバーとして登録する。
  *
@@ -284,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)`,
   );