Skip to content

feat(acp): integrate human permission decisions - #6

Merged
anhle128 merged 1 commit into
mainfrom
integration/permission-policy
Aug 8, 2026
Merged

feat(acp): integrate human permission decisions#6
anhle128 merged 1 commit into
mainfrom
integration/permission-policy

Conversation

@anhle128

@anhle128 anhle128 commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Ports the interactive permission-policy work from block#5106 onto the fork's current main without importing unrelated upstream commits.

What this fixes:

  • ask policy leaves permission requests actionable for a human in Buzz.
  • Buzz renders Allow/Deny choices and sends the selected permission_decision back to the ACP harness.
  • Global and per-agent policies support ask, allow, and reject.
  • Pending requests fail closed on timeout, cancellation, or disconnect.

Fork compatibility work:

  • Fixed upstream formatting/clippy failures.
  • Split over-limit Rust/TypeScript files without changing behavior or raising file-size limits.
  • Added a focused E2E assertion for the human Allow/Deny path.

Verification:

  • just ci
  • Focused Playwright permission-decision E2E
  • GitNexus compare against origin/main (CRITICAL expected: central ACP permission path)

Port the interactive permission policy flow from block#5106 and keep it compatible with the fork's current main.

Signed-off-by: Kevin Le <anhle12892@gmail.com>
@anhle128
anhle128 merged commit d585291 into main Aug 8, 2026
26 of 30 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