novahiz 0.5.1 → 0.5.2

package/CHANGELOG.md

@@ -5,6 +5,33 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.5.2] - 2026-10-05
+
+Maintenance release closing the three leftovers traced right after `0.5.1`:
+the installer's dead `plugin` config key, the unread `tokens` block of the
+example config, and the scheduler's 0 ms wake loop on a busy task.
+
+### Fixed
+
+- `install/install.mjs` no longer writes a `"plugin"` array of **file paths**
+  into a freshly created `opencode.jsonc`. opencode never read it (the plugins
+  are loaded from `<config>/plugins`, proved in E2E: the key pointed at the
+  repo while the loaded copies came from the plugins directory) and a file
+  entry triggered `configured plugin path must be a directory` at every
+  startup. Existing configs are left untouched.
+- `novahiz.config.example.json` no longer advertises a `tokens` block
+  (`enabled`, `trimOutputs`, `keepHeadLines`, `dedupeReads`, `capOutputTokens`,
+  `trimTools`, …) that no code ever read — a leftover of a layer that was never
+  shipped. The example now carries exactly the keys `defaultConfig()` writes,
+  and `docs/TOKENS.md` states it instead of pointing at a block that should
+  not be there.
+- `mcp/clepsydre/src/scheduler.ts`: `arm()` no longer arms a timer for a task
+  that is already running. A past-due schedule on a busy task used to feed a
+  0 ms `wake → skip → arm` loop (one task-store read per turn) for the whole
+  execution, and a manual trigger could sustain it. The lock release now goes
+  through `release()`, which re-evaluates the calendar once the task is free,
+  so a due occurrence is still caught up exactly once instead of by the spin.
+
 ## [0.5.1] - 2026-10-05
 
 Fix-forward release: the `v0.5.0` tag was pushed but its CI never reached

package/docs/TOKENS.md

@@ -95,9 +95,10 @@ from the local enforcement database.
 
 ## Not implemented
 
-`novahiz.config.example.json` still carries a `tokens` block (`enabled`,
-`trimOutputs`, `keepHeadLines`, `keepTailLines`, `keepErrorLines`,
-`dedupeReads`, `capOutputTokens`, `trimTools`, `readTools`). No code reads it:
-it is a leftover of a layer that was never shipped, and neither the plugin nor
-`novahiz tokens` consumes it. Configure the plugin with the environment
-variables above instead.
+There is no `tokens` section in `novahiz.config.example.json`: no code ever
+read the one it carried (`enabled`, `trimOutputs`, `keepHeadLines`,
+`keepTailLines`, `keepErrorLines`, `dedupeReads`, `capOutputTokens`,
+`trimTools`, `readTools`) — a leftover of a layer that was never shipped, and
+neither the plugin nor `novahiz tokens` consumes it. It has been removed from
+the example rather than left to contradict this document. Token economy is
+configured with the environment variables above, and nothing else.

package/install/install.mjs

@@ -236,9 +236,11 @@ async function main() {
     }
   }
 
-  // opencode charge les .ts posés dans <config>/plugins: E2E montre que le
-  // registre des plugins déclare le chemin de ce répertoire, pas celui du
-  // tableau "plugin" de la config — les deux restent en cohérence.
+  // opencode charge automatiquement les .ts de <config>/plugins. Preuve E2E:
+  // le tableau "plugin" de la config pointait vers ce depôt alors que les
+  // plugins charges venaient de <config>/plugins — le tableau est donc retire
+  // du modele de config, une entree fichier y declenchait l'avertissement
+  // "configured plugin path must be a directory" a chaque demarrage.
   for (const pluginFile of ["novahiz-plugin.ts", "novahiz-token-economy.ts"]) {
     const pluginSource = join(home, "adapters", "opencode", pluginFile);
     const pluginTarget = join(pluginsDir, pluginFile);
@@ -519,9 +521,6 @@ async function main() {
         "skills": {
           "paths": [skillsDir]
         },
-        "plugin": [
-          join(home, "adapters", "opencode", "novahiz-plugin.ts")
-        ],
         "compaction": {
           "auto": true,
           "prune": true,

package/mcp/clepsydre/src/scheduler.ts

@@ -11,6 +11,10 @@
 // bascule toujours en traitement special : une tache n'execute jamais deux
 // fois en parallele (verrou `busy`), meme si la cadence est plus rapide que
 // la duree de l'execution.
+// Le verrou ne s'arme jamais : `arm()` ignore les taches busy, sinon une
+// echeance passee donnerait un timer 0 ms en boucle (`skip -> arm`) pendant
+// toute l'execution ; la liberation re-evalue les echeances et rattrape
+// l'occurrence due une seule fois.
 
 import { appendExecution, findTask, loadTasks, mutateTasks, newId, saveTasks, type Task } from "./store.ts";
 import { nextRun, parseSchedule, type Schedule } from "./cron.ts";
@@ -125,6 +129,11 @@ export class Scheduler {
   }
 
   /** Arme le prochain reveil sur l'echeance la plus proche.
+   *  Une tache en execution (verrou `busy`) est ignoree : son echeance passe
+   *  reste en file jusqu'a la liberation, et un reveil dessus ne ferait que
+   *  boucler (`timer 0 ms -> wake -> skip -> arm`) pendant toute
+   *  l'execution, a coup de lecture disque. Le reveil de liberation
+   *  (`release`) re-evalue les echeances quand le verrou saute.
    *  Retourne le delai retenu (null : aucune tache a armer), pour test. */
   arm(): number | null {
     this.stopTimer();
@@ -133,6 +142,7 @@ export class Scheduler {
     let earliest: number | null = null;
     for (const task of tasks) {
       if (!task.enabled || !task.schedule) continue;
+      if (this.busy.has(task.id)) continue;
       const due = this.dueMs(task);
       if (due === null) continue;
       if (earliest === null || due < earliest) earliest = due;
@@ -239,6 +249,17 @@ export class Scheduler {
     this.arm();
   }
 
+  /** Libere le verrou d'une tache puis re-evalue les echeances.
+   *  Sans ce reveil, une occurrence devenue due pendant l'execution
+   *  (`triggerNow` / `runNow` n'avancent pas le calendrier) resterait due sur
+   *  disque en panne depuis que `arm()` ignore les taches busy. Le
+   *  rattrapage reste unique : le premier reveil avance `nextRunAt`, le
+   *  suivant ne trouve plus rien a lancer. */
+  private release(id: string): void {
+    this.busy.delete(id);
+    this.wake();
+  }
+
   /** Declenchement manuel DETACHE : pre-controles synchrones (tache connue,
    *  aucune execution en cours), puis execution en arriere-plan. Le serveur
    *  stdio reste disponible pour les autres requetes ; le resultat se lit
@@ -257,7 +278,7 @@ export class Scheduler {
         // non capte qui tuerait le serveur.
       })
       .finally(() => {
-        this.busy.delete(task.id);
+        this.release(task.id);
       });
     return { ok: true };
   }
@@ -275,7 +296,7 @@ export class Scheduler {
     } catch (error) {
       return { ok: false, reason: error instanceof Error ? error.message : String(error) };
     } finally {
-      this.busy.delete(task.id);
+      this.release(task.id);
     }
   }
 
@@ -300,7 +321,7 @@ export class Scheduler {
         .then(() => runOne(index + 1));
     };
     void runOne(0).finally(() => {
-      this.busy.delete(task.id);
+      this.release(task.id);
     });
   }
 

package/novahiz.config.example.json

@@ -19,17 +19,5 @@
     "autoRegister": true,
     "autoInstall": true,
     "disabled": []
-  },
-  "tokens": {
-    "enabled": true,
-    "trimOutputs": true,
-    "maxOutputBytes": 40000,
-    "keepHeadLines": 120,
-    "keepTailLines": 40,
-    "keepErrorLines": 40,
-    "dedupeReads": true,
-    "capOutputTokens": 0,
-    "trimTools": ["read", "bash", "shell"],
-    "readTools": ["read"]
   }
 }

package/package.json

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