fix: only show error detail when error.original is defined

[Rochet2]

Pull Request check-list

Please make sure to review and check all of these items:

• Does npm run test pass with this change (including linting)?
• Does the description below contain a link to an existing issue (Closes #[issue]) or a description of the issue you are solving?
• Have you added new tests to prevent regressions?
• Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change

Using "sequelize-cli": "6.5.2",

When up handler of a migration throws a custom error.

throw new Error(
  `foobar`,
);

The cli throws the following error

TypeError: Cannot read properties of undefined (reading 'detail')
    at Object.error (/myproject/node_modules/sequelize-cli/lib/helpers/view-helper.js:31:24)
    at /myproject/node_modules/sequelize-cli/lib/commands/migrate.js:68:39
    at async exports.handler (/myproject/node_modules/sequelize-cli/lib/commands/migrate.js:27:7)

The error originates from this line:

    if (error.original.detail) { // this causes the error because original is null/undefined
      console.error(`${_cliColor.default.red('ERROR DETAIL:')} ${error.original.detail}`);
    }

In this PR we change the IF condition to check if the original field is set so the error will not occur.

[WikiRik]

Thanks for this PR!

[WikiRik]

Sorry, approved too quickly. We still support Node 10 which does not have the optional chaining

[WikiRik]

Thanks for the quick update!

[github-actions]

🎉 This PR is included in version 6.6.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀