Skip to content

feat: run comparison - merged graph & change summary logic - #2597

Open
camielvs wants to merge 1 commit into
cmp-01-diff-enginefrom
cmp-02-graph-summary-logic
Open

feat: run comparison - merged graph & change summary logic#2597
camielvs wants to merge 1 commit into
cmp-01-diff-enginefrom
cmp-02-graph-summary-logic

Conversation

@camielvs

@camielvs camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Second PR in the Compare Runs stack. Builds two more pieces of logic on top of the diff engine — still no UI.

  • Merged graph — takes the task/input/output diffs and turns them into a single combined graph that overlays both runs. Every task, input and output from either run becomes one node, and every connection is tagged as belonging to run A only, run B only, or both. This is what the Graph tab later renders.
  • Change summaries — small helpers that turn a task or input/output diff into a short human caption (e.g. "component · 2 arguments · cache disabled", or "source rewired") for use as subdued labels on graph nodes.

Related Issue and Pull requests

Part of the Compare Runs stack. Builds on #2596; the next PR (#2598) builds on this branch.

Type of Change

  • New feature

Checklist

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

Screenshots (if applicable)

Test Instructions

Pure logic, no standalone UI.

  • Run the unit tests: npm run test -- buildMergedGraph summarizeChange
  • Coverage includes unioning tasks from both runs into single nodes, tagging edges as A-only / B-only / shared, dropping edges to unknown tasks, wiring input→task→output edges, rewired outputs, and the wording of the change summaries.

To see it end-to-end, check out the top of the stack (#2603), enable the Compare runs flag, and open the Graph tab of a comparison.

Additional Comments

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: cmp-02-graph-summary-logic/10263af

camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from b799ace to dbfea62 Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from 8c0a61b to 2d2c65a Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from dbfea62 to ec52e0b Compare August 14, 2026 18:17
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from 2d2c65a to 115570a Compare August 14, 2026 18:17
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from ec52e0b to f4272b7 Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch 2 times, most recently from 160bb74 to cfcc893 Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from f4272b7 to 0577e9a Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from cfcc893 to 186b9c5 Compare August 14, 2026 23:47
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 0577e9a to 5f5b90c Compare August 14, 2026 23:47
Comment thread src/routes/v2/pages/CompareView/utils/summarizeChange.ts Outdated
Comment thread src/routes/v2/pages/CompareView/utils/summarizeChange.ts Outdated
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from 186b9c5 to ce6f309 Compare August 17, 2026 20:01
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch 2 times, most recently from 0fc0811 to 5f39e5d Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from ce6f309 to 06a5da8 Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 5f39e5d to d28657f Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from 06a5da8 to be013a9 Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-01-diff-engine branch from be013a9 to e7623f8 Compare August 18, 2026 00:56
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from d28657f to 10263af Compare August 18, 2026 00:56
@camielvs
camielvs marked this pull request as ready for review August 18, 2026 01:20
@camielvs
camielvs requested a review from a team as a code owner August 18, 2026 01:20
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