Skip to content

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

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

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/5f5b90c

@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
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