Android architecture
Skill GuillemRoca/agent-skills-android/skills/android-architecture
Production-grade engineering skills for AI coding agents tailored to Android
npx -y skills add GuillemRoca/agent-skills-android --skill android-architectureAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 2 stars2 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Use when designing module structure, choosing architecture patterns (MVVM/MVI), setting up dependency injection, or defining layer boundaries in an Android project.
SKILL.md
7.5 KB, as published. Nobody here has run it
Android Architecture
Overview
Good architecture makes Android apps testable, maintainable, and scalable. This skill guides architectural decisions: module structure, layer separation (UI → Domain → Data), dependency injection with Hilt, state management with Coroutines and Flow, and patterns that survive configuration changes and process death.
When to Use
- Starting a new Android project or module
- Adding a feature module to an existing multi-module project
- Choosing between MVVM and MVI for a screen
- Setting up or refactoring dependency injection
- Defining boundaries between layers
- Debugging lifecycle or state management issues
Skip when: Making a small change within an already-architected feature.
Core Process
Step 1: Module Structure
- Organize by layer and feature:
:app # Application, navigation, DI entry point
:feature:home # Home screen (UI + ViewModel)
:feature:profile # Profile screen (UI + ViewModel)
:feature:settings # Settings screen (UI + ViewModel)
:core:data # Repositories, data sources, API services
:core:domain # Use cases, domain models (pure Kotlin)
:core:database # Room database, entities, DAOs
:core:network # Retrofit services, API models, interceptors
:core:ui # Shared composables, theme, design tokens
:core:common # Utilities, extensions, constants
:core:testing # Shared test utilities, fakes
- Dependency rules:
:feature:*depends on:core:domain,:core:ui,:core:common:core:datadepends on:core:domain,:core:database,:core:network:core:domaindepends on nothing (pure Kotlin module):appdepends on all:feature:*and:core:data(for DI wiring)- Never: feature → feature, core → feature, domain → data
Step 2: Layer Separation
- Three layers, strict boundaries:
| Layer | Contains | Depends On |
|---|---|---|
| UI | Composables, ViewModels, UiState | Domain |
| Domain | Use cases, domain models, repository interfaces | Nothing |
| Data | Repository implementations, data sources, mappers | Domain (interfaces) |
- Data flows one direction: UI ← Domain ← Data
// Domain layer: pure Kotlin, no Android dependencies
interface TaskRepository {
fun getTasks(): Flow<List<Task>>
suspend fun addTask(task: Task)
}
data class Task(val id: String, val title: String, val completed: Boolean)
class GetTasksUseCase @Inject constructor(
private val repository: TaskRepository
) {
operator fun invoke(): Flow<List<Task>> = repository.getTasks()
}
Step 3: ViewModel and State Management
- MVVM with UiState pattern:
// Sealed interface for exhaustive state handling
sealed interface TaskListUiState {
data object Loading : TaskListUiState
data class Success(val tasks: List<Task>) : TaskListUiState
data class Error(val message: String) : TaskListUiState
}
@HiltViewModel
class TaskListViewModel @Inject constructor(
private val getTasks: GetTasksUseCase
) : ViewModel() {
val uiState: StateFlow<TaskListUiState> = getTasks()
.map<List<Task>, TaskListUiState> { TaskListUiState.Success(it) }
.catch { emit(TaskListUiState.Error(it.message ?: "Unknown error")) }
.stateIn(
scope = viewModelScope,
started = SharingStarted.WhileSubscribed(5_000),
initialValue = TaskListUiState.Loading
)
}
- MVI for complex screens (multiple user actions, side effects):
// Events from UI → ViewModel
sealed interface TaskListEvent {
data class ToggleTask(val taskId: String) : TaskListEvent
data class DeleteTask(val taskId: String) : TaskListEvent
data object RefreshTasks : TaskListEvent
}
// One-time effects (navigation, snackbar)
sealed interface TaskListEffect {
data class ShowSnackbar(val message: String) : TaskListEffect
data class NavigateToDetail(val taskId: String) : TaskListEffect
}
Step 4: Dependency Injection with Hilt
- Module setup:
// In :core:data
@Module
@InstallIn(SingletonComponent::class)
abstract class DataModule {
@Binds
abstract fun bindTaskRepository(impl: TaskRepositoryImpl): TaskRepository
}
// In :core:network
@Module
@InstallIn(SingletonComponent::class)
object NetworkModule {
@Provides
@Singleton
fun provideRetrofit(): Retrofit = Retrofit.Builder()
.baseUrl(BuildConfig.API_BASE_URL)
.addConverterFactory(Json.asConverterFactory("application/json".toMediaType()))
.build()
}
- Hilt rules:
- Use
@Inject constructor(not field injection) - Scope to the narrowest lifecycle:
@Singletononly for truly app-wide instances - Use
@Bindsfor interface→implementation,@Providesfor third-party classes - Test with
@UninstallModules+@BindValueor Hilt test rules
- Use
Step 5: Coroutines and Flow
- Structured concurrency:
// Use viewModelScope — never GlobalScope
viewModelScope.launch {
repository.syncTasks()
}
// Use withContext for thread switching
suspend fun readFromDisk(): Data = withContext(Dispatchers.IO) {
file.readText().parseData()
}
// Expose Flow from repositories, collect in ViewModels
fun getTasks(): Flow<List<Task>> = taskDao.observeAll()
.map { entities -> entities.map { it.toDomain() } }
- Error handling:
// Catch in the ViewModel, not in the repository
viewModelScope.launch {
try {
repository.syncTasks()
} catch (e: IOException) {
_uiState.update { it.copy(error = "Network unavailable") }
}
}
Common Rationalizations
| Shortcut | Why It Fails |
|---|---|
| "We don't need modules, it's a small app" | Modularization pays off at ~10 screens. Build times and code coupling grow fast. |
| "Use cases are unnecessary boilerplate" | Use cases make ViewModels testable without mocking repositories. They also compose. |
| "I'll put the logic in the ViewModel" | Fat ViewModels become untestable. Domain logic belongs in use cases. |
| "Hilt is too complex, I'll use manual DI" | Manual DI doesn't survive multi-module projects. Hilt scales; manual DI doesn't. |
| "GlobalScope is fine for fire-and-forget" | GlobalScope survives activity destruction, leaking resources. Use structured concurrency. |
Red Flags
- Feature module depends on another feature module
- Domain layer imports Android framework classes
- ViewModel directly accesses Room DAOs or Retrofit services
GlobalScope.launchanywhere in production code- Field injection (
@Inject lateinit var) instead of constructor injection - Mutable state exposed from ViewModel (
MutableStateFlowmade public) - Business logic in Composables
Verification
- Module dependency graph follows layer rules (no upward dependencies)
- Domain layer has zero Android dependencies
- ViewModels expose
StateFlow, notMutableStateFloworLiveData - All dependencies provided via Hilt constructor injection
- No
GlobalScopeusage in production code - Repositories return
Flowfor observable data - Use cases are independently testable
-
./gradlew buildsucceeds with no circular dependency errors