fix(webpack): sourceMap option must be a boolean if/when passed to loaders

[janeklb]

Current Behavior

When using sourceMap: 'hidden' as an nx webpack config option you see errors from loaders

eg.

Error: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
ValidationError: Invalid options object. Sass Loader has been initialized using an options object that does not match the API schema.
 - options.sourceMap should be a boolean.
   -> Enables/Disables generation of source maps.
   -> Read more at https://github.com/webpack-contrib/sass-loader#sourcemap

Expected Behavior

No errors when specifying 'hidden' as the sourceMap config.

Note, all modified loader configs expect a boolean:

• https://www.npmjs.com/package/sass-loader#sourcemap
• https://www.npmjs.com/package/less-loader#sourcemap
• https://www.npmjs.com/package/stylus-loader#sourcemap

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated
nx-dev	⬜️ Ignored (Inspect)			Jan 25, 2023 at 9:58PM (UTC)

[janeklb]

This solution is already being used below

nx/packages/webpack/src/utils/with-web.ts

Line 424 in c134efe

sourceMap: !!options.sourceMap,

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 8952b27. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx run-many -t e2e-macos --parallel=1 --exclude=e2e-detox
• nx affected --target=e2e --base=a7485d3ecd3c5db3f296139ca5190da807885dc1 --head=8952b27060f76531813b1d2962cbdc01e6546660 --exclude=e2e-storybook,e2e-storybook-angular --parallel=1
• nx run-many -t root-lint check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=a7485d3ecd3c5db3f296139ca5190da807885dc1 --head=8952b27060f76531813b1d2962cbdc01e6546660 --parallel=3
• nx affected --target=build --base=a7485d3ecd3c5db3f296139ca5190da807885dc1 --head=8952b27060f76531813b1d2962cbdc01e6546660 --parallel=3
• nx affected --target=test --base=a7485d3ecd3c5db3f296139ca5190da807885dc1 --head=8952b27060f76531813b1d2962cbdc01e6546660 --parallel=1
• nx-cloud record -- nx format:check --base=a7485d3ecd3c5db3f296139ca5190da807885dc1 --head=8952b27060f76531813b1d2962cbdc01e6546660
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.