Skip to content

feat: run comparison - run metadata comparison - #2600

Open
camielvs wants to merge 1 commit into
cmp-04-artifact-comparisonfrom
cmp-05-run-metadata-comparison
Open

feat: run comparison - run metadata comparison#2600
camielvs wants to merge 1 commit into
cmp-04-artifact-comparisonfrom
cmp-05-run-metadata-comparison

Conversation

@camielvs

@camielvs camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Fifth PR in the Compare Runs stack. Adds run-level metadata comparison — the context around the two runs, as opposed to their pipeline structure.

It compares who created each run, when, and any run-level annotations and arguments, and surfaces what changed. Two things are deliberately handled specially:

  • Created-at is shown for context but never counts as a "change" on its own — two different runs always have different timestamps.
  • Superficial keys (notes, tags) and layout-only editor annotations are ignored so they don't create noise.

Annotation and argument keys are treated generically, so whatever a given deployment stores shows up without needing hardcoded knowledge of the keys. Rendered as a compact summary bar that expands to show the details when something actually differs.

Related Issue and Pull requests

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

  • Run the unit tests: npm run test -- compareRunMetadata
    • Covers author changes, the created-at exception, generic annotation/argument diffing, and exclusion of superficial/frontend-only keys.
  • End-to-end (via the top of the stack, feat: run comparison - compare page, run selection & routing #2603, with the Compare runs flag on):
    • Compare two runs by different authors and confirm the metadata bar flags the author change and expands to show details.
    • Confirm two runs by the same author with only different timestamps are reported as unchanged.

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-05-run-metadata-comparison/fbe96eb

@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-05-run-metadata-comparison branch 2 times, most recently from 38d1bf5 to d0ef921 Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 6d71a83 to 1b84110 Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-05-run-metadata-comparison branch 2 times, most recently from dc53d9d to 1e12a5d Compare July 31, 2026 01:36
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch 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-05-run-metadata-comparison branch 2 times, most recently from 9ef9026 to d5b078f Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from a10c24a to 3508beb Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-05-run-metadata-comparison branch from d5b078f to c24848f Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 3508beb to 744e5fb Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-05-run-metadata-comparison branch from c24848f to e998ef1 Compare August 14, 2026 23:47
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from 744e5fb to fbae585 Compare August 14, 2026 23:47
Comment thread src/routes/v2/pages/CompareView/utils/compareRunMetadata.ts Outdated
Comment thread src/routes/v2/pages/CompareView/utils/compareRunMetadata.ts Outdated
@camielvs
camielvs force-pushed the cmp-05-run-metadata-comparison branch from e998ef1 to 921fd30 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-05-run-metadata-comparison branch from 921fd30 to 0f02d18 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-05-run-metadata-comparison branch from 0f02d18 to 6980587 Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-05-run-metadata-comparison branch from 6980587 to fbe96eb Compare August 18, 2026 00:56
@camielvs
camielvs force-pushed the cmp-04-artifact-comparison branch from ecc36ff to 556f7e8 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