Skip to content

v1.0.0

@khalilami2005-ctrl khalilami2005-ctrl tagged this 30 Jul 16:08
Aucun changement de comportement. Les 38 tests passent toujours.

Quatre fonctions mortes supprimees dans insights_local.rs
build_analysis_from_stats, build_diagnosis_from_stats, truncate_json_compact,
merge_diagnosis_into_draft. Le nom de la derniere suggerait un cablage oublie :
verification faite, c'est un mecanisme remplace. Chaque section du rapport a
deja son repli calcule (rule_fallback aux lignes 701-777), et ce duo etait une
seconde approche du meme probleme, jamais branchee. Leurs auxiliaires
(build_known_issues, compute_overall_health, ...) gardent d'autres appelants.

agent_pure.rs : un test qui ne decidait rien
`if is_field { parts.push(x) } else { parts.push(x) }` — les deux branches
poussaient la meme valeur. Reduit a un push simple, comportement identique. La
liste de champs qui alimentait le test disparait avec lui. C'est probablement
une intention perdue : les lignes de champ devaient sans doute etre formatees
autrement.

anonymous_analytics.rs
- NaiveDate deplace dans le module de test. Il n'etait utilise que la, donc la
  compilation lib le voyait inutilise et `clippy --fix` l'a retire, cassant
  `cargo test`. L'import appartient au module qui s'en sert.
- DailyUsageEntry passe pub(crate) : un type prive en retour d'une fonction
  pub(crate) rend cette fonction inappelable d'ailleurs.

Le reste est mecanique
clamp(), sort_by_key(Reverse(..)) x5, IoError::other, str::repeat, rows.flatten(),
wildcard redondant dans un or-pattern, et un alias ActivityRow pour le 8-uplet
de insights_local.rs — son commentaire donne l'ordre reel des colonnes, verifie
contre le SELECT.

Deux #[allow(clippy::too_many_arguments)] assumes, sur configure_llama_command
et llm_section : chaque argument est un reglage distinct, et une struct ne ferait
que deplacer la meme liste d'un cran sans retirer une decision aux appelants.

cargo fmt n'est pas lance : il reformaterait presque tout le code pour un gain
cosmetique, et aucun workflow ne l'exige — release.yml fait pnpm install puis
pnpm build, rien d'autre.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Assets 2
Loading