Centraliser la config ruff et rendre le lint bloquant (vrai fix)

Les deux corrections précédentes (de4ff68 sur picarones/, b20a7dc sur
tests/) étaient du rafistolage — elles enlevaient les symptômes sans
traiter la cause. La cause réelle : **trois configurations ruff
divergentes** sur le projet, et **aucun filet local** avant le push.

État de départ problématique :

1. `.github/workflows/ci.yml` : appelait ruff avec des flags
`--select=E,W,F --ignore=E501,E402`, ET avait `continue-on-error:
true` (donc le lint était décoratif).
2. `Makefile::lint` : appelait `ruff check $(PACKAGE)/ tests/` SANS
flags — prenait la config ruff par défaut, DIFFÉRENTE du CI.
3. `pyproject.toml` : aucune section `[tool.ruff]`.

Résultat : `make lint` pouvait dire "OK" localement pendant que le CI
trouvait des erreurs. Et même si le CI trouvait des erreurs, le lint
ne bloquait pas la PR. D'où l'accumulation de 10 erreurs F401/E741 sur
les sprints 16-21, découvertes par vagues successives.

Correctif de fond :

- `pyproject.toml` : ajout de `[tool.ruff]` (line-length=100,
target=py311) et `[tool.ruff.lint]` (select=E,W,F, ignore=E501,E402).
Les règles retenues sont celles du CI d'origine — aucune rupture de
contrat, juste une centralisation.
- `.github/workflows/ci.yml` : la commande passe de
`ruff check picarones/ --select=... && ruff check tests/ --select=...`
à simplement `ruff check picarones/ tests/`. La config vient du
pyproject.toml. **Suppression de `continue-on-error: true`** — le
lint devient bloquant, cohérent avec les jobs tests/build/demo.
- `Makefile::lint` : passe à `ruff check $(PACKAGE)/ tests/` (identique
au CI). Le fallback flake8 est retiré (ruff est la seule voie
officielle, moins de chemins à maintenir). Si ruff n'est pas installé,
on sort avec code 1 et un message explicite au lieu d'un silence qui
laissait croire que c'était vert.
- `CLAUDE.md` : nouvelle règle explicite — `make lint` avant tout push,
le job `lint` du CI est bloquant par design.

Vérifications :
- `ruff check picarones/ tests/` : All checks passed!
- `make lint` : All checks passed!
- `ruff check --show-settings` : line_length=100, py311, select
correctement chargé depuis pyproject.toml
- Suite complète : 1241 passed, 2 skipped (inchangée)

Conséquence : toute future erreur lint sera attrapée localement avant
le push. Plus d'allers-retours CI pour des imports oubliés.

https://claude.ai/code/session_0162FdNNJyNvBuYzkgtsr9VB

.github/workflows/ci.yml

@@ -175,12 +175,14 @@ jobs:
           retention-days: 30
 
   # ──────────────────────────────────────────────────────────────────
-  # Job 4 : Vérification de la qualité du code (optionnel)
+  # Job 4 : Vérification de la qualité du code
+  # La config ruff est centralisée dans pyproject.toml ([tool.ruff]),
+  # donc CI, Makefile et invocations locales produisent exactement les
+  # mêmes résultats — pas de divergence possible entre flags.
   # ──────────────────────────────────────────────────────────────────
   lint:
     name: Code quality
     runs-on: ubuntu-latest
-    continue-on-error: true  # Ne bloque pas le CI si le lint échoue
 
     steps:
       - name: Checkout
@@ -196,9 +198,7 @@ jobs:
         run: pip install ruff
 
       - name: Run ruff
-        run: |
-          ruff check picarones/ --select=E,W,F --ignore=E501,E402
-          ruff check tests/ --select=E,W,F --ignore=E501,E402
+        run: ruff check picarones/ tests/
 
   # ──────────────────────────────────────────────────────────────────
   # Job 5 : CI/CD — Détection de régression CER (optionnel)

CLAUDE.md

@@ -128,6 +128,11 @@ correspondants (`test_sprint15_llm_pipeline_bugs.py`, `test_sprint8_escriptorium
   `logger.warning("[module] fonctionnalité dégradée : %s", e)`.
 - **Toujours utiliser `logger.warning` avec message explicite** quand une fonctionnalité optionnelle
   échoue (confusion, taxonomy, structure, image_quality, etc.).
+- **Avant tout push, lancer `make lint`** (ou `ruff check picarones/ tests/`).
+  La config est centralisée dans `pyproject.toml` sous `[tool.ruff]`, donc
+  CI, Makefile et invocation directe produisent le même résultat. Le job
+  `lint` du CI est bloquant — un F401 (import inutilisé) ou un E741
+  (variable ambiguë) fait échouer la PR, par design.
 - **Les profils de normalisation** sont dans `picarones/core/normalization.py` — l'endpoint
   `/api/normalization/profiles` doit les lire dynamiquement depuis ce fichier, pas depuis une
   liste statique.

Makefile

@@ -84,15 +84,17 @@ test-sprint9:  ## Tests Sprint 9 uniquement
 # Qualité du code
 # ──────────────────────────────────────────────────────────────────
 
-lint:  ## Vérifie le style du code (ruff si disponible, sinon flake8)
+lint:  ## Vérifie le style du code (configuration lue depuis pyproject.toml)
+	@# La config ruff vit dans [tool.ruff] de pyproject.toml : cette cible,
+	@# le job CI et une invocation directe `ruff check` produisent le même
+	@# résultat. Lancez avant tout push pour éviter un échec en PR.
 	@if command -v ruff > /dev/null 2>&1; then \
 	  ruff check $(PACKAGE)/ tests/; \
 	elif $(VENV_BIN)/python -m ruff --version > /dev/null 2>&1; then \
 	  $(VENV_BIN)/python -m ruff check $(PACKAGE)/ tests/; \
-	elif command -v flake8 > /dev/null 2>&1; then \
-	  flake8 $(PACKAGE)/ tests/ --max-line-length=100 --ignore=E501,W503; \
 	else \
-	  echo "Aucun linter disponible (installez ruff : pip install ruff)"; \
+	  echo "ruff non installé — installation : pip install ruff"; \
+	  exit 1; \
 	fi
 
 typecheck:  ## Vérification de types avec mypy (si installé)

pyproject.toml

@@ -94,3 +94,16 @@ picarones = [
 [tool.pytest.ini_options]
 testpaths = ["tests"]
 addopts = "-v --tb=short"
+
+[tool.ruff]
+# Configuration centralisée pour que `ruff check`, `make lint` et le job CI
+# produisent exactement les mêmes résultats sans flags en ligne de commande.
+line-length = 100
+target-version = "py311"
+
+[tool.ruff.lint]
+# E/W = pycodestyle, F = pyflakes. On conserve les mêmes règles que le CI
+# d'origine (avant Sprint 22), qui excluait les lignes longues (E501) et les
+# imports non-top (E402, parfois utiles pour imports conditionnels).
+select = ["E", "W", "F"]
+ignore = ["E501", "E402"]