Skip to content

Commit c136af2

Browse files
author
Tajudeen
committed
Fix: Also handle require() calls when fixing deep relative imports
Add support for require() patterns in addition to import statements. When tsup/esbuild processes external imports, they might be transformed to require() calls, so we need to fix those paths too.
1 parent 2a120c8 commit c136af2

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

  • src/vs/workbench/contrib/cortexide/browser/react

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,7 @@ if (isWatch) {
243243
// This fixes imports like: import { x } from '../../../../../base/common/platform.js'
244244
// which come from files like systemInfo.ts that import VS Code internal modules
245245
// Match any relative import (../...) that leads to base/platform/editor/workbench and add one more level
246+
// Handle: import statements, dynamic imports, and require() calls
246247
content = content.replace(
247248
/(from\s+['\"])((?:\.\.\/)+)(base|platform|editor|workbench)\//g,
248249
(match, prefix, dots, module) => `${prefix}${dots}../${module}/`
@@ -251,6 +252,10 @@ if (isWatch) {
251252
/(import\s*\(\s*['\"])((?:\.\.\/)+)(base|platform|editor|workbench)\//g,
252253
(match, prefix, dots, module) => `${prefix}${dots}../${module}/`
253254
);
255+
content = content.replace(
256+
/(require\(['\"])((?:\.\.\/)+)(base|platform|editor|workbench)\//g,
257+
(match, prefix, dots, module) => `${prefix}${dots}../${module}/`
258+
);
254259

255260
fs.writeFileSync(indexPath, content, 'utf8');
256261
}

0 commit comments

Comments
 (0)