jerry/buzz
1
0
Fork 0
mirror of https://github.com/block/buzz.git synced 2026-10-27 04:30:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "fix(acp): gate relay-signed workflow messages on their attributed author" (#6311)

Browse Source 
Reverts block/buzz#6129
This commit is contained in:
Luke Tornquist
2026-08-19 11:28:04 -04:00
committed by GitHub
parent e5a6e2022f
commit 08eb46ef3c
3 changed files with 1 additions and 438 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
crates/buzz-relay/src/workflow_sink.rs
View File

@@ -255,9 +255,6 @@ impl ActionSink for RelayActionSink {
// - `p` tag attributes the message to the workflow owner
// - `h` tag scopes to the channel (NIP-29, canonical UUID)
// - `buzz:workflow` tag prevents recursive workflow triggering
// - `buzz:workflow-owner` tag names the workflow owner explicitly,
// so consumers (e.g. the ACP inbound author gate) can attribute
// the message without inferring ownership from `p`-tag order
// - one `p` tag per `@Name` that resolves to a channel member,
// so mentioned agents are woken (wake is `p`-tag gated)
let mut tags = vec![
@@ -267,8 +264,6 @@ impl ActionSink for RelayActionSink {
.map_err(|e| ActionSinkError::EventBuild(format!("h tag: {e}")))?,
Tag::parse(["buzz:workflow", "true"])
.map_err(|e| ActionSinkError::EventBuild(format!("workflow tag: {e}")))?,
Tag::parse(["buzz:workflow-owner", &author_pubkey_hex])
.map_err(|e| ActionSinkError::EventBuild(format!("workflow owner tag: {e}")))?,
];
// Resolve `@Name` mentions to channel-member pubkeys and append a
@@ -712,18 +707,5 @@ mod integration_tests {
p_tag_targets.contains(&agent_hex.as_str()),
"mentioned member {agent_hex} must be p-tagged so it wakes; got {p_tag_targets:?}"
);
let owner_tag = stored
.event
.tags
.iter()
.find(|t| t.as_slice().first().map(|s| s.as_str()) == Some("buzz:workflow-owner"))
.and_then(|t| t.as_slice().get(1).map(|s| s.as_str()));
assert_eq!(
owner_tag,
Some(author_hex.as_str()),
"workflow owner must be named explicitly via buzz:workflow-owner \
so consumers never infer ownership from p-tag order"
);
}
}