Skip to content

✨ Expose factual comparison evidence#314

Merged
Robdel12 merged 1 commit into
mainfrom
rd/comparison-evidence
Jul 22, 2026
Merged

✨ Expose factual comparison evidence#314
Robdel12 merged 1 commit into
mainfrom
rd/comparison-evidence

Conversation

@Robdel12

Copy link
Copy Markdown
Contributor

Why

The comparisons command currently treats the API's processing status as the visual outcome. That makes current comparison responses harder for agents and humans to interpret, and hides useful review and Honeydiff evidence the API already returns.

What changed

  • Preserve the existing comparison JSON fields while adding a canonical visual result, reviewState, and visualReview.
  • Normalize current and older API response shapes for screenshot names, browser, viewport, and asset URLs.
  • Expose API-provided Honeydiff projection and region counts alongside the existing fingerprint and diff evidence.
  • Use the visual result for filtering, summaries, icons, and detailed terminal output while retaining the processing status in JSON.
  • Document the distinction between processing status, visual result, and review state.

No auth, pagination, grouping, status-command, or dynamic-content behavior changes are included.

Confidence

The comparison command is covered across current and older response shapes, including preservation of existing fields and filtering by visual result. The full test, lint, build, type-definition, and package checks pass, and the packaged CLI includes the new normalization module.

Preserve existing comparison output while distinguishing visual results from processing status and exposing review and Honeydiff context already returned by the API.
@Robdel12
Robdel12 marked this pull request as ready for review July 22, 2026 02:27
@Robdel12
Robdel12 merged commit 26bef2f into main Jul 22, 2026
27 of 30 checks passed
@Robdel12
Robdel12 deleted the rd/comparison-evidence branch July 22, 2026 02:28
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