Skip to content

Commit

Permalink
fix(template-webpack): only run asset relocator loader on `node_modul…
Browse files Browse the repository at this point in the history
…es` folder (#3060)


Co-authored-by: Erick Zhao <erick@hotmail.ca>
  • Loading branch information
MarshallOfSound and erickzhao committed Nov 14, 2022
1 parent f16a28b commit f86eb28
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 5 deletions.
@@ -1,6 +1,13 @@
module.exports = [
// Add support for native node modules
{
test: /\.(m?js|node)$/,
// We're specifying native_modules in the test because the asset relocator loader generates a
// "fake" .node file which is really a cjs file.
test: /native_modules[/\\].+\.node$/,
use: 'node-loader',
},
{
test: /[/\\]node_modules[/\\].+\.(m?js|node)$/,
parser: { amd: false },
use: {
loader: '@vercel/webpack-asset-relocator-loader',
Expand Down
4 changes: 2 additions & 2 deletions packages/template/webpack-typescript/tmpl/webpack.rules.ts
Expand Up @@ -5,11 +5,11 @@ export const rules: Required<ModuleOptions>['rules'] = [
{
// We're specifying native_modules in the test because the asset relocator loader generates a
// "fake" .node file which is really a cjs file.
test: /native_modules\/.+\.node$/,
test: /native_modules[/\\].+\.node$/,
use: 'node-loader',
},
{
test: /\.(m?js|node)$/,
test: /[/\\]node_modules[/\\].+\.(m?js|node)$/,
parser: { amd: false },
use: {
loader: '@vercel/webpack-asset-relocator-loader',
Expand Down
4 changes: 2 additions & 2 deletions packages/template/webpack/tmpl/webpack.rules.js
Expand Up @@ -3,11 +3,11 @@ module.exports = [
{
// We're specifying native_modules in the test because the asset relocator loader generates a
// "fake" .node file which is really a cjs file.
test: /native_modules\/.+\.node$/,
test: /native_modules[/\\].+\.node$/,
use: 'node-loader',
},
{
test: /\.(m?js|node)$/,
test: /[/\\]node_modules[/\\].+\.(m?js|node)$/,
parser: { amd: false },
use: {
loader: '@vercel/webpack-asset-relocator-loader',
Expand Down

0 comments on commit f86eb28

Please sign in to comment.