Go特有の観点でコードレビューを行うスキル。プロダクトコード・テストコード両方に対応し、coding-golangとtesting-golangスキルの規約に準拠しているかも確認する。

関連スキルへの参照

REQUIRED: レビュー対象がプロダクトコードなら coding-golang、テストコードなら testing-golang の規約に準拠しているか確認すること。

レビューフロー

dot

digraph review_flow {
    rankdir=TB;
    node [shape=box];

    start [label="コードレビュー開始" shape=ellipse];
    check_type [label="コードの種類を判定" shape=diamond];

    prod_skill [label="coding-golang規約を確認"];
    test_skill [label="testing-golang規約を確認"];

    go_issues [label="Go特有の問題をチェック\n(references/go-review-checklist.md)"];
    common_issues [label="よくある問題パターン確認\n(references/common-go-issues.md)"];

    project_check [label="プロジェクト固有の規約確認\n(既存コードとの一貫性)"];
    summary [label="レビュー結果を整理\n(重要度順)"];
    done [label="レビュー完了" shape=ellipse];

    start -> check_type;
    check_type -> prod_skill [label="プロダクトコード"];
    check_type -> test_skill [label="テストコード"];
    check_type -> prod_skill [label="両方"];
    check_type -> test_skill [label="両方"];

    prod_skill -> go_issues;
    test_skill -> go_issues;
    go_issues -> common_issues;
    common_issues -> project_check;
    project_check -> summary;
    summary -> done;
}

レビュー観点

1. Go特有の問題（最重要）

カテゴリ	確認項目
ゴルーチン安全性	リーク、無制限生成、競合状態
チャネル	デッドロック、クローズ忘れ、nil送受信
リソース管理	Close()呼び出し、defer使用、コネクションプール
エラーハンドリング	無視されたエラー、rows.Err()、ラッピング
コンテキスト	伝播、キャンセル対応、タイムアウト
同期	Mutex unlock漏れ、WaitGroup誤用、sync.Once

2. よくある問題パターン

3. コード品質

4. テストコード（該当する場合）

5. セキュリティ

必須チェックリスト

Critical（必ず確認）

High（重要）

Medium

出力フォーマット

markdown

## レビュー結果

### Critical Issues

- [行番号] 問題の説明と修正案

### High Priority

- [行番号] 問題の説明と修正案

### Medium Priority

- [行番号] 問題の説明と修正案

### Suggestions

- 改善提案

### 確認事項

- coding-golang/testing-golang 規約への準拠: [OK/要修正]
- プロジェクト固有の規約: [OK/要修正/未確認]

スキル	確認内容
coding-golang	プロダクトコードの規約（chi、GORM、エラーハンドリング等）
testing-golang	テストコードの規約（testify、mockery、テーブル駆動テスト等）

ファイル	内容
go-review-checklist.md	Go特有のレビュー観点詳細
common-go-issues.md	よくある問題パターンと修正例

reviewing-golang

Go Code Reviewer (reviewing-golang)

Overview