Skip to content

feat(cli): add explanations to migration prompts#1270

Merged
fengmk2 merged 7 commits intovoidzero-dev:mainfrom
hakshu25:feat/migrate-prompt-explanations
Apr 2, 2026
Merged

feat(cli): add explanations to migration prompts#1270
fengmk2 merged 7 commits intovoidzero-dev:mainfrom
hakshu25:feat/migrate-prompt-explanations

Conversation

@hakshu25
Copy link
Copy Markdown
Contributor

@hakshu25 hakshu25 commented Apr 2, 2026

Summary

Closes #1030

When running vp migrate interactively, users are prompted with choices (which agents to use, whether to merge editor configs, whether to migrate ESLint/Prettier, etc.) but are given no context for why. This PR adds a brief gray explanation embedded directly in each prompt message so users understand what they're agreeing to before making a choice.

Demo

スクリーンショット 2026-04-02 17 05 23

Test plan

  • vp check --fix passes
  • pnpm bootstrap-cli && pnpm test passes (snap tests unaffected — all new messages are inside interactive guards)
  • GPG-signed commits

🤖 Generated with Claude Code

Add prompts.log.info() calls before interactive prompts in the migration
flow to explain why each option is being presented to the user.

- Agent selection: explain what agent instruction files are
- Agent conflict (append/skip): explain template contents
- Editor selection: explain what editor configs will be written
- Editor conflict (merge/skip): explain merge behavior
- ESLint migration: explain what Oxlint is and what the migration does
- Prettier migration: explain what Oxfmt is and what the migration does

Closes voidzero-dev#1030

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 2, 2026

Deploy Preview for viteplus-preview ready!

Name Link
🔨 Latest commit 4a66e34
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69ce71f035adad0008a2db28
😎 Deploy Preview https://deploy-preview-1270--viteplus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hakshu25 hakshu25 changed the title feat(migrate): add explanations to migration prompts feat(cli): add explanations to migration prompts Apr 2, 2026
hakshu25 and others added 2 commits April 2, 2026 16:48
Replace standalone prompts.log.info() calls with inline gray subtext
embedded in the prompt message itself via styleText('gray', ...).

This keeps the explanation visually tied to its associated prompt
instead of appearing as a detached message before it.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The previous message only mentioned CLAUDE.md and AGENTS.md, but
each agent has its own file (GEMINI.md, etc.). Use a generic
description that covers all agents.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@fengmk2 fengmk2 requested a review from TheAlexLichter April 2, 2026 08:00
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hakshu25 hakshu25 marked this pull request as ready for review April 2, 2026 08:11
Copy link
Copy Markdown
Member

@TheAlexLichter TheAlexLichter left a comment

Choose a reason for hiding this comment

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

Good start 👍🏻

hakshu25 and others added 2 commits April 2, 2026 21:18
Some editors don't have recommendations — 'Other' is more accurate
than 'None' for this case.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@fengmk2
Copy link
Copy Markdown
Member

fengmk2 commented Apr 2, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@fengmk2 fengmk2 merged commit 0d24eb9 into voidzero-dev:main Apr 2, 2026
32 checks passed
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.

Add explanation when configs are merged/migrated/...

3 participants