mirror of
https://github.com/openclaw/openclaw.git
synced 2026-05-09 09:47:40 +00:00
fix: add telegram command-cap regression test (#12356) (thanks @arosstale)
This commit is contained in:
@@ -17,6 +17,7 @@ Docs: https://docs.openclaw.ai
|
|||||||
|
|
||||||
- Errors: prevent false positive context overflow detection when conversation mentions "context overflow" topic. (#2078) Thanks @sbking.
|
- Errors: prevent false positive context overflow detection when conversation mentions "context overflow" topic. (#2078) Thanks @sbking.
|
||||||
- Model failover: treat HTTP 400 errors as failover-eligible, enabling automatic model fallback when providers return bad request errors. (#1879) Thanks @orenyomtov.
|
- Model failover: treat HTTP 400 errors as failover-eligible, enabling automatic model fallback when providers return bad request errors. (#1879) Thanks @orenyomtov.
|
||||||
|
- Telegram: truncate command registration to 100 entries to avoid `BOT_COMMANDS_TOO_MUCH` failures on startup. (#12356) Thanks @arosstale.
|
||||||
- Exec approvals: format forwarded command text as inline/fenced monospace for safer approval scanning across channels. (#11937)
|
- Exec approvals: format forwarded command text as inline/fenced monospace for safer approval scanning across channels. (#11937)
|
||||||
- Config: clamp `maxTokens` to `contextWindow` to prevent invalid model configs. (#5516) Thanks @lailoo.
|
- Config: clamp `maxTokens` to `contextWindow` to prevent invalid model configs. (#5516) Thanks @lailoo.
|
||||||
- Docs: fix language switcher ordering and Japanese locale flag in Mintlify nav. (#12023) Thanks @joshp123.
|
- Docs: fix language switcher ordering and Japanese locale flag in Mintlify nav. (#12023) Thanks @joshp123.
|
||||||
|
|||||||
@@ -78,4 +78,41 @@ describe("registerTelegramNativeCommands", () => {
|
|||||||
|
|
||||||
expect(listSkillCommandsForAgents).toHaveBeenCalledWith({ cfg });
|
expect(listSkillCommandsForAgents).toHaveBeenCalledWith({ cfg });
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("truncates Telegram command registration to 100 commands", () => {
|
||||||
|
const cfg: OpenClawConfig = {
|
||||||
|
commands: { native: false },
|
||||||
|
};
|
||||||
|
const customCommands = Array.from({ length: 120 }, (_, index) => ({
|
||||||
|
command: `cmd_${index}`,
|
||||||
|
description: `Command ${index}`,
|
||||||
|
}));
|
||||||
|
const setMyCommands = vi.fn().mockResolvedValue(undefined);
|
||||||
|
const runtimeLog = vi.fn();
|
||||||
|
|
||||||
|
registerTelegramNativeCommands({
|
||||||
|
...buildParams(cfg),
|
||||||
|
bot: {
|
||||||
|
api: {
|
||||||
|
setMyCommands,
|
||||||
|
sendMessage: vi.fn().mockResolvedValue(undefined),
|
||||||
|
},
|
||||||
|
command: vi.fn(),
|
||||||
|
} as unknown as Parameters<typeof registerTelegramNativeCommands>[0]["bot"],
|
||||||
|
runtime: { log: runtimeLog } as RuntimeEnv,
|
||||||
|
telegramCfg: { customCommands } as TelegramAccountConfig,
|
||||||
|
nativeEnabled: false,
|
||||||
|
nativeSkillsEnabled: false,
|
||||||
|
});
|
||||||
|
|
||||||
|
const registeredCommands = setMyCommands.mock.calls[0]?.[0] as Array<{
|
||||||
|
command: string;
|
||||||
|
description: string;
|
||||||
|
}>;
|
||||||
|
expect(registeredCommands).toHaveLength(100);
|
||||||
|
expect(registeredCommands).toEqual(customCommands.slice(0, 100));
|
||||||
|
expect(runtimeLog).toHaveBeenCalledWith(
|
||||||
|
"telegram: truncating 120 commands to 100 (Telegram Bot API limit)",
|
||||||
|
);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user