fix: spotlight shaking

Author: logaretm

dev-packages/rollup-utils/plugins/npmPlugins.mjs

@@ -73,7 +73,12 @@ export function makeDebugBuildStatementReplacePlugin() {
 }
 
 export function makeProductionReplacePlugin() {
-  const pattern = /\/\* rollup-include-development-only \*\/[\s\S]*?\/\* rollup-include-development-only-end \*\/\s*/g;
+  // Use legal comments (/*!) so they're preserved by Rolldown
+  // NOTE: Due to a Rolldown limitation, the ending comment must be placed before a statement
+  // (e.g., before a return) rather than after a block, otherwise it gets stripped.
+  // See: https://github.com/rolldown/rolldown/issues/[TODO: file issue]
+  const pattern =
+    /\/\*! rollup-include-development-only \*\/[\s\S]*?\/\*! rollup-include-development-only-end \*\/\s*/g;
 
   function stripDevBlocks(code) {
     if (!code) return null;

packages/browser/src/sdk.ts

@@ -94,15 +94,14 @@ export function init(options: BrowserOptions = {}): Client | undefined {
   let defaultIntegrations =
     options.defaultIntegrations == null ? getDefaultIntegrations(options) : options.defaultIntegrations;
 
-  /* rollup-include-development-only */
+  /*! rollup-include-development-only */
   if (options.spotlight) {
     if (!defaultIntegrations) {
       defaultIntegrations = [];
     }
     const args = typeof options.spotlight === 'string' ? { sidecarUrl: options.spotlight } : undefined;
     defaultIntegrations.push(spotlightBrowserIntegration(args));
   }
-  /* rollup-include-development-only-end */
 
   const clientOptions: BrowserClientOptions = {
     ...options,
@@ -114,6 +113,9 @@ export function init(options: BrowserOptions = {}): Client | undefined {
     }),
     transport: options.transport || makeFetchTransport,
   };
+  // TODO: This is here to avoid the Rolldown limitation of not being able to place the ending comment after a block
+  // Should be moved back after the if block once the Rolldown limitation is fixed
+  /*! rollup-include-development-only-end */
   return initAndBind(BrowserClient, clientOptions);
 }