Refactor the way Coroutines "health" is checked because if a CancellationException is thrown out of the catch block or the ViewModel is told to cancel its scope as soon as it is detected/thrown, code won't be able to return any kind of error to upper layers and that will lead to gray area where the app goes to a zombie state and UI will "forever" wait for a result that never comes

The information that lead to this second change is contained in this article: https://betterprogramming.pub/the-silent-killer-thats-crashing-your-coroutines-9171d1e8f79b

The point is that the examples shown in the article go for a more generic approach and do not reflect how Coroutine scope generation works on Android where the Scope owner is not in most cases in the same file where a CancellationException will likely be thrown so devs have to find a way around in order to tell the Scope owner which usually is ViewModel to check if its scope is still active or not

The way I found to be the better one in my specific case is to instead of letting CancellationExceptions out of catch blocks or warn ViewModels to check on their Coroutine scopes and possibly cancel them if it is found they are not active anymore leading the app to a "Zombie" state, is to check the scope status at the end of every operation.

This way, the UI is warned beforehand and if indeed the scope in question is not active anymore, it is cancelled before another operation is performed under it.

According to Roman Elizarov's comments in the open issue for this matter, it will probably never be fixed due to its high complexity where a new implementation of Coroutines from scratch would almost certainly be required to fix it, which will most likey never happen. So, there's that. Let's enjoy Coroutines as it is :)

More info on it can be found here and I highly recommend you take a look at it: Kotlin/kotlinx.coroutines#1374

Move away from the "By feature + By layer" modularization approach in favor of the "By feature" only approach

Create domain module

Create di module

Create resources module

Create utils module

Delete unused XML layouts

Delete unused ViewHolders

Create specific modules for JVM and Instrumented testing so one cannot access the other's dependencies

Rename midfield module to domain

Convert all module-level build.gradle files to KTS

Remove targetSdk field from build.gradle files as it is now deprecated

Add Koin lazy modules feature

Add Koin startup feature

Migrate to Kotlin 2.1.0

Migrate from packagingOption to packaging as it is now deprecated

Migrate default build.gradle JVM Versioning approach to jvmToolchain

Upgrade to Coil3

Upgrade to Ktor 3.0

Upgrade Gradle to 8.12

Refactor layout ids

Disable "allowBackup" manifest option as database changes will always require migrations even if all app data is removed through the app details screen

Remove empty/unused manifest files

Split ProfileAggregator into individual UseCases

Refactor gradlew script

Add "orderingId" field to Profile table so now when data is retrieved from Database it accurately reflects how data is ordered in Github API since it does not return individual profile scores anymore

Refactor libs.versions.toml

Re-add INSERT to available SQL commands

Refactor the way remote repositories propagate their results so now there is a callback for each type of result and there is no need to check for success or failure more than once

Move ResultHandler to Data layer

Change HTTP codes inside RemoteFetcher as search quota reached error code is not 451 anymore but 403 (Forbidden)

Add checks to UIState so state checking in the UI is more concise

Update dependency versions

Author: caiodev

README.md

@@ -1,4 +1,4 @@
-[![Codacy Badge](https://app.codacy.com/project/badge/Grade/355f2c30bb5746cc8083b0bc684c7650)](https://www.codacy.com/gh/caiodev/GithubProfileSearcher/dashboard?utm_source=github.com&utm_medium=referral&utm_content=caiodev/GithubProfileSearcher&utm_campaign=Badge_Grade)  [![codebeat badge](https://codebeat.co/badges/59a0827f-9c7e-44a3-999b-86ab4662e128)](https://codebeat.co/projects/github-com-caiodev-githubprofilesearcher-edge___often_broken___)
+[![codebeat badge](https://codebeat.co/badges/59a0827f-9c7e-44a3-999b-86ab4662e128)](https://codebeat.co/projects/github-com-caiodev-githubprofilesearcher-edge___often_broken___)  [![CodeFactor](https://www.codefactor.io/repository/github/caiodev/githubprofilesearcher/badge/edge___often_broken___)](https://www.codefactor.io/repository/github/caiodev/githubprofilesearcher/overview/edge___often_broken___)
 
 # GithubProfileSearcher
 

app/build.gradle

@@ -0,0 +1,82 @@
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget
+
+plugins {
+    alias(dep.plugins.androidApplication)
+    alias(dep.plugins.kotlinPlugin)
+}
+
+apply {
+    from("$rootDir/flavors/flavors.gradle")
+}
+
+android {
+    compileSdk =
+        dep.versions.compileSdk
+            .get()
+            .toInt()
+    defaultConfig {
+        applicationId = "githubprofilesearcher.caiodev.com.br.githubprofilesearcher.app"
+        minSdk =
+            dep.versions.minSdk
+                .get()
+                .toInt()
+        versionCode =
+            dep.versions.versionCode
+                .get()
+                .toInt()
+        versionName = dep.versions.versionName.get()
+        vectorDrawables.useSupportLibrary = true
+    }
+
+    buildTypes {
+        debug {
+            applicationIdSuffix = ".debug"
+            isMinifyEnabled = false
+            isShrinkResources = false
+            signingConfig = signingConfigs.getByName("debug")
+        }
+
+        release {
+            applicationIdSuffix = ".release"
+            isMinifyEnabled = true
+            isShrinkResources = true
+            proguardFiles(
+                getDefaultProguardFile("proguard-android.txt"),
+                "proguard-rules.pro",
+            )
+        }
+    }
+
+    kotlin {
+        jvmToolchain(JvmTarget.JVM_23.target.toInt())
+    }
+
+    packaging {
+        resources {
+            excludes +=
+                setOf(
+                    "/META-INF/{AL2.0,LGPL2.1,LICENSE.md,LICENSE-notice.md,licenses/ASM}",
+                    "win32-x86{/attach_hotspot_windows.dll,-64/attach_hotspot_windows.dll}",
+                    "kotlin{/annotation/annotation,/collections/collections,/coroutines/coroutines," +
+                        "/internal/internal,/kotlin,/ranges/ranges,/reflect/reflect}.kotlin_builtins",
+                )
+        }
+    }
+
+    namespace = "githubprofilesearcher.caiodev.com.br.githubprofilesearcher.app"
+}
+
+dependencies {
+    debugImplementation(dep.leakCanary)
+    implementation(projects.modules.common.di) { because("DI Module") }
+    implementation(projects.modules.common.resources) { because("Resources Module") }
+    implementation(projects.modules.features.profile) { because("Profile Module") }
+    implementation(dep.bundles.coil)
+}
+
+tasks.withType<Test> {
+    systemProperty("includeAndroidResources", "true")
+    useJUnitPlatform()
+}
+
+group = "githubprofilesearcher.caiodev.com.br.githubprofilesearcher.app"

app/build.gradle.kts

@@ -2,7 +2,7 @@
 # By default, the flags in this file are appended to flags specified
 # in /home/unknown/Android/Sdk/tools/proguard/proguard-android.txt
 # You can edit the include path and order by changing the proguardFiles
-# directive in build.gradle.
+# directive in build.gradle.kts.
 #
 # For more details, see
 #   http://developer.android.com/guide/developing/tools/proguard.html

app/proguard-rules.pro

@@ -4,23 +4,22 @@
 
     <application
         android:name=".App"
-        android:allowBackup="true"
+        android:allowBackup="false"
         android:icon="@mipmap/ic_launcher"
         android:label="@string/app_name"
         android:localeConfig="@xml/locales_config"
         android:networkSecurityConfig="@xml/network_security_config"
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
         android:theme="@style/AppTheme"
-        tools:ignore="AllowBackup"
         tools:targetApi="tiramisu">
 
         <profileable
             android:shell="true"
             tools:targetApi="q" />
 
         <activity
-            android:name=".ui.profile.activity.ProfileListingActivity"
+            android:name="githubprofilesearcher.caiodev.com.br.githubprofilesearcher.profile.ui.activity.ProfileListingActivity"
             android:configChanges="uiMode"
             android:exported="true">
             <intent-filter>

app/src/main/AndroidManifest.xml

@@ -0,0 +1,73 @@
+package githubprofilesearcher.caiodev.com.br.githubprofilesearcher.app
+
+import android.app.Application
+import androidx.appcompat.content.res.AppCompatResources
+import coil3.ImageLoader
+import coil3.PlatformContext
+import coil3.SingletonImageLoader
+import coil3.asImage
+import coil3.disk.DiskCache
+import coil3.memory.MemoryCache
+import coil3.network.ktor3.KtorNetworkFetcherFactory
+import coil3.request.crossfade
+import coil3.util.DebugLogger
+import githubprofilesearcher.caiodev.com.br.githubprofilesearcher.profile.di.profileModule
+import okio.Path.Companion.toOkioPath
+import org.koin.android.ext.koin.androidContext
+import org.koin.android.ext.koin.androidLogger
+import org.koin.androix.startup.KoinStartup
+import org.koin.core.annotation.KoinExperimentalAPI
+import org.koin.core.lazyModules
+import org.koin.core.logger.Level
+import org.koin.dsl.KoinAppDeclaration
+import githubprofilesearcher.caiodev.com.br.githubprofilesearcher.resources.R as Resources
+
+@OptIn(KoinExperimentalAPI::class)
+class App :
+    Application(),
+    KoinStartup,
+    SingletonImageLoader.Factory {
+    override fun onKoinStartup(): KoinAppDeclaration =
+        {
+            androidContext(this@App)
+            androidLogger(Level.DEBUG)
+            lazyModules(profileModule)
+        }
+
+    override fun newImageLoader(context: PlatformContext): ImageLoader {
+        val defaultImage =
+            AppCompatResources
+                .getDrawable(
+                    applicationContext,
+                    Resources.mipmap.ic_launcher,
+                )
+                ?.asImage()
+        return ImageLoader
+            .Builder(context)
+            .components { add(KtorNetworkFetcherFactory()) }
+            .crossfade(true)
+            .diskCache {
+                DiskCache
+                    .Builder()
+                    .directory(cacheDir.resolve(COIL_CACHE_DIR).toOkioPath())
+                    .maxSizePercent(DISK_CACHE_CAP)
+                    .build()
+            }
+            .memoryCache {
+                MemoryCache
+                    .Builder()
+                    .maxSizePercent(context, MEMORY_CACHE_CAP)
+                    .build()
+            }
+            .placeholder(defaultImage)
+            .error(defaultImage)
+            .logger(DebugLogger())
+            .build()
+    }
+
+    companion object {
+        private const val COIL_CACHE_DIR = "coil_image"
+        private const val DISK_CACHE_CAP = .02
+        private const val MEMORY_CACHE_CAP = .05
+    }
+}