Menü
Goコードレビューや修正後検証で使用。並行処理、goroutineリーク、エラー処理、リソース管理を確認する。
Mit Codex oder Claude installieren Kopieren Sie diesen Prompt, fügen Sie ihn in Codex, Claude oder einen anderen Assistant ein und lassen Sie die Skill-Seite prüfen und installieren.
Basierend auf der SOC-Berufsklassifikation
1Password CLI (op) でサインイン・vault 確認・secret 注入を行う時に使う。Claude セッションでは service account token (OP_SERVICE_ACCOUNT_TOKEN) をtoken ファイル (~/.config/op/service-account-token) から直前注入する方式を使い、op read の stdout を Claude context に流さない方針を強制する。「op signin」「op service-account」「op read」「op run」「op inject」「1Password から取って」などで発火。
Browser automation for opening sites, forms, clicks, screenshots, scraping, login, and web app testing.
Run browser automation on AWS Bedrock AgentCore cloud browsers or AWS-hosted sessions.
Capture architecture decisions as ADRs with context, alternatives, and rationale.
不具合修正のワークフロー（調査→再現→修正→検証）を controller がオーケストレーションするための指揮書。バグ・不具合・障害の修正で使う。explorer で原因特定、implementer で再現テスト→修正、verifier で検証。「バグを直して」「不具合修正」「この障害を調査して直す」などで参照。
Create CodeTour `.tour` walkthroughs with file/line anchors.
| name | reviewing-golang |
| description | Goコードレビューや修正後検証で使用。並行処理、goroutineリーク、エラー処理、リソース管理を確認する。 |
| invocation | auto |
| ecc-imports | [{"upstream-commit":"4e66b2882da9afb9747468b08a253ca2f09c85f3","upstream-path":"agents/go-reviewer.md","sections-merged":[],"conflicts":["Review Priorities","Diagnostic Commands","Approval Criteria"],"imported-at":"2026-04-26T15:00:00.000Z"}] |
Go特有の観点でコードレビューを行うスキル。プロダクトコード・テストコード両方に対応し、coding-golangとtesting-golangスキルの規約に準拠しているかも確認する。
| スキル | 確認内容 |
|---|---|
| coding-golang | プロダクトコードの規約（chi、GORM、エラーハンドリング等） |
| testing-golang | テストコードの規約（testify、mockery、テーブル駆動テスト等） |
REQUIRED: レビュー対象がプロダクトコードなら coding-golang、テストコードなら testing-golang の規約に準拠しているか確認すること。
REQUIRED: レビュー開始時に references/go-review-checklist.md を Read ツールで読み込むこと。リンクを見るだけでなく、ファイルの全内容を読み込んでからレビューを行う。特に修正が状態遷移・ライフサイクル管理・クリーンアップ処理に関わる場合、「並行ライフサイクルの安全性」セクションのチェックリストを全項目適用すること。
| カテゴリ | 確認項目 |
|---|---|
| ゴルーチン安全性 | リーク、無制限生成、競合状態 |
| チャネル | デッドロック、クローズ忘れ、nil送受信 |
| リソース管理 | Close()呼び出し、defer使用、コネクションプール |
| エラーハンドリング | 無視されたエラー、rows.Err()、ラッピング |
| コンテキスト | 伝播、キャンセル対応、タイムアウト |
| 同期 | Mutex unlock漏れ、WaitGroup誤用、sync.Once |
詳細は common-go-issues.md を参照。
gosec ./...context.WithTimeout でタイムアウトを設定する
ctx, cancel := context.WithTimeout(ctx, 5*time.Second)
defer cancel()
os.Getenv("KEY") の戻り値を空文字チェックし、未設定なら error を返す（silent failure 防止）
apiKey := os.Getenv("API_KEY")
if apiKey == "" {
return fmt.Errorf("API_KEY not configured")
}
レビュー時に以下を必ず確認すること：
修正が状態遷移、クリーンアップ処理、リソースのライフサイクル管理に関わる場合、詳細チェックリストは go-review-checklist.md §並行ライフサイクルの安全性 を参照し、全項目のOK/NG判定を出力に含めること。
レビュー結果は以下の形式で整理すること：
## レビュー結果
### Critical Issues
- [行番号] 問題の説明と修正案
### High Priority
- [行番号] 問題の説明と修正案
### Medium Priority
- [行番号] 問題の説明と修正案
### Suggestions
- 改善提案
### 確認事項
- coding-golang/testing-golang 規約への準拠: [OK/要修正]
- プロジェクト固有の規約: [OK/要修正/未確認]
| ファイル | 内容 |
|---|---|
| go-review-checklist.md | Go特有のレビュー観点詳細 |
| common-go-issues.md | よくある問題パターンと修正例 |
ECC base commit
4e66b2882da9afb9747468b08a253ca2f09c85f3のagents/go-reviewer.md（76 行）を検証したが、本 skill の構造（references/に詳細を委譲する索引型 + プロジェクト固有規約優先）と異なるため統合せず、全 H2 を conflicts として記録。ECC
go-reviewerは CRITICAL/HIGH/MEDIUM の severity 階層を持つレビューチェックリスト形式:
- Review Priorities (severity 階層): SQL injection / Command injection / Race conditions / Goroutine leaks / errors.Is/As / Mutex misuse 等
- 既存の §レビュー観点 / §必須チェックリスト +
references/go-review-checklist.md/references/common-go-issues.mdで同等の網羅性をカバー（重複につき conflicts）- Diagnostic Commands:
go vet/staticcheck/golangci-lint/go test -race/govulncheck
- 必要に応じて
references/diagnostic-commands.mdとして将来配置可能（本 spec のスコープ外）- Approval Criteria: Approve/Warning/Block の判定基準
- 本 skill の §レビュー結果 で同等