name: post-completion-review description: タスク完了後の振り返りスキル。問題と対処のまとめ、根本原因分析、リファクタリング検討を行う。「振り返り」「レビュー」「完了後確認」「反省会」「!retro」などの作業に使う。大きなタスクや複数回のエラーを経た実装後に必ず実行。 triggers:

"!retro"
振り返り
レビュー
完了後確認
反省会 metadata: 自己拡張型: true trigger: 実装完了後、特にCI失敗やエラー対応を経た場合

post-completion-review — 完了後振り返りスキル

🎯 目的

タスク完了後に以下3点を確認し、学びを蓄積する:

問題と対処のまとめ — 何が起きて、どう対処したか
根本原因分析 — 対応は適切だったか、再発防止策は打てているか
リファクタリング検討 — 残った技術的負債はないか

🚨 発動条件

以下の場合は必ず実行:

CI/CDが2回以上失敗した
想定外のエラーに遭遇した
「一気にやる」で複数機能を同時実装した
依頼者から「苦戦してましたね」と言われた
30分以上のデバッグ時間を費やした

📋 振り返り手順

Phase 1: 問題と対処のまとめ

テンプレートに従って問題を整理する。 → テンプレート: references/template-issue-summary.md

確認ポイント:

問題は具体的に記述されているか（エラーメッセージ含む）
原因は「なぜ」を掘り下げているか
対処は一時的な回避策か、根本的な解決か

Phase 2: 根本原因分析（RCA）

2-1: 5 Whys 分析

各問題について「なぜ？」を5回繰り返す。 → 分析例: references/example-5whys.md

2-2: 類似問題の調査

MEMORY.mdから類似パターンを検索する。 → スクリプト: scripts/search-similar-issues.sh

確認:

過去に同じパターンの問題があったか
既存の教訓で防げた問題だったか
新しい教訓として追加すべきか

2-3: 再発防止策

教訓	具体策	仕組み化
[学んだこと]	[具体的な行動]	[自動化/チェックリスト化の方法]

仕組み化レベル:

🟢 自動化済み: pre-commit hook, CI, DBトリガー等
🟡 ドキュメント化: MEMORY.md, CHECKLIST.md, スキルに追記
🔴 未対応: 次のアクションを決める

Phase 3: リファクタリング検討

3-1: コードレビュー

今回追加/変更したファイルを確認:

git diff HEAD~N --stat  # Nは関連するコミット数

チェックリスト:

使われていないコードが残っていないか
ハック的な回避策（try-catch import等）がないか
環境依存のハードコードがないか
重複コードがないか
過剰に複雑な部分がないか

3-2: リファクタリング判断

箇所	問題	対応	優先度
[ファイル/関数]	[技術的負債の内容]	[改善方法]	P1/P2/P3

優先度基準:

P1: 今すぐ対応（バグの温床、将来の混乱確実）
P2: 今週中（改善の余地あり）
P3: 時間があれば（nice to have）

3-3: 実行

P1項目は即座にリファクタリングを実行。

📝 最終出力

Slack報告 → references/template-slack-report.md
MEMORY.md追記 → references/template-memory-entry.md

🔄 自己拡張

タスク完了後、このスキル自体も改善する。 → スクリプト: scripts/append-learnings.sh

追記すべき内容:

新しく発見した振り返りパターン
見落としがちなチェックポイント
効果的だった分析手法

📎 関連スキル

lifelog-feedback: 日次フィードバックループ
skill-manager: スキル作成・管理

⚠️ よくある落とし穴

「一気にやる」症候群: 複数機能を同時実装すると複合エラーが発生し、デバッグが困難になる。1機能ずつ実装・検証・コミットを徹底する
CI環境とローカル環境の差異を見落とす: ローカルで動いてもCIで失敗するケースが多い。devDependencies、環境変数、secrets、パス差異に注意。gh secret list で事前確認する
GitHub Actions if式でのsecrets直接参照: if式ではsecretsを直接参照できない。シェル内で $ENV_VAR チェックを使う
リファクタリングの先延ばし: 完了直後が記憶が最も新鮮。振り返り時にP1項目は即座にリファクタリングを実行する
使われないコードの放置: 「将来用」で残したコードが混乱の元になる。使わないなら即削除し、必要になったらgitから復活させる
異なるランタイム（Deno/Node）の混在: tsconfig.json の exclude を最初に設定しないと、ビルドエラーの原因になる

「振り返りなき実装は、同じ失敗を繰り返す。」