code review feedback

Author: mirkoCrobu

cmd/arduino-app-cli/app/restart.go

@@ -77,7 +77,6 @@ func restartHandler(ctx context.Context, cfg config.Configuration, app app.Ardui
 		}
 	}
 
-	// Feedback finale di successo
 	outputResult := getResult()
 	feedback.PrintResult(restartAppResult{
 		AppName: app.Name,

internal/orchestrator/orchestrator.go

@@ -451,11 +451,16 @@ func RestartApp(
 			yield(StreamMessage{error: err})
 			return
 		}
+
+		if runningApp != nil && runningApp.FullPath.String() != appToStart.FullPath.String() {
+			yield(StreamMessage{error: fmt.Errorf("another app %q is running", runningApp.Name)})
+			return
+		}
+
 		if runningApp != nil {
 			stopStream := StopApp(ctx, *runningApp)
 			for msg := range stopStream {
 				if !yield(msg) {
-					cancel()
 					return
 				}
 				if msg.error != nil {
@@ -464,12 +469,7 @@ func RestartApp(
 			}
 		}
 		startStream := StartApp(ctx, docker, provisioner, modelsIndex, bricksIndex, appToStart, cfg, staticStore)
-		for msg := range startStream {
-			if !yield(msg) {
-				cancel()
-				return
-			}
-		}
+		startStream(yield)
 	}
 }