Can this skill help with Kotlin Coroutines and Flow?

Absolutely. It checks for proper coroutine cancellation, the use of appropriate scopes like viewModelScope, and the correct collection of Flows using repeatOnLifecycle.

What critical issues does this Android code review skill detect?

The skill focuses on high-impact issues including memory leaks (Context/View references in ViewModels), lifecycle violations (incorrect lifecycle owners), coroutine leaks (GlobalScope usage), and architectural violations in MVVM/MVI patterns.

Why is 'viewLifecycleOwner' checked in Fragments?

Using 'this' instead of 'viewLifecycleOwner' in Fragments is a common source of memory leaks and crashes because the Fragment's lifecycle often outlives its View's lifecycle.

Does it support both MVVM and MVI patterns?

Yes, it is designed to enforce best practices for both MVVM and MVI, ensuring unidirectional data flow, immutable UIState, and proper business logic placement in UseCases.

Can I use this for general code formatting?

While it checks naming conventions, this skill is specifically optimized for high-level architectural and stability issues. Standard formatting and linting are usually better handled by tools like Ktlint or Detekt.

Android Code Reviewer

Name: Android Code Reviewer
Author: daispacy

bydaispacy

Mobile Development

Analyzes Android Kotlin code to identify critical memory leaks, architecture violations, and lifecycle mismanagement.

Acerca de

The Android Code Review skill acts as a senior mobile architect for your development workflow, focusing specifically on high-impact issues that compromise app stability and scalability. It meticulously audits Kotlin and Java files for lifecycle mismanagement, memory leaks in ViewModels, and improper business logic placement across MVVM or MVI patterns. By highlighting critical architectural flaws and providing actionable refactoring examples, it ensures your mobile application adheres to modern industry standards and remains performant under pressure.

Características Principales

UIState Pattern Validation: Checks for immutable, single-source-of-truth state management using StateFlow.
Lifecycle & Coroutine Auditing: Verifies proper use of viewModelScope and viewLifecycleOwner to prevent crashes.
Critical Memory Leak Detection: Scans for Activity/Context references in ViewModels and long-lived objects.
0 GitHub stars
Naming Convention Checks: Enforces PascalCase for types and prefixed Booleans for improved readability.
Architecture Enforcement: Ensures strict separation of concerns between ViewModels, UseCases, and Repositories.

Casos de Uso

Refactoring legacy Android code to modern MVI/MVVM patterns with centralized UIState.
Pre-merge Pull Request reviews to catch high-priority architectural violations before they reach production.
Debugging complex performance issues related to unhandled coroutine cancellation or resource leaks.

Acerca de

Características Principales

UIState Pattern Validation: Checks for immutable, single-source-of-truth state management using StateFlow.
Lifecycle & Coroutine Auditing: Verifies proper use of viewModelScope and viewLifecycleOwner to prevent crashes.
Critical Memory Leak Detection: Scans for Activity/Context references in ViewModels and long-lived objects.
0 GitHub stars
Naming Convention Checks: Enforces PascalCase for types and prefixed Booleans for improved readability.
Architecture Enforcement: Ensures strict separation of concerns between ViewModels, UseCases, and Repositories.

Casos de Uso

Refactoring legacy Android code to modern MVI/MVVM patterns with centralized UIState.
Pre-merge Pull Request reviews to catch high-priority architectural violations before they reach production.
Debugging complex performance issues related to unhandled coroutine cancellation or resource leaks.