fix: allow 0 for integer values in askForOptionalInteger

Author: rossiam

.changeset/metal-terms-eat.md

@@ -0,0 +1,6 @@
+---
+"@smartthings/cli-lib": patch
+"@smartthings/cli": patch
+---
+
+fixed bug where prompts for optional integers were treating 0 as if no value was entered

packages/lib/src/__tests__/user-query.test.ts

@@ -203,6 +203,14 @@ describe('askForOptionalInteger', () => {
 		expect(promptMock).toHaveBeenCalledTimes(1)
 	})
 
+	it('returns "0" entered as 0', async () => {
+		promptMock.mockResolvedValue({ value: '0' })
+
+		expect(await askForOptionalInteger('prompt message')).toBe(0)
+
+		expect(promptMock).toHaveBeenCalledTimes(1)
+	})
+
 	it('passes validate to inquirer', async () => {
 		promptMock.mockResolvedValue({ value: '' })
 

packages/lib/src/user-query.ts

@@ -124,7 +124,7 @@ export const askForOptionalInteger = async (message: string, options?: AskForInt
 		...options,
 		validate: options?.validate ? allowEmptyFn(options.validate) : undefined,
 	}
-	return await promptForInteger(message, updatedOptions) || undefined
+	return await promptForInteger(message, updatedOptions)
 }
 
 /**