petari 0.2.0 → 0.3.0

package/README.md

@@ -52,7 +52,7 @@ petari の更新で規約文が変わった場合は、`petari init` を再実
 
 | コマンド | 説明 |
 |---|---|
-| `petari [path]` | changes.md を適用。省略時は Downloads から自動検出 (直近 30 分)。`--clip` でクリップボードから |
+| `petari [path]` | changes.md を適用。省略時は Downloads とプロジェクト直下から自動検出 (直近 30 分)。`--clip` でクリップボードから |
 | `petari undo [ID]` | 履歴を巻き戻す (手修正がある場合は警告) |
 | `petari show <ID>` | VS Code の差分ビューで履歴を表示。`--mine` で適用後の手修正分、`--changes` で変更概要 |
 | `petari list` | 履歴の一覧と合計サイズ |
@@ -65,6 +65,10 @@ petari の更新で規約文が変わった場合は、`petari init` を再実
 ## 特徴
 
 - **all-or-nothing**: 全ブロックを事前検証し、1 つでも失敗があれば何も書き込まない
+- **冪等 (再実行に安全)**: SEARCH が見つからなくても REPLACE の内容が既にファイルに
+  存在すれば「適用済み」として成功扱いでスキップ。同じ changes.md を 2 回実行しても
+  失敗せず、適用済み・未適用が混在していても未適用分だけを適用する。
+  結果は `適用 / 済み / 失敗` の 3 状態で表示
 - **エンコーディング保全**: Shift_JIS / UTF-8 (BOM 有無)・CRLF / LF・末尾改行を完全維持。
   変更していない行は元のバイト列をそのまま書き戻す (レガシー VB.NET 資産でも安全)
 - **失敗レポート**: そのまま AI チャットに貼り返せる形式で出力。再依頼が 1 コピペで済む

package/dist/commands/apply.js

@@ -7,6 +7,7 @@ import { existsSync, readFileSync } from "node:fs";
 import { join, resolve } from "node:path";
 import { parseArgs } from "node:util";
 import { invalidPathReason, planChangeSet, } from "../core/applier.js";
+import { PRESENCE_STAGE_LABEL } from "../core/matcher.js";
 import { parseChanges } from "../core/parser.js";
 import { buildFailureReport, buildParseErrorReport } from "../core/report.js";
 import { readClipboard, writeClipboard } from "../infra/clipboard.js";
@@ -18,17 +19,25 @@ import { err, out } from "../infra/term.js";
 import { beginHistory, createHistoryId, finishHistory, pruneHistory, } from "../infra/history.js";
 import { confirm } from "../infra/prompt.js";
 import { findProjectRoot } from "../infra/root.js";
-/** --partial 時: このファイルは書き込み対象か */
+/** このファイルは書き込み対象か (適用済みのみのファイルは書き込み不要なので含めない) */
 function isApplicable(o) {
-    if (o.failures.length === 0)
-        return true;
-    return o.change.op === "replace" && o.appliedBlocks.length > 0;
+    if (o.alreadyApplied)
+        return false;
+    if (o.change.op === "replace")
+        return o.appliedBlocks.length > 0;
+    return o.failures.length === 0;
 }
 function opLabel(o) {
     const c = o.change;
-    if (c.op === "replace")
-        return `replace ${c.path} — ${o.appliedBlocks.length}/${o.totalBlocks} ブロック`;
-    return `${c.op.padEnd(7)} ${c.path}`;
+    if (c.op === "replace") {
+        const already = o.alreadyAppliedBlocks.length;
+        const detail = already > 0
+            ? `適用 ${o.appliedBlocks.length} / 済み ${already} / 全 ${o.totalBlocks} ブロック`
+            : `${o.appliedBlocks.length}/${o.totalBlocks} ブロック`;
+        return `replace ${c.path} — ${detail}`;
+    }
+    const label = `${c.op.padEnd(7)} ${c.path}`;
+    return o.alreadyApplied ? `${label} — 適用済み (スキップ)` : label;
 }
 function printPreview(outcomes) {
     for (const o of outcomes) {
@@ -42,6 +51,23 @@ function printPreview(outcomes) {
         }
     }
 }
+/** 適用済みとしてスキップした内容の一覧 (冪等性)。該当がなければ何も出さない */
+function printAlreadyApplied(outcomes) {
+    const files = outcomes.filter((o) => o.alreadyApplied || o.alreadyAppliedBlocks.length > 0);
+    if (files.length === 0)
+        return;
+    out("適用済みのためスキップ:");
+    for (const o of files) {
+        if (o.change.op !== "replace") {
+            const why = o.change.op === "delete" ? "既に存在しません" : "現在の内容と一致";
+            out(`  ${o.change.op.padEnd(7)} ${o.change.path} — ${why}`);
+            continue;
+        }
+        for (const b of o.alreadyAppliedBlocks) {
+            out(`  replace ${o.change.path} — ブロック ${b.block.index}: REPLACE が既に存在 (${PRESENCE_STAGE_LABEL[b.stage]})`);
+        }
+    }
+}
 export async function applyCommand(argv) {
     const { values, positionals } = parseArgs({
         args: argv,
@@ -104,15 +130,24 @@ export async function applyCommand(argv) {
         source = { type: "clipboard" };
     }
     else {
+        // Downloads とプロジェクトルート直下の両方から候補を集め、最新を採用する
         const downloadsDir = await resolveDownloadsDir(config.downloadsDir);
-        const candidates = findRecentChangesFiles(downloadsDir, new Date());
+        const now = new Date();
+        const tagged = [
+            ...findRecentChangesFiles(downloadsDir, now).map((c) => ({ ...c, origin: "downloads" })),
+            ...findRecentChangesFiles(root, now).map((c) => ({ ...c, origin: "project-root" })),
+        ];
+        const seen = new Set();
+        const candidates = tagged
+            .filter((c) => !seen.has(c.path) && (seen.add(c.path), true))
+            .sort((a, b) => b.mtime.getTime() - a.mtime.getTime());
         if (candidates.length === 0) {
-            err(`petari: ${downloadsDir} に直近 30 分以内の changes*.md が見つかりません`);
+            err(`petari: ${downloadsDir} にもプロジェクト直下にも直近 30 分以内の changes*.md が見つかりません`);
             err("  ファイルパスを指定するか、--clip でクリップボードから読み込んでください");
             return 1;
         }
         const newest = candidates[0];
-        out(`Downloads から検出: ${newest.path}`);
+        out(`${newest.origin === "downloads" ? "Downloads" : "プロジェクト直下"}から検出: ${newest.path}`);
         if (candidates.length > 1) {
             out(`  (他 ${candidates.length - 1} 件の候補があります。最新を使用します)`);
             for (const c of candidates.slice(1))
@@ -129,7 +164,7 @@ export async function applyCommand(argv) {
             err(`petari: ファイルを読めません: ${newest.path}`);
             return 1;
         }
-        source = { type: "downloads", path: newest.path };
+        source = { type: newest.origin, path: newest.path };
     }
     // 1. パース (構文エラーは即時失敗・§4.1)
     const { changeSet, issues } = parseChanges(changesText);
@@ -162,6 +197,13 @@ export async function applyCommand(argv) {
     }
     const applicable = plan.outcomes.filter(isApplicable);
     if (applicable.length === 0) {
+        // 冪等性: 全変更が適用済みなら成功として終了 (書き込み・履歴なし)
+        if (plan.ok && plan.outcomes.some((o) => o.alreadyApplied)) {
+            out("すべての変更は適用済みです。書き込みは行いませんでした。");
+            for (const o of plan.outcomes)
+                out(`  ${opLabel(o)}`);
+            return 0;
+        }
         err("petari: 適用できる変更がありません。\n");
         await emitReport(buildFailureReport(plan.failures));
         return 1;
@@ -169,6 +211,7 @@ export async function applyCommand(argv) {
     if (values["dry-run"]) {
         out(`dry-run: 適用予定 ${applicable.length} ファイル (書き込みなし)`);
         printPreview(applicable);
+        printAlreadyApplied(plan.outcomes);
         if (plan.failures.length > 0) {
             out("");
             await emitReport(buildFailureReport(plan.failures));
@@ -180,6 +223,7 @@ export async function applyCommand(argv) {
     out(`適用予定 ${applicable.length} ファイル:`);
     for (const o of applicable)
         out(`  ${opLabel(o)}`);
+    printAlreadyApplied(plan.outcomes);
     if (plan.failures.length > 0) {
         out(`  (検証失敗 ${plan.failures.length} 件は --partial によりスキップ)`);
     }
@@ -230,6 +274,11 @@ export async function applyCommand(argv) {
         };
         if (skipped.length > 0)
             entry.skippedBlocks = skipped;
+        if (o.alreadyAppliedBlocks.length > 0) {
+            entry.alreadyAppliedBlocks = o.alreadyAppliedBlocks.map((b) => b.block.index);
+        }
+        if (o.alreadyApplied)
+            entry.alreadyApplied = true;
         return entry;
     });
     const manifest = {
@@ -250,16 +299,18 @@ export async function applyCommand(argv) {
     out(`適用しました (履歴 ID: ${id})`);
     for (const o of applicable)
         out(`  ${opLabel(o)}`);
+    printAlreadyApplied(plan.outcomes);
     if (plan.failures.length > 0) {
         out(`スキップした失敗 ${plan.failures.length} 件のレポート:`);
         out("");
         await emitReport(buildFailureReport(plan.failures));
     }
-    // 6. Downloads 由来なら changes.md を履歴へ移動 (原本は history に保存済み・§4.1)
-    if (source.type === "downloads" && source.path !== undefined) {
+    // 6. 自動検出 (Downloads / プロジェクト直下) 由来なら changes.md を履歴へ移動
+    //    (原本は history に保存済み・§4.1)。全件適用済みで履歴を作らなかった場合は削除しない
+    if ((source.type === "downloads" || source.type === "project-root") && source.path !== undefined) {
         try {
             deleteFile(source.path);
-            out(`Downloads の ${source.path} を履歴へ移動しました`);
+            out(`${source.path} を履歴へ移動しました`);
         }
         catch {
             err(`petari: ${source.path} を削除できませんでした (履歴には保存済み)`);

package/dist/core/applier.js

@@ -1,5 +1,5 @@
 import { EncodingError, decodeFile, encodeDocument, findUnencodable, makeDocument, } from "./encoding.js";
-import { STAGE_LABEL, matchBlock } from "./matcher.js";
+import { STAGE_LABEL, contentEqualsStage, findContentStage, matchBlock, } from "./matcher.js";
 /** Windows の予約デバイス名 (拡張子付きも不可) */
 const WINDOWS_RESERVED = /^(con|prn|aux|nul|com[1-9]|lpt[1-9])(\..+)?$/i;
 /** §9: プロジェクトルート相対のみ。絶対パス・`..`・`~`・Windows 特殊パスを拒否 */
@@ -28,9 +28,23 @@ function fileFailure(change, kind, message) {
         failures: [{ path: change.path, kind, message }],
         afterBytes: null,
         appliedBlocks: [],
+        alreadyAppliedBlocks: [],
+        alreadyApplied: false,
         totalBlocks: change.op === "replace" ? change.blocks.length : 0,
     };
 }
+/** rewrite / create / delete がファイル単位で適用済みだったときの成功 outcome */
+function fileAlreadyApplied(change) {
+    return {
+        change,
+        failures: [],
+        afterBytes: null,
+        appliedBlocks: [],
+        alreadyAppliedBlocks: [],
+        alreadyApplied: true,
+        totalBlocks: 0,
+    };
+}
 function planReplace(change, state, fallbackEol) {
     let doc;
     try {
@@ -41,14 +55,28 @@ function planReplace(change, state, fallbackEol) {
     }
     const failures = [];
     const applied = [];
+    const already = [];
     let docLines = doc.lines;
     let textLines = docLines.map((l) => l.text);
     for (const block of change.blocks) {
         const m = matchBlock(textLines, block);
         if (!m.ok) {
+            // 冪等性 (§6): SEARCH 不一致でも REPLACE がまるごと存在すれば「適用済み」として
+            // 成功扱いでスキップする (書き込みなし)。空・空行のみの REPLACE は判定対象外
+            if (m.reason === "not-found") {
+                const stage = findContentStage(textLines, block.replace);
+                if (stage !== null) {
+                    already.push({ block, stage });
+                    continue;
+                }
+            }
+            const checkedReplace = block.replace.some((l) => l.trim() !== "");
             const message = m.reason === "ambiguous"
                 ? `ブロック ${block.index}: SEARCH が ${m.count} 箇所にマッチしました (${STAGE_LABEL[m.stage]})。一意に特定できる範囲を含めてください`
-                : `ブロック ${block.index}: SEARCH が現在のファイル内容に見つかりません`;
+                : `ブロック ${block.index}: SEARCH が現在のファイル内容に見つかりません` +
+                    (checkedReplace
+                        ? " (REPLACE の内容も見つからないため、changes.md の基準スナップショットが現在のコードベースとずれている可能性があります)"
+                        : "");
             failures.push({ path: change.path, kind: m.reason === "ambiguous" ? "block-ambiguous" : "block-not-found", message, block });
             continue;
         }
@@ -80,6 +108,8 @@ function planReplace(change, state, fallbackEol) {
         failures,
         afterBytes,
         appliedBlocks: applied,
+        alreadyAppliedBlocks: already,
+        alreadyApplied: change.blocks.length > 0 && failures.length === 0 && already.length === change.blocks.length,
         totalBlocks: change.blocks.length,
     };
 }
@@ -96,6 +126,18 @@ function planFile(change, state, newFile) {
     switch (change.op) {
         case "create": {
             if (state.exists) {
+                // 冪等性: 既存ファイルの内容が CONTENT と一致するなら適用済みとしてスキップ
+                if (state.bytes !== null) {
+                    try {
+                        const doc = decodeFile(state.bytes);
+                        if (contentEqualsStage(doc.lines.map((l) => l.text), change.content) !== null) {
+                            return fileAlreadyApplied(change);
+                        }
+                    }
+                    catch {
+                        // デコード不能なら判定せず従来のエラーへ
+                    }
+                }
                 return fileFailure(change, "target-exists", "create 指定ですがファイルが既に存在します (全文置き換えなら rewrite を使用)");
             }
             const bad = findUnencodable(change.content.join("\n"), newFile.encoding);
@@ -103,7 +145,15 @@ function planFile(change, state, newFile) {
                 return fileFailure(change, "unencodable", `新規ファイルの既定エンコーディング (${newFile.encoding}) に変換できない文字: ${bad.join(" ")}`);
             }
             const doc = makeDocument(change.content, newFile);
-            return { change, failures: [], afterBytes: encodeDocument(doc), appliedBlocks: [], totalBlocks: 0 };
+            return {
+                change,
+                failures: [],
+                afterBytes: encodeDocument(doc),
+                appliedBlocks: [],
+                alreadyAppliedBlocks: [],
+                alreadyApplied: false,
+                totalBlocks: 0,
+            };
         }
         case "rewrite": {
             if (!state.exists) {
@@ -116,6 +166,10 @@ function planFile(change, state, newFile) {
             catch (e) {
                 return fileFailure(change, "undecodable", e instanceof EncodingError ? e.message : String(e));
             }
+            // 冪等性: 全文が現在の内容と一致するなら適用済みとしてスキップ
+            if (contentEqualsStage(doc.lines.map((l) => l.text), change.content) !== null) {
+                return fileAlreadyApplied(change);
+            }
             const bad = findUnencodable(change.content.join("\n"), doc.encoding);
             if (bad.length > 0) {
                 return fileFailure(change, "unencodable", `Shift_JIS へ変換できない文字が含まれています: ${bad.join(" ")}`);
@@ -123,13 +177,30 @@ function planFile(change, state, newFile) {
             // 既存のエンコーディング・BOM・支配的 EOL・末尾改行の有無を維持して全文を差し替える (§8)
             const lines = change.content.map((text) => ({ text, raw: null, eol: null }));
             const afterBytes = encodeDocument({ ...doc, lines });
-            return { change, failures: [], afterBytes, appliedBlocks: [], totalBlocks: 0 };
+            return {
+                change,
+                failures: [],
+                afterBytes,
+                appliedBlocks: [],
+                alreadyAppliedBlocks: [],
+                alreadyApplied: false,
+                totalBlocks: 0,
+            };
         }
         case "delete": {
             if (!state.exists) {
-                return fileFailure(change, "target-missing", "delete 指定ですがファイルが存在しません");
+                // 冪等性: 既に存在しないなら適用済みとしてスキップ
+                return fileAlreadyApplied(change);
             }
-            return { change, failures: [], afterBytes: null, appliedBlocks: [], totalBlocks: 0 };
+            return {
+                change,
+                failures: [],
+                afterBytes: null,
+                appliedBlocks: [],
+                alreadyAppliedBlocks: [],
+                alreadyApplied: false,
+                totalBlocks: 0,
+            };
         }
         case "replace": {
             if (!state.exists) {

package/dist/core/matcher.js

@@ -8,6 +8,39 @@ const STAGES = [
     { stage: "trim-end", eq: (a, b) => a.trimEnd() === b.trimEnd() },
     { stage: "trim-all", eq: (a, b) => a.trim() === b.trim() },
 ];
+export const PRESENCE_STAGE_LABEL = {
+    ...STAGE_LABEL,
+    "ws-collapse": "空白圧縮一致",
+};
+const collapseWs = (s) => s.trim().replace(/[ \t]+/g, " ");
+const PRESENCE_STAGES = [
+    ...STAGES,
+    { stage: "ws-collapse", eq: (a, b) => collapseWs(a) === collapseWs(b) },
+];
+/**
+ * 適用済み判定: content ブロック全体 (複数行まるごと) が lines 内に存在するか。
+ * 見つかった最初の (最も厳密な) 段階を返す。空・空行のみの content は偶然一致
+ * しやすいため常に null (判定対象外 = 従来どおり失敗扱い)。
+ */
+export function findContentStage(lines, content) {
+    if (!content.some((l) => l.trim() !== ""))
+        return null;
+    for (const { stage, eq } of PRESENCE_STAGES) {
+        if (findMatches(lines, content, eq).length > 0)
+            return stage;
+    }
+    return null;
+}
+/** 適用済み判定 (rewrite/create): content がファイル全行と一致するか */
+export function contentEqualsStage(lines, content) {
+    if (lines.length !== content.length)
+        return null;
+    for (const { stage, eq } of PRESENCE_STAGES) {
+        if (lines.every((line, i) => eq(line, content[i])))
+            return stage;
+    }
+    return null;
+}
 function findMatches(lines, search, eq) {
     const found = [];
     for (let i = 0; i + search.length <= lines.length; i++) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "petari",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "petari — paste AI chat patches onto your codebase",
   "keywords": [
     "cli",