refactor: corrige violações de arquitetura (PopcornGP)

config/detekt/popcorngp-config.json

@@ -4,14 +4,13 @@
     {
       "filePattern": "^.*/data/.+\\.kt$",
       "forbiddenDependencies": [
-        "^.*/domain/.+\\.kt$",
-        "^.*/presentation/.+\\.kt$"
+        ".*\\.presentation\\..+"
       ]
     },
     {
       "filePattern": "^.*/domain/.+\\.kt$",
       "forbiddenDependencies": [
-        "^.*/presentation/.+\\.kt$"
+        ".*\\.presentation\\..+"
       ]
     },
     {

feature-detail/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/detail/data/DetailStreamRepository.kt → feature-detail/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/detail/data/DetailStreamRepositoryImpl.kt

@@ -3,21 +3,14 @@ package com.codandotv.streamplayerapp.feature.detail.data
 import com.codandotv.streamplayerapp.core.local.storage.data.dao.FavoriteDao
 import com.codandotv.streamplayerapp.core.networking.handleError.toFlow
 import com.codandotv.streamplayerapp.feature.detail.domain.DetailStream
+import com.codandotv.streamplayerapp.feature.detail.domain.DetailStreamRepository
 import com.codandotv.streamplayerapp.feature.detail.domain.VideoStream
 import com.codandotv.streamplayerapp.feature.detail.domain.toDetailStream
 import com.codandotv.streamplayerapp.feature.detail.domain.toDetailStreamLocal
 import com.codandotv.streamplayerapp.feature.detail.domain.toVideoStreams
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.map
 
-interface DetailStreamRepository {
-    suspend fun getMovie(): Flow<DetailStream>
-    suspend fun deleteFromMyList(movie: String)
-    suspend fun insertToMyList(movie: DetailStream)
-    suspend fun isFavorite(movieId: String): Boolean
-    suspend fun getVideoStreams(): Flow<List<VideoStream>>
-}
-
 class DetailStreamRepositoryImpl(
     private val movieId: String,
     private val service: DetailStreamService,

feature-detail/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/detail/domain/DetailStreamRepository.kt

@@ -0,0 +1,11 @@
+package com.codandotv.streamplayerapp.feature.detail.domain
+
+import kotlinx.coroutines.flow.Flow
+
+interface DetailStreamRepository {
+    suspend fun getMovie(): Flow<DetailStream>
+    suspend fun deleteFromMyList(movie: String)
+    suspend fun insertToMyList(movie: DetailStream)
+    suspend fun isFavorite(movieId: String): Boolean
+    suspend fun getVideoStreams(): Flow<List<VideoStream>>
+}

feature-profile/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/profile/data/ProfilePickerStreamRepository.kt → feature-profile/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/profile/data/ProfilePickerStreamRepositoryImpl.kt

@@ -2,23 +2,20 @@ package com.codandotv.streamplayerapp.feature.profile.data
 
 import com.codandotv.streamplayerapp.core.networking.handleError.toFlow
 import com.codandotv.streamplayerapp.core.networking.handleError.toResult
+import com.codandotv.streamplayerapp.feature.profile.domain.ProfilePickerStreamRepository
 import com.codandotv.streamplayerapp.feature.profile.domain.ProfileStream
 import com.codandotv.streamplayerapp.feature.profile.domain.toProfiles
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.flowOf
 import kotlinx.coroutines.flow.map
 import org.koin.core.annotation.Factory
 
-interface ProfilePickerStreamRepository {
-    suspend fun getProfiles(): Flow<List<com.codandotv.streamplayerapp.feature.profile.domain.ProfileStream>>
-}
-
 @Factory
 class ProfilePickerStreamRepositoryImpl(
     private val service: ProfilePickerStreamService
 ) : ProfilePickerStreamRepository {
 
-    override suspend fun getProfiles(): Flow<List<com.codandotv.streamplayerapp.feature.profile.domain.ProfileStream>> {
+    override suspend fun getProfiles(): Flow<List<ProfileStream>> {
         with(service.getProfiles()) {
             if (this.toResult().isFailure || this.toResult().getOrNull() == null) {
                 return flowOf(mockProfiles)

feature-profile/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/profile/domain/ProfilePickerStreamRepository.kt

@@ -0,0 +1,7 @@
+package com.codandotv.streamplayerapp.feature.profile.domain
+
+import kotlinx.coroutines.flow.Flow
+
+interface ProfilePickerStreamRepository {
+    suspend fun getProfiles(): Flow<List<ProfileStream>>
+}

feature-search/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/search/domain/mapper/SearchMapper.kt → feature-search/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/search/presentation/mapper/SearchMapper.kt

@@ -1,4 +1,4 @@
-package com.codandotv.streamplayerapp.feature.search.domain.mapper
+package com.codandotv.streamplayerapp.feature.search.presentation.mapper
 
 import com.codandotv.streamplayerapp.core.shared.Url
 import com.codandotv.streamplayerapp.feature.search.data.model.ListSearchStreamResponse

feature-search/src/commonMain/kotlin/com/codandotv/streamplayerapp/feature/search/presentation/screens/SearchScreen.kt

@@ -21,7 +21,7 @@ import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
 import androidx.navigation.NavController
 import com.codandotv.streamplayerapp.core.navigation.extensions.goBack
-import com.codandotv.streamplayerapp.feature.search.domain.mapper.toSearchStreamCardModel
+import com.codandotv.streamplayerapp.feature.search.presentation.mapper.toSearchStreamCardModel
 import com.codandotv.streamplayerapp.feature.search.presentation.widgets.SearchStreamCard
 import com.codandotv.streamplayerapp.feature.search.presentation.widgets.SearchableTopBar
 import com.codandotv.streamplayerapp.feature.search.presentation.widgets.StreamsEmpty

gradle/libs.versions.toml

@@ -25,7 +25,7 @@ cameraCore = "1.4.2"
 moko-permissions = "0.19.1"
 
 #Test
-popcornguineapigDetektRule = "1.0.0"
+popcornguineapigDetektRule = "1.0.1"
 test_junit = "4.13.2"
 androidx_core_testing = "2.2.0"
 mockk = "1.13.7"