Skip to content

fix(discord): ignore thread recap system messages (#230) - #239

Merged
TerrysPOV merged 1 commit into
moazbuilds:masterfrom
rudi193-cmd:fix/issue-230-discord-thread-recap
Jun 28, 2026
Merged

fix(discord): ignore thread recap system messages (#230)#239
TerrysPOV merged 1 commit into
moazbuilds:masterfrom
rudi193-cmd:fix/issue-230-discord-thread-recap

Conversation

@rudi193-cmd

Copy link
Copy Markdown
Contributor

Summary

Fixes #230.

Discord emits system messages (e.g. type 18 thread-creation recap) on the parent channel when a thread is created. Those were routed through handleMessageCreate and could trigger spurious Claude prompts.

Only process regular messages (type 0) and replies (type 19). Bump plugin/marketplace version to 1.0.40 per contributor guidelines.

Note: #235 implements the same filter; this PR is rebased on current master from rudi193-cmd in case maintainers prefer a fresh branch. Happy to close in favor of #235 if that lands first.

Test plan

  • Create a Discord thread in a listened channel — parent channel should not get a Claude prompt from the recap message
  • Normal messages and replies in threads still work

Made with Cursor

Filter Discord message types so thread-creation recaps (type 18) do not
trigger Claude on the parent channel. Fixes moazbuilds#230.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rudi193-cmd

Copy link
Copy Markdown
Contributor Author

Note on the failing claude-review check: this is failing before review runs because the action has no Claude credential available in the repo:

ANTHROPIC_API_KEY:
Failed to authenticate. API Error: 401 Invalid authentication credentials.

The check shows as SUCCESS in the rollup because the workflow exits 0 on auth failure — but mergeStateStatus is UNSTABLE. This is a repo-side secret configuration issue, not a problem with the PR code. Happy to address any actual code feedback.

@TerrysPOV TerrysPOV left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — fix is clean. Enum + filter placement is correct (after bot-author check, before auth); no code concerns.

Heads-up: byte-identical to #235 (azaz44, draft) and the bumps to 1.0.40 are now stale since #233 landed master at 1.0.40 — version-guard will need 1.0.41 on rebase. Thanks for the parallel implementation @rudi193-cmd.

@TerrysPOV TerrysPOV mentioned this pull request Jun 28, 2026
2 tasks
@TerrysPOV
TerrysPOV merged commit 7bc75ca into moazbuilds:master Jun 28, 2026
4 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.

Discord thread handling issue / OpenClaw makes stuff on main channel

2 participants