Skip to content

fix: reset viewed state when file content changes between rounds#310

Merged
tomasz-tomczyk merged 3 commits intomainfrom
fix/309-viewed-reset-on-content-change
Apr 19, 2026
Merged

fix: reset viewed state when file content changes between rounds#310
tomasz-tomczyk merged 3 commits intomainfrom
fix/309-viewed-reset-on-content-change

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Expose file_hash in /api/file response so the frontend can detect content changes
  • Reset "viewed" checkbox and collapsed state when a file's content changes between rounds
  • Previously both states persisted incorrectly, hiding changed files from the reviewer

Review

  • Code review: passed (go-expert + frontend-expert)
  • Parity audit: N/A (viewed state is crit-only)

Test plan

  • Go tests pass (go test ./...)
  • E2E test confirms failure without fix, pass with fix (TDD)
  • All 12 viewed.spec.ts tests pass

Closes #309

🤖 Generated with Claude Code

tomasz-tomczyk and others added 3 commits April 19, 2026 08:07
…ounds

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit 505ecc8 into main Apr 19, 2026
3 of 4 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the fix/309-viewed-reset-on-content-change branch April 19, 2026 07:34
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.

Files marked as viewed should lose the "review stamp" if agent changes them

1 participant