@mediadatafusion/pi-workflow-suite 0.0.13 → 0.0.15

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 All notable public releases will be documented in this file.
 
+## [0.0.15] - 2026-06-09
+
+### Improved
+
+- Improved Plan Mode approval handoffs so approval-ready plans remain visible while action menus are open.
+- Refined the Deep Plan preset so reviewer review remains available without automatically starting by default.
+- Improved workflow recovery around transient connection interruptions so Plan, Mission, and Standard workflows are easier to resume after short transport failures.
+
+### Hardened
+
+- Expanded regression coverage for reviewer routing, sub-agent handoffs, package command surfaces, and interruption recovery.
+
+## [0.0.14] - 2026-06-08
+
+### Fixed
+
+- Restored the npm/pi.dev publish preparation path so the package-safe README is active before npm publish starts, preserving the rendered package README while keeping media pinned to the published `0.0.12` assets.
+
 ## [0.0.13] - 2026-06-08
 
 ### Changed

package/README.md

@@ -4,7 +4,7 @@
 
 [![Install](https://cdn.jsdelivr.net/npm/@mediadatafusion/pi-workflow-suite@0.0.12/docs/assets/readme-link-install.svg)](#installation) [![Quick Start](https://cdn.jsdelivr.net/npm/@mediadatafusion/pi-workflow-suite@0.0.12/docs/assets/readme-link-quick-start.svg)](#quick-start) [![Commands](https://cdn.jsdelivr.net/npm/@mediadatafusion/pi-workflow-suite@0.0.12/docs/assets/readme-link-commands.svg)](#core-commands) [![Settings](https://cdn.jsdelivr.net/npm/@mediadatafusion/pi-workflow-suite@0.0.12/docs/assets/readme-link-settings.svg)](#settings-reference)
 
-**Workflow Suite Version:** `v0.0.13`
+**Workflow Suite Version:** `v0.0.15`
 
 ## Overview
 
@@ -578,15 +578,15 @@ Mission: approval-gated, auto-run after approval, standard planning, milestone v
 
 Deep — Careful
 Plan: deep, asks clarification for non-trivial work
-Review: automatic
-Validation: automatic
+Review: manual/optional; not automatic before execution
+Validation: automatic after execution
 Sub-agents: forced larger teams across planning/execution/repair/review/validation
 Mission: deep planning with final validation
 
 Maximum — Thorough
 Plan: maximum
 Review: automatic
-Validation: automatic
+Validation: automatic after execution
 Sub-agents: forced maximum teams across planning/execution/repair/review/validation
 Mission: supervised auto, final validation, higher retry budget
 ```
@@ -1007,8 +1007,8 @@ pi install -l npm:@mediadatafusion/pi-workflow-suite
 ### Installing specific versions
 
 ```bash
-pi install npm:@mediadatafusion/pi-workflow-suite@0.0.13
-pi install -l npm:@mediadatafusion/pi-workflow-suite@0.0.13
+pi install npm:@mediadatafusion/pi-workflow-suite@0.0.15
+pi install -l npm:@mediadatafusion/pi-workflow-suite@0.0.15
 ```
 
 An unversioned install follows normal update behavior: `pi update` and `pi update --extensions` will pick up new package releases. A versioned install pins the package to that version. Pinned package specs are intentionally skipped by Pi's normal package update commands. To move a pinned install to a newer version, reinstall with the desired version. To switch back to latest tracking, use the unversioned install command without `@<version>`.
@@ -1214,10 +1214,10 @@ See `docs/TROUBLESHOOTING.md` for detailed diagnostics.
 
 ## Versioning
 
-The current preparation version is `v0.0.13`. Version information is intentionally aligned across:
+The current preparation version is `v0.0.15`. Version information is intentionally aligned across:
 
-- `VERSION` (`v0.0.13`),
-- `package.json` (`0.0.13`),
+- `VERSION` (`v0.0.15`),
+- `package.json` (`0.0.15`),
 - `package-lock.json`,
 - this README,
 - Workflow Suite settings/about output.

package/VERSION

@@ -1 +1 @@
-v0.0.13
+v0.0.15

package/extensions/workflow-model-router.ts

@@ -956,9 +956,9 @@ export function builtInWorkflowPresets(): Record<string, WorkflowPresetBundle> {
     },
     deep: {
       displayName: "Deep",
-      description: "Careful end-to-end workflow for risky or codebase-heavy work with stronger clarification, automatic review/validation, final mission validation, and larger worker teams.",
+      description: "Careful end-to-end workflow for risky or codebase-heavy work with stronger clarification, manual Plan review, automatic validation, final mission validation, and larger worker teams.",
       planning: { depth: "deep", clarificationMode: "always_for_nontrivial", maxClarificationQuestions: 5, interactiveClarificationEnabled: true, clarificationQualityGate: true, useSubagentsBeforeClarification: true },
-      workflow: { offerReviewerBeforeExecute: false, autoRunReviewerBeforeExecute: true, offerValidationAfterExecute: true, autoRunValidationAfterExecute: true, validateAfterExecution: true, requirePlanApprovalBeforeExecute: false, requireApprovalBeforeExecution: false, autoRepairReviewFailures: true, autoRepairValidationFailures: true, reviewRetryMode: "safe_only", validationRetryMode: "safe_only", maxReviewRetriesPerPlan: 3, maxReviewRetriesPerWorkflow: 6, maxValidationRetriesPerPlan: 3, maxValidationRetriesPerWorkflow: 6, pauseAfterReviewFailure: false, pauseAfterValidationFailure: false, planProgressEnabled: true, planRuntimeEnabled: true, planShowProgressBar: true },
+      workflow: { offerReviewerBeforeExecute: false, autoRunReviewerBeforeExecute: false, offerValidationAfterExecute: true, autoRunValidationAfterExecute: true, validateAfterExecution: true, requirePlanApprovalBeforeExecute: false, requireApprovalBeforeExecution: false, autoRepairReviewFailures: true, autoRepairValidationFailures: true, reviewRetryMode: "safe_only", validationRetryMode: "safe_only", maxReviewRetriesPerPlan: 3, maxReviewRetriesPerWorkflow: 6, maxValidationRetriesPerPlan: 3, maxValidationRetriesPerWorkflow: 6, pauseAfterReviewFailure: false, pauseAfterValidationFailure: false, planProgressEnabled: true, planRuntimeEnabled: true, planShowProgressBar: true },
       standard: { enabled: true, autoTodoEnabled: true, todoProgressVisible: true, todoTriggerMode: "auto", clarificationEnabled: true, clarificationMode: "auto", maxClarificationQuestions: 2, interactiveClarificationEnabled: true, clarificationTiming: "after_initial_analysis", clarificationQualityGate: true, allowClarificationWithoutAnalysis: false, useSubagentsBeforeClarification: true, allowSubagents: true, subagentScope: "user", subagents: { planningPolicy: "forced", executionPolicy: "forced", repairPolicy: "forced", reviewPolicy: "forced", validationPolicy: "forced", autoUseDuringPlanning: true, autoUseDuringExecution: true, autoUseDuringRepair: true, autoUseDuringReview: true, autoUseDuringValidation: true, minPlanningWorkersForDeep: 2, minPlanningWorkersForMaximum: 2, minExecutionWorkersForDeep: 3, minExecutionWorkersForMaximum: 3, minRepairWorkersForDeep: 2, minRepairWorkersForMaximum: 2, minReviewWorkersForDeep: 3, minReviewWorkersForMaximum: 3, minValidationWorkersForDeep: 3, minValidationWorkersForMaximum: 3 }, statusWidgetVisible: true, useSharedExecutorModel: true, useStandardSpecificModels: false, modelRole: "executor" },
       missions: { defaultAutonomy: "approval_gated", requireValidationPerMilestone: true, autoRunAfterApproval: true, continueAcrossMilestones: true, pauseBetweenMilestones: false, clarificationMode: "always_for_nontrivial", maxClarificationQuestions: 5, planningDepth: "deep", useSubagentsBeforeClarification: true, autoRepairReviewFailures: true, reviewRetryMode: "safe_only", maxReviewRetriesPerMission: 3, autoRepairValidationFailures: true, validationRetryMode: "safe_only", maxValidationRetriesPerMilestone: 3, maxValidationRetriesPerMission: 8, finalValidationEnabled: true, autoRepairFinalValidationFailures: true, maxFinalValidationRetries: 2, subagentPolicy: "forced", minWorkersForDeep: 3, minWorkersForMaximum: 3 },
       subagents: { planningPolicy: "forced", executionPolicy: "forced", repairPolicy: "forced", reviewPolicy: "forced", validationPolicy: "forced", autoUseDuringPlanning: true, autoUseDuringExecution: true, autoUseDuringRepair: true, autoUseDuringReview: true, autoUseDuringValidation: true, minPlanningWorkersForDeep: 3, minPlanningWorkersForMaximum: 3, minExecutionWorkersForDeep: 3, minExecutionWorkersForMaximum: 3, minRepairWorkersForDeep: 2, minRepairWorkersForMaximum: 2, minReviewWorkersForDeep: 3, minReviewWorkersForMaximum: 3, minValidationWorkersForDeep: 3, minValidationWorkersForMaximum: 3, allowBackgroundSubagents: true },
@@ -1077,7 +1077,7 @@ export function workflowPresetPickerLabel(name: string, preset?: WorkflowPresetB
   const summaries: Record<string, string> = {
     simple: "fast autonomous validation, low repair retries, 1-worker phases",
     standard: "balanced autonomous validation and safe repair",
-    deep: "careful autonomous review, validation, and final mission validation",
+    deep: "careful manual review, autonomous validation, and final mission validation",
     maximum: "maximum autonomous rigor with bounded aggressive repair",
   };
   return summaries[name] ? `${title} — ${summaries[name]}` : title;
@@ -1122,7 +1122,7 @@ export function workflowPresetMeaningLines(name: string, preset?: WorkflowPreset
       "Purpose: higher-rigor built-in profile for broad, risky, or codebase-heavy work that still should not stall unnecessarily.",
       "Applies to: Standard Mode, Plan Mode, Mission Mode, and shared sub-agent intensity.",
       "Standard Mode: requires a To Do for substantive work, uses stronger clarification, resumes automatically after answers, and uses deeper Standard worker coverage.",
-      "Plan Mode: uses deep planning, clarifies non-trivial work, avoids a second execution-approval stop after the initial approval, runs automatic review and validation, and retries safe repairs.",
+      "Plan Mode: uses deep planning, clarifies non-trivial work, avoids a second execution-approval stop after the initial approval, leaves review manual/optional, runs validation automatically, and retries safe repairs.",
       "Mission Mode: starts after approval, auto-runs after approval, continues milestones without pause, keeps milestone validation on, and enables final comprehensive validation.",
       "Shared sub-agents: forces larger teams across planning, execution, repair, review, and validation.",
       "Does not change: models/providers/API keys/auth/session/runtime state/shared compaction settings.",

package/extensions/workflow-modes.ts

@@ -1750,7 +1750,7 @@ ${requiredSubagentPreflightSection(options.preflightBlock)}
 
 ${subagentCapabilityTable()}
 
-${priorPlan ? `Current draft/approved plan to revise:\n${priorPlan}\n\n` : ""}${feedbackBlock}${options.forceClarification ? `CLARIFICATION IS REQUIRED BEFORE FINAL PLANNING. Reason: ${options.forceReason ?? "always_for_nontrivial classified this as non-trivial"}\nYour job in this turn is to perform lightweight task analysis, then generate only high-value clarification questions from the actual task. Do not produce an implementation plan in this turn. ${subagentsBeforeClarification ? "If planning depth/policy calls for it and sub-agent use is available, use read-only planning/research sub-agents before asking clarification so the questions are context-aware." : "Do not call sub-agents in this clarification-generation turn unless forced by sub-agent policy."} Do not use generic reusable workflow boilerplate questions.\n\n` : ""}${options.qualityGateFeedback ? `The previous clarification output failed the clarification quality gate: ${options.qualityGateFeedback}\nRegenerate better task-specific clarification questions${options.forceClarification ? ". Do not produce a final plan in this retry." : ", or use PLAN_DECISION: plan if no high-value question exists."}\n\n` : ""}MANDATORY STRUCTURED HANDOFF: Before your final response, call workflow_plan_result with decision=clarify, plan, or blocked. The tool payload is the primary control plane; visible markdown is fallback display only. If workflow_plan_result is unavailable, output the parser-safe PLAN_DECISION fallback once and STOP immediately; do not call subagent as an acknowledgement, do not retry invalid tool names, and do not continue analysis after the fallback.
+${priorPlan ? `Current draft/approved plan to revise:\n${priorPlan}\n\n` : ""}${feedbackBlock}${options.forceClarification ? `CLARIFICATION IS REQUIRED BEFORE FINAL PLANNING. Reason: ${options.forceReason ?? "always_for_nontrivial classified this as non-trivial"}\nYour job in this turn is to perform lightweight task analysis, then generate only high-value clarification questions from the actual task. Do not produce an implementation plan in this turn. ${subagentsBeforeClarification ? "If planning depth/policy calls for it and sub-agent use is available, use read-only planning/research sub-agents before asking clarification so the questions are context-aware." : "Do not call sub-agents in this clarification-generation turn unless forced by sub-agent policy."} Do not use generic reusable workflow boilerplate questions.\n\n` : ""}${options.qualityGateFeedback ? `The previous clarification output failed the clarification quality gate: ${options.qualityGateFeedback}\nRegenerate better task-specific clarification questions${options.forceClarification ? ". Do not produce a final plan in this retry." : ", or use PLAN_DECISION: plan if no high-value question exists."}\n\n` : ""}MANDATORY STRUCTURED HANDOFF: Before your final response, call workflow_plan_result with decision=clarify, plan, or blocked. The tool payload is the primary control plane. After workflow_plan_result returns, print the user-facing markdown for that decision and stop: the approval-ready # Implementation Plan for decision=plan, the clarification questions for decision=clarify, or the blocker summary for decision=blocked. If workflow_plan_result is unavailable, output the parser-safe PLAN_DECISION fallback once and STOP immediately; do not call subagent as an acknowledgement, do not retry invalid tool names, and do not continue analysis after the fallback.
 
 LEGACY FALLBACK: Your VERY FIRST LINE must be exactly one of:
 ${options.forceClarification ? "PLAN_DECISION: clarify" : "PLAN_DECISION: clarify\nPLAN_DECISION: plan"}
@@ -13742,6 +13742,7 @@ ${renderMissionStatus(activeMission ?? paused)}`);
       showMissionPlanMenuFallback("Mission plan menu was skipped because the active mission state changed.");
       return;
     }
+    showMissionPlanReadyCard(mission);
     if (!ctx.hasUI) return;
     const reviewChoice = ["PASS", "NOTES"].includes(mission.reviewerVerdict ?? "") ? [] : ["Review Mission Plan"];
     const choices = mission.autonomy === "manual" || mission.autonomy === "approval_gated"
@@ -13750,6 +13751,7 @@ ${renderMissionStatus(activeMission ?? paused)}`);
     const choice = await ctx.ui.select("Mission plan ready. Choose next action:", choices);
     if (!choice) {
       updateState({ mode: "mission_plan_ready", activeMissionId: mission.id, task: mission.goal, originalTask: mission.goal, draftPlan: mission.planText }, ctx);
+      showMissionPlanMenuFallback("Mission plan menu was dismissed. The mission plan remains ready.");
       return recordWorkflowInternalEvent(ctx, `Mission plan menu dismissed: ${mission.id}`);
     }
     if (choice === "Review Mission Plan") {
@@ -13785,12 +13787,31 @@ ${renderMissionStatus(activeMission ?? paused)}`);
     show(pi, `# Mission Plan Ready\n\n${reason}\n\nUse one of:\n- \`/mission review\`\n- \`/mission approve\`\n- \`/mission revise <feedback>\`\n- \`/mission cancel\``);
   }
 
+  function showMissionPlanReadyCard(mission: MissionState): void {
+    const planText = mission.planText?.trim() ? mission.planText : "(none recorded)";
+    show(pi, `# Mission Plan Ready\n\n${planText}\n\nChoose an action in the Mission menu, or use:\n- \`/mission review\`\n- \`/mission approve\`\n- \`/mission revise <feedback>\`\n- \`/mission cancel\``);
+  }
+
   // ── Menus ──────────────────────────────────────────────────────
 
   function showPlanMenuCommandFallback(reason: string): void {
     show(pi, `# Plan Ready\n\n${reason}\n\nUse one of:\n- \`/plan approve\`\n- \`/plan revise <feedback>\`\n- \`/plan cancel\``);
   }
 
+  function showPlanReadyForApprovalCard(snapshot: WorkflowHandoffSnapshot, options: { includePlanText?: boolean } = {}): void {
+    const planText = snapshot.draftPlan?.trim()
+      ? snapshot.draftPlan
+      : state.draftPlan?.trim()
+        ? state.draftPlan
+        : state.approvedPlan?.trim()
+          ? state.approvedPlan
+          : "(none recorded)";
+    const body = options.includePlanText === true
+      ? `${planText}\n\nChoose an action in the approval menu, or use:`
+      : "Choose an action in the approval menu. The native plan above is the source of truth, and the saved plan remains recoverable with:";
+    show(pi, `# Plan Ready For Approval\n\n${body}\n- \`/plan approve\`\n- \`/plan revise <feedback>\`\n- \`/plan cancel\``);
+  }
+
   function showPlanClarificationFallback(reason: string, questions?: ClarificationQuestion[]): void {
     const rendered = questions?.length ? `\n\n${renderClarificationQuestionsForUser(questions)}` : "";
     show(pi, `# Clarification Needed\n\n${reason}${rendered}\n\nAnswer with: /clarify answer 1A 2C\nSkip with: /clarify answer 1S\nCustom answer: /clarify answer 3D: your text`);
@@ -13845,9 +13866,10 @@ ${renderMissionStatus(activeMission ?? paused)}`);
       return;
     }
     if (!ctx.hasUI) {
-      showPlanMenuCommandFallback(snapshot.fallback);
+      showPlanReadyForApprovalCard(snapshot, { includePlanText: true });
       return;
     }
+    showPlanReadyForApprovalCard(snapshot);
     const settings = loadWorkflowSettings(ctx.cwd);
     const approveLabel = settings.models.reviewer.enabled ? "Approve and Run Workflow" : "Approve and Execute";
     let menuChoice: string | undefined;
@@ -13857,6 +13879,11 @@ ${renderMissionStatus(activeMission ?? paused)}`);
       showPlanMenuCommandFallback(`Interactive approval menu failed to open: ${error instanceof Error ? error.message : String(error)}`);
       return;
     }
+    if (!menuChoice) {
+      updateState({ mode: "plan_draft", activePlanId: snapshot.activePlanId ?? state.activePlanId, draftPlan: snapshot.draftPlan ?? state.draftPlan, task: snapshot.task ?? state.task }, ctx);
+      showPlanMenuCommandFallback("Plan approval menu was dismissed. The plan remains ready for approval.");
+      return;
+    }
     if (menuChoice === "Approve and Execute" || menuChoice === "Approve and Run Workflow") {
       await approveCurrentPlan(ctx, "plan approved by user");
     } else if (menuChoice === "Revise Plan") {
@@ -17432,11 +17459,6 @@ Public workflow commands:
       else if (isMissionWorkflowMode(state)) updateState({ missionTokensUsed: (state.missionTokensUsed ?? 0) + turnTokens }, ctx);
     }
 
-    if (initialPlanParentSuppressed()) {
-      traceWorkflowTracking(ctx, "typed-initial-plan-parent-message-suppressed", { mode: state.mode, lifecycleStatus: state.planProgress?.lifecycleStatus });
-      return { message: { ...message, content: [{ type: "text" as const, text: "" }] } };
-    }
-
     if (planReviewParentSuppressed()) {
       traceWorkflowTracking(ctx, "typed-plan-review-parent-message-suppressed", { mode: state.mode, lifecycleStatus: state.planProgress?.lifecycleStatus });
       return { message: { ...message, content: [{ type: "text" as const, text: "" }] } };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mediadatafusion/pi-workflow-suite",
-  "version": "0.0.13",
+  "version": "0.0.15",
   "description": "Structured workflow orchestration suite for Pi with Standard, Plan, Mission, compaction, diagrams, web access, repo lock, and safety gates.",
   "license": "Apache-2.0",
   "repository": {
@@ -114,7 +114,9 @@
     "prepack": "node scripts/prepare-package-readme.mjs apply",
     "postpack": "node scripts/prepare-package-readme.mjs restore --pack",
     "restore:package-readme": "node scripts/prepare-package-readme.mjs restore",
-    "prepublishOnly": "node scripts/prepare-package-readme.mjs apply --publish",
+    "prepare:publish-readme": "node scripts/prepare-package-readme.mjs apply --publish && node scripts/prepare-package-readme.mjs check",
+    "cleanup:publish-readme": "node scripts/prepare-package-readme.mjs restore",
+    "prepublishOnly": "node scripts/prepare-package-readme.mjs ensure-publish-ready",
     "postpublish": "node scripts/prepare-package-readme.mjs restore --publish",
     "check:package-readme": "node scripts/prepare-package-readme.mjs check"
   }

package/scripts/prepare-package-readme.mjs

@@ -106,11 +106,25 @@ function check() {
   assertPackageReadme(readFileSync(readmePath, 'utf8'));
 }
 
+function ensurePublishReady() {
+  check();
+  if (!existsSync(publishMarkerPath)) {
+    throw new Error(
+      [
+        'package README is package-safe but publish marker is missing.',
+        'Run `npm run prepare:publish-readme` before `npm publish --access public` so npm sees the package README before publish starts.',
+        'If a previous publish failed, run `npm run cleanup:publish-readme` first.',
+      ].join(' '),
+    );
+  }
+}
+
 const command = process.argv[2];
 if (command === 'apply') apply();
 else if (command === 'restore') restore();
 else if (command === 'check') check();
+else if (command === 'ensure-publish-ready') ensurePublishReady();
 else {
-  console.error('Usage: prepare-package-readme.mjs apply|restore|check [--publish|--pack]');
+  console.error('Usage: prepare-package-readme.mjs apply|restore|check|ensure-publish-ready [--publish|--pack]');
   process.exit(1);
 }