feat: bootstrap hilt wiring

[XaaT]

Action par GPT-5 Codex (demandée par @XaaT)

Closes #50

Summary

• ajoute le bootstrap Hilt minimal côté app et :core:data
• introduit RedfaceApplication avec @HiltAndroidApp
• marque MainActivity en @AndroidEntryPoint pour préparer l’injection future côté navigation / ViewModel
• ajoute des bindings de base dans :core:data (Clock, Dispatchers.IO, Dispatchers.Default)
• extrait le wiring Hilt/KSP dans build-logic via deux convention plugins dédiés

Notes

• scope volontairement minimal : pas de modules Hilt par feature, pas de faux repositories, pas de @HiltViewModel tant qu’aucun ViewModel réel n’existe
• bump Hilt vers 2.59.2 dans le version catalog : c’est la première ligne stable qui supporte AGP 9, déjà utilisé par le repo
• le premier point d’injection utile est l’application ; l’activité est rendue Hilt-ready pour les prochains écrans stateful

Validation run

• docker run --rm --security-opt label=disable -v /work/xaat/redface2:/workspace -w /workspace ghcr.io/cirruslabs/android-sdk:36@sha256:f9b3ea9ed2b5fc9522adae82c7b4622ab7aa54207ef532c8e615a347dca08f31 ./gradlew :app:assembleDebug
• docker run --rm --security-opt label=disable -v /work/xaat/redface2:/workspace -w /workspace ghcr.io/cirruslabs/android-sdk:36@sha256:f9b3ea9ed2b5fc9522adae82c7b4622ab7aa54207ef532c8e615a347dca08f31 ./gradlew detektAll lintDebug testDebugUnitTest :app:assembleDebug

Sources

• https://dagger.dev/hilt/gradle-setup
• https://developer.android.com/training/dependency-injection/hilt-android
• Context7 /websites/dagger_dev_hilt
• https://github.com/google/dagger/releases/tag/dagger-2.59

[XaaT]

Action par Claude Opus 4.7 (1M context) (demandée par @XaaT)

Peer review — 4 findings avant merge

[Haute] DispatchersQualifiers.kt placé dans :core:data

core/data/src/main/java/fr/forumhfr/redface2/core/data/di/DispatchersQualifiers.kt

Dès qu'un ViewModel d'une feature voudra @IoDispatcher ou @DefaultDispatcher, il devra importer depuis fr.forumhfr.redface2.core.data.di.* — or le test Konsist ArchitectureKonsistTest.kt a fr.forumhfr.redface2.core.data. dans forbiddenImplementationImports. La CI cassera sur la première feature qui injecte un dispatcher.

Fix : déplacer DispatchersQualifiers.kt (et le provider Main quand il sera ajouté, cf. finding suivant) vers :core:domain, qui est l'emplacement naturel (les interfaces de repositories y vivront avec ces signatures).

[Haute] @MainDispatcher manquant

Seulement @IoDispatcher + @DefaultDispatcher déclarés. Le dispatcher Main est canonique sur Android (coordination UI, callbacks de navigation) — son absence forcera des injections ad-hoc plus tard, incohérentes avec le reste.

Fix :

@Qualifier @Retention(AnnotationRetention.BINARY)
annotation class MainDispatcher

Et dans PlatformBindingsModule :

@Provides @Singleton @MainDispatcher
fun provideMainDispatcher(): CoroutineDispatcher = Dispatchers.Main.immediate

Note : Dispatchers.Main.immediate (pas Dispatchers.Main nu) pour éviter la frame de délai parasite côté UI.

[Moyenne] Version Hilt 2.59.2 vs stack verrouillée 2.57

gradle/libs.versions.toml

La stack canonique annonce Hilt 2.57. La PR bump à 2.59.2 sans documentation. Si intentionnel : mettre à jour stack.md + CHANGELOG. Sinon : aligner sur 2.57.

[Moyenne] RedfaceApplication.clock public + smoke test Log.d

app/src/main/java/fr/forumhfr/redface2/RedfaceApplication.kt

@Inject lateinit var clock: Clock est exposé publiquement sur l'Application, et utilisé pour un Log.d qui n'a pas sa place en production. Mauvaise pratique (l'Application n'est pas un point d'injection général) et couplage involontaire.

Fix : passer private (ce qui casse l'injection lateinit var → utiliser un @EntryPoint si vraiment besoin d'accéder au graph depuis l'Application), ou supprimer complètement l'injection et le log de debug.

---

Points forts

• Convention plugins séparés redface.android.hilt.application / .library : opt-in propre, pas d'injection forcée dans tous les modules
• KSP exclusif (pas KAPT) : aligné Kotlin 2.3
• RedfaceApplication : @HiltAndroidApp, hérite Application, référencé dans le manifest — correct
• MainActivity : @AndroidEntryPoint, modifications minimales

Les 2 findings haute priorité sont triviaux à corriger (déplacement de fichier + 1 qualifier). Une fois appliqués, la PR est merge-ready.

[XaaT]

Action par Claude Opus 4.7 (demandée par @XaaT)

Fixes appliqués (commit 1b3cfca) :

• DispatchersQualifiers déplacé de :core:data vers :core:domain/coroutines/
• @MainDispatcher ajouté (provider Dispatchers.Main.immediate)
• javax.inject exposé en api() depuis :core:domain
• RedfaceApplication réduit à @HiltAndroidApp class RedfaceApplication : Application()
• Hilt 2.59.2 documenté dans stack.md + CHANGELOG

Status : CI ✅, en attente de review humaine.

Note workflow : en parallèle, tout a été consolidé sur la branche internal (HEAD d4d53cd) pour continuer le dev et builder l'AAB de test — elle contient PR#57 HEAD + PR#58 HEAD + fix NoActionBar/statusBarsPadding + PR#62. Cette PR reste la voie canonique vers main une fois la review passée.