Use new approach for setting source

Author: AbhiPrasad

packages/utils/src/env.ts

@@ -15,7 +15,9 @@
 
 declare const __SENTRY_BROWSER_BUNDLE__: boolean | undefined;
 
-declare const __SENTRY_CDN_BUNDLE__: boolean | undefined;
+type SdkSource = 'npm' | 'cdn' | 'loader';
+
+declare const __SENTRY_SDK_SOURCE__: SdkSource | undefined;
 
 /**
  * Figures out if we're building a browser bundle.
@@ -27,8 +29,8 @@ export function isBrowserBundle(): boolean {
 }
 
 /**
- * Figures out if we're a CDN or npm bundle.
+ * Get source of SDK.
  */
-export function isCDNBundle(): 'cdn' | 'npm' {
-  return typeof __SENTRY_CDN_BUNDLE__ !== 'undefined' && !!__SENTRY_CDN_BUNDLE__ ? 'npm' : 'cdn';
+export function getSDKSource(): SdkSource {
+  return typeof __SENTRY_SDK_SOURCE__ !== 'undefined' ? __SENTRY_SDK_SOURCE__ : 'npm';
 }

rollup/bundleHelpers.js

@@ -17,7 +17,7 @@ import {
   makeTerserPlugin,
   makeTSPlugin,
   makeExcludeReplayPlugin,
-  makeIsCDNBundlePlugin,
+  makeSetSDKSourcePlugin,
 } from './plugins/index.js';
 import { mergePlugins } from './utils';
 
@@ -142,29 +142,29 @@ export function makeBundleConfigVariants(baseConfig, options = {}) {
   const includeDebuggingPlugin = makeIsDebugBuildPlugin(true);
   const stripDebuggingPlugin = makeIsDebugBuildPlugin(false);
   const terserPlugin = makeTerserPlugin();
-  const isCDNBundlePlugin = makeIsCDNBundlePlugin(true);
+  const setSdkSourcePlugin = makeSetSDKSourcePlugin('cdn');
 
   // The additional options to use for each variant we're going to create.
   const variantSpecificConfigMap = {
     '.js': {
       output: {
         entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.js`,
       },
-      plugins: [includeDebuggingPlugin, isCDNBundlePlugin],
+      plugins: [includeDebuggingPlugin, setSdkSourcePlugin],
     },
 
     '.min.js': {
       output: {
         entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.min.js`,
       },
-      plugins: [stripDebuggingPlugin, terserPlugin, isCDNBundlePlugin],
+      plugins: [stripDebuggingPlugin, setSdkSourcePlugin, terserPlugin],
     },
 
     '.debug.min.js': {
       output: {
         entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.debug.min.js`,
       },
-      plugins: [includeDebuggingPlugin, terserPlugin, isCDNBundlePlugin],
+      plugins: [includeDebuggingPlugin, setSdkSourcePlugin, terserPlugin],
     },
   };
 

rollup/npmHelpers.js

@@ -13,7 +13,7 @@ import {
   makeCleanupPlugin,
   makeSucrasePlugin,
   makeDebugBuildStatementReplacePlugin,
-  makeIsCDNBundlePlugin,
+  makeSetSDKSourcePlugin,
 } from './plugins/index.js';
 import { mergePlugins } from './utils';
 
@@ -32,7 +32,7 @@ export function makeBaseNPMConfig(options = {}) {
   const debugBuildStatementReplacePlugin = makeDebugBuildStatementReplacePlugin();
   const cleanupPlugin = makeCleanupPlugin();
   const extractPolyfillsPlugin = makeExtractPolyfillsPlugin();
-  const isNPMBundlePlugin = makeIsCDNBundlePlugin(false);
+  const setSdkSourcePlugin = makeSetSDKSourcePlugin('npm');
 
   const defaultBaseConfig = {
     input: entrypoints,
@@ -85,11 +85,11 @@ export function makeBaseNPMConfig(options = {}) {
 
     plugins: [
       nodeResolvePlugin,
+      setSdkSourcePlugin,
       sucrasePlugin,
       debugBuildStatementReplacePlugin,
       cleanupPlugin,
       extractPolyfillsPlugin,
-      isNPMBundlePlugin,
     ],
 
     // don't include imported modules from outside the package in the final output

rollup/plugins/bundlePlugins.js

@@ -63,10 +63,10 @@ export function makeIsDebugBuildPlugin(includeDebugging) {
   });
 }
 
-export function makeIsCDNBundlePlugin(isCDNBundle) {
+export function makeSetSDKSourcePlugin(sdkSource) {
   return replace({
     values: {
-      __SENTRY_CDN_BUNDLE__: isCDNBundle,
+      __SENTRY_SDK_SOURCE__: sdkSource,
     },
   });
 }