@sentry/nextjs: withSentry API helper types incompatible with NextJS

[osdiab]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other: @sentry/nextjs

Version:

6.3.5

Description

nextjs's built in NextApiHandler type for API handlers returns a void | Promise<void> but the withSentry() function from @sentry/nextjs incorrectly only accepts Promise<void>, which means
that using it with non-async API routes makes types fail. withSentry should take in a NextApiHandler, not roll its own overly strict type signature.

[lobsterkatie]

Thanks for pointing this out! Will push a fix.