Skip to content

Commit

Permalink
Simplify Webpack dependencies for Web Worker scenario.
Browse files Browse the repository at this point in the history
  • Loading branch information
DavidAnson committed Nov 18, 2023
1 parent 684f0f1 commit 200f711
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 21 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
"test-watch": "git ls-files | entr npm run test",
"update-snapshots": "ava --timeout=1m --update-snapshots test/markdownlint-cli2-test-exec.js test/markdownlint-cli2-test-fs.js test/markdownlint-cli2-test-main.js",
"webworker": "cd webworker && webpack --mode none",
"webworker-install": "npm install --no-package-lock --no-save path-browserify process setimmediate stream-browserify url util webpack-cli && cpy ./node_modules/setimmediate/setImmediate.js ./webworker --flat"
"webworker-install": "npm install --no-package-lock --no-save path-browserify setimmediate stream-browserify util webpack-cli && cpy ./node_modules/setimmediate/setImmediate.js ./webworker --flat"
},
"engines": {
"node": ">=18"
Expand Down
11 changes: 11 additions & 0 deletions webworker/process-stub.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
// @ts-check

"use strict";

module.exports = {
"cwd": () => "/",
"env": {},
"versions": {
"node": "0.0"
}
};
10 changes: 0 additions & 10 deletions webworker/process-wrapper.js

This file was deleted.

4 changes: 1 addition & 3 deletions webworker/url-stub.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,4 @@

"use strict";

module.exports = {
"fileURLToPath": () => null
};
module.exports = {};
14 changes: 7 additions & 7 deletions webworker/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,22 +19,22 @@ module.exports = {
}
},
"plugins": [
// Rewrite requires to remove "node:" prefix
new webpack.NormalModuleReplacementPlugin(
nodeModulePrefixRe,
(resource) => {
let module = resource.request.replace(nodeModulePrefixRe, "");
if (module === "url") {
module = "url-stub";
}
const module = resource.request.replace(nodeModulePrefixRe, "");
resource.request = module;
}
),
// Intercept existing "unicorn-magic" package to provide missing import
new webpack.NormalModuleReplacementPlugin(
/^unicorn-magic$/u,
(resource) => {
resource.request = require.resolve("./unicorn-magic-stub.js");
}
),
// Intercept use of "process" to provide implementation
new webpack.ProvidePlugin({
"process": "process-wrapper"
})
Expand All @@ -45,10 +45,10 @@ module.exports = {
"fs": false,
"os": require.resolve("./os-stub.js"),
"path": require.resolve("path-browserify"),
"process": require.resolve("./process-wrapper.js"),
"process-wrapper": require.resolve("./process-wrapper.js"),
"process": require.resolve("./process-stub.js"),
"process-wrapper": require.resolve("./process-stub.js"),
"stream": require.resolve("stream-browserify"),
"url-stub": require.resolve("./url-stub.js")
"url": require.resolve("./url-stub.js")
}
}
};

0 comments on commit 200f711

Please sign in to comment.