fix(telegram): preserve inline buttons when capabilities array is empty

hanZeng-08 · hanZeng-08 · commit 96661e886dd0 · 2026-06-23T20:57:46.000+08:00
Empty array capabilities should not disable inline buttons. The default inline buttons scope ('allowlist') should still apply when the user has set an empty capabilities array, rather than treating it as an explicit 'off' signal. Fixes #96098
diff --git a/extensions/telegram/src/inline-buttons.test.ts b/extensions/telegram/src/inline-buttons.test.ts
@@ -109,6 +109,20 @@ describe("resolveTelegramInlineButtonsScope (#75433 SecretRef tolerance)", () =>
     expect(isTelegramInlineButtonsEnabled({ cfg })).toBe(true);
   });
 
+  it("falls back to default when array capabilities is empty", () => {
+    const cfg = {
+      channels: {
+        telegram: {
+          botToken: { source: "exec", provider: "default", id: "telegram-token" },
+          capabilities: [],
+        },
+      },
+    } as unknown as OpenClawConfig;
+
+    expect(resolveTelegramInlineButtonsScope({ cfg })).toBe("allowlist");
+    expect(isTelegramInlineButtonsEnabled({ cfg })).toBe(true);
+  });
+
   it('preserves configured "off" when botToken is an unresolved SecretRef', () => {
     const cfg = {
       channels: {
diff --git a/extensions/telegram/src/inline-buttons.ts b/extensions/telegram/src/inline-buttons.ts
@@ -47,9 +47,13 @@ export function resolveTelegramInlineButtonsScopeFromCapabilities(
     return DEFAULT_INLINE_BUTTONS_SCOPE;
   }
   if (Array.isArray(capabilities)) {
-    const enabled = capabilities.some(
-      (entry) => normalizeLowercaseStringOrEmpty(String(entry)) === "inlinebuttons",
-    );
+    const normalized = capabilities
+      .map((entry) => normalizeLowercaseStringOrEmpty(String(entry)))
+      .filter(Boolean);
+    if (normalized.length === 0) {
+      return DEFAULT_INLINE_BUTTONS_SCOPE;
+    }
+    const enabled = normalized.includes("inlinebuttons");
     return enabled ? "all" : "off";
   }
   if (typeof capabilities === "object") {
