fix: "renovate"

todo.txt

.drone.yml

@@ -18,6 +18,7 @@ steps:
       commands:
           - npm install -g vsce
           - npm ci
+          - xvfb-run -a npm test
           - vsce package
 
     - name: gitea_release

.gitignore

@@ -1,5 +1,4 @@
 out
-dist
 node_modules
 .vscode-test/
 *.vsix
@@ -31,3 +30,6 @@ coverage/
 .config/
 data/
 .aider*
+/dist/
+tsconfig.tsbuildinfo
+.twig-cs-fixer.cache

.gitleaks.toml

@@ -0,0 +1,5 @@
+[extend]
+useDefault = true
+
+[allowlist]
+paths = ['''\.vscode-test/''']

.lintstagedrc

@@ -1,8 +1,9 @@
 {
     "*.js": ["eslint --fix", "prettier --write"],
+    "*.ts": ["eslint --fix", "prettier --write"],
     "*.css": ["prettier --write"],
     "*.jsx": ["prettier --write"],
-    "*.html.twig": ["twig-cs-fixer lint --fix", "myhtmlvalidate -q"],
+    "*.tsx": ["prettier --write"],
     "*.php": ["php-cs-fixer fix --config .php-cs-fixer.php"],
     "Dockerfile": ["hadolint --ignore DL3002 --ignore DL3003 --ignore DL3008 --ignore DL3013 --ignore DL3016 --ignore DL3022"],
     "*.md": ["markdownlint --fix"],

.npmrc

@@ -1 +1,3 @@
 min-release-age=1
+fund=false
+audit=false

.prettierignore

@@ -4,3 +4,7 @@ log/*
 node_modules/*
 coverage/*
 data/*
+.vscode-test/
+/dist/
+tsconfig.tsbuildinfo
+legacy/*

eslint.config.js

@@ -4,6 +4,8 @@ import tsplugin from '@typescript-eslint/eslint-plugin';
 import globals from 'globals';
 
 export default [
+    // remonte (et --fix supprime) tout eslint-disable inutile : regle deja off, inactive, ou qui ne tire pas
+    { linterOptions: { reportUnusedDisableDirectives: 'error' } },
     js.configs.recommended,
     {
         files: ['**/*.ts'],
@@ -61,6 +63,6 @@ export default [
         },
     },
     {
-        ignores: ['out/', 'dist/', '**/*.d.ts'],
+        ignores: ['legacy/', 'out/', 'dist/', '.vscode-test/', '**/*.d.ts'],
     },
 ];

package.json

@@ -1,7 +1,7 @@
 {
     "name": "vscodestat",
     "displayName": "vscodestat",
-    "version": "1.1.160",
+    "version": "1.6.0",
     "description": "",
     "categories": [
         "Other"
@@ -25,10 +25,6 @@
     },
     "contributes": {
         "commands": [
-            {
-                "command": "vscodestat.helloWorld",
-                "title": "Hello World"
-            },
             {
                 "command": "vscodestat.setUrl",
                 "title": "setUrl"
@@ -53,22 +49,24 @@
         "onStartupFinished"
     ],
     "devDependencies": {
+        "@eslint/js": "^10.0.1",
         "@types/mocha": "^10.0.10",
-        "@types/node": "25.x",
+        "@types/node": "26.x",
-        "@types/vscode": "^1.116.0",
+        "@types/vscode": "^1.125.0",
-        "@typescript-eslint/eslint-plugin": "^8.59.1",
+        "@typescript-eslint/eslint-plugin": "^8.62.0",
-        "@typescript-eslint/parser": "^8.59.1",
+        "@typescript-eslint/parser": "^8.62.0",
-        "@vscode/test-cli": "^0.0.12",
+        "@vscode/test-cli": "^0.0.15",
-        "@vscode/test-electron": "^2.5.2",
+        "@vscode/test-electron": "^3.0.0",
         "eslint": "^10.1.0",
+        "globals": "^17.7.0",
         "husky": "^9.1.7",
-        "lint-staged": "^16.4.0",
+        "lint-staged": "^17.0.8",
-        "prettier": "^3.8.3",
+        "prettier": "^3.9.1",
         "ts-api-utils": "^2.5.0",
         "typescript": "^6.0.3"
     },
     "engines": {
-        "vscode": "^1.116.0"
+        "vscode": "^1.125.0"
     },
     "icon": "icon.png",
     "extensionKind": [

src/extension.ts

@@ -3,12 +3,6 @@ import * as vscode from 'vscode';
 export function activate(context: vscode.ExtensionContext) {
     console.log('Congratulations, your extension "vscodestat" is now active!');
 
-    // crée une commande de test
-    const disposable = vscode.commands.registerCommand('vscodestat.helloWorld', () => {
-        vscode.window.showInformationMessage('Hello World from vscodestat!');
-    });
-    context.subscriptions.push(disposable);
-
     // commande pour definir l'url
     const disposable2 = vscode.commands.registerCommand('vscodestat.setUrl', async () => {
         const url = await vscode.window.showInputBox({
@@ -42,7 +36,7 @@ export function activate(context: vscode.ExtensionContext) {
             console.log('Opened file:', filePath);
 
             // vscode.window.showInformationMessage(`Opened file: ${filePath}`);
-            await makeHttpRequest({ event: 'open', project: extractProjectName(filePath) });
+            await makeHttpRequest({ event: 'open', project: extractProjectName(filePath), source: 'vscode' });
         }
     });
 
@@ -52,13 +46,13 @@ export function activate(context: vscode.ExtensionContext) {
         console.log('Saved file:', filePath);
 
         // vscode.window.showInformationMessage(`Saved file: ${filePath}`);
-        await makeHttpRequest({ event: 'save', project: extractProjectName(filePath) });
+        await makeHttpRequest({ event: 'save', project: extractProjectName(filePath), source: 'vscode' });
     });
 
     // detecte un focus / blur de la fenetre vscode
     vscode.window.onDidChangeWindowState(async event => {
         console.log('Window state changed:', event.focused);
-        await makeHttpRequest({ focus: event.focused });
+        await makeHttpRequest({ event: 'focus', focused: event.focused });
     });
 }
 
@@ -69,7 +63,7 @@ export function deactivate() { }
 * /root/docker/monitoringserver/controller/homeController.js
 * => monitoringserver
 */
-function extractProjectName(path: string) {
+export function extractProjectName(path: string) {
     // des c'est l'un des fois l'autre ?
     // /root/docker/vscodestat/src/extension.ts
     // \root\docker\vscodestat\src\extension.ts

src/test/extension.test.ts

@@ -1,15 +1,40 @@
 import * as assert from 'assert';
+import { extractProjectName } from '../extension.js';
 
-// You can import and use all API from the 'vscode' module
+suite('extractProjectName', () => {
-// as well as import your extension to test it
+    test('extrait le nom de projet depuis un path Linux /root/docker/X', () => {
-import * as vscode from 'vscode';
+        assert.strictEqual(
-// import * as myExtension from '../../extension';
+            extractProjectName('/root/docker/vscodestat/src/extension.ts'),
+            'vscodestat'
+        );
+    });
 
-suite('Extension Test Suite', () => {
+    test('extrait le nom de projet depuis un path Windows \\root\\docker\\X', () => {
-	vscode.window.showInformationMessage('Start all tests.');
+        assert.strictEqual(
+            extractProjectName('C:\\root\\docker\\monitoringserver\\controller\\homeController.js'),
+            'monitoringserver'
+        );
+    });
 
-	test('Sample test', () => {
+    test('retourne null pour un path hors /docker/', () => {
-		assert.strictEqual(-1, [1, 2, 3].indexOf(5));
+        assert.strictEqual(extractProjectName('/home/user/projet/file.js'), null);
-		assert.strictEqual(-1, [1, 2, 3].indexOf(0));
+    });
-	});
+
+    test('retourne null pour un path vide', () => {
+        assert.strictEqual(extractProjectName(''), null);
+    });
+
+    test('extrait correctement quand le chemin contient docker plusieurs fois', () => {
+        assert.strictEqual(
+            extractProjectName('/root/docker/flatbay/lib/docker/foo.js'),
+            'flatbay'
+        );
+    });
+
+    test('gere les paths avec tirets et chiffres', () => {
+        assert.strictEqual(
+            extractProjectName('/root/docker/gextra6/bin/console'),
+            'gextra6'
+        );
+    });
 });

todo.txt

@@ -1,144 +0,0 @@
-REVIEW DE CODE — 2026-04-26
-============================================
-
-Extension VSCode (TypeScript) qui collecte des metriques sur
-l'activite du user dans VSCode :
-- ouverture de fichier / changement d'onglet (event 'open')
-- sauvegarde de fichier (event 'save')
-- focus/blur de la fenetre (event 'focus')
-
-Envoie un POST JSON a la URL configuree (`vscodestat.url`).
-113 lignes TypeScript, version 1.1.159 (vsix 1.1.141 commit).
-
-NB : extension force-installed dans les containers vscode + vscodeluigi
-(cf vscode/todo.txt). C'est l'extension de monitoring activite dev.
-
-SECURITE
---------
-[ ] vscodestat.url configurable user-side (CRITIQUE pour exfil)
-    src/extension.ts:14-23 : commande `setUrl` permet au user de
-    changer l'URL en runtime via `vscode.workspace.getConfiguration
-    ().update('vscodestat.url', url, ConfigurationTarget.Global)`.
-    Le user peut donc rediriger ses metriques vers son propre
-    serveur. Dans le contexte (extension force-installed pour
-    tracking employe), le user peut bypass le tracking en
-    pointant sur `https://localhost/dummy` ou similar. Attendu /
-    pas attendu ?
-    NB : entrypoint.sh dans vscode/ overwrite la URL a chaque
-    boot du container. Donc tracking restored. Mais pendant la
-    session, user peut tweak. A documenter le comportement.
-
-[ ] makeHttpRequest envoie au serveur l'event sans auth (RGPD)
-    src/extension.ts:97-118 : `fetch(url, { method: 'POST',
-    headers: ..., body: JSON.stringify(json) })`. Pas de token.
-    Cote serveur (cf monitoringserver/todo.txt), `/vscodestat` est
-    aussi sans auth => tout le monde peut envoyer des metriques.
-    Mais ici c'est cote client, le code est legitime. Ce qui est
-    discutable c'est que l'event contient :
-    - `event: 'open'`, `project: extractProjectName(filePath)`
-    Ces donnees sont des metadonnees d'activite du salarie. RGPD
-    : doit etre dans le declaration des traitements RH.
-
-[ ] extractProjectName fuite des paths potentiellement sensibles
-    src/extension.ts:73-81 :
-        const match = path.match(/\/docker\/([^/]+)/);
-        if (match) return match[1];
-    Si un user ouvre un fichier hors `/docker/X/...`, le
-    `extractProjectName` retourne null, donc `project: null`
-    envoye. Pas un leak direct, mais combine au tracking precis,
-    profile complet de l'activite hors-projet.
-
-[ ] Pas de cap sur la frequence des events
-    src/extension.ts:39-49 : `onDidChangeActiveTextEditor` =>
-    chaque alt-tab entre fichiers => 1 POST. Si user est tres
-    actif, 100+ POST par minute. Pas de debounce. Cote
-    monitoringserver, idem aucun rate-limit (cf monitoringserver/
-    todo.txt). Risque d'epuisement bande passante / spam serveur.
-
-[ ] Le user peut DISABLE l'extension volontairement
-    Standard VSCode : un user peut desactiver toute extension.
-    Le force-install au boot du container la re-active, mais
-    pendant la session, l'employe peut couper le tracking.
-    Pattern de force-tracking discutable (transparency).
-
-[ ] Pas de TLS pinning sur fetch
-    Si l'URL pointe vers HTTPS (probablement),
-    `monitoringserver.raphaelpiccolo.com`, certificat valide. OK
-    mais pas de pinning.
-
-BUGS / FRAGILITE
-----------------
-[ ] vscodestat-1.1.141.vsix commit dans le repo mais version 1.1.159
-    package.json:4 : "version": "1.1.159" mais le vsix commit est
-    1.1.141. Decalage. Si on installe le vsix, c'est l'ancienne
-    version qui est appliquee (vscode/bin/entrypoint.sh:25 :
-    `code-server --install-extension /opt/vsix/vscodestat.vsix`).
-    A rebuild + republier le vsix avec la version courante.
-
-[ ] /opt/vsix/vscodestat.vsix : binaire du dossier autre
-    Cf vscode/todo.txt. Le vsix est COPIE dans l'image vscode
-    (Dockerfile:48). Si le vsix de ce repo est modifie mais pas
-    rebuild dans vscode/, decalage permanent.
-
-[ ] event 'open' sur changement d'onglet, pas vraie ouverture
-    src/extension.ts:39 : `onDidChangeActiveTextEditor`. Fire
-    aussi sur le simple alt-tab entre 2 fichiers deja ouverts.
-    Donc l'event 'open' est mal nomme (en realite "focus de tab").
-    A renommer 'tab_focus' ou similar.
-
-[ ] event 'focus' sans event name
-    src/extension.ts:55 : `await makeHttpRequest({ focus: event.
-    focused })`. Pas de `event: 'focus'`. Cote serveur, comment
-    distinguer ? Cf monitoringserver/homeController.js:78 :
-    `eventName: req.body.name`. Donc le `focus` event ne match
-    pas la convention serveur.
-
-[ ] Pas de batching
-    Chaque event = 1 fetch. Pas de queue + flush periodique.
-    Si reseau down, perte d'events (pas de retry). A capper.
-
-[ ] makeHttpRequest catch silencieux
-    src/extension.ts:114-117 : catch + console.error. Pas de
-    retry, pas de notification user. Si serveur down, events
-    perdus.
-
-[ ] extractProjectName : path Windows hardcode au regex
-    src/extension.ts:79-80 : replace `\\` -> `/` puis match
-    `/docker/(...)/`. OK pour les conventions /root/docker, mais
-    si project hors `docker/`, retourne null. A clarifier.
-
-[ ] Pas de tests unitaires sur extractProjectName
-    Function pure, faciles a tester. Pas de tests dans src/test/
-    visible. A check.
-
-CODE MORT / POLLUTION
----------------------
-[ ] vscodestat-1.1.141.vsix commit
-    Binaire commit dans git. A scrub si rebuild a chaque release.
-
-[ ] vscodestat.helloWorld command
-    src/extension.ts:7-10. Demo command standard de yeoman.
-    Inutile en prod. A delete.
-
-CONVENTIONS
------------
-[ ] Mauvaise pratique : version vsix decalee
-    1.1.141 vs 1.1.159 dans package.json.
-
-[ ] Pas de README detaille sur l'integration serveur
-    README mentionne juste "Sample url". A documenter le format
-    JSON envoye et le comportement.
-
-DECISION SUGGEREE
------------------
-[ ] Rebuild + republier le vsix a la version courante
-    Production decalee de 18 versions (1.1.141 vs 1.1.159).
-
-[ ] Documenter le scope RGPD du tracking
-    Si l'extension force-installed est obligatoire pour tracker
-    l'activite des employes (Thomas, Luigi), declaration RGPD
-    requise (RGPD art 13).
-
-[ ] Considerer un rate-limit cote client
-    Debounce sur onDidChangeActiveTextEditor (ex: 1s) pour
-    eviter le spam.

tsconfig.json

@@ -6,12 +6,13 @@
 		"lib": [
 			"ES2022"
 		],
+		"types": [
+			"node",
+			"mocha",
+			"vscode"
+		],
 		"sourceMap": true,
 		"rootDir": "src",
-		"strict": true   /* enable all strict type-checking options */
+		"strict": true
-		/* Additional Checks */
-		// "noImplicitReturns": true, /* Report error when not all code paths in function return a value. */
-		// "noFallthroughCasesInSwitch": true, /* Report errors for fallthrough cases in switch statement. */
-		// "noUnusedParameters": true,  /* Report errors on unused parameters. */
 	}
 }