From cf9b88cfbcf583414227c1df5249b09bf7c15960 Mon Sep 17 00:00:00 2001
From: Nicolas Fryder <nicolas.fryder@gmail.com>
Date: Wed, 19 Aug 2026 15:13:55 +0200
Subject: [PATCH] chore(lint): supprime 27 avertissements permanents du hook
 pre-push
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Chaque push affichait les 27 mêmes avertissements. Un gate qui répète
toujours la même chose n'est plus lu — et le jour où un VRAI avertissement
apparaît, il passe inaperçu au milieu. Les deux règles en cause étaient les
miennes et sont inadaptées à ce code.

require-await (7 occurrences, apps/api)
Inadapté à Fastify : un handler `async` sans `await` est la façon normale de
renvoyer une valeur — l'API distingue le style async du style callback avec
reply.send. Les rendre synchrones changerait la sémantique. Règle désactivée
pour l'API.

no-console (20 occurrences, apps/web/site)
Traces de diagnostic délibérées dans le site public. Les signaler à chaque
exécution sans jamais les retirer ne faisait que noyer le reste. Règle
désactivée pour les frontends, avec la valeur à restaurer si on décide un
jour de les nettoyer.

Le hook affiche désormais six lignes vertes et rien d'autre : tout ce qui
s'imprime redevient un signal.
---
 eslint.config.js | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/eslint.config.js b/eslint.config.js
index e44f388..7b7d1f8 100644
--- a/eslint.config.js
+++ b/eslint.config.js
@@ -37,7 +37,13 @@ export default [
       "prefer-const": "error",
       "no-var": "error",
       "no-throw-literal": "error",
-      "require-await": "warn",
+      // require-await est inadapté à Fastify : un handler `async` sans `await`
+      // est la façon NORMALE de renvoyer une valeur (l'API distingue le style
+      // async du style callback avec reply.send). Le rendre synchrone
+      // changerait la sémantique. La règle produisait 7 avertissements
+      // permanents — et un gate qui affiche toujours la même chose finit par
+      // ne plus être lu.
+      "require-await": "off",
     },
   },
 
@@ -77,7 +83,11 @@ export default [
       "no-unused-vars": ["error", { argsIgnorePattern: "^_", caughtErrors: "none" }],
       // `try { … } catch {}` volontaire sur les appels best-effort (logout, télémétrie)
       "no-empty": ["error", { allowEmptyCatch: true }],
-      "no-console": ["warn", { allow: ["warn", "error"] }],
+      // Les console.log du site public sont des traces de diagnostic voulues
+      // par l'auteur. Les signaler à chaque exécution sans jamais les retirer
+      // ne fait que noyer les vrais avertissements. Si on décide un jour de
+      // les supprimer, remettre ["warn", { allow: ["warn", "error"] }].
+      "no-console": "off",
       eqeqeq: ["error", "smart"],
       "prefer-const": "error",
       "no-var": "error",