Skip to content

ci(deps): handle upstream help diffs - #2330

Open
liangmiQwQ wants to merge 9 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/report-upstream-help-diffs
Open

ci(deps): handle upstream help diffs#2330
liangmiQwQ wants to merge 9 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/report-upstream-help-diffs

Conversation

@liangmiQwQ

@liangmiQwQ liangmiQwQ commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

This PR includes two primary parts, follow up to #2184

  1. Leave a comment under the PR that notices maintainers the help document differs
  2. Let Claude Code update the help document when there is differences

🤖 Generated with Codex

@liangmiQwQ

liangmiQwQ commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@netlify

netlify Bot commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 1308816
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a743e9f1da8a000080f4c91

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

Reviewed commit: 749b9ad524

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

@liangmiQwQ
liangmiQwQ marked this pull request as ready for review August 5, 2026 02:30
@liangmiQwQ
liangmiQwQ requested a review from fengmk2 August 5, 2026 02:30
@fengmk2 fengmk2 self-assigned this Aug 5, 2026
@fengmk2

fengmk2 commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

test run https://github.com/voidzero-dev/vite-plus/actions/runs/30991987365

@fengmk2

fengmk2 commented Aug 5, 2026

Copy link
Copy Markdown
Member

it works #2337 (comment)

@fengmk2

fengmk2 commented Aug 5, 2026

Copy link
Copy Markdown
Member

why helper message not changed? #2337 (comment)

@liangmiQwQ

liangmiQwQ commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

why helper message not changed? #2337 (comment)

Strange, let me debug

@liangmiQwQ

liangmiQwQ commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

🙇 It seems in #2184, I did not completely follow the underlying tools' help document. Instead, it says Emit a warning for a rule or category there, so Claude treat them as not needed to updated.

@liangmiQwQ liangmiQwQ mentioned this pull request Aug 5, 2026
Merged
@liangmiQwQ

liangmiQwQ commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Resolved in #2345.

@liangmiQwQ

liangmiQwQ commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator Author

@fengmk2 I also left several updates on this branch, sync before your do the test :)

fengmk2 added a commit that referenced this pull request Aug 6, 2026
@liangmiQwQ @fengmk2
fix(cli): align local help with upstream tools (#2345)
ae5f27e 
Refs: [#2330
discussion](#2330 (comment))

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

---------

Co-authored-by: MK (fengmk2) <fengmk2@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fengmk2 fengmk2 Awaiting requested review from fengmk2

At least 1 approving review is required to merge this pull request.

Assignees

@fengmk2 fengmk2

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@liangmiQwQ @fengmk2