feat(merge-pipeline): post_merge_feedback post_steps を有効化 (task 1-D)

[aloekun]

Summary

• .claude/hooks-config.toml の [[merge_pipeline.post_steps]] を有効化し、本プロジェクトで ADR-029 (post-merge-feedback 自動起動) の dogfood を開始
• templates/hooks-config-{typescript,python}.toml に同ブロックをコメントアウト状態で追記 (派生プロジェクトは opt-in)
• docs/todo.md: 1-D を削除 + 「マージ後フィードバックの定常化」section ヘッダー / 前文を除去。1-E / 1-F は ADR-029 参照付きで独立 section に昇格

本 PR は config のみの変更で exe 再ビルド / 再 deploy 不要 (cli-merge-pipeline.exe / hooks-stop-feedback-dispatch.exe は PR #70 / #71 で deploy 済み)。実地検証は次の pnpm merge-pr (別 PR) で pending file が生成され Stop hook が発火するかで行う。

Refs: ADR-029 (task 1-D)

Test plan

• pnpm lint / pnpm test / pnpm build PASS
• takt pre-push-review APPROVE (1 iteration)
• 次の pnpm merge-pr 実行時に .claude/post-merge-feedback-pending.json が生成され、Stop hook が [POST_MERGE_FEEDBACK_TRIGGER] 付き additionalContext を出力することを確認
• skill Phase 0 未実装のため、pending file は skill 側で手動 consume (1-E で自動化予定)

Summary by CodeRabbit

リリースノート

• New Features

  • マージ後の自動フィードバック機能を有効化しました。

• Chores

  • 設定テンプレートに新しいマージパイプライン構成オプションを追加しました。
  • ドキュメントを更新しました。

[coderabbitai]

📝 Walkthrough

Walkthrough

マージパイプラインの設定を更新し、コメントアウトされていたポストマージAIステップを有効化しました。.claude/hooks-config.tomlでpost_merge_feedbackAIステップを有効にし、テンプレートファイルに対応する設定例を追加し、関連するTODOドキュメントを見直しました。

Changes

Cohort / File(s)	Summary
マージパイプライン設定 .claude/hooks-config.toml	コメントアウトされていたpost_merge_learningsをコメント解除し、有効なpost_merge_feedbackAIステップ（type = "ai"、prompt = "post-merge-feedback"）に置き換えました。
設定テンプレート templates/hooks-config-python.toml, templates/hooks-config-typescript.toml	マージパイプラインのpost_stepsセクションを追加し、ポストマージAIフィードバックステップの設定例をコメントアウト状態で提供しました。タイムアウト、デフォルトブランチ、プロンプト名を含みます。
ドキュメント更新 docs/todo.md	ポストマージフィードバック実装の段階をADR-029参照に整理し、詳細な目標・設計・依存関係のテキストを削除しました。スキルリポジトリ側の実装フェーズに焦点を再調整しました。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• docs(adr): ADR-029 post-merge-feedback 自動起動 策定 + 関連 ADR 改訂 (task 1-A) #69: 同じポストマージフィードバック自動トリガー機能の設定・ドキュメント変更を対象とし、本PRが有効化するAIステップの実装計画（ADR-029とペンディングファイル実装）を定義しています。
• feat(cli-merge-pipeline): ai ステップで pending file 書き込み (task 1-B) #70: 本PRが有効化するポストマージAIステップ（post_merge_feedback / プロンプトpost-merge-feedback）が直接的に依存するCLI・ADR-029ペンディングファイルポストマージ処理を実装しています。
• feat(cli): merge-pipeline 導入 (ADR-013) #27: マージパイプラインのpost_steps設定を変更してポストマージAIステップを追加するもので、本PRの有効化と関連しています。

🚥 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	PRタイトルは、.claude/hooks-config.toml の post_merge_feedback post_steps 有効化という主要な変更を正確に反映している。タイトルは簡潔で具体的であり、変更内容を明確に示しており、チームメンバーが履歴を走査する際に主な変更が理解できる。
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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 and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

templates/hooks-config-typescript.toml (1)

57-68: 前提条件コメントの補強を検討

有効化の前提として skill のデプロイのみが記載されていますが、ADR-029 のフロー上は cli-merge-pipeline 側で type = "ai" の post_step をサポートしていること、および pending file を検知する hooks-stop-feedback-dispatch がデプロイ済みであることも必須です。派生プロジェクトがこのブロックを uncomment しただけでは動作しないケースがあるため、コメントに 3 点すべて明記しておくと親切です。

📝 コメント補強案

 # ─── Merge Pipeline: post_steps ───
 # pnpm merge-pr 後に post-merge-feedback skill を自動起動する (ADR-029 試験運用)。
-# 有効化には post-merge-feedback skill が deploy されている必要があります。
+# 有効化には以下すべてが必要です:
+#   - cli-merge-pipeline.exe が `type = "ai"` post_step をサポート (pending file を書き出す)
+#   - hooks-stop-feedback-dispatch.exe がデプロイ済み (pending file を検知)
+#   - post-merge-feedback skill がデプロイ済み (Phase 0 で pending file を consume)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@templates/hooks-config-typescript.toml` around lines 57 - 68, Update the
commented merge_pipeline post_steps block to list all three prerequisites before
enabling: that the post-merge-feedback skill is deployed, that the
cli-merge-pipeline supports post_steps with type = "ai" (so type="ai" is
supported), and that the hooks-stop-feedback-dispatch service (which detects
pending files) is deployed; reference the merge_pipeline/post_steps example and
ADR-029 so readers know these three conditions (post_merge_feedback, type="ai"
support in cli-merge-pipeline, and hooks-stop-feedback-dispatch) must be met
before uncommenting and enabling the block.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@templates/hooks-config-typescript.toml`:
- Around line 57-68: Update the commented merge_pipeline post_steps block to
list all three prerequisites before enabling: that the post-merge-feedback skill
is deployed, that the cli-merge-pipeline supports post_steps with type = "ai"
(so type="ai" is supported), and that the hooks-stop-feedback-dispatch service
(which detects pending files) is deployed; reference the
merge_pipeline/post_steps example and ADR-029 so readers know these three
conditions (post_merge_feedback, type="ai" support in cli-merge-pipeline, and
hooks-stop-feedback-dispatch) must be met before uncommenting and enabling the
block.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c7d75695-70bd-41ea-8d94-646c5a23f536

📥 Commits

Reviewing files that changed from the base of the PR and between dbc95a4 and e21712e.

📒 Files selected for processing (4)

• .claude/hooks-config.toml
• docs/todo.md
• templates/hooks-config-python.toml
• templates/hooks-config-typescript.toml