DYN-4687 - docs browser and docs generator produce .md files with names that are too long.

.gitignore

@@ -112,3 +112,4 @@ logo.png
 /src/DynamoCoreWpf/package-lock.json
 /src/DynamoCoreWpf/package.json
 /src/DynamoCoreWpf/.npm-cache
+/src/DynamoSandbox/node_modules

src/DocumentationBrowserViewExtension/DocumentationBrowserViewModel.cs

@@ -86,6 +86,7 @@ private set
         private Uri link;
         private string graphPath;
         private string content;
+        private string name;
 
         private MarkdownHandler MarkdownHandlerInstance => markdownHandler ?? (markdownHandler = new MarkdownHandler());
         public bool HasContent => !string.IsNullOrWhiteSpace(this.content);
@@ -184,6 +185,7 @@ private void HandleLocalResource(OpenDocumentationLinkEventArgs e)
             {
                 string targetContent;
                 string graph;
+                string graphName;
                 Uri link;
                 switch (e)
                 {
@@ -195,19 +197,22 @@ private void HandleLocalResource(OpenDocumentationLinkEventArgs e)
                         link = string.IsNullOrEmpty(mdLink) ? new Uri(String.Empty, UriKind.Relative) : new Uri(mdLink);
                         graph = GetGraphLinkFromMDLocation(link);
                         targetContent = CreateNodeAnnotationContent(openNodeAnnotationEventArgs);
+                        graphName = openNodeAnnotationEventArgs.MinimumQualifiedName;
                         break;
 
                     case OpenDocumentationLinkEventArgs openDocumentationLink:
                         link = openDocumentationLink.Link;
                         graph = GetGraphLinkFromMDLocation(link);
                         targetContent = ResourceUtilities.LoadContentFromResources(openDocumentationLink.Link.ToString(), GetType().Assembly);
+                        graphName = null;
                         break;
 
                     default:
                         // Navigate to unsupported 
                         targetContent = null;
                         graph = null;
                         link = null;
+                        graphName = null;
                         break;
                 }
 
@@ -220,6 +225,7 @@ private void HandleLocalResource(OpenDocumentationLinkEventArgs e)
                     this.content = targetContent;
                     this.Link = link;
                     this.graphPath = graph;
+                    this.name = graphName;
                 }
             }
             catch (FileNotFoundException)
@@ -358,7 +364,8 @@ internal void InsertGraph()
             {
                 if (graphPath != null)
                 {
-                    raiseInsertGraph(this, new InsertDocumentationLinkEventArgs(graphPath, Path.GetFileNameWithoutExtension(graphPath)));
+                    var graphName = this.name ?? Path.GetFileNameWithoutExtension(graphPath);
+                    raiseInsertGraph(this, new InsertDocumentationLinkEventArgs(graphPath, graphName));
                 }
                 else
                 {

src/DocumentationBrowserViewExtension/PackageDocumentationManager.cs

@@ -5,6 +5,7 @@
 using System.Threading;
 using Dynamo.Interfaces;
 using Dynamo.Logging;
+using Dynamo.Utilities;
 
 namespace Dynamo.DocumentationBrowser
 {
@@ -95,10 +96,13 @@ public string GetAnnotationDoc(string nodeNamespace, string packageName)
                 return output;
             }
 
+            var shortName = Hash.GetHashFilenameFromString(nodeNamespace);
+
             FileInfo matchingDoc = null;
             if (hostDynamoFallbackDocPath != null)
             {
-                matchingDoc = hostDynamoFallbackDocPath.GetFiles($"{nodeNamespace}.md").FirstOrDefault();
+                matchingDoc = hostDynamoFallbackDocPath.GetFiles($"{shortName}.md").FirstOrDefault() ??
+                              hostDynamoFallbackDocPath.GetFiles($"{nodeNamespace}.md").FirstOrDefault();
                 if (matchingDoc != null)
                 {
                     return matchingDoc.FullName;
@@ -107,7 +111,8 @@ public string GetAnnotationDoc(string nodeNamespace, string packageName)
 
             if (dynamoCoreFallbackDocPath != null)
             {
-                matchingDoc = dynamoCoreFallbackDocPath.GetFiles($"{nodeNamespace}.md").FirstOrDefault();
+                matchingDoc = dynamoCoreFallbackDocPath.GetFiles($"{shortName}.md").FirstOrDefault() ??
+                              dynamoCoreFallbackDocPath.GetFiles($"{nodeNamespace}.md").FirstOrDefault();
             }
 
             return matchingDoc is null ? string.Empty : matchingDoc.FullName;

src/DynamoUtilities/Hash.cs

@@ -0,0 +1,38 @@
+using System.Security.Cryptography;
+using System.Text;
+
+namespace Dynamo.Utilities
+{
+    internal class Hash
+    {
+        internal static byte[] GetHash(byte[] bytes)
+        {
+            using (var hashAlgorithm = HashAlgorithm.Create())
+            {
+                return hashAlgorithm.ComputeHash(bytes);
+            }
+        }
+
+        internal static byte[] GetHashFromString(string str)
+        {
+            var bytes = Encoding.UTF8.GetBytes(str);
+            return GetHash(bytes);
+        }
+
+        internal static string GetFilenameFromHash(byte[] bytes)
+        {
+            var builder = new StringBuilder();
+            foreach (var b in bytes)
+            {
+                builder.AppendFormat(@"{0:x2}", b);
+            }
+            return builder.ToString();
+        }
+
+        internal static string GetHashFilenameFromString(string str)
+        {
+            var hash = GetHashFromString(str);
+            return GetFilenameFromHash(hash);
+        }
+    }
+}

src/DynamoUtilities/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 
@@ -20,4 +20,5 @@
 [assembly: InternalsVisibleTo("DocumentationBrowserViewExtension")]
 [assembly: InternalsVisibleTo("DynamoPackages")]
 [assembly: InternalsVisibleTo("ProtoScript")]
-[assembly: InternalsVisibleTo("ProtoCore")]
+[assembly: InternalsVisibleTo("ProtoCore")]
+[assembly: InternalsVisibleTo("NodeDocumentationMarkdownGenerator")]

src/Tools/NodeDocumentationMarkdownGenerator/CommandHandler.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Text;
 using Dynamo.Logging;
 using NodeDocumentationMarkdownGenerator.Commands;
@@ -36,6 +36,20 @@ internal static string HandleFromDirectory(FromDirectoryOptions opts)
             return "";
         }
 
+        internal static string HandleRename(RenameOptions opts)
+        {
+            try
+            {
+                RenameCommand.HandleRename(opts);
+            }
+            catch (Exception e)
+            {
+                LogExceptionToConsole(e);
+            }
+
+            return "";
+        }
+
         internal static void LogExceptionToConsole(Exception e)
         {
             var strBuilder = new StringBuilder();
@@ -76,4 +90,4 @@ public void LogWarning(string warning, WarningLevel level)
             throw new NotImplementedException();
         }
     }
-}
+}

src/Tools/NodeDocumentationMarkdownGenerator/Commands/RenameCommand.cs

@@ -0,0 +1,90 @@
+using System;
+using System.IO;
+using System.Linq;
+using NodeDocumentationMarkdownGenerator.Verbs;
+
+namespace NodeDocumentationMarkdownGenerator.Commands
+{
+    internal static class RenameCommand
+    {
+        internal static void HandleRename(RenameOptions opts)
+        {
+            if (opts.InputMdFile is null && opts.InputMdDirectory != null)
+            {
+                RenameDirectory(opts.InputMdDirectory, opts.MaxLength);
+            }
+            else if (opts.InputMdFile != null && opts.InputMdDirectory is null)
+            {
+                RenameFile(opts.InputMdFile);
+            }
+            else
+            {
+                Console.WriteLine("Invalid options: You can rename a single file using the file option\nor renaming multiple files in a directory (if they are longer than max length)\nusing the directory option");
+            }
+        }
+
+        private static void RenameFile(string file)
+        {
+            var extension = Path.GetExtension(file);
+            if (!extension.Equals(".md", StringComparison.InvariantCultureIgnoreCase))
+            {
+                Console.WriteLine($"Can only rename MD files: {file}");
+                return;
+            }
+
+            if (!File.Exists(file))
+            {
+                Console.WriteLine($"File not found: {file}");
+                return;
+            }
+
+            var baseName = Path.GetFileNameWithoutExtension(file);
+            var shortName = Dynamo.Utilities.Hash.GetHashFilenameFromString(baseName);
+
+            RenameFile(file, baseName, shortName);
+        }
+
+        private static void RenameFile(string file, string baseName, string shortName)
+        {
+            var content = File.ReadAllText(file);
+            content = content.Replace(baseName, shortName);
+            var path = Path.GetDirectoryName(file);
+            var newFile = Path.Combine(path, shortName + ".md");
+            File.WriteAllText(newFile, content);
+            File.Delete(file);
+
+            var allSupportFiles = Directory.GetFiles(path, baseName + ".*", SearchOption.TopDirectoryOnly)
+                .Select(x => new FileInfo(x)).ToList();
+            allSupportFiles.AddRange(Directory.GetFiles(path, baseName + "_img.*", SearchOption.TopDirectoryOnly)
+                .Select(x => new FileInfo(x)).ToList());
+
+            foreach (var supportFile in allSupportFiles)
+            {
+                var newName = Path.Combine(supportFile.DirectoryName,
+                    supportFile.Name.Replace(baseName, shortName));
+                supportFile.MoveTo(newName);
+            }
+        }
+
+        private static void RenameDirectory(string directory, int maxLength)
+        {
+            if (!Directory.Exists(directory))
+            {
+                Console.WriteLine($"Directory not found: {directory}");
+                return;
+            }
+
+            var allMdFiles = Directory.GetFiles(directory, "*.md", SearchOption.TopDirectoryOnly).Select(x => new FileInfo(x)).ToList();
+
+            foreach (var mdFile in allMdFiles)
+            {
+                if (mdFile.Name.Length > maxLength)
+                {
+                    var baseName = Path.GetFileNameWithoutExtension(mdFile.Name);
+                    var shortName = Dynamo.Utilities.Hash.GetHashFilenameFromString(baseName);
+                    RenameFile(mdFile.FullName, baseName, shortName);
+                }
+            }
+        }
+    }
+}

src/Tools/NodeDocumentationMarkdownGenerator/Program.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.IO;
@@ -50,11 +50,12 @@ internal static void Main(string[] args)
 
             ShowWelcomeMessages();
 
-            var result = Parser.Default.ParseArguments<FromDirectoryOptions, FromPackageOptions>(args);
+            var result = Parser.Default.ParseArguments<FromDirectoryOptions, FromPackageOptions, RenameOptions>(args);
             var text = result
                 .MapResult(
                     (FromDirectoryOptions opts) => CommandHandler.HandleFromDirectory(opts),
                     (FromPackageOptions opts) => CommandHandler.HandleFromPackage(opts),
+                    (RenameOptions opts) => CommandHandler.HandleRename(opts),
                     err => "1");
             Console.WriteLine($"docs generation tool {sw.Elapsed.TotalSeconds}");
 # if DEBUG

src/Tools/NodeDocumentationMarkdownGenerator/Verbs/RenameOptions.cs

@@ -0,0 +1,25 @@
+using CommandLine.Text;
+using CommandLine;
+using System.Collections.Generic;
+
+namespace NodeDocumentationMarkdownGenerator.Verbs
+{
+    [Verb("rename", HelpText = "Renaming utilities for fallback MD files")]
+    internal class RenameOptions
+    {
+        [Option('f', "file", HelpText = "Input MD file", Required = false)]
+        public string InputMdFile { get; set; }
+        [Option('d', "directory", HelpText = "Input directory", Required = false)]
+        public string InputMdDirectory { get; set; }
+        [Option('m', "maxlength", HelpText = "Max length before renaming to a shorter name", Required = false, Default = 50)]
+        public int MaxLength { get; set; }
+        [Usage(ApplicationAlias = "Dynamo docs generator")]
+        public static IEnumerable<Example> Examples
+        {
+            get
+            {
+                yield return new Example("Generate docs from package folder", new RenameOptions());
+            }
+        }
+    }
+}