fix(plugins): refine error and success handling during plugin installation

- Updated the `UnraidPluginsService` to ensure that error and success handling for plugin installations only occurs when the operation status is RUNNING.
- Added checks in the `handleFailure` and `handleSuccess` methods to prevent state changes if the operation is not in progress.
- This change improves the reliability of the plugin installation process by preventing unintended state updates after the operation has completed or failed.

Author: elibosley

api/src/unraid-api/graph/resolvers/unraid-plugins/unraid-plugins.service.ts

@@ -83,10 +83,16 @@ export class UnraidPluginsService {
         }
 
         child.on('error', (error) => {
-            this.handleFailure(operation, error);
+            if (operation.status === PluginInstallStatus.RUNNING) {
+                this.handleFailure(operation, error);
+            }
         });
 
         child.on('close', (code) => {
+            if (operation.status !== PluginInstallStatus.RUNNING) {
+                return;
+            }
+
             if (code === 0) {
                 this.handleSuccess(operation);
             } else {
@@ -150,6 +156,10 @@ export class UnraidPluginsService {
     }
 
     private handleSuccess(operation: OperationState) {
+        if (operation.status !== PluginInstallStatus.RUNNING) {
+            return;
+        }
+
         const timestamp = new Date();
         operation.status = PluginInstallStatus.SUCCEEDED;
         operation.finishedAt = timestamp;
@@ -168,6 +178,10 @@ export class UnraidPluginsService {
     }
 
     private handleFailure(operation: OperationState, error: unknown) {
+        if (operation.status !== PluginInstallStatus.RUNNING) {
+            return;
+        }
+
         const timestamp = new Date();
         operation.status = PluginInstallStatus.FAILED;
         operation.finishedAt = timestamp;