Implement @DaveTryon's suggestion using Result property approach

Co-authored-by: DaveTryon <45672944+DaveTryon@users.noreply.github.com>

Author: Copilot

src/Microsoft.Sbom.Api/Output/Telemetry/IRecorder.cs

@@ -5,6 +5,7 @@
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using Microsoft.Sbom.Api.Entities;
+using Microsoft.Sbom.Api.Entities.Output;
 using Microsoft.Sbom.Extensions.Entities;
 
 namespace Microsoft.Sbom.Api.Output.Telemetry;
@@ -105,4 +106,9 @@ public interface IRecorder
     /// Record telemetry for an AggregationSource.
     /// </summary>
     public void RecordAggregationSource(string identifier, int packageCount, int relationshipCount);
+
+    /// <summary>
+    /// Gets the result status of the recorder execution.
+    /// </summary>
+    public Result Result { get; }
 }

src/Microsoft.Sbom.Api/Output/Telemetry/TelemetryRecorder.cs

@@ -124,6 +124,8 @@ public async Task LogException(Exception exception)
 
     public IList<Exception> Exceptions => exceptions;
 
+    public Result Result => result;
+
     /// <summary>
     /// Start recording the duration of exeuction of the given event.
     /// </summary>

src/Microsoft.Sbom.Api/SbomValidator.cs

@@ -9,6 +9,7 @@
 using System.Threading.Tasks;
 using Microsoft.Sbom.Api.Config;
 using Microsoft.Sbom.Api.Config.Extensions;
+using Microsoft.Sbom.Api.Entities.Output;
 using Microsoft.Sbom.Api.Output.Telemetry;
 using Microsoft.Sbom.Api.Workflows;
 using Microsoft.Sbom.Common;
@@ -104,8 +105,7 @@ public async Task<SbomValidationResult> ValidateSbomAsync(
         await recorder.FinalizeAndLogTelemetryAsync();
 
         var errors = recorder.Errors.Select(error => error.ToEntityError()).ToList();
-        var hasExceptions = recorder.Exceptions.Any();
-        return new SbomValidationResult(!errors.Any() && !hasExceptions, errors);
+        return new SbomValidationResult(recorder.Result == Result.Success, errors);
     }
 
     private InputConfiguration ValidateConfig(InputConfiguration config)

test/Microsoft.Sbom.Api.Tests/SbomValidatorTests.cs

@@ -3,8 +3,10 @@
 
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.Sbom.Api.Entities;
+using Microsoft.Sbom.Api.Entities.Output;
 using Microsoft.Sbom.Api.Output.Telemetry;
 using Microsoft.Sbom.Api.Workflows;
 using Microsoft.Sbom.Common;
@@ -89,7 +91,10 @@ private void SetupMocksForValidation(List<FileValidationResult> errors, List<Exc
 
         recorderMock.Setup(r => r.FinalizeAndLogTelemetryAsync()).Returns(Task.CompletedTask);
         recorderMock.Setup(r => r.Errors).Returns(errors);
-        recorderMock.Setup(r => r.Exceptions).Returns(exceptions);
+
+        // Determine the result based on whether there are errors or exceptions
+        var result = (errors.Any() || exceptions.Any()) ? Result.Failure : Result.Success;
+        recorderMock.Setup(r => r.Result).Returns(result);
     }
 
     [TestMethod]

test/Microsoft.Sbom.Api.Tests/VersionSpecificPins/Version_4_0/InterfaceConcretionTests.cs

@@ -15,6 +15,7 @@
 using Microsoft.Sbom.Api.Config.Args;
 using Microsoft.Sbom.Api.Convertors;
 using Microsoft.Sbom.Api.Entities;
+using Microsoft.Sbom.Api.Entities.Output;
 using Microsoft.Sbom.Api.Executors;
 using Microsoft.Sbom.Api.Filters;
 using Microsoft.Sbom.Api.FormatValidator;
@@ -130,6 +131,8 @@ private class PinnedIRecorder : IRecorder
 
         public IList<Exception> Exceptions => throw new NotImplementedException();
 
+        public Result Result => throw new NotImplementedException();
+
         public void AddResult(string propertyName, string value) => throw new NotImplementedException();
         public void AddToTotalCountOfLicenses(int count) => throw new NotImplementedException();
         public void AddToTotalNumberOfPackageDetailsEntries(int count) => throw new NotImplementedException();