Skip to content

refactor: simplifications issues de l'audit de code - #45

Merged
Destynova2 merged 1 commit into
mainfrom
refactor/audit-simplifications
Oct 5, 2026
Merged

Destynova2 merged 1 commit into
mainfrom
refactor/audit-simplifications

Conversation

@Destynova2

Copy link
Copy Markdown
Contributor

Suite d'un audit ponytail + qualité (CQI ≈ 7.3, baseline verte). Aucun changement de comportement voulu ; 2 tests unitaires de variantes retirés avec la politique correspondante.

  • dédoublonne panic_payload_message et les troncatures de chaînes dans engine/runtime_support (fallbacks d'origine conservés)
  • retire le code mort : hover_at_point, Bbox::as_quad, uniform_scale, dev-dep serde_json de dunst-vision
  • supprime l'enum ScrollStrategy à variante unique et BrowserTabReusePolicy (seul Host était câblé, sémantique inlinée)
  • découpe scroll_at, run_demo, run_serve en fonctions nommées (sortie de la démo vérifiée identique par diff)
  • borne les casts de shapes.rs via ratio / to_index, numérique inchangée

Vérifs : fmt, clippy -D warnings, 301 tests, machete, missing-docs = 0.

- dédoublonne panic_payload_message et les troncatures de chaînes
  dans engine/runtime_support
- retire le code mort : hover_at_point, Bbox::as_quad, uniform_scale,
  dev-dep serde_json de dunst-vision
- supprime l'enum ScrollStrategy à variante unique et la politique
  BrowserTabReusePolicy dont seul Host était câblé
- découpe scroll_at, run_demo et run_serve en fonctions nommées
- borne les casts de shapes.rs via les helpers ratio et to_index
@github-actions
github-actions Bot enabled auto-merge (squash) September 27, 2026 19:01
@Destynova2 Destynova2 closed this Oct 5, 2026
auto-merge was automatically disabled October 5, 2026 21:25

Pull request was closed

@Destynova2 Destynova2 reopened this Oct 5, 2026
@Destynova2
Destynova2 enabled auto-merge (squash) October 5, 2026 21:28
@Destynova2
Destynova2 merged commit 16ac958 into main Oct 5, 2026
25 of 26 checks passed
@Destynova2
Destynova2 deleted the refactor/audit-simplifications branch October 5, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant