Skip to content

fix(telegram): show Telegram-safe skill commands in /skills#571

Open
Zx55 wants to merge 1 commit intochenhg5:mainfrom
Zx55:fix/telegram-skill-command-display
Open

fix(telegram): show Telegram-safe skill commands in /skills#571
Zx55 wants to merge 1 commit intochenhg5:mainfrom
Zx55:fix/telegram-skill-command-display

Conversation

@Zx55
Copy link
Copy Markdown

@Zx55 Zx55 commented Apr 11, 2026

Summary

This updates the Telegram /skills output to show Telegram-safe slash command names for discovered skills.

Why

Telegram commands cannot contain hyphens, but skill names often do. cc-connect already sanitizes Telegram command registration and already resolves hyphenated and underscored names equivalently, but the /skills text still rendered the raw skill name. That made the suggested command in Telegram inconsistent with the actual slash command users could invoke.

Changes

  • render skill commands in /skills using a Telegram-safe display name on Telegram
  • preserve the existing display on non-Telegram platforms
  • add a regression test for a hyphenated skill name such as telegram-codex-bot

Validation

  • go test ./core ./platform/telegram
  • manually verified in a real Telegram bot session that /skills shows /telegram_codex_bot

@Zx55 Zx55 marked this pull request as ready for review April 11, 2026 17:39
Copy link
Copy Markdown
Owner

@chenhg5 chenhg5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Telegram-safe skill command names — sanitizeTelegramDisplayCommand strips non-alphanumeric chars and limits to lowercase alphanum. Good companion to #572's recursive discovery.

PR: #571

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.

2 participants