Compare commits
6 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 94b058a6e7 | |||
| a896a8e7fc | |||
| a05d2218c6 | |||
| 8908a32a68 | |||
| 695d1c1b50 | |||
| 7da16a82e5 |
@@ -18,6 +18,7 @@ steps:
|
||||
commands:
|
||||
- npm install -g vsce
|
||||
- npm ci
|
||||
- xvfb-run -a npm test
|
||||
- vsce package
|
||||
|
||||
- name: gitea_release
|
||||
|
||||
+2
-1
@@ -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"],
|
||||
"*.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"],
|
||||
|
||||
@@ -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'],
|
||||
|
||||
Generated
+2
-2
@@ -1,12 +1,12 @@
|
||||
{
|
||||
"name": "vscodestat",
|
||||
"version": "1.3.2",
|
||||
"version": "1.6.0",
|
||||
"lockfileVersion": 3,
|
||||
"requires": true,
|
||||
"packages": {
|
||||
"": {
|
||||
"name": "vscodestat",
|
||||
"version": "1.3.2",
|
||||
"version": "1.6.0",
|
||||
"license": "ISC",
|
||||
"devDependencies": {
|
||||
"@eslint/js": "^10.0.1",
|
||||
|
||||
+1
-1
@@ -1,7 +1,7 @@
|
||||
{
|
||||
"name": "vscodestat",
|
||||
"displayName": "vscodestat",
|
||||
"version": "1.3.2",
|
||||
"version": "1.6.0",
|
||||
"description": "",
|
||||
"categories": [
|
||||
"Other"
|
||||
|
||||
@@ -1,33 +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`).
|
||||
|
||||
NB : extension force-installed dans les containers vscode + vscodeluigi
|
||||
(cf vscode/todo.txt). C'est l'extension de monitoring activite dev.
|
||||
|
||||
SECURITE
|
||||
--------
|
||||
|
||||
[ ] makeHttpRequest envoie au serveur l'event sans auth
|
||||
src/extension.ts : `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.
|
||||
|
||||
BUGS / FRAGILITE
|
||||
----------------
|
||||
[ ] 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 : catch + console.error. Pas de
|
||||
retry, pas de notification user. Si serveur down, events
|
||||
perdus.
|
||||
+1
-5
@@ -13,10 +13,6 @@
|
||||
],
|
||||
"sourceMap": true,
|
||||
"rootDir": "src",
|
||||
"strict": true /* enable all strict type-checking options */
|
||||
/* 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. */
|
||||
"strict": true
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user