Use repository root for output path calculation when the output isn't a child of the working directory

src/MSBuild/TerminalLogger/EvaluationData.cs

@@ -0,0 +1,30 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+internal sealed class EvaluationData
+{
+    /// <summary>
+    /// Captures data that comes from project evaluation, is assumed to not change through execution,
+    /// and is referenced for rendering purposes throughout the execution of the build.
+    /// </summary>
+    /// <param name="targetFramework"></param>
+    public EvaluationData(string? targetFramework)
+    {
+        TargetFramework = targetFramework;
+    }
+
+    /// <summary>
+    /// The target framework of the project or null if not multi-targeting.
+    /// </summary>
+    public string? TargetFramework { get; }
+
+    /// <summary>
+    /// This property is true when the project would prefer to have full paths in the logs and/or for processing tasks.
+    /// </summary>
+    /// <remarks>
+    /// There's an MSBuild property called GenerateFullPaths that would be a great knob to use for this, but the Common
+    /// Targets set it to true if not set, and setting it to false completely destroys the terminal logger output.
+    /// That's why this value is hardcoded to false for now, until we define a better mechanism.
+    /// </remarks>
+    public bool GenerateFullPaths { get; } = false;
+}

src/MSBuild/TerminalLogger/Project.cs

@@ -1,6 +1,11 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+#if NETFRAMEWORK
+using Microsoft.IO;
+#else
+using System.IO;
+#endif
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
@@ -10,16 +15,15 @@ namespace Microsoft.Build.Logging.TerminalLogger;
 /// <summary>
 /// Represents a project being built.
 /// </summary>
+[DebuggerDisplay("{OutputPath}({TargetFramework})")]
 internal sealed class Project
 {
     /// <summary>
     /// Initialized a new <see cref="Project"/> with the given <paramref name="targetFramework"/>.
     /// </summary>
     /// <param name="targetFramework">The target framework of the project or null if not multi-targeting.</param>
-    public Project(string? targetFramework, StopwatchAbstraction? stopwatch)
+    public Project(StopwatchAbstraction? stopwatch)
     {
-        TargetFramework = targetFramework;
-
         if (stopwatch is not null)
         {
             stopwatch.Start();
@@ -39,12 +43,12 @@ public Project(string? targetFramework, StopwatchAbstraction? stopwatch)
     /// <summary>
     /// Full path to the primary output of the project, if known.
     /// </summary>
-    public ReadOnlyMemory<char>? OutputPath { get; set; }
+    public FileInfo? OutputPath { get; set; }
 
     /// <summary>
-    /// The target framework of the project or null if not multi-targeting.
+    /// Full path to the 'root' of this project's source control repository, if known.
     /// </summary>
-    public string? TargetFramework { get; }
+    public DirectoryInfo? SourceRoot { get; set; }
 
     /// <summary>
     /// True when the project has run target with name "_TestRunStart" defined in <see cref="TerminalLogger._testStartTarget"/>.
@@ -56,6 +60,11 @@ public Project(string? targetFramework, StopwatchAbstraction? stopwatch)
     /// </summary>
     public bool IsCachePluginProject { get; set; }
 
+    /// <summary>
+    /// This property is true when the project would prefer to have full paths in the logs and/or for processing tasks.
+    /// </summary>
+    public bool GenerateFullPaths { get; set; }
+
     /// <summary>
     /// A lazily initialized list of build messages/warnings/errors raised during the build.
     /// </summary>