Fix React build: Keep chunk imports relative for ES module compatibility

- Fixed blank screen issue caused by invalid ES module specifiers
- Chunk imports now use relative paths (../chunk-*.js) instead of absolute paths
- Absolute paths starting with 'vs/' are not valid ES module specifiers
- Other imports continue to use absolute paths for proper resolution

Author: Tajudeen

src/vs/workbench/contrib/cortexide/browser/react/build.js

@@ -183,11 +183,21 @@ function fixImportPaths() {
 			}
 
 			// Replace all relative imports with absolute paths
-			// BUT keep chunk imports relative (they're part of the bundle)
+			// BUT keep chunk imports relative (they're in the same directory structure and work with relative paths)
 			const convertRelativeFromImport = (match, quote, dots, rest) => {
-				if (rest.startsWith('./') || rest.startsWith('vs/') || rest.startsWith('chunk-')) {
+				// Skip if already absolute (starts with vs/)
+				if (rest.startsWith('vs/')) {
 					return match;
 				}
+				// Keep chunk imports relative - they're in the same directory structure
+				if (rest.startsWith('chunk-')) {
+					return match; // Keep relative path
+				}
+				// Skip relative imports starting with ./
+				if (rest.startsWith('./')) {
+					return match;
+				}
+				// Convert other relative imports to absolute paths
 				const resolved = resolveRelativeImport(`${dots}${rest}`);
 				if (resolved !== `${dots}${rest}`) {
 					return `from ${quote}${resolved}${quote}`;
@@ -203,9 +213,19 @@ function fixImportPaths() {
 			content = content.replace(
 				/import\s+(['"])((?:\.\.\/)+)([^'";\n]+)\1/g,
 				(match, quote, dots, rest) => {
-					if (rest.startsWith('./') || rest.startsWith('vs/') || rest.startsWith('chunk-')) {
+					// Skip if already absolute (starts with vs/)
+					if (rest.startsWith('vs/')) {
+						return match;
+					}
+					// Keep chunk imports relative - they're in the same directory structure
+					if (rest.startsWith('chunk-')) {
+						return match; // Keep relative path
+					}
+					// Skip relative imports starting with ./
+					if (rest.startsWith('./')) {
 						return match;
 					}
+					// Convert other relative imports to absolute paths
 					const resolved = resolveRelativeImport(`${dots}${rest}`);
 					if (resolved !== `${dots}${rest}`) {
 						return `import ${quote}${resolved}${quote}`;
@@ -214,22 +234,6 @@ function fixImportPaths() {
 				}
 			);
 
-			// Also fix any absolute chunk imports that were incorrectly converted
-			// Convert them back to relative imports
-			content = content.replace(
-				/from\s+(['"])vs\/workbench\/contrib\/cortexide\/browser\/react\/out\/(chunk-[^'"]+)\1/g,
-				(match, quote, chunkName) => {
-					// Get the directory of the current file relative to react/out/
-					const currentFileDir = path.dirname(fileRelPath);
-					// Chunks are in react/out/, entry points are in react/out/subdir/
-					if (currentFileDir === 'vs/workbench/contrib/cortexide/browser/react/out') {
-						return `from ${quote}./${chunkName}${quote}`;
-					}
-					// If in a subdirectory (e.g., void-settings-tsx/), use relative path
-					return `from ${quote}../${chunkName}${quote}`;
-				}
-			);
-
 			// Ensure browser-level modules always point to the browser directory
 			content = content
 				.replace(/vs\/workbench\/contrib\/cortexide\/actionIDs\.js/g, 'vs/workbench/contrib/cortexide/browser/actionIDs.js')
@@ -395,6 +399,7 @@ if (isWatch) {
 	execSync('npx tsup', { stdio: 'inherit' });
 
 	// Fix import paths in bundled files
+	// allow-any-unicode-next-line
 	console.log('🔧 Fixing import paths...');
 	fixImportPaths();