@pecot/tools-mcp 1.0.2 → 1.1.1

package/cli/dist/domain/athena-query.js

@@ -1,12 +1,5 @@
 // Athena に投げるクエリが参照専用かの判定（FR-008 / research R6）。
-//
-// 完全な SQL パーサは持たない。目的は「変更系と複文を awscli 起動前に落とす」ことなので、
-// 判定できない入力（未終端の引用符・コメント）は許可側ではなく拒否側へ倒す。
-// 判定は Trino 構文を前提にする。Athena では `WITH` で始まる文は必ず問い合わせで、
-// `INSERT INTO … WITH` / `CREATE TABLE … AS WITH` の `WITH` は先頭に来ない。
-// STG1 での確認（2026-09-07）: `WITH … INSERT` は StartQueryExecution 自体が失敗する。
-// 失敗の内容は InvalidRequestException / MALFORMED_QUERY で、実行に至らない。
-// 例外は内側の文を実際に実行する `EXPLAIN ANALYZE` だけで、これは `EXPLAIN` を剥がす反復で塞ぐ。
+// 判定できない入力は拒否側へ倒す。`EXPLAIN ANALYZE` は内側の文を実行するので、`EXPLAIN` を剥がす反復で塞ぐ。
 /**
 参照専用として受け付ける先頭キーワード。判定は大小文字を区別しない。
 */
@@ -20,10 +13,10 @@ const EXPLAIN_MODIFIERS = new Set(['ANALYZE', 'VERBOSE']);
 */
 const LEADING_WORD = /^\s*[A-Z]+/i;
 const REJECTION_MESSAGES = {
-    'empty': 'query_string が空です。参照クエリを指定してください',
-    'multi-statement': '複数の文は実行できません。; の後ろに文を続けず 1 文だけ指定してください',
-    'not-read-only': '参照クエリ（SELECT / WITH / SHOW / DESCRIBE / EXPLAIN）だけを実行できます',
-    'unterminated': '引用符またはコメントが閉じていないため、参照クエリだと判定できません',
+    'empty': 'query_string is empty. Specify a read-only query',
+    'multi-statement': 'Multiple statements are not allowed. Specify a single statement with nothing after the ;',
+    'not-read-only': 'Only read-only queries (SELECT / WITH / SHOW / DESCRIBE / EXPLAIN) can be executed',
+    'unterminated': 'Cannot confirm the query is read-only because a quote or comment is not closed',
 };
 function reject(reason, message) {
     return { allowed: false, reason, message };
@@ -43,11 +36,7 @@ function skipBlockComment(query, start) {
     return end === -1 ? undefined : end + 2;
 }
 /**
-引用の終端（閉じ引用符の次）。start は開き引用符の次を指す。
-
-`'…'` 文字列リテラルと `"…"` 引用識別子のエスケープは、同じ引用符の 2 重化だけ。
-Trino では `\` は文字列リテラル中でも普通の文字なので、`'\'` は「閉じていない」ではなく
-「バックスラッシュ 1 文字のリテラル」になる。
+引用の終端（閉じ引用符の次）。エスケープは引用符の 2 重化だけで、Trino では `\` は普通の文字。
 */
 function skipQuoted(query, start, quote) {
     let index = start;
@@ -65,11 +54,7 @@ function skipQuoted(query, start, quote) {
     return undefined;
 }
 /**
-コメントを空白へ、引用の中身を空の引用へ均す。
-
-中身を捨てるのは、後段が見るのは「リテラル外の `;`」と「先頭キーワード」だけだから。
-リテラルの内容は判定材料にならない。均しておけばリテラル内の `;` や `--` が
-後段の単純な文字列走査に引っ掛からない。
+コメントを空白へ、引用の中身を空の引用へ均す。リテラル内の `;` や `--` を後段の走査に拾わせないため。
 */
 function scan(query) {
     let code = '';
@@ -126,10 +111,7 @@ function afterLeadingWord(text) {
     return matched === undefined ? text : text.slice(matched[0].length);
 }
 /**
-先頭が `(` ならその対応括弧までを剥がす（`EXPLAIN (TYPE DISTRIBUTED) SELECT …`）。
-
-対応する `)` が無い場合は剥がさずに返す。剥がせないと、後段の語判定は左括弧を見て
-語を取れず、拒否側へ倒れる。
+先頭が `(` ならその対応括弧までを剥がす。対応する `)` が無ければ剥がさず、拒否側へ倒す。
 */
 function afterOptionGroup(text) {
     const trimmed = text.trimStart();
@@ -149,11 +131,7 @@ function afterOptionGroup(text) {
     return trimmed;
 }
 /**
-1 文が参照専用か。
-
-`EXPLAIN` は修飾語とオプション群を剥がして内側を見直す。1 周ごとに必ず `EXPLAIN` の 1 語を
-消費するので停止する。再帰ではなくループにするのは、`EXPLAIN` を数万段重ねた入力で
-コールスタックを溢れさせないため（溢れると失敗種別の外の例外になる）。
+1 文が参照専用か。再帰でなくループなのは、`EXPLAIN` を重ねた入力でコールスタックを溢れさせないため。
 */
 function isReadOnlyStatement(statement) {
     let current = statement;
@@ -169,10 +147,7 @@ function isReadOnlyStatement(statement) {
     return READ_ONLY_KEYWORDS.has(word);
 }
 /**
-クエリを参照専用 / 拒否に分類する。
-
-判定順は 未終端 → 複文 → 空 → 先頭キーワード。複文を空より先に見るのは、
-`; SELECT 1` のように 1 文目が空の入力を「空」ではなく「複文」と伝えるため。
+クエリを参照専用 / 拒否に分類する。複文を空より先に見るのは、`; SELECT 1` を「複文」と伝えるため。
 */
 export function classifyAthenaQuery(query) {
     const scanned = scan(query);

package/cli/dist/domain/s3-input.d.ts

@@ -1,9 +1,5 @@
 /**
-`s3://<bucket>/<key>` 形式か。
-
-空白を含む値は拒否しない。引数は shell を介さず、1 要素 1 値で awscli へ渡る。
-そのため `s3://b/k --recursive` のような値は「そういう名前のキー」として扱われ、
-別オプションには化けない（research R3）。
+`s3://<bucket>/<key>` 形式か。空白を含む値は、shell を介さず 1 値で渡るので拒否しない（research R3）。
 */
 export declare function validateS3Uri(value: string): string | undefined;
 /**
@@ -11,10 +7,7 @@ export declare function validateS3Uri(value: string): string | undefined;
 */
 export declare function validateS3ListUri(value: string): string | undefined;
 /**
-書き出し先のローカルパスとして妥当か。
-
-`s3://…` を先に見るのは、絶対パスの検証だけでも弾けるが「相対パスだ」という案内になり、
-取得元と書き出し先を取り違えた利用者に誤った直し方を促すため。
+書き出し先のローカルパスとして妥当か。`s3://…` を先に弾くのは、取得元と取り違えた利用者に誤案内しないため。
 */
 export declare function validateLocalPath(value: string): string | undefined;
 //# sourceMappingURL=s3-input.d.ts.map

package/cli/dist/domain/s3-input.js

@@ -1,11 +1,5 @@
-// S3 の入力値（オブジェクト URI・一覧 URI・ローカル出力先）の書式検証。
-//
-// research R7 の方向制約は「入力の形で表現できない」ことで担保しているが、値そのものが
-// 期待どおりの形かはここで見る。書式は contracts/mcp-tools.md の正規表現を逐語で写す。
-//
-// 戻り値は「入力名に続く述部」で、入力名と値の連結は arguments.ts が行う（`<name> <述部>: <value>`）。
-// 述部に助詞まで含めるのは、`s3://…` のような欧文始まりの語で助詞との間に空白が要るかどうかが
-// 語ごとに変わるため。文言の組み立てを 1 箇所に寄せると、規則を足したときに書式が割れない。
+// S3 の入力値（オブジェクト URI・一覧 URI・ローカル出力先）の書式検証。書式は contracts/mcp-tools.md の正規表現を写す。
+// 戻り値は入力名に続く述部で、助詞の前の空白の要否を 1 箇所に寄せるため、連結は arguments.ts が行う。
 import path from 'node:path';
 /**
 `aws_s3_cp` の取得元。バケットとキーの双方を要求する（契約表 aws_s3_cp の s3-uri）。
@@ -21,30 +15,23 @@ const S3_LIST_URI = /^s3:\/\/[^/]*(?:\/.*)?$/;
 */
 const S3_SCHEME = /^s3:\/\//i;
 /**
-`s3://<bucket>/<key>` 形式か。
-
-空白を含む値は拒否しない。引数は shell を介さず、1 要素 1 値で awscli へ渡る。
-そのため `s3://b/k --recursive` のような値は「そういう名前のキー」として扱われ、
-別オプションには化けない（research R3）。
+`s3://<bucket>/<key>` 形式か。空白を含む値は、shell を介さず 1 値で渡るので拒否しない（research R3）。
 */
 export function validateS3Uri(value) {
-    return S3_OBJECT_URI.test(value) ? undefined : 'は s3://<bucket>/<key> の形式で指定してください';
+    return S3_OBJECT_URI.test(value) ? undefined : 'must be in the form s3://<bucket>/<key>';
 }
 /**
 `s3 ls` の位置引数として妥当か（バケット一覧・prefix 指定の双方を許す）。
 */
 export function validateS3ListUri(value) {
-    return S3_LIST_URI.test(value) ? undefined : 'は s3://<bucket>[/<prefix>] の形式で指定してください';
+    return S3_LIST_URI.test(value) ? undefined : 'must be in the form s3://<bucket>[/<prefix>]';
 }
 /**
-書き出し先のローカルパスとして妥当か。
-
-`s3://…` を先に見るのは、絶対パスの検証だけでも弾けるが「相対パスだ」という案内になり、
-取得元と書き出し先を取り違えた利用者に誤った直し方を促すため。
+書き出し先のローカルパスとして妥当か。`s3://…` を先に弾くのは、取得元と取り違えた利用者に誤案内しないため。
 */
 export function validateLocalPath(value) {
     if (S3_SCHEME.test(value))
-        return 'はローカルの絶対パスで指定してください（s3:// は書き出し先にできません）';
-    return path.isAbsolute(value) ? undefined : 'は絶対パスで指定してください';
+        return 'must be a local absolute path (s3:// cannot be a destination)';
+    return path.isAbsolute(value) ? undefined : 'must be an absolute path';
 }
 //# sourceMappingURL=s3-input.js.map

package/cli/dist/infra/awscli.d.ts

@@ -1,30 +1,17 @@
 /**
-起動するコマンド。PATH 解決に委ねる（絶対パス指定はしない）。
-
-SSO のログインレジストリ（infra/sso/login-registry.ts）も同じコマンドで spawn するため
-export する。片方だけ絶対パス化するような差が入らないようにしておく。
+起動するコマンド。PATH 解決に委ねる。SSO のログインレジストリも同じコマンドで spawn するので export する。
 */
 export declare const AWSCLI_COMMAND = "aws";
 /**
-awscli 出力の上限（バイト、research R8）。
-
-応答へ載せる際の切り詰め（mcp/result.ts の limitOutput）と、本モジュールの捕捉上限が
-同じ値を基準にする。依存方向が infra → mcp を許さないため、定義は infra に置く。
+awscli 出力の上限（バイト、research R8）。依存方向が infra → mcp を許さないため、定義は infra に置く。
 */
 export declare const OUTPUT_LIMIT_BYTES: number;
 /**
-子プロセスの stdout / stderr を捕捉する上限（バイト）。超過分は捨てる。
-
-research R8 の `filter-log-events` は既定 10,000 件で数 MiB になり得る。全件を溜めると
-同時実行数（4）と掛かってサーバの常駐メモリに直結するため、応答上限の 4 倍で打ち切る
-（切り詰め後の応答に必要な分は必ず残る）。
+子プロセスの stdout / stderr を捕捉する上限（バイト）。超過分は捨てる。常駐メモリを抑えるため、応答上限の 4 倍で打ち切る。
 */
 export declare const CAPTURE_LIMIT_BYTES: number;
 /**
-生存中の awscli 子プロセス全てへ SIGTERM を送る。
-
-終了処理（mcp/serve.ts の onShutdown）から呼ぶ。既に終了した子への kill は no-op で、
-finalize 側が Set から外すため二重送出にはならない。
+生存中の awscli 子プロセス全てへ SIGTERM を送る。終了済みの子への kill は no-op なので二重送出にならない。
 */
 export declare function killRunningAwscli(): void;
 /**
@@ -44,10 +31,7 @@ export interface SpawnResult {
 export interface RunAwscliOptions {
     readonly timeoutMs: number;
     /**
-    呼び出し元がキャンセルされたら子プロセスへ SIGTERM を送る（任意）。
-  
-    MCP の tools/call は ToolCallContext.signal でキャンセルされる。配線しないと、クライアントが
-    諦めたあとも awscli が孤児として動き続ける。
+    呼び出し元がキャンセルされたら子プロセスへ SIGTERM を送る。配線しないと awscli が孤児として動き続ける。
     */
     readonly signal?: AbortSignal | undefined;
 }
@@ -63,18 +47,11 @@ export interface OutputCapture {
     text(): string;
 }
 /**
-CAPTURE_LIMIT_BYTES まで溜め、超過分は捨てる capture を作る。
-
-SSO のログインレジストリ（infra/sso/login-registry.ts）も同じ上限で子プロセスの出力を
-溜めるため export する。上限が呼び出し元ごとに割れると、常駐メモリの見積もりが立たなくなる。
+CAPTURE_LIMIT_BYTES まで溜め、超過分は捨てる capture を作る。SSO のログインレジストリも同じ上限を使うので export する。
 */
 export declare function createCapture(): OutputCapture;
 /**
-`aws <args>` を起動し、CAPTURE_LIMIT_BYTES まで capture した stdout / stderr を返す。
-
-timeout 到達・abort では SIGTERM を送り、`timedOut` / signal を結果へ載せる。
-起動 error は reject せず `exitCode: null` + `spawnErrorCode` に詰める。
-そのため呼び出し側は、失敗種別の判定を 1 か所（classifyFailure）に集約できる。
+`aws <args>` を起動し、capture した stdout / stderr を返す。起動 error は reject せず spawnErrorCode に詰める。
 */
 export declare function runAwscli(arguments_: readonly string[], options: Readonly<RunAwscliOptions>): Promise<SpawnResult>;
 /**
@@ -89,12 +66,7 @@ export type AwscliProbe = {
     readonly message: string;
 };
 /**
-awscli の存在と版を確認し、成功結果だけをプロセス内に記憶する。
-
-毎回 `aws --version` を起こすとツール 1 件あたりの子プロセスが 2 本になるため、
-Promise 自体を memo する（並行呼び出しでも実行は 1 回）。
-ただし失敗（未導入・版不足）を memo すると、利用者が awscli を導入してもサーバ再起動まで全ツールが awscli-missing を返し続ける。
-そのため失敗時は memo を捨てて、次の呼び出しで再判定する。
+awscli の存在と版を確認し、成功結果だけを memo する。失敗を memo すると導入後も再起動まで awscli-missing が続くため捨てる。
 */
 export declare function ensureAwscli(): Promise<AwscliProbe>;
 /**

package/cli/dist/infra/awscli.js

@@ -1,14 +1,7 @@
-// awscli の子プロセス実行。
-//
-// shell:false で起動するため、引数配列の要素はそのまま 1 値として awscli に渡る
-// （連結や特殊文字による別コマンド委譲が構造的に起きない）。timeout / AbortSignal は
-// SIGTERM で子を落とし、起動失敗（ENOENT 等）は reject せず spawnErrorCode に詰める。
+// awscli の子プロセス実行。shell:false で起動し、起動失敗は reject せず spawnErrorCode に詰める。
 import { spawn } from 'node:child_process';
 /**
-起動するコマンド。PATH 解決に委ねる（絶対パス指定はしない）。
-
-SSO のログインレジストリ（infra/sso/login-registry.ts）も同じコマンドで spawn するため
-export する。片方だけ絶対パス化するような差が入らないようにしておく。
+起動するコマンド。PATH 解決に委ねる。SSO のログインレジストリも同じコマンドで spawn するので export する。
 */
 export const AWSCLI_COMMAND = 'aws';
 /**
@@ -25,34 +18,19 @@ message に載せる最低版の表記。
 const MINIMUM_VERSION_TEXT = MINIMUM_VERSION.join('.');
 const VERSION_PATTERN = /aws-cli\/(\d+)\.(\d+)\.(\d+)/;
 /**
-awscli 出力の上限（バイト、research R8）。
-
-応答へ載せる際の切り詰め（mcp/result.ts の limitOutput）と、本モジュールの捕捉上限が
-同じ値を基準にする。依存方向が infra → mcp を許さないため、定義は infra に置く。
+awscli 出力の上限（バイト、research R8）。依存方向が infra → mcp を許さないため、定義は infra に置く。
 */
 export const OUTPUT_LIMIT_BYTES = 256 * 1024;
 /**
-子プロセスの stdout / stderr を捕捉する上限（バイト）。超過分は捨てる。
-
-research R8 の `filter-log-events` は既定 10,000 件で数 MiB になり得る。全件を溜めると
-同時実行数（4）と掛かってサーバの常駐メモリに直結するため、応答上限の 4 倍で打ち切る
-（切り詰め後の応答に必要な分は必ず残る）。
+子プロセスの stdout / stderr を捕捉する上限（バイト）。超過分は捨てる。常駐メモリを抑えるため、応答上限の 4 倍で打ち切る。
 */
 export const CAPTURE_LIMIT_BYTES = OUTPUT_LIMIT_BYTES * 4;
 /**
-生存中の awscli 子プロセス。
-
-runAwscli が spawn 直後に足し、finalize（close / error のどちらか先）で外す。
-サーバが SIGTERM で畳まれるときは親が先に消える。
-そのため、ここを辿って子へも SIGTERM を送らないと、子が PPID 1 へ再親付けされて動き続ける。
-timeout を張っていた親が居なくなるので、打ち切りも働かない。
+生存中の awscli 子プロセス。サーバが SIGTERM で畳まれるとき、ここを辿って子へも送らないと PPID 1 で動き続ける。
 */
 const runningChildren = new Set();
 /**
-生存中の awscli 子プロセス全てへ SIGTERM を送る。
-
-終了処理（mcp/serve.ts の onShutdown）から呼ぶ。既に終了した子への kill は no-op で、
-finalize 側が Set から外すため二重送出にはならない。
+生存中の awscli 子プロセス全てへ SIGTERM を送る。終了済みの子への kill は no-op なので二重送出にならない。
 */
 export function killRunningAwscli() {
     for (const child of runningChildren)
@@ -72,10 +50,7 @@ export function mergeStderr(captured, extra) {
     return `${captured}\n${extra}`;
 }
 /**
-CAPTURE_LIMIT_BYTES まで溜め、超過分は捨てる capture を作る。
-
-SSO のログインレジストリ（infra/sso/login-registry.ts）も同じ上限で子プロセスの出力を
-溜めるため export する。上限が呼び出し元ごとに割れると、常駐メモリの見積もりが立たなくなる。
+CAPTURE_LIMIT_BYTES まで溜め、超過分は捨てる capture を作る。SSO のログインレジストリも同じ上限を使うので export する。
 */
 export function createCapture() {
     const chunks = [];
@@ -95,11 +70,7 @@ export function createCapture() {
     };
 }
 /**
-`aws <args>` を起動し、CAPTURE_LIMIT_BYTES まで capture した stdout / stderr を返す。
-
-timeout 到達・abort では SIGTERM を送り、`timedOut` / signal を結果へ載せる。
-起動 error は reject せず `exitCode: null` + `spawnErrorCode` に詰める。
-そのため呼び出し側は、失敗種別の判定を 1 か所（classifyFailure）に集約できる。
+`aws <args>` を起動し、capture した stdout / stderr を返す。起動 error は reject せず spawnErrorCode に詰める。
 */
 export async function runAwscli(arguments_, 
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types -- AbortSignal は内部状態を持つ
@@ -140,11 +111,8 @@ options) {
             runningChildren.delete(child);
             clearTimeout(timer);
             options.signal?.removeEventListener('abort', onAbort);
-            // ENOENT 等で 'error' の直後に 'close' も発火する Node の仕様に対し、二重 finalize は
-            // settled で抑止している。稀に child が生存したまま残る場合への防御で kill しておく。
-            // 通常の close 後もここを通る（Node 24 の確認で child.killed は false のまま）。
-            // ただし _handle が既に null で kill() は no-op のため無害。
-            // catch は防御網で、通常は到達しない。
+            // 'error' の直後に 'close' も発火するため、二重 finalize は settled で抑止する。
+            // child が生存したまま残る稀な場合に備えて kill する。_handle が null なら no-op で無害。
             /* c8 ignore start -- catch 節（既に終了済みの kill が throw する場合）は観測されない防御網 */
             if (!child.killed) {
                 try {
@@ -178,10 +146,7 @@ options) {
 */
 const probeCache = { pending: undefined };
 /**
-検出した版が最低版以上か。
-
-major / minor / patch を 1 つの数へ畳む比較は minor / patch が 1000 以上で桁が衝突するため、
-上位から順に見るタプル比較にする。
+検出した版が最低版以上か。minor / patch が 1000 以上で桁が衝突しないよう、上位から順に比べる。
 */
 function isAtLeastMinimum(major, minor, patch) {
     const [minimumMajor, minimumMinor, minimumPatch] = MINIMUM_VERSION;
@@ -197,7 +162,7 @@ async function probeAwscli() {
         return {
             ok: false,
             category: 'awscli-missing',
-            message: `awscli が見つかりません。awscli v2 (${MINIMUM_VERSION_TEXT} 以上) を導入して PATH を通してください`,
+            message: `awscli not found. Install awscli v2 (${MINIMUM_VERSION_TEXT} or later) and add it to PATH`,
         };
     }
     const matched = VERSION_PATTERN.exec(`${result.stdout}\n${result.stderr}`) ?? undefined;
@@ -205,7 +170,7 @@ async function probeAwscli() {
         return {
             ok: false,
             category: 'awscli-missing',
-            message: `awscli の版を判定できませんでした: ${result.stdout.trim()}`,
+            message: `Could not determine the awscli version: ${result.stdout.trim()}`,
         };
     }
     const major = Number(matched[1]);
@@ -217,16 +182,11 @@ async function probeAwscli() {
         : {
             ok: false,
             category: 'awscli-missing',
-            message: `awscli ${MINIMUM_VERSION_TEXT} 以上が必要です（検出: ${version}）`,
+            message: `awscli ${MINIMUM_VERSION_TEXT} or later is required (detected: ${version})`,
         };
 }
 /**
-awscli の存在と版を確認し、成功結果だけをプロセス内に記憶する。
-
-毎回 `aws --version` を起こすとツール 1 件あたりの子プロセスが 2 本になるため、
-Promise 自体を memo する（並行呼び出しでも実行は 1 回）。
-ただし失敗（未導入・版不足）を memo すると、利用者が awscli を導入してもサーバ再起動まで全ツールが awscli-missing を返し続ける。
-そのため失敗時は memo を捨てて、次の呼び出しで再判定する。
+awscli の存在と版を確認し、成功結果だけを memo する。失敗を memo すると導入後も再起動まで awscli-missing が続くため捨てる。
 */
 export async function ensureAwscli() {
     const pending = probeCache.pending ?? probeAwscli();

package/cli/dist/infra/sso/login-registry.d.ts

@@ -4,15 +4,7 @@ import type { FailureCategory } from '../../domain/allowlist.js';
 */
 export declare const SSO_LOGIN_TIMEOUT_MS = 600000;
 /**
-URL とコードが確定するまでの上限。
-
-research R5 と契約書が定めるのは 10 分だけ。
-だが `sso login` が何も出さずに生き続けると、呼び出し元の tools/call が 10 分ブロックする。
-応答を返せない状態を早く畳むための上限で、契約書側には未反映の差分。
-
-確定までの計測値は 0.65 秒。
-この待ちは、mcp/executor.ts の createBoundedExecutor(4) の実行枠を占有し続ける。
-そのため `aws --version` の待ち時間（infra/awscli.ts の PROBE_TIMEOUT_MS）と同じ上限に揃える。
+URL とコードが確定するまでの上限。実行枠を占有し続けないよう、`aws --version` の PROBE_TIMEOUT_MS と揃える。
 */
 export declare const SSO_PROMPT_TIMEOUT_MS = 15000;
 /**
@@ -47,10 +39,7 @@ export interface SsoLoginSession {
     */
     readonly category?: FailureCategory;
     /**
-    awscli の終了コード。子がまだ終わっていなければキーごと無い。
-  
-    応答の exitCode は許可リストの呼び出しと同じく、awscli の終了コードをそのまま返す規約。
-    ここで持たないと、ツール層が値を作れない。シグナルで終わった子は null になる。
+    awscli の終了コード。子がまだ終わっていなければキーごと無い。シグナルで終わった子は null。
     */
     readonly exitCode?: number | null;
 }
@@ -79,17 +68,11 @@ export interface LoginRegistry {
     shutdown(): void;
 }
 /**
-ログインレジストリを作る。
-
-MCP サーバは 1 プロセスなので実運用のインスタンスは 1 つ（loginRegistry）だが、
-テストが状態を持ち越さずに検証できるようファクトリとして公開する。
+ログインレジストリを作る。実運用は loginRegistry の 1 つだが、テストが状態を持ち越さないようファクトリで公開する。
 */
 export declare function createLoginRegistry(options?: Readonly<LoginRegistryOptions>): LoginRegistry;
 /**
-MCP サーバプロセスで共有するレジストリ。
-
-ツール（mcp/tools/sso.ts）と終了配線（mcp/serve.ts）が同じインスタンスを見ないと、
-開始したフローを status が引けず、終了時に子も落ちない。
+MCP サーバプロセスで共有するレジストリ。ツールと終了配線が同じインスタンスを見ないと、status が引けず子も落ちない。
 */
 export declare function loginRegistry(): LoginRegistry;
 /**

package/cli/dist/infra/sso/login-registry.js

@@ -1,18 +1,7 @@
-// `aws sso login` の子プロセスを預かるレジストリ（research R5 / FR-007 / data-model §SsoLoginSession）。
-//
-// `sso login` はデバイス認証が終わるまで戻らないため、close まで全量を溜める runAwscli では
-// URL とコードを即時に返せない。ここでは自前で spawn し、stdout が届くたびに累積出力を
-// parseSsoLoginOutput へ掛けて、URL とコードが揃った時点で start() を resolve する。
-// 子はそのまま生かし、完了・失敗・時間切れは close とタイマで拾う。
-//
-// 子は spawn 直後に生存集合へ登録する。URL / コードの確定前も後始末の対象で、登録が遅れると
-// サーバ終了時に PPID 1 へ再親付けされた子が最大 10 分動き続ける。
-//
-// 出力に対する加工の分担は「秘匿化はここ、応答の上限はツール層」。infra から mcp を import
-// できないため、limitOutput は mcp/tools/sso.ts が掛ける。
-/* eslint-disable @typescript-eslint/prefer-readonly-parameter-types -- LoginProcess は mutable。
-   Node 由来の型を抱える。中身は ChildProcess・出力 capture・タイマ配列。
-   Readonly<T> では deep readonly の要求を満たせない。読み取りしかしない関数として扱う。 */
+// `aws sso login` の子プロセスを預かるレジストリ（research R5 / FR-007）。
+// URL とコードが揃った時点で start() を resolve し、子は生かしたまま close とタイマで拾う。
+// 秘匿化はここで、応答の上限はツール層で掛ける（infra から mcp は import できない）。
+/* eslint-disable @typescript-eslint/prefer-readonly-parameter-types -- LoginProcess は mutable */
 import { spawn } from 'node:child_process';
 import { classifyFailure } from '../../domain/failure.js';
 import { redactSecrets } from '../../domain/redact.js';
@@ -23,15 +12,7 @@ import { AWSCLI_COMMAND, createCapture, mergeStderr } from '../awscli.js';
 */
 export const SSO_LOGIN_TIMEOUT_MS = 600_000;
 /**
-URL とコードが確定するまでの上限。
-
-research R5 と契約書が定めるのは 10 分だけ。
-だが `sso login` が何も出さずに生き続けると、呼び出し元の tools/call が 10 分ブロックする。
-応答を返せない状態を早く畳むための上限で、契約書側には未反映の差分。
-
-確定までの計測値は 0.65 秒。
-この待ちは、mcp/executor.ts の createBoundedExecutor(4) の実行枠を占有し続ける。
-そのため `aws --version` の待ち時間（infra/awscli.ts の PROBE_TIMEOUT_MS）と同じ上限に揃える。
+URL とコードが確定するまでの上限。実行枠を占有し続けないよう、`aws --version` の PROBE_TIMEOUT_MS と揃える。
 */
 export const SSO_PROMPT_TIMEOUT_MS = 15_000;
 function isTerminal(state) {
@@ -61,14 +42,7 @@ function outputOf(record) {
     return record.finalOutput ?? computeOutput(record);
 }
 /**
-子プロセスの stdio が閉じた（close / error）ら呼ぶ。
-
-sessions は終端状態のレコードを次の start か shutdown() まで保持し続ける。
-capture（最大 CAPTURE_LIMIT_BYTES 分の Buffer）を持たせたままにすると、
-常駐する profile 数に比例して解放されない領域が積み上がる。output を確定させた時点で capture を手放す。
-
-Node は 'error' の後に 'close' も発火する（awscli.ts の finalize と同じ）。
-そのため二重呼び出しは finalOutput の有無で防ぐ（2 回目は capture が既に無く output が痩せるため）。
+子の stdio が閉じた（close / error）ら呼ぶ。output を確定させた時点で capture を手放し、二重呼び出しは finalOutput の有無で防ぐ。
 */
 function finalizeOutput(record) {
     if (record.finalOutput !== undefined)
@@ -93,10 +67,7 @@ function snapshotOf(record) {
     };
 }
 /**
-ログインレジストリを作る。
-
-MCP サーバは 1 プロセスなので実運用のインスタンスは 1 つ（loginRegistry）だが、
-テストが状態を持ち越さずに検証できるようファクトリとして公開する。
+ログインレジストリを作る。実運用は loginRegistry の 1 つだが、テストが状態を持ち越さないようファクトリで公開する。
 */
 export function createLoginRegistry(options = {}) {
     const timeoutMs = options.timeoutMs ?? SSO_LOGIN_TIMEOUT_MS;
@@ -114,11 +85,7 @@ export function createLoginRegistry(options = {}) {
     */
     const running = new Set();
     /**
-    shutdown() 済みの印。
-  
-    startLogin は registry.start() の前に ensureAwscli()（最長 15 秒）を await する。
-    その待ち中に stdin の EOF が来て runShutdown が先に実行されると、再開した start() が誰も回収しない子を spawn する。
-    子の stdio パイプがイベントループを保持し、親は loginTimer が最長 10 分後に発火するまで終了できない。start() の冒頭でここを見て spawn 自体をしない。
+    shutdown() 済みの印。ensureAwscli() の待ち中に終了処理が先に動くと、start() が回収されない子を spawn するため、冒頭で見る。
     */
     let isStopped = false;
     const beginLogin = (key, profile) => new Promise((resolve) => {
@@ -175,14 +142,10 @@ export function createLoginRegistry(options = {}) {
         child.stderr.on('error', ignoreStreamError);
         child.stdout.on('data', (chunk) => {
             record.stdout?.push(chunk);
-            // 出力は chunk 単位で届く。累積した全文をそのつど掛け、URL とコードが揃った時点で応答する。
-            // autofill URL はその後の行に届き得るので、揃うまでは再 parse を続けて prompt を差し替える
-            // （確定そのものは遅らせない）。全部そろえば以後の chunk では何もしない。
+            // 累積全文をそのつど掛ける。autofill URL は後の行に届き得るので、揃うまで再 parse して prompt を差し替える。
             if (record.prompt?.verificationUrlComplete !== undefined)
                 return;
-            // 基本の URL / コードが確定した後は、autofill 行を含む chunk が来たときだけ再 parse する。
-            // 含まない awscli（確認結果: 2.36.40 は含む）では、以後の chunk を処理せず返す。
-            // 累積全文（最大 CAPTURE_LIMIT_BYTES）の split と走査を、毎 chunk 実行しないため。
+            // URL / コード確定後は、autofill 行を含む chunk だけ再 parse する。毎 chunk の split と走査を避けるため。
             if (record.prompt !== undefined && !chunk.toString('utf8').includes(AUTOFILL_MARKER))
                 return;
             const prompt = parseSsoLoginOutput(record.stdout?.text() ?? '');
@@ -220,10 +183,7 @@ export function createLoginRegistry(options = {}) {
             // 時間切れ・prompt 待ちの打ち切りで既に終端なら、その理由を SIGTERM 由来の失敗で
             // 上書きしない。
             if (!isTerminal(record.state)) {
-                // 確定前の exit 0 は「デバイスフローを経ず成功した」場合。
-                // awscli 2.36.40 で確認したところ、--use-device-code を指定するとキャッシュトークンの有無に依らず
-                // 毎回デバイスフローへ入る。そのため未観測で、ここは防御として completed に倒している。
-                // research R5 の「抽出できずに終了 = failed」は、SSO 設定なし（exit ≠ 0）を想定した記述。
+                // 確定前の exit 0 は防御として completed に倒す。2.36.40 は --use-device-code で毎回デバイスフローに入り、未観測。
                 if (code === 0)
                     record.state = 'completed';
                 else
@@ -258,7 +218,7 @@ export function createLoginRegistry(options = {}) {
                 profile: key,
                 state: 'failed',
                 startedAt: Date.now(),
-                output: 'サーバ終了中のため新しいログインを開始できません',
+                output: 'Cannot start a new login because the server is shutting down',
                 category: 'other',
             };
         }
@@ -296,10 +256,7 @@ export function createLoginRegistry(options = {}) {
 }
 const sharedRegistry = {};
 /**
-MCP サーバプロセスで共有するレジストリ。
-
-ツール（mcp/tools/sso.ts）と終了配線（mcp/serve.ts）が同じインスタンスを見ないと、
-開始したフローを status が引けず、終了時に子も落ちない。
+MCP サーバプロセスで共有するレジストリ。ツールと終了配線が同じインスタンスを見ないと、status が引けず子も落ちない。
 */
 export function loginRegistry() {
     sharedRegistry.current ??= createLoginRegistry();

package/cli/dist/mcp/serve.d.ts

@@ -9,16 +9,8 @@ export interface WritableTarget {
     write(chunk: string | Uint8Array, callback?: (error?: Error | null) => void): boolean;
 }
 /**
-stdio の stdout を JSON-RPC 専有にするガードを張る。
-
-返した Writable だけが本物の stdout へ届き、それ以外の stdout 書き込み
-(console.log・依存ライブラリの進捗表示など) は stderr へ逃がす。
-
-「process.stdout.write を差し替えるだけ」では、transport 自身の JSON-RPC 応答まで stderr に流れる。
-するとクライアントが無応答になる。
-そのため、差し替え前に束縛した元の write を使う専用 Writable を transport へ渡す、という二段構えが必須になる。
-
-同じ stdout に対しては冪等で、2 回目以降は初回に作った Writable を返す。
+stdout を JSON-RPC 専有にする。返した Writable だけが本物の stdout へ届き、ほかの書き込みは stderr へ逃がす。
+同じ stdout に対しては冪等。
 */
 export declare function installStdoutGuard(stdout: WritableTarget, stderr: WritableTarget): Writable;
 /**
@@ -38,32 +30,18 @@ export interface McpServeDependencies {
     readonly stdin: Readable;
     readonly stderr: WritableTarget;
     /**
-    終了時に呼ぶ後始末。
-  
-    実行中の awscli 子プロセスのように「子プロセスを持つ機能」がここへ登録する。
-    子を kill しない限り親は終了できない (子の stdio パイプが Node のイベントループを
-    保持するため、stdin の EOF で server.close() を呼んでも待ち続ける)。
+    終了時の後始末。子の stdio パイプがイベントループを保持するため、子を kill しないと親は終了できない
     */
     readonly onShutdown: readonly (() => void)[];
     readonly signals: ProcessSignals;
 }
 /**
-MCP サーバを stdio で待ち受ける。
-
-serveStdio が接続ごとに 2025 系と 2026-07-28 系のどちらかを選び、factory のインスタンスを
-1 つ固定して配る。factory は 1 接続で複数回呼ばれ得る (2026-07-28 は探査用インスタンスを捨てることがある)。
-そのため createServer() と onerror の設定以外の副作用は持たせない。
-後始末の配線 (signals / stdin) は factory の外で 1 回だけ行う。
-
-接続が完了しても待ち受けは続く。クライアントが stdin を閉じたら子プロセスを落として
-接続を閉じ、SIGTERM を待たずにプロセスが自力で終了できるようにする。
+MCP サーバを stdio で待ち受ける。factory は 1 接続で複数回呼ばれ得るため、createServer() と onerror 以外の副作用を持たせない。
+stdin が閉じたら子プロセスを落として接続を閉じる
 */
 export declare function runMcpServe(dependencies: Readonly<McpServeDependencies>): StdioServerHandle;
 /**
-`pecot-tools mcp` の action 本体。
-
-program.parse は action の Promise を await しない。
-そのため例外はここで捕まえて、process.exitCode へ反映する (unhandled rejection にしない)。
+`pecot-tools mcp` の action 本体。program.parse は Promise を await しないので、例外はここで process.exitCode へ反映する
 */
 export declare function executeMcpServe(buildDependencies: () => McpServeDependencies): void;
 export declare function mcpServeDefaultDependencies(): McpServeDependencies;