fix(inference): link single-run changelog to its producing run, not the date (#464)

The Config Changelog's single-run date block rendered date-level Git Commit
and Workflow Run links (`data.changelogs.at(-1).head_ref` and
`data.runs.at(-1).html_url`): the last entry/run on the calendar date,
regardless of config. When two unrelated PRs merged the same day, a config
whose data came from the earlier run showed the later run's commit/run links
next to its own correct changelog description.

Drive the links off the run that actually produced the selected config's data
(the sole `dataRunsForDate` entry, the same source the multi-run branch uses)
and drop the date-level `headRef`/`runUrl` fields entirely so the bad
attribution cannot recur. Both branches now share a `renderRunLinks` helper,
and the single-run separator renders only when a link exists (no dangling
em-dash when a changelog entry has no matching benchmark run).

Fixes #408

Author: eicherseiji

packages/app/src/components/inference/ui/ComparisonChangelog.tsx

@@ -16,6 +16,36 @@ import { dataRunsForDate } from '@/components/inference/utils/runEnumeration';
 import { getHardwareConfig } from '@/lib/constants';
 import { getDisplayLabel, updateRepoUrl } from '@/lib/utils';
 
+/** Git Commit and Workflow Run external links for a run, each shown when known. */
+function renderRunLinks(headRef?: string, runUrl?: string) {
+  return (
+    <>
+      {headRef && (
+        <a
+          href={`https://github.com/SemiAnalysisAI/InferenceX/commit/${headRef}`}
+          target="_blank"
+          rel="noopener noreferrer"
+          className="text-sm hover:underline text-foreground underline"
+        >
+          Git Commit
+          <ExternalLinkIcon />
+        </a>
+      )}
+      {runUrl && (
+        <a
+          href={updateRepoUrl(runUrl)}
+          target="_blank"
+          rel="noopener noreferrer"
+          className="text-sm hover:underline text-foreground underline"
+        >
+          Workflow Run
+          <ExternalLinkIcon />
+        </a>
+      )}
+    </>
+  );
+}
+
 /** One changelog entry's description. The GPU and run # are shown in the entry title. */
 function renderDescription(
   entry: { config_keys: string[]; description: string },
@@ -262,28 +292,7 @@ export default function ComparisonChangelog({
                           {gpuLabel ? ` ${gpuLabel}` : ''} #{idx + 1}
                         </span>
                         <span className="text-muted-foreground">&mdash;</span>
-                        {run.headRef && (
-                          <a
-                            href={`https://github.com/SemiAnalysisAI/InferenceX/commit/${run.headRef}`}
-                            target="_blank"
-                            rel="noopener noreferrer"
-                            className="text-sm hover:underline text-foreground underline"
-                          >
-                            Git Commit
-                            <ExternalLinkIcon />
-                          </a>
-                        )}
-                        {run.runUrl && (
-                          <a
-                            href={updateRepoUrl(run.runUrl)}
-                            target="_blank"
-                            rel="noopener noreferrer"
-                            className="text-sm hover:underline text-foreground underline"
-                          >
-                            Workflow Run
-                            <ExternalLinkIcon />
-                          </a>
-                        )}
+                        {renderRunLinks(run.headRef, run.runUrl)}
                         {onChart ? (
                           <button
                             type="button"
@@ -328,7 +337,11 @@ export default function ComparisonChangelog({
                 });
               }
 
-              // Single (or no) matching run → one block keyed by the date.
+              // Single (or no) matching run → one block keyed by the date. Link to
+              // the run that produced this config's data so a date with unrelated
+              // same-day runs never borrows another run's commit/run links. No
+              // matching run (e.g. a pinned endpoint) → no links.
+              const { headRef, runUrl } = runs[0] ?? {};
               const dateGpuLabel = gpuLabelsFor(item.entries);
               return (
                 <div key={item.date} className="flex flex-col gap-1">
@@ -337,31 +350,10 @@ export default function ComparisonChangelog({
                       {item.date}
                       {dateGpuLabel ? ` ${dateGpuLabel}` : ''}
                     </span>
-                    {item.entries.length > 0 && (
+                    {item.entries.length > 0 && (headRef || runUrl) && (
                       <>
                         <span className="text-muted-foreground">&mdash;</span>
-                        {item.headRef && (
-                          <a
-                            href={`https://github.com/SemiAnalysisAI/InferenceX/commit/${item.headRef}`}
-                            target="_blank"
-                            rel="noopener noreferrer"
-                            className="text-sm hover:underline text-foreground underline"
-                          >
-                            Git Commit
-                            <ExternalLinkIcon />
-                          </a>
-                        )}
-                        {item.runUrl && (
-                          <a
-                            href={updateRepoUrl(item.runUrl)}
-                            target="_blank"
-                            rel="noopener noreferrer"
-                            className="text-sm hover:underline text-foreground underline"
-                          >
-                            Workflow Run
-                            <ExternalLinkIcon />
-                          </a>
-                        )}
+                        {renderRunLinks(headRef, runUrl)}
                       </>
                     )}
                     {datesOnChart.has(item.date) ? (

packages/app/src/components/inference/utils/runEnumeration.test.ts

@@ -88,6 +88,8 @@ describe('dataRunsForDate', () => {
   });
 
   it('carries run url and head sha through', () => {
+    // The single-run changelog block links off these per-run fields (#408), so a
+    // date with unrelated same-day runs cannot borrow another run's commit/run links.
     const rows = [
       rc({
         github_run_id: 7,

packages/app/src/hooks/api/use-comparison-changelogs.ts

@@ -26,8 +26,6 @@ export interface ComparisonRun {
 
 export interface ComparisonChangelog {
   date: string;
-  headRef?: string;
-  runUrl?: string;
   /** All of the date's changelog entries (flattened across runs). */
   entries: ComparisonChangelogEntry[];
   /** Individual runs on this date, in chronological order (earliest first). */
@@ -117,8 +115,6 @@ export function useComparisonChangelogs(
 
       results.push({
         date: datesToQuery[i],
-        headRef: data.changelogs.at(-1)?.head_ref,
-        runUrl: data.runs.at(-1)?.html_url ?? undefined,
         entries: data.changelogs.map((c: ChangelogRow) => ({
           config_keys: c.config_keys,
           description: c.description,