fix(acp): admit trusted workflow wake messages - #3209
matthewrball wants to merge 2 commits into
Conversation
Co-authored-by: Matthew Ball <13823657+matthewrball@users.noreply.github.com> Signed-off-by: Matthew Ball <13823657+matthewrball@users.noreply.github.com>
|
@block/buzz-oss-team Ready for review. This unblocks scheduled owner-scoped agent workflows that are currently dropped by the ACP author gate. All GitHub checks and the local ACP/relay validation in the PR description are green; the trust path fails closed for a missing or mismatched workflow signer. A security-focused review would be appreciated. |
Co-authored-by: Matthew Ball <13823657+matthewrball@users.noreply.github.com> Signed-off-by: Matthew Ball <13823657+matthewrball@users.noreply.github.com>
|
Follow-up commit Fresh validation at this commit:
|
|
Additional field evidence for the prompt-attribution part of this PR, beyond whether a workflow wakes the agent: Verified in Buzz Desktop 0.5.25 on macOS, September 30, 2026, at 11:41 AM America/Chicago. A scheduled workflow successfully started the managed agent, but its visible Prompt Context event block presented the relay identity in It eventually proceeded, reasoning that the Current source already has a stronger author-gate check in Suggested acceptance criteria:
This is UI/source inspection evidence, not a passing integration-test claim. No agent permissions, prompts, or running workflows were changed. Related: #3858. |
Summary
ptag as the owner.ptag while keeping mentioned-agent wake targets.Validation
At
c98ca6a:cargo fmt --all -- --checkcargo clippy -p buzz-acp --all-targets --all-features -- -D warningscargo test -p buzz-acp: 623 unit tests and 9 lifecycle tests passed; doc tests passed.At
e46802ebefore the ACP-only follow-up:cargo clippy -p buzz-acp -p buzz-relay --all-targets --all-features -- -D warningscargo test -p buzz-relay nip11::tests --lib: 15 passed.cargo test -p buzz-relay workflow_sink::tests --lib: 18 passed.test-unit,desktop-test,desktop-build,desktop-tauri-check,desktop-tauri-test, andweb-buildpassed.Known unrelated validation gaps
just cireaches mobile analysis, where Flutter 3.44 reports the existingaxisAlignmentdeprecation inmobile/lib/features/pairing/pairing_page/pairing_welcome_view.dart:114.buzz-relaysuite and isolated pre-existing admin/media database tests hitSqlx(PoolTimedOut)in the local database environment. The changed relay modules pass their focused tests.