Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
[world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' [world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' [world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' [world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' [world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); })(); [world] Make the sealed log opt-in instead of default-on by VaguelySerious · Pull Request #3735 · vercel/workflow · GitHub
Skip to content

[world] Make the sealed log opt-in instead of default-on - #3735

Merged
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in
Aug 21, 2026
Merged

[world] Make the sealed log opt-in instead of default-on#3735
VaguelySerious merged 2 commits into
mainfrom
peter/sealed-log-opt-in

Conversation

@VaguelySerious

@VaguelySeriousVaguelySerious commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • default WORKFLOW_SEALED_LOG to off, so new runs are stamped at the slot-identity spec version rather than the sealed log (spec 7)
  • WORKFLOW_SEALED_LOG=1 opts in
  • reading is untouched: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime still floors at slot identity
  • floor the world-testing stamped-version assertion at mintedSpecVersion() rather than SPEC_VERSION_CURRENT

Why

Main's E2E has been red continuously since #3634 landed. Two unmet preconditions for stamping spec 7 on every new run:

1. The Python runtime cannot read a sealed log

It validates specVersion <= 6 and rejects a spec-7 run_started outright:

1 validation error for RunStartedEvent
specVersion
Input should be less than or equal to 6 [type=less_than_equal, input_value=7, input_type=int]

Every run it serves is unrunnable, which is why E2E Python Conformance and E2E Vercel Prod Tests (python - node) fail. The flag's own docs already say to leave it off where a runtime that pins its own spec range has not caught up; nothing was enforcing it.

2. A truncated sealed-log read makes a completed step look in-flight, and inline ownership then waits out a full 860s lease

This one is a composition bug between two individually-sound features.

Spec 7 assigns a log position before the write commits, so a fan-out nearly always has a position whose writer is still in flight. events.ts says so directly:

That proof does NOT hold on a sealed-log run: this page came from the densifying read, which legally cuts itself short below a position whose writer is still in flight, and under fanout there is nearly always such a position.

The densifying read truncates below that hole, and a truncated page comes back empty with hasMore: false. So a resume replay can read a log that stops below the step_completed events — the completed steps still look pending. backfillSealedLogHoles already documents the resulting wedge:

A fanout whose writer died mid-allocation delivers its remaining resumes within milliseconds; every one of those reads lands inside the young window and is told the log ends at the hole, the replay sees no completions and suspends, and the deliveries are then exhausted with the hole still open — a run wedged on a position nobody will ever fill.

What makes this a timed stall rather than a permanent wedge is inline step ownership (#2780). Those pending steps still carry an ownerMessageId on their latest step_started with no step_retrying, so isStepOwnershipActive is true and the lease has time left. The dispatch decision table then deliberately does not re-dispatch, because dispatching would double-execute a step that looks like it is still running:

instead of enqueueing the step. The owning invocation is (likely) still running the body; an immediate step message would bare-start the running step and execute it a second time (workflow#2780).

It arms a delayed run continuation at lastStartedAt + INLINE_OWNERSHIP_LEASE_SECONDS instead. That constant is 860. When the backstop fires the hole is long sealed, the log reads dense, and the run finishes in ~2s.

The data matches the constant, measured from the anchoring step_started on the e2e team, 2026-08-21 20:00-23:30 UTC:

preceding eventnmedianstddev
step_completed330861.31s0.61s
step_failed53861.09s0.47s

Sub-second variance over 383 samples: this is the lease, not queue jitter. (A second, noisier family sits at ~1086s after a step_started, stddev 56s — the owning message's redelivery re-stamping a fresh ownership epoch.)

Spec 6 cannot produce it: positions are allocated by the write that occupies them, so there is no hole, no truncation, and the replay always sees the completions. Same backend, same hour, spec-6 runs coming from PR branches whose base predates #3634:

specVersionrunsstalled >120srate
728,5003801.333%
618,81980.043%

A representative run (abortAnyInStepWorkflow, two parallel steps completing 168ms apart):

22:38:50.255 step_started stepCombiningSignals ┐ one invocation,
22:38:50.257 step_started abortFromStep ┘ inline-owned
22:38:51.379 hook_received
22:38:51.404 step_completed stepCombiningSignals
22:38:51.572 step_completed abortFromStep
... 859s, no events, no /flow invocations on the deployment ...
22:53:10.816 wait_created resumeAt=22:38:51.672 <- step_started + 860.56s
22:53:12.609 run_completed

At a 60s e2e test timeout this surfaces as a diffuse spread of unrelated failures across nearly every framework lane.

Scope

This is the kill switch the flag was built for, not a fix for the composition. It buys back a green main and unblocks the Python lanes.

The actual fix is upstream of the flag and worth doing separately. Two candidates, and the first looks right: a truncated read must not feed the ownership decision. Truncation means "the log might not end here", so concluding a step is still pending from a page that admits it cut itself short is unsound — that path should re-read or re-drive rather than arm an 860s backstop. Relatedly, sealing a hole currently repairs the log but nothing re-drives the runs whose readers were truncated by it; a seal probably owes a resume.

Note this is a separate cause from #3709. That addresses run-status long-poll pool starvation after #3570, which is why main went red on 2026-08-20; these stalls begin 2026-08-21 ~20:00 UTC when #3634 landed. Both are needed for a green main.

Compatibility

Runs created while the flag was on keep working. Verified across two Worlds over one storage directory, writer opted in and reader on the new default:

writer World stamps = 7
run created at = 7
reader World stamps = 6
run read back at = 7
events readable = 2 (run_created, run_started)
requiresNewerWorld = false
advanced under new default= completed (specVersion 7)

assertWorldSupportsRuntimeProtocol accepts [6, 7], so the World this now selects is admitted; that floor was put there for exactly this rollback.

Test plan

  • pnpm --filter @workflow/world test (160 passed)
  • pnpm --filter @workflow/world-local test (557 passed)
  • pnpm --filter @workflow/world-vercel test (548 passed)
  • pnpm --filter @workflow/core test (104 files passed, 1 skipped)
  • vitest run packages/world-testing/test/embedded.test.ts (13 passed, incl. numbers events by position)
  • turbo run typecheck --filter=@workflow/world --filter=@workflow/core
  • biome check on the changed files
  • ad-hoc: mintedSpecVersion across unset/""/0/false/1/true/malformed, and the cross-flag readback above

🤖 Generated with Claude Code

Stamping spec 7 on every new run has two unmet preconditions.
The Python runtime validates `specVersion <= 6` and rejects a spec-7
`run_started` outright, so every run it serves is unrunnable. The docs
for the flag already say to leave it off where a runtime pinning its own
spec range has not caught up; nothing was enforcing that.
Pre-assigned positions are also stranding runs. A spec-7 run stalls
between a step outcome and the resume that should follow it, and only the
queue's own redelivery moves it on ~860s later (~1086s for a step
dispatch). Measured on the e2e team over 2026-08-21 20:00-23:00 UTC,
against the same backend in the same window: spec 7 stalled 380/28500
runs (1.33%) versus spec 6 at 8/18819 (0.04%).
Reading is unchanged: SPEC_VERSION_MAX_SUPPORTED stays 7 and the runtime
still floors at slot identity, so runs created while the flag was on stay
readable and can still be advanced.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-botBot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 862f04e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
NameType
@workflow/worldPatch
@workflow/world-vercelPatch
@workflow/world-localPatch
@workflow/world-postgresPatch
@workflow/corePatch
@workflow/cliPatch
@workflow/vitestPatch
@workflow/web-sharedPatch
@workflow/webPatch
@workflow/world-testingPatch
@workflow/buildersPatch
@workflow/nextPatch
@workflow/nitroPatch
workflowPatch
@workflow/astroPatch
@workflow/nestPatch
@workflow/rollupPatch
@workflow/sveltekitPatch
@workflow/vitePatch
@workflow/nuxtPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-nextjs-workflow-webpackReadyReadyPreview, v0Aug 21, 2026 11:34pm
example-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-astro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-express-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-fastify-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-hono-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nestjs-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nitro-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-nuxt-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-python-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-sveltekit-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-tanstack-start-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workbench-vite-workflowReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-swc-playgroundReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-tarballsReadyReadyPreview, v0Aug 21, 2026 11:34pm
workflow-webReadyReadyPreview, v0Aug 21, 2026 11:34pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
workflow-docsSkippedSkippedv0Aug 21, 2026 11:34pm

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (1 failed)

nextjs-webpack-quickjs (1 failed):

  • outputStreamWorkflow - getTailIndex and getChunks getChunks returns same content as reading the stream

⚠️ Flaky E2E Tests (passed on retry)

These tests failed at least once and passed on a retry. A recurring entry here is a real race worth investigating.

19 flaky tests
  • abortAnyInStepWorkflow: AbortSignal.any inside a step composes deserialized signals (sveltekit)
  • abortHookOrderingWorkflow [hook-first-hook-first]: hook.then → addEventListener → resumeHook → abort() (vite)
  • abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort() (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (nest)
  • abortThrowIfAbortedWorkflow: throwIfAborted causes FatalError, no retries (sveltekit)
  • addTenWorkflow (nitro)
  • cancelRun via CLI - cancelling a running workflow (nextjs-turbopack)
  • cancelRun via CLI - cancelling a running workflow (sveltekit)
  • concurrent hook token conflict - two workflows cannot use the same hook token simultaneously (nextjs-webpack)
  • distributedAbortController - TTL expiration triggers signal (hono)
  • getTailIndex returns correct index after stream completes (express)
  • getTailIndex returns correct index after stream completes (nest)
  • getTailIndex returns correct index after stream completes (sveltekit)
  • no startIndex (reads all chunks) (astro)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (nuxt)
  • outputStreamInsideStepWorkflow - getWritable() called inside step functions (tanstack-start)
  • regular Error retries until success (astro)
  • sleepWinsRaceWorkflow (astro)
  • sleepWinsRaceWorkflow (tanstack-start)

🛠 Infra Events (absorbed by the harness)

Platform anomalies the e2e harness detected and worked around (e.g. a run the queue never picked up, replaced by a fresh run). Clustered timestamps indicate a backend blip; a steady drip indicates a platform issue worth escalating.

  • cold-start-warmup · suite warmup (tanstack-start) · at 23:34:49Z · abandoned wrun_01M0KASVA7EE4P9NPEWY71NBP2
  • run-pickup-stall · 'hookGetConflictWithPriorStepWorkflow' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RQWRQ62FPRWSVAHH2EF
  • run-pickup-stall · 'hookGetConflictWithParallelStepWorkfl…' - hook.getConflict() does not block step execution (nextjs-webpack) · at 23:38:56Z · abandoned wrun_01M0KB1RVYZHER39BC2H2BCPBF
  • run-pickup-stall · hookGetConflictThenStepParallelWorkflow - hook.getConflict() continuation step runs alongside other steps (nextjs-webpack) · at 23:38:57Z · abandoned wrun_01M0KB1S4J2AMZVMQM1ZGAGX83

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production357717424320
✅ 💻 Local Development392205584480
✅ 📦 Local Production392205584480
✅ 🐘 Local Postgres392205584480
✅ 🪟 Windows32000320
✅ 🌐 Cross-language Conformance90132141
✅ vercel-multi-region270027
Total156991254818248
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro-node132028
✅ astro-quickjs132028
✅ example-node132028
✅ example-quickjs132028
✅ express-node132028
✅ express-quickjs132028
✅ fastify-node132028
✅ fastify-quickjs132028
✅ hono-node132028
✅ hono-quickjs132028
✅ nest-node132028
✅ nest-quickjs132028
✅ nextjs-turbopack-node15703
✅ nextjs-turbopack-quickjs15703
✅ nextjs-webpack-node15703
❌ nextjs-webpack-quickjs15613
✅ nitro-node132028
✅ nitro-quickjs132028
✅ nuxt-node132028
✅ nuxt-quickjs132028
✅ python-node80152
✅ sveltekit-node15109
✅ sveltekit-quickjs15109
✅ tanstack-start-node132028
✅ tanstack-start-quickjs132028
✅ vite-node132028
✅ vite-quickjs132028

✅ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable-node134026
✅ astro-stable-quickjs134026
✅ express-stable-node134026
✅ express-stable-quickjs134026
✅ fastify-stable-node134026
✅ fastify-stable-quickjs134026
✅ hono-stable-node134026
✅ hono-stable-quickjs134026
✅ nest-stable-node134026
✅ nest-stable-quickjs134026
✅ nextjs-turbopack-canary-node141019
✅ nextjs-turbopack-canary-quickjs141019
✅ nextjs-turbopack-stable-node16000
✅ nextjs-turbopack-stable-quickjs16000
✅ nextjs-webpack-canary-node141019
✅ nextjs-webpack-canary-quickjs141019
✅ nextjs-webpack-stable-node16000
✅ nextjs-webpack-stable-quickjs16000
✅ nitro-stable-node134026
✅ nitro-stable-quickjs134026
✅ nuxt-stable-node134026
✅ nuxt-stable-quickjs134026
✅ sveltekit-stable-node15307
✅ sveltekit-stable-quickjs15307
✅ tanstack-start-node134026
✅ tanstack-start-quickjs134026
✅ vite-stable-node134026
✅ vite-stable-quickjs134026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack-node16000
✅ nextjs-turbopack-quickjs16000

✅ 🌐 Cross-language Conformance

AppPassedFailedSkipped
✅ python90132

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit 862f04e · Fri, 21 Aug 2026 23:49:46 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1270 (+38%) 🔻1420 🔴 (+39%) 🔻1491 🔴 (+43%) 🔻1591 🔴 (+45%) 🔻30
TTFSstream463 (+70%) 🔻1393 🔴 (+33%) 🔻1409 🔴 (+30%) 🔻1473 🔴 (+28%) 🔻30
TTFShook + stream660 (+23%) 🔻1682 🔴 (+25%) 🔻1729 🔴 (+20%) 🔻1918 🔴 (+24%) 🔻30
Fan-out TTFSPromise.all(100 steps)483 (-10%)874 (-44%) 💚1904 (+20%) 🔻2013 (+25%) 🔻10
Fan-out TTLSPromise.all(100 steps)4576 (+202%) 🔻6658 (-5.7%)7411 (-7.9%)7512 (-23%) 💚10
STSO1020 steps (inline)143 (+5.1%)215 (+11%)242 (+5.2%)348 (-4.4%)1019
WO1020 steps212130 (+12%)212130 (+12%)212130 (+12%)212130 (+12%)1
CRTTfirst chunk (pooled)85 (+3.7%)130 (+10%)148 (-22%) 💚206 (-49%) 💚28

Streams

ScenarioCRTT 1stp75p90p99CDV maxiters
paced control (100/s, 60B)105 (-3%)120 (-15%)194 (-10%)378 (-58%)131 (-19%)10
size sweep (100/s, 160B-12KB)98.5 (-4%)123 (-10%)202 (-11%)619 (+36%)142 (-11%)10
replay gateway-gpt-5.4-nano-2000t (1x)126 (+12%)114 (-33%)155 (-57%)563 (-23%)224 (-62%)3
replay eve-gpt-5.6-sol-2000t (1x)151 (+4%)117 (-16%)175 (-5%)666 (-8%)460 (-19%)2
replay eve-gpt-5.6-sol-2000t (2x)114 (-10%)146 (-33%)207 (-38%)373 (-50%)202 (-34%)3
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 188787ms → this run 211947ms (Δ +23160ms, +12%)

100-150 ms ┃█ main 69 this 3 -66
150-200 ms ████████████████┃███████ main 748 this 522 -226
200-250 ms █████░░░░░░░┃ main 141 this 412 +271
250-300 ms █┃ main 40 this 54 +14
300-350 ms ┃ main 7 this 18 +11
350-400 ms ┃ main 5 this 6 +1
400-450 ms ┃ main 3 this 1 -2
450-500 ms ┃ main 2 this 2 +0
550-600 ms ┃ main 1 this 0 -1
600-650 ms ┃ main 0 this 1 +1
650-700 ms ┃ main 1 this 0 -1
700-750 ms ┃ main 1 this 0 -1
850-900 ms ┃ main 1 this 0 -1
📈 CRTT drill-down vs main (RTT distributions & profiles)
variant RTT 1ms→5s+ avg p50 p90 p99 n
control ······██▁···· 106.8 (-11%) 99 (+1%) 194 (-10%) 378 (-58%) 3000
sweep ·····▁█▇▁▁··· 108.7 (-10%) 97 (-7%) 202 (-11%) 619 (+36%) 3000
gw 1x ·····▁█▅▁▁··· 102.4 (-33%) 94 (-18%) 155 (-57%) 563 (-23%) 5295
eve 1x ·····▁█▅▁▁··· 113.6 (-9%) 89 (-17%) 175 (-5%) 666 (-8%) 5186
eve 2x ·····▁▅█▂···· 123.5 (-27%) 110 (-18%) 207 (-38%) 373 (-50%) 7779

RTT over stream progress (avg per tenth of stream, bars scaled min→max):

control █▆▃▅▅▃▂▁▂▂ 95–126ms
sweep ▁▂▄▂▂▅█▃▁▂ 97–137ms
gw 1x ▄▃▅▂▂▁▂█▇▂ 90–124ms
eve 1x ▂▃▁▂▁▃█▂▂▁ 85–209ms
eve 2x ▂▂▃▂▁▄▅█▃▁ 100–180ms

RTT by chunk size (avg per log size bin, ~160B → ~12KB serialized, bars scaled min→max):

sweep ██▇▄▃▁▂ 107–111ms

Delivery jitter over stream progress (avg positive CDV per tenth of stream, bars scaled min→max):

control ▆▆▄██▃▃▃▄▁ 27–37ms
sweep ▁▆▇▅▆██▂▃▅ 32–50ms
gw 1x ▅▃█▁▄▂▄█▂▃ 25–35ms
eve 1x ▃▇▁▃▄▃█▁▃▂ 18–29ms
eve 2x ▄▂▃▁▅█▃▆▂▅ 20–23ms
ℹ️ Metric definitions & methodology

Streams: first-chunk RTT (the stream-open path, before any buffering/backpressure), CRTT percentiles, and worst delivery stall (CDV max). Cells are medians across iterations; per-run values in the artifacts. No 🔴/🟢 marks until targets attach.

The collapsed STSO distribution section above buckets every step gap, split inline (same warm process — pure framework overhead) vs queue-hop (fresh process — dispatch, reinit, replay). = main, = this run, = fill.

The collapsed CRTT drill-down: per-variant RTT histograms (fixed log bins, · = empty) and mean RTT/positive-CDV profile lines over stream progress and chunk size. Histograms, avgs, and profiles merge exactly across runs; p50–p99 are percentile-of-percentiles. Per-index rows live in the artifacts.

Best/P75/P90/P99 deltas compare against the most recent benchmark run on main at the time of this run. 🔻 flags a delta worse than +15%, 💚 one better than −15%.

Metrics — TTFS: time to first step body (in-deployment start() → first step body) · Fan-out TTFS: fan-out time to first step (in-deployment start() → first of the parallel step bodies to complete) · Fan-out TTLS: fan-out time to last step (in-deployment start() → last of the parallel step bodies to complete, i.e. when the Promise.all resolves) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · CRTT: chunk round-trip time (per-chunk write → read latency, one clock domain: deployment → stream backend → same deployment) · CDV: chunk delay variation / delivery jitter (inter-arrival gap minus inter-write gap per seq-adjacent pair; skew-free; the row is each run's MAX positive value, so one stall moves it)

Scenarios — step: one trivial no-op step, no stream; no hooks, so the run stays in turbo mode (in-process fast path) · stream: one streaming step; no hooks, so the run stays in turbo mode (in-process fast path) · hook + stream: registers a hook before one step, which exits turbo mode (dispatch path) · 1020 steps: 1020 trivial sequential steps; STSO is measured between consecutive steps in the given step ranges, and WO is the whole-run overhead outside step bodies · Promise.all(100 steps): 100 trivial no-op steps started together in a single Promise.all; Fan-out TTFS is the first of them to complete and Fan-out TTLS the last, both from the in-deployment clientStart, so their gap is the spread the runtime adds across the fan-out · paced control (100/s, 60B): the control: 300 tiny (~60B) deltas metronome-paced at 100/s — zero workload structure, so it reads the transport floor and flush cadence, and disambiguates transport-wide vs workload-specific when a replay row moves · size sweep (100/s, 160B-12KB): same pacing as the control with deltas padded in rotation across seven log-spaced sizes (~160B–12KB) — rotation decouples size from stream position, so it isolates whether chunk size causes latency · replay gateway-gpt-5.4-nano-2000t (1x): raw provider SSE cadence captured at the AI gateway boundary (gpt-5.4-nano, the most popular gateway model; per-token deltas p50 208B = the modal production chunk size), replayed exactly as measured — the typical customer's workload; its CDV is the typical customer's real delivery jitter · replay eve-gpt-5.6-sol-2000t (1x): a captured eve turn (gpt-5.6-sol, the most-used demanding eve model; ~2000 output tokens = production p50 turn length) replayed exactly as measured — eve's envelope protocol re-ships the cumulative message so sizes ramp 142B→13KB; the demanding outlier tenant's reality · replay eve-gpt-5.6-sol-2000t (2x): the same eve capture at 2x — the headroom/stress row; real fast-tier models emit the same chunk sizes at proportionally higher rate, so time compression is a faithful speed model · first chunk (pooled): every run's seq-0 RTT pooled across all stream scenarios — the first chunk precedes any workload differentiation, so pooling samples one shared stream-open path with exact percentiles

Replay cadences (semantic sha256) — eve-gpt-5.6-sol-2000teaf22f5946e7c61f3c65c7006d550df180cfabd4e706254a09f22aec0cfb420d · gateway-gpt-5.4-nano-2000t6f24ac518b6b83ff1d0e85a5fe78230db192716d66a7fc6b2fe022752001d041

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600

All timestamps are deployment-side; runs are triggered in-deployment, so the CI runner and api.vercel.com sit outside every measured window. TTFS = start() → first step body (includes dispatch + any cold start); Fan-out TTFS/TTLS = first/last step completion of one Promise.all from the same anchor (the gap is the runtime’s fan-out spread); STSO/WO between step bodies; CRTT inside the workflow (excludes the api.vercel.com read path).

Cold starts stay in the numbers (real bursty-workload latency, inflates P75+); Best is the warm floor.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 world-sim scenario book — 1 fail of 41 total

fence=per-spec

scenariooutcomeeventsvirtreplayviolations
smoke-no-stepscompleted30msok0
smoke-one-stepcompleted60msok0
hook-at-step-startedcompleted120msok0
hook-at-step-completedcompleted120msok0
hook-at-hook-createdcompleted120msok0
deadline-hook-winscompleted71.0hok0
deadline-expirescompleted71.0hok0
long-sleepcompleted1130.0dok0
hook-never-arrivesstalled30msskipped0
step-retries-twicecompleted102.0sok0
parallel-stepscompleted90msok0
hook-on-execution-statecompleted120msok0
peek-hook-before-branchcompleted120msok0
peek-hook-after-branchcompleted120msok0
peek-hook-at-registrationcompleted120msok0
race-hook-before-probecompleted120msok0
race-hook-after-probecompleted120msok0
race-duplicate-deliverycompleted130msok0
attr-hook-before-stepcompleted110msok0
attr-hook-after-stepcompleted110msok0
attr-from-step-bodycompleted130msok0
fork-hook-after-timeoutcompleted141.0mok0
fork-hook-before-timeoutcompleted141.0mok0
count-hook-after-timeoutcompleted171.0mok0
count-hook-before-timeoutcompleted201.0mok0
stale-read-step-count-forkcompleted201.0mok0
stale-read-equal-step-countscompleted141.0mok0
step-vs-step-forkcompleted120msok0
step-vs-step-fork-fencedcompleted120msok0
fence-catches-benign-directioncompleted125msok0
in-flight-before-decisioncompleted171.0mok0
in-flight-before-decision-countedcompleted171.0mok0
in-flight-after-decisioncompleted192.0mok0
stale-read-step-count-fork-fencedcompleted201.0mok0
fork-hook-winscompleted131.0mok0
fork-timeout-winscompleted131.0mok0
unclaimed-payload-under-forkcompleted171.0mok0
claimed-payload-under-forkcompleted171.0mok0
writers-independent-step-bodiescompleted120msok0
writers-scripted-tempocompleted120msok0
cancel-mid-stepcancelled70msskipped0

Full trace: world-sim.txt

@VaguelySerious
VaguelySerious marked this pull request as ready for review August 21, 2026 23:30
The conformance suite pinned a run's stamped specVersion to
SPEC_VERSION_CURRENT. What a World is told to stamp is mintedSpecVersion(),
and the two differ whenever a version is readable before it is mintable —
the normal mid-bump state, not a conformance defect.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel
vercelBottemporarily deployed to Preview – workflow-docs August 21, 2026 23:31 Inactive
@VaguelySerious
VaguelySerious merged commit b2cac62 into mainAug 21, 2026
76 of 82 checks passed
@VaguelySerious
VaguelySerious deleted the peter/sealed-log-opt-in branch August 21, 2026 23:32
@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for b2cac62 (AI decision).

This commit flips the WORKFLOW_SEALED_LOG default for the sealed-log spec version (spec 7), which does not exist on stable: git show origin/stable:packages/world/src/spec-version.ts caps at SPEC_VERSION_CURRENT = 3 with no mintedSpecVersion or SEALED_LOG_ENV_VAR, and packages/world-testing/src/event-ids.mts plus the docs/content/.../v5/ pages it touches are absent there. It is a mitigation for a regression introduced only on main (#3634), so there is nothing on the maintenance line for it to fix.

To override, re-run the Backport to stable workflow manually via workflow_dispatch and paste this commit SHA into the ref input:

b2cac623d33685e84090abc391178219f4edc960

@github-actionsgithub-actionsBot mentioned this pull request Aug 21, 2026
pranaygp pushed a commit that referenced this pull request Aug 22, 2026
* Revert "[world] Make the sealed log opt-in instead of default-on (#3735)"
Reverts b2cac62. New runs are stamped at spec 7 again, now that a
read which cannot see past an unfilled position waits for it instead of
reporting a log that ends there (workflow-server: derive the in-request
seal poll budget from the staleness bound).
Two things are kept from #3735 rather than reverted:
- the world-testing conformance floor at mintedSpecVersion(), which was
wrong for any staged bump and not specific to this default
- a note on mintedSpecVersion recording what default-on rests on: the
events density requirement, and that a sealed log meets it by repair
rather than by construction, so the READ has to wait
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* TEMPORARY: point world-vercel at workflow-server#839 preview
Validating the seal-poll-budget fix end to end with spec 7 on. Reverted
before merge; the override lint guard is expected to fail meanwhile.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
* Revert "TEMPORARY: point world-vercel at workflow-server#839 preview"
This reverts commit 5e17cc9.
---------
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@VaguelySerious@pranaygp