Éditer la recette

Titre

Description **Revue experte du point de vue d'une pianiste exigeante** (joueuse réelle), sur la branche `pivot-piano-recette` (working copy hôte, audit code + théorie). Objectif : garder une TRACE priorisée des pistes d'amélioration UX du pivot guitare → piano. **Méthode** — 4 axes : (1) cohérence produit & style gothique/corbeau, (2) justesse de la THÉORIE MUSICALE (vérifiée à la main sur les data-structures), (3) pertinence/complétude des modules, (4) UX/ergonomie sur les 13 écrans. **Verdict en une ligne** : le MOTEUR musical est solide (gammes, accords, détection de tonalité tous justes), mais le PIVOT piano est inachevé (reliquats guitare visibles dès le 1er lancement) et il manque les fondamentaux d'un outil pianiste (choix d'inversion, main G/D, spelling enharmonique tonal, patterns piano exposés). **Décompte** : 🔴 bloquants ~8 · 🟠 majeurs ~10 · 🟡 mineurs ~8.

Cas d'usage Une pianiste (joueuse de piano réelle et exigeante) découvre puis utilise GrooveGrave de bout en bout : onboarding, dictionnaire d'accords, gammes/modes, composer, impro, feuille de chanson, éditeur, métronome, réglages, paywall. Elle attend une théorie juste, un spelling correct, le choix des inversions/voicings et une séparation main gauche/main droite.

Étapes Audit conduit sur le code de la branche `pivot-piano-recette` : 1. Extraction verbatim des data-structures théoriques (ChordDatabase, ScaleDatabase, PianoVoicingProvider, HarmonyAnalysis, PianoPatterns) puis vérification manuelle des formules d'intervalles, du spelling et des voicings. 2. Recherche exhaustive des reliquats guitare (guitar/fret/string/capo/strum/pick/médiator/tab) et évaluation de la cohérence de l'univers gothique-corbeau. 3. Revue UX écran par écran (Views/) du point de vue pianiste : découvrabilité, libellés, feedback, réglages piano manquants. 4. Priorisation par impact utilisateur (bloquant / majeur / mineur).

Cas de test ## A. THÉORIE MUSICALE — vérifiée à la main > Cross-check binôme : détail théorie par fichier dans la recette **458f5c36** (annexe, par Bill/QA). Résolutions consolidées en bas de cette recette. ### ✅ Ce qui est JUSTE (à valoriser) - **24 gammes / modes** : formules d'intervalles TOUTES exactes — majeure, mineures (naturelle/harmonique/mélodique), 7 modes grecs, pentatoniques, blues, exotiques (phrygien dominant, hongroise mineure, napolitaine, diminuée, par tons, bebop dominante). - **Accords** : triades (maj/min/dim/aug/sus2/sus4/power) + extensions (7, maj7, 9, maj9, 11, 13, add9/11, b5/#5, b9/#9, 6, 6/9, 7sus) — toutes les formules exactes. Renversements corrects (note grave +12). - **Détection de tonalité** : profils Krumhansl-Kessler 1982 (valeurs exactes) + corrélation de Pearson — implémentation sophistiquée et correcte. - **Patterns piano** (PianoPatterns.swift) : basse d'Alberti [0,2,1,2], oom-pah, valse 3/4, charleston, ballade, arpèges up/down/brisé — musicalement justes. ### 🔴 CRITIQUE — Spelling enharmonique NON tonal `ChordDatabase.normalizeChordName` force **tous les bémols en dièses** (Db→C#, Eb→D#, Gb→F#, Ab→G#, Bb→A#) et l'affichage repose sur des pitch-classes entières (0–11). - **Constat** : un accord de **Mi♭ mineur** s'affiche **Ré#–Fa#–La#** au lieu de **Mi♭–Sol♭–Si♭** ; les gammes en tonalités bémols (Ré♭, Mi♭, La♭…) sont mal orthographiées (dièses au lieu de bémols, risque de lettres répétées type Fa/Fa#). - **Sévérité** : 🔴 Bloquant pour une pianiste (lecture de partition faussée). - **Amélioration** : spelling **key-aware** — choisir #/♭ selon la tonalité/le degré (cercle des quintes), pas une normalisation globale en dièses. Exposer aussi un toggle enharmonique manuel. ### 🟠 MAJEUR — Familles d'accords ø et °7 pas first-class - **Constat** : `m7♭5` (demi-diminué ø) et `dim7` (°7) doivent être composés ; un dim7 obtenu via `dim`+`6` s'afficherait **« dim6 »** (label théorique faux). - **Amélioration** : ajouter half-diminished (ø / m7♭5) et diminished 7th (°7) comme qualités/labels de premier ordre dans ChordQuality + le parser. ### 🟠 MAJEUR — Degrés romains en tonalité mineure - **Constat** : `minorRomanLabels` note les degrés diatoniques **♭III / ♭VI / ♭VII** ; or le bémol n'est correct que par rapport au MAJEUR parallèle. En analyse d'une tonalité mineure on écrit **III / VI / VII** (sans bémol). - **Amélioration** : table de degrés dépendante du mode (majeur vs mineur), sans préfixe ♭ pour les degrés diatoniques du mineur. ### 🟡 MINEUR - Analyse harmonique limitée à **major/minor** (pas de modal) alors que l'app propose 7 modes → écart de complétude. - Gamme **« Chinoise » [0,4,6,7,11]** = invalide/obscure → remplacer par une pentatonique **gong [0,2,4,7,9]** ou **yu [0,3,5,7,10]**. --- ## B. COHÉRENCE PRODUIT & RELIQUATS GUITARE ### 🔴 Bloquant (visible utilisateur) - **Chanson par défaut = « Le Squelette Guitariste » / « The Skeleton Guitarist »** avec paroles guitare (« Je gratte ma guitare »…) — `AppFirstLaunch.swift` l.27-162. **Vue à CHAQUE premier lancement.** → Réécrire en « Le Squelette Pianiste » + paroles piano. - **Onboarding guitare** : « L'app ultime pour les guitaristes » (`LocalizationManager` l.384) et page « Accordez votre guitare » (l.389). 1res pages vues. → « musiciens » / « Métronome ». - **ChordDiagramView** : `enum Instrument { guitar, ukulele }` — **pas de piano** ; `ShowMeView` force `getFingerings(instrument:.guitar)` → la pianiste a l'audio piano MAIS un diagramme d'accord GUITARE. → Ajouter le cas `.piano` (le clavier PianoKeyboardView existe déjà). - **InstrumentProfile.InstrumentType** : pas de `.piano` (app piano-first mais le piano n'existe pas comme instrument → « Mon piano » impossible). - **Contradiction splash ↔ onboarding** : le splash montre piano/corbeau, l'onboarding parle guitare. ### 🟠 Majeur - Libellés Settings « Grattage / Strum », warnings « guitare acoustique » (écho, reset) ; tutoriels « à la guitare » (pages 13, 17). - `PickingPatterns.swift` = guitare pure (cordes 1-6, fret offsets) exposé dans l'UX alors que **`PianoPatterns.swift` (Alberti, oom-pah, valse, arpèges) existe déjà mais n'est PAS branché à l'UI**. → Brancher l'UI sur PianoPatterns et retirer picking côté piano. - Code mort à supprimer : `FretboardView.swift`, `MediatorView.swift` (médiator). ### 🟡 Mineur - Univers « corbeau/raven » présent seulement dans la vidéo splash, pas nommé/tenu dans le thème code (`GrooveGraveTheme` = « Undead Chords »). --- ## C. UX PAR ÉCRAN (point de vue pianiste) **Transverses CRITIQUES 🔴** 1. **Pas de choix d'inversion / voicing** : tout joue `firstVoicing()` (Composer l.703, Dictionnaire) — impossible d'imposer une fondamentale ou une position (drop-2, shell…). 2. **Patterns incompatibles piano** : rhythm/picking pensés strum/plectre ; manquent Alberti/walking/stride/broken chords — alors qu'ils existent dans PianoPatterns (voir B). 3. **Pas de séparation main gauche / main droite** — cœur du jeu pianiste (accompagnement vs mélodie). 4. **Transposition ambiguë** : `TransposeControl` libellé « Tonalité » alors que c'est +X demi-tons ; pas de notion de capo/tonalité d'origine. 5. **Métronome sans subdivisions / count-in / accents par temps.** **Par écran (constat → amélioration)** - **Accueil** : libellé « Métronome » sur image `scene-tuner` (accordeur/métronome ambigu) ; pas de raccourci « clavier seul ». → Clarifier l'icône, ajouter un mode clavier libre. - **Dictionnaire d'accords** : n'affiche que **4 voicings** (`prefix(4)`) sans le dire ; `improMode` = shuffle staccato de notes isolées (trompeur) ; pas d'annotation d'inversion ni d'enharmonie. → Montrer tous les voicings + étiqueter position/inversion. - **Gammes & modes** : pas de doigtés suggérés ; notes non étiquetées par intervalle en lecture ; relatifs vs parallèles non expliqués. → Doigtés, étiquettes T/2/3…, note pédagogique modes. - **Composer** : pas de choix d'inversion ; tempo non partagé (reset au retour) ; pas de drag-to-reorder ; progressions non nommées (afficher « ii-V-I »). → Inversions par accord, tempo global, réordonnancement, libellés fonctionnels. - **Impro / ShowMe** : terme « fingering » (guitare) au lieu d'« inversion » ; patterns guitare ; pas de MG/MD ni mélodie. → Vocabulaire piano, patterns PianoPatterns, jeu accord+mélodie. - **Feuille de chanson** : « Tonalité » trompeur (= transposition) ; accords non tapables pour écouter ; toggle « analyse harmonique » invisible ; pas de highlight de l'accord en cours de lecture. → Renommer, rendre les accords tapables, exposer le toggle, surligner la lecture. - **Éditeur** : syntaxe d'accord non documentée ; pas de validation temps réel ni undo/redo. → Aide syntaxe, validation, undo. - **Métronome** : pas de subdivisions ; cercles visuels désynchronisés du clic ; tempo isolé. → Subdivisions/swing, sync visuel, count-in. - **Réglages** : effets non documentés (reverb/sustain/delay), « profils d'instrument » obscurs ; pas de transposition clavier ni d'inversion par défaut. → Descriptions + preview, réglage inversion défaut. - **Onboarding** : page « Tuner » = en fait un métronome (confusion) ; passif, rien sur voicings/MG-MD ; workflow piano non expliqué. → Corriger, rendre interactif, présenter le parcours piano. - **Paywall** : gates premium/free pas explicités avant blocage. → Marquer les fonctions premium en amont. - **Clavier piano** : pas d'indication d'octave ; glow compact quasi invisible ; pas de doigté ni MG/MD. → Repère d'octave, contraste du glow, rôles de main. **HAUTE 🟠** : accords étendus (maj13, 11) réduits sans warning (ChordQualityMapper) ; tempo non partagé entre modules ; pas de choix enharmonique (Do#/Réb — rejoint A) ; pas de doigtés suggérés. --- ## D. PARCOURS RÉELS (pianiste) — découvrabilité & flow Ce que la revue écran-par-écran ne montre pas : la friction se cumule le long d'un parcours. **Parcours 1 — Premier lancement (découverte)** Splash piano/corbeau ✅ → onboarding « L'app ultime pour les guitaristes » puis page « Accordez votre guitare » 🔴 → 1re chanson « Le Squelette Guitariste » (paroles guitare) 🔴. → Friction : dissonance immédiate entre l'univers annoncé (piano/corbeau) et le contenu (guitare). Perte de confiance dès l'entrée. **P0.** **Parcours 2 — « Je veux jouer Fa#m7 correctement »** Dictionnaire → F# → m7 → seulement 4 voicings (sans l'indiquer), tous en `firstVoicing()`, aucun choix d'inversion ni annotation de position → passage à ShowMe → **diagramme GUITARE** alors qu'elle joue piano. → Friction : impossible de choisir la position/inversion ; diagramme non pianistique. En tonalité bémol le spelling serait faux (cf. A). **P1.** **Parcours 3 — « Accompagner une chanson, main gauche + main droite »** SongSheet → contrôle « Tonalité » (= en fait transposition, libellé trompeur) → accords non tapables pour écouter → analyse harmonique dont le toggle est invisible → ShowMe : patterns strum/picking (guitare), pas d'Alberti/stride/walking, **aucune séparation main G / main D**. → Friction : le cœur du jeu pianiste (accompagnement 2 mains + pattern idiomatique) est absent — alors que `PianoPatterns` existe déjà en code mais n'est pas branché. **P0 (patterns) + P1 (mains).** --- ## SUIVI / RÉSOLUTIONS (cross-check binôme Bill ✕ Alice) - 🎉 **P0 « PIVOT VISIBLE » COMPLET (2026-07-01)** — Alice : chanson « Squelette Pianiste » + onboarding guitare→piano + 18 clés loc (commits 1bcc6b0, 631f44e) ; Bill : ShowMe câblé patterns piano (c2e1207) + `ShowMeView` l.376/433 `FingeringDiagramView(.guitar)` → `PianoKeyboardView`/voicings (renversements navigables, f2ca022) + splash find.mp4 piano/corbeau (ab9d6d4). ✅ **Plus AUCUN diagramme guitare visible par une pianiste.** - 🗒️ Reste (backlog P1 théorie, côté données — Bill, quand Antoine priorise) : spelling enharmonique key-aware ; ø (m7♭5) / °7 (dim7) first-class ; degrés mineurs III/VI/VII sans ♭. - 🧹 Cleanup optionnel non-bloquant : wrapper guitare `ChordDiagramView` (#Preview only) + cache fingerings interne ShowMe = vestigial non visible. - 🔧 **P0 EN COURS (Alice, commits 1bcc6b0 + 631f44e sur pivot-piano-recette)** : - ✅ Chanson par défaut « Le Squelette Guitariste » → « Le Squelette Pianiste » (FR+EN, paroles piano). - ✅ Onboarding : « guitaristes » → « pianistes » ; page 4 « Accordez votre guitare » (accordeur disparu) → **Métronome** (clés onboarding.tuner.*→metronome.*). - ✅ Retrait du reliquat mort `home.tuner` ; ajout des 18 clés dict manquantes (styles/patterns piano) FR+EN — corrige les affichages en clé brute (dont `style.classical`). - 🔎 **Palier 2 — reclassé après investigation (Alice)** : ajouter `.piano` aux enums Instrument/InstrumentType est INUTILE et à éviter. - `PianoChordDiagramView` (équivalent piano complet : voicings, renversements, écoute) EXISTE déjà et est branché dans SongSheetView (l.674) ; ChordDictionaryView utilise déjà PianoKeyboardView/PianoVoicingProvider. Le diagramme piano est donc essentiellement fait. - Le wrapper guitare `ChordDiagramView` n'est utilisé que dans son propre #Preview → mort en prod (candidat suppression ; garder `FingeringDiagramView` tant que ShowMe l'utilise). - ⛔ **Seul reste réel** : `ShowMeView` l.376 & l.433 affiche encore `FingeringDiagramView(instrument:.guitar)` → swap vers `PianoChordDiagramView` (fichier de Bill). C'est le dernier diagramme guitare visible par une pianiste. - `InstrumentType.piano` : mis-fit (enum modèle-à-cordes : stringCount/openFrequencies) → NE PAS forcer en P0 ; design séparé si l'enregistrement d'un vrai piano devient un besoin. - 🟡 [A/B] **Finding localisation** : la loc réelle = **FR + EN uniquement** (dicts hardcodés `frenchTranslations`/`englishTranslations`, résolution via `translations[key] ?? key`). Les `.lproj` es/de/pt ne sont PAS chargés → 3 langues de fichiers mais 2 langues réellement servies. À arbitrer (charger les .strings ou assumer FR/EN). - ✅ [A] `HarmonyAnalysis` l.102-104 : les extensions 9/11/13 omettaient la 7e de dominante implicite (vecteur chroma d'analyse faussé, incohérent avec PianoVoicingProvider) — **corrigé par Bill, commit aaec74b**. - ✅ [B] Dictionnaire : styles réduits à **Classique / Jazz / Blues** (retrait rock/pop/manouche/bossa/folk/reggae/… ) + ajout d'un vrai style Classique (cadence + Pachelbel + Alberti) — **Bill, commit 8fa4d28**. - 🔗 Annexe théorie par fichier : recette **458f5c36**. - ⛔ [A] **CONFIRMÉ à faire** (Bill) : ø (m7♭5) / °7 (dim7) pas first-class dans ChordQuality (seul `.diminished`) → dim7 composé affiché « dim6 » + voicing douteux. Fix : `.diminished7` / `.halfDiminished` natifs (ChordQuality + parsing + affichage °7 / ø). - ⛔ [A] **CONFIRMÉ à faire** (Bill) : degrés romains en mineur — HarmonyAnalysis l.185-187 note ♭III/♭VI/♭VII (correct seulement vs majeur parallèle). Fix low-risk : retirer les ♭ pour III/VI/VII en mode mineur.

Diagramme Mermaid (optionnel)

Exemple: flowchart LR\n A --> B

Ajouter des images (PNG, JPEG, GIF, WebP — max 50MB)

Conclusions Le **moteur théorique est fiable** (formules d'accords et de gammes exactes, détection de tonalité de niveau recherche) — c'est un vrai atout à mettre en avant. Le blocage n'est pas la justesse musicale mais **(1)** la finition du pivot (reliquats guitare visibles dès le 1er écran) et **(2)** l'absence des primitives attendues par une pianiste : choix d'inversion/voicing, séparation main gauche/droite, spelling enharmonique tonal, et exposition des patterns piano DÉJÀ présents dans le code mais non branchés à l'UI. Bonne nouvelle coût/impact : plusieurs correctifs à fort effet sont peu coûteux (réécrire la chanson/onboarding, brancher PianoPatterns à l'UI, ajouter le cas `.piano` au diagramme, corriger les libellés).

Recommandations ### Priorisation proposée (à trancher avec Bill / QA) **P0 — Bloquants pivot (rapides, fort impact perçu)** 0. **Brancher `PianoPatterns` (Alberti/oom-pah/valse/arpèges) à l'UI et retirer `PickingPatterns` côté piano** — meilleur ratio impact/coût (accord binôme Bill/Alice). 1. Réécrire la chanson par défaut « Squelette Guitariste → Pianiste » + paroles (AppFirstLaunch). 2. Onboarding : retirer « guitaristes » et la page « Accordez votre guitare ». 3. Ajouter le cas `.piano` à ChordDiagramView / brancher le diagramme sur PianoKeyboardView pour ShowMe. 4. Ajouter `.piano` à InstrumentType (profil instrument). **P1 — Fondamentaux pianiste** 5. **Spelling enharmonique tonal** (key-aware #/♭) + toggle manuel — chantier théorie critique. 6. **Choix d'inversion / voicing** dans Dictionnaire, Composer, ShowMe (au lieu de firstVoicing systématique). 7. **Brancher PianoPatterns** (Alberti/oom-pah/valse/arpèges) à l'UI ; retirer picking guitare côté piano. 8. Séparer **main gauche / main droite** (rôles, sonorités, illumination clavier). **P2 — Théorie & complétude** 9. Familles ø (m7♭5) et °7 first-class ; corriger les degrés mineurs (III/VI/VII sans ♭). 10. Repère d'octave sur le clavier ; libellés « Tonalité » → « Transposition » ; tempo partagé entre modules. 11. Métronome : subdivisions, count-in, accents. **P3 — Nettoyage / polish** 12. Supprimer FretboardView & MediatorView ; harmoniser warnings/tutoriels guitare→piano ; nommer/tenir l'univers corbeau dans le thème ; renommer la gamme « Chinoise » ; étendre l'analyse harmonique aux modes.