npx husky init generates invalid pre-commit script

[dexmlee]

npx husky init is generating this invalid pre-commit script:

npm

See bin.js:
w('.husky/pre-commit', p.env.npm_config_user_agent?.split('/')[0] ?? 'npm' + ' test\n')

So possible values are either:

• npm
• npm test

On my mac:
p.env.npm_config_user_agent == 'npm/10.2.4 node/v20.11.1 darwin arm64 workspaces/false'
Order of operations error: + is higher than null coalesce. Looks like ?? was introduced in v9.1.0. Needs parens.

Easy problem to fix on new scripts, but confuses people following the documentation.

Troubleshoot

• Before creating an issue, please check:
  https://typicode.github.io/husky/troubleshoot.html

If you're migrating from husky 4, see:
https://typicode.github.io/husky/migrate-from-v4.html

Context
Please describe your issue and provide some context:

• Terminal or GUI client (PowerShell, Git Bash, GitHub Desktop, ...)
• If applicable, content of the failing hook
• If possible, minimal steps to reproduce the issue

Thank you!

[typicode]

Thanks for spotting it! Feel free to make the PR if you want. Otherwise let me know, I'll do it.

[dexmlee]

@typicode PR sent.

[Munaf-Divan]

@typicode, out of curiosity, do we have a system in place where we label the issue, or it occurs automatically when a PR is created?