refactor(workflow-executor): rename history to previousSteps and document executionResult

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: alban bertolini

packages/workflow-executor/src/executors/base-step-executor.ts

@@ -36,7 +36,7 @@ export default abstract class BaseStepExecutor<TStep extends StepDefinition = St
    * Empty array when there is no history. Ready to spread into a messages array.
    */
   protected async buildPreviousStepsMessages(): Promise<SystemMessage[]> {
-    if (!this.context.history.length) return [];
+    if (!this.context.previousSteps.length) return [];
 
     const summary = await this.summarizePreviousSteps();
 
@@ -52,7 +52,7 @@ export default abstract class BaseStepExecutor<TStep extends StepDefinition = St
   private async summarizePreviousSteps(): Promise<string> {
     const allStepExecutions = await this.context.runStore.getStepExecutions();
 
-    return this.context.history
+    return this.context.previousSteps
       .map(({ stepDefinition, stepOutcome }) => {
         const execution = allStepExecutions.find(e => e.stepIndex === stepOutcome.stepIndex);
 

packages/workflow-executor/src/types/execution.ts

@@ -39,7 +39,7 @@ export interface ExecutionContext<TStep extends StepDefinition = StepDefinition>
   readonly agentPort: AgentPort;
   readonly workflowPort: WorkflowPort;
   readonly runStore: RunStore;
-  readonly history: ReadonlyArray<Readonly<Step>>;
+  readonly previousSteps: ReadonlyArray<Readonly<Step>>;
   readonly remoteTools: readonly unknown[];
   readonly userInput?: UserInput;
 }

packages/workflow-executor/src/types/step-execution-data.ts

@@ -45,6 +45,7 @@ export interface ReadRecordStepExecutionData extends BaseStepExecutionData {
 export interface UpdateRecordStepExecutionData extends BaseStepExecutionData {
   type: 'update-record';
   executionParams?: { fieldDisplayName: string; value: string };
+  /** User confirmed → values returned by updateRecord. User rejected → skipped. */
   executionResult?: { updatedValues: Record<string, unknown> } | { skipped: true };
   pendingUpdate?: {
     fieldDisplayName: string;

packages/workflow-executor/test/executors/base-step-executor.test.ts

@@ -73,7 +73,7 @@ function makeContext(overrides: Partial<ExecutionContext> = {}): ExecutionContex
     agentPort: {} as ExecutionContext['agentPort'],
     workflowPort: {} as ExecutionContext['workflowPort'],
     runStore: makeMockRunStore(),
-    history: [],
+    previousSteps: [],
     remoteTools: [],
     ...overrides,
   };
@@ -90,7 +90,7 @@ describe('BaseStepExecutor', () => {
     it('includes prompt and executionParams from previous steps', async () => {
       const executor = new TestableExecutor(
         makeContext({
-          history: [makeHistoryEntry({ stepId: 'cond-1', stepIndex: 0, prompt: 'Approve?' })],
+          previousSteps: [makeHistoryEntry({ stepId: 'cond-1', stepIndex: 0, prompt: 'Approve?' })],
           runStore: makeMockRunStore([
             {
               type: 'condition',
@@ -115,7 +115,7 @@ describe('BaseStepExecutor', () => {
     it('uses Input for matched steps and History for unmatched steps', async () => {
       const executor = new TestableExecutor(
         makeContext({
-          history: [
+          previousSteps: [
             makeHistoryEntry({ stepId: 'cond-1', stepIndex: 0 }),
             makeHistoryEntry({ stepId: 'cond-2', stepIndex: 1, prompt: 'Second?' }),
           ],
@@ -145,7 +145,7 @@ describe('BaseStepExecutor', () => {
     it('falls back to History when no matching step execution in RunStore', async () => {
       const executor = new TestableExecutor(
         makeContext({
-          history: [
+          previousSteps: [
             makeHistoryEntry({ stepId: 'orphan', stepIndex: 5, prompt: 'Orphan step' }),
             makeHistoryEntry({ stepId: 'matched', stepIndex: 1, prompt: 'Matched step' }),
           ],
@@ -181,7 +181,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([]),
         }),
       );
@@ -205,7 +205,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([]),
         }),
       );
@@ -234,7 +234,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([]),
         }),
       );
@@ -270,7 +270,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [condEntry, aiEntry],
+          previousSteps: [condEntry, aiEntry],
           runStore: makeMockRunStore([
             {
               type: 'ai-task',
@@ -297,7 +297,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([
             {
               type: 'condition',
@@ -334,7 +334,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([
             {
               type: 'ai-task',
@@ -359,7 +359,7 @@ describe('BaseStepExecutor', () => {
 
       const executor = new TestableExecutor(
         makeContext({
-          history: [entry],
+          previousSteps: [entry],
           runStore: makeMockRunStore([
             {
               type: 'condition',

packages/workflow-executor/test/executors/condition-step-executor.test.ts

@@ -53,7 +53,7 @@ function makeContext(
     agentPort: {} as ExecutionContext['agentPort'],
     workflowPort: {} as ExecutionContext['workflowPort'],
     runStore: makeMockRunStore(),
-    history: [],
+    previousSteps: [],
     remoteTools: [],
     ...overrides,
   };
@@ -175,7 +175,7 @@ describe('ConditionStepExecutor', () => {
       const context = makeContext({
         model: mockModel.model,
         runStore,
-        history: [
+        previousSteps: [
           {
             stepDefinition: {
               type: StepType.Condition,

packages/workflow-executor/test/executors/read-record-step-executor.test.ts

@@ -111,7 +111,7 @@ function makeContext(
     agentPort: makeMockAgentPort(),
     workflowPort: makeMockWorkflowPort(),
     runStore: makeMockRunStore(),
-    history: [],
+    previousSteps: [],
     remoteTools: [],
     ...overrides,
   };
@@ -650,7 +650,7 @@ describe('ReadRecordStepExecutor', () => {
       const context = makeContext({
         model: mockModel.model,
         runStore,
-        history: [
+        previousSteps: [
           {
             stepDefinition: {
               type: StepType.Condition,

packages/workflow-executor/test/executors/update-record-step-executor.test.ts

@@ -112,7 +112,7 @@ function makeContext(
     agentPort: makeMockAgentPort(),
     workflowPort: makeMockWorkflowPort(),
     runStore: makeMockRunStore(),
-    history: [],
+    previousSteps: [],
     remoteTools: [],
     ...overrides,
   };
@@ -620,7 +620,7 @@ describe('UpdateRecordStepExecutor', () => {
       const context = makeContext({
         model: mockModel.model,
         runStore,
-        history: [
+        previousSteps: [
           {
             stepDefinition: {
               type: StepType.Condition,