Fix: preserve Telegram slash command arguments for command-dispatch by Octane0411 · Pull Request #9 · OasAIStudio/openclaw

Octane0411 · 2026-04-03T20:32:01Z

Summary

Preserve Telegram-native slash command arguments when dispatching to command-dispatch skills.
Normalize native command matching so variants like /dispatch_command 115, /dispatch_command@bot 115, and /dispatch_command@bot:115 resolve to the same configured skill command and keep argument payloads.
Keep command argument forwarding unchanged for existing non-Telegram dispatch paths.

Security Impact

No security behavior changes.
No new capabilities or trust boundaries are introduced.

Repro+Verification

Before: /dispatch_command 115 via Telegram command dispatch only forwarded /dispatch_command, dropping 115.
After: /dispatch_command 115 now resolves to command dispatch-command with args 115 and passes command=115 to command-dispatch tool execution.
Also covers Telegram bot-suffixed command variants in existing coverage paths.

Testing

Added/updated regression tests in src/telegram/bot-native-commands.test.ts for normalized skill command names and argument preservation.
Existing command-dispatch inline-action regression cases for slash variants remain in place in src/auto-reply/reply/get-reply-inline-actions.command-dispatch.test.ts.

AI Assisted

ChatGPT assisted with tracing the dispatch path and identifying where normalized command mismatch caused argument stripping.
Changes were limited to dispatch parsing and targeted tests.

Fixes

Fixes openclaw#59228

@velvet-shark

Merged via squash. Prepared head SHA: f9b2375 Co-authored-by: velvet-shark <126378+velvet-shark@users.noreply.github.com> Co-authored-by: velvet-shark <126378+velvet-shark@users.noreply.github.com> Reviewed-by: @velvet-shark

…ema instead of OpenAI … (openclaw#40008) Verified: - pnpm install --frozen-lockfile - pnpm build - pnpm check - pnpm test:macmini Co-authored-by: opriz <51957849+opriz@users.noreply.github.com> Co-authored-by: Tak Hoffman <781889+Takhoffman@users.noreply.github.com>

…formatters (openclaw#39969)

@jalehman

…penclaw#40460) Merged via squash. Prepared head SHA: 44622ab Co-authored-by: dsantoreis <220753637+dsantoreis@users.noreply.github.com> Co-authored-by: jalehman <550978+jalehman@users.noreply.github.com> Reviewed-by: @jalehman

@jalehman

…estart (openclaw#40324) Merged via squash. Prepared head SHA: cfd9956 Co-authored-by: cgdusek <38732970+cgdusek@users.noreply.github.com> Co-authored-by: jalehman <550978+jalehman@users.noreply.github.com> Reviewed-by: @jalehman

@mbelinky

…#40995) Merged via squash. Prepared head SHA: c1191ed Co-authored-by: xaeon2026 <264572156+xaeon2026@users.noreply.github.com> Co-authored-by: mbelinky <132747814+mbelinky@users.noreply.github.com> Reviewed-by: @mbelinky

@pejmanjohn

…al (openclaw#41187) * fix(acp): map error states to end_turn instead of unconditional refusal * fix: map ACP error stop reason to end_turn (openclaw#41187) (thanks @pejmanjohn) --------- Co-authored-by: Pejman Pour-Moezzi <481729+pejmanjohn@users.noreply.github.com> Co-authored-by: Onur <onur@textcortex.com>

@pejmanjohn

…#41185) * fix(acp): propagate setSessionMode gateway errors to client * fix: add changelog entry for ACP setSessionMode propagation (openclaw#41185) (thanks @pejmanjohn) --------- Co-authored-by: Pejman Pour-Moezzi <481729+pejmanjohn@users.noreply.github.com> Co-authored-by: Onur <onur@textcortex.com>

…penclaw#40185) * telegram: bridge direct delivery message hooks * telegram: align sent hooks with command session

@mbelinky

Merged via squash. Prepared head SHA: 5877389 Co-authored-by: mbelinky <132747814+mbelinky@users.noreply.github.com> Co-authored-by: mbelinky <132747814+mbelinky@users.noreply.github.com> Reviewed-by: @mbelinky

@altaywtf

Merged via squash. Prepared head SHA: 4900042 Co-authored-by: altaywtf <9790196+altaywtf@users.noreply.github.com> Co-authored-by: altaywtf <9790196+altaywtf@users.noreply.github.com> Reviewed-by: @altaywtf

…ered

… lists

Octane0411 · 2026-04-04T20:16:17Z

Root Cause

Telegram-native dispatch matching in parseTelegramNativeCommandArgs used only case normalization, so configured slash command aliases that differ by normalization (such as hyphen vs underscore forms or bot suffix variants) could fail matching. In those mismatch paths, the payload was not reliably preserved for command-dispatch execution.

Fix

Updated parseTelegramNativeCommandArgs to compare names through normalizeTelegramCommandName(...) in both matching branches, keeping capture behavior unchanged. Added regression coverage in src/telegram/bot-native-commands.test.ts to verify dispatch_command style invocations keep 115 and route to the configured skill command.

Verification

New regression test in src/telegram/bot-native-commands.test.ts covering normalized Telegram command names with arguments.
Existing slash-variant dispatch assertions already cover the /command and /command@bot forms in src/auto-reply/reply/get-reply-inline-actions.command-dispatch.test.ts.
Current HEAD: 5caa889de0288238ab558a24614826a725cc1781.

steipete and others added 30 commits March 9, 2026 06:34

docs: move 2026.3.8 entries back to unreleased

2d55ad0

test: fix Node 24+ test runner and subagent registry mocks

8d2d6db

test: fix Windows fake runtime bin fixtures

912aa87

fix: normalize windows runtime shim executables

66c581c

chore: prepare 2026.3.8-beta.1 release

5fca4c0

chore: update appcast for 2026.3.8-beta.1

250d3c9

test: fix windows runtime and restart loop harnesses

cc0f30f

fix(update): re-enable launchd service before updater bootstrap

1d3dde8

chore: prepare 2026.3.8 npm release

d0847ee

test: narrow gateway loop signal harness

3caab92

fix(launchd): harden macOS launchagent install permissions

ce9e91f

fix(onboard): avoid persisting talk fallback on fresh setup

7217b97

build: bump unreleased version to 2026.3.9

f9706fd

fix: stabilize launchd paths and appcast secret scan

6c579d7

build: sync plugin versions for 2026.3.9

f6d0712

fix(swiftformat): exclude HostEnvSecurityPolicy.generated.swift from …

98ea71a

…formatters (openclaw#39969)

Update CONTRIBUTING.md

258b790

Add Robin Waslander to maintainers

2cce459

Update CONTRIBUTING.md

4815dc0

plugins: harden global hook runner state (openclaw#40184)

12702e1

fix(telegram): bridge direct delivery to internal message:sent hooks (o…

7b88249

…penclaw#40185) * telegram: bridge direct delivery message hooks * telegram: align sent hooks with command session

Cron: enforce cron-owned delivery contract (openclaw#40998)

d4e59a3

Merged via squash. Prepared head SHA: 5877389 Co-authored-by: mbelinky <132747814+mbelinky@users.noreply.github.com> Co-authored-by: mbelinky <132747814+mbelinky@users.noreply.github.com> Reviewed-by: @mbelinky

Agents: add embedded error observations (openclaw#41336)

87d939b

Merged via squash. Prepared head SHA: 4900042 Co-authored-by: altaywtf <9790196+altaywtf@users.noreply.github.com> Co-authored-by: altaywtf <9790196+altaywtf@users.noreply.github.com> Reviewed-by: @altaywtf

wangruobing02 added 29 commits March 12, 2026 18:45

Fix: Clear stale function artifacts when switching models

405682c

Fix: allow ACP lineage keys for spawnedBy in sessions patch

eeaa8c0

Fix: Allow legacy ACP session keys in spawnedBy validation

8e99a41

Fix: honor dangerouslyDisableDeviceAuth to bypass device identity

d212e37

Fix: return not-requested for delivery.mode=none instead of not-deliv…

005e81a

…ered

Fix(slack): replace files.uploadV2 with 3-step upload flow

f29142e

Fix: propagate onboard auth profiles across sibling agents

26cb617

Fix: refresh model catalog cache during in-process gateway restart

a659030

Fix: Retry Telegram media download on code-less fetch failures

95d0bca

Fix(slack): use auth.scopes only, apps.permissions.info is deprecated

8c0dda8

Fix: respect configured imageModel.primary for image tool routing

a6d07eb

Fix: Deduplicate cross-channel outbound replies by inbound message id

8dc9126

Fix: dedupe route reply by effective target

2d3019b

Fix: deduplicate cross-channel route replies without messageId

20f87c5

Fix: apply media understanding to followup-queued messages

9850d89

Fix: add Gemini batch embedding env proxy test

e96cce1

Fix: use trusted env proxy mode in memory remote HTTP

0d4cdaa

Fix: align memory remote fetch proxy mode for Gemini embeddings

2ba2594

macOS: recover LaunchAgent after kickstart restart failures

996866a

Fix: keep macOS launchd service registered when restart kickstart fails

8e5b80f

Fix: Recover macOS LaunchAgent when kickstart restart fails

682864f

macOS: recover LaunchAgent when restart fails to load

eb1ffdd

Fix: validate launchd restart state after kickstart recovery

028cbae

Fix: recover not-loaded launch services for gateway start/restart

9ba2140

Fix: preserve telegram slash command args in dispatch flow

0735f0e

Fix: preserve Telegram bot-suffixed slash command args

0528765

Fix: preserve telegram native command args from split match payload

4e7a2a7

Fix: Preserve command-dispatch args for empty preloaded skill command…

732b69c

… lists

Fix: preserve telegram native command args for skill dispatch

5caa889

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix: preserve Telegram slash command arguments for command-dispatch#9

Fix: preserve Telegram slash command arguments for command-dispatch#9
Octane0411 wants to merge 1214 commits intomainfrom
fix/issue-59228-command-dispatch-args

Octane0411 commented Apr 3, 2026 •

edited

Loading

Uh oh!

Octane0411 commented Apr 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Conversation

Octane0411 commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Security Impact

Repro+Verification

Testing

AI Assisted

Fixes

Uh oh!

Octane0411 commented Apr 4, 2026

Root Cause

Fix

Verification

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Octane0411 commented Apr 3, 2026 •

edited

Loading