# 一括処理が「全記事」に当たった：書く前に数える、テストは壊して確かめる > 記事の状態をまとめて直す処理が、全記事に当たり、18件が誤って表示に戻った。履歴から戻した経緯と、テストが効くかを、規則を1つずつ外して確かめた記録。 - URL: https://koshinstudio.com/notes/bulk-accident/ - 公開: 2026-10-08 - 書いた人: Koshin（https://koshinstudio.com/about/） - サイト: Koshin Studio English version (https://koshinstudio.com/en/notes/bulk-accident/) トレカダッシュ 開発の記録 ／ 2026年10月 iOS アプリ「トレカダッシュ」で、記事の状態をまとめて直す新しい処理が、最初に流したとき、対象にするつもりのなかった記事まで、全部に当たりました。別のサイト由来の 18 件が、誤って、表示に戻っていました。事故の直し方と、その後に、同じ穴をふさぐテストを作ったとき、「そのテストが、本当に効くか」を、どう確かめたかを、HANDOFF の記録で書きます。 このサイトには、トレカダッシュの作品ページがあります（トレカダッシュ (https://koshinstudio.com/works/torecadash/)）。ここに書くのは、開発の記録です。 ## 何をする処理だったか アプリには、出品の価格が妥当かを調べる「価格の関門」があります（別の記事 (https://koshinstudio.com/notes/fail-open/)に書きました）。この関門で、価格だけを理由に非表示にされた記事のうち、商品の取り違えが原因で、本当は正しい記事を、表示に戻す処理を足しました。戻す条件は、「正しい商品の定価と、出品価格が一致すること」です。 ## 事故：全記事に当たった この処理は、最初、取り込み元を区別せず、全記事に当たりました。その結果、別のサイト（Amazon、カードラボ、ミニストップなど）由来の記事 18 件が、表示に戻ってしまいました。本来、非表示のままであるべきものです。 直し方は、二つです。 - 範囲を絞る。処理が当たる記事を、その取り込み元のものだけに、限りました（サイトの識別子が、その取り込み元で始まるもの）。 - 戻したものを、元に戻す。誤って表示に戻した 18 件は、記事の状態の履歴（statusHistory）から特定して、元の状態（非表示で、価格で却下されたもの）に戻しました。 ただし、この 18 件には、取り返せなかったものがあります。処理が、元の定価の欄を、出品価格で上書きしていました。値は、出品価格のままです。 原因は、書き込みの前に、対象の範囲を、数えていなかったことでした。記録にも、そう書いてあります。「一括の書き込みは、対象の範囲を先に数えてから」。 ## 同じ穴をふさぐ、テストを作る この日の点検で見つかった問題（9 種セットの誤分割、単品カードの混入、カードではない商品の混入など）を、二度と起こさないために、テストを作りました。catalog-guards.test.js に 66 件、ほかと合わせて、全 6 本を、1 つの手順（run-all.sh）で通せるようにしてあります。 あわせて、毎日の運用連絡に、警告を足しました。突き合わせの取りこぼし率が、基準（抽選図鑑 80%、ポケチュー 75%、X のまとめ 75%）を下回ったとき、30 時間動いていないとき、商品リストに、単品カードや、カードではない名前の商品が現れたときです。判定は、副作用のない純粋な関数で書いてあり、直れば、次の回で、自然に消えます。 ## テストが効くかを、壊して確かめる テストは、書いただけでは、効いているか分かりません。常に通るテストは、何も守っていないかもしれないからです。そこで、本番のコードの複製の、規則を 1 つずつ外して、テストが失敗することを、確かめました。 - 規則を 1 つ外す（壊す）。 - テストを流す。 - 失敗すれば、そのテストは、その規則を、守っている。 この手順で、6 種類の改変を試して、6 種類とも、テストが失敗しました（全部、検出できた）。 運用のきまりも、記録に残しました。functions/index.js を触ったら、配備の前に、必ず、全テストを通します。 ## 分かったこと > 一括の書き込みは、先に数える。テストは、壊して確かめる。 - 書く前に、「何件に当たるか」を数える。下見をせずに流すと、範囲の間違いに、書き終わってから気づきます。 - 元に戻せる形で書く。履歴から特定できたので、18 件を戻せました。ただし、上書きした欄は、戻せません。 - 通るテストより、壊れたら落ちるテスト。規則を外して、落ちることを見て、はじめて、そのテストを信じられます。 この記事は、HANDOFF（§272）の記録（2026年10月7日）から書きました。件数は、記録に書かれたものです。 この話に出てくるもの - 確認できなかったときに、通していた (https://koshinstudio.com/notes/fail-open/) — 同じ「価格の関門」の、別の記録 - 「重複は実質ゼロ」と数えたのに、監査は112組を出していた (https://koshinstudio.com/notes/dup-zero/) — 数え方を疑った、別の記録