Skip to content

Fix Discord component interaction dispatch - #80

Merged
pattyboy813 merged 2 commits into
mainfrom
patfaint-fix-bill-setup-channel
Aug 22, 2026
Merged

Fix Discord component interaction dispatch#80
pattyboy813 merged 2 commits into
mainfrom
patfaint-fix-bill-setup-channel

Conversation

@pattyboy813

@pattyboy813 pattyboy813 commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Summary

Resolve Discord Components V2 channel-select partials through the interaction guild cache, falling back to Discord fetch only when absent. Validate the resolved channel is a standard text channel in the same guild before checking Bill's required permissions and persisting it.

Make the persistent profile button and select DynamicItem templates disjoint while preserving existing custom IDs. The supported-action registry ensures every emitted wizard action has exactly one dispatcher and prevents unknown actions from being generated.

Add focused regressions for real discord.py application-command partials, cache and fetch resolution, rejected channel kinds and guilds, fetch errors, persisted channel IDs, profile select/button dispatcher exclusivity, and action exhaustiveness.

Related work

None.

Checks

  • Python compile, Ruff, and pytest pass (70 tests)
  • Worker typecheck and Vitest pass (228 tests)
  • No secrets, private IDs, or local environment files are included
  • Documentation and manual steps are updated when needed (not applicable)

Risk and rollout

Low. Existing persistent custom IDs remain compatible. Authorization, stale-revision handling, permission validation, and persistence contracts are unchanged. No deployment or data changes are included.

pattyboy813 and others added 2 commits August 22, 2026 21:36
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@pattyboy813 pattyboy813 changed the title Fix setup channel selection resolution Fix Discord component interaction dispatch Aug 22, 2026
@pattyboy813
pattyboy813 merged commit 2e5fdeb into main Aug 22, 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.

1 participant