pi-soly 2.5.2 → 2.5.4

package/index.ts

@@ -353,7 +353,17 @@ export default function solyExtension(pi: ExtensionAPI) {
 		d.home = os.homedir();
 		const model = ctx.model as { id?: string; provider?: string; reasoning?: boolean } | undefined;
 		d.modelId = model?.id ?? null;
+		const prevProvider = d.modelProvider;
 		d.modelProvider = model?.provider ?? null;
+		// TEMP DEBUG
+		if (prevProvider !== d.modelProvider) {
+			try {
+				const fs = require("node:fs");
+				const os = require("node:os");
+				const p = require("node:path");
+				fs.appendFileSync(p.join(os.tmpdir(), "pi-soly-quota-debug.log"), `[${new Date().toISOString()}] updateChromeData: modelProvider changed ${prevProvider} -> ${d.modelProvider}, modelId=${d.modelId}\n`);
+			} catch {}
+		}
 		d.reasoning = Boolean(model?.reasoning);
 		try {
 			d.thinkingLevel = pi.getThinkingLevel();
@@ -552,6 +562,14 @@ export default function solyExtension(pi: ExtensionAPI) {
 		// quotaPercent/quotaResetsLabel back for the footer to render).
 		if (quotaPoller) quotaPoller.stop();
 		quotaPoller = startQuotaPoller(chrome.data, () => getActiveConfig().chrome.enabled, () => chrome.poke());
+		// TEMP DEBUG
+		try {
+			const fs = await import("node:fs");
+			const os = await import("node:os");
+			const p = await import("node:path");
+			const dbgLog = p.join(os.tmpdir(), "pi-soly-quota-debug.log");
+			fs.appendFileSync(dbgLog, `[${new Date().toISOString()}] session_start: poller started, chrome.enabled=${getActiveConfig().chrome.enabled}, modelProvider=${chrome.data.modelProvider}\n`);
+		} catch {}
 		// Editors save in bursts (write to .tmp, rename, touch). Coalesce
 		// those rapid reload events into a single sub-line event under the
 		// Working indicator (└─ reloaded 47 rules). Errors here are real

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "2.5.2",
+  "version": "2.5.4",
   "description": "Workflow + project management for pi-coding-agent. Plans, state, MANDATORY rules, self-review, multi-question picker. One npm install, zero config. LLM drives the workflow inline via the soly_workflow tool — no external subagent plugin.",
   "type": "module",
   "main": "index.ts",

package/quota/minimax.ts

@@ -26,8 +26,21 @@
 // =============================================================================
 
 import { execFile } from "node:child_process";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
 import type { QuotaProvider, QuotaSnapshot } from "./types.ts";
 
+/** TEMP DEBUG. */
+const DEBUG_LOG = path.join(os.tmpdir(), "pi-soly-quota-debug.log");
+function dbg(msg: string): void {
+	try { fs.appendFileSync(DEBUG_LOG, `[${new Date().toISOString()}] [minimax] ${msg}\n`); } catch { /* best effort */ }
+}
+
+/** Detect Windows (where npm-installed CLIs are `.cmd` shims that execFile
+ *  can't spawn directly without a shell). */
+const IS_WIN = process.platform === "win32";
+
 /** Minimal shape we read from `mmx quota show --output json`. */
 type MinimaxQuotaResponse = {
 	model_remains?: Array<{
@@ -39,12 +52,28 @@ type MinimaxQuotaResponse = {
 };
 
 /** Run mmx and capture stdout. Resolves to null on any failure
- *  (mmx missing, non-zero exit, timeout, bad JSON). Never throws. */
+ *  (mmx missing, non-zero exit, timeout, bad JSON). Never throws.
+ *
+ *  On Windows, npm-global CLIs are `.cmd` shims (e.g. `mmx.cmd`). Node's
+ *  `execFile` spawns the executable directly without a shell, so it can't
+ *  resolve `mmx` → `mmx.cmd` and fails with ENOENT. `shell: true` lets the
+ *  shell do that resolution. Safe here: args are fixed (no user input). */
 function runMmx(args: string[], timeoutMs: number): Promise<string | null> {
 	return new Promise((resolve) => {
-		execFile("mmx", args, { encoding: "utf-8", timeout: timeoutMs, maxBuffer: 1024 * 1024 }, (err, stdout) => {
-			if (err) resolve(null);
-			else resolve(stdout ?? "");
+		const opts = {
+			encoding: "utf-8" as const,
+			timeout: timeoutMs,
+			maxBuffer: 1024 * 1024,
+			shell: IS_WIN, // resolve .cmd shims on Windows
+		};
+		execFile("mmx", args, opts, (err, stdout, stderr) => {
+			if (err) {
+				dbg(`runMmx error: ${err instanceof Error ? err.message : String(err)} (code=${(err as { code?: string }).code ?? "?"}) stderr=${(stderr ?? "").slice(0, 200)}`);
+				resolve(null);
+			} else {
+				dbg(`runMmx ok: ${String(stdout).length} bytes`);
+				resolve(stdout ?? "");
+			}
 		});
 	});
 }
@@ -54,16 +83,26 @@ function parseGeneralQuota(raw: string): QuotaSnapshot | null {
 	let parsed: unknown;
 	try {
 		parsed = JSON.parse(raw);
-	} catch {
+	} catch (e) {
+		dbg(`parse: JSON.parse failed: ${e instanceof Error ? e.message : String(e)}; raw[0..100]=${raw.slice(0, 100)}`);
 		return null; // not valid JSON — mmx printed an error to stdout
 	}
 	// Cast at the boundary (single documented place, per code-style rules).
 	const data = parsed as MinimaxQuotaResponse;
-	if (data.base_resp?.status_code !== 0) return null;
+	if (data.base_resp?.status_code !== 0) {
+		dbg(`parse: status_code=${data.base_resp?.status_code} (expected 0)`);
+		return null;
+	}
 	const general = data.model_remains?.find((m) => m.model_name === "general");
-	if (!general) return null;
+	if (!general) {
+		dbg(`parse: no 'general' model entry; have: ${(data.model_remains ?? []).map((m) => m.model_name).join(",")}`);
+		return null;
+	}
 	const pct = general.current_interval_remaining_percent;
-	if (typeof pct !== "number") return null;
+	if (typeof pct !== "number") {
+		dbg(`parse: pct is ${typeof pct}`);
+		return null;
+	}
 	return {
 		remainingPercent: pct,
 		resetsInMs: typeof general.remains_time === "number" ? general.remains_time : null,

package/quota/poller.ts

@@ -21,10 +21,21 @@
 //   It only runs while a session is active.
 // =============================================================================
 
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
 import type { ChromeData } from "../visual/data.ts";
 import { resolveQuotaProvider } from "./registry.ts";
 import { formatReset } from "./format.ts";
 
+/** TEMP DEBUG — append a line to the quota debug log. Remove after diagnosis. */
+const DEBUG_LOG = path.join(os.tmpdir(), "pi-soly-quota-debug.log");
+function dbg(msg: string): void {
+	try {
+		fs.appendFileSync(DEBUG_LOG, `[${new Date().toISOString()}] ${msg}\n`);
+	} catch { /* best effort */ }
+}
+
 /** Default poll interval: 60 seconds. */
 const POLL_INTERVAL_MS = 60_000;
 
@@ -59,33 +70,44 @@ export function startQuotaPoller(
 
 	const tick = async (): Promise<void> => {
 		if (stopped) return;
+		dbg(`tick: enabled=${isEnabled()} provider=${data.modelProvider}`);
 		if (!isEnabled()) {
+			dbg("disabled, skipping");
 			scheduleNext();
 			return;
 		}
 
 		const providerId = data.modelProvider;
 		if (!providerId) {
+			dbg("no modelProvider");
 			scheduleNext();
 			return;
 		}
 
 		const provider = resolveQuotaProvider(providerId);
 		if (!provider) {
-			// No adapter for this provider — clear and skip.
+			dbg(`no adapter for ${providerId}`);
 			data.quotaPercent = null;
 			data.quotaResetsLabel = null;
 			scheduleNext();
 			return;
 		}
 
-		const snapshot = await provider.fetch();
+		let snapshot;
+		try {
+			snapshot = await provider.fetch();
+			dbg(`fetch result: ${JSON.stringify(snapshot)}`);
+		} catch (e) {
+			dbg(`fetch threw: ${e instanceof Error ? e.message : String(e)}`);
+			snapshot = null;
+		}
 		if (snapshot) {
 			data.quotaPercent = snapshot.remainingPercent;
 			data.quotaResetsLabel = snapshot.resetsInMs !== null ? formatReset(snapshot.resetsInMs) : null;
+			dbg(`wrote data: pct=${data.quotaPercent} label=${data.quotaResetsLabel}`);
 			onUpdate();
+			dbg("called onUpdate (poke)");
 		}
-		// On null (fetch failed), keep the previous snapshot — don't clear.
 		scheduleNext();
 	};