Skip to content

Commit d0e7681

Browse files
committed
fix: fix handling of yargs fail callback to handle message argument
1 parent b293d15 commit d0e7681

2 files changed

Lines changed: 11 additions & 4 deletions

File tree

‎.changeset/thin-books-appear.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@smartthings/cli": patch
3+
---
4+
5+
Fixed error message when no command is given.

‎src/index.ts‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,11 @@ export const buildInstance = (commands: CommandModule<object, any>[]): Argv => {
3131
})
3232
/* eslint-enable @typescript-eslint/naming-convention */
3333
.completion('generate-completions-script', 'output completion script setup')
34-
.fail((message, error, yargs) => {
35-
if ('isAxiosError' in error && error.isAxiosError) {
34+
.fail((message, error) => {
35+
if (message) {
36+
console.error(message)
37+
console.error('Run with --help for usage details.')
38+
} else if (error && 'isAxiosError' in error && error.isAxiosError) {
3639
// We don't print axiosError.message here because it just duplicates the things
3740
// we're displaying but unformatted.
3841
const axiosError = error as AxiosError
@@ -43,8 +46,7 @@ export const buildInstance = (commands: CommandModule<object, any>[]): Argv => {
4346
} else if (error) {
4447
console.error(error.message)
4548
} else {
46-
console.error(message)
47-
console.error(yargs.help())
49+
console.error('An unknown error occurred.')
4850
}
4951

5052
// eslint-disable-next-line no-process-exit

0 commit comments

Comments
 (0)