Skip to content

feat(telegram): support copy text buttons - #59443

Closed
kilhyeonjun wants to merge 7 commits into
NousResearch:mainfrom
kilhyeonjun:feat/telegram-copy-text-buttons
Closed

feat(telegram): support copy text buttons#59443
kilhyeonjun wants to merge 7 commits into
NousResearch:mainfrom
kilhyeonjun:feat/telegram-copy-text-buttons

Conversation

@kilhyeonjun

Copy link
Copy Markdown

Summary

  • add Telegram COPY_BUTTON: label | text markers that render as Bot API CopyTextButton inline keyboard buttons
  • support both normal sends and streaming final edits, while leaving markers visible when CopyTextButton is unavailable
  • document the marker syntax and constraints for Telegram users

Test Plan

  • uv run --extra messaging python -m pytest tests/gateway/test_telegram_copy_buttons.py tests/gateway/test_telegram_clarify_buttons.py tests/gateway/test_telegram_rich_messages.py tests/gateway/test_telegram_thread_fallback.py -q -o 'addopts='
  • git diff --check
  • uv run --extra messaging python -m py_compile plugins/platforms/telegram/adapter.py tests/gateway/test_telegram_copy_buttons.py

@alt-glitch alt-glitch added type/feature New feature or request comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery platform/telegram Telegram bot adapter provider/openai OpenAI / Codex Responses API P3 Low — cosmetic, nice to have labels Jul 6, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Scope note for reviewers: the title/body describe a Telegram CopyTextButton feature, but the diff (+1147/-10 across 15 files) is dominated by an undisclosed OpenAI Codex quota-usage feature: new agent/credential_usage.py, new 455-line hermes_cli/codex_usage.py (hermes codex-usage CLI hitting the Codex quota endpoint), plus agent/codex_runtime.py/conversation_loop.py, gateway/run.py/slash_commands.py (_handle_usage_command), hermes_cli/main.py (cmd_insights/cmd_sessions), and hermes_state.py token counting. Only plugins/platforms/telegram/adapter.py + its test + the telegram doc match the title. Recommend splitting the Codex usage-tracking feature into its own PR (with a matching title) so it gets proper review, separate from the Telegram copy-button change.

@kilhyeonjun

Copy link
Copy Markdown
Author

Closing this PR because unrelated Codex usage commits were accidentally included in the branch. Reopened a clean Telegram-only PR here: #59445

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have platform/telegram Telegram bot adapter provider/openai OpenAI / Codex Responses API type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants