Menú
効果的なコードレビューの実施方法、レビュー観点チェックリスト、建設的なフィードバック技術、セルフレビュー手法、自動化ツール活用まで、品質とチームコラボレーションを向上させる包括的ガイド。
Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.
Basado en la clasificación ocupacional SOC
コンピュータサイエンスの基礎知識を網羅的にカバー。ハードウェアの仕組みからデータ表現、アルゴリズム、データ構造、計算理論、プログラミングパラダイム、ソフトウェアエンジニアリング基礎まで、エンジニアに必要な全てのCS基礎知識を体系的に解説。
Fundamentals of modern web development. Framework selection (React, Vue, Next.js), project architecture, state management, routing, build tools, and CSS strategy best practices.
A comprehensive guide covering the fundamentals of computer science. From hardware internals and data representation to algorithms, data structures, computation theory, programming paradigms, and software engineering basics — a systematic guide to all the CS foundations every engineer needs.
DJテクニック（Rekordbox 7.2.7）と楽曲制作（Ableton Live 12 Suite）を体系的に学べる包括的なガイド。基礎音楽理論から応用テクニック、キャリア構築まで、DJ・音楽プロデューサーに必要な全スキルをカバー。
CLIツール開発ガイド。Node.js（Commander、Inquirer）、Python（Click、Typer）、Go（Cobra）、引数パース、インタラクティブUI、アーキテクチャ設計、テスト、配布方法など、プロフェッショナルなCLIツール開発のベストプラクティス。
スクリプト開発ガイド。Shell、Python、Node.jsスクリプト、自動化、バッチ処理、環境変数管理など、効率的なスクリプト開発のベストプラクティス。
| name | code-review |
| description | 効果的なコードレビューの実施方法、レビュー観点チェックリスト、建設的なフィードバック技術、セルフレビュー手法、自動化ツール活用まで、品質とチームコラボレーションを向上させる包括的ガイド。 |
このSkillは、効果的なコードレビューの全てをカバーします：
このガイドで学べること: レビュープロセス設計、効果的なフィードバック方法、自動化ツール活用、チーム文化構築 公式で確認すべきこと: 最新のレビューツール機能、GitHub/GitLab新機能、ベストプラクティス
GitHub Pull Request Documentation - PR運用ガイド
Google Engineering Practices - Googleコードレビューガイド
GitLab Code Review Guidelines - GitLabレビューガイド
| 観点 | 確認内容 | 詳細ガイド |
|---|---|---|
| 1. 機能性 | 要件を満たしているか | guides/01-functionality.md |
| 2. 設計 | アーキテクチャに従っているか | guides/02-design.md |
| 3. 可読性 | 理解しやすいか | guides/03-readability.md |
| 4. テスト | 十分なテストがあるか | guides/04-testing.md |
| 5. セキュリティ | 脆弱性はないか | guides/05-security.md |
| 6. パフォーマンス | 効率的か | guides/06-performance.md |
| 7. 保守性 | 将来の変更が容易か | guides/07-maintainability.md |
完全チェックリスト: checklists/review-checklist.md
必須ステップ:
1. 変更をdiffで確認
2. セルフレビューチェックリスト実行
3. デバッグコード・コメント削除
4. テスト実行
5. ビルド確認
詳細: guides/08-self-review.md チェックリスト: checklists/self-review.md
レビュワーのステップ:
1. PRの目的・背景を理解
2. テスト・ビルドの通過確認
3. コード全体の構造把握
4. 詳細レビュー（7つの観点）
5. コメント作成
6. 総合判断（Approve/Request Changes）
作成者のステップ:
1. コメントを全て読む
2. 質問に回答
3. 修正が必要なものを修正
4. 議論が必要なものを議論
5. Re-review依頼
詳細: guides/10-feedback-response.md
「ここでnil checkを追加すると、クラッシュを防げます。
Optional bindingを使うのはどうでしょうか？
if let user = user {
// ...
}
」
「これだとクラッシュするよ」
| プレフィックス | 意味 | 例 |
|---|---|---|
| [必須] | 修正必須 | [必須] メモリリークの可能性 |
| [推奨] | 修正推奨 | [推奨] この関数は分割できます |
| [質問] | 質問・確認 | [質問] この仕様で良いですか？ |
| [提案] | 代替案提示 | [提案] Combineを使うと簡潔に |
| [学習] | 学習機会 | [学習] こういう書き方もあります |
| [賞賛] | 良い点 | [賞賛] いいリファクタですね！ |
詳細: guides/11-communication.md
| ツール | 用途 | 設定ガイド |
|---|---|---|
| SwiftLint | コーディング規約 | scripts/swiftlint-setup.sh |
| SwiftFormat | フォーマット | scripts/swiftformat-setup.sh |
| Danger | PR自動チェック | scripts/danger-setup.sh |
| SonarQube | 静的解析 | scripts/sonarqube-setup.sh |
→ references/github-actions-integration.md
❌ let name = user!.name
✅ guard let user = user else { return }
let name = user.name
❌ if items.count > 10 { ... }
✅ private let maxItemsCount = 10
if items.count > maxItemsCount { ... }
❌ func processUser() { // 100行 }
✅ func processUser() {
validateUser()
saveUser()
notifyUser()
}
全パターン: references/common-issues.md
→ references/review-efficiency.md
→ references/conflict-resolution.md
code-review-agent
thoroughstyle-checker-agent
quicksecurity-scanner-agent
thoroughtest-coverage-agent
mediumcode-review-agent (包括的レビュー) +
style-checker-agent (スタイルチェック) +
security-scanner-agent (セキュリティ) +
test-coverage-agent (カバレッジ)
→ 結果統合 → PRに自動コメント
self-review-agent (セルフレビュー支援)
→ 指摘事項を確認・修正
→ 再度確認
[必須] エッジケースの考慮不足
現在の実装だと、空配列の場合にクラッシュします。
提案:
guard !items.isEmpty else { return }
テストケース:
- 空配列
- 要素1つ
- 大量の要素
[推奨] 責務の分離
現在ViewControllerに全てのロジックがありますが、
ViewModelに移動することで、テスタビリティが向上します。
メリット:
- テストが容易
- 再利用性向上
- 関心の分離
[賞賛] エラーハンドリング
きちんとエラーケースを考慮されていて素晴らしいです！
ユーザーにわかりやすいメッセージも表示されていますね。
包括的ガイド（新規追加）
自動化完全ガイド 🤖
PRテンプレート - 包括的なPull Requestテンプレート
Dangerfile - 自動PRチェック
git-workflow - PR作成プロセスtesting-strategy - テストレビュー観点ios-development - iOS固有の観点ci-cd-automation - 自動レビュー統合このSkill自体の変更履歴は CHANGELOG.md を参照