Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(vercel-edge): create @sentry/vercel-edge package and make @sentry/sveltekit edge ready #8688

Closed

Conversation

isaacharrisholt
Copy link

@isaacharrisholt isaacharrisholt commented Aug 1, 2023

This PR is an attempt to solve #8087 and make the SvelteKit SDK usable on the Vercel Edge runtime.


Task tracker:

  • Create @sentry/vercel-edge SDK package
  • Copy code from Nextjs SDK to Vercel Edge runtime SDK
  • Add tests to Vercel Edge runtime SDK
  • Fix lint issues in Vercel Edge runtime SDK
  • Make Nextjs SDK depend on and use Vercel Edge runtime SDK
  • Make SvelteKit SDK use Vercel Edge runtime SDK

@Lms24
Copy link
Member

Lms24 commented Aug 1, 2023

Hi @isaacharrisholt thanks for opening this PR!

Before you spend more time working on this: We're currently working on establishing a concrete plan for proper non-Node runtime support (which includes Vercel's edge runtime). We're evaluating a few options how to move forward and we'll probably do it a bit differently than what's outlined in #8087. Therefore I'd recommend to pause work on your PR until we have this plan. Again, thanks a lot for contributing - we'll let you know once we know more and of course also make sure you're properly credited if we can move forward with this PR!

@isaacharrisholt
Copy link
Author

isaacharrisholt commented Aug 1, 2023

@Lms24 sounds good! Is there a rough timeline for when I can expect this to be worked on? We use Sentry at my company, but we're currently limited to only frontend error monitoring using the Svelte (not SvelteKit) SDK, which really isn't ideal.

It would also be great if you could share any plans for the non-Node runtimes :)

@Lms24
Copy link
Member

Lms24 commented Aug 1, 2023

@isaacharrisholt it's best to follow #8693 for the progress. We're taking a look at this soon :)

@AbhiPrasad
Copy link
Member

Going to close this as we have merged in #9041. Sorry for the long delay on this but we had to make some architecture changes under the hood to accommodate everything.

Vercel Edge Runtime SDK will be out next release!

@AbhiPrasad AbhiPrasad closed this Sep 19, 2023
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.

3 participants