fix(logger): #630 escape Spectre markup on all console paths + non-throwing render fallback

[jsboige]

Problème (#630)

Quand un harvest set échoue, le marqueur [HARVEST-FAILURE] injecté par la résilience per-set #614 est rendu par Logger.Log(..., MessageType.Problem). Ce chemin passait le message non-escapé à AnsiConsole.MarkupLine → le StyleParser de Spectre interprète [HARVEST-FAILURE] comme un token de style → InvalidOperationException levée depuis le catch de la résilience → le run entier meurt au lieu de dégrader gracieusement. Le filet #614 était donc court-circuité par son propre message d'échec.

L'audit du Logger a montré que le bug est une asymétrie : le chemin Info/Warning/Success escape déjà (Markup.Escape), mais Problem, Title (Rule) et Instructions/Explanations (Panel — le ctor Panel(string) parse aussi le markup) ne le faisaient pas. Les handlers internes du logger interpolaient aussi ex.Message/LogFile sans escape.

Fix (les deux options du body de l'issue, complémentaires)

1. Escape systématique : Markup.Escape(message) sur les 3 chemins manquants + sur ex.Message/LogFile des handlers internes. Le marqueur [HARVEST-FAILURE] s'affiche désormais littéralement (greppable).
2. Ceinture non-throwing : le rendu console est enveloppé d'un try/catch avec fallback plain-text (AnsiConsole.WriteLine, sans parsing markup) — aucun échec de rendu ne peut plus se propager hors de Logger.Log, quelle que soit une future régression d'escape. Le garde-fou ArgumentOutOfRangeException (enum invalide) est préservé via un exception filter.

Le fichier log (File.AppendAllText) écrivait déjà le texte brut — non affecté.

Tests

LoggerMarkupSafetyTests (10 tests, console routée vers un StringWriter via IAnsiConsole custom) :

• Repro exact fix(harvest): spectre [HARVEST-FAILURE] interprété comme markup couleur → InvalidOperationException fatale (tue #614) #630 : [HARVEST-FAILURE] Card set ... failed and was skipped en Problem → ne throw plus et rend le marqueur littéralement.
• Theory sur les 7 MessageType avec contenu à crochets/moustaches/pseudo-styles.
• L'enum invalide throw toujours ArgumentOutOfRangeException (la ceinture ne l'avale pas).
• LogException avec crochets dans les messages d'exception.

Contre-vérification : contre l'ancien code (stash du fix), 5/10 échouent — exactement les 5 chemins non-escapés ; les chemins déjà sûrs passent. Les tests pinnent donc réellement le fix.

Suite complète : 566 pass / 1 known-fail (OWLSharp round-trip #133, pré-existant) / 5 skip. 0 warning introduit.

Impact

• Hardening du chemin régén : un set en échec logge proprement et le run continue (fix(pipeline): #613 harvest resilient to per-set failures (no more abort-all on one timeout) #614 restauré).
• Milestone v0.9.x mais mergeable dès maintenant : protège les prochaines régénérations (dont d'éventuels re-runs bundle).

Closes #630. Relates #613, #614.

🤖 Coordinator ai-01