fix: background scripts for firefox build (#790)

Co-authored-by: Jack Steam <jacksteamdev@gmail.com>
crxjs · Oct 16, 2023 · f5c4bd7 · f5c4bd7
1 parent 54002ea
commit f5c4bd7
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 0 deletions.
diff --git a/.changeset/fluffy-geckos-flash.md b/.changeset/fluffy-geckos-flash.md
@@ -0,0 +1,5 @@
+---
+"@crxjs/vite-plugin": patch
+---
+
+fix: background scripts for firefox build
diff --git a/packages/vite-plugin/src/node/plugin-background.ts b/packages/vite-plugin/src/node/plugin-background.ts
@@ -108,6 +108,7 @@ export const pluginBackground: CrxPluginFn = () => {
         } else {
           manifest.background = {
             scripts: [this.getFileName(refId)],
+            type: 'module',
           }
         }
 

diff --git a/packages/vite-plugin/src/node/plugin-manifest.ts b/packages/vite-plugin/src/node/plugin-manifest.ts
@@ -239,6 +239,17 @@ export const pluginManifest: CrxPluginFn = () => {
             manifest.background.service_worker = refId
           }
 
+          if (manifest.background && 'scripts' in manifest.background) {
+            const file = manifest.background.scripts[0]
+            const id = join(config.root, file)
+            const refId = this.emitFile({
+              type: 'chunk',
+              id,
+              name: basename(file),
+            })
+            manifest.background.scripts = [refId]
+          }
+
           for (const file of htmlFiles(manifest)) {
             const id = join(config.root, file)
             this.emitFile({
@@ -278,6 +289,12 @@ export const pluginManifest: CrxPluginFn = () => {
             manifest.background.service_worker = name
           }
 
+          if (manifest.background && 'scripts' in manifest.background) {
+            const ref = manifest.background.scripts[0]
+            const name = this.getFileName(ref)
+            manifest.background.scripts = [name]
+          }
+
           // update content script file names from refs
           // TODO: emit and parse css
           manifest.content_scripts = manifest.content_scripts?.map(