Merge pull request #106 from CodinGame/webpack-loader-windows

Fix webpack-loader on Windows
CodinGame · Apr 22, 2023 · f94aaa3 · f94aaa3
2 parents 6a7bdc8 + 2c8b376
commit f94aaa3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/override/vs/editor/contrib/suggest/browser/suggest.js b/src/override/vs/editor/contrib/suggest/browser/suggest.js
@@ -14,7 +14,7 @@ It's treeshaked out of monaco editor and needs to be reintroduced.
 If you're using webpack, you can add a loader that will polyfill it:
 \`\`\`
 {
-  test: /node_modules\\/monaco-editor\\//,
+  test: /node_modules[\\/]monaco-editor/,
   loader: 'vscode/webpack-loader'
 }
 \`\`\`

diff --git a/src/webpack-loader.ts b/src/webpack-loader.ts
@@ -1,7 +1,8 @@
 import type { RawLoaderDefinitionFunction } from 'webpack'
+import { sep as s } from 'path'
 
 const monacoPolyfillLoader: RawLoaderDefinitionFunction = function (source) {
-  if (this.resourcePath.endsWith('monaco-editor/esm/vs/editor/contrib/suggest/browser/suggest.js')) {
+  if (this.resourcePath.endsWith(`monaco-editor${s}esm${s}vs${s}editor${s}contrib${s}suggest${s}browser${s}suggest.js`)) {
     return `${source.toString()}
 export function setSnippetSuggestSupport(support) {
   const old = _snippetSuggestSupport;