G-EDIT-TRIM-H: ASR呼び出し失敗時のロバスト性テストを追加、G-EDIT-TRIM-Iは既存実装を確認

[rahiseko-alt]

概要

• G-EDIT-TRIM-H（ASR呼び出しが失敗しても詰め処理がクラッシュしない）: tests/trim-hi-asr-robustness-check.mjs を新設し、video-shorts/package.json の test チェーンへ登録した。
  • モック注入点は pipeline.mjs の export renderSegment の words 引数（cmdRender は transcript.words || [] を渡すだけで、実際の ASR は別プロセス src/transcribe.py が事前に書く出荷経路のため）。
  • mockTranscribe(scenario) で (a) タイムアウト (b) 不正JSON (c) 空words配列 を模し、resolveWordsFromAsr() が例外を投げず [] へ落とすことを確認したうえで renderSegment を実行。
  • 実測（ffprobe）で (1) 例外を投げない (2) 出力尺が入力尺と一致する、の2点を確認。
  • 対照として、無音2.0秒を含む本物の words を渡すと出力尺が明確に縮むことも確認（測り方が「詰めた／詰めていない」を区別できることの担保）。
  • 実行結果: 12 PASS / 0 FAIL。本体コード（trim-plan.mjs 等）は既にこのロバスト性を満たしていたため、テスト追加のみで対応。
• G-EDIT-TRIM-I（語データが壊れていても詰め処理が落ちない）: 既に tests/trim-plan-check.mjs に対応するテスト「時刻が壊れた語は無視する（落ちない）」が存在し、20 PASS / 0 FAIL であることを確認。追加実装・追加テストは不要と判断。
• docs/roadmap.html: 両葉を status: "doing" にし、実施内容・確認結果を detail に追記（criteria.text/criteria.verify 本文は無変更）。meta.active/meta.next/meta.handoff も更新。

ローカル確認

node video-shorts/tests/trim-plan-check.mjs        # 20 PASS / 0 FAIL
node video-shorts/tests/trim-hi-asr-robustness-check.mjs  # 12 PASS / 0 FAIL
node scripts/verify-roadmap-evidence.mjs            # OK
BASE_REF=origin/main node scripts/verify-criteria-freeze.mjs  # OK

video-shorts の pnpm test（全体）はローカル実行時に同一ホスト上で他セッションの重いテスト（顔検出・モザイク処理）が並走しており、CPU競合により無関係なタイミング系テスト（M-5-B: モザイク処理の追加時間）が一時的にFAILした（この変更が触れていない face-mosaic-check.py 側の性能しきい値テストで、trim/ASR経路とは無関係）。孤立環境である CI の結果を正とする。

テスト計画

• node video-shorts/tests/trim-plan-check.mjs が全PASS
• node video-shorts/tests/trim-hi-asr-robustness-check.mjs が全PASS（新規）
• node scripts/verify-roadmap-evidence.mjs が緑
• BASE_REF=origin/main node scripts/verify-criteria-freeze.mjs が緑
• CI（ci-green）が緑であることを確認

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_014LU9XgyUv1kaJdG3t3XVyY

---

Generated by Claude Code

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a46ae613-ec2f-4171-9a58-2003ddd21671

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[rahiseko-alt]

クローズします（マスター判断）

このPRが対象とする G-EDIT-TRIM-H / G-EDIT-TRIM-I の2葉を、ロードマップから削除することになりました（#85）。

経緯

• 調査の結果、このPRの凍結verify文言が前提とする resolveWordsFromAsr() は本番コードに存在しないことが判明しました。ASRはレンダリング前に走る別プロセスで、pipeline.mjs 側にASR呼び出し自体が無いというアーキテクチャと、verify文言の前提が根本的に噛み合っていませんでした（実装漏れでも単純誤記でもなく、着手前の言語化ミスでした）。
• GOALの受入定義に対して両葉は不要であり、永久に未実装のままでも trim-plan.mjs の既存の防御コード（語配列が空なら全区間keepへフォールバック、不正な語はfilterで除外）により実害発生確率は極めて低いことを確認しました。
• マスターより「必要性の低い葉は存在がデメリットである」との判断で、削除が決定しました。

このPRで積み上げていただいた検証（renderSegmentへの実データ疎通確認等）自体に問題があったわけではなく、対象とする受入基準（ロードマップの葉）自体が無くなったための対応です。ありがとうございました。

---

Generated by Claude Code

---

Generated by Claude Code