@agimon-ai/doompi-workflow 0.0.1-alpha.125 → 0.0.1-alpha.127

package/README.md

@@ -108,12 +108,15 @@ session creates a child **workflow session** (provenance `workflow-session`), ne
 rail, and hands it the launch. The workflow session owns the run (`PI_SESSION_ID`), runs its step
 sessions, and has Workflow mode on, so its agent can troubleshoot and recover the run. The run's
 environment also carries `DOOMPI_WORKFLOW_LAUNCHER_SESSION_ID`, naming the launching session, which
-lists the run in its Activity group, is told when it ends, and may read its status.
+lists the run in its Activity group, is told when it completes, and may read its status. Failures only
+raise a UI toast there, without adding a message to the launching agent's context.
 
 - The workflow session's rail row shows the run (`workflow · build › test · 4m`, or a failure or
   pause marked like a pending question) through the host's per-session activity line.
-- A failure or finish is posted into the workflow session's conversation as a notice, without
-  starting an agent turn. The owner agent also gets a live brief of its runs on every turn.
+- A failure starts a turn in the workflow session to diagnose and report locally. Recovery requires
+  a user request or explicit launcher delegation. Completion is posted without starting a turn,
+  and completion after recovery still notifies the launcher. The owner agent also gets a live brief
+  of its runs on every turn.
 - After a success, a workflow session nobody interacted with asks its launcher to release it, and it
   becomes `stopped · open to wake`. A session whose run failed or paused, or whose agent you
   prompted, stays live until you remove it. Removing or restarting a live workflow session stops its
@@ -175,3 +178,9 @@ readiness discovery; dependent calls await that session's readiness. On shutdown
 also handles startup failure. Both use the same idempotent cleanup. A generation
 fence prevents retired callbacks from sending messages or updating the UI; only
 explicit resource teardown may use a retired UI context.
+
+## Step completion
+
+In-process steps wait, without a timeout or decision reminders, for the child session to be idle with no queued messages or owned subagent/task/runner work. Cancellation interrupts this wait. Missing coordinator snapshots and holding-provider errors defer the decision gate; a failed inspection fails the step. Workflow monitoring itself does not hold completion.
+
+Pi terminal steps must load this package: its native settled handler owns the decision gate, not generic `hooks.yaml`. It skips host-steered sessions and subagents, and defers the gate while owned background work remains. Missing pane coordinators retain the no-work fallback; snapshot failures defer gating with a diagnostic.

package/dist/extensions/pi.cjs

@@ -2,17 +2,18 @@ Object.defineProperties(exports, {
 	__esModule: { value: true },
 	[Symbol.toStringTag]: { value: "Module" }
 });
-const require_workflowTerminal = require("../workflowTerminal-DeJ9BBta.cjs");
+const require_workflowTerminal = require("../workflowTerminal-ILzqDAL9.cjs");
 const require_workflowLaunchCommand = require("../workflowLaunchCommand-DJyDB96d.cjs");
 let _agimon_ai_doompi_core_piExtension = require("@agimon-ai/doompi-core/piExtension");
 let _agimon_ai_doompi_core_extensionFile = require("@agimon-ai/doompi-core/extensionFile");
+let _agimon_ai_doompi_core_backgroundWork = require("@agimon-ai/doompi-core/backgroundWork");
 let _agimon_ai_doompi_core_skills = require("@agimon-ai/doompi-core/skills");
 let _agimon_ai_doompi_core_uiHub = require("@agimon-ai/doompi-core/uiHub");
+let _agimon_ai_workflow_mcp = require("@agimon-ai/workflow-mcp");
 let _agimon_ai_doompi_core_childProcess = require("@agimon-ai/doompi-core/childProcess");
 let node_fs = require("node:fs");
 let node_fs_promises = require("node:fs/promises");
 let node_path = require("node:path");
-let _agimon_ai_workflow_mcp = require("@agimon-ai/workflow-mcp");
 let zod = require("zod");
 let node_crypto = require("node:crypto");
 let _agimon_ai_doompi_ui_toolChrome = require("@agimon-ai/doompi-ui/toolChrome");
@@ -23,6 +24,18 @@ let _agimon_ai_doompi_core_readiness = require("@agimon-ai/doompi-core/readiness
 let _agimon_ai_doompi_minor_mode = require("@agimon-ai/doompi-minor-mode");
 let _agimon_ai_doompi_telemetry = require("@agimon-ai/doompi-telemetry");
 let _earendil_works_pi_tui = require("@earendil-works/pi-tui");
+//#region src/services/stepStopGate/index.ts
+function isStepStopGateEligible(environment) {
+	return Boolean(environment[_agimon_ai_workflow_mcp.WORKFLOW_DECISION_FILE_ENV] && environment[_agimon_ai_workflow_mcp.WORKFLOW_DECISION_STEERING_ENV] !== _agimon_ai_workflow_mcp.WORKFLOW_DECISION_STEERING_HOST && !environment.PI_SUBAGENT_CHILD);
+}
+/** A pane's root agent may settle to wait without spending a workflow reminder. */
+async function stepStopGate(environment, snapshot) {
+	if (!isStepStopGateEligible(environment)) return void 0;
+	if (snapshot?.items.some(_agimon_ai_doompi_core_backgroundWork.holdsSettledSession) || snapshot?.errors.some((error) => _agimon_ai_doompi_core_backgroundWork.BACKGROUND_WORK_HOLDING_PROVIDERS.includes(error.provider))) return void 0;
+	const gate = await (0, _agimon_ai_workflow_mcp.gateStepDecision)(environment);
+	return gate.action === "block" ? gate.message : void 0;
+}
+//#endregion
 //#region src/services/workflowTools/index.ts
 const PI_LAUNCH_FIELDS = {
 	workflowPath: true,
@@ -3485,6 +3498,7 @@ function createWorkflowPiRuntime(pi, options) {
 				const startedDuringSession = previous === void 0 && (sessionStartedAt === 0 || !Number.isFinite(startedAt) || startedAt >= sessionStartedAt);
 				if (lifecycleNarrationReady && require_workflowTerminal.isSessionRun(record, sessionId)) narrateWorkflowTransition(narrationSink, record, previous === RUNNING_STATUS || previous === COMPLETED_STATUS || previous === ERROR_STAGE ? previous : void 0, startedDuringSession);
 				if (!notifyChanges || !rootProcess || record.stage === RUNNING_STATUS) continue;
+				if (record.stage === ERROR_STAGE && require_workflowTerminal.isLaunchedRun(record, sessionId)) continue;
 				if (!deliveredTerminalRuns.has(identity) && (previous === RUNNING_STATUS || startedDuringSession)) pendingTerminalRuns.set(identity, record);
 			}
 			const retainedWork = new Map(mine.map((record) => [runIdentity(record), record]));
@@ -3513,8 +3527,7 @@ function createWorkflowPiRuntime(pi, options) {
 					}
 					const identity = runIdentity(record);
 					const events = runEvents.get(identity) ?? progressEventsForRun(record, await (0, _agimon_ai_workflow_mcp.readWorkflowProgress)(registry.runDirectoryFor(record)));
-					const summary = require_workflowTerminal.finishedRunSummary(record, (0, _agimon_ai_workflow_mcp.summarizeWorkflowProgress)(events));
-					return require_workflowTerminal.isLaunchedRun(record, sessionId) ? `${summary}\n${require_workflowTerminal.launchedRunHint(record)}` : summary;
+					return require_workflowTerminal.finishedRunSummary(record, (0, _agimon_ai_workflow_mcp.summarizeWorkflowProgress)(events));
 				}));
 				if (!refreshIsCurrent()) return;
 				const message = (indexes) => ({
@@ -3536,8 +3549,8 @@ function createWorkflowPiRuntime(pi, options) {
 						})
 					}
 				});
-				const quiet = terminalRecords.flatMap((record, index) => require_workflowTerminal.isOwnedDelegatedRun(record, sessionId) ? [index] : []);
-				const loud = terminalRecords.flatMap((record, index) => require_workflowTerminal.isOwnedDelegatedRun(record, sessionId) ? [] : [index]);
+				const quiet = terminalRecords.flatMap((record, index) => require_workflowTerminal.isOwnedDelegatedRun(record, sessionId) && record.stage !== ERROR_STAGE ? [index] : []);
+				const loud = terminalRecords.flatMap((record, index) => require_workflowTerminal.isOwnedDelegatedRun(record, sessionId) && record.stage !== ERROR_STAGE ? [] : [index]);
 				if (quiet.length > 0) pi.sendMessage(message(quiet), { triggerTurn: false });
 				if (loud.length > 0) pi.sendMessage(message(loud), {
 					triggerTurn: true,
@@ -4521,6 +4534,7 @@ function createWorkflowPiRuntime(pi, options) {
 var index_cli_default = (({ pi, context, options, signal }) => {
 	const environment = options?.environment ?? Object.freeze({ ...process.env });
 	const dispatcher = require_workflowTerminal.isWorkflowDispatcherProcess(environment);
+	let backgroundWork;
 	const fence = require_workflowTerminal.createWorkflowFence(pi, signal);
 	let workflowMode = false;
 	let leader;
@@ -4547,6 +4561,13 @@ var index_cli_default = (({ pi, context, options, signal }) => {
 	const selectedTools = dispatcher ? require_workflowTerminal.dispatcherTools(nativeTools, require_workflowTerminal.resolveDispatcherParentSession(environment), environment) : nativeTools;
 	return {
 		services: [...runtime.services ?? [], (cordis) => {
+			cordis.inject([_agimon_ai_doompi_core_backgroundWork.DOOM_BACKGROUND_WORK_SERVICE], (serviceContext) => {
+				const service = (0, _agimon_ai_doompi_core_backgroundWork.readDoomBackgroundWorkService)(serviceContext);
+				backgroundWork = service;
+				return () => {
+					if (backgroundWork === service) backgroundWork = void 0;
+				};
+			});
 			cordis.inject([_agimon_ai_doompi_core_skills.DOOM_SKILL_SOURCES_SERVICE], (skillContext) => {
 				const directory = require_workflowTerminal.workflowSkillDirectory();
 				const contribution = (0, _agimon_ai_doompi_core_skills.requireDoomSkillSourcesService)(skillContext).register({
@@ -4573,7 +4594,32 @@ var index_cli_default = (({ pi, context, options, signal }) => {
 		}))),
 		commands: runtime.commands.map(([name, command]) => fence.command(name, command)),
 		shortcuts: runtime.shortcuts?.map(([key, shortcut]) => fence.shortcut(key, shortcut)),
-		events: fence.events(runtime.events ?? {}),
+		events: fence.events({
+			...runtime.events,
+			async agent_settled(_event, ctx) {
+				if (!fence.isCurrentInvocation() || !isStepStopGateEligible(environment)) return;
+				let snapshot;
+				try {
+					snapshot = backgroundWork?.snapshot(ctx.sessionManager.getSessionId());
+				} catch (error) {
+					pi.sendMessage({
+						customType: "workflow-step",
+						content: `Cannot inspect step background work: ${String(error)}`,
+						display: true
+					});
+					return;
+				}
+				const message = await stepStopGate(environment, snapshot);
+				if (message && fence.isCurrentInvocation()) pi.sendMessage({
+					customType: "workflow-step",
+					content: message,
+					display: true
+				}, {
+					triggerTurn: true,
+					deliverAs: "followUp"
+				});
+			}
+		}),
 		toolRestrictions: [...runtime.toolRestrictions ?? [], ...dispatcher ? [{
 			source: `${require_workflowTerminal.PACKAGE_SOURCE}/dispatcher`,
 			restrict: require_workflowTerminal.dispatcherToolRestriction()
@@ -4618,6 +4664,9 @@ var root_cli_default = (0, _agimon_ai_doompi_core_extensionFile.defineRoot)((con
 //#region src/extensions/workspaces/sessions/(backend)/command/workflow-launch.cli.ts
 var workflow_launch_cli_default = (0, _agimon_ai_doompi_core_extensionFile.defineRoutedContribution)((context) => context.root.commands[0], { cardinality: "optional" });
 //#endregion
+//#region src/extensions/workspaces/sessions/(backend)/hook/agent-settled.cli.ts
+var agent_settled_cli_default = (0, _agimon_ai_doompi_core_extensionFile.defineRoutedContribution)((context) => context.root.events?.agent_settled, { cardinality: "optional" });
+//#endregion
 //#region src/extensions/workspaces/sessions/(backend)/hook/resources-discover.cli.ts
 var resources_discover_cli_default = (0, _agimon_ai_doompi_core_extensionFile.defineRoutedContribution)((context) => context.root.events?.resources_discover, { cardinality: "optional" });
 //#endregion
@@ -4750,6 +4799,7 @@ const extension = (0, _agimon_ai_doompi_core_piExtension.definePiExtension)("@ag
 			return defined([at(workflow_launch_cli_default, contextSession)]);
 		},
 		events: {
+			...keyed("agent_settled", at(agent_settled_cli_default, contextSession)),
 			...keyed("resources_discover", at(resources_discover_cli_default, contextSession)),
 			...keyed("session_start", at(session_start_cli_default, contextSession))
 		},