| name | code-review |
| description | Review code changes for bugs, pattern violations, and common pitfalls in ecency-mobile |
| argument-hint | ["file-or-branch"] |
| disable-model-invocation | true |
Code Review
Review code for bugs, anti-patterns, and issues specific to the ecency-mobile codebase.
How to Review
- Read the changed files — understand what changed and why
- Verify each finding against current code — don't assume; read the actual file before flagging
- Categorize findings by severity:
- Inline (must fix): Bugs, security issues, data corruption risks
- Outside-diff (should fix): Issues in unchanged code exposed by the change
- Nitpick (nice to have): Style, naming improvements
- Only flag what's confirmed — if a finding doesn't apply to current code, skip it
What to Check
React & Hooks
Class Components (Legacy)
React Query / SDK
SDK Mutations
Sheets (Action Sheets)
Styling
Hive Blockchain Specific
i18n
Common Bug Patterns Found in This Codebase
- Stale isAmountValid state — validation flags not reset when inputs clear or errors occur
- Silent no-ops in auth fallbacks — empty case blocks that silently do nothing for certain auth types
- Race conditions in async setState — multiple concurrent state updates in class components
- Missing error.message extraction — error objects displayed as
[object Object] instead of .message
- Inconsistent validation — amount validated against minimum in one path but not maximum (available balance)
- Unused props not cleaned up — TypeScript interface still declares props that are no longer passed
Output Format
For each finding:
**[SEVERITY]** file:line — Description
- What's wrong
- Why it matters
- Suggested fix
Severities: BUG, SECURITY, PERF, STYLE, NITPICK