[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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('^' + ".*" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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('^' + ".*" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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('^' + ".*" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, '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); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent
, 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

[1/2] Replace shared observability component styles with Tailwind - #3295

Merged
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1
Aug 12, 2026
Merged

[1/2] Replace shared observability component styles with Tailwind#3295
mitul-s merged 6 commits into
mainfrom
c+ms/remove-inline-styles-86b1

Conversation

@mitul-s

@mitul-smitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in shared sidebar, stream, and reusable UI components with Tailwind utilities.
  • Map Geist design-system colors to existing semantic theme utilities.
  • Keep computed animation geometry and consumer-provided style overrides inline.

Stack

  1. This PR — shared UI components
  2. Trace viewer

Merge in order from 1 to 2.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 11 changed files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in WebOpen in Cursor

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
example-nextjs-workflow-turbopackReadyReadyPreviewAug 12, 2026 7:11pm
example-nextjs-workflow-webpackReadyReadyPreviewAug 12, 2026 7:11pm
example-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-astro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-express-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-fastify-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-hono-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nestjs-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nitro-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-nuxt-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-python-workflowErrorErrorAug 12, 2026 7:11pm
workbench-sveltekit-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-tanstack-start-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workbench-vite-workflowReadyReadyPreviewAug 12, 2026 7:11pm
workflow-docsReadyReadyPreview, v0Aug 12, 2026 7:11pm
workflow-swc-playgroundReadyReadyPreviewAug 12, 2026 7:11pm
workflow-tarballsReadyReadyPreviewAug 12, 2026 7:11pm
workflow-webReadyReadyPreviewAug 12, 2026 7:11pm

@changeset-bot

changeset-botBot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b4f68e

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

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

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

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

📊 Workflow Benchmarks

commit e9f8abf · Mon, 03 Aug 2026 19:15:51 GMT · run logs

Backend: vercel · app: nextjs-turbopack

MetricScenarioBest (ms)P75 (ms)P90 (ms)P99 (ms)Samples
TTFSstep1187 (+351%) 🔻1457 🔴 (+34%) 🔻1491 🔴 (+34%) 🔻1604 🔴 (+21%) 🔻30
TTFSstream1151 (+423%) 🔻1263 🔴 (+23%) 🔻1342 🔴 (+29%) 🔻1415 🔴 (+30%) 🔻30
TTFShook + stream1472 (+275%) 🔻1712 🔴 (+36%) 🔻1774 🔴 (+36%) 🔻2119 🔴 (+34%) 🔻30
STSO1020 steps (inline)210 (+23%) 🔻502 (±0%)556 (+0.7%)795 (+7.1%)1016
STSO1020 steps (queue-hop)2276 (+78%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3445 (+31%) 🔻3
WO1020 steps442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)442990 (+6.2%)1
SLstream latency131 (+42%) 🔻190 🔴 (+29%) 🔻242 🔴 (+41%) 🔻3846 🔴 (+818%) 🔻30
SOstream overhead (text)149 (+27%) 🔻221 (-11%)276 (-16%) 💚681 (-98%) 💚30
SOstream overhead (structured)145 (+28%) 🔻213 (+3.9%)223 (-18%) 💚303 (-24%) 💚30
📈 STSO distribution vs main (inline / queue-hop histograms)

1020 steps (inline)

Cumulative STSO time: main 409957ms → this run 432357ms (Δ +22400ms, +5%)

 150-200 ms ┃█ main 14 this 0 -14
200-250 ms ███████┃██████ main 92 this 54 -38
250-300 ms ███████████████┃████ main 136 this 104 -32
300-350 ms ██████████████████████┃ main 151 this 154 +3
350-400 ms █████████████████████░░┃ main 143 this 161 +18
400-450 ms ██████████████████░░┃ main 118 this 138 +20
450-500 ms ███████████████░░░░░┃ main 101 this 141 +40
500-550 ms ██████████████████████┃ main 150 this 154 +4
550-600 ms ██████┃█ main 57 this 49 -8
600-650 ms ███┃ main 23 this 25 +2
650-700 ms █┃ main 13 this 15 +2
700-750 ms ┃ main 8 this 7 -1
750-800 ms ┃ main 4 this 4 +0
800-850 ms ┃ main 3 this 4 +1
850-900 ms ┃ main 2 this 1 -1
950-1000 ms ┃ main 0 this 2 +2
1100-1150 ms ┃ main 0 this 1 +1
1250-1300 ms ┃ main 1 this 1 +0
4050-4100 ms ┃ main 0 this 1 +1

1020 steps (queue-hop)

Cumulative STSO time: main 6303ms → this run 9140ms (Δ +2837ms, +45%)

1000-1500 ms ┃███████████ main 1 this 0 -1
2000-2500 ms ███████████┃ main 1 this 1 +0
2500-3000 ms ┃███████████ main 1 this 0 -1
3000-3500 ms ░░░░░░░░░░░░░░░░░░░░░░░┃ main 0 this 2 +2
ℹ️ Metric definitions & methodology

The collapsed STSO distribution section above buckets every step gap of the sequential-steps run (not a sampled window), split by whether the step ending the gap ran inline — in the same warm process as the step before it, so the gap is pure framework overhead — or after a queue-hop — the first step of a fresh process, which pays queue dispatch, client reinit and event-log replay. Bars overlay the two runs: is main, marks where this run lands, bridges the gap when this run has more samples in a bucket.

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, deployment clocks) · STSO: step-to-step overhead (gap between consecutive step bodies) · WO: workflow overhead (whole-run time outside step bodies, in-deployment anchored) · SL: stream latency (in-deployment write → read propagation, readAt - writtenAt) · SO: stream overhead (end-to-end write+consume time beyond the modelled generation window)

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 · stream latency: parallel reader/writer steps on a dedicated stream; SL is the in-deployment write->read propagation (readAt - writtenAt) · stream overhead (text): writer streams 300 variable-length text token deltas paced at 100/s for 3s (a haiku-size LLM's token throughput) while a parallel reader drains the whole stream; SO is the end-to-end write+consume time beyond the 3s generation window (overhead/backpressure) · stream overhead (structured): same workload as stream overhead (text), but each delta is an AI-SDK-style structured object ({ type: 'text-delta', id, text }) instead of a raw string, so the SO gap vs the text scenario is the added serialization cost

🔴 marks a percentile over its target (within target is left unmarked). Targets (p75/p90/p99, ms) — TTFS 200/300/600 · SL 50/60/125 · SO 250/500/1000

All metrics are measured from deployment-side timestamps only. Runs are triggered by an in-deployment route that stamps the anchor (clientStart) right before start(), so the CI runner’s request and its path through api.vercel.com sit outside every measured window. TTFS = in-deployment start() → first step body (turbo uses the in-process fast path, non-turbo the dispatch path), and includes the VQS dispatch hop plus any /flow cold start. STSO/WO are measured between step bodies on the deployment. SL is measured inside the workflow (parallel reader/writer steps), so it no longer includes the api.vercel.com read path.

Cold starts are kept in the numbers on purpose — they are part of real bursty-workload latency. The workbench deployment cold-starts the /flow invocation for a large fraction of runs, inflating P75+; the Best column shows the fastest (warm-start) sample for comparison.

@github-actions

github-actionsBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪 E2E Test Results

Some tests failed

❌ Failed E2E Tests

▲ Vercel Production (2 failed)

nextjs-turbopack (2 failed):

  • AbortController abortHookOrderingWorkflow [listener-first-hook-first]: addEventListener → hook.then → resumeHook → abort()
  • setAttributes setAttributesInsideStepWorkflow: step-body calls append attributed native events

💻 Local Development (1 failed)

nextjs-webpack-canary (1 failed):

  • AbortController abortSurvivesReplayWorkflow: controller state consistent across replay

E2E Test Summary

Summary
PassedFailedSkippedTotal
❌ ▲ Vercel Production145322391694
❌ 💻 Local Development146612271694
✅ 📦 Local Production162102271848
✅ 🐘 Local Postgres162102271848
✅ 🪟 Windows15400154
✅ 📋 Other102002121232
✅ vercel-multi-region270027
Total7362311328497
Details by Category

❌ ▲ Vercel Production

AppPassedFailedSkipped
✅ astro126028
✅ example126028
✅ express126028
✅ fastify126028
✅ hono126028
❌ nextjs-turbopack14923
✅ nextjs-webpack15103
✅ nitro126028
✅ nuxt126028
✅ sveltekit14509
✅ vite126028

❌ 💻 Local Development

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
❌ nextjs-webpack-canary134119
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 📦 Local Production

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🐘 Local Postgres

AppPassedFailedSkipped
✅ astro-stable128026
✅ express-stable128026
✅ fastify-stable128026
✅ hono-stable128026
✅ nextjs-turbopack-canary135019
✅ nextjs-turbopack-stable15400
✅ nextjs-webpack-canary135019
✅ nextjs-webpack-stable15400
✅ nitro-stable128026
✅ nuxt-stable128026
✅ sveltekit-stable14707
✅ vite-stable128026

✅ 🪟 Windows

AppPassedFailedSkipped
✅ nextjs-turbopack15400

✅ 📋 Other

AppPassedFailedSkipped
✅ e2e-local-dev-nest-stable128026
✅ e2e-local-dev-tanstack-start-128026
✅ e2e-local-postgres-nest-stable128026
✅ e2e-local-postgres-tanstack-start-128026
✅ e2e-local-prod-nest-stable128026
✅ e2e-local-prod-tanstack-start-128026
✅ e2e-vercel-prod-nest126028
✅ e2e-vercel-prod-tanstack-start126028

✅ vercel-multi-region

AppPassedFailedSkipped
✅ nextjs-turbopack2700

📋 View full workflow run

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursorcursorBot changed the title Replace hard-coded UI styles with TailwindReplace shared observability component styles with TailwindAug 3, 2026
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Sim World

Simulated world deterministic testing for races. Traces

🟠 The mint run produced no summary — see the job log.

🟠 The append-only run produced no summary — see the job log.

@github-actions

Copy link
Copy Markdown
Contributor

No backport to stable for 14b52ac (AI decision).

This is a pure cosmetic refactor that swaps inline style objects for Tailwind utility classes across @workflow/web-shared observability components, including renaming the exported getResourceColor/RESOURCE_COLORS API to getResourceClassNames/RESOURCE_CLASS_NAMES. It fixes no user-visible defect and carries a small risk of visual regressions plus an export-surface change, so it does not belong on a maintenance line.

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

14b52ac04bb3e97abf4f01541072ef494eecdf6a

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.

4 participants

@mitul-s@VaguelySerious@cursoragent