Skip to content

fix(cli): align local help with upstream tools - #2345

Merged
fengmk2 merged 5 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/sync-local-tool-help
Aug 6, 2026
Merged

fix(cli): align local help with upstream tools#2345
fengmk2 merged 5 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/sync-local-tool-help

Conversation

@liangmiQwQ

@liangmiQwQ liangmiQwQ commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Refs: #2330 discussion

Vite+ owns only top-level help for tool-backed commands. Exact vp <command> --help and vp <command> -h requests use the local themed help, while requests with additional arguments, such as vp test --help --coverage or vp test list --help, delegate to the bundled tool. This keeps deep and subcommand help complete without mirroring every upstream help level.

The local CLI maintains its own top-level help entries for commands backed by Vite, Vitest, Oxlint, Oxfmt, and tsdown. Some entries paraphrased or shortened the upstream help, so option types, defaults, caveats, and lint-category guidance drifted from the bundled tool versions.

This PR updates those entries to match the upstream wording and casing. It also wraps long descriptions to the available terminal width while keeping option columns aligned, so the fuller text remains readable.

It does not add or remove options or change non-help command behavior.

🤖 Generated with Codex

@liangmiQwQ
liangmiQwQ changed the base branch from deps/upstream-update to main August 5, 2026 13:04
@liangmiQwQ
liangmiQwQ marked this pull request as ready for review August 5, 2026 13:10
@liangmiQwQ
liangmiQwQ marked this pull request as draft August 5, 2026 13:10
@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview ready!

Name Link
🔨 Latest commit 18d7854
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a7435f48d5aac0008a1afa4
😎 Deploy Preview https://deploy-preview-2345--viteplus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@liangmiQwQ
liangmiQwQ marked this pull request as ready for review August 5, 2026 14:00
@liangmiQwQ
liangmiQwQ requested a review from fengmk2 August 5, 2026 14:00
@liangmiQwQ

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5a1d89ab82

ℹ️ 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".

Comment thread packages/cli/src/help.ts
Comment thread packages/cli/src/help.ts
Comment thread packages/cli/src/help.ts Outdated
@liangmiQwQ
liangmiQwQ marked this pull request as draft August 5, 2026 14:18
@liangmiQwQ
liangmiQwQ removed the request for review from fengmk2 August 5, 2026 14:18
@liangmiQwQ
liangmiQwQ force-pushed the liang/codex/sync-local-tool-help branch from 35f9dd3 to fef0ed0 Compare August 6, 2026 02:59
@liangmiQwQ
liangmiQwQ marked this pull request as ready for review August 6, 2026 05:57
@liangmiQwQ
liangmiQwQ requested a review from fengmk2 August 6, 2026 06:03
@fengmk2
fengmk2 merged commit ae5f27e into voidzero-dev:main Aug 6, 2026
45 checks passed
@liangmiQwQ
liangmiQwQ deleted the liang/codex/sync-local-tool-help branch August 6, 2026 07:57
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