@mjasnikovs/pi-task 0.13.31 → 0.13.33

package/dist/task/auto-orchestrator.d.ts

@@ -21,8 +21,12 @@ export interface AutoDeps {
      * CLAUDE.md guidelines (and auto-fix violations) BEFORE it is committed.
      * Optional: when absent (tests, or the feature disabled), the loop treats it
      * as a pass. ok === false blocks the commit and fails the task.
+     *
+     * Takes the loop's CURRENT ctx (not the one captured at defaultDeps time):
+     * each task runs in a fresh session, so the captured ctx is stale by the time
+     * enforcement runs and using it for the loader widget throws "stale ctx".
      */
-    enforce?: (cwd: string, taskTitle: string) => Promise<EnforceOutcome>;
+    enforce?: (ctx: ExtensionCommandContext, cwd: string, taskTitle: string) => Promise<EnforceOutcome>;
 }
 /**
  * Expand any @file references in the feature text by appending each referenced

package/dist/task/auto-orchestrator.js

@@ -13,7 +13,7 @@ import { renderInlineMarkdown, stripInlineMarkdown } from './inline-markdown.js'
 import { AUTO_CLARIFY_PROMPT, AUTO_DECOMPOSE_PROMPT } from './auto-prompts.js';
 import { isDuplicateQuestion, MAX_DUP_STRIKES, DUP_REPROMPT_HINT } from './question-dedup.js';
 import { allocateAutoId, buildAutoBody, parseDecomposeList, parseTaskList, checkOffTask, stampTaskInProgress, findResumableAuto } from './auto-io.js';
-import { writeTaskFile, readTaskFile, updateTaskFrontMatter, taskFilePath } from './task-io.js';
+import { writeTaskFile, readTaskFile, updateTaskFrontMatter, taskFilePath, tasksDir } from './task-io.js';
 import { gitCommitAll } from './auto-commit.js';
 import { runGuidelineEnforcement, classifyEnforceChildFailure, ENFORCE_TIMEOUT_MS } from './enforce-guidelines.js';
 import { runWorker } from '../workers/pi-worker-core.js';
@@ -306,7 +306,7 @@ function defaultDeps(ctx, cwd, signal, title) {
         commit: (cwd2, message) => getConfig().autoCommit ?
             gitCommitAll(cwd2, message, signal)
             : Promise.resolve({ committed: false, reason: 'auto-commit disabled' }),
-        enforce: (cwd2, taskTitle) => {
+        enforce: (enforceCtx, cwd2, taskTitle) => {
             if (!getConfig().enforceGuidelines) {
                 return Promise.resolve({ ok: true, reason: 'disabled' });
             }
@@ -328,7 +328,21 @@ function defaultDeps(ctx, cwd, signal, title) {
                     lastLine = undefined;
                     contextUsage = undefined;
                     const startedAt = Date.now();
-                    const stopLoader = startAutoLoader(ctx, () => ({
+                    // Per-pass debug log. The enforce child is otherwise
+                    // unobservable (it has no per-task file like the impl runner),
+                    // so a loop/timeout in the verifier was undiagnosable. One
+                    // rolling file under .pi-tasks/; each pass brackets its
+                    // tool/text lines with start/end markers naming the task and
+                    // the terminal outcome, so a kill shows the exact repeated
+                    // calls that tripped the loop detector. Fire-and-forget.
+                    const enforceLogPath = path.join(tasksDir(cwd2), 'enforce-debug.log');
+                    const logEnforce = (msg) => {
+                        void fsp
+                            .appendFile(enforceLogPath, `${new Date().toISOString()} ${msg}\n`)
+                            .catch(() => { });
+                    };
+                    logEnforce(`=== enforce start: ${taskTitle} ===`);
+                    const stopLoader = startAutoLoader(enforceCtx, () => ({
                         title: taskTitle,
                         kind: 'enforce',
                         step: 'guidelines',
@@ -347,10 +361,13 @@ function defaultDeps(ctx, cwd, signal, title) {
                             timeoutMs: ENFORCE_TIMEOUT_MS,
                             onLine: line => {
                                 lastLine = line;
-                                phaseDeps.logDebug?.(`enforce: ${line}`);
+                                logEnforce(line);
                             }
                         });
                         const failure = classifyEnforceChildFailure(r);
+                        logEnforce(failure ?
+                            `=== enforce end: FAIL — ${failure} ===`
+                            : '=== enforce end: verdict captured ===');
                         if (failure)
                             throw new Error(failure);
                         return r.text;
@@ -470,7 +487,7 @@ export async function runAutoLoop(ctx, cwd, id, deps) {
             // dep.) Fixes it makes are folded into this task's snapshot below.
             if (deps.enforce) {
                 active.ui.notify(`${id}: enforcing AGENTS.md/CLAUDE.md on "${next.title}"…`, 'info');
-                const verdict = await deps.enforce(cwd, next.title);
+                const verdict = await deps.enforce(active, cwd, next.title);
                 if (!verdict.ok) {
                     await updateTaskFrontMatter(cwd, id, { state: 'failed' });
                     announceDone(active, `${id} stopped at "${next.title}" — ${verdict.reason ?? 'guideline check failed'} — fix and run /task-auto-resume.`, 'error');

package/dist/task/enforce-guidelines.js

@@ -159,6 +159,11 @@ export async function runGuidelineEnforcement(deps) {
         text = await deps.runChild(ENFORCE_TOOLS, buildEnforcePrompt(doc.text, diff), deps.signal);
     }
     catch (err) {
+        // A user cancellation is not an enforcement failure — re-throw it so the
+        // /task-auto loop's USER_CANCELLED handler reports a clean "cancelled —
+        // resume" instead of wrapping it as "enforcement pass could not run".
+        if (err instanceof Error && err.message === USER_CANCELLED)
+            throw err;
         const msg = err instanceof Error ? err.message : String(err);
         return { ok: false, reason: `enforcement pass could not run: ${msg}` };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.13.31",
+    "version": "0.13.33",
     "description": "Deterministic spec-orchestration for local models, with a bundled real-time remote web view and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",