ci: Vitest unit tests + ESLint + GitHub Actions (CI & release)

[sebdraven]

Stack sur la PR de review (#23). Contient l'infra qualité qui manquait au projet.

Pourquoi cette PR

Le repo n'avait ni tests, ni lint, ni workflow. Cette PR pose une base minimale mais sérieuse pour:

• détecter les régressions sur le scoring DIMA et la détection de sites suspects ;
• attraper les fautes de frappe et la dette JS via ESLint ;
• générer le .zip Chrome de release automatiquement à partir d'un tag.

Ce qui est ajouté

Tests (tests/)

Vitest + happy-dom, 5 fichiers, 83 tests, ~1.5s :

• contentExtractor.test.js — cleanText préserve cyrillique/chinois/arabe (régression locked-in), detectPageType (14 cas dont edge cases tels que businessnews.example.com → general et example.com/products/12 → commerce), extractTitle, shouldSkipElement (pattern strict avec frontières [-_]|$)
• techniqueAnalyzer.test.js — bandes de score Faible/Modéré/Élevé/Très Élevé/Critique, findKeywordMatches (frontières de mot, multi-mots, multi-occurrences, case-insensitive), pondération contextuelle (TE0153 sur news → 1.4, TE0221 sur social → 1.6), pondération dynamique (escalade selon occurrences, bonus techniques critiques), cap du score global à 100, filtrage tactique/technique
• suspiciousSitesManager.test.js — checkSite pour les 3 matchType (exact/contains/pattern), format Storm1516 natif sur X/Twitter et Telegram, refus si plateforme incompatible, extractSocialHandle, getRiskConfig (4 niveaux), gating console.log derrière DIMA_DEBUG
• uiManager.test.js — escapeHtml (XSS), sanitizeHexColor (rejette shorthand, named colors, payloads d'injection CSS), isSafeHttpUrl (rejette javascript: / data: / file:), adjustColor (clamp black/white), generateTooltip
• manifest.test.js — JSON valide, MV3, chaque chemin déclaré (content_scripts, icons, web_accessible_resources) existe sur disque

Loader (tests/helpers/loadScript.js) : les content scripts du plugin exposent leurs classes via window.X = X. Un eval indirect dans le window happy-dom de Vitest les rend disponibles sans modifier la source.

Lint

• eslint.config.js (flat config, ESLint 9) — globals navigateur pour la source, globals Node + Vitest pour les tests, eqeqeq: smart. 0 erreur, 0 warning.

Workflows GitHub Actions

.github/workflows/ci.yml — sur chaque push main et chaque PR :

• setup Node 20 avec cache npm
• npm ci
• npm run lint
• npm test
• Validation manifest.json parseable

.github/workflows/release.yml — sur tag v*.*.* :

• lint + tests d'abord (fail-fast)
• vérifie que le tag matche manifest.version (sinon erreur explicite)
• zippe les fichiers runtime uniquement (manifest.json, content.js, modules/, data/, documentation/, README.md, LICENSE, icônes — pas de node_modules, pas de tests)
• upload comme artifact + crée une release draft GitHub avec le zip attaché et release notes auto

Source touchée

Une seule modif côté source pour rendre la classe testable :

• modules/Suspicioussitesmanager.js : expose window.SuspiciousSitesManager (la classe, pas seulement l'instance) + gate du console.warn "aucune base chargée" derrière DIMA_DEBUG (cohérent avec le reste).

Comment l'utiliser

Localement :

npm install
npm run ci         # lint + tests
npm run test:watch # mode dev
npm run test:coverage

Pour publier une version :

# 1. bump manifest.json -> "version": "2.1.0"
# 2. commit + merge sur main
git tag v2.1.0
git push origin v2.1.0
# release.yml crée la release draft avec le .zip

Base

Cette branche part de review/code-review-fixes (#23) — les tests verrouillent les comportements corrigés là-bas (Unicode cleanText, escapeHtml, sanitizeHexColor, detectPageType strict). À merger après #23.

🤖 Generated with Claude Code