@coworker-jp/aidr 0.1.302 → 0.1.303

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coworker-jp/aidr",
-  "version": "0.1.302",
+  "version": "0.1.303",
   "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 にマップする。
  *