Been writing compose apps for like a year+ (3 fully compose apps shipped to prod) I've never writte...

# compose

[Colton Idle]

Been writing compose apps for like a year+ (3 fully compose apps shipped to prod)

I've never written @Stable in any of my codebases. 🙈
Am I missing something basic here? Should I be marking classes as stable?

[Siyamed]

🙊🙉🙈

[Sean McQuillan [G]]

You’re good ignoring it. It’s a useful tool for helping hint some skipping, but tbh I’d say you should have evidence of it’s need prior to applying it.

[Colton Idle]

gotcha. From what I picked up in context clues while lurking in #compose over the past year is that classes have to be "stable" in order for compose to be able to recompose properly... but yeah all of my state is typically just

class Foo {
   var fieldA: Boolean by mutableState...
   var fieldB: String by mutableState...
   var fieldC: List<String> by mutableState...
   var fieldD: List<SomeDataClass> by mutableState...
}

and so I assumed that's "stable".

[Landry Norris]

I’m not even sure that it helps. I have a composable that seems to recompose everything, regardless of what actually changed. I marked my state class as Stable, and it still recomposes everything. Tried Immutable, and same thing.

[Colton Idle]

Hm. That seems somewhat problematic? you should file a minimal repro case if possible.

[Bryan Herbst]

Fantastic article to help guide you: https://medium.com/androiddevelopers/jetpack-compose-stability-explained-79c10db270c8

[Sean McQuillan [G]]

To clarify one important point: Recomposition without any stables will be “correct”

[Sean McQuillan [G]]

Adding @Stable is not necessary for correctness, and if used incorrectly it can lead to faults in recomposition

[Sean McQuillan [G]]

It just helps increase skipping, which can improve performance.

[Zoltan Demant]

Your mileage may vary, but I saw pretty great performance improvements by annotating my models with `@Immutable`; especially during animations. I just wish we could specify stability for external classes, but wrapping where needed has been a good alternative for now 🙂