skills/bpsai/reviewing-code/SKILL.md
Use when reviewing code changes, checking PRs, or evaluating code quality.
npx skillsauth add aiskillstore/marketplace reviewing-codeInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
# Run ALL checks at once (tests + linting)
bpsai-pair ci
# Validate project structure
bpsai-pair validate
# See what changed
git diff main...HEAD --stat
git diff main...HEAD
## Code Review: [Description]
### Summary
Brief assessment.
### Must Fix
1. **[File:Line]** - Issue and fix
### Should Fix
1. **[File:Line]** - Suggestion
### Consider
1. **[File:Line]** - Optional improvement
### Positive Notes
- What was done well
### Verdict
- [ ] Approve
- [ ] Approve with comments
- [ ] Request changes
# Find debug statements
git diff main...HEAD | grep -E "print\(|breakpoint|pdb"
# Find TODOs in changes
git diff main...HEAD --name-only | xargs grep -n "TODO\|FIXME"
# Check for secrets
git diff main...HEAD | grep -iE "password|secret|api.?key|token"
development
Apple Human Interface Guidelines for content display components. Use this skill when the user asks about charts component, collection view, image view, web view, color well, image well, activity view, lockup, data visualization, content display, displaying images, rendering web content, color pickers, or presenting collections of items in Apple apps. Also use when the user says how should I display charts, what's the best way to show images, should I use a web view, how do I build a grid of items, what component shows media, or how do I present a share sheet. Cross-references: hig-foundations for color/typography/accessibility, hig-patterns for data visualization patterns, hig-components-layout for structural containers, hig-platforms for platform-specific component behavior.
tools
Automate HelpDesk tasks via Rube MCP (Composio): list tickets, manage views, use canned responses, and configure custom fields. Always search tools first for current schemas.
testing
Expert Haskell engineer specializing in advanced type systems, pure functional design, and high-reliability software. Use PROACTIVELY for type-level programming, concurrency, and architecture guidance.
tools
GraphQL gives clients exactly the data they need - no more, no less. One endpoint, typed schema, introspection. But the flexibility that makes it powerful also makes it dangerous. Without proper controls, clients can craft queries that bring down your server. This skill covers schema design, resolvers, DataLoader for N+1 prevention, federation for microservices, and client integration with Apollo/urql. Key insight: GraphQL is a contract. The schema is the API documentation. Design it carefully.