feat(sveltekit): Introduce client-side handleError wrapper

[AbhiPrasad]

ref #7402

Adds a wrapper for the client-side handleError hook.

Usage is like the following:

src/hooks.client.js

import { wrapHandleError } from '@sentry/sveltekit';

/** @type {import('@sveltejs/kit').HandleClientError} */
function myCustomHandleError() {
 ...
}

export const handleError = wrapHandleError(myCustomHandleError);

[github-actions]

size-limit report 📦

Path	Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified)	20.24 KB (+0.12% 🔺)
@sentry/browser - ES5 CDN Bundle (minified)	62.95 KB (+0.11% 🔺)
@sentry/browser - ES6 CDN Bundle (gzipped + minified)	18.87 KB (+0.1% 🔺)
@sentry/browser - ES6 CDN Bundle (minified)	55.87 KB (+0.12% 🔺)
@sentry/browser - Webpack (gzipped + minified)	20.62 KB (+0.12% 🔺)
@sentry/browser - Webpack (minified)	67.39 KB (+0.1% 🔺)
@sentry/react - Webpack (gzipped + minified)	20.65 KB (+0.12% 🔺)
@sentry/nextjs Client - Webpack (gzipped + minified)	48.54 KB (+0.18% 🔺)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified)	27.51 KB (+0.23% 🔺)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified)	25.76 KB (+0.21% 🔺)
@sentry/replay ES6 CDN Bundle (gzipped + minified)	43.17 KB (+0.01% 🔺)
@sentry/replay - Webpack (gzipped + minified)	37.19 KB (0%)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified)	61.18 KB (+0.09% 🔺)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified)	54.35 KB (+0.04% 🔺)

[Lms24]

Nice, looks great! Let's just make sure that the conditional export paths are fine in the tarball, then this is good to 🚀

[psytrx]

Currently, sequence does not support HandleClientError/HandleServerError. See source.

This would allow us to filter out, transform and enrich errors in an idiomatic way.

export const handleError = sequence(
  filterUselessErrors,
  addContextOrMetadata,
  sentryHandleError,
  myCustomHandleError);

Would it make sense for you to suggest such a change to the SvelteKit team? I'm open to starting a discussion. I'm not quite sure about the consequences, but it feels like it should be a non-breaking change.

[Lms24]

@fjdumont thanks for the suggestion. We're definitely going to consider sequence for the server-side handle hook. If SvelteKit would support sequence for handleError, we could also make consider it in the SDK.

Feel free to go ahead and open a discussion on the SvelteKit repo 👍