Skip to content

fix(cloudflare,deno,node): Align types of vercelai#22343

Merged
JPeer264 merged 2 commits into
developfrom
jp/align-vercelai-types
Jul 21, 2026
Merged

fix(cloudflare,deno,node): Align types of vercelai#22343
JPeer264 merged 2 commits into
developfrom
jp/align-vercelai-types

Conversation

@JPeer264

Copy link
Copy Markdown
Member

recordInput and recordOuput types were missing for integrations that were using the vercelAiIntegration from the server-utils package. By exporting them this is now aligned.

Only node had also force in it, so it is only extended there.

Cloudflare (the older entrypoint) and Vercel Edge only have enableTrucation

@JPeer264 JPeer264 self-assigned this Jul 17, 2026
@JPeer264
JPeer264 requested review from a team as code owners July 17, 2026 08:19
@JPeer264
JPeer264 requested review from isaacs and mydea and removed request for a team July 17, 2026 08:19
@JPeer264
JPeer264 removed request for a team, isaacs and mydea July 17, 2026 08:21
Comment thread packages/server-utils/src/index.ts Outdated

@nicohrubec nicohrubec 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.

🚀

Co-authored-by: Charly Gomez <charly.gomez1310@gmail.com>
@JPeer264
JPeer264 enabled auto-merge (squash) July 17, 2026 15:37
@JPeer264
JPeer264 merged commit fc5cad6 into develop Jul 21, 2026
609 of 611 checks passed
@JPeer264
JPeer264 deleted the jp/align-vercelai-types branch July 21, 2026 08:05
JPeer264 added a commit to getsentry/sentry-docs that referenced this pull request Jul 21, 2026
<!-- Use this checklist to make sure your PR is ready for merge. You may
delete any sections you don't need. -->

## DESCRIBE YOUR PR

Clarifying the docs. There were also some code changes needed to fix
them: getsentry/sentry-javascript#22343

## IS YOUR CHANGE URGENT?  

Help us prioritize incoming PRs by letting us know when the change needs
to go live.
- [ ] Urgent deadline (GA date, etc.): <!-- ENTER DATE HERE -->
- [ ] Other deadline: <!-- ENTER DATE HERE -->
- [x] None: Not urgent, can wait up to 1 week+

## SLA

- Teamwork makes the dream work, so please add a reviewer to your PRs.
- Please give the docs team up to 1 week to review your PR unless you've
added an urgent due date to it.
Thanks in advance for your help!

## PRE-MERGE CHECKLIST

*Make sure you've checked the following before merging your changes:*

- [ ] Checked Vercel preview for correctness, including links
- [ ] PR was reviewed and approved by any necessary SMEs (subject matter
experts)
- [ ] PR was reviewed and approved by a member of the [Sentry docs
team](https://github.com/orgs/getsentry/teams/docs)

## LEGAL BOILERPLATE

<!-- Sentry employees and contractors can delete or ignore this section.
-->

Look, I get it. The entity doing business as "Sentry" was incorporated
in the State of Delaware in 2015 as Functional Software, Inc. and is
gonna need some rights from me in order to utilize my contributions in
this here PR. So here's the deal: I retain all rights, title and
interest in and to my contributions, and by keeping this boilerplate
intact I confirm that Sentry can use, modify, copy, and redistribute my
contributions, under Sentry's choice of terms.

## EXTRA RESOURCES

- [Sentry Docs contributor guide](https://docs.sentry.io/contributing/)
Sign up for free to 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