@mjasnikovs/pi-task 0.38.17 → 0.38.18

package/dist/config/config.d.ts

@@ -215,5 +215,16 @@ export declare const CONFIG_LOADERS: {
  * into the config object.
  */
 export declare function loadConfig(raw: unknown): PiTaskConfig;
+/**
+ * Where the saved config lives. `PI_TASK_CONFIG_PATH` overrides it.
+ *
+ * The override exists because this module loads the real file at import time,
+ * which made every test that reads a config value depend on the developer's own
+ * `~/.config/pi-task/config.json` — a machine-local `"debugLogs": "off"` failed
+ * 12 tests here while CI (no config file at all) stayed green. The test preload
+ * points this at a path under the tmp dir that never exists, so tests always see
+ * DEFAULT_CONFIG. Read once at module eval: the preload runs before any import.
+ */
+export declare const CONFIG_PATH_ENV = "PI_TASK_CONFIG_PATH";
 export declare function getConfig(): PiTaskConfig;
 export declare function saveConfig(config: PiTaskConfig): Promise<void>;

package/dist/config/config.js

@@ -182,7 +182,19 @@ export function loadConfig(raw) {
     }
     return out;
 }
-const CONFIG_PATH = path.join(os.homedir(), '.config', 'pi-task', 'config.json');
+/**
+ * Where the saved config lives. `PI_TASK_CONFIG_PATH` overrides it.
+ *
+ * The override exists because this module loads the real file at import time,
+ * which made every test that reads a config value depend on the developer's own
+ * `~/.config/pi-task/config.json` — a machine-local `"debugLogs": "off"` failed
+ * 12 tests here while CI (no config file at all) stayed green. The test preload
+ * points this at a path under the tmp dir that never exists, so tests always see
+ * DEFAULT_CONFIG. Read once at module eval: the preload runs before any import.
+ */
+export const CONFIG_PATH_ENV = 'PI_TASK_CONFIG_PATH';
+const CONFIG_PATH = process.env[CONFIG_PATH_ENV]?.trim()
+    || path.join(os.homedir(), '.config', 'pi-task', 'config.json');
 const _g = globalThis;
 if (!_g.__piTaskConfig) {
     _g.__piTaskConfig = { config: { ...DEFAULT_CONFIG }, loaded: false };

package/dist/task/widget.js

@@ -115,21 +115,35 @@ export function buildWidgetData(s) {
 export function startWidget(ctx, getState) {
     if (!ctx.hasUI)
         return () => { };
+    // `ctx.ui` THROWS once the ctx goes stale (/reload, session replacement), so
+    // the theme read belongs INSIDE the guard, not one line above it. render()
+    // runs from a timer, where an unguarded throw is an uncaughtException that
+    // kills the whole pi process — and a swallowed one would throw again on
+    // every tick, so a stale ctx latches and stops the timer. Issue #15.
+    let stale = false;
     const render = () => {
+        if (stale)
+            return;
         const s = getState();
-        const lines = s ? buildWidgetLines(s, ctx.ui.theme) : undefined;
-        const plain = s ? buildWidgetLines(s, undefined) : undefined; // un-themed for the wire
+        const plain = s ? buildWidgetLines(s, undefined) : undefined; // un-themed for the wire; needs no ctx
         try {
+            const lines = s ? buildWidgetLines(s, ctx.ui.theme) : undefined;
             ctx.ui.setWidget(WIDGET_KEY, lines);
         }
         catch {
-            /* stale ctx */
+            stale = true;
+            clearInterval(timer);
+            return;
         }
         setTaskWidget(plain, s ? buildWidgetData(s) : null);
     };
-    render();
+    // The timer is created BEFORE the first render so `timer` is always bound
+    // when render's catch reaches for it: a ctx already stale on the very first
+    // paint now stops the loop there, instead of arming an interval that wakes
+    // up and returns early forever.
     const timer = setInterval(render, WIDGET_REFRESH_MS);
     timer.unref?.();
+    render();
     return () => {
         clearInterval(timer);
         try {
@@ -192,21 +206,32 @@ export function buildAutoLoaderData(s) {
 export function startAutoLoader(ctx, getState) {
     if (!ctx.hasUI)
         return () => { };
+    // Same staleness hazard as startWidget: the theme read must sit inside the
+    // guard, and a stale ctx stops the timer rather than throwing every tick.
+    let stale = false;
     const render = () => {
+        if (stale)
+            return;
         const s = getState();
-        const lines = s ? buildAutoLoaderLines(s, ctx.ui.theme) : undefined;
-        const plain = s ? buildAutoLoaderLines(s, undefined) : undefined;
+        const plain = s ? buildAutoLoaderLines(s, undefined) : undefined; // needs no ctx
         try {
+            const lines = s ? buildAutoLoaderLines(s, ctx.ui.theme) : undefined;
             ctx.ui.setWidget(AUTO_WIDGET_KEY, lines);
         }
         catch {
-            /* stale ctx */
+            stale = true;
+            clearInterval(timer);
+            return;
         }
         setTaskWidget(plain, s ? buildAutoLoaderData(s) : null);
     };
-    render();
+    // The timer is created BEFORE the first render so `timer` is always bound
+    // when render's catch reaches for it: a ctx already stale on the very first
+    // paint now stops the loop there, instead of arming an interval that wakes
+    // up and returns early forever.
     const timer = setInterval(render, WIDGET_REFRESH_MS);
     timer.unref?.();
+    render();
     return () => {
         clearInterval(timer);
         try {
@@ -249,7 +274,14 @@ export function buildImplData(s) {
 export function flashTerminalWidget(ctx, state, taskId, reason) {
     if (!ctx.hasUI)
         return;
-    const theme = ctx.ui.theme;
+    // Same staleness hazard as the render timers: bail rather than throw.
+    let theme;
+    try {
+        theme = ctx.ui.theme;
+    }
+    catch {
+        return;
+    }
     let line;
     let clearMs;
     if (state === 'cancelled') {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.38.17",
+    "version": "0.38.18",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",