Improve link generation in renderer

Author: nyamsprod

src/Ide.php

@@ -7,6 +7,7 @@
 use function http_build_query;
 use function is_string;
 use function ltrim;
+use function str_replace;
 use function strtolower;
 
 enum Ide: string
@@ -27,7 +28,7 @@ public static function fromEnv(?string $default = null): self
         return self::tryFrom(strtolower((string) $value)) ?? self::PhpStorm;
     }
 
-    public function uri(CallLocation $location, ?string $project = null): string
+    public function uri(CallLocation $location, string $project = 'UNKNOWN_PROJECT'): string
     {
         return match ($this) {
             self::JetBrains => 'jetbrains://php-storm/navigate/reference?'.http_build_query(['project' => $project, 'path' => $location->path, 'line' => $location->line]),
@@ -38,7 +39,7 @@ public function uri(CallLocation $location, ?string $project = null): string
 
     public function path(CallLocation $location): string
     {
-        return $location->path.':'.$location->line;
+        return str_replace('\\', '/', (string) $location->path).':'.$location->line;
     }
 
     public function link(CallLocation $location, ?string $project = null, AnsiStyle ...$styles): string
@@ -48,6 +49,6 @@ public function link(CallLocation $location, ?string $project = null, AnsiStyle
             return $path;
         }
 
-        return '<a class="'.AnsiStyle::inlineClasses(...$styles).'" href="'.$this->uri($location, $project).'">'.$path.'</a>';
+        return '<a class="'.AnsiStyle::inlineClasses(...$styles).'" href="'.$this->uri($location, $project ?? 'UNKNOWN_PROJECT').'">'.$path.'</a>';
     }
 }

src/Renderer.php

@@ -191,7 +191,7 @@ private function renderHeader(?Profile $profile, ?CallLocation $callLocation): s
         $header[] = '<p class="bkm-sw-header">';
         if (null !== $callLocation) {
             $header[] = AnsiStyle::wrapHtml('class', 'Path: ', AnsiStyle::BrightGreen);
-            $header[] = Ide::fromEnv()->link($callLocation, null, AnsiStyle::White);
+            $header[] = Ide::fromEnv()->link($callLocation, null, AnsiStyle::BrightCyan, AnsiStyle::Bold);
             $header[] = '<br>';
         }
 

src/ViewExporter.php

@@ -65,7 +65,12 @@ private function exportLeaderPrinter(array $data): void
 
     public function exportSnapshot(Snapshot $snapshot): void
     {
-        $this->exportLeaderPrinter($snapshot->toHuman());
+        $data = $snapshot->toHuman();
+        $location = new CallLocation($snapshot->originPath, $snapshot->originLine);
+        unset($data['origin_path'], $data['origin_line']);
+        $data['call_location'] = Ide::fromEnv()->link($location, null, AnsiStyle::BrightCyan, AnsiStyle::Bold);
+
+        $this->exportLeaderPrinter($data);
     }
 
     public function exportMetrics(Result|Span|Metrics $metrics): void
@@ -86,10 +91,11 @@ public function exportSpan(Result|Span $span): void
             Span::class => $span,
         };
 
+        $ide = Ide::fromEnv();
         $this->exportLeaderPrinter([
             'label' => $source->label,
-            'call_location_start' => Ide::fromEnv()->link($source->range->start, null, AnsiStyle::White),
-            'call_location_end' => Ide::fromEnv()->link($source->range->end, null, AnsiStyle::White),
+            'call_location_start' => $ide->link($source->range->start, null, AnsiStyle::BrightCyan, AnsiStyle::Bold),
+            'call_location_end' => $ide->link($source->range->end, null, AnsiStyle::BrightCyan, AnsiStyle::Bold),
         ]);
         $this->exportMetrics($source->metrics);
     }
@@ -211,7 +217,7 @@ public function exportTimeline(Timeline $timeline, ?callable $filter = null): vo
             $data = $snapshot->toHuman();
             $tableRenderer->addRow([
                 $data['label'],
-                $data['origin_path'].':'.$data['origin_line'],
+                Ide::fromEnv()->link(new CallLocation($data['origin_path'], (int) $data['origin_line']), 'UNKNOWN_PROJECT', AnsiStyle::Bold, AnsiStyle::BrightCyan),
                 $snapshot->timestamp->format('U.u'),
                 DurationUnit::format($snapshot->cpuUserTime + $snapshot->cpuSystemTime, 3),
                 $data['memory_usage'],