Skip to content

feat: run comparison - graph diff view - #2602

Draft
camielvs wants to merge 1 commit into
cmp-06-structured-diff-viewfrom
cmp-07-graph-diff-view
Draft

feat: run comparison - graph diff view#2602
camielvs wants to merge 1 commit into
cmp-06-structured-diff-viewfrom
cmp-07-graph-diff-view

Conversation

@camielvs

@camielvs camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Seventh PR in the Compare Runs stack. Adds the Graph view — a visual, side-by-side-in-one-canvas take on the comparison.

It renders the merged graph (from #2597) with colour-coded nodes and edges: green for added, red for removed, amber for changed, grey for unchanged, plus a legend. Nodes show each task's status, digest and a short change summary; clicking one opens a detail popover reusing the same task/IO diff detail from the Structured view. A Highlight A / B / Both control dims everything not present in the chosen run and switches nodes to show that run's actual values.

Also refactors the run view's status indicator to extract a reusable StatusTab so the merged task nodes can show per-run (A/B) status tabs.

Related Issue and Pull requests

Part of the Compare Runs stack. Builds on #2601; the final PR (#2603) builds on this branch.

Type of Change

  • New feature
  • Cleanup/Refactor

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

Reachable once the stack is wired up (#2603).

  • Check out the top of the stack, enable the Compare runs beta flag, and compare two runs.
  • On the Graph tab, confirm:
    • The merged graph lays out and node/edge colours match the legend (added / removed / changed / unchanged).
    • Clicking a node opens its detail popover; clicking the background closes it.
    • Highlight A / B / Both dims the other run and shows the highlighted run's values.
  • Regression check the refactor: open a normal run's graph and confirm task status indicators (including the cache-disabled state) still render as before.

Additional Comments

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: cmp-07-graph-diff-view/eefad45

@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch from e34d81d to 22f985a Compare July 30, 2026 23:22
@camielvs
camielvs force-pushed the cmp-06-structured-diff-view branch 2 times, most recently from f58787e to 6d8d1a9 Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch 2 times, most recently from a1c4ec9 to 69226d7 Compare July 31, 2026 01:05
@camielvs
camielvs force-pushed the cmp-06-structured-diff-view branch from 6d8d1a9 to 15d5b88 Compare July 31, 2026 01:05
@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch from 69226d7 to cf080ba Compare July 31, 2026 01:36
@camielvs
camielvs force-pushed the cmp-06-structured-diff-view branch from 15d5b88 to 16384ae Compare July 31, 2026 01:36
@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch 2 times, most recently from e6b562b to 659f92c Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-06-structured-diff-view branch 2 times, most recently from 1c31e0a to 63ee9b6 Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch from 659f92c to 3605a8d Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-07-graph-diff-view branch from 3605a8d to eefad45 Compare August 14, 2026 23:47
@camielvs
camielvs force-pushed the cmp-06-structured-diff-view branch from 63ee9b6 to e57e483 Compare August 14, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant