fix(tools): fail on Vite branding source drift#2205
Conversation
✅ Deploy Preview for viteplus-preview canceled.
|
7178aac to
d78a8d0
Compare
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d78a8d02f0
ℹ️ 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".
|
@codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
2 participants
The Vite branding patcher currently treats a missing removable build-banner pattern as "Already patched". When upstream changes, sync can continue even though the branding patch no longer applies.
This PR restores the nested Vite checkout from
HEADbefore each patch pass, so repeated calls still start from raw upstream sources. If a restored source no longer matches a known pattern, the patcher raises an actionable upstream-drift error instead.It leaves the build-banner behavior and CLI snapshots unchanged.
A small follow-up to #2200
🤖 Generated with Codex