Review Changes

SkillDev tools

Review code changes against project architecture, style, and conventions

Available today. Use it from your connected AI after setup.

Connect ahel once, and every AI you use reads what you have installed.

Then ask your AI: use the Review Changes skill

What this skill tells your AI

The instructions your AI receives, as published by woocommerce/woocommerce-android in .agents/skills/review/SKILL.md and read by ahel’s review.

Review the current changes against the project's conventions and architecture rules.

Load the appropriate docs based on which code area is being reviewed:

  • Store code: @docs/store-compose.md, @docs/store-viewmodel-patterns.md, @docs/store-tracking-events.md, @docs/store-testing.md
  • POS code: @docs/pos-architecture.md, @docs/pos-tracking-events.md, @docs/pos-testing.md
  • Shared: @docs/coding-style.md

Steps

  1. Determine the diff to review. First check git diff trunk...HEAD for branch changes (most common: pre-PR review). If empty, fall back to git diff --cached for staged changes, then git diff for unstaged changes.

  2. Determine if the code is POS or store. POS: ui/woopos/, WooPos* prefix. Everything else: store management. Read the relevant docs above.

  3. Read the changed files in full to understand context, not just the diff hunks.

  4. Check against each category below. Only report actual issues found — do not report categories with no issues.

Architecture

  • POS: ViewModels extend ViewModel(), use parent-child SharedFlow event bus, pure Compose, Compose Navigation
  • Store: ViewModels extend ScopedViewModel, use triggerEvent() / MultiLiveEvent, Compose inside Fragments, XML nav graphs
  • Both: @HiltViewModel + @Inject constructor, data flows through repositories, ViewModels never access Room/network directly

Kotlin Style

  • Max 120 character line length (test names excepted)
  • No wildcard imports, no FIXME (use TODO), no !! force unwraps
  • Constants: UPPER_SNAKE_CASE, companion objects at bottom

Jetpack Compose

  • Check against patterns in docs/store-compose.md (store) or docs/pos-architecture.md (POS)

Testing

  • Store: BaseUnitTest, testBlocking, captureValues, AssertJ
  • POS: WooPosCoroutineTestRule, runTest, advanceUntilIdle, AssertJ

Analytics

  • Store: AnalyticsEvent enum, AnalyticsTrackerWrapper
  • POS: WooPosAnalyticsEvent sealed class, WooPosAnalyticsTracker

Output Format

## Blockers
- [file:line] Description of blocking issue

## Suggestions
- [file:line] Description of improvement suggestion

## Positives
- Description of what was done well

Signals

GitHub stars
319
Forks
146
Last commit
Sep 2026
Hacker News mentions
20
Advanced
Catalog kind
skill
Gateway key
review-woocommerce
Source
github.com/woocommerce/woocommerce-android