Skip to content

feat: Telegram connector overhaul — auth, trading panel, hot-reload#104

Merged
luokerenx4 merged 1 commit intomasterfrom
dev
Apr 6, 2026
Merged

feat: Telegram connector overhaul — auth, trading panel, hot-reload#104
luokerenx4 merged 1 commit intomasterfrom
dev

Conversation

@luokerenx4
Copy link
Copy Markdown
Contributor

Summary

  • Auth hot-reload: chatIds now hot-read from connectors.json per request — no restart needed to add/remove users
  • Magic link onboarding: unauthorized users receive a clickable link to Web UI with ?addChatId= pre-filled, one-click authorization
  • /trading panel: single command with multi-account overview, per-account detail panel (equity, pending commits, history), inline keyboard Approve/Reject/Back navigation
  • Connector-neutral tool descriptions: tradingPush no longer hardcodes "approve in the UI"
  • Fix misleading "Empty = allow all" comment on chatIds (actual behavior: reject all)

Commits

  • 4e1fab1 fix: telegram auth guard hot-reload chatIds + fix misleading comment
  • 89fec76 feat: telegram magic link onboarding for chat authorization
  • 6799ddf feat: Telegram /trading panel with multi-account support

Test plan

  • Send message to bot from unauthorized chat → receive magic link
  • Click link → Web UI shows authorization banner → click Authorize → chatId saved
  • Send message again → bot responds normally (hot-reload, no restart)
  • /trading with single account → shows full panel
  • /trading with multiple accounts → shows overview + account selector buttons
  • Approve/Reject buttons on pending commit → executes and refreshes panel
  • Back button → returns to overview

🤖 Generated with Claude Code

Single /trading command replaces /wallet, /push, /reject. Shows account
overview with inline keyboard for multi-account selection, per-account
panel with equity/pending/history, and Approve/Reject buttons. After
action, panel refreshes in-place. Also updates tradingPush tool
descriptions to be connector-neutral.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@luokerenx4 luokerenx4 merged commit c6625c2 into master Apr 6, 2026
2 checks passed
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