[DYN-3343] Automatically add loaded packages as crash detail

src/DynamoCoreWpf/UI/Prompts/CrashPrompt.xaml.cs

@@ -1,11 +1,15 @@
-using System.Diagnostics;
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
 using System.IO;
+using System.Linq;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Documents;
 using System.Windows.Markup;
 using System.Xml;
 using Dynamo.Core;
+using Dynamo.PackageManager;
 using Dynamo.ViewModels;
 
 namespace Dynamo.Nodes.Prompts
@@ -18,17 +22,18 @@ public partial class CrashPrompt : Window
         private string details; // Store here for clipboard copy
         private string folderPath;
         private string productName;
+        private string markdownPackages;
 
         public CrashPrompt(string details, DynamoViewModel dynamoViewModel)
         {
             InitializeComponent();
             this.CrashDetailsContent.Text = details;
 
             productName = dynamoViewModel.BrandingResourceProvider.ProductName;
-            Title = string.Format(Wpf.Properties.Resources.CrashPromptDialogTitle,productName);
+            Title = string.Format(Wpf.Properties.Resources.CrashPromptDialogTitle, productName);
             txtOverridingText.Text = string.Format(Wpf.Properties.Resources.CrashPromptDialogCrashMessage, productName);
         }
-        
+
         public CrashPrompt(DynamoViewModel dynamoViewModel)
         {
             InitializeComponent();
@@ -42,6 +47,9 @@ public CrashPrompt(CrashPromptArgs args, DynamoViewModel dynamoViewModel)
         {
             InitializeComponent();
 
+            var packageLoader = dynamoViewModel.Model.GetPackageManagerExtension()?.PackageLoader;
+            markdownPackages = Wpf.Utilities.CrashUtilities.PackagesToMakrdown(packageLoader);
+
             productName = dynamoViewModel.BrandingResourceProvider.ProductName;
             Title = string.Format(Wpf.Properties.Resources.CrashPromptDialogTitle, productName);
             txtOverridingText.Text = string.Format(Wpf.Properties.Resources.CrashPromptDialogCrashMessage, productName);
@@ -92,7 +100,7 @@ private void Continue_Click(object sender, RoutedEventArgs e)
 
         private void PostOnGithub_Click(object sender, RoutedEventArgs e)
         {
-            DynamoViewModel.ReportABug(this.CrashDetailsContent.Text);
+            DynamoViewModel.ReportABug(markdownPackages);
         }
 
         private void Details_Click(object sender, RoutedEventArgs e)

src/DynamoCoreWpf/Utilities/CrashUtilities.cs

@@ -1,6 +1,9 @@
 using Dynamo.Configuration;
+using Dynamo.PackageManager;
 using Dynamo.Utilities;
 using System;
+using System.Collections.Generic;
+using System.Linq;
 
 namespace Dynamo.Wpf.Utilities
 {
@@ -26,28 +29,61 @@ internal static string GithubNewIssueUrlFromCrashContent(object crashContent)
             return baseUri.ToString();
         }
 
+        /// <summary>
+        /// Converts packages information into markdown format for use in Github issue body
+        /// </summary>
+        /// <param name="packageLoader">Package loader</param>
+        /// <returns>A markdown format string to use in issue body.</returns>
+        internal static string PackagesToMakrdown(PackageLoader packageLoader)
+        {
+            if (packageLoader != null)
+            {
+                //List of the names of all the loaded packages
+                var packagesNames = packageLoader.LocalPackages.Select(o => o.Name + " " + o.VersionName).ToList();
+                //Package's issue section in markdown format
+                string markdownText;
+                if (packagesNames.Any())
+                {
+                    markdownText = "### Loaded Packages" + Environment.NewLine;
+                    foreach (var name in packagesNames)
+                    {
+                        markdownText += "- " + name + Environment.NewLine;
+                    }
+                }
+                else
+                {
+                    markdownText = "No loaded packages were found.";
+                }
+                return markdownText;
+            }
+            else
+            {
+                return "(Fill in here)";
+            }
+        }
+
         /// <summary>
         /// Formats crash details and adds metadata for use in Github issue body
         /// </summary>
         /// <param name="details">Crash details, such as a stack trace.</param>
         /// <returns>A formatted, but not escaped, string to use as issue body.</returns>
         private static string GitHubCrashReportBody(object details)
         {
-            var stackTrace = details?.ToString() ?? string.Empty;
+            var markdownPackages = details?.ToString() ?? string.Empty;
 
             // This functionality was not available prior to version 2.1.0, so it should be the fallback value
             var dynamoVersion = AssemblyHelper.GetDynamoVersion().ToString() ?? "2.1.0+";
 
-            return BuildMarkdownContent(dynamoVersion, stackTrace);
+            return BuildMarkdownContent(dynamoVersion, markdownPackages);
         }
 
         /// <summary>
         /// Builds a Markdown string that will be posted to our new GitHub issue
         /// </summary>
         /// <param name="dynamoVersion">Dynamo version that should be recorded in the issue report</param>
-        /// <param name="stackTrace">The crash stack trace to be included in the issue report</param>
+        /// <param name="markdownPackages">Section of the issue with the loaded packages in markdown format</param>
         /// <returns></returns>
-        internal static string BuildMarkdownContent(string dynamoVersion, string stackTrace)
+        internal static string BuildMarkdownContent(string dynamoVersion, string markdownPackages)
         {
             return
                 "# Issue Description" + Environment.NewLine +
@@ -70,7 +106,7 @@ internal static string BuildMarkdownContent(string dynamoVersion, string stackTr
                 "(Fill in here)" + Environment.NewLine + Environment.NewLine +
 
                 "## What packages or external references (if any) were used?" + Environment.NewLine +
-                "(Fill in here)" + Environment.NewLine + Environment.NewLine + Environment.NewLine +
+                markdownPackages + Environment.NewLine + Environment.NewLine +
 
                 "## Stack Trace" + Environment.NewLine +
                 "```" + Environment.NewLine +

test/DynamoCoreWpfTests/CrashReportingTests.cs

@@ -42,6 +42,11 @@ at System.Windows.Application.RunDispatcher(Object ignore)
             at System.Windows.Application.RunInternal(Window window)
             at DynamoSandbox.DynamoCoreSetup.RunApplication(Application app)";
 
+        // This are example packages names used to simulate loaded packages information
+        private string Packages = @"
+            - Package Example A
+            - Package Example B";
+
         [Test]
         [Ignore("Test ignored because the web browser is not closed after the test execution")]
         public void CanReportBugWithNoContent()
@@ -66,7 +71,7 @@ public void CanReportBugWithContent()
             var dynamoVersion = "2.1.0";
 
             // Create a crash report to submit
-            var crashReport = Wpf.Utilities.CrashUtilities.BuildMarkdownContent(dynamoVersion, StackTrace);
+            var crashReport = Wpf.Utilities.CrashUtilities.BuildMarkdownContent(dynamoVersion, Packages);
             Assert.IsNotNullOrEmpty(crashReport);
 
             // Report a bug with a stack trace
@@ -80,7 +85,7 @@ public void StackTraceIncludedInReport()
             var dynamoVersion = AssemblyHelper.GetDynamoVersion().ToString();
 
             // Create a crash report to submit
-            var crashReport = Wpf.Utilities.CrashUtilities.BuildMarkdownContent(dynamoVersion, StackTrace);
+            var crashReport = Wpf.Utilities.CrashUtilities.BuildMarkdownContent(dynamoVersion, Packages);
             Assert.IsNotNullOrEmpty(crashReport);
 
             // Mock url for request