fix(@inquirer/confirm): Fix when using with unix yes command. (#1815)

Fixes #365

Author: SBoudrias

.github/workflows/main.yml

@@ -24,7 +24,7 @@ jobs:
       - run: corepack enable
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: latest
           cache: 'yarn'
       - run: yarn install --immutable --immutable-cache
       - name: Typescript - packages
@@ -46,7 +46,7 @@ jobs:
       - run: corepack enable
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: latest
           cache: 'yarn'
       - run: yarn install --immutable --immutable-cache
       - run: yarn tsc
@@ -65,9 +65,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version:
-          - 20
-          - 18
+        node-version: [18, 20, 22]
     steps:
       - uses: actions/checkout@v4
       - run: corepack enable
@@ -83,12 +81,6 @@ jobs:
     name: Integration Tests
     runs-on: ubuntu-latest
     needs: Compile
-    strategy:
-      fail-fast: false
-      matrix:
-        node-version:
-          - 20
-          - 18
     steps:
       - uses: actions/checkout@v4
       - name: Cache turbo build setup
@@ -100,7 +92,7 @@ jobs:
       - run: corepack enable
       - uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: latest
           cache: 'yarn'
       - run: yarn install --immutable --immutable-cache
       - run: yarn tsc
@@ -124,7 +116,7 @@ jobs:
       - run: corepack enable
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: latest
           cache: 'yarn'
       - run: yarn install --immutable --immutable-cache
       - run: yarn tsc
@@ -168,7 +160,7 @@ jobs:
       - run: corepack enable
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: latest
           cache: 'yarn'
       - run: yarn install --immutable --immutable-cache
       - run: yarn turbo attw

integration/esm/integration.test.mjs

@@ -1,13 +1,16 @@
-import { createRequire } from 'node:module';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
+import child_process from 'node:child_process';
+import { writeFileSync, unlinkSync } from 'node:fs';
+import path from 'node:path';
+import { promisify } from 'node:util';
 import { input } from '@inquirer/prompts';
 import defaultInput from '@inquirer/input';
 import { createPrompt } from '@inquirer/core';
 import inquirer, { createPromptModule } from 'inquirer';
 import fixturePrompt from './fixturePrompt.mjs';
 
-const require = createRequire(import.meta.url);
+const exec = promisify(child_process.exec);
 
 describe('ESM Integration', () => {
   it('@inquirer/prompts should be exported', () => {
@@ -24,7 +27,7 @@ describe('ESM Integration', () => {
 
   it('works when prompt throws an error', async () => {
     await assert.rejects(() => fixturePrompt({}), {
-      message: `Prompt functions must return a string.\n    at file://${require.resolve('./fixturePrompt.mjs')}`,
+      message: `Prompt functions must return a string.\n    at file://${path.join(import.meta.dirname, './fixturePrompt.mjs')}`,
     });
   });
 
@@ -33,4 +36,36 @@ describe('ESM Integration', () => {
     assert.ok(typeof inquirer.createPromptModule === 'function');
     assert.ok(typeof createPromptModule === 'function');
   });
+
+  it('works with Unix yes command piped input', async () => {
+    try {
+      await exec('which yes');
+    } catch {
+      assert.ok(true);
+      console.warn('WARN: Skipping test due to absence of `yes` in the PATH');
+    }
+
+    const testScript = path.join(import.meta.dirname, 'test-yes-pipe.js');
+    writeFileSync(
+      testScript,
+      `
+          import { confirm } from '@inquirer/prompts';
+
+          const answer = await confirm({
+            message: 'Do you want to proceed?'
+          });
+
+          process.exit(answer ? 0 : 1);
+        `,
+    );
+
+    try {
+      await exec(`yes | node ${testScript} > /dev/null`);
+      assert.ok(true);
+    } catch {
+      assert.fail('Command thew');
+    } finally {
+      unlinkSync(testScript);
+    }
+  }, 10000);
 });

packages/confirm/src/index.ts

@@ -36,6 +36,8 @@ export default createPrompt<boolean, ConfirmConfig>((config, done) => {
   const prefix = usePrefix({ status, theme });
 
   useKeypress((key, rl) => {
+    if (status !== 'idle') return;
+
     if (isEnterKey(key)) {
       const answer = getBooleanValue(value, config.default);
       setValue(transformer(answer));