Add links to nuget.org only for packages actually available on nuget.org

Author: 0xced

CHANGELOG.md

@@ -4,6 +4,29 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased][Unreleased]
+
+* Links to nuget.org are now added only for packages which are actually available on the NuGet Gallery. Previously, a non-working link to nuget.org would be added for packages coming from private NuGet feeds.
+* Packages which are unavailable on the NuGet Gallery are now rendered in beige (moccasin), unless the `--no-links` options is specified.
+
+Before:
+
+```mermaid
+graph LR
+classDef default fill:aquamarine,stroke:#009061,color:#333333
+MyPrivatePackage --> Serilog
+```
+
+After:
+
+```mermaid
+graph LR
+classDef default fill:aquamarine,stroke:#009061,color:#333333
+classDef private fill:moccasin,stroke:#AF8844
+MyPrivatePackage --> Serilog
+class MyPrivatePackage private
+```
+
 ## [0.4.0][0.4.0] - 2025-06-26
 
 * Fixed an issue where a project reference (blue) could be wrongly identified as package reference (green)

src/Chisel

@@ -43,9 +43,10 @@ protected override async Task<int> ExecuteAsync(CommandContext commandContext, G
         var source = settings.Source ?? environment.CurrentWorkingDirectory;
         var graphUrl = await console.Status().StartAsync($"Generating dependency graph for {source}".EscapeMarkup(), async context =>
         {
+            var logger = new SpectreLogger(console, settings.LogLevel);
             var graph = await source.Match(
-                file => ComputeDependencyGraphAsync(file, settings, cancellationToken),
-                package => ComputeDependencyGraphAsync(package, settings, Settings.LoadDefaultSettings(settings.NuGetRoot), new SpectreLogger(console, settings.LogLevel), context, cancellationToken)
+                file => ComputeDependencyGraphAsync(file, settings, logger, cancellationToken),
+                package => ComputeDependencyGraphAsync(package, settings, Settings.LoadDefaultSettings(settings.NuGetRoot), logger, context, cancellationToken)
             );
             return await WriteGraphAsync(graph, settings);
         });
@@ -103,7 +104,7 @@ private static async Task<int> DiagnoseAsync(TextWriter stdOut, DirectoryInfo? s
         return result.ExitCode;
     }
 
-    private static async Task<DependencyGraph> ComputeDependencyGraphAsync(FileSystemInfo source, GraphCommandSettings settings, CancellationToken cancellationToken)
+    private static async Task<DependencyGraph> ComputeDependencyGraphAsync(FileSystemInfo source, GraphCommandSettings settings, ILogger logger, CancellationToken cancellationToken)
     {
         var name = Path.GetFileNameWithoutExtension(source.Name);
         if (settings.Title == GraphCommandSettings.DefaultTitle)
@@ -115,7 +116,12 @@ private static async Task<DependencyGraph> ComputeDependencyGraphAsync(FileSyste
         var lockFile = new LockFileFormat().Read(projectInfo.ProjectAssetsFile.FullName);
         Predicate<Package> filter = projectInfo.CopyLocalPackages.Count > 0 ? package => projectInfo.CopyLocalPackages.Contains(package.Name) : _ => true;
         var (packages, roots) = lockFile.ReadPackages(targetFramework.GetShortFolderName(), settings.RuntimeIdentifier, filter);
-        return new DependencyGraph(packages, roots, ignores: settings.GraphIgnore);
+        var dependencyGraph = new DependencyGraph(packages, roots, ignores: settings.GraphIgnore);
+        if (!settings.NoLinks)
+        {
+            await dependencyGraph.AddLinksAsync(logger, cancellationToken);
+        }
+        return dependencyGraph;
     }
 
     private static async Task<DependencyGraph> ComputeDependencyGraphAsync(PackageIdentity package, GraphCommandSettings settings, ISettings nugetSettings, ILogger logger, StatusContext context, CancellationToken cancellationToken)
@@ -126,7 +132,7 @@ private static async Task<DependencyGraph> ComputeDependencyGraphAsync(PackageId
             settings.Title = $"Dependency graph of {project.Package.Id} {project.Package.Version} ({project.TargetFramework.GetShortFolderName()})";
         }
         context.Status = $"Generating dependency graph for {project.Package.Id} {project.Package.Version} ({project.TargetFramework.GetShortFolderName()})".EscapeMarkup();
-        return await ComputeDependencyGraphAsync(project.File, settings, cancellationToken);
+        return await ComputeDependencyGraphAsync(project.File, settings, logger, cancellationToken);
     }
 
     private static async Task<Uri?> WriteGraphAsync(DependencyGraph graph, GraphCommandSettings settings)
@@ -143,7 +149,6 @@ private static async Task<DependencyGraph> ComputeDependencyGraphAsync(PackageId
             {
                 Direction = settings.GraphDirection,
                 Title = settings.Title,
-                IncludeLinks = !settings.NoLinks,
                 IncludeVersions = settings.GraphIncludeVersions,
                 WriteIgnoredPackages = settings.GraphWriteIgnoredPackages,
             };