[Mineur] Dead code `_process_batch` + docstring dupliquée #101

New issue

Open

opened 2026-08-26 05:33:45 +00:00 by mat · 0 comments

mat commented

2026-08-26 05:33:45 +00:00

Owner

Problème

stream.py:209-219 : _process_batch (doublon synchrone de _async_process_batch) n'est référencé nulle part dans le code de production — uniquement par les tests (tests/test_stream.py:146,156,404,418,430,649,666). Migrer les tests vers la version async et supprimer.
coordinator.py:310-311 : la docstring contient deux fois la ligne « Only fields that survive a restart are saved: live_total_wh, ».

Recommandation

Nettoyage cosmétique mais visible dans une release 1.0.

Constaté lors de la review de code pré-v1.0 (v0.8.1).

## Problème 1. `stream.py:209-219` : `_process_batch` (doublon synchrone de `_async_process_batch`) n'est référencé nulle part dans le code de production — uniquement par les tests (`tests/test_stream.py:146,156,404,418,430,649,666`). Migrer les tests vers la version async et supprimer. 2. `coordinator.py:310-311` : la docstring contient deux fois la ligne « Only fields that survive a restart are saved: live_total_wh, ». ## Recommandation Nettoyage cosmétique mais visible dans une release 1.0. --- Constaté lors de la review de code pré-v1.0 (v0.8.1).

mat added the

v1.0 · mineur

label

2026-08-26 05:33:45 +00:00