Skip to content

feat(core): add accessible prompt option - #595

Open
dreyfus92 wants to merge 1 commit into
mainfrom
feat/accessible-thread
Open

feat(core): add accessible prompt option#595
dreyfus92 wants to merge 1 commit into
mainfrom
feat/accessible-thread

Conversation

@dreyfus92

Copy link
Copy Markdown
Member

What does this PR do?

this threads the accessible option through the shared prompt options so every prompt can opt into accessible mode individually.

tests cover the resolution order from the prompt's side.

related: #585

Type of change

  • Bug fix
  • Feature
  • Refactor (no behavior change)
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • pnpm test passes (or targeted tests for my change)
  • pnpm format has been run
  • I have added/updated tests for my changes (if applicable)
  • I have added a changeset

AI-generated code disclosure

  • This PR includes AI-generated code

@changeset-bot

changeset-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4aa3b28

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@clack/core Minor
@clack/prompts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Aug 13, 2026

Copy link
Copy Markdown

commit: 4aa3b28

@dreyfus92
dreyfus92 requested a review from 43081j August 13, 2026 22:47
export type ClackState = 'initial' | 'active' | 'cancel' | 'submit' | 'error';

/**
* Options shared by all Clack prompts

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't this just PromptOptions? maybe we don't need to extract it into its own interface. PromptOptions is already a base

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.

2 participants