Skip to content

feat: run comparison - artifact comparison - #2599

Open
camielvs wants to merge 1 commit into
cmp-03-diff-ui-primitivesfrom
cmp-04-artifact-comparison
Open

feat: run comparison - artifact comparison#2599
camielvs wants to merge 1 commit into
cmp-03-diff-ui-primitivesfrom
cmp-04-artifact-comparison

Conversation

@camielvs

@camielvs camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Fourth PR in the Compare Runs stack. Adds artifact comparison — the ability to see how a task's output artifacts differ between the two runs.

For each output artifact it shows whether it was added, removed, changed or unchanged. Small inline values are shown directly side by side; larger artifacts show their type and size, with a Compare button that opens a full-screen dialog previewing both runs' versions next to each other (reusing the existing artifact preview components). Artifacts that are too large or can't be previewed are handled gracefully.

Not wired into a page yet — the later task diff view consumes this.

Related Issue and Pull requests

Part of the Compare Runs stack. Builds on #2598; the next PR (#2600) 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

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 that both produced output artifacts on the same task.
  • Expand a task and confirm:
    • Artifacts show the correct Added / Removed / Changed / Unchanged status.
    • Small inline values are shown side by side for both runs.
    • Larger/previewable artifacts show a Compare button that opens the dialog with both runs' previews.
    • Non-previewable or oversized artifacts show a sensible message instead of erroring.

Additional Comments

camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: cmp-04-artifact-comparison/d39ca43

@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 18500a7 to 4370669 Compare July 30, 2026 23:22
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from bbd75eb to 6d71a83 Compare July 30, 2026 23:22
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 4370669 to cb9779a Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch 2 times, most recently from 1b84110 to 378e505 Compare July 31, 2026 01:36
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 378e505 to a10c24a Compare August 14, 2026 18:17
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch 2 times, most recently from 45ac2fb to b881d15 Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch 2 times, most recently from 3508beb to 744e5fb Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from b881d15 to 1f63058 Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 744e5fb to fbae585 Compare August 14, 2026 23:47
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 1f63058 to 8a70f0f Compare August 14, 2026 23:47
Comment thread src/routes/v2/pages/CompareView/components/ArtifactDiffSection.tsx Outdated
Comment thread src/routes/v2/pages/CompareView/components/ArtifactDiffSection.tsx Outdated
Comment thread src/routes/v2/pages/CompareView/components/ArtifactDiffSection.tsx
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 8a70f0f to 46feccf Compare August 17, 2026 20:01
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from fbae585 to 5095ff0 Compare August 17, 2026 20:01
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 5095ff0 to df8cad2 Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 46feccf to 907eae2 Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from df8cad2 to ecc36ff Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 907eae2 to 1a21fb6 Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from ecc36ff to 556f7e8 Compare August 18, 2026 00:56
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 1a21fb6 to 0440072 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
Comment thread src/routes/v2/pages/CompareView/hooks/useRunComparisonSide.ts Outdated
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 556f7e8 to 97bfdd5 Compare August 18, 2026 21:00
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 0440072 to 1a8193d Compare August 18, 2026 21:00
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 97bfdd5 to 76a1953 Compare August 18, 2026 21:59
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 1a8193d to 9b26a52 Compare August 18, 2026 21:59
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 76a1953 to d39ca43 Compare August 18, 2026 23:07
@camielvs
camielvs requested a review from Mbeaulne August 19, 2026 00:00
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.

2 participants