Return status codes from Fusion CLI commands (#7001)

Author: tobias-tengler

src/HotChocolate/Fusion/src/CommandLine/Commands/ComposeCommand.cs

@@ -63,7 +63,7 @@ public ComposeCommand() : base("compose")
 
     [RequiresUnreferencedCode(
         "Calls System.Text.Json.JsonSerializer.SerializeToDocument<TValue>(TValue, JsonSerializerOptions)")]
-    private static async Task ExecuteAsync(
+    private static async Task<int> ExecuteAsync(
         IConsole console,
         FileInfo packageFile,
         List<string>? subgraphPackageFiles,
@@ -125,7 +125,7 @@ private static async Task ExecuteAsync(
         if (settings is null)
         {
             console.WriteLine("Fusion graph settings are invalid.");
-            return;
+            return 1;
         }
 
         if (enableNodes.HasValue && enableNodes.Value)
@@ -145,7 +145,7 @@ private static async Task ExecuteAsync(
         if (fusionGraph is null)
         {
             console.WriteLine("Fusion graph composition failed.");
-            return;
+            return 1;
         }
 
         var fusionGraphDoc = Utf8GraphQLParser.Parse(SchemaFormatter.FormatAsString(fusionGraph));
@@ -164,6 +164,8 @@ private static async Task ExecuteAsync(
         }
 
         console.WriteLine("Fusion graph composed.");
+
+        return 0;
     }
 
     private static FusionFeatureCollection CreateFeatures(
@@ -382,4 +384,4 @@ public sealed class Transport
         [JsonPropertyOrder(10)]
         public string? DefaultClientName { get; set; } = "Fusion";
     }
-}
+}

src/HotChocolate/Fusion/src/CommandLine/Commands/ExportGraphCommand.cs

@@ -19,7 +19,7 @@ public ExportGraphCommand() : base("graph")
 
         var graphFile = new Option<FileInfo?>("--file");
         graphFile.AddAlias("-f");
-        
+
         AddOption(fusionPackageFile);
         AddOption(graphFile);
 
@@ -31,40 +31,42 @@ public ExportGraphCommand() : base("graph")
             Bind.FromServiceProvider<CancellationToken>());
     }
 
-    private static async Task ExecuteAsync(
+    private static async Task<int> ExecuteAsync(
         IConsole console,
         FileInfo? packageFile,
         FileInfo? graphFile,
         CancellationToken cancellationToken)
     {
         packageFile ??= new FileInfo(Combine(Environment.CurrentDirectory, "gateway" + FusionPackage));
-        
+
         if (!packageFile.Exists)
         {
             if (Directory.Exists(packageFile.FullName))
             {
-                packageFile = new FileInfo(Combine(packageFile.FullName, "gateway" + FusionPackage));   
+                packageFile = new FileInfo(Combine(packageFile.FullName, "gateway" + FusionPackage));
             }
             else if (!packageFile.Extension.EqualsOrdinal(FusionPackage) &&
                 !packageFile.Extension.EqualsOrdinal(ZipPackage))
             {
-                packageFile = new FileInfo(packageFile.FullName + FusionPackage);   
+                packageFile = new FileInfo(packageFile.FullName + FusionPackage);
             }
 
             if (!packageFile.Exists)
             {
                 console.WriteLine($"The package file `{packageFile.FullName}` does not exist.");
-                return;
+                return 1;
             }
         }
 
         graphFile ??= new FileInfo(Combine(packageFile.DirectoryName!, "fusion.graphql"));
-        
+
         await using var package = FusionGraphPackage.Open(packageFile.FullName);
 
         var graph = await package.GetFusionGraphAsync(cancellationToken);
         var options = new SyntaxSerializerOptions { Indented = true, MaxDirectivesPerLine = 0, };
-        
+
         await File.WriteAllTextAsync(graphFile.FullName, graph.ToString(options), Encoding.UTF8, cancellationToken);
+
+        return 0;
     }
-}
+}

src/HotChocolate/Fusion/src/CommandLine/Commands/SubgraphPackCommand.cs

@@ -37,7 +37,7 @@ public SubgraphPackCommand() : base("pack")
             Bind.FromServiceProvider<CancellationToken>());
     }
 
-    private static async Task ExecuteAsync(
+    private static async Task<int> ExecuteAsync(
         IConsole console,
         FileInfo? packageFile,
         FileInfo? schemaFile,
@@ -56,13 +56,13 @@ private static async Task ExecuteAsync(
         if (!schemaFile.Exists)
         {
             console.WriteLine($"The schema file `{schemaFile.FullName}` does not exist.");
-            return;
+            return 1;
         }
 
         if (!configFile.Exists)
         {
             console.WriteLine($"The config file `{configFile.FullName}` does not exist.");
-            return;
+            return 1;
         }
 
         if (extensionFiles.Count == 0)
@@ -81,7 +81,7 @@ private static async Task ExecuteAsync(
         {
             console.WriteLine(
                 $"The extension file `{extensionFiles.First(t => !t.Exists).FullName}` does not exist.");
-            return;
+            return 1;
         }
 
         if (packageFile is null)
@@ -115,5 +115,7 @@ await CreateSubgraphPackageAsync(
             cancellationToken);
 
         console.WriteLine($"{packageFile.FullName} created.");
+
+        return 0;
     }
 }