i-repo 2.2.0 → 2.3.0

package/dist/bin/irepo.js

@@ -285,9 +285,12 @@ if (isMain) {
     // パイプ下流が早く閉じると stdout への書き込みが EPIPE を投げる
     // (`i-repo rep list | head` など)。握りつぶさないと Node が生の
     // スタックトレースを吐く — ndjson パイプを売りにする CLI で最も目立つ
-    // 行儀の悪さなので、EPIPE のみ静かに終了する (他のエラーは握りつぶさない)。
+    // 行儀の悪さなので、パイプ切断系のみ静かに終了する。
+    // Windows の閉じパイプは EPIPE とは限らず EOF / ERR_STREAM_DESTROYED を
+    // 出すため、それらも同じ「下流が消えた」扱いにする (他のエラーは握りつぶさない)。
+    const PIPE_CLOSED = new Set(["EPIPE", "EOF", "ERR_STREAM_DESTROYED"]);
     const onStreamError = (err) => {
-        if (err.code === "EPIPE")
+        if (PIPE_CLOSED.has(err.code ?? ""))
             process.exit(0);
         throw err;
     };
@@ -312,6 +315,16 @@ if (isMain) {
     // ループが終わらずハングしうる。ハンドラを登録した時点で Node 既定の
     // 即時終了を奪っているので、整形後に明示的に終了させて非ゼロを保証する。
     const onFatal = (error) => {
+        // Ink 描画中の致命例外でもここに来て process.exit するため、Ink の
+        // クリーンアップ (raw mode 解除) を経ずに即死しうる。Windows conhost は
+        // raw mode のまま終了するとエコー不能になるので、落とす前に端末を素に戻す。
+        try {
+            if (process.stdin.isTTY)
+                process.stdin.setRawMode(false);
+        }
+        catch {
+            /* 既に detach 済み等は無視 */
+        }
         process.exit(printFatal(error));
     };
     process.on("uncaughtException", onFatal);

package/dist/commands/raw.js

@@ -9,6 +9,7 @@ import { readFile, writeFile } from "node:fs/promises";
 import { basename } from "node:path";
 import { createClient } from "../config/client.js";
 import { handleError, ValidationError } from "../core/errors.js";
+import { assertSafeOutputPath } from "../utils/file.js";
 import { withSpinner, out } from "../ui/index.js";
 export const rawCommand = new Command("raw")
     .description("Execute raw API command (returns XML)")
@@ -29,6 +30,11 @@ export const rawCommand = new Command("raw")
             }
             parsedParams[param.substring(0, eqIdx)] = param.substring(eqIdx + 1);
         }
+        // 出力先の検証は API 送信より前に行う。raw は状態変更コマンド
+        // (作成/更新/削除) も実行しうるため、送信後に保存パスで弾くと
+        // 「実行済みなのに検証エラー → 直して再実行で二重実行」になる。
+        if (opts.output)
+            assertSafeOutputPath(opts.output);
         const client = await createClient(globalOpts);
         let xml;
         if (opts.file) {
@@ -49,6 +55,7 @@ export const rawCommand = new Command("raw")
             });
         }
         if (opts.output) {
+            // 検証は送信前に済ませてある (上記)
             await writeFile(opts.output, xml, "utf-8");
             if (!globalOpts.quiet) {
                 console.error(`Saved to ${opts.output}`);

package/dist/config/scoped-store.js

@@ -101,6 +101,28 @@ function readJsonFile(filePath) {
     }
     return data;
 }
+/**
+ * renameSync を EPERM/EBUSY/EACCES に短い同期バックオフでリトライする。
+ * Windows ではアンチウイルス/インデクサが書込直後のファイルを一瞬開くため
+ * temp+rename の rename が散発的に共有違反で失敗する (config set がたまにコケる)。
+ * 同期コンテキストなので Atomics.wait で待つ。POSIX では通常1回で成功。
+ */
+function renameSyncWithRetry(from, to) {
+    const transient = new Set(["EPERM", "EBUSY", "EACCES"]);
+    const sab = new Int32Array(new SharedArrayBuffer(4));
+    for (let attempt = 0;; attempt++) {
+        try {
+            renameSync(from, to);
+            return;
+        }
+        catch (err) {
+            const code = err.code ?? "";
+            if (attempt >= 9 || !transient.has(code))
+                throw err;
+            Atomics.wait(sab, 0, 0, 10 + attempt * 15);
+        }
+    }
+}
 function writeJsonFile(filePath, data) {
     const dir = dirname(filePath);
     if (!existsSync(dir)) {
@@ -124,7 +146,7 @@ function writeJsonFile(filePath, data) {
         catch {
             /* 権限是正の失敗で書込自体を失敗扱いにはしない */
         }
-        renameSync(tmpPath, filePath);
+        renameSyncWithRetry(tmpPath, filePath);
     }
     catch (err) {
         // rename 前に失敗したら中途半端な tmp を残さない

package/dist/ui/formatters/writer.js

@@ -16,6 +16,7 @@
 import { writeFileSync, appendFileSync } from "node:fs";
 import { resolve } from "node:path";
 import { stripAnsi } from "../width.js";
+import { assertSafeOutputPath } from "../../utils/file.js";
 const BOM = "\uFEFF";
 let outputFile;
 let wroteAnything = false;
@@ -32,6 +33,7 @@ export function setOutputFile(path, opts) {
     outputFile = path ? resolve(path) : undefined;
     wroteAnything = false;
     if (outputFile) {
+        assertSafeOutputPath(outputFile); // Windows 予約デバイス名への沈黙保存を防ぐ
         writeFileSync(outputFile, opts?.bom ? BOM : "", "utf-8");
     }
 }

package/dist/utils/file.d.ts

@@ -1,3 +1,9 @@
+/**
+ * ローカル保存パスが Windows 予約デバイス名でないか検証する (送信前/書込前に呼ぶ)。
+ * Windows は予約名を**末尾ファイル名だけでなく親ディレクトリ成分としても**拒否する
+ * (CON\response.xml も無効) ため、`/` `\` どちらの区切りも分割して全成分を検査する。
+ */
+export declare function assertSafeOutputPath(outputPath: string): void;
 export interface DownloadResult {
     /** Saved file path (absolute) */
     path: string;

package/dist/utils/file.js

@@ -6,6 +6,70 @@
 import { writeFile, mkdir, rename, unlink } from "node:fs/promises";
 import { dirname, resolve, basename } from "node:path";
 import { t, icons } from "../ui/theme.js";
+import { ValidationError } from "../core/errors.js";
+/**
+ * Windows の予約 DOS デバイス名。これらに書き込むとファイルにならずデバイスへ
+ * 消える/ハングする — 「保存した」表示なのに実体が無い沈黙の失敗になる。
+ * MS の RtlIsDosDeviceName_U が認識する名前に揃える: CON/PRN/AUX/NUL、
+ * COM1-9/LPT1-9、CONIN$/CONOUT$、CLOCK$。
+ */
+const WIN_RESERVED_NAMES = new Set([
+    "CON", "PRN", "AUX", "NUL", "CLOCK$", "CONIN$", "CONOUT$",
+]);
+for (let i = 1; i <= 9; i++) {
+    WIN_RESERVED_NAMES.add(`COM${i}`);
+    WIN_RESERVED_NAMES.add(`LPT${i}`);
+}
+/**
+ * basename が Windows 予約デバイス名に解決するか。Windows は以下を同じデバイスと
+ * 見なす: 拡張子付き (NUL.txt)・末尾コロン (NUL:)・末尾のドット/空白 ("NUL " /
+ * "NUL.") は無視される。大文字小文字も無視。
+ */
+function isWindowsReservedName(base) {
+    // 末尾の空白・ドット、先頭の空白を除去 (Windows のパス正規化が落とす)
+    let s = base.replace(/^[ ]+/, "").replace(/[ .]+$/, "");
+    // ドライブ相対パスの接頭辞を剥がす: `C:NUL` は drive C のカレント相対 `NUL`
+    // (= デバイス) を指す。これを残すと先頭の `C` を device 候補と誤判定する。
+    s = s.replace(/^[A-Za-z]:/, "");
+    // 拡張子・(ADS等の)コロンより前の device 名候補を取り出す (NUL.txt / NUL: → NUL)
+    const candidate = s.split(/[.:]/, 1)[0].toUpperCase();
+    return WIN_RESERVED_NAMES.has(candidate);
+}
+/**
+ * ローカル保存パスが Windows 予約デバイス名でないか検証する (送信前/書込前に呼ぶ)。
+ * Windows は予約名を**末尾ファイル名だけでなく親ディレクトリ成分としても**拒否する
+ * (CON\response.xml も無効) ため、`/` `\` どちらの区切りも分割して全成分を検査する。
+ */
+export function assertSafeOutputPath(outputPath) {
+    if (process.platform !== "win32")
+        return;
+    for (const seg of outputPath.split(/[\\/]/)) {
+        if (seg && isWindowsReservedName(seg)) {
+            throw new ValidationError(`Invalid output path: "${outputPath}" — "${seg}" is a reserved Windows device name (the write would fail or vanish). Choose another name.`);
+        }
+    }
+}
+/**
+ * rename を EPERM/EBUSY/EACCES に対して短い指数バックオフでリトライする。
+ * Windows では Defender 等のアンチウイルス/インデクサが書込直後の tmp や
+ * ターゲットを一瞬開くため、temp+rename の rename が散発的に共有違反で失敗する
+ * (npm/esbuild 等も同種のリトライを実装)。POSIX では通常1回で成功する。
+ */
+async function renameWithRetry(from, to) {
+    const transient = new Set(["EPERM", "EBUSY", "EACCES"]);
+    for (let attempt = 0;; attempt++) {
+        try {
+            await rename(from, to);
+            return;
+        }
+        catch (err) {
+            const code = err.code ?? "";
+            if (attempt >= 9 || !transient.has(code))
+                throw err;
+            await new Promise((r) => setTimeout(r, 10 + attempt * 15));
+        }
+    }
+}
 /**
  * Save binary data from an SDK download to a local file.
  *
@@ -21,6 +85,7 @@ import { t, icons } from "../ui/theme.js";
  */
 export async function saveBinaryDownload(result, outputPath, defaultFilename, quiet = false) {
     const resolvedPath = resolve(outputPath ?? defaultFilename);
+    assertSafeOutputPath(resolvedPath); // Windows 予約デバイス名への沈黙保存を防ぐ
     // Ensure parent directory exists
     await mkdir(dirname(resolvedPath), { recursive: true });
     const buffer = Buffer.from(result.data);
@@ -30,7 +95,7 @@ export async function saveBinaryDownload(result, outputPath, defaultFilename, qu
     const tmpPath = `${resolvedPath}.${process.pid}.part`;
     try {
         await writeFile(tmpPath, buffer);
-        await rename(tmpPath, resolvedPath);
+        await renameWithRetry(tmpPath, resolvedPath);
     }
     catch (err) {
         await unlink(tmpPath).catch(() => { });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "i-repo",
-  "version": "2.2.0",
+  "version": "2.3.0",
   "description": "Modern CLI for ConMas i-Reporter - Built for humans and AI",
   "type": "module",
   "bin": {