+5

.changeset/password-empty-submit.md

Reviewed

··· 1 1 + --- 2 2 + "@clack/core": patch 3 3 + --- 4 4 + 5 5 + Fix `password` prompt resolving to `undefined` on empty submit. It now normalizes an empty submission to `""`, matching the `text` prompt behavior and the documented `Promise<string | symbol>` return type.

+5

packages/core/src/prompts/password.ts

Reviewed

··· 34 34 this.on('userInput', (input) => { 35 35 this._setValue(input); 36 36 }); 37 37 + this.on('finalize', () => { 38 38 + if (this.value === undefined) { 39 39 + this.value = ''; 40 40 + } 41 41 + }); 37 42 } 38 43 }

+12

packages/core/test/prompts/password.test.ts

Reviewed

··· 29 29 expect(output.buffer).to.deep.equal([cursor.hide, 'foo']); 30 30 }); 31 31 32 32 + test('returns empty string on empty submit', async () => { 33 33 + const instance = new PasswordPrompt({ 34 34 + input, 35 35 + output, 36 36 + render: () => 'foo', 37 37 + }); 38 38 + const resultPromise = instance.prompt(); 39 39 + input.emit('keypress', '', { name: 'return' }); 40 40 + const result = await resultPromise; 41 41 + expect(result).to.equal(''); 42 42 + }); 43 43 + 32 44 describe('cursor', () => { 33 45 test('can get cursor', () => { 34 46 const instance = new PasswordPrompt({