fix(remix): Normalise Windows separators in the deletion glob

`glob` reads a backslash as an escape, so a Windows `outDir` produced a
pattern that matched nothing and left the generated source maps on disk.
Both path flavours are now checked for absoluteness, since
`path.isAbsolute` only recognises the host platform's form.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: chargome

packages/remix/src/vite/sourceMaps.ts

@@ -54,10 +54,13 @@ export function makeAddSentryVitePlugin(options: SentryRemixVitePluginOptions):
         // Derived from the configured `outDir` rather than hardcoding `build`, so a custom Remix
         // `buildDirectory` still gets cleaned up. Remix runs a client and an SSR build, each with
         // its own `outDir`, so each pass deletes only what it emitted.
-        // Vite resolves `outDir` to an absolute path, so only a relative one may be prefixed with
-        // `./` - `.//abs/path/**/*.map` matches nothing and silently leaves the maps on disk.
-        const outDir = config.build?.outDir?.replace(/\/+$/, '') || 'build';
-        const outDirGlob = path.isAbsolute(outDir) ? outDir : `./${outDir.replace(/^\.\//, '')}`;
+        //
+        // Separators are normalised because `glob` reads a backslash as an escape, and both path
+        // flavours are checked because `path.isAbsolute` only recognises the host platform's form.
+        // Only a relative path may take the `./` prefix: `.//abs/path/**/*.map` matches nothing.
+        const outDir = (config.build?.outDir || 'build').replace(/\\/g, '/').replace(/\/+$/, '');
+        const isAbsoluteOutDir = path.posix.isAbsolute(outDir) || path.win32.isAbsolute(outDir);
+        const outDirGlob = isAbsoluteOutDir ? outDir : `./${outDir.replace(/^\.\//, '')}`;
         const filesToDelete = [`${outDirGlob}/**/*.map`];
 
         if (debug) {

packages/remix/test/vite/sourceMaps.test.ts

@@ -164,6 +164,19 @@ describe('makeAddSentryVitePlugin', () => {
     ]);
   });
 
+  // `glob` reads a backslash as an escape, so an unnormalised Windows `outDir` matches nothing and
+  // leaves the maps on disk. Both path flavours are checked, so this holds on POSIX runners too.
+  it('normalises a Windows absolute outDir', async () => {
+    const plugins = makeAddSentryVitePlugin({});
+    const configPlugin = plugins.find(plugin => plugin.name === 'sentry-remix-files-to-delete-after-upload');
+
+    (configPlugin?.config as (config: UserConfig) => void)({ build: { outDir: 'C:\\proj\\build\\client' } });
+
+    await expect(capturedOptions?.sourcemaps?.filesToDeleteAfterUpload).resolves.toEqual([
+      'C:/proj/build/client/**/*.map',
+    ]);
+  });
+
   it('keeps the source maps when the user set their own build.sourcemap', async () => {
     const plugins = makeAddSentryVitePlugin({});
     const configPlugin = plugins.find(plugin => plugin.name === 'sentry-remix-files-to-delete-after-upload');