add pm2 fixes, upgrade pm2

Author: claytercek

package-lock.json

@@ -34,7 +34,7 @@
     "cross-spawn": "^7.0.3",
     "neverthrow": "^8.1.1",
     "node-window-manager": "^2.2.4",
-    "pm2": "^5.1.2",
+    "pm2": "^5.4.3",
     "semver": "^7.3.5",
     "tail": "^2.2.4",
     "zod": "^3.23.8"

packages/monitor/package.json

@@ -36,7 +36,7 @@ export class ProcessManager {
 	}
 
 	getProcesses(): ResultAsync<pm2.ProcessDescription[], Error> {
-		return wrapPm2Function("Failed to get PM2 processes", pm2.list);
+		return wrapPm2Function("Failed to get PM2 processes", (cb) => pm2.list(cb));
 	}
 
 	startProcess(options: pm2.StartOptions): ResultAsync<pm2.ProcessDescription, Error> {
@@ -86,8 +86,10 @@ function wrapPm2Function<T>(
 }
 
 const safeDisconnect = Result.fromThrowable(
-	pm2.disconnect,
-	(error) => new Error("Failed to disconnect from PM2", { cause: error }),
+	() => pm2.disconnect(),
+	(error) => {
+		return new Error("Failed to disconnect from PM2", { cause: error })
+	},
 );
 
 function pingDaemon(): ResultAsync<boolean, Error> {

packages/monitor/src/core/process-manager.ts

@@ -134,7 +134,7 @@ export const monitorConfigSchema = z.object({
 	/** Set this to true to delete existing PM2 processes before connecting. If you're running volatile apps or your node process might be quit unexpectedly, this can be helpful to start with a clean slate on startup. */
 	deleteExistingBeforeConnect: z
 		.boolean()
-		.default(false)
+		.default(true)
 		.describe(
 			"Set this to true to delete existing PM2 processes before connecting. If you're running volatile apps or your node process might be quit unexpectedly, this can be helpful to start with a clean slate on startup.",
 		),