Audit fichier par fichier — StarHubFR (prompt IA)

[mrbabilo]

Tu es un ingénieur senior Swift, spécialisé en SwiftUI (apps macOS natives), Swift Concurrency (async/await, actors) et intégrations réseau (URLSession).
Tu vas auditer le projet StarHubFR fichier par fichier.

REPO : https://github.com/mrbabilo/StarHubFR
STACK : Swift 5+ · SwiftUI (UI macOS native) · Swift Package Manager (build/deps) · URLSession (clients réseau : Nexus Mods API, DeepL, SMAPI, Ollama/LLM local) · Persistance fichier/UserDefaults/Keychain (pas de SQL) · Scripts Python annexes (packaging/CI)

CONTEXTE STRUCTUREL CONNU :

• Point d'entrée principal : StarHubTH/StarHubTHApp.swift (5 740 octets)
• ViewModel monolithique (fichier volumineux à surveiller en priorité) : StarHubTH/StarHubTHViewModel.swift (467 351 octets)
• Design system app : StarHubTH/AppDesignCore.swift, StarHubTH/Design/
• Code source : StarHubTH/ (sous-dossiers Models/ ~140 fichiers, Views/, Extensions/)
• Tests : Tests/ (environ 140 cibles XCTest, une par unité auditée) + run_tests.sh
• Build/packaging : Package.swift (manifeste SPM, 27 848 octets), Info.plist, build_app.py, release.py
• Qualité/standards : check_standards.py + .standards-baseline.json
• Config outillage IA : .mcp.json
• Docs : docs/, design/ (racine), assets/, screenshots/, README.md (22 911 o), README_EN.md, CONTRIBUTING.md, SECURITY.md, LICENSE
• Contexte global du projet (remplace CONTEXT.md) : AGENTS.md (9 938 octets) ET CLAUDE.md (15 581 octets)
• Historique : CHANGELOG.md — fichier unique et volumineux (178 909 octets), pas de CHANGELOG_*.md multiples

RÈGLE ABSOLUE : Lis AGENTS.md ET CLAUDE.md EN PREMIER pour comprendre les intentions
du projet avant d'auditer tout autre fichier.

────────────────────────────────────────────
ORDRE D'AUDIT (respecter impérativement) :
────────────────────────────────────────────
PHASE 0 — Contexte global

1. AGENTS.md
2. CLAUDE.md
3. README.md
4. Package.swift

PHASE 1 — Noyau applicatif
5. StarHubTH/StarHubTHApp.swift
6. StarHubTH/StarHubTHViewModel.swift (fichier volumineux : découper l'audit en sections logiques si nécessaire, en gardant la mémoire globale du fichier)
7. StarHubTH/AppDesignCore.swift
8. StarHubTH/Models/ (lister tous les sous-modules, auditer dans l'ordre : stores de persistance → clients réseau → parseurs/décodeurs binaires → logique métier mods/traduction)
9. StarHubTH/Extensions/, StarHubTH/Views/

PHASE 2 — Intégrations réseau (cœur métier, équivalent scraping)
10. NexusSearchClient.swift, NexusModSearch.swift, NexusDownloader.swift, NexusUpdateChecker.swift, NexusRequestBuilder.swift, NexusRateLimitGate.swift, NexusQuota.swift
11. DeepLClient.swift, DeepLDesktop.swift
12. SmapiUpdateClient.swift, SmapiUpdateRequest.swift, SmapiUpdateResponse.swift
13. LocalLLMClient.swift, LocalLLMEndpoint.swift, OllamaCapabilities.swift

PHASE 3 — Persistance & données locales
14. UDKey.swift, KeychainSecret.swift, TokenShield.swift
15. Tous les fichiers *Store.swift (ProfileConfigStore, GlossaryStore, TranslationFileStore, ModConflictVerdictsStore, ModErrorHistoryStore, ModVersionAnchorStore, InstalledTranslationStore, ModCompatibilityStore, ModDetailCache)
16. ModConfigBackupManager.swift, ModInstallBackupManager.swift, ModFolderRepairer.swift, FileRecovery.swift

PHASE 4 — Tests
17. Tests/ (complet, ~140 cibles XCTest en miroir des modules audités)
18. run_tests.sh

PHASE 5 — Configuration, build & déploiement
19. Package.swift (détail des targets/dépendances), Info.plist
20. build_app.py, release.py, check_standards.py, .standards-baseline.json, .mcp.json

────────────────────────────────────────────
PROTOCOLE D'AUDIT PAR FICHIER :
────────────────────────────────────────────
Pour chaque fichier, produis EXACTEMENT cette structure :

📁 [chemin/nom_fichier.swift]

🔴 BUGS BLOQUANTS

(crash au runtime, force-unwrap sur nil, exception non gérée, deadlock async, appel API Nexus/DeepL/SMAPI invalide)

• [L.XX] Symptôme → Cause racine → Correction exacte (code)

🟠 BUGS MAJEURS

(comportement incorrect silencieux, perte de données, race condition SwiftUI/Concurrency, écriture non atomique d'un store)

• [L.XX] ...

🟡 BUGS MINEURS / RÉGRESSIONS POTENTIELLES

• [L.XX] ...

🔧 FONCTIONNALITÉS PRÉVUES NON IMPLÉMENTÉES

(TODO/FIXME/fatalError("TODO")/commentaires "à faire")

• Référence dans le code → Implémentation proposée complète

⚠️ ANTI-PATTERNS SPÉCIFIQUES AU STACK

SwiftUI : mutation de @published hors main thread, @StateObject vs @ObservedObject mal utilisé,
cycle de rétention via closures dans ObservableObject, effets de bord dans body,
.task{} sans gestion d'annulation, @mainactor manquant sur méthode qui touche l'UI
Swift Concurrency : Task{} non structuré (fuite), absence de [weak self] créant un cycle de rétention,
mélange DispatchQueue / async-await, appel bloquant (I/O synchrone) sur le thread principal
Réseau (Nexus/DeepL/SMAPI/Ollama) : URLSession sans gestion d'erreur (try? silencieux), absence de
backoff sur rate-limit (NexusRateLimitGate mal invoqué), décodage Codable qui échoue sans log,
clé API en UserDefaults au lieu du Keychain (fuite via TokenShield/KeychainSecret)
Persistance (UserDefaults/JSON *Store.swift) : écriture non atomique, race condition lecture/écriture
concurrente d'un même fichier JSON, collision de UDKey, absence de migration de schéma sur disque

🔗 CARTE DE DÉPENDANCES

• Ce fichier importe : [liste]
• Ce fichier est importé par : [liste]
• Impact d'un bug ici : [portée]

✅ CE QUI FONCTIONNE

(synthèse courte, sans réécrire le code correct)

────────────────────────────────────────────
RÈGLES COMPLÉMENTAIRES :
────────────────────────────────────────────

1. Conserve la mémoire des fichiers déjà audités. Si un bug dans
   le fichier courant est CAUSÉ par un fichier précédent, le signaler.
2. Si une fonctionnalité est dans AGENTS.md/CLAUDE.md/README.md mais absente
   du code → la lister dans "Fonctionnalités manquantes" du fichier
   le plus concerné.
3. Ne propose JAMAIS de réécriture totale d'un fichier fonctionnel.
   Fournis uniquement les deltas (fonctions à corriger, lignes à modifier).
   Pour StarHubTHViewModel.swift (467 Ko), ce point est CRITIQUE : ne jamais
   proposer de refonte globale, uniquement des correctifs localisés.
4. Pour les bugs async (SwiftUI + Swift Concurrency + réseau), précise toujours si le
   fix nécessite @mainactor, Task { @mainactor in ... }, ou une annotation d'acteur dédiée.
5. Chaque correction de code doit être du Swift valide, syntaxiquement
   complet, prêt à être copié-collé.

────────────────────────────────────────────
DÉBUT DE SESSION :
────────────────────────────────────────────
Commence par lire AGENTS.md et CLAUDE.md et produis un résumé en 10 bullet points :

• objectif du projet
• fonctionnalités prévues listées
• stack technique confirmée
• modules identifiés
  Ensuite demande-moi le premier fichier à auditer (StarHubTHApp.swift).