Skip to content

fix: omit unchanged revision formatting details - #50

Merged
JSv4 merged 1 commit into
mainfrom
fix/47-show-only-actual-changes
Sep 17, 2026
Merged

JSv4 merged 1 commit into
mainfrom
fix/47-show-only-actual-changes

Conversation

@JSv4

@JSv4 JSv4 commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Formatting cards no longer show identical before/after attributes such as Left → Left or 0 → 0. Redundant formatting records backed by equal supplied snapshots are omitted from the list, counts, and filters.

Preserve engine-reported changes whose details are unmodeled, missing, or opaque, along with unresolved native revision diagnostics. Equal modeled values alone do not disprove a real engine change.

Validation:

  • TDD regressions cover comparison/native records, changed/added/removed attributes, counts, filtering, arriving evidence, and preservation of opaque changes and diagnostics.
  • npm run check: lint, 87 unit tests, library/demo builds and types, and API checks pass.
  • Browser suite: 55 passed, 2 optional fixture tests skipped; comparison/studio checks also pass after review fixes.
  • Adversarial review with gpt-5.6-sol; both findings reproduced with failing tests, fixed, and re-reviewed clean.

Closes #47.

@JSv4
JSv4 merged commit d58a384 into main Sep 17, 2026
4 checks passed
@JSv4
JSv4 deleted the fix/47-show-only-actual-changes branch September 17, 2026 13:06
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.

Change Lists Introduce Meaningless Noise

1 participant