#23182: fix(discord): pass statusReactions emojis and timing config to controller
channel: discord
size: XS
Cluster:
Discord and MS Teams Fixes
## Summary
- **Problem:** Discord status reaction emojis and timing configuration are not forwarded to the controller, causing status reactions to use hardcoded defaults instead of user-configured values.
- **Why it matters:** Users who customize their status reaction emojis or timing in config see no effect.
- **What changed:** Passed \`statusReactions\` emojis and timing config from the Discord handler to the controller.
- **What did NOT change:** Default behavior when no custom config is set remains the same.
## Change Type (select all)
- [x] Bug fix
- [ ] Feature
- [ ] Refactor
- [ ] Docs
- [ ] Security hardening
- [ ] Chore/infra
## Scope (select all touched areas)
- [ ] Gateway / orchestration
- [ ] Skills / tool execution
- [ ] Auth / tokens
- [ ] Memory / storage
- [x] Integrations
- [ ] API / contracts
- [ ] UI / DX
- [ ] CI/CD / infra
## Linked Issue/PR
- Related to Discord status reactions config
## User-visible / Behavior Changes
- Custom status reaction emojis and timing now take effect in Discord
## Security Impact (required)
- New permissions/capabilities? \`No\`
- Secrets/tokens handling changed? \`No\`
- New/changed network calls? \`No\`
- Command/tool execution surface changed? \`No\`
- Data access scope changed? \`No\`
## Repro + Verification
### Environment
- OS: macOS 15.3 (arm64)
- Runtime: Node v22+
- Integration/channel: Discord
### Steps
1. Configure custom status reaction emojis in Discord config
2. Trigger a bot response
3. Verify custom emojis are used
### Expected
- Custom emojis and timing are respected
### Actual
- Before fix: Hardcoded defaults used
- After fix: Config values forwarded to controller
## Evidence
Config forwarding follows the same pattern used for other Discord controller parameters.
## Human Verification (required)
- Verified scenarios: Traced config flow from handler to controller
- Edge cases checked: Missing config falls back to defaults
- What I did **not** verify: Live Discord bot with custom emojis
## Compatibility / Migration
- Backward compatible? \`Yes\`
- Config/env changes? \`No\`
- Migration needed? \`No\`
## Failure Recovery (if this breaks)
- How to disable/revert this change quickly: Revert the config forwarding changes
- Known bad symptoms: None expected
## Risks and Mitigations
None — additive config forwarding only.
Most Similar PRs
#23192: fix(slack): remove ack reaction on NO_REPLY when removeAckAfterRepl...
by SidQin-cyber · 2026-02-22
72.8%
#22642: fix(discord): Discord status state machine 2.0 (clean restart, foll...
by victorGPT · 2026-02-21
70.9%
#19917: feat(discord): allow disabling intermediate status reactions
by Gitjay11 · 2026-02-18
70.9%
#22557: fix(discord): coerce exec approval approver IDs to string to preven...
by zwffff · 2026-02-21
69.5%
#17254: fix(discord): intercept text-based slash commands instead of forwar...
by robbyczgw-cla · 2026-02-15
69.3%
#20009: fix(discord): immediately defer interactions to prevent timeouts
by Gitjay11 · 2026-02-18
69.3%
#19615: fix(discord): include default account when sub-accounts are configured
by prue-starfield · 2026-02-18
69.2%
#22965: fix(discord): pass accountId to ack reaction calls for named accounts
by arosstale · 2026-02-21
67.5%
#23336: feat : Enhance chat message handling: extract origin from __opencla...
by jayy-77 · 2026-02-22
67.1%
#20239: feat : add text policy configuration for Discord channels and imple...
by jayy-77 · 2026-02-18
67.0%