Skip to content

Preserve changed episode snapshots in dataset comparisons - #225

Open
Mo-Elshenawy1 wants to merge 5 commits into
mostafa/api-sync-75851e2c-episode-readfrom
mostafa/api-sync-75851e2c-compare
Open

Mo-Elshenawy1 wants to merge 5 commits into
mostafa/api-sync-75851e2c-episode-readfrom
mostafa/api-sync-75851e2c-compare

Conversation

@Mo-Elshenawy1

@Mo-Elshenawy1 Mo-Elshenawy1 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Changelog

Preserve changed episode snapshots in dataset comparisons.

Docs

None; command help is included in this PR.

Description

Handle changed episode versions in dataset comparisons, including both snapshots in JSON and selected/previous version columns in table and CSV output. Older responses without changed counts keep their existing summary.

Depends on #220; this PR targets that branch so the diff contains only this change.

Validated with make rust-lint rust-test-ignored rust-doc rust-audit build, affected command help and Bash completion. HTTP tests use local fixtures. Live API smoke tests and deployment verification have not been performed.

BeforeAfter

Comparison summaries omitted changed counts and prior episode snapshots.

foxglove datasets versions compare ds_example 1 2 --format json — Preserves previousVersion/previousEpisode and reports changedCount alongside additions and removals.

Links

Episode API stack, listed in merge order. Each PR targets its dependency branch.

claude[bot]

This comment was marked as outdated.

claude[bot]

This comment was marked as outdated.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Mo-Elshenawy1
Mo-Elshenawy1 marked this pull request as ready for review October 15, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant