Skip to content

Drop @inquirer/prompts#61

Merged
4nkur merged 1 commit intomainfrom
drop-inquirer
Apr 29, 2026
Merged

Drop @inquirer/prompts#61
4nkur merged 1 commit intomainfrom
drop-inquirer

Conversation

@4nkur
Copy link
Copy Markdown
Owner

@4nkur 4nkur commented Apr 29, 2026

Summary

Removes Draftwise's last remaining stdin dependency. With api mode gone in #60, the four interactive prompts (init's idea input, tech/tasks' spec picker, scaffold's confirm) had nothing meaningful to gate — the CLI's job is to load context and hand off to the host coding agent.

  • tech / tasks with multiple specs and no slug → always error with the available list
  • scaffold requires --yes to confirm before writing
  • init greenfield without --idea → always prints the structured agent handoff (a plain-terminal user reads it as a usage hint)
  • TTY/non-TTY distinction gone: src/utils/tty.js, test/setup.js, and vitest.config.js deleted; deps.isInteractive is no longer a DI seam
  • @inquirer/prompts removed from package.json; lockfile regenerated

Test plan

  • npm test — 222 passing
  • npm run lint — clean
  • CLAUDE.md, README.md, CHANGELOG.md, plugin reference files updated to drop the "TTY-only fallback" framing

Removes the four remaining interactive prompts: init's idea input, tech/tasks' spec picker, and scaffold's confirm. Each was a TTY-only convenience layer on top of flags that already drove the canonical input path. With api mode gone, those prompts had nothing meaningful to gate — the CLI's job is to load context and hand off to the host coding agent, not run a Q&A loop.

Concrete behavior changes: tech / tasks with multiple specs and no slug arg now error with the available list (was: TTY → inquirer picker, non-TTY → error); scaffold requires --yes to confirm before writing (was: TTY → inquirer confirm, non-TTY → error); init greenfield without --idea always prints the structured agent handoff (was: TTY → inquirer input, non-TTY → handoff). The TTY/non-TTY distinction is gone entirely — src/utils/tty.js, test/setup.js, and vitest.config.js are deleted; deps.isInteractive is no longer a dependency-injection seam.

CLAUDE.md and README.md updated to drop "TTY-only fallback" framing. 222 tests passing; lint clean.
@4nkur 4nkur merged commit 56cb499 into main Apr 29, 2026
2 checks passed
@4nkur 4nkur deleted the drop-inquirer branch April 29, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant