Skill detail
code-review
Direct pull-request code-review checklist.
Inspect before use
Automated review checks relevance, not safety or endorsement. Read the source instructions before using this skill.
SKILL.md
The saved excerpt is a snapshot from review. The external source remains the complete and most current version.
--- name: code-review description: Comprehensive code review checklist for pull requests author: goose version: "1.0" tags: - development - quality - review --- # Code Review Checklist When reviewing code, check each of these areas: ## Functionality - [ ] Code does what the PR description claims - [ ] Edge cases are handled - [ ] Error handling is appropriate ## Code Quality - [ ] Follows project style guide - [ ] No hardcoded values that should be configurable - [ ] Functions are focused and well-named ## Testing - [ ] New functionality has tests - [ ] Tests are meaningful, not just for coverage - [ ] Existing tests still pass ## Security - [ ] No credentials or secrets in code - [ ] User input is validated - [ ] SQL queries are parameterized ## Performance - [ ] No obvious performance issues - [ ] Database queries are optimized - [ ] No unnecessary loops or iterations ## Documentation - [ ] Public APIs are documented - [ ] Complex logic has comments explaining "why" - [ ] README updated if neededRead the full source on GitHub (opens external page)