Hey all, I have a query related specifically to th...

# multiplatform

[GurpreetSK]

Hey all, I have a query related specifically to the Voyager ScreenModel library (v1.0.0). Is it a requirement to have the property to be observed as a class property instead of a function? If I have the flow defined as a property it works as expected, however it unexpectedly emits multiple items when defined as a function. Attaching code snippets in thread

[GurpreetSK]

Code emitting single item, as expected:

internal data object HomeScreen : Screen {

    private val requests: Flow<GetRandomRant?> = MutableStateFlow(null)

    @Composable
    override fun Content() {
        val navigator = LocalNavigator.currentOrThrow
        val viewModel: HomeScreenViewModel = getScreenModel()

        val state by viewModel.models.collectAsState()

        Box(
            contentAlignment = Alignment.Center,
            modifier = Modifier.fillMaxSize()
        ) {
            Text(state)
        }
    }
}


internal class HomeScreenViewModel(
    private val repository: RantsRepository
) : ScreenModel {

    val models: StateFlow<Rant> = repository
        .getRandom()
        .retry(3)
        .catch { emit("") }
        .stateIn(screenModelScope, SharingStarted.WhileSubscribed(5000), "")
}

[GurpreetSK]

Code emitting an undefined (but not always same) amount of items:

internal data object HomeScreen : Screen {

    private val requests: Flow<GetRandomRant?> = MutableStateFlow(null)

    @Composable
    override fun Content() {
        val navigator = LocalNavigator.currentOrThrow
        val viewModel: HomeScreenViewModel = getScreenModel()

        val state by viewModel.models().collectAsState()

        Box(
            contentAlignment = Alignment.Center,
            modifier = Modifier.fillMaxSize()
        ) {
            Text(state)
        }
    }
}


internal class HomeScreenViewModel(
    private val repository: RantsRepository
) : ScreenModel {

    fun models(): StateFlow<Rant> = repository
        .getRandom()
        .retry(3)
        .catch { emit("") }
        .stateIn(screenModelScope, SharingStarted.WhileSubscribed(5000), "")
}

[GurpreetSK]

RantsRepository#getRandom() returns a random rant of type string.