df0baeff36
Tracked: CLAUDE.md, agents, skills, settings, memory. Ephemeral data (sessions, history, telemetry, tasks) excluded via .gitignore. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
940 B
940 B
name, description
| name | description |
|---|---|
| code-review | Performs thorough code review checking quality, security, patterns, and best practices for trading bot code |
Code Review Skill
Performs comprehensive code review for the OTCryptoRobot trading bot.
Trigger
When user asks to review code, check changes, or review a PR.
Process
- Identify Changes - Run
git diffto see changes - Review Each File - Check against patterns in CLAUDE.md
- Check Categories
- Generic
<C, F>pattern compliance - Async/await correctness (critical for real-time trading)
- CancellationToken propagation
- EF Core patterns (AsNoTracking, DbContextFactory)
- DI registration in RegisterService.cs
- Error handling (no empty catch blocks)
- Security (API keys, credentials)
- Thread safety (singleton services)
- SignalR/gRPC patterns
- Generic
- Output - Summary with severity-rated issues