Skip to content

chore: Fork @clack/prompts.#587

Merged
cpojer merged 2 commits intomainfrom
prompts
Feb 10, 2026
Merged

chore: Fork @clack/prompts.#587
cpojer merged 2 commits intomainfrom
prompts

Conversation

@cpojer
Copy link
Copy Markdown
Member

@cpojer cpojer commented Feb 10, 2026

This PR forks @clack/prompts and replaces cyan with blueBright for interactive commands. It gives Vite+ a bit more identity, and we can customize things as we like.

This is lightweight because it still uses @clack/core under the hood. Unfortunately @clack/prompts doesn't allow any customization at all.

Fixed lint errors in the prompts folder but left everything else untouched.

Copy link
Copy Markdown
Member Author

cpojer commented Feb 10, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cpojer cpojer marked this pull request as ready for review February 10, 2026 08:56
@cpojer cpojer force-pushed the prompts branch 2 times, most recently from ff0b8a9 to b8c3828 Compare February 10, 2026 09:02
Comment thread package.json Outdated
@cpojer cpojer force-pushed the prompts branch 2 times, most recently from 688f893 to 5aa689b Compare February 10, 2026 09:30
@cpojer cpojer force-pushed the prompts branch 2 times, most recently from ad66fa2 to 2838472 Compare February 10, 2026 11:30
@cpojer cpojer merged commit 7328beb into main Feb 10, 2026
17 checks passed
@cpojer cpojer deleted the prompts branch February 10, 2026 16:15
@cpojer cpojer mentioned this pull request Feb 10, 2026
cpojer added a commit that referenced this pull request Feb 11, 2026
All the e2e tests started failing after merging #587, see https://github.com/voidzero-dev/vite-plus/actions/runs/21872859706/job/63134846956

I *believe* this is due to an ordering issue and it's why I added the explicit first build step earlier, but I removed it. This change ensures the prompts package is built before the global CLI.

Please let me know if there is a quick way to verify this locally since the usual tests all seem to pass without this change.
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