Skip to content

Releases: felipereisdev/code-review-skill

v2.0.0

20 Apr 21:59

Choose a tag to compare

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 with Http::fake() + preventStrayRequests().
  • Queues & Jobs: retry_after > $timeout, exponential backoff, ShouldBeUnique, failed() method, retryUntil(), RateLimited middleware, Bus::chain() / Bus::batch().
  • Events & Notifications: ShouldDispatchAfterCommit, event discovery, ShouldQueue, assertQueued().
  • Validation & Forms: Rule::when(), array notation, after() hook.
  • Collections: cursor() vs lazy() vs lazyById(), 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/inject with string keys, watch without { deep: true }, missing cleanup.
  • Inertia.js section (when @inertiajs/vue3 detected): 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.md and pest.md

Full Changelog

  • SKILL.md: Added Step 1.5 (Consistency Check), API validation instruction, and inertia.md / pest.md to 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