[JEWEL-723] Fix excessive recompositions in SelectableLazyColumn

platform/jewel/decorated-window/api/decorated-window.api

@@ -10,7 +10,7 @@ public abstract interface class com/jetbrains/DesktopActions$Handler {
 	public fun print (Ljava/io/File;)V
 }
 
-public class com/jetbrains/JBR {
+public final class com/jetbrains/JBR {
 	public static fun getApiVersion ()Ljava/lang/String;
 	public static fun getDesktopActions ()Lcom/jetbrains/DesktopActions;
 	public static fun getRoundedCornersManager ()Lcom/jetbrains/RoundedCornersManager;

platform/jewel/foundation/api/foundation.api

@@ -1023,3 +1023,4 @@ public final class org/jetbrains/jewel/foundation/util/JewelLogger$Companion {
 	public final fun getInstance (Ljava/lang/Class;)Lorg/jetbrains/jewel/foundation/util/JewelLogger;
 	public final fun getInstance (Ljava/lang/String;)Lorg/jetbrains/jewel/foundation/util/JewelLogger;
 }
+

platform/jewel/foundation/src/main/kotlin/org/jetbrains/jewel/foundation/lazy/SelectableLazyColumn.kt

@@ -15,9 +15,7 @@ import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.rememberCoroutineScope
-import androidx.compose.runtime.rememberUpdatedState
 import androidx.compose.runtime.setValue
-import androidx.compose.runtime.snapshotFlow
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.focus.FocusDirection
@@ -59,19 +57,20 @@ public fun SelectableLazyColumn(
     content: SelectableLazyListScope.() -> Unit,
 ) {
     val scope = rememberCoroutineScope()
-    val container = SelectableLazyListScopeContainer().apply(content)
+    val container = remember(content) { SelectableLazyListScopeContainer().apply(content) }
 
     val keys = remember(container) { container.getKeys() }
     var isFocused by remember { mutableStateOf(false) }
 
-    val latestOnSelectedIndexesChanged = rememberUpdatedState(onSelectedIndexesChange)
-    LaunchedEffect(state, container) {
-        snapshotFlow { state.selectedKeys }
-            .collect { selectedKeys ->
-                val indices = selectedKeys.mapNotNull { key -> container.getKeyIndex(key) }
-                latestOnSelectedIndexesChanged.value.invoke(indices)
-            }
+    var lastSelectedKeys by remember { mutableStateOf(state.selectedKeys) }
+    LaunchedEffect(state.selectedKeys, onSelectedIndexesChange, container) {
+        if (lastSelectedKeys == state.selectedKeys) return@LaunchedEffect
+
+        val indices = state.selectedKeys.mapNotNull { key -> container.getKeyIndex(key) }
+        lastSelectedKeys = state.selectedKeys
+        onSelectedIndexesChange(indices)
     }
+
     val focusManager = LocalFocusManager.current
     val focusRequester = remember { FocusRequester() }
     LazyColumn(

platform/jewel/samples/ide-plugin/src/main/kotlin/org/jetbrains/jewel/samples/ideplugin/releasessample/ContentSource.kt

@@ -21,8 +21,10 @@ internal abstract class ContentSource<T : ContentItem> {
         }
 }
 
-internal data class FilteredContentSource<T : ContentItem>(override val items: List<T>, val original: ContentSource<*>) :
-    ContentSource<T>() {
+internal data class FilteredContentSource<T : ContentItem>(
+    override val items: List<T>,
+    val original: ContentSource<*>,
+) : ContentSource<T>() {
     override val displayName: String
         get() = original.displayName
 }

platform/jewel/samples/standalone/src/main/kotlin/org/jetbrains/jewel/samples/standalone/view/component/SplitLayouts.kt

@@ -1,4 +1,5 @@
-// Copyright 2000-2024 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license.
+// Copyright 2000-2024 JetBrains s.r.o. and contributors. Use of this source code is governed by the
+// Apache 2.0 license.
 package org.jetbrains.jewel.samples.standalone.view.component
 
 import androidx.compose.foundation.border