docs: update upgrade guide for http-proxy-middleware v3#7122
docs: update upgrade guide for http-proxy-middleware v3#7122chenjiahan merged 3 commits intomainfrom
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary of ChangesHello @chenjiahan, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request primarily enhances the Rsbuild upgrade documentation by providing comprehensive guidance for migrating proxy configurations due to the update of Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
Deploying rsbuild with
|
| Latest commit: |
dc68600
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://55c1cb35.rsbuild-v2.pages.dev |
| Branch Preview URL: | https://proxy-migration-guide-0202.rsbuild-v2.pages.dev |
There was a problem hiding this comment.
Pull request overview
This PR updates the v1→v2 upgrade guides (EN & ZH) and related server proxy documentation to explain the http-proxy-middleware v3 migration details and reorganizes existing notes about removed HTML template parameters.
Changes:
- Add a new "Proxy middleware upgraded" section to the EN and ZH v1-to-v2 upgrade guides, documenting
http-proxy-middlewarev3 changes: explicittargetpaths,context→pathFilter, and unifiedonevent configuration. - Tweak the ZH
server.proxyconfig doc wording around v2 vs v3 behavior to match the new upgrade guidance. - Move the HTML template parameter removal note from its own subsection into the "Others/其他" bullet list in both EN and ZH upgrade guides.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
website/docs/zh/guide/upgrade/v1-to-v2.mdx |
Adds Chinese upgrade guidance for http-proxy-middleware v3 (target path, pathFilter, on events) and relocates the HTML template parameter removal note into the “其他” section with minor wording improvements. |
website/docs/zh/config/server/proxy.mdx |
Adjusts the ZH description of v2 vs v3 proxy path behavior to read more smoothly while remaining consistent with the new upgrade section. |
website/docs/en/guide/upgrade/v1-to-v2.mdx |
Adds English upgrade guidance for http-proxy-middleware v3 and moves the HTML template parameter removal details into the “Others” section. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request updates the upgrade guide to include breaking changes from http-proxy-middleware v3. The changes are clear and well-structured, covering the new requirements for target, the replacement of context with pathFilter, and the new event configuration. I've found a small inconsistency in one of the examples that could be misleading for users, and I've provided suggestions to correct it in both the English and Chinese documentation.
Summary
Added a new section to the upgrade guide explaining the upgrade to
http-proxy-middlewarev3, including explicit path requirements intarget, replacement ofcontextwithpathFilter, and unified event configuration using theonoption.Related Links
Checklist