Skip to content

Telegram: fix dock native command registration#929

Closed
grp06 wants to merge 4 commits intoopenclaw:mainfrom
grp06:fix-telegram-command
Closed

Telegram: fix dock native command registration#929
grp06 wants to merge 4 commits intoopenclaw:mainfrom
grp06:fix-telegram-command

Conversation

@grp06
Copy link
Copy Markdown
Member

@grp06 grp06 commented Jan 15, 2026

Summary

Testing

  • pnpm test

Notes

  • AI-assisted changes (Clawdbot/Codex).
  • Formatting updates touch src/agents/pi-tools.policy.ts and src/agents/pi-tools.ts.

@grp06 grp06 force-pushed the fix-telegram-command branch from 4037eb5 to 0de3a1a Compare January 15, 2026 04:16
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4037eb5ef3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 242 to 246
defineChatCommand({
key: `dock:${dock.id}`,
nativeName: `dock-${dock.id}`,
nativeName: `dock_${dock.id}`,
description: `Switch to ${dock.id} for replies.`,
textAlias: `/dock-${dock.id}`,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep dock command text alias aligned with native name

nativeName now uses dock_${dock.id} while the only text alias remains /dock-${dock.id}. The command parser (normalizeCommandBody/hasControlCommand) only recognizes textAliases, but /commands output uses nativeName as the primary label (see buildCommandsMessage in src/auto-reply/status.ts). In text-only contexts (e.g., commands.native disabled or surfaces without native commands), users will be shown /dock_… yet that string is not a recognized alias, so the command won’t be handled. Consider adding a /dock_${dock.id} text alias or switching the primary label to the text alias when native commands are off.

Useful? React with 👍 / 👎.

@grp06 grp06 force-pushed the fix-telegram-command branch from 0de3a1a to b0e5e00 Compare January 15, 2026 04:30
@steipete steipete self-assigned this Jan 15, 2026
steipete added a commit that referenced this pull request Jan 15, 2026
Thanks @grp06.

Co-authored-by: George Pickett <gpickett00@gmail.com>
@steipete
Copy link
Copy Markdown
Contributor

Landed via fast-forward to main after rebasing on latest main and resolving conflicts.

  • Squash commit: 54fb59b
  • Tests: pnpm lint && pnpm build && pnpm test

Thanks @grp06!

@steipete steipete closed this Jan 15, 2026
zooqueen pushed a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
Thanks @grp06.

Co-authored-by: George Pickett <gpickett00@gmail.com>
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