pi-goal-list-loop-audit 0.38.1 → 0.38.2

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.38.2 — thinking selected with model, remove standalone rows (2026-09-02)
+
+### Changed
+  `Drafter thinking` and `Auditor thinking` standalone rows removed from `/glla` — thinking is now selected **with** the model pick (the existing `Drafter agent` / `Auditor agent` flows already prompt for thinking immediately after the model, and that ladder is model-aware via `auditorThinkingLevels`). Main had no standalone thinking row; Drafter/Auditor now match it. Stored `drafterThinkingLevel` / `auditorThinkingLevel` keys remain read for fallback-chain rendering (`modelThinkingText`) and for the model-pick ladder, but there is no separate editor — re-pick the model to change thinking. Prevents stale thinking selection and the "Main has no selector, others do" inconsistency.
+
 ## 0.38.1 — drafter/auditor fallback parity with Main (2026-09-02)
 
 ### Fixed

package/docs/INDEX.md

@@ -18,7 +18,7 @@ For shipped docs, the relevant entry points are:
   failback; v0.35.9 hardened cross-version npm tarball checks; v0.35.10
   handles multi-entry npm dry-run reports; v0.35.11 accepts both npm report
   shapes; v0.35.12 supports npm 12's keyed pack reports; v0.35.13 fixes stale-API recovery loops.
-  v0.35.14–v0.38.1 continue through the supervisor freeze (`/glla pause`),
+  v0.35.14–v0.38.2 continue through the supervisor freeze (`/glla pause`),
   load hold, auditor picker parity, Windows launch fix, zombie-watchdog
   subagent carve-out, due-wait backstop, the `/glla agents` visibility panel,
   durable state-root selection, blank-until-resume auditor context, frozen

package/extensions/loops/goal-settings-ui.ts

@@ -1187,40 +1187,6 @@ export async function handleSettingChoice(id: string, ctx: ExtensionContext): Pr
       );
       return;
     }
-    case "drafterThinkingLevel": {
-      const settings = loadSettings(ctx.cwd);
-      const pickedModel = settings.drafterModel
-        ? resolvePickedModel(ctx, { kind: "ref", ref: settings.drafterModel })
-        : (ctx.model as any);
-      const levels = auditorThinkingLevels(pickedModel);
-      if (levels.length <= 1) {
-        ctx.ui.notify(
-          `Drafter agent ${pickedModel ? `${pickedModel.provider}/${pickedModel.id}` : "(session)"} exposes no thinking levels — drafting runs with thinking off.`,
-          "info",
-        );
-        return;
-      }
-      const preferred = settings.drafterThinkingLevel ?? ctx.thinkingLevel ?? "high";
-      const thinking = await ctx.ui.select(
-        "Drafter thinking — TEMPORARY DRAFTING AGENT ONLY (your session thinking is restored afterward)",
-        drafterThinkingChoiceOptions(
-          levels,
-          levels.includes(preferred) ? preferred : levels.includes("high") ? "high" : levels[levels.length - 1],
-          settings.drafterThinkingLevel === undefined,
-        ),
-      );
-      if (thinking) {
-        if (thinking.startsWith("session —")) {
-          saveSettings("global", ctx.cwd, { drafterThinkingLevel: undefined });
-          ctx.ui.notify("Drafter thinking: inherited from the session.", "info");
-        } else {
-          const level = thinking.split(" ")[0] as ConfiguredThinkingLevel;
-          saveSettings("global", ctx.cwd, { drafterThinkingLevel: level });
-          ctx.ui.notify(`Drafter thinking: ${level}.`, "info");
-        }
-      }
-      return;
-    }
     case "drafterModelFallbacks": {
       const global = loadGlobalSettings();
       const current = normalizeMainModelFallbackRefs(global.drafterModelFallbacks);
@@ -1316,30 +1282,6 @@ export async function handleSettingChoice(id: string, ctx: ExtensionContext): Pr
       ctx.ui.notify(`Auditor model: ${pick.kind === "session" ? "session model (override cleared)" : pick.ref}${t ? ` · thinking ${t.split(" ")[0]}` : ""}`, "info");
       return;
     }
-    case "auditorThinkingLevel": {
-      // v0.34.127: standalone path — the v0.31.4 comment claimed "/glla
-      // thinking=" is the direct path, but no such action ever existed; the
-      // ONLY way to change the level was re-picking the auditor model. The
-      // Auditor thinking row fixes that with the same ladder + dialog the
-      // model flow uses.
-      const curThinking = loadSettings(ctx.cwd).auditorThinkingLevel;
-      const inheritedThinking = ctx.thinkingLevel ?? "max";
-      const resolved = resolveAuditorModel(ctx);
-      const levels = auditorThinkingLevels(resolved.model);
-      if (levels.length <= 1) {
-        ctx.ui.notify(
-          `Auditor model ${resolved.model ? `${resolved.model.provider}/${resolved.model.id}` : "(session)"} exposes no thinking levels — the auditor runs with thinking off.`,
-          "info",
-        );
-        return;
-      }
-      const t = await ctx.ui.select(
-        "Auditor thinking — DETACHED auditor worker ONLY (your session model's thinking is untouched)",
-        levels.map((lv) => `${lv} — ${THINKING_DESCR[lv] ?? ""}${lv === (curThinking ?? inheritedThinking) ? " (current)" : ""}`),
-      );
-      if (t) saveSettings("global", ctx.cwd, { auditorThinkingLevel: t.split(" ")[0] as Settings["auditorThinkingLevel"] });
-      return;
-    }
     case "auditorModelFallbacks": {
       const settings = loadGlobalSettings();
       const current = normalizeMainModelFallbackRefs(settings.auditorModelFallbacks);

package/extensions/settings-menu.ts

@@ -311,14 +311,6 @@ export function buildSettingsRows(
       sourceText: src("drafterModel"),
       description: "temporary agent/model used only during /goal, /list, and /loop drafting; after confirmation or interruption the session agent is restored",
     },
-    {
-      id: "drafterThinkingLevel",
-      section: "drafter",
-      label: "Drafter thinking",
-      valueText: settings.drafterThinkingLevel ? settings.drafterThinkingLevel : `inherit ${sessionThinking}`,
-      sourceText: src("drafterThinkingLevel"),
-      description: "requested drafting-agent reasoning level — the same request follows the drafter fallback chain and Pi clamps it per model; unset inherits the session level",
-    },
     {
       id: "drafterModelFallbacks",
       section: "drafter",
@@ -358,14 +350,6 @@ export function buildSettingsRows(
       sourceText: src("auditorModel"),
       description: "provider/model override for the isolated auditor agent — the fallback chain below runs after this primary; its thinking defaults to the parent session",
     },
-    {
-      id: "auditorThinkingLevel",
-      section: "auditor",
-      label: "Auditor thinking",
-      valueText: settings.auditorThinkingLevel ?? sessionThinking,
-      sourceText: src("auditorThinkingLevel"),
-      description: "DETACHED auditor worker's reasoning level — unset inherits the parent session dial (including max); an explicit value overrides it",
-    },
     {
       id: "auditorModelFallbacks",
       section: "auditor",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.38.1",
+  "version": "0.38.2",
   "description": "Mission control for autonomous pi: interview-drafted goals, an audited task queue, and forever-loops (metric, spec, project-audit) that run for hours. A detached extension-less auditor process re-verifies every completion with raw evidence without holding the main pi turn; confirmed drafts, decision pauses and consent gates keep you in charge.",
   "license": "AGPL-3.0-only",
   "author": "dracon",