romtuck/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(telegram): react action accepts numeric messageId and chatId

Browse Source 
The react action used readStringParam for messageId and chatId, which
rejected numeric values with a misleading "messageId required" error.
Switched to readStringOrNumberParam to match the delete/edit actions.

Closes #1459

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Ayush Ojha 2026-01-30 01:06:49 -08:00 committed by Ayaan Zaidi
parent 3a85cb1833
commit f760aa302c
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/channels/plugins/actions/telegram.ts
View File

@ -85,7 +85,7 @@ export const telegramMessageActions: ChannelMessageActionAdapter = {
} }
if (action === "react") { if (action === "react") {
const messageId = readStringParam(params, "messageId", { const messageId = readStringOrNumberParam(params, "messageId", {
required: true, required: true,
}); });
const emoji = readStringParam(params, "emoji", { allowEmpty: true }); const emoji = readStringParam(params, "emoji", { allowEmpty: true });
@ -94,7 +94,9 @@ export const telegramMessageActions: ChannelMessageActionAdapter = {
{ {
action: "react", action: "react",
chatId: chatId:
readStringParam(params, "chatId") ?? readStringParam(params, "to", { required: true }), readStringOrNumberParam(params, "chatId") ??
readStringOrNumberParam(params, "channelId") ??
readStringParam(params, "to", { required: true }),
messageId, messageId,
emoji, emoji,
remove, remove,