Skip to content

Align persona review verification flow#788

Merged
selfcontained merged 1 commit into
mainfrom
agt_e90a9c10588d/agent-10588d
Jul 17, 2026
Merged

Align persona review verification flow#788
selfcontained merged 1 commit into
mainfrom
agt_e90a9c10588d/agent-10588d

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

  • instruct parent agents to request reviewer verification after fixing persona feedback, while preserving the existing human-review resolve/reopen flow
  • grant persona reviewers the feedback-resolution tool and notify the parent through structured review-thread updates
  • remove polling/sleep guidance after persona launch because Dispatch sends review notifications automatically
  • retain exceptional parent/assignee resolution authority while limiting review-role callers to persona reviews they authored
  • make resolution conditional on open state so stale actors cannot overwrite an existing outcome or append duplicate resolution messages
  • update docs and regression coverage for the unified lifecycle

Validation

  • pnpm run check
  • pnpm run finalize:web
  • pnpm run test (server: 2274 passed, web: 224 passed, browser extension: 51 passed)
  • focused review suites: 44 passed
  • persona review E2E flows passed
  • isolated E2E functional coverage passed; the pre-existing overflow-layout.spec.ts afterAll cleanup hook intermittently exceeded 30 seconds, while its layout assertions passed. The remaining serial E2E project passed, and the cleanup issue is being handled separately on fix/overflow-e2e-cleanup-timeout.

@selfcontained
selfcontained merged commit 0a04b81 into main Jul 17, 2026
1 check passed
@selfcontained
selfcontained deleted the agt_e90a9c10588d/agent-10588d branch July 17, 2026 03:57
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