novahiz 0.11.2 → 0.11.4

package/CHANGELOG.md

@@ -5,6 +5,32 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.11.4] - 2026-10-22
+
+### Fixed
+
+- **`novahiz setup` now completes on Linux** - `runSync()` executed
+  `home/src/cli.ts` directly with Node, which cannot read TypeScript
+  without `--experimental-strip-types`. On a fresh global install the home
+  has no `dist/` build, so the sync step failed with
+  `ERR_UNKNOWN_FILE_EXTENSION`. The flag is now added so the two-pass
+  install (core then extras) completes on Linux, macOS and Windows.
+
+## [0.11.3] - 2026-10-22
+
+### Fixed
+
+- **`--clean` now sees the files an older version left behind** - the install
+  manifest never recorded what earlier installers had copied, so a home
+  upgraded across several versions kept dead entry points (`install/bootstrap.mjs`,
+  `install/install.sh`, `install/install.ps1`, `bin/veille-mensuelle.ps1`) that
+  `--clean` could not detect. `install/retired.json` now lists them with the
+  sha256 of the published content, line endings normalised to LF, and cleanup
+  removes an entry only when the file on disk still matches that fingerprint.
+  A file edited by hand no longer matches, comes back as kept, and is never
+  deleted. The inventory is read from the package, never from the home, so an
+  installation cannot widen the list of what it erases.
+
 ## [0.11.2] - 2026-10-22
 
 ### Fixed

package/README.md

@@ -4,7 +4,7 @@
 
 17 catégories, 94 skills, 13 gate rules, 8 MCP providers — tout déterministe, tout local, tout JSON.
 
-> **Version 0.11.0** — `npm install -g novahiz` pose le CLI, et le premier `novahiz` lancé termine l'installation (core puis extras). `CHANGELOG.md` et `scripts/` sont dans le tarball.
+> **Version 0.11.3** — `npm install -g novahiz` pose le CLI, et le premier `novahiz` lancé termine l'installation (core puis extras). `CHANGELOG.md` et `scripts/` sont dans le tarball.
 
 ```
 ┌──────────────────────────────────────────────────────────────────────────┐

package/dist/cli.js

@@ -35,7 +35,10 @@ function runSync() {
     const home = homeDir();
     const cli = join(home, "src", "cli.ts");
     if (existsSync(cli)) {
-        spawnSync(process.execPath, [cli, "sync"], {
+        // src/cli.ts est du TypeScript : Node ne le lit pas nativement. On ajoute
+        // --experimental-strip-types pour pouvoir l'exécuter depuis le home,
+        // qui ne contient pas le build dist/ (réservé au paquet publié).
+        spawnSync(process.execPath, ["--experimental-strip-types", cli, "sync"], {
             encoding: "utf8",
             stdio: "inherit",
             env: { ...process.env, NOVAHIZ_HOME: home }

package/install/install.mjs

@@ -52,6 +52,21 @@ const STEPS = [
   "Cleanup previous install"
 ];
 
+/**
+ * Inventaire des fichiers retires du paquet (`install/retired.json`).
+ *
+ * Le manifeste d'installation ne peut pas connaitre ce qu'une version
+ * anterieure a copie avant que ce champ existe : cet inventaire comble le trou.
+ * Il est lu depuis le paquet installe, jamais depuis le home, pour qu'une
+ * installation ne puisse pas elargir elle-meme la liste de ce qu'elle efface.
+ * Un fichier absent ou illisible rend une liste vide : le nettoyage retombe
+ * alors sur le seul manifeste, jamais sur une supposition.
+ */
+function readRetired() {
+  const parsed = readJson(join(repoRoot(import.meta.url), "install", "retired.json"), { files: [] });
+  return Array.isArray(parsed?.files) ? parsed : { files: [] };
+}
+
 const CORE_ITEMS = [
   "src",
   "catalog",
@@ -896,9 +911,10 @@ async function main() {
   // --- Nettoyage de l'installation precedente ------------------------------
   // Une mise a jour ne peut pas laisser derriere elle des skills renommes, des
   // plugins retires ou des commandes devenues obsoles : ils restent charges au
-  // prochain demarrage. L'effacement ne s'appuie que sur une preuve de
-  // propriete, le manifeste d'installation precedent ; un fichier pose par un
-  // pack ou par l'utilisateur n'y figure pas et n'est jamais retenu.
+  // prochain demarrage. L'effacement s'appuie sur deux preuves de propriete : le
+  // manifeste d'installation precedent, et pour les fichiers anterieurs a ce
+  // manifeste, l'empreinte publiee dans `retired.json`. Un fichier pose par un
+  // pack, par l'utilisateur, ou modifie a la main n'est jamais retenu.
   ui.step("Cleanup previous install");
   const ownedRoots = [
     { dest: skillsDir, src: join(home, "skills") },
@@ -906,9 +922,11 @@ async function main() {
     { dest: join(configDir, "agent"), src: join(home, "adapters", "opencode", "agent") },
     { dest: join(configDir, "commands"), src: join(home, "adapters", "opencode", "commands") }
   ];
-  const cleanup = cleanupPreviousInstall({ home, configDir, roots: ownedRoots, clean, dryRun, stepFailure });
+  const retired = readRetired();
+  const cleanup = cleanupPreviousInstall({ home, configDir, roots: ownedRoots, clean, dryRun, stepFailure, retired });
   for (const path of cleanup.removed) note(`  removed: ${relative(home, path)}`);
   for (const path of cleanup.stale ?? []) note(`  would remove: ${relative(home, path)}`);
+  for (const path of cleanup.kept ?? []) note(`  kept (modified since install): ${relative(home, path)}`);
   if (cleanup.removed.length > 0) {
     // L'index tient encore la liste des skills supprimes : on le rejoue une
     // fois, sinon le prochain routage classifiera vers un skill parti.

package/install/lib.d.mts

@@ -100,6 +100,8 @@ export interface CleanupResult {
   removed: string[];
   /** Present uniquement en mode a blanc : ce qui partirait. */
   stale?: string[];
+  /** Fichiers retires mais retouches depuis : jamais supprimes, seulement dits. */
+  kept: string[];
 }
 
 export declare function withinDir(value: string, dir: string): boolean;
@@ -108,6 +110,29 @@ export declare function staleOwnedPaths(
   roots: OwnedRoot[],
   exists?: (path: string) => boolean,
 ): string[];
+
+// --- Fichiers retires du paquet -------------------------------------------
+export interface RetiredFile {
+  /** Chemin relatif au home, tel que l'ancienne version l'installait. */
+  path: string;
+  /** Version qui a cesse de le livrer. */
+  version: string;
+  /** sha256 du contenu publie, fins de ligne normalisees en LF. */
+  sha256: string;
+}
+
+export interface RetiredInventory {
+  files: RetiredFile[];
+}
+
+/** Empreinte d'un contenu, `\r\n` ramene a `\n` avant hachage. */
+export declare function normalisedSha256(content: string | Buffer | Uint8Array): string;
+export declare function retiredOwnedPaths(
+  retired: RetiredInventory | null | undefined,
+  home: string,
+  readFile?: (path: string) => Buffer,
+  exists?: (path: string) => boolean,
+): { removed: string[]; kept: string[] };
 export declare function cleanupPreviousInstall(options: {
   home: string;
   configDir: string;
@@ -115,6 +140,7 @@ export declare function cleanupPreviousInstall(options: {
   clean: boolean;
   dryRun?: boolean;
   stepFailure?: string | null;
+  retired?: RetiredInventory | null;
 }): CleanupResult;
 export declare function detectedHarnesses(dirs: Record<string, string>, whichFn?: (cmd: string) => boolean): string[];
 /**

package/install/lib.mjs

@@ -1,4 +1,5 @@
 import { cpSync, existsSync, mkdirSync, readFileSync, readdirSync, rmSync, writeFileSync } from "node:fs";
+import { createHash } from "node:crypto";
 import { homedir } from "node:os";
 import { dirname, join, relative, resolve, sep } from "node:path";
 import { spawnSync } from "node:child_process";
@@ -306,14 +307,58 @@ export function staleOwnedPaths(created, roots, exists = existsSync) {
   return [...new Set(out)];
 }
 
+/**
+ * Empreinte d'un fichier, fins de ligne normalisees en LF.
+ *
+ * Le paquet est livre en LF, l'installation ecrit en CRLF sous Windows : sans
+ * normalisation, le meme fichier ne se reconnait plus lui-meme d'une machine a
+ * l'autre. C'est cette empreinte, et pas les octets bruts, qui sert de preuve
+ * de propriete aux fichiers retires.
+ */
+export function normalisedSha256(content) {
+  return createHash("sha256").update(content.toString("utf8").replace(/\r\n/g, "\n"), "utf8").digest("hex");
+}
+
+/**
+ * Fichiers retires du paquet et encore presents dans le home.
+ *
+ * Le manifeste ne peut pas les connaitre : un fichier copie par une version
+ * anterieure a l'invention du champ `created` n'y figure pas, et rien ne le
+ * reference plus cote source. La preuve de propriete devient donc le contenu :
+ * l'entree ne part que si le fichier sur disque porte exactement l'empreinte
+ * publiee dans `retired.json`. Un fichier modifie a la main ne correspond plus
+ * et n'est jamais supprime ; il ressort dans `kept` pour etre dit a l'utilisateur.
+ */
+export function retiredOwnedPaths(retired, home, readFile = readFileSync, exists = existsSync) {
+  const removed = [];
+  const kept = [];
+  for (const entry of retired?.files ?? []) {
+    if (!entry || typeof entry.path !== "string" || typeof entry.sha256 !== "string") continue;
+    if (entry.path.length === 0 || entry.path.startsWith("..")) continue;
+    const target = join(home, entry.path);
+    if (!exists(target)) continue;
+    let digest;
+    try {
+      digest = normalisedSha256(readFile(target));
+    } catch {
+      continue;
+    }
+    if (digest === entry.sha256.toLowerCase()) removed.push(target);
+    else kept.push(target);
+  }
+  return { removed: [...new Set(removed)], kept: [...new Set(kept)] };
+}
+
 /**
  * Decide et applique le nettoyage de l'installation precedente.
  *
- * Renvoie `{ status, detail, removed }`, `status` etant la valeur attendue par
- * ui.finishStep ("done" ou "skip"). Seule preuve de propriete : le manifeste
- * d'installation precedent, champ `created`, qui liste les chemins ecrits par
- * novahiz lui-meme. Un fichier de pack ou de l'utilisateur n'y figure pas, il
- * ne peut donc pas partir. Trois cas de sortie :
+ * Renvoie `{ status, detail, removed, kept }`, `status` etant la valeur attendue
+ * par ui.finishStep ("done" ou "skip"). Deux preuves de propriete, selon l'age
+ * du fichier : le manifeste d'installation (`created`, `backups`) pour ce que
+ * les versions recentes ont ecrit, et l'empreinte de `retired.json` pour ce que
+ * les versions anterieures ont pose sans jamais l'enregistrer. Un fichier de
+ * pack, de l'utilisateur, ou modifie a la main ne satisfait ni l'une ni l'autre
+ * et reste en place. Cas de sortie :
  *   - `--clean` absent, ou echec d'une etape precedente : skip, rien ne bouge ;
  *   - rien de retire : done ;
  *   - au moins un fichier retire : done, manifeste mis a jour.
@@ -321,12 +366,20 @@ export function staleOwnedPaths(created, roots, exists = existsSync) {
  * Vit ici et non dans install.mjs : ce module est sans effet de bord a
  * l'import, l'installateur execute main() au chargement.
  */
-export function cleanupPreviousInstall({ home, configDir, roots, clean, dryRun = false, stepFailure = null }) {
+export function cleanupPreviousInstall({
+  home,
+  configDir,
+  roots,
+  clean,
+  dryRun = false,
+  stepFailure = null,
+  retired = null
+}) {
   if (!clean) {
-    return { status: "skip", detail: "pass --clean to remove files from a previous install", removed: [] };
+    return { status: "skip", detail: "pass --clean to remove files from a previous install", removed: [], kept: [] };
   }
   if (stepFailure) {
-    return { status: "skip", detail: `an earlier step failed (${stepFailure}), cleanup aborted`, removed: [] };
+    return { status: "skip", detail: `an earlier step failed (${stepFailure}), cleanup aborted`, removed: [], kept: [] };
   }
   const manifest = loadManifest(home);
   // Deux preuves de propriete, pas une. `created` ne liste que les fichiers
@@ -338,15 +391,18 @@ export function cleanupPreviousInstall({ home, configDir, roots, clean, dryRun =
     ...(manifest.created ?? []),
     ...(manifest.backups ?? []).map((entry) => (entry && entry.path) || "").filter((path) => path.length > 0)
   ];
-  const stale = staleOwnedPaths(owned, roots);
+  const retiredScan = retiredOwnedPaths(retired, home);
+  const stale = [...new Set([...staleOwnedPaths(owned, roots), ...retiredScan.removed])];
+  const kept = retiredScan.kept.filter((item) => !stale.includes(item));
   if (stale.length === 0) {
-    return { status: "done", detail: "no leftover file from a previous install", removed: [] };
+    return { status: "done", detail: "no leftover file from a previous install", removed: [], kept };
   }
   if (dryRun) {
     return {
       status: "skip",
       detail: `${stale.length} file(s) would be removed by a real run with --clean`,
       removed: [],
+      kept,
       stale
     };
   }
@@ -364,7 +420,7 @@ export function cleanupPreviousInstall({ home, configDir, roots, clean, dryRun =
     ),
     cleanedAt: new Date().toISOString()
   });
-  return { status: "done", detail: `${removed.length} file(s) from a previous install removed`, removed };
+  return { status: "done", detail: `${removed.length} file(s) from a previous install removed`, removed, kept };
 }
 
 export function pruneEmptyDirs(paths, stops = []) {

package/install/retired.json

@@ -0,0 +1,25 @@
+{
+  "note": "Files Novahiz used to ship and no longer does. `install --clean` removes one only when the file on disk still carries the published sha256, computed with line endings normalised to LF. A file edited by hand no longer matches and is never deleted; the installer reports it as kept.",
+  "files": [
+    {
+      "path": "install/bootstrap.mjs",
+      "version": "0.11.0",
+      "sha256": "d4f90e6d091844b06590de1c7d201cc9bac2f2bd9f548f3c93b2bcfc68bd646c"
+    },
+    {
+      "path": "install/install.sh",
+      "version": "0.11.0",
+      "sha256": "9b2e7073f83c3aa9c0e9b269d8bec9187c22f0faebbcf0e71c2c74fb596dc2de"
+    },
+    {
+      "path": "install/install.ps1",
+      "version": "0.11.0",
+      "sha256": "6b44ef53bc9f0761c84a27776461c4c79683d832af037cc9fbbe82ec8dc7fb02"
+    },
+    {
+      "path": "bin/veille-mensuelle.ps1",
+      "version": "0.11.0",
+      "sha256": "4c88aeb86e274b4d5c2ddfd513e1c152189823e66959bbd578669277e4ffaabc"
+    }
+  ]
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novahiz",
-  "version": "0.11.2",
+  "version": "0.11.4",
   "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/cli.ts

@@ -39,7 +39,10 @@ function runSync(): void {
   const home = homeDir();
   const cli = join(home, "src", "cli.ts");
   if (existsSync(cli)) {
-    spawnSync(process.execPath, [cli, "sync"], {
+    // src/cli.ts est du TypeScript : Node ne le lit pas nativement. On ajoute
+    // --experimental-strip-types pour pouvoir l'exécuter depuis le home,
+    // qui ne contient pas le build dist/ (réservé au paquet publié).
+    spawnSync(process.execPath, ["--experimental-strip-types", cli, "sync"], {
       encoding: "utf8",
       stdio: "inherit",
       env: { ...process.env, NOVAHIZ_HOME: home }