Option 'bail' causes failed build

[meskill]

Bug report

What is the current behavior?

Setting option bail to true will break the build for unresolved modules wrapped in try...catch
After setting bail to false bundle builds successfully

If the current behavior is a bug, please provide the steps to reproduce.

1. Set bail: true in config
2. Add require to unresolved module wrapped in try...catch

   let packageJson = {};
   
   try {
     packageJson = require('../../../../package.json');
   } catch (e) {
   }
   

3. Start build as always

What is the expected behavior?

There is should be only a warning about the unresolved module, bundle should be built successfully.

Other relevant information:
webpack version: 5.0.0-rc.3
Node.js version: 12.18.4

[alexander-akait]

Unresolved module is an error, try to run this code in node wrong, it is a warning in try/catch

[meskill]

@evilebottnawi, please, look carefully
require is wrapped in try..catch so error for an unresolved module is handled, and it works fine without the option

[alexander-akait]

Oh, I was wrong here, it is bug on webpack-cli, please use latest rc release of webpack-cli

[alexander-akait]

From my test:

WARNING in ./test.js 4:18-53
Module not found: Error: Can't resolve '../../../../package.json' in '/home/user/path/to/package'

# echo $?
0

[meskill]

@evilebottnawi I've created a test repo to show this issue and the difference between webpack4 and webpack5
https://github.com/meskill/webpack5-bail-option-repro

[meskill]

is it a bug for sure?

shouldn't webpack only care if the required source is not found while the bundler is running even though the require is inside a try catch.

and not if a require is resolving in runtime because of the try catch.

I think those cases should be handled by setting the externals property in config?!?
https://webpack.js.org/configuration/externals/

this bug describes only bundle-time case.
we have a tool to wrap a webpack execution from our users and thats why we may face case when user app doesn't have some dependencies in their modules tree.
And it works as expected, except for this bug, when option bail: true leads to unsuccessful build

[webpack-bot]

This issue had no activity for at least three months.

It's subject to automatic issue closing if there is no activity in the next 15 days.

[alexander-akait]

bump

[webpack-bot]

Issue was closed because of inactivity.

If you think this is still a valid issue, please file a new issue with additional information.