Add feature-level telemetry for dotnet run command (#50793)

Author: marcpopMSFT

src/Cli/dotnet/Commands/Run/RunCommand.cs

@@ -124,14 +124,15 @@ public int Execute()
 
         Func<ProjectCollection, ProjectInstance>? projectFactory = null;
         RunProperties? cachedRunProperties = null;
+        VirtualProjectBuildingCommand? virtualCommand = null;
         if (ShouldBuild)
         {
             if (string.Equals("true", launchSettings?.DotNetRunMessages, StringComparison.OrdinalIgnoreCase))
             {
                 Reporter.Output.WriteLine(CliCommandStrings.RunCommandBuilding);
             }
 
-            EnsureProjectIsBuilt(out projectFactory, out cachedRunProperties);
+            EnsureProjectIsBuilt(out projectFactory, out cachedRunProperties, out virtualCommand);
         }
         else
         {
@@ -143,11 +144,11 @@ public int Execute()
             if (EntryPointFileFullPath is not null)
             {
                 Debug.Assert(!ReadCodeFromStdin);
-                var command = CreateVirtualCommand();
-                command.MarkArtifactsFolderUsed();
+                virtualCommand = CreateVirtualCommand();
+                virtualCommand.MarkArtifactsFolderUsed();
 
-                var cacheEntry = command.GetPreviousCacheEntry();
-                projectFactory = CanUseRunPropertiesForCscBuiltProgram(BuildLevel.None, cacheEntry) ? null : command.CreateProjectInstance;
+                var cacheEntry = virtualCommand.GetPreviousCacheEntry();
+                projectFactory = CanUseRunPropertiesForCscBuiltProgram(BuildLevel.None, cacheEntry) ? null : virtualCommand.CreateProjectInstance;
                 cachedRunProperties = cacheEntry?.Run;
             }
         }
@@ -163,6 +164,9 @@ public int Execute()
                 targetCommand.EnvironmentVariable(name, value);
             }
 
+            // Send telemetry about the run operation
+            SendRunTelemetry(launchSettings, virtualCommand);
+
             // Ignore Ctrl-C for the remainder of the command's execution
             Console.CancelKeyPress += (sender, e) => { e.Cancel = true; };
 
@@ -297,22 +301,23 @@ internal bool TryGetLaunchProfileSettingsIfNeeded(out ProjectLaunchSettingsModel
         }
     }
 
-    private void EnsureProjectIsBuilt(out Func<ProjectCollection, ProjectInstance>? projectFactory, out RunProperties? cachedRunProperties)
+    private void EnsureProjectIsBuilt(out Func<ProjectCollection, ProjectInstance>? projectFactory, out RunProperties? cachedRunProperties, out VirtualProjectBuildingCommand? virtualCommand)
     {
         int buildResult;
         if (EntryPointFileFullPath is not null)
         {
-            var command = CreateVirtualCommand();
-            buildResult = command.Execute();
-            projectFactory = CanUseRunPropertiesForCscBuiltProgram(command.LastBuild.Level, command.LastBuild.Cache?.PreviousEntry) ? null : command.CreateProjectInstance;
-            cachedRunProperties = command.LastBuild.Cache?.CurrentEntry.Run;
+            virtualCommand = CreateVirtualCommand();
+            buildResult = virtualCommand.Execute();
+            projectFactory = CanUseRunPropertiesForCscBuiltProgram(virtualCommand.LastBuild.Level, virtualCommand.LastBuild.Cache?.PreviousEntry) ? null : virtualCommand.CreateProjectInstance;
+            cachedRunProperties = virtualCommand.LastBuild.Cache?.CurrentEntry.Run;
         }
         else
         {
             Debug.Assert(ProjectFileFullPath is not null);
 
             projectFactory = null;
             cachedRunProperties = null;
+            virtualCommand = null;
             buildResult = new RestoringCommand(
                 MSBuildArgs.CloneWithExplicitArgs([ProjectFileFullPath, .. MSBuildArgs.OtherMSBuildArgs]),
                 NoRestore,
@@ -753,4 +758,135 @@ public static ParseResult ModifyParseResultForShorthandProjectOption(ParseResult
         var newParseResult = Parser.Parse(tokensToParse);
         return newParseResult;
     }
+
+    /// <summary>
+    /// Sends telemetry about the run operation.
+    /// </summary>
+    private void SendRunTelemetry(
+        ProjectLaunchSettingsModel? launchSettings,
+        VirtualProjectBuildingCommand? virtualCommand)
+    {
+        try
+        {
+            if (virtualCommand != null)
+            {
+                SendFileBasedTelemetry(launchSettings, virtualCommand);
+            }
+            else
+            {
+                SendProjectBasedTelemetry(launchSettings);
+            }
+        }
+        catch (Exception ex)
+        {
+            // Silently ignore telemetry errors to not affect the run operation
+            if (CommandLoggingContext.IsVerbose)
+            {
+                Reporter.Verbose.WriteLine($"Failed to send run telemetry: {ex}");
+            }
+        }
+    }
+
+    /// <summary>
+    /// Builds and sends telemetry data for file-based app runs.
+    /// </summary>
+    private void SendFileBasedTelemetry(
+        ProjectLaunchSettingsModel? launchSettings,
+        VirtualProjectBuildingCommand virtualCommand)
+    {
+        Debug.Assert(EntryPointFileFullPath != null);
+        var projectIdentifier = RunTelemetry.GetFileBasedIdentifier(EntryPointFileFullPath, Sha256Hasher.Hash);
+
+        var directives = virtualCommand.Directives;
+        var sdkCount = RunTelemetry.CountSdks(directives);
+        var packageReferenceCount = RunTelemetry.CountPackageReferences(directives);
+        var projectReferenceCount = RunTelemetry.CountProjectReferences(directives);
+        var additionalPropertiesCount = RunTelemetry.CountAdditionalProperties(directives);
+
+        RunTelemetry.TrackRunEvent(
+            isFileBased: true,
+            projectIdentifier: projectIdentifier,
+            launchProfile: LaunchProfile,
+            noLaunchProfile: NoLaunchProfile,
+            launchSettings: launchSettings,
+            sdkCount: sdkCount,
+            packageReferenceCount: packageReferenceCount,
+            projectReferenceCount: projectReferenceCount,
+            additionalPropertiesCount: additionalPropertiesCount,
+            usedMSBuild: virtualCommand.LastBuild.Level is BuildLevel.All,
+            usedRoslynCompiler: virtualCommand.LastBuild.Level is BuildLevel.Csc);
+    }
+
+    /// <summary>
+    /// Builds and sends telemetry data for project-based app runs.
+    /// </summary>
+    private void SendProjectBasedTelemetry(ProjectLaunchSettingsModel? launchSettings)
+    {
+        Debug.Assert(ProjectFileFullPath != null);
+        var projectIdentifier = RunTelemetry.GetProjectBasedIdentifier(ProjectFileFullPath, GetRepositoryRoot(), Sha256Hasher.Hash);
+        
+        // Get package and project reference counts for project-based apps
+        int packageReferenceCount = 0;
+        int projectReferenceCount = 0;
+        
+        // Try to get project information for telemetry if we built the project
+        if (ShouldBuild)
+        {
+            try
+            {
+                var globalProperties = MSBuildArgs.GlobalProperties?.ToDictionary() ?? new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
+                globalProperties[Constants.EnableDefaultItems] = "false";
+                globalProperties[Constants.MSBuildExtensionsPath] = AppContext.BaseDirectory;
+                
+                using var collection = new ProjectCollection(globalProperties: globalProperties);
+                var project = collection.LoadProject(ProjectFileFullPath).CreateProjectInstance();
+                
+                packageReferenceCount = RunTelemetry.CountPackageReferences(project);
+                projectReferenceCount = RunTelemetry.CountProjectReferences(project);
+            }
+            catch
+            {
+                // If project evaluation fails for telemetry, use defaults
+                // We don't want telemetry collection to affect the run operation
+            }
+        }
+
+        RunTelemetry.TrackRunEvent(
+            isFileBased: false,
+            projectIdentifier: projectIdentifier,
+            launchProfile: LaunchProfile,
+            noLaunchProfile: NoLaunchProfile,
+            launchSettings: launchSettings,
+            packageReferenceCount: packageReferenceCount,
+            projectReferenceCount: projectReferenceCount);
+    }
+
+    /// <summary>
+    /// Attempts to find the repository root directory.
+    /// </summary>
+    /// <returns>Repository root path if found, null otherwise</returns>
+    private string? GetRepositoryRoot()
+    {
+        try
+        {
+            var currentDir = ProjectFileFullPath != null 
+                ? Path.GetDirectoryName(ProjectFileFullPath)
+                : Directory.GetCurrentDirectory();
+                
+            while (currentDir != null)
+            {
+                if (Directory.Exists(Path.Combine(currentDir, ".git")))
+                {
+                    return currentDir;
+                }
+                currentDir = Directory.GetParent(currentDir)?.FullName;
+            }
+        }
+        catch
+        {
+            // Ignore errors when trying to find repo root
+        }
+        
+        return null;
+    }
 }