feat(core): split exports by browser/server for bundle size

Split the exports from `@sentry/core` into three options:

- `@sentry/core`, the default (unchanged)
- `@sentry/core/browser`, containing _only_ shared and browser-specific
  functionality, nothing server-specific.
- `@sentry/core/server`, containing _only_ shared and server-specific
  functionality, nothing browser-specific.

This allows us to make the bundle sizes quite a bit smaller in our
browser SDKs where this is important, while adding more functionality to
our server-specific SDKs, in `@sentry/core` where they can be easily
shared across runtimes.

Requires updating our `tsconfig` settings so that tsc knows it is
allowed to look on `package.json` exports. This then, requires setting
some other packages to `"moduleResolution": "bundler"` and `"module":
"esnext"` in order to continue working as before. In the case of
sveltekit, some types appear to just not be exported in a
node16-compatible way, meaning that we now need to read them from the
exported function rather than dive into the package contents.

fix: #20434
fix: JS-2243

Author: isaacs

dev-packages/test-utils/tsconfig.json

@@ -3,6 +3,7 @@
   "compilerOptions": {
     "target": "ES2022",
     "module": "ES2022",
+    "moduleResolution": "bundler",
     // package-specific options
     "esModuleInterop": true
   },

packages/angular/package.json

@@ -38,7 +38,7 @@
     "@types/node": "^14.8.0",
     "ng-packagr": "^14.2.2",
     "rxjs": "7.8.2",
-    "typescript": "4.6.4",
+    "typescript": "4.8",
     "zone.js": "^0.12.0"
   },
   "scripts": {

packages/angular/src/errorhandler.ts

@@ -3,8 +3,8 @@ import type { ErrorHandler as AngularErrorHandler, OnDestroy } from '@angular/co
 import { Inject, Injectable } from '@angular/core';
 import type { ReportDialogOptions } from '@sentry/browser';
 import * as Sentry from '@sentry/browser';
-import type { Event } from '@sentry/core';
-import { consoleSandbox, isString } from '@sentry/core';
+import type { Event } from '@sentry/core/browser';
+import { consoleSandbox, isString } from '@sentry/core/browser';
 import { runOutsideAngular } from './zone';
 
 /**

packages/angular/src/sdk.ts

@@ -10,15 +10,15 @@ import {
   linkedErrorsIntegration,
   setContext,
 } from '@sentry/browser';
-import type { Client, Integration } from '@sentry/core';
+import type { Client, Integration } from '@sentry/core/browser';
 import {
   applySdkMetadata,
   conversationIdIntegration,
   debug,
   dedupeIntegration,
   functionToStringIntegration,
   inboundFiltersIntegration,
-} from '@sentry/core';
+} from '@sentry/core/browser';
 import { IS_DEBUG_BUILD } from './flags';
 
 /**

packages/angular/src/tracing.ts

@@ -20,8 +20,8 @@ import {
   startBrowserTracingNavigationSpan,
   startInactiveSpan,
 } from '@sentry/browser';
-import type { Integration, Span } from '@sentry/core';
-import { debug, stripUrlQueryAndFragment, timestampInSeconds } from '@sentry/core';
+import type { Integration, Span } from '@sentry/core/browser';
+import { debug, stripUrlQueryAndFragment, timestampInSeconds } from '@sentry/core/browser';
 import type { Observable } from 'rxjs';
 import { Subscription } from 'rxjs';
 import { filter, tap } from 'rxjs/operators';

packages/angular/test/errorhandler.test.ts

@@ -1,6 +1,6 @@
 import { HttpErrorResponse } from '@angular/common/http';
 import * as SentryBrowser from '@sentry/browser';
-import type { Client, Event } from '@sentry/core';
+import type { Client, Event } from '@sentry/core/browser';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import { createErrorHandler, SentryErrorHandler } from '../src/errorhandler';
 

packages/angular/test/tracing.test.ts

@@ -4,7 +4,7 @@ import {
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
   SentrySpan,
   spanToJSON,
-} from '@sentry/core';
+} from '@sentry/core/browser';
 import { describe, it } from 'vitest';
 import { browserTracingIntegration, init, TraceDirective } from '../src/index';
 import { _updateSpanAttributesForParametrizedUrl, getParameterizedRouteFromSnapshot } from '../src/tracing';

packages/angular/vitest.config.ts

@@ -1,4 +1,11 @@
+// angular has to use an old ts version, so we can't make this as perfectly
+// typed as most of our other code, which is why linting is disabled.
+// thankfully, it's small enough to not be a problem.
+
+// oxlint-disable
+
 import { defineConfig } from 'vitest/config';
+//@ts-ignore
 import baseConfig from '../../vite/vite.config';
 
 export default defineConfig({

packages/astro/src/client/browserTracingIntegration.ts

@@ -3,13 +3,13 @@ import {
   startBrowserTracingPageLoadSpan,
   WINDOW,
 } from '@sentry/browser';
-import type { Integration, TransactionSource } from '@sentry/core';
+import type { Integration, TransactionSource } from '@sentry/core/browser';
 import {
   browserPerformanceTimeOrigin,
   debug,
   SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
-} from '@sentry/core';
+} from '@sentry/core/browser';
 import { DEBUG_BUILD } from '../debug-build';
 
 /**

packages/astro/src/client/sdk.ts

@@ -1,7 +1,7 @@
 import type { BrowserOptions } from '@sentry/browser';
 import { getDefaultIntegrations as getBrowserDefaultIntegrations, init as initBrowserSdk } from '@sentry/browser';
-import type { Client, Integration } from '@sentry/core';
-import { applySdkMetadata } from '@sentry/core';
+import type { Client, Integration } from '@sentry/core/browser';
+import { applySdkMetadata } from '@sentry/core/browser';
 import { browserTracingIntegration } from './browserTracingIntegration';
 
 // Tree-shakable guard to remove all code related to tracing