Git rid of immutable list in RV adapters

adapter/build.gradle.kts

@@ -8,7 +8,6 @@ android {
 }
 
 dependencies {
-  api(libs.kotlinX.immutable)
   api(libs.androidX.collection)
   api(libs.androidX.paging)
   api(libs.androidX.recyclerView)

adapter/src/main/kotlin/io/goooler/demoapp/adapter/rv/core/IRvAdapter.kt

@@ -9,7 +9,6 @@ import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.StaggeredGridLayoutManager
 import io.goooler.demoapp.adapter.rv.core.ISpanSize.Companion.SPAN_SIZE_FULL
 import java.util.Collections
-import kotlinx.collections.immutable.toImmutableList
 
 /**
  * Keep the same signature as [RecyclerView.Adapter].
@@ -69,12 +68,12 @@ internal interface IRvAdapter<M : IVhModelType> : RecyclerViewAdapter<BindingVie
 
     private val ivdManager = ViewTypeDelegateManager<M>()
 
-    @Suppress("PropertyName", "VariableNaming", "ktlint:standard:backing-property-naming")
-    protected val _list = mutableListOf<M>()
-
     lateinit var adapter: AP
 
-    override val list: List<M> get() = _list.toImmutableList()
+    /**
+     * Have to override this property in [adapter], we don't store any list data in this delegate.
+     */
+    override val list: List<M> get() = adapter.list
 
     override fun onAttachedToRecyclerView(recyclerView: RecyclerView) {
       initManager(ivdManager)
@@ -86,10 +85,7 @@ internal interface IRvAdapter<M : IVhModelType> : RecyclerViewAdapter<BindingVie
       recyclerView.adapter = null
     }
 
-    override operator fun get(position: Int): M = _list.getOrElse(position) {
-      // Override get in adapters as a fallback.
-      adapter[position] ?: error("No such a element in position $position in adapter $adapter.")
-    }
+    override operator fun get(position: Int): M = list[position]
 
     @LayoutRes
     override fun getItemViewType(position: Int): Int = get(position).viewType
@@ -184,13 +180,18 @@ internal interface IMutableRvAdapter<M : IVhModelType> : IRvAdapter<M> {
           AP : IRvBinding<M>,
           AP : RecyclerView.Adapter<BindingViewHolder> {
 
+    private val _list = mutableListOf<M>()
+
     override var list: List<M>
-      get() = super.list
+      // Copy a new list to avoid the original list being modified.
+      get() = _list.toMutableList()
       set(value) {
         _list.clear()
         _list.addAll(flat(value))
       }
 
+    override operator fun get(position: Int): M = _list[position]
+
     override fun addItems(vararg items: Pair<Int, M>) {
       items.forEach { (index, item) ->
         check(index in _list.indices) { "Index $index out of bounds for length ${_list.size}" }

adapter/src/main/kotlin/io/goooler/demoapp/adapter/rv/paging/BaseRvPagingAdapter.kt

@@ -26,15 +26,20 @@ abstract class BaseRvPagingAdapter<M : IDiffVhModelType> private constructor(
   IRvBinding<M>,
   IRvAdapter<M> by delegate {
 
-  var onLoadStatusListener: OnLoadStatusListener? = null
-
-  override val list: List<M> get() = snapshot().items
-
   constructor(callback: DiffCallback<M> = DiffCallback()) : this(callback, IRvAdapter.Impl()) {
     @Suppress("LeakingThis")
     delegate.adapter = this
   }
 
+  var onLoadStatusListener: OnLoadStatusListener? = null
+
+  override val list: List<M> get() = snapshot().items
+
+  override operator fun get(position: Int): M? = getItem(position)
+
+  @LayoutRes
+  override fun getItemViewType(position: Int): Int = getItem(position)?.viewType ?: 0
+
   override fun onAttachedToRecyclerView(recyclerView: RecyclerView) {
     super.onAttachedToRecyclerView(recyclerView)
     delegate.onAttachedToRecyclerView(recyclerView)
@@ -47,11 +52,6 @@ abstract class BaseRvPagingAdapter<M : IDiffVhModelType> private constructor(
     removeLoadStateListener(loadStateListener)
   }
 
-  @LayoutRes
-  override fun getItemViewType(position: Int): Int = getItem(position)?.viewType ?: 0
-
-  override operator fun get(position: Int): M? = getItem(position)
-
   private val loadStateListener: (CombinedLoadStates) -> Unit = {
     when {
       it.refresh is LoadState.Loading -> onLoadStatusListener?.onRefresh()