taskplane 0.6.0 → 0.6.1

package/extensions/task-runner.ts

@@ -1406,6 +1406,20 @@ export const _readExitSummary = readExitSummary;
 export const _buildExitDiagnostic = buildExitDiagnostic;
 export type { BuildExitDiagnosticInput };
 
+/**
+ * Determine whether a step is "low-risk" and should skip reviews.
+ * Low-risk steps: Step 0 (Preflight) and the final step (Delivery/Docs).
+ *
+ * @param stepNumber  The 0-based step number being evaluated
+ * @param totalSteps  Total number of steps in the task
+ * @returns true if the step should skip plan and code reviews
+ */
+export function isLowRiskStep(stepNumber: number, totalSteps: number): boolean {
+	if (totalSteps <= 0) return false;
+	const lastStepIndex = totalSteps - 1;
+	return stepNumber === 0 || stepNumber === lastStepIndex;
+}
+
 // ── TMUX Agent Spawner ───────────────────────────────────────────────
 
 /**
@@ -2081,11 +2095,20 @@ export default function (pi: ExtensionAPI) {
 		logExecution(statusPath, `Step ${step.number} started`, step.name);
 		updateWidgets();
 
+		// Skip reviews for low-risk steps (Step 0 / Preflight and final step / Delivery)
+		const _isLowRiskStep = isLowRiskStep(step.number, task.steps.length);
+
 		// Plan review (level ≥ 1)
 		if (task.reviewLevel >= 1) {
-			const verdict = await doReview("plan", step, ctx, stepBaselineCommit);
-			if (verdict === "RETHINK") {
-				ctx.ui.notify(`Reviewer: RETHINK on Step ${step.number} plan. Proceeding with caution.`, "warning");
+			if (_isLowRiskStep) {
+				const label = step.number === 0 ? "Preflight" : "final step";
+				logExecution(statusPath, `Skip plan review`, `Step ${step.number} (${label}) — low-risk`);
+				ctx.ui.notify(`⏭️ Skipping plan review for Step ${step.number} (${label})`, "info");
+			} else {
+				const verdict = await doReview("plan", step, ctx, stepBaselineCommit);
+				if (verdict === "RETHINK") {
+					ctx.ui.notify(`Reviewer: RETHINK on Step ${step.number} plan. Proceeding with caution.`, "warning");
+				}
 			}
 		}
 
@@ -2135,10 +2158,16 @@ export default function (pi: ExtensionAPI) {
 
 		// Code review (level ≥ 2)
 		if (task.reviewLevel >= 2 && state.phase === "running") {
-			const verdict = await doReview("code", step, ctx, stepBaselineCommit);
-			if (verdict === "REVISE") {
-				ctx.ui.notify(`Reviewer: REVISE on Step ${step.number}. Running worker to fix...`, "warning");
-				await runWorker(step, ctx); // One more pass to address issues
+			if (_isLowRiskStep) {
+				const label = step.number === 0 ? "Preflight" : "final step";
+				logExecution(statusPath, `Skip code review`, `Step ${step.number} (${label}) — low-risk`);
+				ctx.ui.notify(`⏭️ Skipping code review for Step ${step.number} (${label})`, "info");
+			} else {
+				const verdict = await doReview("code", step, ctx, stepBaselineCommit);
+				if (verdict === "REVISE") {
+					ctx.ui.notify(`Reviewer: REVISE on Step ${step.number}. Running worker to fix...`, "warning");
+					await runWorker(step, ctx); // One more pass to address issues
+				}
 			}
 		}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "taskplane",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "AI agent orchestration for pi — parallel task execution with checkpoint discipline",
   "keywords": [
     "pi-package",