--- name: code-corrector description: >- Corrige et durcit le code d'une application web de façon disciplinée et mesurée. À utiliser quand l'utilisateur veut corriger les bugs d'une web app, faire un audit de sécurité (OWASP Top 10 : injections, XSS, CSRF, contrôle d'accès), lever les plafonds d'affichage/pagination SANS retirer les protections serveur, optimiser un code en préservant à 100 % le design existant, ou obtenir un rapport de correction structuré avec correctifs testés, y compris un composant de la stack personnelle (mode privé). Déclencheurs : "corrige les bugs de mon app", "audit sécurité", "OWASP", "déverrouille l'affichage / flux illimité", "optimise sans casser le design", "failles XSS/CSRF/injection". Règle cardinale : toute modification STRUCTURALE est proposée puis exécutée seulement après approbation explicite. Ne pas utiliser pour un débogage ponctuel d'un script ou d'un timer → debugging-and-error-recovery, ni pour auditer la configuration de l'agent (MCP, hooks, permissions) → agent-config-audit. license: MIT metadata: version: "1.1" author: GeekFamilyCorp langues: fr --- # code-corrector — Correction + durcissement web, sous contrôle humain Transforme un « corrige tout » flou en processus discipliné, testé et réversible : inventorier, reproduire, corriger avec un test par correctif, auditer la sécurité, lever les plafonds d'affichage sans démonter les protections serveur, préserver le design, et **proposer** toute modification structurale avant de la faire. Règles maison citées (R53 : confirmation avant action irréversible ou externe ; R70 : jamais de commande par concaténation ; R43 : secret exposé = compromis) — définitions dans `${CLAUDE_PLUGIN_ROOT}/references/regles.md`. ## Règle cardinale — approbation humaine des changements structuraux Classer CHAQUE changement avant de l'appliquer : - **Local / sûr (applicable directement)** : correction de bug isolé, validation ou échappement d'entrée, ajout de test, correctif d'une faille évidente sans changement d'API, micro-optimisation sans effet visuel. → appliquer, accompagné d'un test. - **STRUCTURAL (proposer + attendre le « oui » explicite)** : schéma de données, contrat d'API, flux d'authentification/autorisation, architecture, dépendances, configuration de sécurité (CORS, CSP, sessions), suppression ou altération d'un contrôle de sécurité, toute modification visuelle, toute écriture sur un service en production. → ne pas exécuter ; présenter le diff envisagé + le pourquoi. En cas de doute sur la catégorie : STRUCTURAL. Ne jamais affaiblir un contrôle de sécurité « pour que ça marche » sans accord explicite et conscient. ## « Flux illimité » : lever les plafonds UI, garder les protections serveur - **À lever (artificiel, présentation)** : caps codés en dur (`limit 50`), pagination qui masque des données accessibles, troncatures UI → pagination propre, virtualisation de liste, scroll paginé, streaming. Jamais tout en mémoire d'un coup. - **À conserver (protection)** : rate-limiting, quotas anti-abus, timeouts, taille max de payload, garde-fous anti-DoS. Une limite qui protège est un contrôle de sécurité → STRUCTURAL. ## Deux modes | Mode | Quand | Ce qui change | |---|---|---| | **Générique** (défaut) | code d'un tiers, projet public, application inconnue | rien de nommé ; rapport standard | | **Privé** | l'utilisateur vise un composant de sa propre stack (graphe, vecteurs, workflows, gateway, agent de veille, VPS) | lire votre fiche de stack (`references/stack-privee.md`, à écrire vous-même : elle n'est pas fournie) : protocole PROD, composants à conserver, inventaire à revérifier | Le mode privé ajoute en tête du rapport : composant visé, environnement (poste / VPS), mode (lecture seule / modification approuvée). Ce fichier de référence ne franchit jamais la frontière du dépôt public (R37). ## Workflow 1. **Inventaire & périmètre.** Cartographier front, back, endpoints, points d'entrée de données ; localiser les bugs signalés. Vérifier la date du jour si la logique en dépend. 2. **Reproduire avant de corriger.** Par bug : test rouge → cause racine (pas le symptôme) → correctif → test vert. Pas de correctif sans reproduction. 3. **Audit sécurité.** Passer la grille `references/owasp.md` (A01→A10, quatre questions par item), produire le tableau de gravité, corriger les failles *locales*, proposer les structurales. 4. **Flux de données.** Lever les plafonds d'affichage (section dédiée), mesurer l'impact perf. 5. **Préservation du design.** Aucune modification visuelle sans justification + accord ; si un correctif touche au rendu, capturer un avant/après. 6. **Rapport + tests.** Livrer selon `report_template.md` (même dossier) + les tests ajoutés. ## Exemples **Correctif local (appliqué directement).** Entrée : « la recherche plante quand je tape une apostrophe ». Reproduction : `GET /search?q=l'été` → 500, stacktrace SQL. Cause : concaténation dans la requête. Correctif : requête paramétrée. Test : `test_search_apostrophe` rouge → vert. Statut : corrigé (local), ligne A03 du tableau. **Modification structurale (proposée, non exécutée).** Entrée : « les utilisateurs voient les factures des autres en changeant l'id dans l'URL ». Cause : A01, aucune vérification d'appartenance. Le correctif minimal (vérifier `invoice.owner == user`) est local et appliqué. Mais l'audit montre que le même défaut existe sur 6 routes : la solution propre est un middleware d'autorisation par ressource → **changement d'architecture, proposé dans la section 6 du rapport avec diff, risque et rollback ; en attente d'accord.** **Mode privé.** Entrée : « le dashboard de la gateway tronque la liste des runs à 100 ». Lecture de votre fiche de stack (`references/stack-privee.md`, à écrire vous-même : elle n'est pas fournie) → environnement poste, service en marche → mode lecture seule d'abord. Diagnostic : cap UI côté template. Correctif : pagination côté client, aucun paramètre serveur touché (le throttle reste). Rapport en-tête : composant = agent-gateway, env = poste, mode = modif locale approuvée. ## Garde-fous - Human-in-the-loop sur tout ce qui est structural, visuel ou prod (R53). - Un correctif = un test ; pas de « corrigé » sans preuve exécutable. - Jamais affaiblir un contrôle, retirer un rate-limit/CSRF/auth, ni introduire de secret en clair ; pas de code malveillant. - Réversibilité : backup daté avant d'éditer un fichier sensible ; changements atomiques. - Modèle : `default` pour la correction, `deep` seulement pour une faille d'architecture — alias dans `${CLAUDE_PLUGIN_ROOT}/models.json`. - Frontières : bug d'un script ou d'un timer sans app web → `debugging-and-error-recovery` ; config de l'agent → `agent-config-audit` ; décision irréversible à stresser → `doubt-driven-development`. ## Changelog - **2026-09-07 — 1.1** : absorbe `code-corrector-local` (archivé) ; checklist OWASP déplacée en `references/owasp.md` ; mode privé + protocole PROD sortis dans une fiche de stack non publiée ; renvois fantômes supprimés ; frontmatter normalisé ; trois exemples ajoutés. - **2026-06 — 1.0** : version initiale.