catch json error (#9369)

danmoseley · web-flow · commit ef5ab9a5b321 · 2025-05-16T16:49:56.000-06:00
* catch json error

* spc

* 'using
diff --git a/src/Aspire.Cli/DotNetCliRunner.cs b/src/Aspire.Cli/DotNetCliRunner.cs
@@ -659,34 +659,42 @@ public async Task<int> AddPackageAsync(FileInfo projectFilePath, string packageN
             }
 
             var foundPackages = new List<NuGetPackage>();
-            var document = JsonDocument.Parse(stdout);
-
-            var searchResultsArray = document.RootElement.GetProperty("searchResult");
-
-            foreach (var sourceResult in searchResultsArray.EnumerateArray())
+            try
             {
-                var source = sourceResult.GetProperty("sourceName").GetString();
-                var sourcePackagesArray = sourceResult.GetProperty("packages");
-
-                foreach (var packageResult in sourcePackagesArray.EnumerateArray())
-                {
-                    var id = packageResult.GetProperty("id").GetString();
+                using var document = JsonDocument.Parse(stdout);
 
-                    // var version = prerelease switch {
-                    //     true => packageResult.GetProperty("version").GetString(),
-                    //     false => packageResult.GetProperty("latestVersion").GetString()
-                    // };
+                var searchResultsArray = document.RootElement.GetProperty("searchResult");
 
-                    var version = packageResult.GetProperty("latestVersion").GetString();
+                foreach (var sourceResult in searchResultsArray.EnumerateArray())
+                {
+                    var source = sourceResult.GetProperty("sourceName").GetString();
+                    var sourcePackagesArray = sourceResult.GetProperty("packages");
 
-                    foundPackages.Add(new NuGetPackage
+                    foreach (var packageResult in sourcePackagesArray.EnumerateArray())
                     {
-                        Id = id!,
-                        Version = version!,
-                        Source = source!
-                    });
+                        var id = packageResult.GetProperty("id").GetString();
+
+                        // var version = prerelease switch {
+                        //     true => packageResult.GetProperty("version").GetString(),
+                        //     false => packageResult.GetProperty("latestVersion").GetString()
+                        // };
+
+                        var version = packageResult.GetProperty("latestVersion").GetString();
+
+                        foundPackages.Add(new NuGetPackage
+                        {
+                            Id = id!,
+                            Version = version!,
+                            Source = source!
+                        });
+                    }
                 }
             }
+            catch (JsonException ex)
+            {
+                logger.LogError($"Failed to read JSON returned by the package search. {ex.Message}");
+                return (ExitCodeConstants.FailedToAddPackage, null);
+            }
 
             return (result, foundPackages.ToArray());
         }
