petari 0.9.1 → 0.10.0

package/README.md

@@ -94,7 +94,8 @@ slnmix の出力) が実ファイルとずれている**可能性が高いです
 
 主なオプション: `--dry-run` (検証と差分プレビューのみ) / `--partial` (成功分のみ適用) /
 `--root <dir>` / `--yes` / `--clip-report` (失敗レポートをクリップボードへ。v0.5.0 から既定で自動コピーされるため、config で無効化した場合の個別指定用) /
-`--no-vbproj` (create した .vb の .vbproj 自動登録を今回だけ無効化)
+`--no-vbproj` (create した .vb の .vbproj 自動登録を今回だけ無効化) /
+`--force` (適用済みの changes.md でも再適用する。下記「二重適用の防止」を今回だけ解除)
 
 ## 特徴
 
@@ -102,7 +103,11 @@ slnmix の出力) が実ファイルとずれている**可能性が高いです
 - **冪等 (再実行に安全)**: SEARCH が見つからなくても REPLACE の内容が既にファイルに
   存在すれば「適用済み」として成功扱いでスキップ。同じ changes.md を 2 回実行しても
   失敗せず、適用済み・未適用が混在していても未適用分だけを適用する。
+  SEARCH の前後に行を足す追記型の変更も、足す行が既にあれば「適用済み」になる。
   結果は `適用 / 済み / 失敗` の 3 状態で表示
+- **二重適用の防止**: 適用した changes.md の指紋を履歴に記録し、同じ changes.md を
+  もう一度渡して書き込みが発生する場合は何も書かずに止める (ファイル名・前置き・改行コードが
+  違っても同じ変更なら検出)。`petari undo` で戻したものは再適用でき、それ以外は `--force` で再適用
 - **エンコーディング保全**: Shift_JIS / UTF-8 (BOM 有無)・CRLF / LF・末尾改行を完全維持。
   変更していない行は元のバイト列をそのまま書き戻す (レガシー VB.NET 資産でも安全)
 - **規約逸脱への耐性 (寛容パース)**: AI がマーカーの記号数・見出しレベルを崩したり

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 { changeSetFingerprint } from "../core/fingerprint.js";
 import { PRESENCE_STAGE_LABEL } from "../core/matcher.js";
 import { fixedNewFile, resolveNewFileStyle } from "../core/new-file-style.js";
 import { parseChangesRecovering } from "../core/parser.js";
@@ -18,7 +19,7 @@ import { deleteFile, isInsideRoot, readFileState, sha256, writeBytes } from "../
 import { gitDirtyFiles } from "../infra/git.js";
 import { collectStyleSampleGroups } from "../infra/new-file-style.js";
 import { err, out, outEmphasis } from "../infra/term.js";
-import { beginHistory, createHistoryId, finishHistory, pruneHistory, } from "../infra/history.js";
+import { beginHistory, createHistoryId, findAppliedHistory, finishHistory, pruneHistory, } from "../infra/history.js";
 import { confirm } from "../infra/prompt.js";
 import { findProjectRoot } from "../infra/root.js";
 import { formatVbprojPlan, planVbprojRegistrations } from "./vbproj-register.js";
@@ -67,7 +68,8 @@ function printAlreadyApplied(outcomes) {
             continue;
         }
         for (const b of o.alreadyAppliedBlocks) {
-            out(`  replace ${o.change.path} — ブロック ${b.block.index}: REPLACE が既に存在 (${PRESENCE_STAGE_LABEL[b.stage]})`);
+            const what = b.insertion === true ? "追記する行が SEARCH の前後に既に存在" : "REPLACE が既に存在";
+            out(`  replace ${o.change.path} — ブロック ${b.block.index}: ${what} (${PRESENCE_STAGE_LABEL[b.stage]})`);
         }
     }
 }
@@ -83,6 +85,8 @@ export async function applyCommand(argv) {
             "clip-report": { type: "boolean", default: false },
             // create した .vb の .vbproj 自動登録を今回だけ無効化 (config vbproj.register の一時上書き)
             "no-vbproj": { type: "boolean", default: false },
+            // 履歴で適用済みの changes.md でも再適用する (二重適用防止の解除・§6.1)
+            force: { type: "boolean", default: false },
         },
         allowPositionals: true,
     });
@@ -190,6 +194,14 @@ export async function applyCommand(argv) {
             out(`  ${r.line} 行目: ${r.message}`);
         out("");
     }
+    // 同じ changes.md を以前に適用していないか (取り違え・二重実行の検出・§6.1)。
+    // 旧形式の履歴は保存済みの changes.md 原本をパースして指紋を算出する
+    const fingerprint = changeSetFingerprint(changeSet);
+    const previousId = findAppliedHistory(root, fingerprint, (text) => {
+        const r = parseChangesRecovering(stripBom(text));
+        return r.issues.length > 0 ? null : changeSetFingerprint(r.changeSet);
+    });
+    const previousNote = previousId !== null ? `この changes.md は適用済みです (履歴 ID: ${previousId})` : null;
     // 2. 全ブロックのドライラン検証
     const states = new Map();
     for (const f of changeSet.files) {
@@ -224,6 +236,8 @@ export async function applyCommand(argv) {
     // 3. 失敗があれば何も書き込まずレポート (§4.1, §7)
     if (!plan.ok && !values.partial) {
         err(`petari: 検証に失敗しました (${plan.failures.length} 件)。何も書き込んでいません。\n`);
+        if (previousNote !== null)
+            err(`注: ${previousNote}。取り違えていないか確認してください\n`);
         await emitReport(buildFailureReport(plan.failures, { outcomes: plan.outcomes, nothingWritten: true }));
         return 1;
     }
@@ -246,6 +260,8 @@ export async function applyCommand(argv) {
         // 冪等性: 全変更が適用済みなら成功として終了 (書き込み・履歴なし)
         if (plan.ok && plan.outcomes.some((o) => o.alreadyApplied)) {
             out("すべての変更は適用済みです。書き込みは行いませんでした。");
+            if (previousNote !== null)
+                out(`  (${previousNote})`);
             for (const o of plan.outcomes)
                 out(`  ${opLabel(o)}`);
             return 0;
@@ -254,6 +270,24 @@ export async function applyCommand(argv) {
         await emitReport(buildFailureReport(plan.failures, { outcomes: plan.outcomes, nothingWritten: true }));
         return 1;
     }
+    // 適用済みの changes.md で、なお書き込みが発生する = 二重適用のおそれ。--force なしでは止める
+    if (previousNote !== null) {
+        if (values.force) {
+            out(`--force: ${previousNote}。再適用します`);
+        }
+        else if (values["dry-run"]) {
+            out(`警告: ${previousNote}。実際の適用は --force を付けない限り止まります`);
+        }
+        else {
+            err(`petari: ${previousNote}。何も書き込んでいません。`);
+            err("  このまま適用すると次のファイルに書き込みが発生します (二重適用のおそれ):");
+            for (const o of applicable)
+                err(`    ${opLabel(o)}`);
+            err("  取り違えでなければ --force を付けて再実行してください");
+            err("  (petari undo で巻き戻した履歴は対象外です。git 等で手動で戻した場合は --force が必要です)");
+            return 1;
+        }
+    }
     if (values["dry-run"]) {
         out(`dry-run: 適用予定 ${applicable.length} ファイル (書き込みなし)`);
         printPreview(applicable);
@@ -368,6 +402,7 @@ export async function applyCommand(argv) {
         success: plan.ok,
         partial: values.partial,
         source,
+        changesFingerprint: fingerprint,
         files: entries,
     };
     finishHistory(root, id, manifest, after);

package/dist/commands/list.js

@@ -63,7 +63,8 @@ export async function listCommand(argv) {
             continue;
         }
         const appliedCount = m.files.filter((f) => f.applied).length;
-        const status = m.success ? "成功" : m.partial ? "部分適用" : "失敗";
+        const status = (m.success ? "成功" : m.partial ? "部分適用" : "失敗") +
+            (m.undoneAt !== undefined ? " (巻き戻し済み)" : "");
         const summary = summaryLine(dir);
         out(`${id}  ${status}  ${appliedCount} ファイル  ${summary}`);
     }

package/dist/commands/undo.js

@@ -10,7 +10,7 @@ import { parseArgs } from "node:util";
 import { invalidPathReason } from "../core/applier.js";
 import { loadConfig } from "../infra/config.js";
 import { deleteFile, isInsideRoot, readFileState, sha256, writeBytes } from "../infra/files.js";
-import { historyRoot, listHistoryIds, readManifest } from "../infra/history.js";
+import { historyRoot, listHistoryIds, markUndone, readManifest, } from "../infra/history.js";
 import { confirm } from "../infra/prompt.js";
 import { findProjectRoot } from "../infra/root.js";
 import { err, out } from "../infra/term.js";
@@ -103,5 +103,13 @@ export async function undoCommand(argv) {
         }
     }
     out(`巻き戻しました (履歴 ID: ${id})`);
+    // 巻き戻した changes.md は再適用できるよう、二重適用検出の対象から外す (§6.1)
+    try {
+        markUndone(root, id, new Date());
+    }
+    catch (e) {
+        err(`petari: 履歴 ${id} に巻き戻し済みの印を付けられませんでした (${e instanceof Error ? e.message : String(e)})。` +
+            "同じ changes.md を再適用するには --force が必要です");
+    }
     return 0;
 }

package/dist/core/applier.js

@@ -1,5 +1,5 @@
 import { EncodingError, decodeFile, encodeDocument, findUnencodable, makeDocument, } from "./encoding.js";
-import { STAGE_LABEL, contentEqualsStage, findContentStage, matchBlock, } from "./matcher.js";
+import { STAGE_LABEL, contentEqualsStage, findContentStage, findInsertionPresenceStage, matchBlock, } from "./matcher.js";
 import { analyzeNearest } from "./nearest.js";
 /** Windows の予約デバイス名 (拡張子付きも不可) */
 const WINDOWS_RESERVED = /^(con|prn|aux|nul|com[1-9]|lpt[1-9])(\..+)?$/i;
@@ -111,6 +111,13 @@ function planReplace(change, state, fallbackEol) {
             continue;
         }
         const replacement = m.replacement;
+        // 冪等性 (§6.1): 追記型ブロックは適用後も SEARCH が一致し続けるため、追記する行が
+        // 一致範囲の前後に既にあれば「適用済み」としてスキップする (再実行での重複追記を防ぐ)
+        const insertionStage = findInsertionPresenceStage(textLines, m.start, m.end, replacement === block.replace ? [replacement] : [replacement, block.replace]);
+        if (insertionStage !== null) {
+            already.push({ block, stage: insertionStage, insertion: true });
+            continue;
+        }
         const bad = findUnencodable(replacement.join("\n"), doc.encoding);
         if (bad.length > 0) {
             failures.push({

package/dist/core/fingerprint.js

@@ -0,0 +1,23 @@
+/**
+ * changes.md の指紋 (§5, §6.1)。同じ changes.md の再適用 (取り違え・二重実行) を
+ * 履歴と突き合わせて止めるために使う。
+ *
+ * パース後の変更内容 (パス・操作・ブロック/本文の行) だけから算出する。
+ * チャットの前置き・CHANGES 概要・改行コード・寛容パースで補正したマーカー表記の
+ * 違いは指紋に影響しない (同じ変更なら同じ指紋)。
+ */
+import { createHash } from "node:crypto";
+export function changeSetFingerprint(changeSet) {
+    const normalized = changeSet.files.map((f) => {
+        switch (f.op) {
+            case "replace":
+                return { path: f.path, op: f.op, blocks: f.blocks.map((b) => [b.search, b.replace]) };
+            case "create":
+            case "rewrite":
+                return { path: f.path, op: f.op, content: f.content };
+            case "delete":
+                return { path: f.path, op: f.op };
+        }
+    });
+    return createHash("sha256").update(JSON.stringify(normalized), "utf8").digest("hex");
+}

package/dist/core/matcher.js

@@ -32,6 +32,63 @@ export function findContentStage(lines, content) {
     }
     return null;
 }
+/**
+ * 追記型ブロックの適用済み判定 (§6.1)。SEARCH が [start, end) に一意に一致したうえで、
+ * 挿入しようとしている行 (candidates のいずれか) が「一致範囲を内側に含む形で」既に
+ * ファイルにあるかを調べる。REPLACE が SEARCH を包含する追記型 (SEARCH `A` → REPLACE
+ * `A` + `B`) は適用後も SEARCH が一致し続けるため、SEARCH 不一致を起点とする
+ * findContentStage では拾えず、再実行で `B` が重複していた。
+ *
+ * - 比較は空行を除いた行列で行う (空行の欠落・追加だけの差は同一視)。空行まで含めて
+ *   一致すれば exact / trim-end、空行の並びだけ異なれば blank-insensitive を返す
+ * - 各行の比較は exact → trim-end のみ。インデント違いは別物として扱う
+ *   (入れ子の End If を補う変更を、隣の外側の End If を根拠に「済み」と誤判定しないため)
+ * - 一致範囲の外に内容のある行を 1 行以上足すものだけが対象。SEARCH と同数以下
+ *   (インデント修正などの書き換え) は null
+ *
+ * candidates には reindent 済みの置換行と REPLACE の原文を渡す (前回 trim-all で一致して
+ * 補正後の行が入っている場合と、原文のまま入っている場合の両方を拾うため)。
+ */
+export function findInsertionPresenceStage(lines, start, end, candidates) {
+    const nonBlank = [];
+    lines.forEach((line, i) => {
+        if (line.trim() !== "")
+            nonBlank.push(i);
+    });
+    const first = nonBlank.findIndex((i) => i >= start);
+    const matched = nonBlank.filter((i) => i >= start && i < end).length;
+    if (first < 0 || matched === 0)
+        return null;
+    for (const { stage, eq } of STAGES.slice(0, 2)) {
+        for (const rep of candidates) {
+            const repIdx = [];
+            rep.forEach((line, i) => {
+                if (line.trim() !== "")
+                    repIdx.push(i);
+            });
+            if (repIdx.length <= matched)
+                continue;
+            // q = REPLACE 内で一致範囲 (の最初の非空行) が始まる位置 (非空行単位)
+            for (let q = 0; q + matched <= repIdx.length; q++) {
+                const from = first - q;
+                if (from < 0 || from + repIdx.length > nonBlank.length)
+                    continue;
+                const all = repIdx.every((r, j) => eq(lines[nonBlank[from + j]], rep[r]));
+                if (!all)
+                    continue;
+                // 空行の並びまで同じか (ラベル用)
+                const lo = nonBlank[from];
+                const hi = nonBlank[from + repIdx.length - 1] + 1;
+                const repLo = repIdx[0];
+                const repHi = repIdx[repIdx.length - 1] + 1;
+                const sameLayout = hi - lo === repHi - repLo &&
+                    lines.slice(lo, hi).every((line, k) => eq(line, rep[repLo + k]));
+                return sameLayout ? stage : "blank-insensitive";
+            }
+        }
+    }
+    return null;
+}
 /** 適用済み判定 (rewrite/create): content がファイル全行と一致するか */
 export function contentEqualsStage(lines, content) {
     if (lines.length !== content.length)

package/dist/core/report.js

@@ -50,7 +50,9 @@ function outcomeSummary(outcomes) {
             byIndex.set(b.block.index, `OK 一致 (${STAGE_LABEL[b.stage]})`);
         }
         for (const b of o.alreadyAppliedBlocks) {
-            byIndex.set(b.block.index, "OK 適用済み (REPLACE が既に存在)");
+            byIndex.set(b.block.index, b.insertion === true
+                ? "OK 適用済み (追記する行が SEARCH の前後に既に存在)"
+                : "OK 適用済み (REPLACE が既に存在)");
         }
         for (const f of o.failures) {
             if (f.block === undefined)

package/dist/infra/history.js

@@ -69,4 +69,50 @@ export function readManifest(root, id) {
         return null;
     return JSON.parse(readFileSync(p, "utf8"));
 }
+/** undo の完了を manifest に記録する (再適用検出で「巻き戻し済み」を区別するため) */
+export function markUndone(root, id, at) {
+    const p = join(historyRoot(root), id, "manifest.json");
+    const manifest = JSON.parse(readFileSync(p, "utf8"));
+    manifest.undoneAt = at.toISOString();
+    writeFileSync(p, JSON.stringify(manifest, null, 2) + "\n", "utf8");
+}
+/**
+ * 同じ指紋の changes.md を適用した履歴 (巻き戻し済み・書き込みなしを除く) を新しい順に
+ * 探し、最初に見つかった履歴 ID を返す。指紋を持たない旧形式の manifest は、保存済みの
+ * changes.md 原本から fallback で算出する (null = 算出不能として対象外)。
+ * manifest は非信頼入力のため、読めない・形が違うものは黙って対象外にする。
+ */
+export function findAppliedHistory(root, fingerprint, fallback) {
+    for (const id of listHistoryIds(root).reverse()) {
+        let manifest;
+        try {
+            manifest = readManifest(root, id);
+        }
+        catch {
+            continue;
+        }
+        if (manifest === null || typeof manifest !== "object")
+            continue;
+        if (manifest.undoneAt !== undefined)
+            continue;
+        if (!Array.isArray(manifest.files) || !manifest.files.some((f) => f?.applied === true))
+            continue;
+        let fp = null;
+        if (typeof manifest.changesFingerprint === "string") {
+            fp = manifest.changesFingerprint;
+        }
+        else {
+            const p = join(historyRoot(root), id, "changes.md");
+            try {
+                fp = fallback(readFileSync(p, "utf8"));
+            }
+            catch {
+                fp = null;
+            }
+        }
+        if (fp === fingerprint)
+            return id;
+    }
+    return null;
+}
 export { sha256 };

package/package.json

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