@bastani/atomic 0.9.13-alpha.1 → 0.9.13-alpha.2

package/dist/builtin/workflows/src/durable/types.ts

@@ -61,6 +61,10 @@ export interface DurableWorkflowHandle {
 	readonly rootWorkflowId?: string;
 	/** Explicit resumability flag for failed/blocked runs. */
 	readonly resumable?: boolean;
+	/** True when the terminal status came from an author-selected ctx.exit(). */
+	readonly exited?: boolean;
+	/** Author-supplied reason from ctx.exit(), when present. */
+	readonly exitReason?: string;
 	readonly error?: string;
 	readonly failureKind?: WorkflowFailureKind;
 	readonly failureCode?: WorkflowFailureCode;
@@ -278,6 +282,10 @@ export interface DurableWorkflowMetadata {
 	readonly failedToolNodeId?: string;
 	readonly rootWorkflowId?: string;
 	readonly resumable?: boolean;
+	/** True when the terminal status came from an author-selected ctx.exit(). */
+	readonly exited?: boolean;
+	/** Author-supplied reason from ctx.exit(), when present. */
+	readonly exitReason?: string;
 	readonly invocationCwd?: string;
 	readonly origin?: WorkflowActor;
 	readonly workflowCwd?: string;
@@ -304,6 +312,10 @@ export interface ResumableWorkflowEntry {
 	readonly failedToolNodeId?: string;
 	readonly rootWorkflowId?: string;
 	readonly resumable?: boolean;
+	/** True when the terminal status came from an author-selected ctx.exit(). */
+	readonly exited?: boolean;
+	/** Author-supplied reason from ctx.exit(), when present. */
+	readonly exitReason?: string;
 	readonly invocationCwd?: string;
 	readonly origin?: WorkflowActor;
 	readonly workflowCwd?: string;
@@ -314,7 +326,11 @@ export interface ResumableWorkflowEntry {
 }
 
 export interface DurableWorkflowFailureMetadata {
-	readonly error: string;
+	readonly error?: string;
+	/** True when the terminal status came from an author-selected ctx.exit(). */
+	readonly exited?: boolean;
+	/** Author-supplied reason from ctx.exit(), when present. */
+	readonly exitReason?: string;
 	readonly failureKind?: WorkflowFailureKind;
 	readonly failureCode?: WorkflowFailureCode;
 	readonly failureRecoverability?: WorkflowFailureRecoverability;

package/dist/builtin/workflows/src/durable/workflow-child-result.ts

@@ -49,7 +49,9 @@ export function isWorkflowChildResult(
 }
 
 function isWorkflowExitStatus(value: WorkflowSerializableValue | undefined): value is WorkflowExitStatus {
-	return value === "completed" || value === "skipped" || value === "cancelled" || value === "blocked";
+	return (
+		value === "completed" || value === "skipped" || value === "cancelled" || value === "blocked" || value === "failed"
+	);
 }
 
 function isOrdinaryObject(

package/dist/builtin/workflows/src/engine/primitives/workflow.ts

@@ -143,7 +143,7 @@ export function createChildWorkflowRunner(input: {
 			) {
 				throw new WorkflowGracefulQuitError(childRunId, `child workflow "${childName}" (${child.name})`);
 			}
-			if (!isWorkflowExitStatus(childRun.status)) {
+			if (!isWorkflowExitStatus(childRun.status) || (childRun.status === "failed" && childRun.exited !== true)) {
 				const failedChildStage = childRun.stages.find((stage) => stage.failureKind !== undefined);
 				throw new Error(
 					`atomic-workflows: child workflow "${childName}" (${child.name}) failed with status ${childRun.status}${childRun.error !== undefined ? `: ${childRun.error}` : ""}`,

package/dist/builtin/workflows/src/engine/run-durable-finalize.ts

@@ -40,9 +40,17 @@ export async function finalizeDurableTerminalStatus(input: DurableTerminalFinali
 			recordRunTimingCheckpoint(input.durableBackend, input.runSnapshot);
 		}
 		const failure =
-			durableStatus === "failed" && input.runSnapshot.error !== undefined
+			durableStatus === "failed"
 				? {
-						error: input.runSnapshot.error,
+						...(input.runSnapshot.error !== undefined ? { error: input.runSnapshot.error } : {}),
+						...(input.runSnapshot.exited === true
+							? {
+									exited: true,
+									...(input.runSnapshot.exitReason !== undefined
+										? { exitReason: input.runSnapshot.exitReason }
+										: {}),
+								}
+							: {}),
 						...(input.runSnapshot.failureKind !== undefined
 							? { failureKind: input.runSnapshot.failureKind }
 							: {}),

package/dist/builtin/workflows/src/engine/run-durable-topology.ts

@@ -238,6 +238,7 @@ export function reconcileCachedDirectChildParentStage(input: {
 	readonly boundary: StageSnapshot;
 }): boolean {
 	const existingRun = input.store.runs().find((run) => run.id === input.checkpointChildRunId);
+	if (existingRun === undefined) return false;
 	const runById = new Map(input.store.runs().map((run) => [run.id, run]));
 	const storedParentRun = runById.get(input.parentRun.id);
 	if (
@@ -250,6 +251,13 @@ export function reconcileCachedDirectChildParentStage(input: {
 	const expectedRootRunId = reciprocalWorkflowRootRunId(runById, input.parentRun.id);
 	if (expectedRootRunId === undefined) return false;
 	const storedBoundary = storedParentRun?.stages.find((stage) => stage.id === input.boundary.id);
+	const cachedChildStatusMatches =
+		(existingRun?.status === "completed" && input.catalogRun.status === "completed") ||
+		(input.boundary.workflowChild?.exited === true &&
+			input.boundary.workflowChild.runId === input.checkpointChildRunId &&
+			existingRun?.status === input.catalogRun.status &&
+			input.boundary.workflowChild.status === input.catalogRun.status);
+
 	if (
 		!rootMatches(input.parentRun.rootRunId, expectedRootRunId) ||
 		!rootMatches(storedParentRun?.rootRunId, expectedRootRunId) ||
@@ -257,8 +265,7 @@ export function reconcileCachedDirectChildParentStage(input: {
 		(storedParentRun !== undefined &&
 			authoritativeWorkflowChildRunId(storedBoundary) !== input.checkpointChildRunId) ||
 		input.catalogRun.id !== input.checkpointChildRunId ||
-		existingRun?.status !== "completed" ||
-		input.catalogRun.status !== "completed" ||
+		!cachedChildStatusMatches ||
 		existingRun.parentRunId !== input.parentRun.id ||
 		input.catalogRun.parentRunId !== input.parentRun.id ||
 		input.catalogRun.parentStageId === undefined ||

package/dist/builtin/workflows/src/extension/atomic-stage-session.ts

@@ -49,9 +49,15 @@ export interface PrepareAtomicStageSessionOptions {
 	resourceLoaderInheritanceSnapshot?: DefaultResourceLoaderInheritanceSnapshot;
 	onSettingsManager?: (settingsManager: PiSdkSettingsManager) => void;
 }
+/**
+ * Workflow stages are top-level sessions that carry a policy object; they are
+ * not subagent children. They keep full management and are authorized to
+ * delegate, as `packages/coding-agent/docs/workflows.md` documents. Nesting
+ * stays bounded by the depth guard in the subagent executor.
+ */
 const WORKFLOW_STAGE_SUBAGENT_POLICY: SubagentChildPolicy = {
 	managementActions: "full",
-	fanoutAuthorized: false,
+	fanoutAuthorized: true,
 	inheritProjectContext: true,
 	inheritSkills: true,
 };

package/dist/builtin/workflows/src/extension/companions.ts

@@ -65,7 +65,7 @@ export const COMPANIONS: readonly CompanionSpec[] = [
 		installSpec: "npm:pi-subagents",
 		purpose: "delegate stages to focused child agents",
 		pathHints: ["/pi-subagents/", "/pi-subagents-", "/pi-subagents."],
-		commandHints: ["subagents-doctor", "run", "chain", "parallel", "run-chain"],
+		commandHints: ["subagents-doctor", "run", "parallel"],
 		toolHints: ["subagent"],
 	},
 	{