fix(acp): deliver forum mentions under the default subscription - #5945
Open
Chessing234 wants to merge 2 commits into
Open
fix(acp): deliver forum mentions under the default subscription#5945Chessing234 wants to merge 2 commits into
Chessing234 wants to merge 2 commits into
Conversation
The kind list for `--subscribe mentions` was an inline `unwrap_or_else` closure buried in `tokio_main`, which is where a subscription gap can sit unnoticed for a long time. Lift it to `default_mention_kinds()` next to the other subscription helpers and pin the current coverage with a test, so any later edit to the list has to keep kind:9, workflow approvals and reminders deliberately rather than by accident. No behaviour change. Signed-off-by: Taksh <takshkothari09@gmail.com>
`--subscribe mentions` listened to kind:9, workflow approvals and reminders. A forum channel carries its conversation as kind:45001 posts and kind:45003 comments, so an agent on the default subscription was deaf in exactly the channels built for threaded discussion: it joined, logged the subscription, showed online, and an `@mention` in a forum post produced no inbound event at all (block#5268). Add both forum kinds to the default list. `--kinds` still overrides it wholesale, and the test added in the previous commit asserts the stream kinds are still covered, so this cannot quietly narrow. Signed-off-by: Taksh <takshkothari09@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #5268.
An agent started with the default
--subscribe mentionssubscribed to kind:9 stream messages, workflow approval requests and reminders. Forum channels don't use kind:9 — their conversation is kind:45001 posts and kind:45003 comments — so an agent invited to a forum channel joined, logged the subscription and showed online, and then never received an event for an@mentionthere. From the outside it looks like the agent is ignoring you; there's nothing in the log to suggest the mention was filtered out, because it was never delivered in the first place.Both forum kinds are now in the default list.
--kindsstill overrides the default wholesale, so anyone who has narrowed it explicitly is unaffected.tokio_mainintodefault_mention_kinds()(first commit, no behaviour change) so it's testable and visible next to the other subscription helpers.Verified locally on the pinned 1.95.0 toolchain:
cargo test -p buzz-acp --lib— 780 passed (778 before, plus the two new)cargo clippy -p buzz-acp --all-targets -- -D warnings— cleancargo clippy --workspace --all-targets -- -D warnings— cleancargo fmt --all -- --check— cleanNote: I'm an outside contributor, so the workflow runs on this PR will sit at
action_requireduntil a maintainer approves them; only the DCO check reports on its own.