Skip to content

feat: add example custom commands to agentic mode bot menu#179

Open
sakhnenkoff wants to merge 1 commit intoRichardAtCT:mainfrom
sakhnenkoff:patch/custom-commands
Open

feat: add example custom commands to agentic mode bot menu#179
sakhnenkoff wants to merge 1 commit intoRichardAtCT:mainfrom
sakhnenkoff:patch/custom-commands

Conversation

@sakhnenkoff
Copy link
Copy Markdown

Summary

Related to #173 and #176.

Adds example user-facing commands to get_bot_commands() in agentic mode. These commands are NOT registered as CommandHandlers — they appear in Telegram's / menu for discoverability but are forwarded to Claude as natural language via _handle_unknown_command.

This is a minimal approach — a stepping stone toward the configurable custom commands feature discussed in #173. The current PR hardcodes 5 example commands; a follow-up could make these configurable via environment variable or config file.

Commands added:

  • /today — Morning briefing and daily plan
  • /closeday — End of day review and reflection
  • /capture — Quick capture a thought
  • /schedule — Plan your week
  • /context — Load context for a topic

How it works:

  • Commands are added to get_bot_commands() only (menu visibility)
  • They are NOT added to the handlers list in _register_agentic_handlers
  • Therefore they are NOT in _known_commands
  • _handle_unknown_command (group 10) catches them and forwards to Claude

Tests

2 new tests + 1 updated test in tests/unit/test_orchestrator.py:

  • test_vault_commands_in_bot_command_list — verifies commands appear in menu
  • test_vault_commands_not_in_known_commands — verifies they pass through to Claude
  • Updated test_agentic_bot_commands count from 6 to 11

All existing tests continue to pass.

Add /today, /closeday, /capture, /schedule, /context to get_bot_commands()
for Telegram menu visibility. These commands are forwarded to Claude via
_handle_unknown_command — this patch only affects menu display.

Ref: upstream issue RichardAtCT#173
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