Stability configuration: Stability of list of stable classes [327643787]

Fixed

Bug

Status Update

No update yet.

Description

pa...@gmail.com

created issue #1

Feb 29, 2024 09:18PM

Versions

Jetpack Compose version: androidx.compose:compose-bom:2023.08.00
Android Studio Build: Android Studio Hedgehog | 2023.1.1 Patch 2
Kotlin version: 1.9.22

Steps to Reproduce or Code Sample to Reproduce:

Create empty Jetpack Compose project
Update Compose Compiler to 1.5.8 and Kotlin version to 1.9.22
Enable Compose Metrics
Make Lists stable by Stability Configuration file
Build the app
Check the composeMetrics

Stability configuration file

kotlin.collections.List

Used `data class`

data class UiState(
    val items: List<Item>,
) {
    data class Item(
        val texts: List<String>,
    )
}

Output stability

stable class Item {
  stable val texts: List<String>
  <runtime stability> = 
}

unstable class UiState {
  unstable val items: List<Item>
  <runtime stability> = Unstable
}

restartable scheme("[androidx.compose.ui.UiComposable]") fun Greeting(
  unstable state: UiState
)

Expected behavior

As the Item is stable thanks to forced List stability I would expect to have the UiState stable as it contains list (stable) of stable classes. Am I missing something?

Comments

as...@google.com <as...@google.com> #2Mar 4, 2024 10:55AM

Reassigned to be...@google.com.

I think hardcoded list of classes might be overriding ones provided by the config. Ben, can you take a look?

be...@google.com <be...@google.com> Mar 5, 2024 12:25AM

Accepted by be...@google.com.

ap...@google.com <ap...@google.com> #3Mar 7, 2024 09:51PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main

commit 3030beac7f68cc08154e86072775a007aaae3b06
Author: Ben Trengrove <bentrengrove@google.com>
Date: Tue Mar 05 11:23:47 2024

Fix nested external generic types being inferred Unstable

currentlyAnalysing didn't contain any information about the typeParameters.
When checking of the members of B, List was returning unstable as List was already
being analyzed. Added type parameter info to the currently analyzing Set to fix this.

Also fixed localSrc not taking into account externalTypes when being compiled in tests.

Fixes: 327643787
Test: testNestedExternalTypesAreStable
Relnote: Fix nested external generic types being inferred Unstable
Change-Id: I3437fad769ba3eee58a299584ac4e352a3173b0f

M compose/compiler/compiler-hosted/integration-tests/src/jvmTest/kotlin/androidx/compose/compiler/plugins/kotlin/ClassStabilityTransformTests.kt
M compose/compiler/compiler-hosted/src/main/java/androidx/compose/compiler/plugins/kotlin/ComposePlugin.kt
M compose/compiler/compiler-hosted/src/main/java/androidx/compose/compiler/plugins/kotlin/analysis/Stability.kt

https://android-review.googlesource.com/2988009

pr...@google.com <pr...@google.com> #4Mar 20, 2024 08:26PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.compose.compiler:compiler-hosted:1.5.11