@osolmaz/pi-workflows 0.12.0 → 0.13.0

package/src/workflows/index.ts

@@ -1,5 +1,6 @@
 export {
   agent,
+  assistantMessage,
   action,
   checkpoint,
   compute,
@@ -23,8 +24,8 @@ export {
   decisionPresentationFingerprint,
   digestCanonical,
   humanDecisionChannelRequest,
-  legacyDecisionPresentation,
   normalizeDecisionPresentation,
+  valueDecisionPresentation,
   validateHumanDecisionRequestIntegrity,
   type DecisionDocumentSegment,
 } from "./decision-presentation.js";
@@ -38,7 +39,7 @@ export {
   digest,
   humanDecision,
   humanDecisionEdge,
-  humanDecisionStateRoot,
+  humanDecisionDatabasePath,
   textInput,
   validateHumanDecisionResponse,
   validateHumanDecisionSubmission,
@@ -112,42 +113,37 @@ export {
   validateWorkflowUpdate,
 } from "./updates.js";
 export { sanitizeText, stripAnsi } from "./text.js";
-export {
-  ARTIFACT_THRESHOLD_BYTES,
-  ArtifactWriter,
-  decodeValueWith,
-  encodeValue,
-  isArtifactValue,
-  resolveArtifacts,
-} from "./artifacts.js";
 export {
   DEFINITION_SNAPSHOT_SCHEMA,
-  RUN_BUNDLE_SCHEMA,
   RUN_STATE_SCHEMA,
   SESSION_BINDING_SCHEMA,
-  TRACE_EVENT_SCHEMA,
   WorkflowRunStore,
   createDefinitionSnapshot,
   createRunId,
-  listRunBundles,
-  readRunBundle,
-  workflowRunsBaseDir,
-  type LoadedRunBundle,
-  type ReadRunBundleOptions,
+  listWorkflowRuns,
+  readLastTraceEvent,
+  readWorkflowRun,
+  workflowStateDatabasePath,
+  type LoadedWorkflowRun,
+  type ReadWorkflowRunOptions,
+  type RunWriteAuthority,
 } from "./store.js";
 export type {
   AcceptedHumanDecision,
-  AcceptedHumanDecisionV1,
-  AcceptedHumanDecisionV2,
+  DefaultedHumanDecision,
+  ResolvedHumanDecision,
   AgentNodeDefinition,
+  AgentExpectedOutput,
+  AgentStepCompletion,
   AgentStepContract,
+  AssistantAgentNodeDefinition,
+  AssistantMessageOutput,
+  AssistantMessageReceipt,
   AgentStepExecutor,
   AgentStepPresentation,
   AgentStepRequest,
   AgentStepSubmission,
   ActionNodeDefinition,
-  ArtifactRef,
-  ArtifactValue,
   CheckpointNodeDefinition,
   ConversationRange,
   ComputeNodeDefinition,
@@ -167,26 +163,21 @@ export type {
   HumanDecisionChoiceMap,
   HumanDecisionContinuationRecord,
   HumanDecisionDeliveryRecord,
-  HumanDecisionDeliveryRecordV1,
-  HumanDecisionDeliveryRecordV2,
   HumanDecisionNodeContract,
   HumanDecisionPrompt,
   HumanDecisionReceipt,
-  HumanDecisionReceiptV1,
-  HumanDecisionReceiptV2,
   HumanDecisionRequest,
-  HumanDecisionRequestV1,
-  HumanDecisionRequestV2,
   HumanDecisionResponse,
   HumanDecisionSettlementRecord,
   HumanDecisionSubmission,
   HumanDecisionTextInput,
-  LegacyHumanDecisionPrompt,
+  HumanDecisionTimeout,
+  HumanDecisionTimeoutPolicy,
   MaybePromise,
   NotifyNodeDefinition,
-  PresentedHumanDecisionPrompt,
   ShellActionExecution,
   ShellActionNodeDefinition,
+  SubmittedAgentNodeDefinition,
   ShellActionResult,
   WorkflowActionContext,
   WorkflowActionReceipt,
@@ -216,7 +207,6 @@ export type {
   WorkflowPresentationContext,
   WorkflowProgressData,
   WorkflowProgressStatus,
-  WorkflowRunManifest,
   WorkflowRunResult,
   WorkflowRunState,
   WorkflowRunStatus,

package/src/workflows/progress.ts

@@ -18,6 +18,7 @@ export type ProgressEstimate = {
   confidence?: ProgressConfidence;
   sourceEstimatedFinishAt?: string;
   unavailableReason?: string;
+  elapsedMs?: number;
 };
 
 export type ProgressSample = {
@@ -42,7 +43,16 @@ export function estimateProgress(
   for (const sample of samples) validateProgressData(sample.data as Record<string, unknown>);
   const latest = samples.at(-1) as ProgressSample;
   const data = latest.data;
-  const base: ProgressEstimate = { key, data, sampleCount: 1 };
+  const firstAt = Date.parse(samples[0]!.at);
+  const endAt = TERMINAL.has(data.status) ? Date.parse(latest.at) : now.getTime();
+  const elapsedMs =
+    Number.isFinite(firstAt) && Number.isFinite(endAt) ? Math.max(0, endAt - firstAt) : undefined;
+  const base: ProgressEstimate = {
+    key,
+    data,
+    sampleCount: 1,
+    ...(elapsedMs !== undefined ? { elapsedMs } : {}),
+  };
   if (TERMINAL.has(data.status)) return base;
   const sourceFinish = validSourceFinish(latest, now);
   if (sourceFinish !== undefined) base.sourceEstimatedFinishAt = sourceFinish;
@@ -212,10 +222,15 @@ export function formatProgressLine(estimate: ProgressEstimate, now = new Date())
         ? `${formatRemaining(estimate.remainingLowMs)}–${formatRemaining(estimate.remainingHighMs)}`
         : formatRemaining(estimate.remainingMedianMs);
     eta = `ETA ${range}`;
-  } else if (!TERMINAL.has(data.status)) {
+  } else if (!TERMINAL.has(data.status) && data.phase === undefined) {
     eta = `ETA unavailable${estimate.unavailableReason ? ` (${estimate.unavailableReason})` : ""}`;
   }
-  return [label, count, eta].filter(Boolean).join("  ");
+  const phase = data.phase === undefined ? "" : sanitizeText(data.phase);
+  const elapsed =
+    data.phase === undefined || estimate.elapsedMs === undefined
+      ? ""
+      : `elapsed ${formatRemaining(estimate.elapsedMs)}`;
+  return [label, count, phase, elapsed, eta].filter(Boolean).join("  ");
 }
 
 export function formatProgressReport(

package/src/workflows/schema.ts

@@ -51,7 +51,25 @@ export function assertValidAgentNode(node: AgentNodeDefinition, nodeId = "agent"
     fail(`node ${nodeId} requires a prompt function`);
   }
   if (node.expectedOutput !== undefined && typeof node.expectedOutput !== "string") {
-    fail(`node ${nodeId} expectedOutput must be a string`);
+    const output = node.expectedOutput as { kind?: unknown; maxChars?: unknown };
+    if (output === null || typeof output !== "object" || output.kind !== "assistant-message") {
+      fail(`node ${nodeId} expectedOutput must be a string or assistantMessage()`);
+    }
+    const unknown = Object.keys(output).filter((key) => key !== "kind" && key !== "maxChars");
+    if (unknown.length > 0) {
+      fail(`node ${nodeId} assistantMessage has unknown field ${JSON.stringify(unknown[0])}`);
+    }
+    if (
+      output.maxChars !== undefined &&
+      (typeof output.maxChars !== "number" ||
+        !Number.isInteger(output.maxChars) ||
+        output.maxChars <= 0)
+    ) {
+      fail(`node ${nodeId} assistantMessage maxChars must be a positive integer`);
+    }
+    if (node.validate !== undefined) {
+      fail(`node ${nodeId} cannot use validate with assistantMessage()`);
+    }
   }
   assertOptionalFunction(node.validate, `node ${nodeId} validate`);
   assertCommonNodeFields(node, nodeId);
@@ -151,6 +169,23 @@ export function assertValidCheckpointNode(
     if (typeof node.humanDecision.request !== "function") {
       fail(`node ${nodeId} humanDecision request must be a function`);
     }
+    if (
+      node.humanDecision.onTimeout !== undefined &&
+      typeof node.humanDecision.onTimeout !== "function"
+    ) {
+      assertRecord(node.humanDecision.onTimeout, `node ${nodeId} humanDecision onTimeout`);
+      if (
+        typeof node.humanDecision.onTimeout.afterMs !== "number" ||
+        !Number.isFinite(node.humanDecision.onTimeout.afterMs) ||
+        node.humanDecision.onTimeout.afterMs <= 0
+      ) {
+        fail(`node ${nodeId} humanDecision onTimeout afterMs must be a finite positive number`);
+      }
+      assertRecord(
+        node.humanDecision.onTimeout.response,
+        `node ${nodeId} humanDecision onTimeout response`,
+      );
+    }
   }
   assertCommonNodeFields(node, nodeId);
 }