Skip to content

fix(eve): bind approval callbacks to responders - #1354

Closed
benpankow wants to merge 1 commit into
ben/hitl-concurrent-authorizationfrom
ben/hitl-bind-candidate-responder
Closed

fix(eve): bind approval callbacks to responders#1354
benpankow wants to merge 1 commit into
ben/hitl-concurrent-authorizationfrom
ben/hitl-bind-candidate-responder

Conversation

@benpankow

@benpankow benpankow commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Closes the cross-user callback race for concurrent approval candidates:

  • restores the persisted candidate responder into the active authorization context before resumed policy/token resolution
  • prevents a later Slack click from substituting its principal into an earlier candidate OAuth callback
  • retains candidate identity as the canonical responder for audit and settlement
  • adds a regression with an original responder and a different latest session caller

Validation

  • pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/harness/input-requests.test.ts
  • pnpm --filter eve typecheck

Stack

Depends on #1353.

Signed-off-by: benpankow <ben.pankow@vercel.com>
@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
eve-docs Ready Ready Preview Jul 29, 2026 8:29pm
eve-docs-4759 Ready Ready Preview, v0 Jul 29, 2026 8:29pm

@benpankow

Copy link
Copy Markdown
Collaborator Author

Superseded by the consolidated 7-PR implementation stack starting at #1368 (GitHub stack #1375).

@benpankow benpankow closed this Jul 29, 2026
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