fix: onBundleReady called before manifest is generated in build mode (#221)

Author: alvkn

packages/vite-plugin-web-extension/src/plugins/manifest-loader-plugin.ts

@@ -153,12 +153,14 @@ export function manifestLoaderPlugin(options: ResolvedOptions): vite.Plugin {
       });
     }
 
-    if (options.onBundleReady) {
+    await copyPublicDirToOutDir({ mode, paths });
+
+    // Handle the onBundleReady callback in dev mode here, as writeBundle is not called in dev mode
+    if (mode === BuildMode.DEV && options.onBundleReady) {
+      logger.verbose("Running onBundleReady");
       await options.onBundleReady();
     }
 
-    await copyPublicDirToOutDir({ mode, paths });
-
     // In dev mode, open up the browser immediately after the build context is finished with the
     // first build.
     if (mode === BuildMode.DEV && !options.disableAutoLaunch) {
@@ -287,6 +289,14 @@ export function manifestLoaderPlugin(options: ResolvedOptions): vite.Plugin {
       noInput.cleanupBundle(bundle);
     },
 
+    // Runs during: build, watch
+    async writeBundle() {
+      if (options.onBundleReady) {
+        logger.verbose("Running onBundleReady");
+        await options.onBundleReady();
+      }
+    },
+
     // Runs during: watch, dev
     async watchChange(id) {
       if (