novahiz 0.3.1 → 0.3.2

package/NOTICE.md

@@ -43,6 +43,7 @@ Installed on demand with `npx skills add` (or an upstream tarball when git trans
 | `flutter-skills` | https://github.com/flutter/agent-plugins | BSD-3-Clause |
 | `dart-skills` | https://github.com/dart-lang/skills | BSD-3-Clause |
 | `expo-skills` | https://github.com/expo/skills | MIT |
+| `impeccable` | https://github.com/pbakaus/impeccable | Apache-2.0 |
 
 ## opencode plugins (third-party)
 

package/README.md

@@ -274,7 +274,7 @@ Novahiz auto-registers external MCP servers based on the prompt category:
 | cron | `scheduler-mcp` (local venv clone) | MIT | devops |
 | dart | `dart mcp-server` (Dart SDK) | BSD-3-Clause | code, debug, design-ui, flutter |
 
-Skill packs (installed from official repos, never vendored): `flutter/agent-plugins` (25 skills), `dart-lang/skills` (15 skills), `expo/skills` (17 skills, the `expo-*` group only; `eas-*` paid services excluded). See [docs/PROVIDERS.md](docs/PROVIDERS.md).
+Skill packs (installed from official repos, never vendored): `flutter/agent-plugins` (25 skills), `dart-lang/skills` (15 skills), `expo/skills` (17 skills, the `expo-*` group only; `eas-*` paid services excluded), `pbakaus/impeccable` (1 skill, the upstream `impeccable` design skill). See [docs/PROVIDERS.md](docs/PROVIDERS.md).
 
 Upstream repositories and full provenance for MCP providers and opencode plugins: [docs/PROVIDERS.md](docs/PROVIDERS.md), [docs/HARNESSES.md](docs/HARNESSES.md), [NOTICE.md](NOTICE.md).
 

package/catalog/providers.json

@@ -39,20 +39,21 @@
     "id": "cron",
     "label": "Cron Scheduler",
     "kind": "mcp",
-    "command": ["C:/Users/hiz/.local/share/mcp-scheduler/.venv/Scripts/python.exe", "C:/Users/hiz/.local/share/mcp-scheduler/main.py"],
+    "command": ["python", "C:/path/to/scheduler-mcp/main.py"],
     "source": "https://github.com/PhialsBasement/scheduler-mcp",
     "license": "MIT",
     "categories": ["devops"],
-    "purpose": "Local MIT scheduler clone at C:/Users/hiz/.local/share/mcp-scheduler (requirements pin mcp<2)."
+    "purpose": "MIT scheduler clone (requirements pin mcp<2); no npm package exists. Clone PhialsBasement/scheduler-mcp, install its requirements into a venv, then point `command` at that venv's python and your clone's main.py (setup in docs/PROVIDERS.md). Shipped disabled: enable it after the local setup."
   },
   {
     "id": "novahiz",
     "label": "novahiz",
     "kind": "mcp",
-    "command": ["node", "C:/Users/hiz/.config/novahiz/mcp/novahiz-tools/index.mjs"],
+    "command": ["node", "~/.config/novahiz/mcp/novahiz-tools/index.mjs"],
     "source": "https://github.com/novahiz/novahiz",
     "license": "Apache-2.0",
-    "categories": ["code", "planning"]
+    "categories": ["code", "planning"],
+    "purpose": "Novahiz stdio MCP (novahiz_providers, novahiz_deps, memory tools). The opencode plugin resolves this path from the home directory itself; edit the command for other harnesses."
   },
   {
     "id": "dart",
@@ -94,5 +95,15 @@
     "license": "MIT",
     "categories": ["code", "expo"],
     "purpose": "17 official Expo framework skills (expo-* group); the 7 eas-* paid-service skills are excluded. Installed to ~/.agents/skills; when git clone fails, fetch the tarball from codeload.github.com and copy the skill folders."
+  },
+  {
+    "id": "impeccable",
+    "label": "Impeccable design skill",
+    "kind": "skill",
+    "install": ["npx", "-y", "skills@1.7.0", "add", "pbakaus/impeccable", "--skill", "impeccable", "-g", "-a", "opencode", "-y"],
+    "source": "https://github.com/pbakaus/impeccable",
+    "license": "Apache-2.0",
+    "categories": ["design-ui"],
+    "purpose": "Upstream Impeccable skill (critique, audit and polish playbooks) required by gate rule R14. Installed to ~/.agents/skills; rerun the command to update."
   }
 ]

package/docs/CATALOG.md

@@ -8,7 +8,7 @@ The catalog is the knowledge base of Novahiz. It defines what categories exist,
 catalog/
 ├── categories.json    # 17 categories with keywords, skills, roadmaps
 ├── rules.json         # 11 pre-edit rules (file class + content triggers)
-├── providers.json     # 10 external providers (7 MCP + 3 skill packs)
+├── providers.json     # 11 external providers (7 MCP + 4 skill packs)
 └── overrides.json     # Manual skill curation (power, stars, tags)
 ```
 

package/docs/CONFIGURATION.md

@@ -41,7 +41,7 @@ These files live in `catalog/` and are part of the git repository:
   "providers": {
     "autoRegister": true,
     "autoInstall": false,
-    "disabled": []
+    "disabled": ["cron"]
   },
   "ledger": {
     "enabled": true,
@@ -69,7 +69,7 @@ These files live in `catalog/` and are part of the git repository:
 | `classify.fallbackCategory` | `general` | Fallback when no category matches |
 | `providers.autoRegister` | `true` | Auto-register MCP providers |
 | `providers.autoInstall` | `false` | Auto-install provider dependencies |
-| `providers.disabled` | `[]` | Providers to skip |
+| `providers.disabled` | `["cron"]` | Providers to skip (`cron` needs a local scheduler clone first) |
 | `ledger.enabled` | `true` | Enable the task ledger |
 | `ledger.review.edits` | `3` | Force review after N edits |
 | `ledger.review.todos` | `2` | Force review after N completed todos |

package/docs/PROVIDERS.md

@@ -46,7 +46,7 @@ MCP servers, with provenance from `catalog/providers.json`:
 | `novahiz` | local (`mcp/novahiz-tools`) | [novahiz/novahiz](https://github.com/novahiz/novahiz) | Apache-2.0 | code, planning |
 | `dart` | `dart mcp-server` (Dart SDK) | [dart-lang/ai · dart_mcp_server](https://github.com/dart-lang/ai/tree/main/pkgs/dart_mcp_server) | BSD-3-Clause | code, debug, design-ui, flutter |
 
-`cron` runs `scheduler-mcp` (MIT) from a local clone at `C:/Users/hiz/.local/share/mcp-scheduler` with its own virtualenv. It replaced the former AGPL `mcp-cron` npm package; there is no npm install step for it.
+`cron` runs `scheduler-mcp` (MIT) from a local clone with its own virtualenv. It replaced the former AGPL `mcp-cron` npm package; there is no npm install step for it. It ships disabled because the `command` has to point at your own clone: install the upstream requirements into a venv, set `command` to that venv's python plus your clone's `main.py`, then remove `cron` from `providers.disabled`.
 
 `dart` requires the Dart SDK on `PATH` (`requires: ["dart"]`). Without it the MCP entry still registers, but the server fails to start; disable it or install the SDK.
 
@@ -57,12 +57,13 @@ Skill and command packs:
 | `flutter-skills` | `npx skills add flutter/agent-plugins --skill '*' -g -a opencode -y` | [flutter/agent-plugins](https://github.com/flutter/agent-plugins) | BSD-3-Clause | code, design-ui, flutter |
 | `dart-skills` | `npx skills add dart-lang/skills --skill '*' -g -a opencode -y` | [dart-lang/skills](https://github.com/dart-lang/skills) | BSD-3-Clause | code, debug, flutter |
 | `expo-skills` | `npx skills add expo/skills --skill expo-overview ... -g -a opencode -y` | [expo/skills](https://github.com/expo/skills) | MIT | code, expo |
+| `impeccable` | `npx skills add pbakaus/impeccable --skill impeccable -g -a opencode -y` | [pbakaus/impeccable](https://github.com/pbakaus/impeccable) | Apache-2.0 | design-ui |
 
 Installed skills land under `~/.agents/skills` for OpenCode. They are referenced by install command, never vendored in this repository. `dart-lang/skills` is a subset of `flutter/agent-plugins` (same 15 Dart skills); both are listed for provenance.
 
 `expo-skills` names the 17 `expo-*` skills explicitly; the 7 `eas-*` skills (paid EAS services) are excluded. When `npx skills add` cannot reach the repo (git clone failures), fetch the tarball from `codeload.github.com` and copy the skill folders into `~/.agents/skills`, then run `Novahiz sync` — that fallback was used on this machine.
 
-Design and text skills ship as ordinary Novahiz skills under `skills/`, not as providers.
+`impeccable` is the one design skill referenced as a provider: gate rule R14 requires it, and it installs from upstream under its own Apache-2.0 terms instead of being vendored. The other design and text skills ship as ordinary Novahiz skills under `skills/`, not as providers.
 
 ## Mapping
 
@@ -92,7 +93,7 @@ Control it in `novahiz.config.json`:
 
 - `autoRegister`: register missing MCP servers on startup.
 - `autoInstall`: run the official install commands during `node install/install.mjs`.
-- `disabled`: provider ids to skip.
+- `disabled`: provider ids to skip. `cron` ships disabled by default; enable it after the local scheduler setup described above.
 
 Run the install commands on demand:
 

package/install/install.mjs

@@ -58,7 +58,7 @@ function defaultConfig(skillsDir) {
     providers: {
       autoRegister: true,
       autoInstall: false,
-      disabled: []
+      disabled: ["cron"]
     }
   };
 }
@@ -314,8 +314,8 @@ async function main() {
     { pkg: "narsil-mcp", bin: "narsil-mcp", name: "narsil" },
     { pkg: "security-mcp", bin: "security-mcp", name: "security" },
   ];
-  // `cron` has no npm package: the plugin registers it from catalog/providers.json
-  // (scheduler-mcp local venv, see docs/PROVIDERS.md).
+  // `cron` has no npm package and ships disabled (local scheduler clone only);
+  // enable it after the local setup documented in docs/PROVIDERS.md.
 
   if (!dryRun) {
     note("\nInstalling MCP servers...");
@@ -418,8 +418,8 @@ async function main() {
             "timeout": 120000,
             "enabled": true
           },
-          // `cron` is registered by the plugin from catalog/providers.json
-          // (scheduler-mcp local venv); no npm install, no template entry.
+          // `cron` ships disabled by default: no template entry here, enable it
+          // after the local scheduler clone setup (docs/PROVIDERS.md).
           "playwright": {
             "type": "local",
             "command": ["npx", "@playwright/mcp@latest", "--browser=msedge"],

package/novahiz.config.example.json

@@ -18,7 +18,7 @@
   "providers": {
     "autoRegister": true,
     "autoInstall": false,
-    "disabled": []
+    "disabled": ["cron"]
   },
   "tokens": {
     "enabled": true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novahiz",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Enforce AI agent skills before every file edit: prompt classifier, skill catalog, and pre-edit gate.",
   "type": "module",
   "license": "Apache-2.0",
@@ -60,6 +60,7 @@
     "mcp",
     "skills",
     "docs",
+    "!docs/audit-2026-09-25.md",
     "README.md",
     "NOTICE.md",
     "novahiz.config.example.json",

package/src/spec.ts

@@ -171,7 +171,7 @@ export const DEFAULT_CONFIG: NovahizConfig = {
   providers: {
     autoRegister: true,
     autoInstall: false,
-    disabled: []
+    disabled: ["cron"]
   },
   ledger: {
     enabled: true,

package/docs/audit-2026-09-25.md

@@ -1,275 +0,0 @@
-# Audit en profondeur du système Novahiz — 2026-09-25
-
-## Méthode
-
-- Demande utilisateur : auditer en profondeur Novahiz, plugins un à un, serveurs MCP, tools, rules, catalogue.
-- 5 sous-agents reviewer en lecture seule, un par axe. Session IDs pour traçabilité :
-  - Plugins : `ses_f285bcda6ffef7ox95YLcQxx0H`
-  - MCP : `ses_f285b9f3fffe27HJYbhwpe3t1N`
-  - Rules & gate : `ses_f285b6f8bffeWWcf0e5nSMCyBi`
-  - Tools : `ses_f285b405bffe7MkZh0NAxD9Rbc`
-  - Catalogue : `ses_f285b0a9cffe7J5YB25MiVtPZ8`
-- Périmètre : `C:\Users\hiz\.config\novahiz`, `C:\Users\hiz\.config\opencode`, `opencode.jsonc`, `dcp.jsonc`, `.config\.mcp\`, `~/.agents/skills`, `.config\.agents\skills`.
-- Audit seul : aucun correctif n'a été appliqué. Chaque finding cite un fichier et une ligne vérifiés par l'agent qui l'a produit.
-- Hors périmètre : comportement runtime du navigateur, contenu des sessions, audit de dépendances des paquets tiers installés.
-
-## Vue d'ensemble
-
-| Axe | Findings | Répartition |
-|---|---|---|
-| 1. Plugins opencode | 24 | 3 HIGH, 5 MEDIUM, 6 LOW, 10 INFO |
-| 2. Serveurs MCP (7 résolus) | 31 listés (l'agent déclare 32, voir Inconnus) | 2 HIGH, 9 MEDIUM, 11 LOW, 9 INFO |
-| 3. Rules & gate | 29 | 3 CRITICAL, 6 MAJEUR (=HIGH), 9 MINEUR (=MEDIUM), 11 SUGGESTION |
-| 4. Tools (CLI, MCP tools, scripts) | 25 | 3 HIGH, 8 MEDIUM, 10 LOW, 4 INFO |
-| 5. Catalogue de skills | 22 | 2 HIGH, 4 MEDIUM, 10 LOW, 6 INFO |
-| **Total** | **131** | **3 CRITICAL, 16 HIGH, 35 MEDIUM, 37 LOW, 40 INFO/SUGGESTION** |
-
-Six findings sont présents sous deux angles différents (doublons inter-axes listés plus bas) : environ 125 uniques.
-
-Verdict global : **le gate, qui est le cœur du système, échoue en fail-open sur trois axes d'entrée** (classification du prompt, config éditable, index des skills). Les garde-fous existent et sont testés sur les chemins heureux, mais plusieurs chemins d'erreur contournent l'enforcement sans log visible.
-
-## Top des blocages
-
-1. **[CRITICAL]** `novahiz/src/gate.ts:280` — le tier est calculé sur le contenu édité et non sur le prompt : R6 et la roadmap sont sautées dès que l'édit est court, donc l'étape « plan » n'est presque jamais exigée mécaniquement.
-2. **[CRITICAL]** `novahiz/src/commands/gate.ts:48` — `gate.enabled: false` ou `gate.tools: []` dans `novahiz.config.json` désactive tout l'enforcement sans aucune barrière ni log. Un kill-switch silencieux, éditable par la même session que le gate protège.
-3. **[CRITICAL]** `novahiz/src/spec.ts:137` — l'index des skills est lu sous `**/build/**` : retirer un skill de `build/installed-skills.json` le rend `unmatchedRequired` et donc jamais exigé, sans warning.
-4. **[HIGH]** `plugins/novahiz.ts:401` — un échec de `classify` vide les catégories et neutralise 6 des 11 règles à `promptCategories`, avec seulement un warning en log.
-5. **[HIGH]** `plugins/novahiz.ts:486` — un sessionID vide ou invalide fait retourner le hook avant tout appel au gate : l'outil s'exécute sans contrôle.
-6. **[HIGH]** `plugins/novahiz.ts:494` — `loaded.add` précède la vérification de session-load, et le nom de skill est joint puis resplitté par virgules : des skills inexistants passent pour chargés.
-7. **[HIGH]** `scripts/capture-cli.mjs:69` — `--label` ou `--out` contenant `..` résout hors du dossier de sortie et déclenche un `rmSync` récursif avec `force: true`.
-8. **[HIGH]** `package.json:52` — le `postinstall` exécute `install.mjs --yes` : 5 `npm i -g` non pinnés, écriture de `~/.config/opencode/opencode.jsonc` et `npm update`, sans consentement.
-9. **[HIGH]** `src/exec.ts:53` — `SAFE_BOOTSTRAP_BINS` contient `cmd` et `bash` : un bootstrap `["cmd","/c",…]` dans `catalog/providers.json` fait sauter toute la barrière d'exécution, arguments non validés.
-10. **[HIGH]** `mcp_scheduler/executor.py:119` — l'outil cron planifie des commandes shell persistantes (`cmd.exe /c`) hors du périmètre du gate. La DB est vide aujourd'hui : risque capacitaire.
-11. **[HIGH]** `opencode.jsonc:17` — narsil indexe tout `.config` avec `--git --persist` : journaux et base du scheduler deviennent lisibles via `narsil_get_file` par tout appelant d'outil.
-12. **[HIGH]** `build/installed-skills.json` — index à 84 skills sur 95 présents : 11 skills (eas-* ×7, supabase, supabase-postgres-best-practices, expo-migrate-module, expo-skill-eval) sont invisibles pour le gate et le ranking.
-13. **[HIGH]** `opencode/skills-lock.json` — fichier vide (`{"version":1,"skills":{}}`), lu par aucun code : aucune détection de non-conformité des 95 skills.
-
-## 1. Plugins opencode (24 findings)
-
-Deux plugins chargés : `plugins/novahiz.ts` (le gate, cœur critique) et `plugins/orca-opencode-status.js` (report d'état externe). Le chargement est automatique par scan `plugin/` et `plugins/` (non récursif) ; `opencode.jsonc` ne référence que les plugins npm `@mohak34/opencode-notifier` et `@tarquinen/opencode-dcp`.
-
-### plugins/novahiz.ts (14 findings)
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| HIGH | Échec de classification = fail-open sur tout le cœur du gate | plugins/novahiz.ts:401-404 | 6 des 11 règles neutralisées, éditions possibles sans skills pipeline |
-| HIGH | Bypass total si sessionID invalide | plugins/novahiz.ts:486-487 | Retour avant appel gate, outil exécuté sans contrôle |
-| HIGH | Skills comptés chargés sans vérification + injection par virgule | plugins/novahiz.ts:494-501 | Skills inexistants ou multiples passent pour chargés |
-| MEDIUM | Prompt passé en argv : injection d'options, fuite, troncature | plugins/novahiz.ts:400 | `--home` détourne le spec, prompt >32767 casse argv Windows, texte visible dans les processus |
-| MEDIUM | spawnSync bloque l'événementiel d'opencode | plugins/novahiz.ts:254-260 | Serveur gelé jusqu'à 10 s par appel sur chaque chemin chaud |
-| MEDIUM | Enregistrement de commandes MCP depuis un catalog git-oppable non signé | plugins/novahiz.ts:334-340 | Une compromission de `catalog/providers.json` devient exécution au démarrage |
-| LOW | TOCTOU et écritures d'état non atomiques | plugins/novahiz.ts:119-121 | État autodocs corrompu ou perdu |
-| LOW | Garde anti-traversal incomplète vers un autre lecteur | plugins/novahiz.ts:568-572 | Chemins absolus étrangers dans l'état `.novahiz` |
-| LOW | Compteur `sessions` mort et mal nommé | plugins/novahiz.ts:180 | Incrémenté à chaque édition, jamais lu |
-| INFO | Duplication à trois niveaux (src/, adapters/, nettoyage) | plugins/novahiz.ts:7-8 | Exposé à la dérive entre copies |
-| INFO | Détection de timeout par heuristique fragile | plugins/novahiz.ts:264 | Timeout remonté comme exit 1 générique sur Windows |
-| INFO | Effets de bord disque à chaque message | plugins/novahiz.ts:322 | 3 existsSync + création de project-memory au démarrage et par message |
-| INFO | Texte d'enforcement en dur dupliqué de la doc | plugins/novahiz.ts:459-462 | Évolution d'AGENTS.md sans mise à jour = system prompt mensonger |
-| INFO | Aucune injection shell (point conforme) | plugins/novahiz.ts:254 | spawn sans shell:true : CWE-78 non applicable |
-
-### plugins/orca-opencode-status.js (10 findings)
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| MEDIUM | Port non validé : le POST peut sortir de la boucle locale | orca-opencode-status.js:342 | Port contenant `@` redirige l'URL avec le header token vers un hôte distant |
-| MEDIUM | Un événement malformé tue l'abonnement pour tout le processus | orca-opencode-status.js:1089 | TypeError dans la boucle consume arrête tout report d'état jusqu'au redémarrage |
-| LOW | Aucun fichier d'état (correction de prémisse notifier-state) | orca-opencode-status.js:101-133 | État purement en mémoire, rien à recréer |
-| LOW | pendingAttentionByKey non borné avant dispose | orca-opencode-status.js:763 | Fuite mémoire jusqu'au dispose |
-| LOW | Logique de conversion v2→v1 tripliquée | orca-opencode-status.js:805-817 | Événement ajouté à un seul endroit disparaît silencieusement |
-| INFO | console.warn hors canal de log produit | orca-opencode-status.js:65 | Avertissements perdus du flux structuré |
-| INFO | IO synchrone sur l'événementiel | orca-opencode-status.js:33 | statSync/readFileSync empilés par événement |
-| INFO | Sortie de données de conversation vers un autre processus | orca-opencode-status.js:143 | Jusqu'à 4000 caractères de prompts/réponses vers Orca, secrets possibles |
-| INFO | Aucune commande exécutée (point conforme) | orca-opencode-status.js:356 | CWE-78 non applicable |
-| INFO | Bornes et timeouts conformes à leurs commentaires | orca-opencode-status.js:96 | Fail-open assumé documenté, non critique |
-
-## 2. Serveurs MCP (31 findings listés)
-
-Sept serveurs résolus : novahiz (15 tools), context7 (2), narsil (~60 tools), cron/scheduler, dart (11), playwright (25), security (46). Les trois derniers points forts vérifiés : fail-closed du gate, validation Zod, HMAC constant-time.
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| HIGH | Commandes shell arbitraires persistantes sans validation | mcp_scheduler/executor.py:119-131 | cron planifie `cmd.exe /c` hors gate (DB vide : risque capacitaire) |
-| HIGH | narsil indexe .config avec journaux et base de tâches | opencode.jsonc:17 | Logs et scheduler.db lisibles via narsil_get_file par tout appelant |
-| MEDIUM | Clé `env` rejetée silencieusement par le schéma McpLocalConfig | opencode.jsonc:24-27 | Config d'environnement de cron inopérante, artefacts dans le workspace |
-| MEDIUM | Échecs dart et playwright non journalisés | opencode.log:run 4410c095 | 55 et 30 échecs, zéro outil en session, aucune cause dans le log |
-| MEDIUM | Authentification security-mcp inactive | opencode.jsonc:30-35 | SECURITY_MCP_SHARED_SECRET absent : 46 tools (dont fortify auto-apply) utilisables sans auth |
-| MEDIUM | Journal d'audit des tools tracké par git | .mcp/audit/tool-calls.jsonl:1 | Inputs/outputs de tools exposés par l'historique git |
-| MEDIUM | Injection possible via commandes flottantes `npx -y pkg@latest` | catalog/providers.json:6 | Retrait d'une clé MCP = exécution auto d'un paquet non pinné au démarrage |
-| MEDIUM | Mises à jour automatiques de code DCP activées par défaut | dcp.jsonc:1 | Plugin en-process se recharge depuis npm et raw.githubusercontent sans revue |
-| MEDIUM | SSRF sur les tâches API du scheduler | mcp_scheduler/executor.py:168-203 | Toute URL acceptée (loopback, link-local), méthode et headers contrôlés |
-| MEDIUM | Disponibilité du gate : spawnSync node ETIMEDOUT récurrent | plugins/novahiz.ts:245 | 14 refus fail-closed sur les 23-24/09 (timeout 10 s dépassé) |
-| MEDIUM | mcp-audit-report.md périmé | opencode/mcp-audit-report.md:4 | 11 affirmations fausses sur une config antérieure au 24/09, posture GOOD usurpée |
-| LOW | root arbitraire dans les tools memory | novahiz/mcp/novahiz-tools/index.mjs:579 | memory_* créent/écrivent project-memory/ n'importe où (doublon axe tools) |
-| LOW | slotId non validé côté MCP | novahiz/mcp/novahiz-tools/index.mjs:590-593 | Format d'identifiant non contraint |
-| LOW | Versions context7 et security non pinnées | opencode/opencode.jsonc:12 | Shims npm user-remplaçables sans revue |
-| LOW | npx invoqué sans flag -y | opencode/opencode.jsonc:38 | Installation et exécution sans confirmation |
-| LOW | Timeout par défaut de 5000 ms sur context7 et cron | opencode/opencode.jsonc:10 | Démarrage lent = échec alors que le scheduler prévoit 300 s |
-| LOW | Dépendances flottantes du scheduler | mcp-scheduler/requirements.txt:1 | Réinstallation pip non reproductible |
-| LOW | Tokens OAuth résiduels en clair (expo, supabase) | ~/.local/share/opencode/mcp-auth.json:1 | Jetons de serveurs retirés encore lisibles sur disque |
-| LOW | Double source Playwright divergente | opencode/opencode.jsonc:36 | Global 0.0.81 vs npx 0.0.82 invoqué |
-| LOW | Schéma DCP chargé depuis raw.githubusercontent.com | opencode/dcp.jsonc:2 | Disponibilité et intégrité chez un tiers |
-| LOW | Profil navigateur persistant avec sessions vivantes | opencode/instructions.md:20 | browser_cookie_* lisent les jetons de session Edge |
-| LOW | Postinstall novahiz (doublon axe tools) | novahiz/package.json:52 | `install.mjs --yes` à l'installation npm |
-| INFO | Télémétrie OTel context7 active par défaut | context7 telemetry-config.js:1 | Métriques d'usage émises sans OTEL_SDK_DISABLED |
-| INFO | Clés d'embedding narsil non configurées | opencode/opencode.jsonc:15 | Recherche sémantique dégradée |
-| INFO | security-mcp en stdio pur | security-mcp/dist/mcp/server.js:1 | Surface réseau nulle (conforme) |
-| INFO | Shims npm orphelins de serveurs retirés | ~/AppData/Roaming/npm/obsidian-mcp.cmd:1 | supabase, chrome-devtools, obsidian, sequential-thinking, mcp-cron encore installés |
-| INFO | Deux racines .mcp, policies et reports absents | ~/.config/.mcp/audit/ | generate_policy et generate_compliance_report sans emplacement |
-| INFO | Panne réseau 24-25/09 rétablie | ~/.local/share/opencode/log/opencode.log:1 | ENOTFOUND opencode.ai avant rétablissement (HTTP 200 vérifié) |
-| INFO | Secrets uniquement en variables d'environnement | opencode/opencode.jsonc:171 | OPENROUTER_API_KEY hors fichiers trackés, filtrage enfants MCP non déterminé |
-| INFO | opencode.db 2343,7 Mo et opencode.log 56,7 Mo | ~/.local/share/opencode/opencode.db:1 | Hygiène du poste |
-| INFO | Garde-fous positifs vérifiés | novahiz/plugins/novahiz.ts:245 | Fail-closed, timeouts 10 s / 1 Mo, Zod, HMAC constant-time conformes |
-
-## 3. Rules & gate (29 findings)
-
-Source de vérité : `~/.config/novahiz/catalog/rules.json` (11 règles, R1 et R2 référencées mais inexistantes dans ce fichier). Enforcement : `plugins/novahiz.ts` (hook `tool.execute.before`, throw) + `src/commands/gate.ts` (CLI).
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| CRITICAL | Tier calculé sur le contenu édité et non sur le prompt | novahiz/src/gate.ts:280 | R6 et la roadmap sautées dès que l'édit est court |
-| CRITICAL | Config du gate éditable sans garde-fou | novahiz/src/commands/gate.ts:48 | `gate.enabled:false` désactive l'enforcement silencieusement |
-| CRITICAL | Index des skills sous `**/build/**` ignoré | novahiz/src/spec.ts:137 | Skill retiré de l'index = jamais exigé, sans warning |
-| MAJEUR | AGENTS.md pointe vers un rules.json mort | opencode/AGENTS.md:7 | Le gate lit `~/.config/novahiz/catalog/rules.json`, pas le fichier cité |
-| MAJEUR | Slash-commands écrivant vers le rules.json mort | opencode/commands/novahiz_new_rule.md:5 | novahiz_new_rule, novahiz_rule, novahiz_rule_audit sans effet sur le gate |
-| MAJEUR | R8-docs n'utilise que des chemins relatifs | novahiz/catalog/rules.json:51 | Tout chemin absolu vers novahiz-docs échappe à la règle (vérifié par exécution) |
-| MAJEUR | bash/shell sans cible détectée renvoie allow:true | novahiz/src/commands/gate.ts:105 | python -e, curl -o, git apply, patch, tar -x passent sans compétence exigée |
-| MAJEUR | Catégorie `test` n'exige aucune compétence | novahiz/catalog/rules.json:35 | R6 exclut `test`, roadmap testing sans étape kind:skill : requiredSkills vide |
-| MAJEUR | tsx/jsx/vue/svelte/astro hors R9, R13, R14 | novahiz/catalog/rules.json:98 | Classés `design` : ni code-review ni skills design sans prompt design-ui |
-| MINEUR | Échec du classifieur vide les catégories de session | opencode/plugins/novahiz.ts:400 | 7 règles à promptCategories inapplicables, log warning seulement (doublon axe plugins) |
-| MINEUR | contentExcludes regex invalide = fail-open silencieux | novahiz/src/gate.ts:306 | catch `continue` alors que le commentaire annonce fail-closed |
-| MINEUR | Règle avec `when` vide jamais applicable | novahiz/src/gate.ts:215 | Corruption partielle = règle morte, non remontée |
-| MINEUR | R9 déclenché sur tout fichier code | novahiz/catalog/rules.json:59 | novahiz-code-review exigé sur chaque édition de code, hors tâche de revue |
-| MINEUR | R11 plus large que sa description | novahiz/catalog/rules.json:78 | Tout prompt design-ui exige novahiz-wcag-audit |
-| MINEUR | Sensibilité à la casse divergente plugin vs CLI | novahiz/src/commands/gate.ts:60 | Outil en majuscules passe le filtre plugin, bloqué côté CLI |
-| MINEUR | mode warn/audit renvoie allow:false mais laisse passer | novahiz/src/commands/gate.ts:239 | JSON suggère un blocage inexistant (exit 0) |
-| MINEUR | Base SQLite indisponible bloque tout édit sans dégradation | novahiz/src/commands/gate.ts:135 | Seul NOVAHIZ_GATE=off y échappe |
-| MINEUR | Outils MCP hors GATE_TOOLS non gateés | opencode/plugins/novahiz.ts:236 | cron_add_command_task, playwright_*, security_* écrivent sans friction |
-| SUGGESTION | Sélecteur mort `promptCategories:["security"]` | novahiz/catalog/rules.json:70 | Id inexistant parmi les 17 catégories (doublon axe catalogue) |
-| SUGGESTION | `fileClasses:["design"]` et contentMatches/minChange inutilisés | novahiz/src/gate.ts:210 | Sélecteurs sans consommateur |
-| SUGGESTION | R4 trop large sur `**/*.spec.ts` | novahiz/catalog/rules.json:19 | Tests Jest/Vitest déclenchent à tort novahiz-browser |
-| SUGGESTION | novahiz-gate SKILL.md : étapes edit/verify/advisory fausses | opencode/skills/novahiz-gate/SKILL.md:35 | Affirmation non conforme au code |
-| SUGGESTION | novahiz-gate SKILL.md : référence de lignes périmée | opencode/skills/novahiz-gate/SKILL.md:37 | Repère 229-236 au lieu de 352-359 |
-| SUGGESTION | novahiz-gate SKILL.md : « sans message » faux | opencode/skills/novahiz-gate/SKILL.md:46 | Le CLI émet bien un warning |
-| SUGGESTION | docs/RULES.md ne liste que 9 règles sur 11 | novahiz/docs/RULES.md:34 | R7-assessment et R8-docs manquent |
-| SUGGESTION | docs/CATALOG.md defaultSkills obsolètes | novahiz/docs/CATALOG.md:26 | Anciens noms supabase au lieu de novahiz-supabase/novahiz-postgres |
-| SUGGESTION | docs/GATE.md classes de fichiers incorrectes | novahiz/docs/GATE.md:17 | tsx/json/sql décrits à tort, contraire à CLASS_BY_EXTENSION |
-| SUGGESTION | Tests nommant des règles supprimées | novahiz/tests/gate.test.ts:33 | 4 assertions passeraient aussi avec un gate vide |
-
-## 4. Tools — CLI, MCP tools, scripts (25 findings)
-
-Inventaire : 22 commandes CLI, 15 tools MCP, 3 scripts bin/scripts, 10 scripts dans skills. 0 CRITICAL, 3 HIGH, 8 MEDIUM, 10 LOW, 4 INFO.
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| HIGH | rmSync hors périmètre par traversal de chemin | scripts/capture-cli.mjs:69-71 | `--label`/`--out` avec `..` + `force:true` efface une arborescence arbitraire |
-| HIGH | postinstall sans consentement | package.json:52 | 5 npm i -g non pinnés + écriture d'opencode.jsonc + npm update |
-| HIGH | SAFE_BOOTSTRAP_BINS contient des shells | src/exec.ts:53-55 | `["cmd","/c",…]` dans providers.json saute toute la barrière |
-| MEDIUM | Gate CLI et gate MCP divergent | src/commands/gate.ts:141-198 | Le MCP saute traceCheck, recordEdit, enforcement_log |
-| MEDIUM | Binaire PATH exécuté à chaque ouverture de DB | src/db.ts:118-124 | `graft` via execFileSync hors de exec.ts, sans SAFE_TOKEN |
-| MEDIUM | Champ `file` non validé dans l'index memory | src/memory.ts:203-211 | Écriture hors project-memory/ via `..\..` dans index.json |
-| MEDIUM | Tools memory_* acceptent un root arbitraire | mcp/novahiz-tools/index.mjs:579 | Création/écriture n'importe où sur le disque (doublon axe MCP) |
-| MEDIUM | graft restore/export destructifs sans confirmation | src/commands/graft.ts:98-113 | Réécrit novahiz.sqlite sans --force ni sauvegarde |
-| MEDIUM | `task resume|current` annoncés mais non implémentés | src/commands/task.ts:66 | Actions listées valides, jamais traitées ; erreur gravée en référence de non-régression |
-| MEDIUM | Test novahiz_step n'exerce jamais l'écriture | tests/mcp.test.ts:191 | Assertion passe sur « aucune étape », régression invisible |
-| MEDIUM | npx -y sans pin pour skills tiers | catalog/providers.json:72 | Dernière version publiée exécutée sans hash |
-| LOW | confirm() interactif toujours `false` | src/commands/context.ts:81-85 | stdin sur TTY renvoie null : confirmation inexistante et non testée |
-| LOW | reviewTask n'isole pas les mutations par tâche | src/ledger.ts:531-539 | amendTodo/dropTodo sur les todos d'une autre tâche |
-| LOW | `kind` de todo non validé côté CLI | src/commands/task.ts:19 | Chaîne libre sans CHECK : todo verify validé sans preuve |
-| LOW | Entrées stdin non bornées | src/commands/context.ts:101-109 | readFileSync(0) sans limite vers JSON.parse |
-| LOW | Validation de session inégale CLI vs MCP | mcp/novahiz-tools/index.mjs:406 | Clés de table sans contrainte de longueur côté MCP |
-| LOW | Owner-globs sans garde ReDoS | src/ledger.ts:306-315 | globToRegExp sans isSafeRegexPattern |
-| LOW | `upgrade` exécute git pull sans contrôle | src/cli.ts:116 | Code auto-mis à jour avec hooks, sans --dry-run |
-| LOW | install/ écrit hors home sans garde | install/lib.mjs:39 | OPENCODE_CONFIG_DIR/XDG_CONFIG_HOME sans contrôle underHome |
-| LOW | chmod 0600 non appliqué sur Windows | src/db.ts:13-19 | novahiz.sqlite lisible par tout compte local |
-| LOW | openDb suit NOVAHIZ_DB sans garde | src/db.ts:10-11 | mkdirSync récursif à un chemin arbitraire |
-| INFO | README MCP : 10 tools annoncés pour 15 exposés | mcp/novahiz-tools/README.md:5 | memory_* manquants dans la doc |
-| INFO | doctor vérifie memory tools par source.includes | src/commands/doctor.ts:186 | Faux négatif silencieux au reformatage |
-| INFO | Test shell:true limité à src/ | tests/exec.test.ts:68 | mcp/, install/, scripts/ hors contrôle |
-| INFO | Redondances positives non documentées | src/commands/gate.ts:54 | Doublons CLI/MCP difficiles à maintenir |
-
-## 5. Catalogue de skills (22 findings)
-
-3 racines dans `skills.paths` : `opencode/skills` (41) + `~/.agents/skills` (53) + un skill (`impeccable`, R14) = 95. Aucun conflit de noms, `frontmatter.name == dossier` pour les 95.
-
-| Sévérité | Finding | Fichier:ligne | Impact |
-|---|---|---|---|
-| HIGH | Index Novahiz périmé : 84 sur 95 | novahiz/build/installed-skills.json:1 | 11 skills ignorés par gate et ranking |
-| HIGH | skills-lock.json vide, lu par personne | opencode/skills-lock.json:3 | Aucune détection de non-conformité |
-| MEDIUM | R10-security référence la catégorie "security" inexistante | novahiz/catalog/rules.json:66 | Seuls les prompts "audit" chargent novahiz-security |
-| MEDIUM | skills.paths indexe opencode/skills non indexé par Novahiz | opencode/opencode.jsonc:51 | Skill ajouté = invisible pour Novahiz sans resync manuel |
-| MEDIUM | skillRoots cible le sous-dossier impeccable | novahiz/novahiz.config.json:5 | Skills ajoutés dans .agents/skills ignorés par Novahiz |
-| MEDIUM | 55 skills sur 95 sans entrée dans overrides.json | novahiz/catalog/overrides.json:1 | Aucun boost de ranking, impeccable inclus alors que R14 l'exige |
-| LOW | README annonce 84 skills (×3 occurrences) | novahiz/README.md:5,55,245 | Doc en décalage avec l'arborescence |
-| LOW | Test d'hygiène asserte "84 skills" | novahiz/tests/plugin.hygiene.test.ts:91 | CI valide un chiffre obsolète, ne détecte pas la dérive |
-| LOW | Descriptions sans clause de routage "quand" | gate, memory, dart-collect-coverage (faute "packge"), expo-upgrade, expo-native-ui | Routage par description moins fiable |
-| LOW | Paquet yaml cassé : dist/compose/composer.js absent | opencode/node_modules/yaml/dist/index.js:3 | require('yaml') MODULE_NOT_FOUND depuis la racine OpenCode |
-| INFO | Aucun code ne synchronise novahiz/skills ↔ opencode/skills | novahiz/install/install.mjs:167 | Égalité actuelle par resync git manuel uniquement |
-| INFO | novahiz-postgres cite une reference inexistante | opencode/skills/novahiz-postgres/SKILL.md:93 | Référence morte documentée |
-| INFO | skill-authoring cite references/x.md (placeholder) | opencode/skills/skill-authoring/SKILL.md:67 | Exemple générique |
-| INFO | expo-overview cite references/native-ios.md absent | ~/.agents/skills/expo-overview/SKILL.md:77 | Résout seulement dans eas-app-stores |
-| INFO | Doublons sémantiques gate/novahiz-gate, memory/novahiz-memory, supabase/novahiz-supabase | opencode/skills/ | Chevauchement sans collision d'index |
-| INFO | Aucun parasite .bak/.old ni skill orphelin, 26 sous-dossiers agents/ sans SKILL.md = définitions de sous-agents | opencode/skills | Conforme |
-
-## Doublons inter-axes
-
-1. postinstall `package.json:52` (tools HIGH, MCP LOW)
-2. memory_* root arbitraire `index.mjs:579` (tools MEDIUM, MCP LOW)
-3. providers.json npx flottant (tools MEDIUM, MCP MEDIUM)
-4. classify fail-open `novahiz.ts:401` (plugins HIGH, rules MINEUR)
-5. catégorie "security" inexistante (catalogue MEDIUM, rules SUGGESTION)
-6. spawnSync 10 s (plugins MEDIUM côté gel UI, MCP MEDIUM côté refus fail-closed) : angles différents, même cause racine
-
-## Conformité doc ↔ code
-
-| Document | Affirme | Réalité |
-|---|---|---|
-| `opencode/AGENTS.md` | rules.json dans opencode/catalog | Fichier mort ; le gate lit `~/.config/novahiz/catalog/rules.json` |
-| `novahiz/docs/RULES.md` | 9 règles | 11 règles existent (R7, R8 absentes du tableau) |
-| `novahiz/docs/CATALOG.md` | defaultSkills supabase/* | Renommés novahiz-supabase/novahiz-postgres |
-| `novahiz/docs/GATE.md` | tsx=code, json=config, sql=data | Contraire à CLASS_BY_EXTENSION |
-| `opencode/skills/novahiz-gate/SKILL.md` | étapes edit/verify/advisory dans requiredSkills ; lignes 229-236 ; « sans message » | Faux sur les 3 points |
-| `opencode/mcp-audit-report.md` | posture GOOD | 11 affirmations fausses sur la config antérieure au 24/09 |
-| `novahiz/README.md` | 84 skills | 95 présents |
-| `mcp/novahiz-tools/README.md` | 10 tools | 15 exposés |
-
-## Couverture tests — signaux faibles
-
-- `tests/mcp.test.ts:191` : test `novahiz_step` qui n'exerce jamais l'écriture (assertion sur « aucune étape »).
-- `src/commands/context.ts:81` : `confirm()` toujours false sur TTY, comportement non testable en l'état.
-- `tests/exec.test.ts:68` : parcours `shell:true` limité à `src/` (mcp, install, scripts hors garde-fou).
-- `tests/gate.test.ts:33` : assertions sur des règles supprimées, passeraient avec un gate vide.
-- `tests/plugin.hygiene.test.ts:91` : CI valide le chiffre « 84 skills ».
-- `scripts/capture-cli.mjs:42` : une erreur obtenue est gravée comme référence de non-régression (`task resume`).
-
-## Recommandations priorisées
-
-Rien n'a été appliqué. Ordre de traitement proposé.
-
-**P0 — fail-open du gate (sécurité du système lui-même)**
-1. gate.ts:280 : calculer le tier sur le prompt, pas sur le contenu.
-2. commands/gate.ts:48 : rendre `gate.enabled`/`gate.tools` non désactivables sans changement externe tracé (ou au minimum logger fort).
-3. spec.ts:137 : sortir l'index de `build/` ou le signer/valider à la lecture, avec warning si skill retiré.
-4. plugins/novahiz.ts:401, 486, 494 : classifieur en échec = refus (fail-closed), sessionID invalide = refus, validation stricte des noms de skills.
-5. exec.ts:53 : retirer cmd/bash de l'allowlist, valider les arguments bootstrap.
-6. capture-cli.mjs:69 : résoudre puis vérifier que la cible reste sous le dossier de sortie avant rmSync.
-
-**P1 — surfaces d'exécution et d'index**
-7. package.json:52 : supprimer ou rendre opt-in le postinstall.
-8. executor.py:119 : valider/whitelist les commandes cron ; SSRF : restreindre les URL.
-9. opencode.jsonc:17 : exclure logs/DB de l'index narsil.
-10. opencode.jsonc:30 : activer SECURITY_MCP_SHARED_SECRET ou retirer les outils auto-apply.
-11. opencode.jsonc:24 : passer `env` à `environment` si le schéma l'exige, ou corriger le schéma.
-12. Rebuild de l'index : 84 → 95 ; remplir ou supprimer skills-lock.json ; corriger skillRoots.
-
-**P2 — dette, doc, cohérence**
-13. Resynchroniser AGENTS.md, RULES.md, CATALOG.md, GATE.md, novahiz-gate/SKILL.md, mcp-audit-report.md, README (84 → 95).
-14. R8 en chemins absolus normalisés ; catégorie `test` avec compétences ; .tsx/.jsx dans R9/R13/R14.
-15. Dé-dupliquer le code du plugin (src/adapters/plugins) et documenter les redondances de sécurité.
-16. Purger `mcp-auth.json`, shims npm orphelins, `.mcp/audit/tool-calls.jsonl` tracké.
-
-## Inconnus et réserves
-
-- L'agent MCP a déclaré `TOTAL: 32` mais liste 31 lignes de findings (écart de comptage non résolu ; le total général de 131 suit les lignes listées).
-- Non évalué : comportement runtime réel des sessions Edge/Orca, contenu des conversations, CVE des dépendances installées (hors périmètre).
-- Les décomptes de sévérité par axe viennent des agents eux-mêmes, dont les barèmes diffèrent (CRITICAL/MAJEUR/MINEUR/SUGGESTION côté gate).
-- Aucun finding n'a été re-testé indépendamment après coup : les exécutions de preuve citées (R8 absolu, tests passants) sont celles des sous-agents.