Expose collected BuildEvents publically

src/Buildalyzer/AnalyzerManagerOptions.cs

@@ -7,15 +7,15 @@ namespace Buildalyzer;
 
 public class AnalyzerManagerOptions
 {
-    public ILoggerFactory LoggerFactory { get; set; }
+    public ILoggerFactory? LoggerFactory { get; set; }
 
     /// <summary>
     /// A filter that indicates whether a give project should be loaded.
     /// Return <c>true</c> to load the project, <c>false</c> to filter it out.
     /// </summary>
-    public Func<ProjectInSolution, bool> ProjectFilter { get; set; }
+    public Func<ProjectInSolution, bool>? ProjectFilter { get; set; }
 
-    public TextWriter LogWriter
+    public TextWriter? LogWriter
     {
         set
         {

src/Buildalyzer/AnalyzerResults.cs

@@ -1,4 +1,5 @@
 using System.Collections.Concurrent;
+using Microsoft.Build.Framework;
 
 namespace Buildalyzer;
 
@@ -19,6 +20,9 @@
         _overallSuccess = _overallSuccess.HasValue ? _overallSuccess.Value && overallSuccess : overallSuccess;
     }
 
+    /// <inheritdoc />
+    public ImmutableArray<BuildEventArgs> BuildEventArguments { get; set; } = [];
+
     public IAnalyzerResult this[string targetFramework] => _results[targetFramework];
 
     public IEnumerable<string> TargetFrameworks => _results.Keys.OrderBy(e => e, TargetFrameworkComparer.Instance);
@@ -39,7 +43,7 @@
     {
         public static TargetFrameworkComparer Instance { get; } = new TargetFrameworkComparer();
 
         public int Compare(string x, string y)
         {
             return (string.IsNullOrEmpty(x), string.IsNullOrEmpty(y)) switch
             {

src/Buildalyzer/BuildEventArgsCollector.cs

@@ -17,6 +17,10 @@ public BuildEventArgsCollector(EventArgsDispatcher server)
     /// <inheritdoc />
     public int Count => Bag.Count;
 
+    /// <summary>Indicates that no events has been collected.</summary>
+    [Pure]
+    public bool None() => Count == 0;
+
     /// <inheritdoc />
     public IEnumerator<BuildEventArgs> GetEnumerator() => Bag.GetEnumerator();
 

src/Buildalyzer/IAnalyzerResults.cs

@@ -1,10 +1,13 @@
-namespace Buildalyzer;
+using Microsoft.Build.Framework;
 
-public interface IAnalyzerResults : IEnumerable<IAnalyzerResult>
+namespace Buildalyzer;
+
+public interface IAnalyzerResults : IReadOnlyCollection<IAnalyzerResult>
 {
-    IAnalyzerResult this[string targetFramework] { get; }
+    /// <summary>The collected <see cref="BuildEventArgs"/> during the analysis.</summary>
+    ImmutableArray<BuildEventArgs> BuildEventArguments { get; }
 
-    int Count { get; }
+    IAnalyzerResult this[string targetFramework] { get; }
 
     bool OverallSuccess { get; }
 

src/Buildalyzer/ProjectAnalyzer.cs

@@ -148,61 +148,55 @@ public IAnalyzerResults Build(string targetFramework, BuildEnvironment buildEnvi
     private IAnalyzerResults BuildTargets(
         BuildEnvironment buildEnvironment, string targetFramework, string[] targetsToBuild, AnalyzerResults results)
     {
-        using (CancellationTokenSource cancellation = new CancellationTokenSource())
-        {
-            using var pipeLogger = new AnonymousPipeLoggerServer(cancellation.Token);
-            bool receivedAnyEvent = false;
-
-            void OnPipeLoggerOnAnyEventRaised(object o, BuildEventArgs buildEventArgs)
-            {
-                receivedAnyEvent = true;
-            }
-
-            pipeLogger.AnyEventRaised += OnPipeLoggerOnAnyEventRaised;
+        using var cancellation = new CancellationTokenSource();
 
-            using var eventProcessor = new EventProcessor(Manager, this, BuildLoggers, pipeLogger, results != null);
+        using var pipeLogger = new AnonymousPipeLoggerServer(cancellation.Token);
+        using var eventCollector = new BuildEventArgsCollector(pipeLogger);
+        using var eventProcessor = new EventProcessor(Manager, this, BuildLoggers, pipeLogger, true);
 
-            // Run MSBuild
-            int exitCode;
-            string fileName = GetCommand(
-                buildEnvironment,
-                targetFramework,
-                targetsToBuild,
-                pipeLogger.GetClientHandle(),
-                out string arguments);
-
-            using (ProcessRunner processRunner = new ProcessRunner(
-                fileName,
-                arguments,
-                buildEnvironment.WorkingDirectory ?? Path.GetDirectoryName(ProjectFile.Path),
-                GetEffectiveEnvironmentVariables(buildEnvironment),
-                Manager.LoggerFactory))
+        // Run MSBuild
+        int exitCode;
+        string fileName = GetCommand(
+            buildEnvironment,
+            targetFramework,
+            targetsToBuild,
+            pipeLogger.GetClientHandle(),
+            out string arguments);
+
+        using (ProcessRunner processRunner = new ProcessRunner(
+            fileName,
+            arguments,
+            buildEnvironment.WorkingDirectory ?? Path.GetDirectoryName(ProjectFile.Path)!,
+            GetEffectiveEnvironmentVariables(buildEnvironment)!,
+            Manager.LoggerFactory))
+        {
+            void OnProcessRunnerExited()
             {
-                void OnProcessRunnerExited()
-                {
-                    if (!receivedAnyEvent && processRunner.ExitCode != 0)
-                    {
-                        pipeLogger.Dispose();
-                    }
-                }
-
-                processRunner.Exited += OnProcessRunnerExited;
-                processRunner.Start();
-                try
-                {
-                    pipeLogger.ReadAll();
-                }
-                catch (ObjectDisposedException)
+                if (eventCollector.None() && processRunner.ExitCode != 0)
                 {
-                    // Ignore
+                    pipeLogger.Dispose();
                 }
-                processRunner.WaitForExit();
-                exitCode = processRunner.ExitCode;
             }
 
-            // Collect the results
-            results?.Add(eventProcessor.Results, exitCode == 0 && eventProcessor.OverallSuccess);
+            processRunner.Exited += OnProcessRunnerExited;
+            processRunner.Start();
+            try
+            {
+                pipeLogger.ReadAll();
+            }
+            catch (ObjectDisposedException)
+            {
+                // Ignore
+            }
+            processRunner.WaitForExit();
+            exitCode = processRunner.ExitCode;
         }
+
+        results.BuildEventArguments = [.. eventCollector];
+
+        // Collect the results
+        results.Add(eventProcessor.Results, exitCode == 0 && eventProcessor.OverallSuccess);
+
         return results;
     }
 

tests/Buildalyzer.Tests/Integration/SimpleProjectsFixture.cs

@@ -81,6 +81,16 @@ public void Builds_DesignTime(
         results.Should().AllSatisfy(r => r.Succeeded.Should().BeTrue());
     }
 
+    [Test]
+    public void Collects_BuildEventArguments()
+    {
+        using var ctx = Context.ForProject(@"SdkNet6Project\SdkNet6Project.csproj");
+
+        var results = ctx.Analyzer.Build(new EnvironmentOptions());
+
+        results.BuildEventArguments.Should().HaveCount(18);
+    }
+
     [Test]
     public void BuildsProject(
         [ValueSource(nameof(Preferences))] EnvironmentPreference preference,