MAS-076: isDuplicate_ 冪等性バグ修正

概要

問題

isDuplicate_() 関数は、RPA起票の冪等性を担保するために「同じ摘要のINVが既に存在するか」をチェックする。しかし、現在の実装では 有効フラグ=FALSE の行をスキップ しているため、ユーザーが誤起票されたINVを論理削除（有効フラグ=FALSE）した後にRPAを再実行すると、同一の摘要で二重登録が発生する。

現在のコード（401_bat_rpa.js L146-157）

function isDuplicate_(invData, headers, memoPattern) {
  const memoCol = headers.indexOf('摘要');
  const flagCol = headers.indexOf('有効フラグ');
  if (memoCol === -1) return false;
  for (let i = 1; i < invData.length; i++) {
    if (flagCol !== -1) {
      const flag = invData[i][flagCol];
      if (flag === false || String(flag).toUpperCase() === 'FALSE') continue; // ← 問題箇所
    }
    if (String(invData[i][memoCol]) === memoPattern) return true;
  }
  return false;
}

修正方針

有効フラグに関わらず、摘要が一致する行が存在すれば true を返すように変更する。つまり、有効フラグのフィルタリング（L151-153 の continue）を削除する。

影響範囲

• isDuplicate_() は6つのRPA関数（SaaS, HC, CAPEX, Finance, Adhoc, Pipeline）から合計25箇所で呼び出されている
• 修正は関数本体のみ。呼び出し側の変更は不要
• 既に 200_data/202_repository.js にも isDuplicate メソッドがあるが、こちらも同様のロジックのため合わせて修正する

関連ドキュメント

人間が検討すべき事項

なし（明確なバグ。即修正可）

実装プロンプト（Claude Code 用）

以下のプロンプトを Claude Code に入力して実装を実行する。

S-04: isDuplicate_ 冪等性バグ修正

背景: 401_bat_rpa.js の isDuplicate_() 関数（L146-158）は、RPA の二重実行を防ぐ冪等性ガードだが、有効フラグ=FALSE
の行をスキップするバグがある。ユーザーが INV を無効化（FALSE）した後に RPA を再実行すると、同一の摘要パターンを持つ INV
が再作成されてしまう。

修正内容:

1. isDuplicate_() の修正 (401_bat_rpa.js L145-158)
   - L151-153 の有効フラグフィルタを削除。全行（TRUE/FALSE問わず）を対象に摘要の重複チェックを行う
   - JSDoc コメントも「有効行のみ」→「全行（無効化済み含む）」に修正
2. テストの確認 (901_test_runner.js L1102-1149)
   - 既存の T7-01〜T7-06（冪等性テスト）が PASS することを確認
   - 可能であれば、「有効フラグ=FALSE にした INV がある状態で RPA 再実行→0件追加」のテストケースを T7 セクションに追加
3. 手順:
   - fix/s04-idempotency ブランチを作成
   - コード修正
   - npm run push:dev → dev で動作確認（RPA 実行 → INV 1件を FALSE → 再実行 → 追加されないこと）
   - テストランナー実行
   - git push → PR → main マージ

影響範囲: isDuplicate_() の呼び出し元は6つのRPA関数・計25箇所だが、関数シグネチャは変更しないため呼び出し元の修正は不要。

作成者: Claude Code

拡張思考（ultrathink）の使用状況

変更履歴