From 1c4e218946d1ae134f38f6202948ab6058ff9d0d Mon Sep 17 00:00:00 2001
From: Andrew Chou <andrewchou@fastmail.com>
Date: Tue, 17 Dec 2024 11:58:35 -0500
Subject: [PATCH] use preStart hook instead of startLogic in custom forge
 plugin

---
 forge.config.js | 39 ++++++++++++++-------------------------
 1 file changed, 14 insertions(+), 25 deletions(-)

diff --git a/forge.config.js b/forge.config.js
index 51543dc..c25956d 100644
--- a/forge.config.js
+++ b/forge.config.js
@@ -58,6 +58,7 @@ class CoMapeoDesktopForgePlugin extends PluginBase {
 	 */
 	getHooks() {
 		return {
+			preStart: [this.#initViteDevServer],
 			resolveForgeConfig: [this.#updatePackagerConfig],
 			postStart: [this.#hookViteDevServer],
 			prePackage: [this.#buildRender],
@@ -66,34 +67,22 @@ class CoMapeoDesktopForgePlugin extends PluginBase {
 	}
 
 	/**
-	 * Starts the Vite dev server as part of the `forge start` command
-	 *
-	 * @type {PluginBase<{}>['startLogic']}
-	 * @override
+	 * @type {ForgeHookFn<'preStart'>}
 	 */
-	startLogic = async (_opts) => {
-		if (this.#viteDevServer) return false
+	#initViteDevServer = async (_opts) => {
+		if (this.#viteDevServer) return
 
-		return {
-			result: false,
-			tasks: [
-				{
-					title: 'Start Vite dev server',
-					task: async () => {
-						const server = await createServer({
-							configFile: RENDERER_VITE_CONFIG_PATH,
-						})
+		const server = await createServer({
+			configFile: RENDERER_VITE_CONFIG_PATH,
+		})
 
-						try {
-							await server.listen()
-							server.printUrls()
-							this.#viteDevServer = server
-						} catch {
-							console.log('Vite dev server already running.')
-						}
-					},
-				},
-			],
+		try {
+			await server.listen()
+			console.log('Started Vite dev server')
+			server.printUrls()
+			this.#viteDevServer = server
+		} catch {
+			console.log('Vite dev server already running.')
 		}
 	}