Dep: replace inquirer with existing node:readline for all interactive prompts

[sacOO7]

The project targets Node 22+ which has a mature node:readline module. Inquirer pulled in ~15-20 transitive
dependencies (rxjs, ansi-escapes, cli-cursor, mute-stream, etc.) adding supply chain surface and ~3-5 MB to
node_modules for features we don't need (password input, checkboxes, editor prompts). All prompt use cases
(confirmation + list selection) are well served by node:readline

Summary

• Remove inquirer, @inquirer/prompts, and @types/inquirer dependencies (~44 transitive packages eliminated)
• Add defaultValue parameter to promptForConfirmation() to preserve Enter-as-yes behavior for "Did you mean?"
  prompts
• Create promptForSelection() utility (src/utils/prompt-selection.ts) — numbered-list selector using
  node:readline, replacing inquirer's arrow-key list prompt
• Migrate InteractiveHelper.selectAccount/selectApp/selectKey to use promptForSelection()
• Migrate "Did you mean?" confirm prompts in base-topic-command.ts and did-you-mean.ts hook to
  promptForConfirmation()
• Delete dead code: prompt-utils.ts (entire file), InteractiveHelper.confirm(), readline-helper.ts
• Update dependabot config, docs, and AGENTS.md references

TODO

• - Check all of the existing commands that use removed inquirer dependency ( will mostly be switch commands )
• - Test each of those commands replaced with node:readline, check changes behaviour, make sure operational consistency is preserved.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
cli-web-cli	Ready	Preview, Comment	May 3, 2026 8:46am

[Copilot]

Pull request overview

This PR replaces inquirer-based interactive prompts with lightweight node:readline utilities to reduce dependency surface area while preserving existing “confirm” and “select from list” UX in the CLI.

Changes:

• Introduces promptForSelection() (numbered list prompt) and migrates InteractiveHelper selection flows to it.
• Extends promptForConfirmation() with a defaultValue parameter (and updates “Did you mean?” flows to default to “yes” on Enter).
• Removes now-dead inquirer/readline-restore helper code and updates tests/docs/config accordingly.

Reviewed changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/utils/prompt-selection.ts	New readline-based numbered selection prompt utility.
src/utils/prompt-confirmation.ts	Adds defaultValue behavior + suffix updates for confirmation prompts.
src/services/interactive-helper.ts	Migrates account/app/key selection to promptForSelection().
src/hooks/command_not_found/did-you-mean.ts	Uses promptForConfirmation(..., true) for interactive “Did you mean?” confirmation.
src/base-topic-command.ts	Uses promptForConfirmation(..., true) for “Did you mean?” confirmation on topic commands.
src/utils/readline-helper.ts	Deletes inquirer-specific readline restoration helper.
src/hooks/command_not_found/prompt-utils.ts	Deletes dead prompt helper utilities.
test/unit/utils/prompt-selection.test.ts	Adds unit tests for promptForSelection().
test/unit/utils/prompt-confirmation.test.ts	Adds tests for defaultValue + suffix behavior.
test/unit/services/interactive-helper.test.ts	Updates tests to mock promptForSelection() instead of inquirer.
test/unit/hooks/interactive-did-you-mean.test.ts	Updates tests to mock promptForConfirmation() and removes readline-restore assertions.
test/unit/commands/did-you-mean.test.ts	Updates expected prompt formatting from (Y/n) to [Y/n].
package.json	Removes inquirer, @inquirer/prompts, and @types/inquirer.
pnpm-lock.yaml	Lockfile updates after dependency removals.
docs/Interactive-REPL.md	Updates docs to reference node:readline over inquirer.
AGENTS.md	Updates guidance to use promptForConfirmation() for destructive confirmations.
.github/dependabot.yml	Removes inquirer-specific ignore rule.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 16 out of 17 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 16 out of 17 changed files in this pull request and generated no new comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

Comments suppressed due to low confidence (1)

src/utils/readline-helper.ts:48

• In strict TS mode, process.stdin.isRaw is typically typed as boolean | undefined, and setRawMode may not exist unless stdin is a TTY. As written, stdin.setRawMode(isRaw) can fail type-checking (and can throw at runtime if setRawMode is missing). Consider guarding with typeof stdin.setRawMode === "function" and passing a definite boolean (e.g., stdin.setRawMode(Boolean(isRaw)) or stdin.setRawMode(isRaw ?? false)).

    // Restore terminal settings
    if (stdin.isTTY) {
      stdin.setRawMode(isRaw);
    }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 17 out of 18 changed files in this pull request and generated 5 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 17 out of 18 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

Comments suppressed due to low confidence (1)

test/unit/hooks/interactive-did-you-mean.test.ts:67

• promptForConfirmation is a vi.fn() module mock, but afterEach only calls vi.restoreAllMocks(), which doesn't clear the call history of vi.fn mocks. This file reads vi.mocked(promptForConfirmation).mock.calls[0], so earlier calls from previous tests can shift indexes and make assertions unreliable. Clear/reset the mock in beforeEach/afterEach (e.g. vi.mocked(promptForConfirmation).mockClear() or vi.clearAllMocks() alongside restoring spies).

    // Reset promptForConfirmation mock to auto-confirm
    vi.mocked(promptForConfirmation).mockResolvedValue(true);
  });

  afterEach(function () {
    process.env = originalEnv;
    vi.restoreAllMocks();
  });

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.