Skip to content

fix(visualizer): tolérer suffixe Make collé sur l'argument run - #277

Merged
raveriss merged 1 commit into
mainfrom
codex/fix-make-visualizer-command-error
Feb 5, 2026
Merged

fix(visualizer): tolérer suffixe Make collé sur l'argument run#277
raveriss merged 1 commit into
mainfrom
codex/fix-make-visualizer-command-error

Conversation

@raveriss

@raveriss raveriss commented Feb 5, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Corriger l'échec de make visualizer 1 9 provoqué par un suffixe alphabétique collé au paramètre run (ex: 9my) qui était rejeté comme "Run invalide".

Description

  • Ajout d'un nettoyage regex dans scripts/visualize_raw_filtered.py (import re + pré-traitement dans _parse_run) pour extraire la portion numérique avant la normalisation Rxx et ajout d'un test de non-régression test_parse_run_accepts_make_goal_suffix_for_positional_usage dans tests/test_visualize_raw_filtered.py.

Testing

  • Exécutions réalisées : poetry run pytest -q tests/test_visualize_raw_filtered.py -k make_goal_suffix (passé), make visualizer 1 9 (passé), poetry run pre-commit run --all-files (tous les hooks passés), vérifications style/type (black, isort, ruff, mypy) et audit (pip-audit après mise à jour de pip), puis make cov (suite complète: 577 passed; couverture totale = 90%); toutes les vérifications automatisées rapportées ci‑dessus ont réussi.

Codex Task

@codecov

codecov Bot commented Feb 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@raveriss
raveriss marked this pull request as ready for review February 5, 2026 12:20
@raveriss
raveriss merged commit b06b94d into main Feb 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant