Skip to content

Commit

Permalink
Merge pull request #24018 from storybookjs/yann/string-width-issue
Browse files Browse the repository at this point in the history
CLI: Add uncaughtException handler
(cherry picked from commit a66b8b2)
  • Loading branch information
yannbf authored and storybook-bot committed Sep 8, 2023
1 parent 6ae602c commit b9b940a
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions code/lib/cli/bin/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,21 @@ if (majorNodeVersion < 16) {
process.exit(1);
}

// The Storybook CLI has a catch block for all of its commands, but if an error
// occurs before the command even runs, for instance, if an import fails, then
// such error will fall under the uncaughtException handler.
// This is the earliest moment we can catch such errors.
process.once('uncaughtException', (error) => {
if (error.message.includes('string-width')) {
console.error(
[
'🔴 Error: It looks like you are having a known issue with package hoisting.',
'Please check the following issue for details and solutions: https://github.com/storybookjs/storybook/issues/22431#issuecomment-1630086092\n\n',
].join('\n')
);
}

throw error;
});

require('../dist/generate.js');

0 comments on commit b9b940a

Please sign in to comment.