Skip to content

add client-only-plugins#543

Merged
cm-ayf merged 1 commit intomainfrom
client-only-plugins
Mar 1, 2026
Merged

add client-only-plugins#543
cm-ayf merged 1 commit intomainfrom
client-only-plugins

Conversation

@cm-ayf
Copy link
Copy Markdown
Owner

@cm-ayf cm-ayf commented Mar 1, 2026

No description provided.

@cm-ayf cm-ayf self-assigned this Mar 1, 2026
Copilot AI review requested due to automatic review settings March 1, 2026 03:18
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comiacapay Building Building Preview, Comment Mar 1, 2026 3:18am

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a small Vite plugin utility to force selected plugins to run only in the client environment, and applies it in the Vite config to keep the PWA build plugin from running outside the client build.

Changes:

  • Add clientOnlyPlugins helper plugin under plugins/ that patches matching plugins with applyToEnvironment.
  • Update vite.config.ts to use clientOnlyPlugins to scope vite-plugin-pwa:build to the client environment.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
vite.config.ts Registers the new helper and applies it to the PWA build plugin by name.
plugins/client-only-plugins.ts Introduces the helper plugin that mutates matched plugins to be client-only via applyToEnvironment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cm-ayf cm-ayf merged commit 5560047 into main Mar 1, 2026
15 checks passed
@cm-ayf cm-ayf deleted the client-only-plugins branch March 1, 2026 03:27
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