TypeError: format is not a function

[reintroducing]

Expected Behavior

I'm trying to commit in a new repository that uses husky and my commit should work and pass husky checks as expected.

Current Behavior

When I try to commit, I see the following:

husky > commit-msg (node v10.3.0)
/Users/mprzybylski/Work/Livongo/fe-utils/node_modules/@commitlint/cli/lib/cli.js:113
	throw err;
	^

TypeError: format is not a function
    at /Users/mprzybylski/Work/Livongo/fe-utils/node_modules/@commitlint/cli/lib/cli.js:193:18
    at run (/Users/mprzybylski/Work/Livongo/fe-utils/node_modules/babel-polyfill/node_modules/core-js/modules/es6.promise.js:75:22)
    at /Users/mprzybylski/Work/Livongo/fe-utils/node_modules/babel-polyfill/node_modules/core-js/modules/es6.promise.js:92:30
    at flush (/Users/mprzybylski/Work/Livongo/fe-utils/node_modules/babel-polyfill/node_modules/core-js/modules/_microtask.js:18:9)
    at process._tickCallback (internal/process/next_tick.js:61:11)
husky > commit-msg hook failed (add --no-verify to bypass)

Affected packages

• cli
• core
• prompt
• config-angular

Possible Solution

I'm not sure, and the odd thing is this works in my pre-existing repos, but it fails in this new one.

Steps to Reproduce (for bugs)

1. Make changes to files.
2. Commit changes. Error.

I've created a small repo that reproduces the problem here: https://github.com/reintroducing/commit-test

commitlint.config.js

```js module.exports = { extends: ['@spothero/commitlint-config'] }; ```

Context

I can't commit to the new repo unless i bypass husky checks. I'm using a custom commitlint config which can be found here: https://github.com/spothero/commitlint-config

Your Environment

Executable	Version
commitlint --version	7.5.2
git --version	2.20.1 (Apple Git-117)
node --version	10.3.0

[aaryn101]

I am seeing the same thing. It only appeared after upgrading to husky@2.2.0.

[reintroducing]

for what its worth, my commitlint setup is pinned to husky@1.3.1.

[reintroducing]

I went ahead and added a link to a repo I just created which reproduces the problem: https://github.com/reintroducing/commit-test

[mattwills8]

Also seeing it, started today after package updates!

Using VSCode with prettier and eslint

[marionebl]

Thanks for reporting, I'll look into this.

[marionebl]

Could reproduce - this is caused by a mismatch between the @commitlint/format (7.6.0) and @commitlint/cli (7.5.1) package versions. Could you install the latest cli and check if this fixes the issue for you?

npm install @commitlint/cli@latest

[keenwon]

@marionebl @commitlint/core exported the wrong format?

[mattwills8]

@marionebl worked for me, thanks!!

[reintroducing]

@marionebl that did indeed fix it, thank you, appreciate the quick turn around on this.

[varl]

I'm importing from @commitlint/format and I have the same problem.

The docs for @commitlint/format are no longer correct.

Test code for @commitlint/format:

const format = require('@commitlint/format')
console.log('format', typeof format)
format({})

Result: 7.5.0

format function

Result: 7.6.0

format object
// { default: [Function: format],
//  formatResult: [Function: formatResult] }
[ERROR] TypeError: format is not a function

Test code for @commitlint/core:

const { read, load, lint, format } = require('@commitlint/core')

console.log('format', typeof format)
console.log('load', typeof load)
console.log('read', typeof read)
console.log('lint', typeof lint)

Result: 7.5.0

format function
load function
read function
lint function

Result: 7.6.0

format object
load function
read function
lint function
[ERROR] TypeError: format is not a function

To me this is actually a breaking change for @commitlint/format and @commitlint/core since the API has changed in a non-backwards compatible way.

Is this intentional and going forward we should import from require('@commitlint/format').default?

[marionebl]

To me this is actually a breaking change for @commitlint/format and @commitlint/core since the API has changed in a non-backwards compatible way.

Thanks for the analysis - you are right about the accidental breaking change. I'm in the process of creating a patch release that addresses this.

Is this intentional and going forward we should import from require('@commitlint/format').default?

No, when 7.6.1 and (later) 8.0.0 are released you'll be able to require the default export of @commitlint/format directly.

[varl]

@marionebl thanks for your response, and effort in fixing it. 👍