Write JSON payloads straight to stdout instead of through Spectre

`submission get` and every other command that emits a machine-readable
payload wrote it via Spectre.Console's static `AnsiConsole.WriteLine`.
That renders the string through Spectre's layout pipeline, which
word-wraps at `Profile.Width`. The wrap is unaware of JSON syntax, so it
injected raw U+000A/U+000D characters inside string values and could
split a `\uXXXX` escape in half.

The result was invalid JSON (RFC 8259 §7 forbids unescaped control
characters in strings), and — worse — lenient parsers such as
PowerShell's `ConvertFrom-Json` accepted it and silently corrupted the
listing text, so the documented `submission get` -> edit ->
`submission update` round-trip could republish mangled store copy.
Redirected stdout was the worst case, since Spectre falls back to 80
columns when there is no console.

Program.cs already puts all human-facing output on stderr and reserves
stdout for the payload; only the renderer was wrong. Add a
`StandardOutput.WriteLine` helper that writes to `Console.Out` directly
and route the 19 JSON call sites through it.

`package` writes its output directory to stdout for the same reason, so
it moves too — that also fixes a latent Spectre markup parse error for
paths containing `[`. The path is no longer green/bold.

Also fix a latent test-harness bug this uncovered: `OutputCapture` called
`Console.SetOut(this)` from its constructor, so the error capture
silently hijacked `Console.Out`. It is now set explicitly for stdout
only, matching how Program.cs keeps the two streams apart.

Fixes #150

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: af8e5c71-6935-4efa-b777-64b180bb20c7

Author: azchohfi

MSStore.CLI.UnitTests/BaseCommandLineTest.cs

@@ -372,7 +372,7 @@ internal void AddFakeAccount(AccountEnrollment? accountEnrollment)
                 });
         }
 
-        protected void AddDefaultFakeSubmission()
+        protected void AddDefaultFakeSubmission(string listingDescription = "BaseListingDescription")
         {
             var fakeSubmission = new DevCenterSubmission
             {
@@ -406,7 +406,7 @@ protected void AddDefaultFakeSubmission()
                             {
                                 BaseListing = new BaseListing
                                 {
-                                    Description = "BaseListingDescription"
+                                    Description = listingDescription
                                 }
                             }
                         }
@@ -764,6 +764,10 @@ protected void SetupBasedOnTestDataProjectSubPath(DirectoryInfo dirInfo, string[
             var outputCapture = new OutputCapture(Console.Out);
             var errorCapture = RefreshAnsiConsole();
 
+            // Only stdout is redirected: the error capture is reached exclusively through
+            // ErrorAnsiConsole, mirroring how Program.cs keeps the two streams apart.
+            Console.SetOut(outputCapture);
+
             AnsiConsole.Console = AnsiConsole.Create(new AnsiConsoleSettings
             {
                 Ansi = AnsiSupport.Yes,
@@ -871,7 +875,6 @@ internal sealed class OutputCapture : TextWriter, IDisposable
             public OutputCapture(TextWriter textWriter)
             {
                 _stdOutWriter = textWriter;
-                Console.SetOut(this);
                 Captured = new StringWriter();
             }
 

MSStore.CLI.UnitTests/PackageCommandUnitTests.cs

@@ -99,7 +99,7 @@ public async Task PackageCommandForUWPAppsShouldCallMSBuildWithOutputParameterIf
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(customPath);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(customPath));
 
             ExternalCommandExecutor.VerifyAll();
         }
@@ -216,7 +216,7 @@ public async Task PackageCommandForWinUIAppsShouldCallMSBuildWithOutputParameter
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(customPath);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(customPath));
 
             ExternalCommandExecutor.VerifyAll();
         }
@@ -295,7 +295,7 @@ public async Task PackageCommandForMauiAppsShouldCallMSBuildWithOutputParameterI
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(customPath);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(customPath));
 
             ExternalCommandExecutor.VerifyAll();
         }
@@ -375,7 +375,7 @@ public async Task PackageCommandForFlutterAppsShouldCallFlutter()
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(path);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(path));
         }
 
         [TestMethod]
@@ -426,7 +426,7 @@ public async Task PackageCommandForFlutterAppsShouldCallFlutterWithOutputParamet
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(customPath);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(customPath));
         }
 
         private void SetupPubGet(DirectoryInfo dirInfo)
@@ -478,7 +478,7 @@ public async Task PackageCommandForElectronNpmAppsShouldCallElectronNpm()
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(path);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(path));
         }
 
         [TestMethod]
@@ -514,7 +514,7 @@ public async Task PackageCommandForElectronYarnAppsShouldCallElectronYarn()
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(path);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(path));
         }
 
         [TestMethod]
@@ -566,7 +566,7 @@ public async Task PackageCommandForReactNativeNpmAppsShouldCallMSBuild(string ma
                 ]);
 
             result.Error.Should().Contain("The packaged app is here:");
-            result.Output.Should().Contain(path);
+            result.Output.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries).Should().ContainSingle(line => line.Contains(path));
         }
     }
 }

MSStore.CLI.UnitTests/SubmissionCommandPackagedUnitTests.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
+using System.Text.Json;
 using MSStore.API.Packaged.Models;
 
 namespace MSStore.CLI.UnitTests
@@ -68,6 +69,40 @@ public async Task PackagedSubmissionGetCommand()
             result.Output.Should().Contain("\"FileUploadUrl\": \"https://azureblob.com/fileupload\"");
         }
 
+        [TestMethod]
+        public async Task PackagedSubmissionGetCommandShouldNotWrapJsonOutput()
+        {
+            // Longer than the width the test console renders at, and sprinkled with characters
+            // that the serializer escapes as \uXXXX, so a wrap would both break the JSON and
+            // corrupt the description.
+            var longDescription = string.Concat(
+                Enumerable.Repeat("Sync your mail & calendar across every device without a fuss. ", 12));
+
+            AddDefaultFakeSubmission(longDescription);
+
+            FakeApps[0].LastPublishedApplicationSubmission = new ApplicationSubmissionInfo
+            {
+                Id = "123456789"
+            };
+
+            var result = await ParseAndInvokeAsync(
+                [
+                    "submission",
+                    "get",
+                    FakeApps[0].Id!
+                ]);
+
+            using var json = JsonDocument.Parse(result.Output);
+
+            json.RootElement
+                .GetProperty("Listings")
+                .GetProperty("en-us")
+                .GetProperty("BaseListing")
+                .GetProperty("Description")
+                .GetString()
+                .Should().Be(longDescription);
+        }
+
         [TestMethod]
         public async Task PackagedSubmissionGetListingAssetsCommand()
         {

MSStore.CLI.UnitTests/SubmissionCommandUnpackagedUnitTests.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
+using System.Text.Json;
 using MSStore.API.Models;
 
 namespace MSStore.CLI.UnitTests
@@ -68,6 +69,50 @@ public async Task UnpackagedSubmissionGetCommand()
             result.Output.Should().Contain("\"PackageId\": \"12345\"");
         }
 
+        [TestMethod]
+        public async Task UnpackagedSubmissionGetCommandShouldNotWrapJsonOutput()
+        {
+            // Longer than the width the test console renders at, and sprinkled with characters
+            // that the serializer escapes as \uXXXX, so a wrap would both break the JSON and
+            // corrupt the description.
+            var longDescription = string.Concat(
+                Enumerable.Repeat("Sync your mail & calendar across every device without a fuss. ", 12));
+
+            FakeStoreAPI
+                .Setup(x => x.GetDraftAsync(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<string>(), It.IsAny<CancellationToken>()))
+                .ReturnsAsync(new ResponseWrapper<ListingsMetadataResponse>
+                {
+                    IsSuccess = true,
+                    ResponseData = new ListingsMetadataResponse
+                    {
+                        Listings =
+                            [
+                                new Listing
+                                {
+                                    Language = "en-us",
+                                    Description = longDescription
+                                }
+                            ]
+                    }
+                });
+
+            var result = await ParseAndInvokeAsync(
+                [
+                    "submission",
+                    "get",
+                    Guid.Empty.ToString()
+                ]);
+
+            using var json = JsonDocument.Parse(result.Output);
+
+            json.RootElement
+                .GetProperty("ResponseData")
+                .GetProperty("Listings")[0]
+                .GetProperty("Description")
+                .GetString()
+                .Should().Be(longDescription);
+        }
+
         [TestMethod]
         public async Task UnpackagedSubmissionGetListingAssetsCommand()
         {

MSStore.CLI/Commands/Apps/GetCommand.cs

@@ -94,7 +94,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
                     }
                     else
                     {
-                        AnsiConsole.WriteLine(JsonSerializer.Serialize(app, app.GetType(), SourceGenerationContext.GetCustom(true)));
+                        StandardOutput.WriteLine(JsonSerializer.Serialize(app, app.GetType(), SourceGenerationContext.GetCustom(true)));
                         return await _telemetryClient.TrackCommandEventAsync<Handler>(0, ct);
                     }
                 }

MSStore.CLI/Commands/Flights/CreateCommand.cs

@@ -111,7 +111,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
 
                 if (flight != null)
                 {
-                    AnsiConsole.WriteLine(JsonSerializer.Serialize(flight, SourceGenerationContext.GetCustom(true).DevCenterFlight));
+                    StandardOutput.WriteLine(JsonSerializer.Serialize(flight, SourceGenerationContext.GetCustom(true).DevCenterFlight));
                     return await _telemetryClient.TrackCommandEventAsync<Handler>(0, ct);
                 }
 

MSStore.CLI/Commands/Flights/GetCommand.cs

@@ -75,7 +75,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
 
                 if (flight != null)
                 {
-                    AnsiConsole.WriteLine(JsonSerializer.Serialize(flight, SourceGenerationContext.GetCustom(true).DevCenterFlight));
+                    StandardOutput.WriteLine(JsonSerializer.Serialize(flight, SourceGenerationContext.GetCustom(true).DevCenterFlight));
                     return await _telemetryClient.TrackCommandEventAsync<Handler>(0, ct);
                 }
 

MSStore.CLI/Commands/Flights/Submission/GetCommand.cs

@@ -88,7 +88,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
                     return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, -1, ct);
                 }
 
-                AnsiConsole.WriteLine(JsonSerializer.Serialize(flightSubmission, SourceGenerationContext.GetCustom(true).DevCenterFlightSubmission));
+                StandardOutput.WriteLine(JsonSerializer.Serialize(flightSubmission, SourceGenerationContext.GetCustom(true).DevCenterFlightSubmission));
 
                 return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, 0, ct);
             }

MSStore.CLI/Commands/Flights/Submission/Rollout/FinalizeCommand.cs

@@ -101,7 +101,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
                     return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, -1, ct);
                 }
 
-                AnsiConsole.WriteLine(JsonSerializer.Serialize(flightSubmissionRollout, SourceGenerationContext.GetCustom(true).PackageRollout));
+                StandardOutput.WriteLine(JsonSerializer.Serialize(flightSubmissionRollout, SourceGenerationContext.GetCustom(true).PackageRollout));
 
                 return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, 0, ct);
             }

MSStore.CLI/Commands/Flights/Submission/Rollout/GetCommand.cs

@@ -101,7 +101,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
                     return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, -1, ct);
                 }
 
-                AnsiConsole.WriteLine(JsonSerializer.Serialize(flightSubmissionRollout, SourceGenerationContext.GetCustom(true).PackageRollout));
+                StandardOutput.WriteLine(JsonSerializer.Serialize(flightSubmissionRollout, SourceGenerationContext.GetCustom(true).PackageRollout));
 
                 return await _telemetryClient.TrackCommandEventAsync<Handler>(productId, 0, ct);
             }