Fix timeout for ephemeral paginators and hybrid commands

[AutonomousCat]

Pull Request Type

• Feature addition
• Bugfix
• Documentation update
• Code refactor
• Tests improvement
• CI/CD pipeline enhancement
• Other: [Replace with a description]

Description

While I was writing a command that uses ephemeral paginators, I noticed the timeout resulted in a 404 NotFound. While looking for ways to resolve this, I noticed that HybridContext required a message object in it's edit() function, so I used _prefixed_ctx to determine HybridContext. Probably not the best way to handle this, but I did what I could.

Changes

• Added edit field to Timeout class to store the message edit coroutine
• Updated send() and reply() to use the appropriate edit method based on command type

Test Scenarios

I tested with ephemeral, non-ephemeral, send, reply, slash, hybrid, and prefixed. Now works in all cases without errors.

Python Compatibility

• I've ensured my code works on Python 3.10.x
• I've ensured my code works on Python 3.11.x

Checklist

• I've run the pre-commit code linter over all edited files
• I've tested my changes on supported Python versions
• I've added tests for my code, if applicable
• I've updated / added documentation, where applicable

[mifuyutsuki]

Any reason to pass edit to Timeout instead of, say, ctx? At least that way you can forgo the if-else chain added in send() below to determine the right edit() by just using ctx.edit() regardless of command type, as have been done by the _on_button() callback. (Alternatively, add ctx to Paginator attributes instead of Timeout, then use self.paginator.ctx.edit().)

[AutonomousCat]

@mifuyutsuki HybridContext requires message. Storing ctx was the first thing I tried, but when testing, HybridContext types would error out since the message param was missing.

[silasary]

I'm trying to figure out why HybridContext needs the message parameter in the first place.

[AstreaTSS]

This is likely because said context was made before SlashContext had a default for message. It appears when a default was added, HybridContext was missed.