agentskills.codes
CO

compose-state-holder-ui-split

Use when a Jetpack Compose screen-level composable takes a ViewModel/component/controller, collects state or effects, handles navigation/snackbars, or wires callbacks while also rendering layout.

Install

mkdir -p .claude/skills/compose-state-holder-ui-split && curl -L -o skill.zip "https://agentskills.codes/api/skills/download/16133" && unzip -o skill.zip -d .claude/skills/compose-state-holder-ui-split && rm skill.zip

Installs to .claude/skills/compose-state-holder-ui-split

Activation

This is the description your AI agent reads to decide when to run this skill — the better it matches your request, the more reliably it fires.

Use when a Jetpack Compose screen-level composable takes a ViewModel/component/controller, collects state or effects, handles navigation/snackbars, or wires callbacks while also rendering layout.
195 chars✓ has a “when” trigger

About this skill

Compose: state holder/UI split

Core principle

Separate state-holder wiring from UI rendering. The state-holder composable talks to ViewModels, components, flows, navigation, and side effects. The UI composable takes plain immutable UI state plus callbacks and describes layout.

This keeps screens previewable, testable, and easier to reuse across Android, Desktop, TV, and KMP/CMP targets.

When to use this skill

Use this when a Compose screen:

  • Takes a ViewModel, component, controller, navigator, repository, or service directly.
  • Collects app/business state or side effects in the same function that lays out most UI.
  • Passes a whole state holder into child composables instead of explicit state and callbacks.
  • Is hard to preview because it needs dependency injection, navigation, lifecycle, or fake services.
  • Has UI tests that must construct a full app stack to verify a simple layout branch.

The pattern

Use a small public state-holder composable:

@Composable
fun ProfileScreen(component: ProfileComponent, modifier: Modifier = Modifier) {
    val state by component.state.collectAsStateWithLifecycle()

    ProfileScreen(
        state = state,
        onNameChange = component::onNameChange,
        onSaveClick = component::save,
        onBackClick = component::back,
        modifier = modifier,
    )
}

Then put UI in a plain composable that knows nothing about the state holder:

@Composable
fun ProfileScreen(
    state: ProfileUiState,
    onNameChange: (String) -> Unit,
    onSaveClick: () -> Unit,
    onBackClick: () -> Unit,
    modifier: Modifier = Modifier,
) {
    ProfileContent(
        name = state.name,
        isSaving = state.isSaving,
        canSave = state.canSave,
        onNameChange = onNameChange,
        onSaveClick = onSaveClick,
        onBackClick = onBackClick,
        modifier = modifier,
    )
}

Private content functions can break up layout:

@Composable
private fun ProfileContent(
    name: String,
    isSaving: Boolean,
    canSave: Boolean,
    onNameChange: (String) -> Unit,
    onSaveClick: () -> Unit,
    onBackClick: () -> Unit,
    modifier: Modifier = Modifier,
) {
    // Layout only.
}

Rules of thumb

ConcernState-holder composableUI composable
Collect ViewModel/component stateYesNo
Collect one-shot effectsYes, or a tiny sibling effect handlerUsually no
Hold dependency-injected objectsYesNo
Accept immutable UI stateUsually passes it throughYes
Accept lambdas for user eventsWires themCalls them
Own layout, modifiers, semantics, test tagsNo/minimalYes
Own UI-local state like scroll, focus, text input, animation, interactionSometimes seeds itYes
Preview/screenshot friendlyNot necessarilyYes

The "no collection in UI composables" rule is about app/business state and side-effect streams. Plain UI composables can still own UI-local framework state: rememberScrollState, rememberLazyListState, FocusRequester, focus state, animation state, TextFieldState, MutableInteractionSource.collectIsPressedAsState(), and similar behavior that belongs to the rendered widget.

If that UI-local state grows into coordinated behavior with multiple related fields and operations, use compose-state-hoisting to decide whether it should become a plain state holder class remembered in composition.

What to pass

Pass the smallest useful UI contract:

  • Prefer a dedicated UiState/State object over many unrelated primitives when the screen has real state.
  • Prefer explicit lambdas (onRetryClick, onItemSelected) over passing a whole component.
  • Keep domain models out of the UI composable if they force business rules into UI. Map to UI models when the UI needs a different shape.
  • Keep navigation as callbacks. The UI composable says "user clicked back", not "navigate to route X".
  • Frame-rate or UI-local values that should not force whole-tree recomposition when they change: prefer provider lambdas and deferred reads per compose-state-deferred-reads.

Side effects

compose-side-effects covers effect APIs (LaunchedEffect, DisposableEffect, SideEffect), keys, cleanup, and rememberUpdatedState.

Handle effects near the state holder, where the effect source and imperative target are both available:

@Composable
fun ProfileScreen(component: ProfileComponent, snackbarHostState: SnackbarHostState) {
    val state by component.state.collectAsStateWithLifecycle()

    LaunchedEffect(component) {
        component.effects.collect { effect ->
            when (effect) {
                ProfileEffect.Saved -> snackbarHostState.showSnackbar("Saved")
            }
        }
    }

    ProfileScreen(state = state, onSaveClick = component::save)
}

If effect handling grows, extract ProfileEffects(component, snackbarHostState) rather than pushing the component into the UI composable.

Common mistakes

MistakeWhy it hurtsFix
fun Screen(viewModel: MyViewModel) contains all layoutHard to preview/test without Android lifecycle and DIAdd a plain UI overload that takes state and callbacks
Child composables take componentDependencies leak through the treePass only the state/callbacks that child needs
UI composable launches navigationUI becomes coupled to app routingExpose onBackClick, onItemClick, etc.
UI composable collects app/business flowsCollection lifecycle is hidden in layoutCollect near the state holder and pass values down
UI-local state is hoisted into the state holder for no reasonState holder starts owning layout mechanicsKeep scroll/focus/animation/text-field interaction state in the UI composable when it is only UI behavior
Every tiny composable gets a state-holder overloadToo much ceremonySplit at screen/section boundaries, not every Row

When NOT to apply

  • Tiny one-off composables that already take plain values and callbacks.
  • Design-system primitives such as Button, Card, or ListItem; those should expose slots and modifiers, not state holders.
  • Cases where the state-holder composable would only forward one primitive and add no isolation.

Related

Search skills

Search the agent skills registry