Compare commits
8 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 66933b1e2e | |||
| 18bbde19b6 | |||
| fa5cdf3e1d | |||
| 1416375e40 | |||
| de572af675 | |||
| 234f663f04 | |||
| 8c3bacb2f2 | |||
| 20ea4f9291 |
2
.gitignore
vendored
2
.gitignore
vendored
@ -1,5 +1,5 @@
|
|||||||
# claude
|
# claude
|
||||||
CLAUDE.md
|
.claude/
|
||||||
|
|
||||||
# python
|
# python
|
||||||
__pycache__/
|
__pycache__/
|
||||||
|
|||||||
10
README.md
10
README.md
@ -9,17 +9,19 @@ buttons) behind previous / jump / next navigation, with an optional cache button
|
|||||||
`requirements.txt`:
|
`requirements.txt`:
|
||||||
|
|
||||||
```
|
```
|
||||||
dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.1
|
dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.4
|
||||||
```
|
```
|
||||||
|
|
||||||
Direct:
|
Direct:
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
pip install "dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.1"
|
pip install "dpy_paginator @ git+ssh://git@git.rethinkstudios.io/rethink-public/dpy_paginator.git@v0.1.4"
|
||||||
```
|
```
|
||||||
|
|
||||||
Requires `discord.py` (pulled transitively).
|
Requires `discord.py` (pulled transitively).
|
||||||
|
|
||||||
|
Drop the `@v0.1.4` suffix from the line above to install the latest unpinned.
|
||||||
|
|
||||||
## Basic usage
|
## Basic usage
|
||||||
|
|
||||||
Plain pages — just navigation:
|
Plain pages — just navigation:
|
||||||
@ -97,6 +99,8 @@ echoes its `data`.
|
|||||||
## Cache button (mention priming)
|
## Cache button (mention priming)
|
||||||
|
|
||||||
Discord clients render `<@id>` as a raw id until the user object is cached locally.
|
Discord clients render `<@id>` as a raw id until the user object is cached locally.
|
||||||
|
Omit `cache` (or pass `None` / `[]`) and the button never appears; a non-empty cache
|
||||||
|
must have one entry per page or construction raises.
|
||||||
The cache button fixes that: pass `cache=[...]` with one entry per page, where each
|
The cache button fixes that: pass `cache=[...]` with one entry per page, where each
|
||||||
entry is a string of the user mentions on that page (`"<@111> <@222> <@333>"`).
|
entry is a string of the user mentions on that page (`"<@111> <@222> <@333>"`).
|
||||||
|
|
||||||
@ -139,4 +143,4 @@ in an embed). It may be sync or async.
|
|||||||
|
|
||||||
## Versioning
|
## Versioning
|
||||||
|
|
||||||
Tagged `vX.Y.Z`. Pin the tag in `requirements.txt`.
|
Releases are tagged `vX.Y.Z`. The install line above pins a release; drop the `@vX.Y.Z` suffix to install the latest unpinned. Pin deliberately for reproducible installs.
|
||||||
|
|||||||
@ -4,7 +4,7 @@ build-backend = "hatchling.build"
|
|||||||
|
|
||||||
[project]
|
[project]
|
||||||
name = "dpy_paginator"
|
name = "dpy_paginator"
|
||||||
version = "0.1.1"
|
version = "0.1.4"
|
||||||
description = "Button-navigated paginator for discord.py — config-free, injectable emojis, installable."
|
description = "Button-navigated paginator for discord.py — config-free, injectable emojis, installable."
|
||||||
requires-python = ">=3.10"
|
requires-python = ">=3.10"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
|
|||||||
@ -29,6 +29,7 @@ config-free: no host config import; everything is passed at construction.
|
|||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
import asyncio
|
import asyncio
|
||||||
|
import logging
|
||||||
from typing import (
|
from typing import (
|
||||||
Any,
|
Any,
|
||||||
Dict,
|
Dict,
|
||||||
@ -69,6 +70,8 @@ DEFAULT_EMOJIS = {
|
|||||||
"cache": "\U0001f5c2\ufe0f", # 🗂️
|
"cache": "\U0001f5c2\ufe0f", # 🗂️
|
||||||
}
|
}
|
||||||
|
|
||||||
|
log = logging.getLogger(__name__)
|
||||||
|
|
||||||
PageT_co = TypeVar("PageT_co", bound=Page, covariant=True)
|
PageT_co = TypeVar("PageT_co", bound=Page, covariant=True)
|
||||||
|
|
||||||
|
|
||||||
@ -161,6 +164,10 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
super().__init__(timeout=timeout)
|
super().__init__(timeout=timeout)
|
||||||
if not pages:
|
if not pages:
|
||||||
raise ValueError("ButtonPaginator requires at least one page")
|
raise ValueError("ButtonPaginator requires at least one page")
|
||||||
|
if per_page < 1:
|
||||||
|
# per_page <= 0 would ZeroDivisionError (==0) or yield a negative max_pages
|
||||||
|
# (<0) at the divmod below; fail loud like the other construction guards
|
||||||
|
raise ValueError("per_page must be >= 1")
|
||||||
self.author_id: Optional[int] = author_id
|
self.author_id: Optional[int] = author_id
|
||||||
self.delete_message_after: bool = delete_message_after
|
self.delete_message_after: bool = delete_message_after
|
||||||
self.mentions_allowed = mentions_allowed or discord.AllowedMentions.all()
|
self.mentions_allowed = mentions_allowed or discord.AllowedMentions.all()
|
||||||
@ -176,7 +183,9 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
total_pages, left_over = divmod(len(self.pages), self.per_page)
|
total_pages, left_over = divmod(len(self.pages), self.per_page)
|
||||||
self.max_pages: int = total_pages + (1 if left_over else 0)
|
self.max_pages: int = total_pages + (1 if left_over else 0)
|
||||||
if cache is not None and len(cache) < self.max_pages:
|
# a falsy cache (None or []) disables the cache button (see the render path's
|
||||||
|
# `if self.cache:`); only a non-empty cache must have one entry per page
|
||||||
|
if cache and len(cache) < self.max_pages:
|
||||||
raise ValueError(
|
raise ValueError(
|
||||||
f"cache has {len(cache)} entries but there are {self.max_pages} pages; "
|
f"cache has {len(cache)} entries but there are {self.max_pages} pages; "
|
||||||
"cache needs one entry per page"
|
"cache needs one entry per page"
|
||||||
@ -200,7 +209,7 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
async def interaction_check(self, interaction: Interaction) -> bool:
|
async def interaction_check(self, interaction: Interaction) -> bool:
|
||||||
"""restrict interaction to author_id when set"""
|
"""restrict interaction to author_id when set"""
|
||||||
if not self.author_id or self.author_id == interaction.user.id:
|
if self.author_id is None or self.author_id == interaction.user.id:
|
||||||
return True
|
return True
|
||||||
await interaction.response.send_message("You cannot interact with this menu.", ephemeral=True)
|
await interaction.response.send_message("You cannot interact with this menu.", ephemeral=True)
|
||||||
return False
|
return False
|
||||||
@ -259,6 +268,14 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
if isinstance(value, discord.Attachment):
|
if isinstance(value, discord.Attachment):
|
||||||
value = await value.to_file()
|
value = await value.to_file()
|
||||||
self._page_kwargs["files"].append(value)
|
self._page_kwargs["files"].append(value)
|
||||||
|
elif key in ("embed", "embeds", "file", "files"):
|
||||||
|
# a wrong-typed embed/file would otherwise fall to the catch-all and
|
||||||
|
# be forwarded verbatim, colliding with the base embeds=[]/files=[]
|
||||||
|
# and raising an opaque TypeError from inside discord.py — reject it
|
||||||
|
# here with a clear, paginator-side message
|
||||||
|
raise ValueError(
|
||||||
|
f"page key {key!r} has unexpected type {type(value).__name__}"
|
||||||
|
)
|
||||||
else:
|
else:
|
||||||
self._page_kwargs[key] = value
|
self._page_kwargs[key] = value
|
||||||
elif isinstance(formatted_page, str):
|
elif isinstance(formatted_page, str):
|
||||||
@ -280,10 +297,20 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
|
|
||||||
return self._page_kwargs
|
return self._page_kwargs
|
||||||
|
|
||||||
def update_buttons(self) -> None:
|
def update_buttons(self, nav: bool = True) -> None:
|
||||||
"""rebuild the action row for the current page state"""
|
"""rebuild the action row for the current page state
|
||||||
|
|
||||||
|
nav=False rebuilds with only the page's custom buttons and no navigation
|
||||||
|
items (prev/jump/cache/next) — a single-page result that still carries
|
||||||
|
custom buttons keeps them (and their live callbacks) without a nav row.
|
||||||
|
the nav row is also suppressed for a single page (max_pages < 2) even when
|
||||||
|
nav is left at its default, so a re-render (update_page) never resurrects it.
|
||||||
|
"""
|
||||||
self.clear_items()
|
self.clear_items()
|
||||||
|
|
||||||
|
nav = nav and self.max_pages >= 2
|
||||||
|
|
||||||
|
if nav:
|
||||||
self.previous_page.emoji = self.emojis["previous"]
|
self.previous_page.emoji = self.emojis["previous"]
|
||||||
self.previous_page.disabled = self.current_page <= 0
|
self.previous_page.disabled = self.current_page <= 0
|
||||||
self.add_item(self.previous_page)
|
self.add_item(self.previous_page)
|
||||||
@ -294,6 +321,7 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
for button in self.current_page_buttons:
|
for button in self.current_page_buttons:
|
||||||
self.add_item(button)
|
self.add_item(button)
|
||||||
|
|
||||||
|
if nav:
|
||||||
if self.cache:
|
if self.cache:
|
||||||
self.cache_button.emoji = self.emojis["cache"]
|
self.cache_button.emoji = self.emojis["cache"]
|
||||||
self.add_item(self.cache_button)
|
self.add_item(self.cache_button)
|
||||||
@ -367,9 +395,13 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
) -> Optional[Union[discord.Message, discord.WebhookMessage]]:
|
) -> Optional[Union[discord.Message, discord.WebhookMessage]]:
|
||||||
"""send the first page; obj is an Interaction or a Messageable"""
|
"""send the first page; obj is an Interaction or a Messageable"""
|
||||||
kwargs = await self.get_page_kwargs(self.get_page(self.current_page))
|
kwargs = await self.get_page_kwargs(self.get_page(self.current_page))
|
||||||
|
# update_buttons already suppresses the nav row for a single page (max_pages < 2),
|
||||||
|
# keeping only the page's custom buttons — so a single page with custom buttons
|
||||||
|
# renders them (live callbacks) with no nav row
|
||||||
self.update_buttons()
|
self.update_buttons()
|
||||||
|
|
||||||
if self.max_pages < 2:
|
if self.max_pages < 2 and not self.current_page_buttons:
|
||||||
|
# single page, no custom buttons: no interactive row at all, drop the view
|
||||||
self.stop()
|
self.stop()
|
||||||
kwargs.pop("view", None)
|
kwargs.pop("view", None)
|
||||||
|
|
||||||
@ -402,3 +434,7 @@ class ButtonPaginator(Generic[PageT_co], discord.ui.View):
|
|||||||
await self.message.delete()
|
await self.message.delete()
|
||||||
except (discord.NotFound, discord.Forbidden):
|
except (discord.NotFound, discord.Forbidden):
|
||||||
pass
|
pass
|
||||||
|
except discord.HTTPException:
|
||||||
|
# on_timeout runs as a fire-and-forget task; a transient delete failure must
|
||||||
|
# not surface as an unretrieved-task traceback on a best-effort cleanup
|
||||||
|
log.warning("paginator on_timeout: failed to delete message", exc_info=True)
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user