Skip to content

feat: add Telegram clarify prompts#22532

Open
kamellperry wants to merge 58 commits into
NousResearch:mainfrom
kamellperry:patched-main
Open

feat: add Telegram clarify prompts#22532
kamellperry wants to merge 58 commits into
NousResearch:mainfrom
kamellperry:patched-main

Conversation

@kamellperry

Copy link
Copy Markdown

Summary

  • Wire gateway clarify callbacks into Telegram sessions
  • Render structured clarify choices with labels, descriptions, inline buttons, and Other flow
  • Allow clarify choices to be plain strings or {label, description, value} objects

Test Plan

  • python -m py_compile tools/clarify_tool.py gateway/platforms/telegram.py gateway/run.py
  • python -m pytest tests/tools/test_clarify_tool.py tests/gateway/test_telegram_clarify.py -q -o 'addopts='

Notes

This lets messaging-platform agents ask a structured follow-up question and wait for the same user/thread response instead of returning "Clarify tool is not available in this execution context."

@alt-glitch alt-glitch added type/feature New feature or request P2 Medium — degraded but workaround exists comp/gateway Gateway runner, session dispatch, delivery platform/telegram Telegram bot adapter comp/tools Tool registry, model_tools, toolsets labels May 9, 2026
# Conflicts:
#	hermes_cli/env_loader.py
#	tests/tools/test_computer_use.py
#	tools/computer_use/cua_backend.py
#	tools/computer_use/tool.py
# Conflicts:
#	hermes_cli/main.py
# Conflicts:
#	cli.py
#	tests/hermes_cli/test_env_loader.py
#	tests/tools/test_computer_use.py
#	tools/computer_use/cua_backend.py
#	tools/computer_use/schema.py
#	tools/computer_use/tool.py
#	website/docs/reference/skills-catalog.md
# Conflicts:
#	tools/computer_use/cua_backend.py
#	website/docs/reference/tools-reference.md
…ve-20260531182256-71921

# Conflicts:
#	hermes_cli/gateway.py
Preserve text-only screenshot analysis for non-vision main models and treat an empty app catalog as a valid cached result so launch_app name resolution does not fall through to live cua-driver calls in tests.
…ve-20260604144931-27314

# Conflicts:
#	hermes_cli/main.py
…ve-20260605200930-38811

# Conflicts:
#	hermes_cli/main.py
#	tests/hermes_cli/test_tui_npm_install.py
…ve-20260606151658-5953

# Conflicts:
#	gateway/run.py
#	hermes_cli/gateway.py
…ve-20260608234034-82037

# Conflicts:
#	cli.py
#	hermes_cli/main.py
#	tests/hermes_cli/test_cmd_update.py
#	tests/hermes_cli/test_web_ui_build.py
#	tests/tools/test_computer_use.py
#	tools/computer_use/cua_backend.py
#	website/docs/reference/tools-reference.md
#	website/docs/reference/toolsets-reference.md
@kamellperry kamellperry force-pushed the patched-main branch 5 times, most recently from bbb4572 to fd7542c Compare June 12, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery comp/tools Tool registry, model_tools, toolsets P2 Medium — degraded but workaround exists platform/telegram Telegram bot adapter type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants