pi-goal-list-loop-audit 0.28.6 → 0.28.7

package/extensions/goal-loop-core.ts

@@ -897,7 +897,12 @@ export interface EffectiveAggressiveSettings {
   stuckMaxInterventions: number;
   /** 0 = wedge alerts off. */
   wedgeAlertMinutes: number;
-  autoResume: boolean;
+  /** Tri-state: true = always auto-resume; false = never; undefined =
+   * DEFAULT (hold on human session loads, resume on reload/fork).
+   * v0.28.7: must stay tri-state here — coercing unset→false broke the
+   * restore gate's default branch (the 0.28.3 regression the behavioral
+   * harness caught). */
+  autoResume: boolean | undefined;
   aggressiveMode: boolean;
 }
 
@@ -918,7 +923,7 @@ export function resolveEffectiveAggressiveSettings(s: {
     stuckMaxInterventions:
       s.stuckMaxInterventions ?? (aggressiveMode ? AGGRESSIVE_STUCK_MAX_INTERVENTIONS : BASE_STUCK_MAX_INTERVENTIONS),
     wedgeAlertMinutes: s.wedgeAlertMinutes ?? (aggressiveMode ? 0 : 30),
-    autoResume: s.autoResume ?? aggressiveMode,
+    autoResume: s.autoResume ?? (aggressiveMode ? true : undefined),
   };
 }
 

package/extensions/loops/goal.ts

@@ -224,6 +224,13 @@ function goStaleTerminal(ctx: ExtensionContext, where: string): void {
   notifyExternal(ctx, `glla: extension api stale — restart pi. (${where})`);
 }
 
+/** TEST-ONLY hook (tests/harness): the stale flag is process-terminal in
+ * production — only a pi restart clears it — so behavioral tests reset it
+ * between stale scenarios. Never called by production code. */
+export function __testOnlyResetStaleFlag(): void {
+  extensionApiStale = false;
+}
+
 /** v0.28.1 (S3): side-effect-free staleness probe — getSessionName()
  * routes through pi's assertActive() and throws the stale signature iff
  * pi invalidated this factory handle (session replacement). A positive
@@ -2523,11 +2530,12 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
   pi.registerTool(defineTool({
     name: "complete_task",
     label: "Complete task",
-    description: "Mark a task in the active goal's task list as complete (does not stop the turn).",
-    parameters: Type.Object({
+    description: "Mark a task in the active goal's task list as complete (does not stop the turn).",    parameters: Type.Object({
       id: Type.String({ description: "Task id to complete" }),
     }),
-    async execute(_id, params) {
+    async execute(_id, params, _signal, _onUpdate, execCtx) {
+      const foreign7 = foreignToolGuard(execCtx);
+      if (foreign7) return { content: [{ type: "text", text: foreign7 }], details: {} };
       const p = params as { id: string };
       if (!state.goal || !state.goal.taskList) {
         return { content: [{ type: "text", text: "No task list in this goal." }], details: {} };
@@ -2555,7 +2563,9 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
       id: Type.String(),
       status: Type.Union([Type.Literal("pending"), Type.Literal("in_progress"), Type.Literal("complete")]),
     }),
-    async execute(_id, params) {
+    async execute(_id, params, _signal, _onUpdate, execCtx) {
+      const foreign8 = foreignToolGuard(execCtx);
+      if (foreign8) return { content: [{ type: "text", text: foreign8 }], details: {} };
       const p = params as { id: string; status: "pending" | "in_progress" | "complete" };
       if (!state.goal || !state.goal.taskList) {
         return { content: [{ type: "text", text: "No task list in this goal." }], details: {} };
@@ -3012,6 +3022,8 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
       })),
     }),
     async execute(_id, params, _signal, _onUpdate, execCtx) {
+      const foreign9 = foreignToolGuard(execCtx);
+      if (foreign9) return { content: [{ type: "text", text: foreign9 }], details: {} };
       if (!state.goal || state.goal.status !== "active") {
         return { content: [{ type: "text", text: "No active goal to break down." }], details: {} };
       }
@@ -3179,7 +3191,9 @@ async function promptSettingsMenu(
  * Same handlers as v0.27.0's if/else chain — only the trigger changed from
  * `startsWith(label)` strings to stable ids.
  */
-async function handleSettingChoice(id: string, ctx: ExtensionContext): Promise<void> {
+// v0.28.7 (T4): exported for the behavioral settings-editor tests
+// (tests/settings-editors.test.ts drives each editor class end-to-end).
+export async function handleSettingChoice(id: string, ctx: ExtensionContext): Promise<void> {
   switch (id) {
     case "autoResume": {
       const v = await ctx.ui.select("Auto-resume goals/loops on session start", [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.28.6",
+  "version": "0.28.7",
   "description": "Goal. Loop. Audit. Done. \u2014 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 \u2014 only the read tools needed to verify your goal.",
   "license": "MIT",
   "author": "dracon",