Ts review
Skill widnyana/eyay-toolkits/plugins/ts-backend-dev/skills/ts-review
Review TypeScript code for quality, security, and correctness. Use this skill when the user asks for a code review, wants feedback on a PR or diff, or says "review this". Covers architecture, security, error handling, database patterns, type safety, and code quality. Produces categorized findings: critical issues, improvements, suggestions, and positive notes. Also triggers on phrases like "check this code", "what do you think of this", or "any issues with".From its SKILL.md
npx -y skills add widnyana/eyay-toolkits --skill ts-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 4 stars4 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
2.5 KB, 482 tokens by cl100k_base, as published. Nobody here has run it
Code Review
Review: $ARGUMENTS
Review Checklist
1. Architecture & Patterns
- Clear separation of concerns (routing, business logic, data access)
- Dependency injection or explicit dependencies (no hidden globals)
- Service/module layer handles business logic, not handlers/controllers
- Input validation at the boundary (DTOs, schemas, zod, class-validator)
2. Security
- Auth checks on sensitive endpoints
- Role-based or scope-based access control where needed
- No hardcoded secrets, credentials, or API keys
- Input validation and sanitization on all public inputs
- Parameterized queries (no string interpolation in SQL)
- Webhook signature verification
- No sensitive data in logs or error responses
3. Error Handling
- Errors are properly typed and propagated
- Error messages don't leak internals to clients
- External service calls wrapped in try-catch
- Graceful degradation where appropriate
- No swallowed errors (empty catch blocks)
4. Database
- Transactions for multi-step operations
- Proper indexing for query patterns
- No N+1 queries
- Connection handling (pools, timeouts, cleanup)
5. TypeScript-Specific
- No
anywithout justification - Proper type annotations on public APIs
- Discriminated unions over optional sprawl
- Null/undefined handled explicitly (no implicit
!abuse) - Generics used where they add value, not everywhere
6. Code Quality
- Single responsibility per function/class
- No duplicated logic that should be shared
- Descriptive names (no
data,info,resultas sole identifiers) - No dead code or unreachable branches
- Consistent style with the rest of the codebase
Output Format
Provide:
- Summary: Overall assessment in 1-2 sentences
- Critical Issues: Must fix before merge
- Improvements: Should fix
- Suggestions: Nice to have
- Positive Notes: What's done well
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.