Skip to content

fix: add explicit document review acceptance - #6

Merged
julismo merged 1 commit into
mainfrom
agent/document-review-acceptance
Aug 7, 2026
Merged

fix: add explicit document review acceptance#6
julismo merged 1 commit into
mainfrom
agent/document-review-acceptance

Conversation

@julismo

@julismo julismo commented Aug 7, 2026

Copy link
Copy Markdown
Owner

What changed

  • Added an explicit, accessible Accept review action to the synthetic document-review flow.
  • Shows a local-only confirmation and clears it when a correction changes the reviewed result.
  • Added behaviour tests and a real synthetic-workbench screenshot to the README.

Why

The approved showcase design requires a reviewer to accept or correct a result. The panel previously exposed correction only.

User impact

Acceptance stays in React component state: no data is uploaded, stored, or sent to an external service.

Validation

  • npm run lint
  • npm run typecheck
  • npm run test (14 passing)
  • npm run build
  • Browser checks at 1440px and 375px with zero console errors or warnings

@julismo
julismo marked this pull request as ready for review August 7, 2026 16:15
@julismo
julismo merged commit 0aa1766 into main Aug 7, 2026
3 checks passed
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