Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .changeset/bright-waits-refresh.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
---
'@workflow/core': patch
'@workflow/world': patch
'@workflow/world-local': patch
'@workflow/world-postgres': patch
'@workflow/world-vercel': patch
---

Refresh workflow events after completing elapsed waits so concurrent hook events preserve deterministic replay order.
7 changes: 7 additions & 0 deletions .changeset/corrupted-event-log-code.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
"@workflow/core": patch
"@workflow/errors": patch
"@workflow/world": patch
---

Report corrupted event logs with a distinct `CorruptedEventLogError` type and `CORRUPTED_EVENT_LOG` run error code.
5 changes: 5 additions & 0 deletions .changeset/fix-inline-execution-flow-count-race.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@workflow/world-testing": patch
---

Fix race condition in test server's flow invocation counter that caused intermittent failures in the inline-execution test suite (e.g. "sequential steps complete in a single flow invocation"). The counter is now incremented before awaiting the flow handler, so the count is observable as soon as the run transitions to completed.
2 changes: 2 additions & 0 deletions .changeset/fix-observability-getworld-import.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
---
---
6 changes: 6 additions & 0 deletions .changeset/guard-step-consumer-events.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
'@workflow/core': patch
'@workflow/world': patch
---

Validate step, wait, and hook lifecycle events against replay ownership metadata.
5 changes: 5 additions & 0 deletions .changeset/modern-penguins-peel.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@workflow/web-shared": patch
---

trace viewer bug fix + file cleanup
5 changes: 5 additions & 0 deletions .changeset/rare-badgers-judge.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@workflow/web-shared": patch
---

updated colours on the trace viewer
5 changes: 5 additions & 0 deletions .changeset/yellow-pianos-relax.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@workflow/web-shared": patch
---

Add reduced motion for the trace viewer
2 changes: 2 additions & 0 deletions .github/actions/wait-for-vercel-project/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ outputs:
description: 'The URL of the ready deployment (e.g. https://my-project-abc123.vercel.app).'
deployment-id:
description: 'The Vercel deployment ID (dpl_xxx), resolved from the matching commit status. Empty string if it could not be resolved.'
deployment-state:
description: 'The GitHub Deployment status state that made the deployment terminal-ready, such as success or inactive.'

runs:
using: 'node20'
Expand Down
1 change: 1 addition & 0 deletions .github/actions/wait-for-vercel-project/dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -19995,6 +19995,7 @@ async function run() {
core.info(`Deployment ID: ${deploymentId}`);
core.setOutput("deployment-url", deploymentUrl);
core.setOutput("deployment-id", deploymentId);
core.setOutput("deployment-state", latest.state);
return;
}
throw new Error(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -279,6 +279,7 @@ async function run(): Promise<void> {
core.info(`Deployment ID: ${deploymentId}`);
core.setOutput('deployment-url', deploymentUrl);
core.setOutput('deployment-id', deploymentId);
core.setOutput('deployment-state', latest.state);
return;
}

Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/docs-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ jobs:
environment: ${{ github.ref == 'refs/heads/main' && 'production' || 'preview' }}

- name: Verify OpenGraph images and sitemap
if: steps.waitForDocsDeployment.outputs.deployment-state == 'success'
run: pnpm --filter docs test:smoke
env:
DEPLOYMENT_URL: ${{ steps.waitForDocsDeployment.outputs.deployment-url }}
Expand All @@ -90,6 +91,7 @@ jobs:
# See: https://vercel.com/docs/deployment-protection/methods-to-bypass-deployment-protection/trusted-sources

- name: Verify rendered docs links
if: steps.waitForDocsDeployment.outputs.deployment-state == 'success'
run: pnpm --filter docs lint:rendered-links
env:
DEPLOYMENT_URL: ${{ steps.waitForDocsDeployment.outputs.deployment-url }}
Expand Down
9 changes: 7 additions & 2 deletions packages/core/src/abort-consistency.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -340,15 +340,20 @@ describe('AbortController consistency', () => {
runId: 'wrun_test',
eventType: 'hook_created',
correlationId: hookItem.correlationId,
eventData: {},
eventData: {
token: 'test-token',
},
createdAt: new Date(),
},
{
eventId: 'evnt_1',
runId: 'wrun_test',
eventType: 'hook_received',
correlationId: hookItem.correlationId,
eventData: { payload: { reason: 'hook worked' } },
eventData: {
token: 'test-token',
payload: { reason: 'hook worked' },
},
createdAt: new Date(),
},
];
Expand Down
48 changes: 48 additions & 0 deletions packages/core/src/abort-controller.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,15 @@
* (for real-time step propagation).
*/

import { CorruptedEventLogError } from '@workflow/errors';
import { withResolvers } from '@workflow/utils';
import type { Event } from '@workflow/world';
import * as nanoid from 'nanoid';
import { monotonicFactory } from 'ulid';
import { describe, expect, it, vi } from 'vitest';
import { EventsConsumer } from './events-consumer.js';
import type { WorkflowOrchestratorContext } from './private.js';
import { dehydrateStepReturnValue } from './serialization.js';
import { createContext } from './vm/index.js';
import {
createCreateAbortController,
Expand Down Expand Up @@ -115,6 +118,51 @@ describe('AbortController in workflow VM', () => {
expect(controller.signal.aborted).toBe(true);
});

it('reports a CorruptedEventLogError when abort hook_received token mismatches the controller', async () => {
ctx = setupWorkflowContext([]);
const ProbeAbortController = createCreateAbortController(ctx);
new ProbeAbortController();
const probeHookItem = [...ctx.invocationsQueue.values()].find(
(item) => item.type === 'hook'
);
expect(probeHookItem).toBeDefined();
if (!probeHookItem || probeHookItem.type !== 'hook') {
throw new Error('Expected abort hook item');
}

const ops: Promise<any>[] = [];
ctx = setupWorkflowContext([
{
eventId: 'evnt_0',
runId: 'wrun_test',
eventType: 'hook_received',
correlationId: probeHookItem.correlationId,
eventData: {
token: 'wrong-token',
payload: await dehydrateStepReturnValue(
{ reason: 'aborted' },
'wrun_test',
undefined,
ops
),
},
createdAt: new Date(),
},
]);

const errorReceived = withResolvers<Error>();
ctx.onWorkflowError = errorReceived.resolve;

const AbortController = createCreateAbortController(ctx);
new AbortController();

const workflowError = await errorReceived.promise;
expect(workflowError).toBeInstanceOf(CorruptedEventLogError);
expect(workflowError?.message).toContain('hook_received');
expect(workflowError?.message).toContain('wrong-token');
expect(workflowError?.message).toContain(probeHookItem.token);
});

it('signal.aborted is false initially', () => {
ctx = setupWorkflowContext([]);
const AbortController = createCreateAbortController(ctx);
Expand Down
Loading
Loading