Releases: felipereisdev/code-review-skill
Releases · felipereisdev/code-review-skill
v2.0.0
v2.0.0 — Expanded Stack Checklists & Consistency Framework
Highlights
- New Step 1.5 — Consistency Check: Before flagging any pattern violation, the reviewer now checks what the codebase already does. Inconsistency is worse than a suboptimal pattern, so existing conventions take priority unless no pattern exists yet.
- API Validation via live docs: Reviewers are instructed to validate library API usage against current documentation (via context7) before flagging issues, reducing false positives from outdated training data.
- README updated: Features list, stack count (19), project structure, and new stack entries (Inertia.js, Pest PHP, Tailwind v3/v4 detection) all reflect v2.0.0 changes.
New Stack Files
stacks/inertia.md— Full Inertia.js checklist covering navigation, forms, deferred props (v2), polling (usePoll), infinite scroll (WhenVisible), and server-side patterns (Inertia::render,Inertia::defer, shared data).stacks/pest.md— Pest PHP checklist covering test structure, named assertions, datasets, mocking, database isolation, browser tests (Pest 4), and architecture tests (arch()).
Expanded Stack Files
Laravel (stacks/laravel.md)
- Advanced Query Patterns:
addSelect()subqueries, dynamic relationships via subquery, conditional aggregates,setRelation(), compound indexes. - Caching:
Cache::remember(),Cache::flexible(),Cache::memo()/once(), cache tags,Cache::add(),Cache::lock(), failover stores. - HTTP Client: explicit timeouts, retry with backoff, status checks,
Http::pool(), testing withHttp::fake()+preventStrayRequests(). - Queues & Jobs:
retry_after>$timeout, exponential backoff,ShouldBeUnique,failed()method,retryUntil(),RateLimitedmiddleware,Bus::chain()/Bus::batch(). - Events & Notifications:
ShouldDispatchAfterCommit, event discovery,ShouldQueue,assertQueued(). - Validation & Forms:
Rule::when(), array notation,after()hook. - Collections:
cursor()vslazy()vslazyById(),toQuery().
Tailwind CSS (stacks/tailwind.md)
- Version detection: v3 vs v4 import syntax and config conventions (flag mismatched syntax).
- Structured sections: Conflicts & Redundancy, Class Order, Dark Mode, Spacing, Dynamic Classes, Componentization.
- Common Pitfalls: table of frequent mistakes with explanations.
Vue (stacks/vue.md)
- Common Pitfalls: destructuring without
toRefs(), array index as key, methods vs computed, global state in composables,provide/injectwith string keys,watchwithout{ deep: true }, missing cleanup. - Inertia.js section (when
@inertiajs/vue3detected): navigation, forms, deferred props, polling, infinite scroll, server-side patterns.
README Changes
- Added Consistency-First Review and API Validation to features list
- Stack count updated from 17+ to 19
- Added Inertia.js and Pest PHP to supported stacks
- Added Testing category section
- Tailwind CSS entry now notes v3/v4 detection
- Project structure now includes
inertia.mdandpest.md
Full Changelog
SKILL.md: Added Step 1.5 (Consistency Check), API validation instruction, andinertia.md/pest.mdto stack file list.README.md: Updated for v2.0.0 — features, stacks, project structure.stacks/inertia.md: New file — Inertia.js code review checklist.stacks/pest.md: New file — Pest PHP code review checklist.stacks/laravel.md: Expanded from 45 to 133 lines — caching, HTTP client, queues, events, validation, advanced queries, collections.stacks/tailwind.md: Expanded from 12 to 58 lines — version detection, structured sections, common pitfalls.stacks/vue.md: Expanded from 38 to 103 lines — common pitfalls, Inertia.js section, improved formatting.
Full diff: v1.0.0...v2.0.0