augmnt/webdev-skills

reviewing-code

Provides structured code review with prioritized feedback.

查看源码
仓库原始内容

按源仓库内容呈现,保留标题、案例、代码、表格、链接以及原文引用的演示图片。

Reviewing Code

Prioritized, actionable feedback. Assume good intent.

Priority Levels

LevelPrefixMeaning
Blocker🚨Must fix before merge
Suggestion⚠️Should consider
Nit💭Optional polish
QuestionNeed clarification
PraiseDone well

Checklist

Blockers (Security & Correctness)

  • [ ] No secrets/credentials in code
  • [ ] User input validated/sanitized
  • [ ] Auth checks present where needed
  • [ ] Edge cases handled (null, empty, errors)
  • [ ] No race conditions in async code

High Priority (Performance & Logic)

  • [ ] No N+1 queries
  • [ ] Large lists paginated
  • [ ] Expensive ops not in render loops
  • [ ] Error handling appropriate

Medium Priority (Maintainability)

  • [ ] Functions have single responsibility
  • [ ] No copy-paste duplication
  • [ ] Naming is clear

Output Format

markdown
## Summary
[Approve / Request Changes / Discuss]

## 🚨 Blockers
- `file:line` - Issue and fix

## ⚠️ Suggestions
- `file:line` - Suggestion and why

## 💭 Nits
- `file:line` - Minor improvement

## ✨ Good Stuff
- What was done well

Common Issues

React: Missing useEffect deps, state that should be derived, prop drilling TypeScript: any types, missing null checks API: Missing loading/error states, unbounded queries

For detailed React/hooks patterns and accessibility checks, see REACT-PATTERNS.md.

来自同一仓库

更多 Skills

全部 Skills