Skip to content

ref(server-utils): Add orchestrion support for Vercel AI v5 - #21979

Merged
mydea merged 5 commits into
developfrom
fn/vercel-ai-v5
Jul 8, 2026
Merged

ref(server-utils): Add orchestrion support for Vercel AI v5#21979
mydea merged 5 commits into
developfrom
fn/vercel-ai-v5

Conversation

@mydea

@mydeamydea commented Jul 6, 2026

Copy link
Copy Markdown
Member

This adds orchestrion support for Vercel AI v5 (v6 support already exists).

@mydeamydea self-assigned this Jul 6, 2026
@mydea
mydeaforce-pushed the fn/vercel-ai-v5 branch 2 times, most recently from f47a718 to d2d3306CompareJuly 7, 2026 10:12

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d2d3306. Configure here.

@github-actions

github-actionsBot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser27.59 kB--
@sentry/browser - with treeshaking flags26.03 kB--
@sentry/browser (incl. Tracing)46.27 kB--
@sentry/browser (incl. Tracing + Span Streaming)48.03 kB--
@sentry/browser (incl. Tracing, Profiling)51.03 kB--
@sentry/browser (incl. Tracing, Replay)85.56 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags75.17 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)90.26 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)102.91 kB--
@sentry/browser (incl. Feedback)44.76 kB--
@sentry/browser (incl. sendFeedback)32.38 kB--
@sentry/browser (incl. FeedbackAsync)37.51 kB--
@sentry/browser (incl. Metrics)28.67 kB--
@sentry/browser (incl. Logs)28.91 kB--
@sentry/browser (incl. Metrics & Logs)29.59 kB--
@sentry/react29.38 kB--
@sentry/react (incl. Tracing)48.53 kB--
@sentry/vue33.03 kB--
@sentry/vue (incl. Tracing)48.16 kB--
@sentry/svelte27.61 kB--
CDN Bundle30 kB--
CDN Bundle (incl. Tracing)48.26 kB--
CDN Bundle (incl. Logs, Metrics)31.57 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)49.57 kB--
CDN Bundle (incl. Replay, Logs, Metrics)70.81 kB--
CDN Bundle (incl. Tracing, Replay)85.73 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)87.04 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)91.55 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)92.83 kB--
CDN Bundle - uncompressed89.35 kB--
CDN Bundle (incl. Tracing) - uncompressed145.99 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed94.05 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed149.96 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed218.75 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed265.16 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed269.12 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed278.86 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed282.81 kB--
@sentry/nextjs (client)50.94 kB--
@sentry/sveltekit (client)46.68 kB--
@sentry/core/server78.39 kB--
@sentry/core/browser64.74 kB--
@sentry/node-core63.21 kB--
@sentry/node124.73 kB+0.01%+1 B 🔺
@sentry/node (incl. diagnostics channel injection)131.22 kB+0.19%+247 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection)69.95 kB--
@sentry/node/light50.72 kB--
@sentry/node - without tracing74.82 kB+0.01%+1 B 🔺
@sentry/aws-serverless85.6 kB+0.01%+1 B 🔺
@sentry/cloudflare (withSentry) - minified181.68 kB--
@sentry/cloudflare (withSentry)449.39 kB--

View base workflow run

@mydea
mydea marked this pull request as ready for review July 7, 2026 12:35
@mydea
mydea requested a review from a team as a code ownerJuly 7, 2026 12:35
@mydea
mydea requested review from JPeer264 and andreiborza and removed request for a teamJuly 7, 2026 12:35
Comment threadpackages/server-utils/src/orchestrion/config/vercel-ai.ts Outdated
@mydea
mydea requested a review from andreiborzaJuly 8, 2026 07:32

@andreiborzaandreiborza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for taking this over!

@JPeer264JPeer264 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

enrichSpanOnEnd(span, message, options);
span.end();
return value;
}, failSpan);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

super-l: I guess this can be omitted, then we wouldn't need failSpan to have its own function and it can live in the catch block directly

Suggested change
},failSpan);
});

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this does not work because we are not awaiting the result and we likely do not want to because that would change things we return possibly 🤔

@mydea
mydea merged commit 5d85e5c into developJul 8, 2026
214 checks passed
@mydea
mydea deleted the fn/vercel-ai-v5 branch July 8, 2026 08:27
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.

3 participants

@mydea@JPeer264@andreiborza