fix(core): Match Sentry package paths with Windows separators in Metro resolvers

CHANGELOG.md

@@ -11,6 +11,7 @@
 ### Fixes
 
 - Find `ios/.xcode.env` in the Expo plugin "Upload Debug Symbols to Sentry" phase when the iOS project uses Swift Package Manager instead of CocoaPods ([#6911](https://github.com/getsentry/sentry-react-native/pull/6911))
+- Remove the `@sentry/browser` feedback modules and the server-only `@sentry/core` modules from native bundles built on Windows. Before this fix, `includeWebFeedback: false` made these bundles fail at launch ([#6915](https://github.com/getsentry/sentry-react-native/pull/6915))
 
 ## 8.31.0
 

packages/core/src/js/tools/metroconfig.ts

@@ -384,7 +384,7 @@ export function withSentryResolver(config: MetroConfig, includeWebReplay: boolea
 const SENTRY_BROWSER_FEEDBACK_WRAPPER_RE = /\/feedback(Sync|Async)/;
 
 function isFromSentryBrowser(originModulePath: string): boolean {
-  return originModulePath.includes('@sentry/browser');
+  return /[\\/]@sentry[\\/]browser[\\/]/.test(originModulePath);
 }
 
 /**
@@ -413,7 +413,7 @@ const SERVER_ONLY_MODULE_RE =
   /\/(mcp-server|integrations\/(http|express|postgresjs|requestdata|consola|spanStreaming)|tracing\/(vercel-ai|openai|anthropic-ai|google-genai|langchain|langgraph)|utils\/ai)(\/|\.js|$)/;
 
 function isFromSentryCore(originModulePath: string): boolean {
-  return originModulePath.includes('@sentry/core');
+  return /[\\/]@sentry[\\/]core[\\/]/.test(originModulePath);
 }
 
 /**

packages/core/test/tools/metroconfig.test.ts

@@ -549,6 +549,18 @@ describe('metroconfig', () => {
           ExpectToBeCalledWithMetroParameters(originalResolverMock, browserContextMock, wrapperModule, 'web');
         });
 
+        test('removes wrapper when origin path uses Windows separators', () => {
+          const windowsContextMock = {
+            resolveRequest: jest.fn(),
+            originModulePath: 'C:\\project\\node_modules\\@sentry\\browser\\build\\npm\\esm\\prod\\index.js',
+          };
+          const modifiedConfig = withSentryFeedbackResolver(config, false);
+          const result = resolveRequest(modifiedConfig, windowsContextMock, wrapperModule, 'ios');
+
+          expect(result).toEqual({ type: 'empty' });
+          expect(originalResolverMock).not.toHaveBeenCalled();
+        });
+
         test('keeps wrapper when origin is not @sentry/browser', () => {
           const nonBrowserContextMock = {
             resolveRequest: jest.fn(),
@@ -713,6 +725,18 @@ describe('metroconfig', () => {
       });
     });
 
+    test('removes module when origin path uses Windows separators', () => {
+      const windowsContext = {
+        ...contextMock,
+        originModulePath: 'C:\\project\\node_modules\\@sentry\\core\\build\\esm\\index.js',
+      };
+      const modifiedConfig = withSentryExcludeServerOnlyResolver(config);
+      const result = modifiedConfig.resolver?.resolveRequest?.(windowsContext, './tracing/openai/index.js', 'android');
+
+      expect(result).toEqual({ type: 'empty' });
+      expect(originalResolverMock).not.toHaveBeenCalled();
+    });
+
     test('does not exclude modules when origin is not @sentry/core', () => {
       const nonSentryContext = {
         ...contextMock,