fix(node): Correctly handle Windows paths when resolving module name

packages/node/src/module.ts

@@ -1,20 +1,29 @@
 import { basename, dirname } from '@sentry/utils';
 
+/** normalizes Windows paths */
+function normalisePath(path: string): string {
+  return path
+    .replace(/^[A-Z]:/, '') // remove Windows-style prefix
+    .replace(/\\/g, '/'); // replace all `\` instances with `/`
+}
+
 /** Gets the module from a filename */
 export function getModule(filename: string | undefined): string | undefined {
   if (!filename) {
     return;
   }
 
+  const normalizedFilename = normalisePath(filename);
+
   // We could use optional chaining here but webpack does like that mixed with require
-  const base = `${
-    (require && require.main && require.main.filename && dirname(require.main.filename)) || global.process.cwd()
-  }/`;
+  const base = normalisePath(
+    `${(require && require.main && require.main.filename && dirname(require.main.filename)) || global.process.cwd()}/`,
+  );
 
   // It's specifically a module
-  const file = basename(filename, '.js');
+  const file = basename(normalizedFilename, '.js');
 
-  const path = dirname(filename);
+  const path = dirname(normalizedFilename);
   let n = path.lastIndexOf('/node_modules/');
   if (n > -1) {
     // /node_modules/ is 14 chars

packages/node/test/module.test.ts

@@ -0,0 +1,15 @@
+import { getModule } from '../src/module';
+
+describe('getModule', () => {
+  test('Windows', async () => {
+    (require.main as any) = { filename: 'C:\\Users\\Tim\\app.js' };
+
+    expect(getModule('C:\\Users\\users\\Tim\\Desktop\\node_modules\\module.js')).toEqual('module');
+  });
+
+  test('POSIX', async () => {
+    (require.main as any) = { filename: '/Users/Tim/app.js' };
+
+    expect(getModule('/Users/users/Tim/Desktop/node_modules/module.js')).toEqual('module');
+  });
+});