review-frontendlisted

# Review: Frontend (React + MobX + TypeScript) ## Purpose Validates frontend-specific patterns in the changed code of the current branch. Covers React component correctness (observer wrapping, MVVM boundary, useEffect misuse, accessibility basics, key props), full MobX store patterns (structure, member ordering, accessibility modifiers, observable collections, reactions, async actions, bidirectional sync, lifecycle, API placement), and TypeScript safety. This skill consolidates what was previously split between `code-style-review` (frontend parts) and `code-mobx-store-review`. It supersedes both for frontend changes. --- ## Input Contract | Field | Required | Description | |-------|----------|-------------| | Branch / diff range | No | Defaults to merge-base..HEAD + uncommitted changes | | Explicit commit hash/range | No | Review only that range when provided | | `JOB_NAME` | No | Job name when dispatched by orchestrator | | `CONTEXT_PATH` | No | Path to context doc when dispatched by orchestrator | --- ## Scope Boundaries | Concern | This skill | Use instead | |---------|-----------|-------------| | React component patterns (observer, MVVM, useEffect, keys, a11y) | YES | — | | MobX store: structure, actions, computed, reactions, lifecycle | YES | — | | MobX: observable collections, bidirectional sync, disposers | YES | — | | View↔Store boundary violations | YES | — | | TypeScript safety in frontend code (no any, I* prefix, ?./??/!) | YES | — | | Backend patterns, Ne