android-code-review-expert

Android Code Review Excellence 🧐🛡️

A rigorous framework for AI agents and developers to perform high-quality, professional Android code reviews.

⚡ When to Use

• PR Review: Analyzing a Pull Request for logic, style, and regressions.
• Pre-commit Audit: Self-checking code before pushing.
• Architectural Check: Ensuring new features follow the established team patterns.
• Security Audit: Spotting sensitive data exposure or insecure networking.
• Performance Audit: Detecting potential memory leaks or UI jank.

🏗️ Review Checklist

1. Architecture & Design

• Clean Architecture: Is logic properly separated into Domain/Data/Presentation?
• State Flow: Does the ViewModel expose StateFlow? Is state properly hoisted?
• Dependency Injection: Are dependencies injected via Hilt/Koin? No manual new Class() for dependencies.
• Manifest Scaffolding: Are required permissions (like INTERNET for Coil/Retrofit) actually defined? Is the @HiltAndroidApp class correctly designated in AndroidManifest.xml via android:name?
• Type-Safe Navigation: Is the app using Compose NavHost with Kotlin Serialization (type-safe routes) instead of anti-pattern manual state swapping?
• Modularity: Is the code placed in the correct module?

2. Jetpack Compose (UI)

• Recomposition: Are there expensive calculations inside @Composable? Use remember.
• Previews: Does the new component have @Preview for light/dark mode and accessibility?
• Stability: Are domain models marked @Stable or @Immutable?
• Material 3: Is the code using the theme's MaterialTheme.colorScheme instead of hardcoded hex values?
• Theme Bridge: If XML resources are used (e.g. res/), is the Material Components XML Bridge correctly set up in themes?

3. Stability & Networking

• Error Handling: Are network calls wrapped in Result or try/catch with proper UI feedback?
• Coroutine Scopes: Use viewModelScope for ViewModels and lifecycleScope for Fragments/Activities.
• Flow Lifecycle: Is collectAsStateWithLifecycle() used instead of base collectAsState()? (Critical for avoiding resource leaks when the app goes into the background).
• Room TypeConverters: If caching domain models with complex definitions (e.g. LocalDateTime, List<String>), did you verify a @TypeConverter is written and registered on the Database? ROOM WILL CRASH without this.