novahiz 0.7.0 → 0.8.0

package/CHANGELOG.md

@@ -5,6 +5,72 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [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
+memory MCP degraded on machines without a home catalog, and the docs reading
+layer collapsed under the SQLite planner of node 22.
+
+### Fixed
+
+- **`packageRoot()` walk-up** (`src/spec.ts`): the community-path assumption
+  (entry script exactly one directory below the package root) held for `bin/`
+  and `src/` but not `mcp/<server>/index.mjs` — two levels down it resolved to
+  `mcp/catalog/` (absent), so on any machine whose home has no catalog yet
+  (CI, fresh `npm install -g novahiz` before the installer ran) every memory
+  call returned `failed: true` « Invalid or missing catalog file ». The
+  resolver now walks up to the nearest `package.json`, preferring
+  `name: "novahiz"`, pinned by a regression test that reproduces the exact CI
+  condition (`tests/memory-p5.test.ts`, `NOVAHIZ_HOME` empty) — red CI on
+  Linux since 0.7.0, dead memory tools on the documented community path.
+- **Docs search plan pinned with CROSS JOIN** (`mcp/novahiz-docs/src/store.ts`):
+  SQLite 3.51.3 (node:sqlite of node 22.23.x) reorders the plain JOIN — it
+  drives from `chunks(library)` and re-evaluates the full FTS MATCH for every
+  chunk of the library (50 × ~6 ms ≈ 315 ms instead of ~14 ms), collapsing
+  the 100 ms reading budget (742-1085 ms observed on CI runners). SQLite
+  never reorders a CROSS JOIN (`optoverview` 7.1.2, `lang_select` 2.2), so
+  the plan is now pinned on every runtime: 315.8 → 13.3 ms on 3.51.3,
+  unchanged 14.1 ms on 3.53.3 — same result set, only the plan changes.
+
 ## [0.7.0] - 2026-10-08
 
 Feature release: the Obsidian vault becomes a first-class citizen of the

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/spec.js

@@ -282,15 +282,40 @@ function readCatalog(path) {
 // gates lifecycle scripts, and the postinstall only previews anyway), so a
 // brand-new machine has no catalog in its home yet — every command used to die
 // with ENOENT right after the documented install. The package ships its own
-// catalog/: fall back to it before failing. The entry script (bin/novahiz.mjs,
-// dist/cli.js or src/cli.ts) always sits exactly one directory below the
-// package root, so dirname(argv[1])/.. resolves in every supported shape.
+// catalog/: fall back to it before failing.
+// Audit 2026-10-08: the old one-level assumption (dirname(argv[1])/..) held for
+// bin/ and src/ but NOT for mcp/<server>/index.mjs — two levels down it pointed
+// at mcp/catalog/ (absent), so on any machine without a home catalog (CI,
+// fresh npm-global) the memory MCP degraded on every call: red CI on Linux,
+// memory_write/search dead for the community path. Walk up instead: the package
+// root is the nearest ancestor carrying a package.json, preferring name
+// "novahiz"; a stray nested manifest is kept only as a last resort.
 // Exported: doctor resolves the shipped skills/ the same way.
 export function packageRoot() {
     const entry = process.argv[1];
     if (!entry || entry.length === 0)
         return null;
-    return join(dirname(resolve(entry)), "..");
+    let dir = dirname(resolve(entry));
+    let loose = null;
+    for (let depth = 0; depth < 6; depth += 1) {
+        const manifest = join(dir, "package.json");
+        if (existsSync(manifest)) {
+            try {
+                const name = JSON.parse(readFileSync(manifest, "utf8")).name;
+                if (name === "novahiz")
+                    return dir;
+                loose ??= dir;
+            }
+            catch {
+                // manifest illisible: on remonte quand meme.
+            }
+        }
+        const parent = dirname(dir);
+        if (parent === dir)
+            break;
+        dir = parent;
+    }
+    return loose;
 }
 // The home copy always wins when present, because that is what the installer
 // refreshes.

package/mcp/novahiz-docs/src/store.ts

@@ -148,10 +148,19 @@ export function search(db: DatabaseSync, query: string, options: SearchOptions =
   if (match === null) return [];
   const limit = Math.min(Math.max(Math.trunc(options.limit ?? 5), 1), 50);
   const filterLibrary = options.library !== undefined && options.library.length > 0;
+  // CROSS JOIN et non JOIN : SQLite ne reordonne JAMAIS les operandes d'un
+  // CROSS JOIN (optoverview 7.1.2 / lang_select 2.2 — le mecanisme documente
+  // pour imposer l'ordre des boucles imbriquees). Avec un JOIN simple, le
+  // planner de SQLite 3.51.3 (node 22.23.x) sort chunks (index library) en
+  // boucle externe et re-evalue le MATCH complet pour chaque chunk : 50 x
+  // ~6 ms = ~315 ms au lieu de ~14 ms — le budget de lecture de 100 ms
+  // s'ecroule, CI rouge sur node 22. Le resultat est identique (jointure
+  // interne), seul le plan change. Bench du 2026-10-08 : 315.8 -> 13.3 ms
+  // sous 3.51.3, 14.0 -> 14.1 ms sous 3.53.3.
   const sql =
     "SELECT c.library, c.version, c.source_url, c.license, c.fetched_at, c.heading_path, c.ord, " +
     "snippet(chunks_fts, 0, '[', ']', '…', 16) AS snippet, bm25(chunks_fts) AS score " +
-    "FROM chunks_fts JOIN chunks c ON c.id = chunks_fts.rowid " +
+    "FROM chunks_fts CROSS JOIN chunks c ON c.id = chunks_fts.rowid " +
     `WHERE chunks_fts MATCH ?${filterLibrary ? " AND c.library = ?" : ""} ` +
     "ORDER BY bm25(chunks_fts) LIMIT ?";
   const params: Array<string | number> = [match];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novahiz",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "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/spec.ts

@@ -474,14 +474,36 @@ function readCatalog<T>(path: string): T {
 // gates lifecycle scripts, and the postinstall only previews anyway), so a
 // brand-new machine has no catalog in its home yet — every command used to die
 // with ENOENT right after the documented install. The package ships its own
-// catalog/: fall back to it before failing. The entry script (bin/novahiz.mjs,
-// dist/cli.js or src/cli.ts) always sits exactly one directory below the
-// package root, so dirname(argv[1])/.. resolves in every supported shape.
+// catalog/: fall back to it before failing.
+// Audit 2026-10-08: the old one-level assumption (dirname(argv[1])/..) held for
+// bin/ and src/ but NOT for mcp/<server>/index.mjs — two levels down it pointed
+// at mcp/catalog/ (absent), so on any machine without a home catalog (CI,
+// fresh npm-global) the memory MCP degraded on every call: red CI on Linux,
+// memory_write/search dead for the community path. Walk up instead: the package
+// root is the nearest ancestor carrying a package.json, preferring name
+// "novahiz"; a stray nested manifest is kept only as a last resort.
 // Exported: doctor resolves the shipped skills/ the same way.
 export function packageRoot(): string | null {
   const entry = process.argv[1];
   if (!entry || entry.length === 0) return null;
-  return join(dirname(resolve(entry)), "..");
+  let dir = dirname(resolve(entry));
+  let loose: string | null = null;
+  for (let depth = 0; depth < 6; depth += 1) {
+    const manifest = join(dir, "package.json");
+    if (existsSync(manifest)) {
+      try {
+        const name = (JSON.parse(readFileSync(manifest, "utf8")) as { name?: string }).name;
+        if (name === "novahiz") return dir;
+        loose ??= dir;
+      } catch {
+        // manifest illisible: on remonte quand meme.
+      }
+    }
+    const parent = dirname(dir);
+    if (parent === dir) break;
+    dir = parent;
+  }
+  return loose;
 }
 
 // The home copy always wins when present, because that is what the installer