بنقرة واحدة
بنقرة واحدة
| name | review |
| description | Review code changes against project architecture, style, and conventions |
| allowed-tools | Bash, Read, Grep, Glob |
| user-invocable | true |
Review the current changes against the project's conventions and architecture rules.
Load the appropriate docs based on which code area is being reviewed:
@docs/store-compose.md, @docs/store-viewmodel-patterns.md, @docs/store-tracking-events.md, @docs/store-testing.md@docs/pos-architecture.md, @docs/pos-tracking-events.md, @docs/pos-testing.md@docs/coding-style.mdDetermine 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.
Determine if the code is POS or store. POS: ui/woopos/, WooPos* prefix. Everything else: store management. Read the relevant docs above.
Read the changed files in full to understand context, not just the diff hunks.
Check against each category below. Only report actual issues found — do not report categories with no issues.
ViewModel(), use parent-child SharedFlow event bus, pure Compose, Compose NavigationScopedViewModel, use triggerEvent() / MultiLiveEvent, Compose inside Fragments, XML nav graphs@HiltViewModel + @Inject constructor, data flows through repositories, ViewModels never access Room/network directlyFIXME (use TODO), no !! force unwrapsUPPER_SNAKE_CASE, companion objects at bottomdocs/store-compose.md (store) or docs/pos-architecture.md (POS)BaseUnitTest, testBlocking, captureValues, AssertJWooPosCoroutineTestRule, runTest, advanceUntilIdle, AssertJAnalyticsEvent enum, AnalyticsTrackerWrapperWooPosAnalyticsEvent sealed class, WooPosAnalyticsTracker## Blockers
- [file:line] Description of blocking issue
## Suggestions
- [file:line] Description of improvement suggestion
## Positives
- Description of what was done well
Run the Android AI Assistant headless smoke regression harness without launching UI.
Generate screenshot coverage for screen-level Compose previews from the current diff or a specific target, check required visual and data variations, and produce a compact visual report
Build, install, and visually verify the app on an Android emulator or device. Uses the Android CLI for agents (android) when available with a full mobile-mcp/adb fallback.
Create a pull request following project conventions. Triggers on any request to create, open, make, submit, file, send, push, spin up, put up, draft, raise, or prepare a PR/pull request.
POS analytics tracking patterns (WooPosAnalyticsEvent sealed class, WooPosAnalyticsTracker, WooPosAnalyticsEventConstant). Use when writing, editing, exploring, debugging, or reviewing analytics tracking in POS (WooPos*) code. NOT for main store app — use the `store-analytics` skill instead.
POS unit testing patterns (WooPosCoroutineTestRule, runTest, advanceUntilIdle, mockito-kotlin, event bus mocking, analytics verification). Use when writing, editing, exploring, debugging, or reviewing unit tests for POS (WooPos*) code. NOT for main store app tests — use the `store-tests` skill instead.