fix(push-runner): quality_gate 失敗 step の出力を全量表示 (push パイプライン改善 R1)

[aloekun]

Summary

• quality_gate の step 失敗時、出力を run_cmd_shell_capped (40 行 silent truncate) で表示していたため cargo test の失敗一覧が消えて診断できなかった問題を修正
• step 実行を run_step に集約し run_cmd_shell_unlimited へ切替え、失敗経路は全量表示・成功経路は従来どおり quiet (退行なし)
• 回帰テスト mod r1_failure_output_not_truncated 2 本を追加 (cap の外の診断行が残ること + 成功 step 非退行)
• run_cmd_shell_unlimited は T5 (PR fix(cli-push-runner): push 拒否検知の 40 行 truncate 依存を修正 (push T5) #282) で追加済みのため lib-subprocess は無変更

Context

• Why: push パイプライン改善計画 (plan2) の残タスク R1。T5 が push stage で確立した「判定は exit status・失敗経路は診断を落とさない」原則の残り半分を quality_gate に横展開する。
• Trigger: 2026-07-16 の push パイプライン調査で backlog 化 (§6 項目 1)、2026-07-18 にユーザー判定で採用確定 (T13)。
• Scope decision: 変更は stages/quality_gate.rs に閉じる。判定 (ok) は exit status 由来で出力量に依存しないため、全量保持のコストは失敗 step の診断のためだけに払う。capped_reporting + cap 引き上げ案は truncate を明示するだけで受け入れ基準「truncate されず表示」を満たさず不採用。

Validation

• cargo test -p cli-push-runner: 252 pass (R1 の 2 本追加、250 → 252)
• cargo clippy -p cli-push-runner --all-targets --all-features: warning 0
• 回帰テストが素通りしない実証: run_step を capped 版に戻すと bad が「40 行に切り詰め」で fail、good は通る
• pnpm push pre-push review (pre-push-review-refute): verdict=APPROVE (2026-07-18、security + simplicity 両 reviewer 承認)
• exe 再ビルド + dogfood push で quality_gate 全 group PASS

References

• ADR-049 (incident→eval 回帰スイート) / ADR-043 (fail-closed) / ADR-044 (subprocess utility boundary)
• PR fix(cli-push-runner): push 拒否検知の 40 行 truncate 依存を修正 (push T5) #282 (T5: push 拒否検知の truncate 依存修正、run_cmd_shell_unlimited 追加元)
• docs/push-pipeline-fix-plan2.md §3 R1

Summary by CodeRabbit

• バグ修正
  • 品質チェック失敗時の診断出力が途中で切り詰められず、問題の原因を確認しやすくなりました。
  • 成功時の出力抑制と判定動作は従来どおり維持されています。
• テスト
  • 長い出力を伴う失敗ケースと、成功ケースの回帰テストを追加しました。
• ドキュメント
  • 出力改善の実装内容と検証結果を計画書に追記しました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fb7b78e2-3e60-451d-994e-50d1f87b3bb6

📥 Commits

Reviewing files that changed from the base of the PR and between bcf6842 and 76314b3.

📒 Files selected for processing (2)

• docs/push-pipeline-fix-plan2.md
• src/cli-push-runner/src/stages/quality_gate.rs

---

📝 Walkthrough

Walkthrough

quality_gate の失敗時 step 実行を無制限出力取得へ変更し、pre と command に適用した。40 行を超える失敗出力と成功経路の回帰テスト、および実装結果の文書記録を追加した。

Changes

Quality gate output handling

Layer / File(s)	Summary
失敗時の無制限出力取得 src/cli-push-runner/src/stages/quality_gate.rs	run_step が run_cmd_shell_unlimited を使用し、run_group の pre と command 実行で失敗時の全量出力を保持するよう変更した。
回帰テストと実施記録 src/cli-push-runner/src/stages/quality_gate.rs, docs/push-pipeline-fix-plan2.md	40 行を超える失敗出力と成功 step の挙動を検証するテストを追加し、実装方針・テスト結果・ビルド結果を文書化した。

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#33: quality gate の実装と失敗出力取得経路に直接関連する。
• aloekun/claude-code-hook-test#208: capped 出力から unlimited 出力へ切り替える同一系統の変更である。
• aloekun/claude-code-hook-test#282: 40 行出力制限を unlimited capture へ変更する同一系統の対応である。

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	quality_gate の失敗 step 出力を全量表示する変更を端的に表しており、主な変更内容と一致しています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/quality-gate-failure-full-output

---

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.}

[github-actions]

🤖 PR Monitor 分析 (GitHub Actions バックストップ)

• トリガー: issue_comment (created) / 実行 run
• CI: CodeRabbit のみが check として登録されており pending (Review in progress)。他の失敗 check なし。mergeStateStatus は UNSTABLE (pending check 由来と推定)、mergeable は MERGEABLE。
• レビュー状況: CodeRabbit はレビュー未着 (walkthrough 生成中のプレースホルダコメントのみ、findings なし)。人間レビューも reviews API 上 0 件。
• Verdict: approved (現時点で applicable な findings が 0 件のため .takt/facets/instructions/analyze-coderabbit.md Step 4 の規則「findings array が空なら approved」に従う。CodeRabbit 未着のため今後 findings が追加され次第、再評価が必要)

Applicable Findings (Critical / High / Major)

該当なし (レビュー未着・指摘 0 件)

Applicable Findings (Medium 以下)

該当なし

Filtered (not applicable)

該当なし

変更概要 (軽量サマリー)

• docs/push-pipeline-fix-plan2.md: R1 タスクの実施結果セクションを追記 (+28 行)。方針・実装範囲・回帰テスト内容・検証範囲の記録で、コード変更なし。
• src/cli-push-runner/src/stages/quality_gate.rs: quality_gate の step 出力を run_cmd_shell_capped (40 行 truncate) から run_cmd_shell_unlimited に切替える run_step ヘルパーを追加。失敗 step は全量出力、成功 step は従来どおり非表示 (quiet 維持)。ADR-049 の流儀に沿った回帰テスト2本 (failing_step_output_is_not_truncated, passing_step_still_succeeds) を追加。

次のアクション

• CodeRabbit のレビュー完了を待って再度本 workflow (issue_comment トリガー等) が走った際に findings の有無を再評価する。
• 現時点で CI 失敗・レビュー指摘なし。ローカルセッションでは push 前チェックリスト (cargo test / clippy) が plan2.md の記載通り完了していることを確認済みなので、CodeRabbit 完了後に問題なければマージ判断へ進めてよい。