pi-soly 2.5.3 → 2.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "2.5.3",
+  "version": "2.5.5",
   "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/visual/footer.ts

@@ -75,6 +75,21 @@ export function buildFooterLine(data: ChromeData, fd: FooterData, width: number,
 		const quotaText = ascii ? `${data.quotaPercent}%` : `⬢ ${data.quotaPercent}%`;
 		const label = data.quotaResetsLabel ? `${quotaText} · ${data.quotaResetsLabel}` : quotaText;
 		left.push({ id: "quota", text: styler.fg("muted", label), priority: 6 });
+		// TEMP DEBUG
+		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()}] [footer] render: pushed quota segment pct=${data.quotaPercent}\n`);
+		} catch {}
+	} else {
+		// TEMP DEBUG
+		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()}] [footer] render: quotaPercent=null (segment skipped)\n`);
+		} catch {}
 	}
 
 	if (data.rulesActive > 0) {