القائمة
Goコードレビューや修正後検証で使用。並行処理、goroutineリーク、エラー処理、リソース管理を確認する。
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
استنادا إلى تصنيف SOC المهني
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 の §レビュー結果 で同等