Keep msstore package stdout clean, and print an absolute path

Follow-up from self-review of the previous commit.

Making `package` treat stdout as its machine-readable result exposed two
other writers on that same path that still went through the static
Spectre console (which targets stdout):

- `WinUIProjectConfigurator` hosted its "Building MSIX..." status on the
  static `AnsiConsole` — the only configurator that did; every sibling
  uses the injected stderr console, and the block already reported
  success via `ctx.SuccessStatus(ErrorAnsiConsole, ...)`. When stdout is
  redirected Spectre falls back to `FallbackStatusRenderer`, which writes
  the description permanently rather than clearing it, so the status text
  landed in the piped output ahead of the path.

- `ValidateImagesAsync` takes an `IAnsiConsole` and forwards it to
  `ProjectImagesHelper`, but used the static console for its own two
  warning lines, so the default-images warning went to stdout.

Both now use the console they already had in hand.

Also print `outputDirectory.FullName` rather than `ToString()`.
`DirectoryInfo.ToString()` returns the original, unnormalised path, and
`PWAProjectConfigurator.PackageAsync` returns `new DirectoryInfo(pathOrUrl)`
built straight from the command argument — so `msstore package .` emitted
a bare `.`, which is useless to a caller consuming stdout. The other five
packagers derive their result from `FileInfo(...).Directory`, whose path
is already absolute, so this is a no-op for them.

Tests now assert that `package` writes exactly one line to stdout, which
is what pins the behaviour these fixes restore.

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

Author: azchohfi

MSStore.CLI.UnitTests/PackageCommandUnitTests.cs

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

MSStore.CLI/Commands/PackageCommand.cs

@@ -100,7 +100,7 @@ public override async Task<int> InvokeAsync(ParseResult parseResult, Cancellatio
                 if (returnCode == 0 && outputDirectory != null)
                 {
                     _ansiConsole.WriteLine($"The packaged app is here:");
-                    StandardOutput.WriteLine(outputDirectory.ToString());
+                    StandardOutput.WriteLine(outputDirectory.FullName);
                 }
 
                 return await _telemetryClient.TrackCommandEventAsync<Handler>(returnCode, props, ct);

MSStore.CLI/Helpers/IProjectConfiguratorExtensions.cs

@@ -21,10 +21,10 @@ internal static async Task ValidateImagesAsync(this IProjectConfigurator configu
                 var defaultImages = ProjectImagesHelper.GetDefaultImagesUsedByApp(ansiConsole, appImages, projectSpecificDefaultImages, imageConverter, logger);
                 if (defaultImages.Count > 0)
                 {
-                    AnsiConsole.MarkupLine($"[bold yellow]The following images are using the default values and should be updated:[/]");
+                    ansiConsole.MarkupLine($"[bold yellow]The following images are using the default values and should be updated:[/]");
                     foreach (var image in defaultImages)
                     {
-                        AnsiConsole.MarkupLine($"[bold yellow]  {image}[/]");
+                        ansiConsole.MarkupLine($"[bold yellow]  {image}[/]");
                     }
                 }
             }

MSStore.CLI/ProjectConfigurators/WinUIProjectConfigurator.cs

@@ -71,7 +71,7 @@ public override async Task<bool> CanConfigureAsync(string pathOrUrl, Cancellatio
 
             version = AppXManifestManager.UpdateManifestVersion(manifestFile.FullName, version);
 
-            var bundleUploadFile = await AnsiConsole.Status().StartAsync("Building MSIX...", async ctx =>
+            var bundleUploadFile = await ErrorAnsiConsole.Status().StartAsync("Building MSIX...", async ctx =>
             {
                 try
                 {