novahiz 0.7.1 → 0.8.1

package/CHANGELOG.md

@@ -5,6 +5,60 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.8.1] - 2026-10-10
+
+Patch: `novahiz upgrade` always reported "up to date" on the npm channel.
+
+### Fixed
+
+- **`novahiz upgrade` reads the published version again**
+  (`src/commands/upgrade.ts`): the `npm view` call ran with `stdio: "ignore"`,
+  which makes `spawnSync` return `stdout: null` — `latest` was always empty,
+  `compareVersions(local, "")` always won, and the command printed
+  `up to date (npm latest: )` even when a newer release existed, so `--apply`
+  could never trigger. The call now captures stdout (`"pipe"`), with a
+  regression test spawning `npm --version` locally (no network). Inherited
+  since 0.7.1.
+
+## [0.8.0] - 2026-10-09
+
+Minor release: total Impeccable coverage on the design roadmap, a repair
+command for the optional skill packs, and an honest diagnosis of why packs go
+missing on fresh installs.
+
+### Added
+
+- **Design-ui roadmap: total Impeccable coverage** (`catalog/categories.json`):
+  23 `kind: skill` steps carry the whole official playbook — init, document,
+  shape, critique, then the targeted fixes (layout, typeset, colorize,
+  animate, adapt, bolder, quieter, clarify, delight, distill, extract,
+  onboard, optimize, overdrive, generate), then audit, harden, polish and the
+  detect → verify loop. Labels match the official descriptions published on
+  https://impeccable.style/cheatsheet. Every step is `optional: true` (a
+  checklist the agent follows, gate enforcement stays with R14);
+  `impeccable-live` is excluded on purpose (live browser session, not a
+  code path) and the deprecated `craft` alias is dropped. Rollback:
+  `categories.json.pre-impeccable-full.bak`.
+- **`/novahiz-skills-update` command**
+  (`adapters/opencode/commands/novahiz-skills-update.md`): reinstalls every
+  optional skill pack — dart, flutter, expo, impeccable — cleanly from their
+  official providers (`novahiz deps --install --yes`), refreshes the
+  Impeccable CLI/engine from npm, resyncs the skill index and verifies with
+  doctor. The one command a community member runs when packs are missing.
+
+### Changed
+
+- **`doctor` now names the git root cause** (`src/commands/doctor.ts`): a new
+  non-blocking `git (skill packs)` row — the official skills CLI behind every
+  `providers.json` `kind: skill` entry (dart-lang/skills, flutter/agent-plugins,
+  expo/skills, pbakaus/impeccable) requires git; without it those installs fail
+  silently on a fresh setup, which is exactly how a machine ends up with 33
+  pack skills missing and no explanation. Present → "installable"; missing →
+  install hint pointing at `/novahiz-skills-update`.
+- **`Referenced skills` hint** (`src/commands/doctor.ts`): the
+  "novahiz-install --dart-skills --flutter-skills" hint now points at
+  `/novahiz-skills-update, or novahiz deps --install`, the actual repair path.
+
 ## [0.7.1] - 2026-10-08
 
 Patch release: two installation-level repairs found while cutting 0.7.0 — the

package/README.md

@@ -2,9 +2,9 @@
 
 > **Couche d'application sans dépendance pour les agents de code IA** — classe les prompts, attribue des roadmaps d'exécution, bloque les éditions non sûres tant que les bonnes skills ne sont pas chargées, et persiste les décisions d'une session à l'autre — tout cela de façon déterministe, sans appel de modèle.
 
-17 catégories, 99 skills, 13 gate rules, 8 MCP providers — tout déterministe, tout local, tout JSON.
+17 catégories, 94 skills, 13 gate rules, 8 MCP providers — tout déterministe, tout local, tout JSON.
 
-> **Version 0.6.0** — publiée sur npm (`npm install -g --allow-scripts=novahiz novahiz` installe et configure tout : core, plugin, agent, skills packs et Playwright MCP). `CHANGELOG.md` et `scripts/` sont dans le tarball.
+> **Version 0.8.0** — publiée sur npm (`npm install -g --allow-scripts=novahiz novahiz` installe et configure tout : core, plugin, agent, skills packs et Playwright MCP). `CHANGELOG.md` et `scripts/` sont dans le tarball.
 
 ```
 ┌──────────────────────────────────────────────────────────────────────────┐
@@ -35,7 +35,7 @@
                  └─ tier, skills ──────│         └─ block : charger les skills nommées, retry une fois
 ```
 
-**17 catégories**, **99 skills**, **13 gate rules**, **8 MCP providers** — tout déterministe, tout local, tout JSON.
+**17 catégories**, **94 skills**, **13 gate rules**, **8 MCP providers** — tout déterministe, tout local, tout JSON.
 
 ---
 
@@ -66,7 +66,7 @@ npm antérieur à 11.16 exécute les scripts par défaut : `npm install -g novah
 Puis vérifiez :
 
 ```bash
-novahiz doctor   # Santé : 15 checks (17 avec --deep)
+novahiz doctor   # Santé : 17 checks (22 avec --deep)
 novahiz classify "fix the auth bug"
 ```
 
@@ -197,6 +197,8 @@ Huit catégories (`code`, `debug`, `browser`, `design-ui`, `database-supabase`,
 
 Clarify renvoie le travail au plan quand une réponse change l'architecture ; converge le renvoie aux tâches quand il trouve un écart. `flutter` et `expo` gardent les six mêmes étapes et insèrent leurs skills qualité autour d'implement. Le **tier** du classifier filtre la suite : `trivial` n'exécute presque rien, `lite` garde implement et converge, `full` parcourt toute la roadmap. Référence complète : [docs/ROADMAPS.md](docs/ROADMAPS.md).
 
+La roadmap `design-ui` porte en plus la couverture totale du playbook Impeccable : 23 étapes `skill` optionnelles couvrant l'intégralité des commandes officielles (`init`, `document`, `shape`, `critique`, ciblées `layout` → `generate`, `audit`, `harden`, `polish`, boucle `detect → verify`), avec les libellés des descriptions publiées sur [impeccable.style/cheatsheet](https://impeccable.style/cheatsheet). Elles forment une checklist pour l'agent ; l'application des règles design reste portée par `R14-impeccable`. `impeccable-live` est volontairement hors roadmap (session navigateur en direct, pas un chemin de code).
+
 ```mermaid
 flowchart LR
     subgraph "Fonctionnalité (code)"
@@ -310,7 +312,7 @@ Deux pièces complémentaires : la session consomme moins sans perdre en puissan
 
 ## Skills installées
 
-Novahiz livre 99 skills couvrant toutes les catégories :
+Novahiz livre 94 skills couvrant toutes les catégories — 46 dans le cœur du dépôt, les autres dans les packs officiels installés par défaut au setup :
 
 | Catégorie | Skills | Objectif |
 |-----------|--------|----------|
@@ -344,7 +346,9 @@ Novahiz auto-enregistre les serveurs MCP externes selon la catégorie du prompt
 | novahiz-scheduler | `mcp/clepsydre/index.mjs` (local, zéro dépendance) | Apache-2.0 | devops |
 | dart | `dart mcp-server` (Dart SDK) | BSD-3-Clause | code, debug, design-ui, flutter |
 
-Packs de skills (installés depuis les dépôts officiels, jamais vendorés) : `flutter/agent-plugins` (25 skills), `dart-lang/skills` (15 skills), `expo/skills` (19 skills, le groupe `expo-*` seulement ; les services payants `eas-*` exclus), `pbakaus/impeccable` (1 skill, la skill design upstream `impeccable`). Voir [docs/PROVIDERS.md](docs/PROVIDERS.md).
+Packs de skills (installés depuis les dépôts officiels, jamais vendorés, et livrés **par défaut** au setup via `providers.autoInstall`) : `flutter/agent-plugins` (10 skills), `dart-lang/skills` (15 skills), `expo/skills` (19 skills, le groupe `expo-*` seulement ; les services payants `eas-*` exclus), `pbakaus/impeccable` (1 skill, la skill design upstream `impeccable`).
+
+**`git` est requis** par le CLI officiel des skills : sans lui, l'installation des packs échoue silencieusement et les skills manquent au premier lancement. `novahiz doctor` le nomme (ligne `git (skill packs)`), et **`/novahiz-skills-update`** — ou `novahiz deps --install` — réinstalle proprement tous les packs depuis leurs providers officiels. Voir [docs/PROVIDERS.md](docs/PROVIDERS.md).
 
 Dépôts upstream et provenance complète des providers MCP et plugins opencode : [docs/PROVIDERS.md](docs/PROVIDERS.md), [docs/HARNESSES.md](docs/HARNESSES.md), [NOTICE.md](NOTICE.md).
 
@@ -374,7 +378,7 @@ Voir [docs/CONFIGURATION.md](docs/CONFIGURATION.md) pour toutes les options.
 | Commande | Objectif |
 |----------|----------|
 | `novahiz init` | Installation en une passe |
-| `novahiz doctor` | Diagnostic de santé 15-check (17 avec `--deep`) |
+| `novahiz doctor` | Diagnostic de santé 17-check (22 avec `--deep`) |
 | `novahiz status` | Classification + état du gate actuels |
 | `novahiz classify <text>` | Classer un prompt |
 | `novahiz gate` | Vérifier si une édition est autorisée |

package/adapters/opencode/commands/novahiz-skills-update.md

@@ -0,0 +1,29 @@
+---
+description: Reinstall every optional skill pack (dart, flutter, expo, impeccable) cleanly from their official providers. Arguments: $ARGUMENTS
+---
+
+1. Locate the Novahiz home: `NOVAHIZ_HOME` if set, otherwise `~/.config/novahiz`.
+
+2. Confirm `git` is on the PATH (`git --version`). The official skills CLI
+   behind every skill provider needs it; without it the install fails
+   silently. If git is missing, tell the user to install it first
+   (Windows: `winget install Git.Git`) and stop.
+
+3. Reinstall/update every provider from the official registry
+   (`catalog/providers.json` - dart-lang/skills, flutter/agent-plugins,
+   expo/skills, pbakaus/impeccable, plus the provider MCP servers):
+
+   `node <home>/src/cli.ts deps --install --yes`
+
+4. Refresh the Impeccable CLI and engine from npm (official package):
+
+   `npm install -g impeccable`
+   `impeccable install --global --yes --force`
+
+5. Realign the skill index:
+
+   `node <home>/src/cli.ts sync`
+
+6. Verify with `node <home>/src/cli.ts doctor`: the `Referenced skills` row
+   must show every pack present (the 7 `eas-*` paid-service skills stay
+   excluded by design). Report what was reinstalled.

package/catalog/categories.json

@@ -832,9 +832,27 @@
           ],
           "optional": true
         },
+        {
+          "id": "impeccable-init",
+          "label": "Impeccable init (give Impeccable the project context: PRODUCT.md audience and goals)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-document",
+          "label": "Impeccable document (record your visual system in DESIGN.md so future work can follow it)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
         {
           "id": "impeccable-shape",
-          "label": "Impeccable shape (UX/UI design brief before code)",
+          "label": "Impeccable shape (turn an idea into a design brief you can build from)",
           "kind": "skill",
           "requireSkills": [
             "impeccable"
@@ -851,7 +869,142 @@
         },
         {
           "id": "impeccable-critique",
-          "label": "Impeccable critique (systematic after UI work)",
+          "label": "Impeccable critique (find what holds the design back and what to improve first)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-layout",
+          "label": "Impeccable layout (arrange the page so people know where to look and what belongs together)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-typeset",
+          "label": "Impeccable typeset (make text easier to read, scan and recognize)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-colorize",
+          "label": "Impeccable colorize (color clarifies actions, states and the design's character)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-animate",
+          "label": "Impeccable animate (motion clarifies changes, relationships and feedback)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-adapt",
+          "label": "Impeccable adapt (make the design work on a different screen, device or platform)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-bolder",
+          "label": "Impeccable bolder (more presence, still true to the design)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-quieter",
+          "label": "Impeccable quieter (reduce visual noise, keep the personality)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-clarify",
+          "label": "Impeccable clarify (help people understand what happened and what to do next)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-delight",
+          "label": "Impeccable delight (meaningful moments: surprising, satisfying, unmistakably yours)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-distill",
+          "label": "Impeccable distill (reduce clutter so the important things are easier to find)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-extract",
+          "label": "Impeccable extract (turn repeated patterns into shared components and styles)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-onboard",
+          "label": "Impeccable onboard (help new users reach their first useful result)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-optimize",
+          "label": "Impeccable optimize (find what slows the interface, fix it, measure the result)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-overdrive",
+          "label": "Impeccable overdrive (push past the expected, make it feel extraordinary)",
+          "kind": "skill",
+          "requireSkills": [
+            "impeccable"
+          ],
+          "optional": true
+        },
+        {
+          "id": "impeccable-generate",
+          "label": "Impeccable generate (name an element, compare variations, keep the favorite)",
           "kind": "skill",
           "requireSkills": [
             "impeccable"
@@ -860,7 +1013,7 @@
         },
         {
           "id": "impeccable-audit",
-          "label": "Impeccable audit (a11y/perf/responsive, when the UI warrants it)",
+          "label": "Impeccable audit (find implementation problems and prioritize the fixes)",
           "kind": "skill",
           "requireSkills": [
             "impeccable"
@@ -869,7 +1022,7 @@
         },
         {
           "id": "impeccable-harden",
-          "label": "Impeccable harden (errors, i18n, edge cases before polish)",
+          "label": "Impeccable harden (keep the interface usable when data, connections or actions go wrong)",
           "kind": "skill",
           "requireSkills": [
             "impeccable"
@@ -878,7 +1031,7 @@
         },
         {
           "id": "impeccable-polish",
-          "label": "Impeccable polish (final pass before ship)",
+          "label": "Impeccable polish (make the page feel finished and work better)",
           "kind": "skill",
           "requireSkills": [
             "impeccable"

package/dist/commands/doctor.js

@@ -469,7 +469,7 @@ export function referencedSkillsCheck(spec, indexSkills, packageRootDir) {
             ? `${referenced.length} present`
             : absentShipped.length > 0
                 ? `missing from index: ${absent.join(", ")}`
-                : `${referenced.length - absent.length} present, ${absentPack} pack skills not installed (novahiz-install --dart-skills --flutter-skills)`,
+                : `${referenced.length - absent.length} present, ${absentPack} pack skills not installed (run /novahiz-skills-update, or novahiz deps --install)`,
         blocking: absentShipped.length > 0
     };
 }
@@ -537,6 +537,20 @@ export async function commandDoctor(parsed) {
                 : "all present",
         blocking: missingCli.length > 0
     });
+    // git backs the official skills CLI behind every providers.json kind:skill
+    // entry (dart, flutter, expo, impeccable). Without git those installs fail
+    // silently on a fresh setup, leaving the pack skills missing with no
+    // explanation - surface the root cause here as a non-blocking warning.
+    const hasGit = hasCommand("git");
+    checks.push({
+        id: "git",
+        label: "git (skill packs)",
+        ok: hasGit,
+        detail: hasGit
+            ? "present - provider skill packs (dart, flutter, expo, impeccable) installable"
+            : "missing - provider skill packs cannot install; run: winget install Git.Git (or install git, then rerun /novahiz-skills-update)",
+        blocking: false
+    });
     // Two probes, one per live enforcement layer, both deterministic against the
     // current ruleset (the old single probe fed README.md with empty categories,
     // which matches no rule since the prose content rules were removed):

package/dist/commands/upgrade.js

@@ -64,8 +64,12 @@ function readLocalVersion(root) {
         return "unknown";
     }
 }
-/** npm sous Windows est npm.cmd : spawnSync a besoin du shell, sinon ENOENT. */
-function runNpm(args, stdio) {
+/** npm sous Windows est npm.cmd : spawnSync a besoin du shell, sinon ENOENT.
+ *  `pipe` capture stdout/stderr (obligatoire pour `npm view` : `ignore`
+ *  retourne stdout=null et `latest` restait toujours vide), `inherit` pour
+ *  les sorties visibles de l'utilisateur (installation). Exporte pour le
+ *  test de regression. */
+export function runNpm(args, stdio) {
     const result = spawnSync("npm", args, {
         encoding: "utf8",
         stdio,
@@ -79,7 +83,7 @@ function runNpm(args, stdio) {
 }
 function npmFlow(parsed, root) {
     const local = readLocalVersion(root);
-    const view = runNpm(["view", "novahiz", "version"], "ignore");
+    const view = runNpm(["view", "novahiz", "version"], "pipe");
     if (view.status !== 0) {
         process.stderr.write(`novahiz upgrade: npm view failed (${view.stderr.trim() || `exit ${view.status}`}).\n`);
         process.exitCode = 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novahiz",
-  "version": "0.7.1",
+  "version": "0.8.1",
   "description": "Enforce AI agent skills before every file edit: prompt classifier, skill catalog, and pre-edit gate.",
   "type": "module",
   "license": "Apache-2.0",

package/src/commands/doctor.ts

@@ -469,7 +469,7 @@ export function referencedSkillsCheck(
         ? `${referenced.length} present`
         : absentShipped.length > 0
           ? `missing from index: ${absent.join(", ")}`
-          : `${referenced.length - absent.length} present, ${absentPack} pack skills not installed (novahiz-install --dart-skills --flutter-skills)`,
+          : `${referenced.length - absent.length} present, ${absentPack} pack skills not installed (run /novahiz-skills-update, or novahiz deps --install)`,
     blocking: absentShipped.length > 0
   };
 }
@@ -544,6 +544,21 @@ export async function commandDoctor(parsed: Parsed): Promise<void> {
     blocking: missingCli.length > 0
   });
 
+  // git backs the official skills CLI behind every providers.json kind:skill
+  // entry (dart, flutter, expo, impeccable). Without git those installs fail
+  // silently on a fresh setup, leaving the pack skills missing with no
+  // explanation - surface the root cause here as a non-blocking warning.
+  const hasGit = hasCommand("git");
+  checks.push({
+    id: "git",
+    label: "git (skill packs)",
+    ok: hasGit,
+    detail: hasGit
+      ? "present - provider skill packs (dart, flutter, expo, impeccable) installable"
+      : "missing - provider skill packs cannot install; run: winget install Git.Git (or install git, then rerun /novahiz-skills-update)",
+    blocking: false
+  });
+
   // Two probes, one per live enforcement layer, both deterministic against the
   // current ruleset (the old single probe fed README.md with empty categories,
   // which matches no rule since the prose content rules were removed):

package/src/commands/upgrade.ts

@@ -71,8 +71,12 @@ function readLocalVersion(root: string): string {
   }
 }
 
-/** npm sous Windows est npm.cmd : spawnSync a besoin du shell, sinon ENOENT. */
-function runNpm(args: string[], stdio: "inherit" | "ignore"): { status: number; stdout: string; stderr: string } {
+/** npm sous Windows est npm.cmd : spawnSync a besoin du shell, sinon ENOENT.
+ *  `pipe` capture stdout/stderr (obligatoire pour `npm view` : `ignore`
+ *  retourne stdout=null et `latest` restait toujours vide), `inherit` pour
+ *  les sorties visibles de l'utilisateur (installation). Exporte pour le
+ *  test de regression. */
+export function runNpm(args: string[], stdio: "inherit" | "pipe"): { status: number; stdout: string; stderr: string } {
   const result = spawnSync("npm", args, {
     encoding: "utf8",
     stdio,
@@ -87,7 +91,7 @@ function runNpm(args: string[], stdio: "inherit" | "ignore"): { status: number;
 
 function npmFlow(parsed: Parsed, root: string): void {
   const local = readLocalVersion(root);
-  const view = runNpm(["view", "novahiz", "version"], "ignore");
+  const view = runNpm(["view", "novahiz", "version"], "pipe");
   if (view.status !== 0) {
     process.stderr.write(`novahiz upgrade: npm view failed (${view.stderr.trim() || `exit ${view.status}`}).\n`);
     process.exitCode = 1;