pi-goal-list-loop-audit 0.24.5 → 0.24.7

package/extensions/goal-loop-display.ts

@@ -101,7 +101,12 @@ export function buildStatusText(state: State, audit?: AuditDisplayProgress | nul
     return `glla: ${paint(theme, pauseIsError(g) ? "error" : "warning", label)}`;
   }
   if (g.status === "active") {
-    const queue = state.list?.length ? ` · list ${state.list.length}` : "";
+    // v0.24.7: list policy gets its own wording — a queue item is not a goal.
+    // Before: "glla: list ● 3m 19s · list 29" (policy label AND queue counter
+    // both said "list"). After: "glla: list ● 3m 19s · 29 queued". Goal
+    // policy keeps the bare "list N" suffix (no duplication there).
+    const n = state.list?.length ?? 0;
+    const queue = n === 0 ? "" : g.policy === "list" ? ` · ${n} queued` : ` · list ${n}`;
     const tasks = g.taskList ? ` ${countDone(g)}/${countTotal(g)} tasks ·` : "";
     return `glla: ${g.policy} ${paint(theme, "success", "●")}${tasks} ${fmtElapsed(now - Date.parse(g.createdAt))}${queue}`;
   }
@@ -158,12 +163,16 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
         ? paint(theme, "accent", "⟡")
         : paint(theme, "success", "●");
   const head = `${icon} ${truncate(g.objective.replace(/\s+/g, " "), budgetFor(width, 3, 64))}`;
+  // v0.24.7: a list item is named as such and points at /list — before,
+  // the widget called it "active" and hinted "/goal status", reading as if
+  // queue work were a standalone goal.
+  const isList = g.policy === "list";
   const statusWord = g.status === "active" ? paint(theme, "success", "active") : g.status;
   // Token segment only when a budget is set (v0.22.0): the guard is opt-in,
   // and "0/0 tok" carried no information when off.
   const tokenLimit = g.usage?.tokensLimit ?? 0;
   const tokens = tokenLimit > 0 ? ` · ${paint(theme, "dim", `${fmtTokens(g.usage?.tokensUsed ?? 0)}/${fmtTokens(tokenLimit)} tok`)}` : "";
-  const lines = [head, `├─ ${statusWord} · ${fmtElapsed(now - Date.parse(g.createdAt))}${tokens}`];
+  const lines = [head, `├─ ${isList ? "list item · " : ""}${statusWord} · ${fmtElapsed(now - Date.parse(g.createdAt))}${tokens}`];
   if (g.status === "auditing") {
     lines.push(`├─ auditor: ${audit?.label ?? "running"}${audit?.currentTool ? ` · ${truncate(audit.currentTool, 30)}` : ""}`);
     if (audit?.elapsedMs) lines.push(`└─ ${paint(theme, "dim", `${fmtElapsed(audit.elapsedMs)} in isolated session`)}`);
@@ -178,7 +187,10 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
   const next = nextPending(g);
   if (next) lines.push(`├─ next: ${truncate(next, budgetFor(width, 9, 56))}`);
   const queue = state.list?.length ?? 0;
-  lines.push(`└─ ${paint(theme, "dim", `${queue > 0 ? `list ${queue} · ` : ""}/goal status · /glla`)}`);
+  const footer = isList
+    ? `${queue > 0 ? `${queue} queued · ` : ""}/list · /glla`
+    : `${queue > 0 ? `list ${queue} · ` : ""}/goal status · /glla`;
+  lines.push(`└─ ${paint(theme, "dim", footer)}`);
   return lines;
 }
 

package/extensions/goal-loop-subagents.ts

@@ -0,0 +1,191 @@
+// pi-goal-list-loop-audit — v0.24.6
+// extensions/goal-loop-subagents.ts
+//
+// Subagent model inheritance fix (v0.24.6, Section I of the eager-continuation
+// contract). pi-subagents v0.14.3's default Explore agent pins
+// "anthropic/claude-haiku-4-5" (default-agents.ts:40). Its model resolution is
+// explicit option > agent config > parent model (agent-runner.ts:720), so an
+// Explore spawn NEVER inherits the session model — it silently routes to a
+// different provider with a different quota pool. On rigs where the session
+// model is local/alternative (e.g. MiniMax-M3) and claude-haiku-4-5 resolves
+// through a quota-capped key (OpenRouter), three concurrent Explore spawns
+// exhaust the key with 403 "Key limit exceeded" while the parent session is
+// unaffected.
+//
+// pi-subagents has no per-agent model setting (subagents.json covers
+// concurrency/scope/etc., not models). Its supported override mechanism is
+// user agent files: <agentDir>/agents/<Name>.md fully replaces the default
+// config of the same name (custom-agents.ts + agent-types.ts overlay by
+// exact key). A file that omits "model:" falls through to the parent model.
+//
+// This module manages exactly one override file — Explore.md — the only
+// default agent with a model pin today. It writes/updates/removes the file
+// according to glla settings, and NEVER touches a file it didn't write
+// (marker frontmatter field). If tintinweb pins more defaults later, the
+// drift test in tests/subagent-model-override.test.ts fails and prompts us
+// to add embedded copies here.
+
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+
+/** Frontmatter marker identifying files this module wrote. Files without it
+ * are user-owned: never modified, never deleted. */
+export const SUBAGENT_MANAGED_MARKER = "pi-goal-list-loop-audit";
+
+/** Default pi-subagents agents that pin a model. Today: only Explore
+ * (anthropic/claude-haiku-4-5). Plan and general-purpose already inherit the
+ * parent model. Guarded by the drift test. */
+export const KNOWN_PINNED_DEFAULT_AGENTS = ["Explore"] as const;
+
+/** Strategy for subagent model selection. Default "inherit-parent": managed
+ * override drops the model pin so subagents share the session model AND its
+ * quota pool. "agent-default": upstream behavior (Explore pins haiku). */
+export type SubagentModelStrategy = "inherit-parent" | "agent-default";
+
+// ---- Embedded default-agent copies (verbatim from pi-subagents v0.14.3
+// src/default-agents.ts; drift-tested against the installed package) ----
+
+export const EXPLORE_DEFAULT_DESCRIPTION = "Fast read-only search agent for locating code. Use it to find files by pattern (eg. \"src/components/**/*.tsx\"), grep for symbols or keywords (eg. \"API endpoints\"), or answer \"where is X defined / which files reference Y.\" Do NOT use it for code review, design-doc auditing, cross-file consistency checks, or open-ended analysis \u2014 it reads excerpts rather than whole files and will miss content past its read window. When calling, specify search breadth: \"quick\" for a single targeted lookup, \"medium\" for moderate exploration, or \"very thorough\" to search across multiple locations and naming conventions.";
+
+export const EXPLORE_DEFAULT_SYSTEM_PROMPT = `# CRITICAL: READ-ONLY MODE - NO FILE MODIFICATIONS
+You are a file search specialist. You excel at thoroughly navigating and exploring codebases.
+Your role is EXCLUSIVELY to search and analyze existing code. You do NOT have access to file editing tools.
+
+You are STRICTLY PROHIBITED from:
+- Creating new files
+- Modifying existing files
+- Deleting files
+- Moving or copying files
+- Creating temporary files anywhere, including /tmp
+- Using redirect operators (>, >>, |) or heredocs to write to files
+- Running ANY commands that change system state
+
+Use Bash ONLY for read-only operations: ls, git status, git log, git diff, find, cat, head, tail.
+
+# Tool Usage
+- Use the find tool for file pattern matching (NOT the bash find command)
+- Use the grep tool for content search (NOT bash grep/rg command)
+- Use the read tool for reading files (NOT bash cat/head/tail)
+- Use Bash ONLY for read-only operations
+- Make independent tool calls in parallel for efficiency
+- Adapt search approach based on thoroughness level specified
+
+# Output
+- Use absolute file paths in all references
+- Report findings as regular messages
+- Do not use emojis
+- Be thorough and precise`;
+
+export const EXPLORE_DEFAULT_TOOLS = "read, bash, grep, find, ls";
+
+interface EmbeddedAgentDefault {
+  description: string;
+  systemPrompt: string;
+  tools: string;
+}
+
+/** Embedded copies keyed by agent name. Only agents in
+ * KNOWN_PINNED_DEFAULT_AGENTS need entries. */
+const EMBEDDED_DEFAULTS: Record<string, EmbeddedAgentDefault> = {
+  Explore: {
+    description: EXPLORE_DEFAULT_DESCRIPTION,
+    systemPrompt: EXPLORE_DEFAULT_SYSTEM_PROMPT,
+    tools: EXPLORE_DEFAULT_TOOLS,
+  },
+};
+
+/** Default global agent dir (pi-subagents reads $PI_CODING_AGENT_DIR/agents,
+ * default ~/.pi/agent/agents). Parameterized in sync for tests. */
+export function defaultAgentDir(): string {
+  return path.join(os.homedir(), ".pi", "agent");
+}
+
+/** Build the override .md file content. With `model`, the pin is written;
+ * without, the file falls through to the parent session model. */
+export function buildAgentOverrideMd(name: string, model?: string): string {
+  const def = EMBEDDED_DEFAULTS[name];
+  if (!def) throw new Error(`no embedded default config for agent "${name}"`);
+  const yamlDesc = "'" + def.description.replace(/'/g, "''") + "'";
+  const lines = [
+    "---",
+    `description: ${yamlDesc}`,
+    `tools: ${def.tools}`,
+    "prompt_mode: replace",
+  ];
+  if (model) lines.push(`model: ${model}`);
+  lines.push(
+    `x-managed-by: ${SUBAGENT_MANAGED_MARKER}`,
+    model
+      ? `x-glla-note: model pinned to ${model} by glla subagentModelOverrides. Remove the file or change /glla subagent settings to inherit the session model.`
+      : "x-glla-note: model pin removed (upstream default pins a fixed model) so this agent inherits the parent session model and its quota pool. Managed by glla — flip /glla subagent strategy to agent-default to restore upstream behavior.",
+    "---",
+    "",
+    def.systemPrompt,
+    "",
+  );
+  return lines.join("\n");
+}
+
+function hasManagedMarker(content: string): boolean {
+  return content.includes(`x-managed-by: ${SUBAGENT_MANAGED_MARKER}`);
+}
+
+export interface SubagentSyncResult {
+  written: string[];
+  removed: string[];
+  /** Files left untouched because the user owns them (no marker). */
+  skipped: Array<{ name: string; reason: string }>;
+}
+
+/** Sync <agentDir>/agents/<Name>.md with the desired state. Idempotent:
+ * writes only when content differs. Never touches non-managed files. */
+export function syncSubagentModelOverrides(opts: {
+  agentDir: string;
+  strategy: SubagentModelStrategy;
+  overrides?: Record<string, string>;
+}): SubagentSyncResult {
+  const result: SubagentSyncResult = { written: [], removed: [], skipped: [] };
+  const overrides = opts.overrides ?? {};
+  const names = new Set<string>([...KNOWN_PINNED_DEFAULT_AGENTS, ...Object.keys(overrides)]);
+
+  for (const name of names) {
+    const overrideModel = overrides[name];
+    if (overrideModel !== undefined && !EMBEDDED_DEFAULTS[name]) {
+      result.skipped.push({
+        name,
+        reason: `no embedded default config for "${name}" — create ${name}.md manually (only ${KNOWN_PINNED_DEFAULT_AGENTS.join("/")} are glla-managed)`,
+      });
+      continue;
+    }
+    const file = path.join(opts.agentDir, "agents", `${name}.md`);
+    const desired = overrideModel
+      ? buildAgentOverrideMd(name, overrideModel)
+      : opts.strategy === "inherit-parent"
+        ? buildAgentOverrideMd(name)
+        : undefined; // agent-default + no per-type override → file should be absent
+
+    const exists = fs.existsSync(file);
+    const current = exists ? fs.readFileSync(file, "utf-8") : undefined;
+
+    if (desired === undefined) {
+      if (exists && hasManagedMarker(current!)) {
+        fs.unlinkSync(file);
+        result.removed.push(name);
+      } else if (exists) {
+        result.skipped.push({ name, reason: "user-owned file (no glla marker) — left untouched" });
+      }
+      continue;
+    }
+
+    if (exists && !hasManagedMarker(current!)) {
+      result.skipped.push({ name, reason: "user-owned file (no glla marker) — left untouched" });
+      continue;
+    }
+    if (current === desired) continue; // idempotent no-op
+    fs.mkdirSync(path.dirname(file), { recursive: true });
+    fs.writeFileSync(file, desired);
+    result.written.push(name);
+  }
+  return result;
+}

package/extensions/loops/goal.ts

@@ -75,6 +75,11 @@ import {
   pushCapped as pushRepetitionCapped,
 } from "../goal-loop-repetition.js";
 import { buildStatusText, buildWidgetLines, type AuditDisplayProgress } from "../goal-loop-display.js";
+import {
+  defaultAgentDir,
+  syncSubagentModelOverrides,
+  type SubagentModelStrategy,
+} from "../goal-loop-subagents.js";
 import {
   applyMeasurement,
   applyMetriclessTick,
@@ -590,6 +595,8 @@ async function cmdStatus(ctx: ExtensionContext): Promise<void> {
   const lines = [
     `[${g.id}] ${statusLabel(g.status)}`,
     `Objective: ${g.objective}`,
+    // v0.24.7: name WHERE the work came from — a queue item is not a goal.
+    ...(g.policy === "list" ? [`Source: /list queue (${listQueue().length} waiting) — /list to manage`] : []),
     `Auto-continue: ${g.autoContinue ? "on" : "off"}`,
     `Iteration: ${iterationCounter}`,
     `Tokens: ${(g.usage?.tokensUsed ?? 0).toLocaleString()}${(g.usage?.tokensLimit ?? 0) > 0 ? ` / ${(g.usage!.tokensLimit).toLocaleString()}` : " (no cap — /glla tokenlimit=<n> to set)"}`,
@@ -2264,6 +2271,18 @@ interface Settings {
    * interview floor is skipped — the seed carries the intent (unattended
    * rigs). Default off: nothing activates before the user confirms. */
   autoAcceptDrafts?: boolean;
+  /** v0.24.6: subagent model strategy for pi-subagents default agents that
+   * pin a model (Explore pins claude-haiku-4-5, which silently routes
+   * subagents to a different provider/quota pool than the session).
+   * "inherit-parent" (default) writes a managed ~/.pi/agent/agents/Explore.md
+   * override without the model pin so subagents share the session model and
+   * its quota; "agent-default" restores upstream behavior. Applies to NEW
+   * sessions (pi-subagents registers agents at session start). */
+  subagentModelStrategy?: SubagentModelStrategy;
+  /** v0.24.6: per-agent-type model pin, e.g. { "Explore": "minimax/MiniMax-M3" }.
+   * Always wins over subagentModelStrategy — the managed override is written
+   * WITH this pin regardless of strategy. */
+  subagentModelOverrides?: Record<string, string>;
 }
 
 const DEFAULT_SETTINGS: Settings = {
@@ -2271,6 +2290,9 @@ const DEFAULT_SETTINGS: Settings = {
   // pi, auditor follows), floor "high" — the auditor is the verification
   // gate, depth is worth more there than speed. /glla thinking= overrides.
   auditorThinkingLevel: undefined,
+  // v0.24.6: subagents inherit the session model by default — one quota
+  // pool, no surprise 403s from a pinned default agent's provider.
+  subagentModelStrategy: "inherit-parent",
 };
 
 // Two-tier config (v0.7.0): GLOBAL is the normal home — you set things once
@@ -2408,6 +2430,8 @@ async function openSettingsUI(ctx: ExtensionContext): Promise<void> {
           `Notify command — ${show("notifyCmd", "(off)")}`,
           `Token limit per goal — ${show("tokenLimit", "(off)")}`,
           `Wedge alert minutes — ${show("wedgeAlertMinutes", `(${WEDGE_ALERT_DEFAULT_MINUTES}m default)`)}`,
+          `Subagent model strategy — ${show("subagentModelStrategy", "(inherit-parent)")}`,
+          `Subagent Explore model pin — ${loadSettings(ctx.cwd).subagentModelOverrides?.Explore ?? "(follows strategy)"}`,
           "Done",
         ],
       );
@@ -2441,6 +2465,26 @@ async function openSettingsUI(ctx: ExtensionContext): Promise<void> {
           else if (!v.trim()) saveSettings("global", ctx.cwd, { wedgeAlertMinutes: undefined });
           else ctx.ui.notify(`Not a non-negative integer: ${v}`, "warning");
         }
+      } else if (choice.startsWith("Subagent model strategy")) {
+        const v = await ctx.ui.select("Subagent model (pi-subagents default agents)", [
+          "inherit-parent — subagents share your session model + its quota pool (fixes separate-provider 403s; search agents may run on a pricier model)",
+          "agent-default — upstream behavior: Explore pins claude-haiku-4-5 (cheap search, but a SEPARATE provider quota from your session)",
+        ]);
+        if (v) {
+          const strategy: SubagentModelStrategy = v.startsWith("agent-default") ? "agent-default" : "inherit-parent";
+          saveSettings("global", ctx.cwd, { subagentModelStrategy: strategy });
+          ctx.ui.notify("Subagent model strategy saved — applies to NEW pi sessions (pi-subagents registers agents at session start).", "info");
+        }
+      } else if (choice.startsWith("Subagent Explore model pin")) {
+        const v = await ctx.ui.input("Model pin for Explore subagents", "provider/model-id e.g. minimax/MiniMax-M3 — always wins over strategy; empty = follow strategy");
+        if (v !== undefined) {
+          const current = loadSettings(ctx.cwd).subagentModelOverrides ?? {};
+          const next = { ...current };
+          if (v.trim()) next.Explore = v.trim();
+          else delete next.Explore;
+          saveSettings("global", ctx.cwd, { subagentModelOverrides: Object.keys(next).length > 0 ? next : undefined });
+          ctx.ui.notify("Explore model pin saved — applies to NEW pi sessions.", "info");
+        }
       }
     } catch {
       return;
@@ -2800,6 +2844,22 @@ export default function (pi: ExtensionAPI): void {
     ensureAgentToolsActive(pi, ctx);
     warnOnCommandCollision(ctx);
     warnIfAuditorProviderRisky(ctx);
+    // v0.24.6: sync the pi-subagents model override (managed Explore.md) with
+    // settings. Idempotent; applies to NEW sessions (pi-subagents registers
+    // its agents at its own session start).
+    try {
+      const s = loadSettings(ctx.cwd);
+      const sync = syncSubagentModelOverrides({
+        agentDir: defaultAgentDir(),
+        strategy: s.subagentModelStrategy ?? "inherit-parent",
+        overrides: s.subagentModelOverrides,
+      });
+      for (const skip of sync.skipped) {
+        ctx.ui.notify(`glla subagent override skipped [${skip.name}]: ${skip.reason}`, "warning");
+      }
+    } catch (err) {
+      ctx.ui.notify(`glla subagent override sync failed: ${err instanceof Error ? err.message : String(err)}`, "warning");
+    }
     // Restore gate (v0.21.0): a fresh session ("startup"/"new", or a pi too
     // old to report a reason) has no conversation context for the restored
     // work — HOLD instead of auto-firing, so opening pi in a folder never

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.24.5",
+  "version": "0.24.7",
   "description": "Goal. Loop. Audit. Done. — a pi-coding-agent extension that supervises long-running work, with isolated auditor on each completion. Beat bamboozling by design: the auditor runs in a fresh session with no extensions, no skills, no editor — only the read tools needed to verify your goal.",
   "license": "MIT",
   "author": "dracon",