Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Create plugin for resolve multiple entry points for content and content-ui scripts #846

Draft
wants to merge 31 commits into
base: dev
Choose a base branch
from

Conversation

PatrykKuniczak
Copy link
Collaborator

@PatrykKuniczak PatrykKuniczak commented Jan 27, 2025

* denotes required fields

Priority*

  • High: This PR needs to be merged first, before other tasks.
  • Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
  • Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*

I've created plugin which allows users to create multiple content and content-ui scripts.

Changes*

I've refactored old approach with only one script. to accept multiple entries.
I've needed to split bundling process into 2 steps, because esbuild or rollup config of vite doesn't work well with multiple entries which need to be bundle to iife format.

First step is for add HMR code into files.
Second bundle files from first step to be iife

How to check the feature

Run pnpm dev or pnpm build and check if it works

@PatrykKuniczak PatrykKuniczak force-pushed the feat/create-matches-for-content-scripts branch 3 times, most recently from ab6a772 to 5149d5c Compare February 2, 2025 15:14
@PatrykKuniczak PatrykKuniczak force-pushed the feat/create-matches-for-content-scripts branch 2 times, most recently from f5b8585 to b39e686 Compare February 13, 2025 19:23
…content-script-with-hmr.ts and adjust to be compatible with watch-rebuild-plugin.ts
@PatrykKuniczak PatrykKuniczak force-pushed the feat/create-matches-for-content-scripts branch from 57188d5 to ce562d6 Compare February 14, 2025 09:17
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