Skip to content

STA-278: Explain binary file changes in reviews - #37

Merged
github-actions[bot] merged 2 commits into
mainfrom
feature/sta-278
Sep 15, 2026
Merged

github-actions[bot] merged 2 commits into
mainfrom
feature/sta-278

Conversation

@claudecafe

Copy link
Copy Markdown
Collaborator

Capture mixed text and binary changes as file-level review blocks, preserving paths, lifecycle status, modes, sizes, and hashes without emitting binary bytes.

This lets authors explain binary additions, modifications, deletions, renames, and text/binary transitions alongside ordinary diffs, with local, exported, and hosted reviews retaining metadata cards.

Owner-approved receipt: 91b9308c-6abe-49f0-b87d-4ceccd8310f8.

minipai and others added 2 commits September 16, 2026 00:40
Binary sides keep their byte size and SHA-256 hash instead of aborting the
capture, so a change set mixing text and binary files can be captured and
explained. Binary changes render as metadata cards with their path, status,
modes, and before/after sizes, and their hashes participate in capture
identity. Text-only captures hash exactly as before, keeping existing walks
and explanations matched.

Co-Authored-By: ことね <kotone@claudecafe.dev>
The dev:report sample only exercised textual diffs, so the new binary card had no public example to inspect. Add an explained section covering an addition, a same-size modification, a deletion, a rename, and both text/binary transitions.

Co-Authored-By: ことね <kotone@claudecafe.dev>
@github-actions
github-actions Bot enabled auto-merge (rebase) September 15, 2026 14:43
@github-actions
github-actions Bot merged commit bce603d into main Sep 15, 2026
2 checks passed
@claudecafe
claudecafe deleted the feature/sta-278 branch September 17, 2026 02:04
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