feat: Compare execution data, not just pipeline definitions (#2620) - #2641
Closed
camielvs wants to merge 1 commit into
Closed
feat: Compare execution data, not just pipeline definitions (#2620)#2641camielvs wants to merge 1 commit into
camielvs wants to merge 1 commit into
Conversation
Two runs of the same pipeline reported "no differences" even when they produced different artifacts, ended differently, or took wildly different amounts of time — the comparison only looked at the pipeline definition. - Outputs gain an artifact axis: each run's root execution artifacts are fetched (one request per run, regardless of pipeline width) and compared by content — inline value, size and directory-ness. Per-task artifacts stay lazy behind the task rows that ask for them. - `ioDisplayStatus()` keeps the spec diff and the artifact diff as separate fields, so an output whose declaration matched but whose artifact differs reads as changed in the counts row, the output node and the detail panel without corrupting the definition-level status. - Run metadata now compares overall status and duration. Both come from data already fetched: status from the polled graph state, duration from the root execution's status history. Duration only counts as a difference when the gap is both absolutely and proportionally large, so ordinary run-to-run jitter stays quiet, and `createdAt` still does not count. - Replaced the copy that overclaimed a match in the verdict box and the task detail panel.
🎩 PreviewA preview build has been created at: |
This was referenced Aug 17, 2026
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
camielvs
changed the base branch from
cmp-08-compare-page-and-wiring
to
graphite-base/2641
August 18, 2026 00:44
Collaborator
Author
|
This PR was create erroneously by Claude - closing. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Related Issue and Pull requests
Type of Change
Checklist
Screenshots (if applicable)
Test Instructions
Additional Comments