mirror of
https://github.com/openclaw/openclaw.git
synced 2026-05-09 19:24:31 +00:00
In multi-account Telegram configurations, `mergeTelegramAccountConfig()` performs a shallow merge of channel-level config onto each account. This causes channel-level `groups` to be inherited by ALL accounts, including those whose bots are not members of the configured groups. When a secondary bot attempts to handle group messages for a group it is not in, the failure disrupts message delivery for all accounts — causing silent message loss with no errors in logs. Fix: exclude `groups` from the base spread (like `accounts` already is) and only apply channel-level groups as fallback in single-account setups for backward compatibility. Multi-account setups must use account-level groups config. Added 5 test cases covering single-account inheritance, multi-account isolation, account-level priority, and backward compatibility. Fixes #30673
272 lines
7.8 KiB
TypeScript
272 lines
7.8 KiB
TypeScript
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
import type { OpenClawConfig } from "../config/config.js";
|
|
import { withEnv } from "../test-utils/env.js";
|
|
import { listTelegramAccountIds, resolveTelegramAccount } from "./accounts.js";
|
|
|
|
const { warnMock } = vi.hoisted(() => ({
|
|
warnMock: vi.fn(),
|
|
}));
|
|
|
|
vi.mock("../logging/subsystem.js", () => ({
|
|
createSubsystemLogger: () => {
|
|
const logger = {
|
|
warn: warnMock,
|
|
child: () => logger,
|
|
};
|
|
return logger;
|
|
},
|
|
}));
|
|
|
|
describe("resolveTelegramAccount", () => {
|
|
afterEach(() => {
|
|
warnMock.mockClear();
|
|
});
|
|
|
|
it("falls back to the first configured account when accountId is omitted", () => {
|
|
withEnv({ TELEGRAM_BOT_TOKEN: "" }, () => {
|
|
const cfg: OpenClawConfig = {
|
|
channels: {
|
|
telegram: { accounts: { work: { botToken: "tok-work" } } },
|
|
},
|
|
};
|
|
|
|
const account = resolveTelegramAccount({ cfg });
|
|
expect(account.accountId).toBe("work");
|
|
expect(account.token).toBe("tok-work");
|
|
expect(account.tokenSource).toBe("config");
|
|
});
|
|
});
|
|
|
|
it("uses TELEGRAM_BOT_TOKEN when default account config is missing", () => {
|
|
withEnv({ TELEGRAM_BOT_TOKEN: "tok-env" }, () => {
|
|
const cfg: OpenClawConfig = {
|
|
channels: {
|
|
telegram: { accounts: { work: { botToken: "tok-work" } } },
|
|
},
|
|
};
|
|
|
|
const account = resolveTelegramAccount({ cfg });
|
|
expect(account.accountId).toBe("default");
|
|
expect(account.token).toBe("tok-env");
|
|
expect(account.tokenSource).toBe("env");
|
|
});
|
|
});
|
|
|
|
it("prefers default config token over TELEGRAM_BOT_TOKEN", () => {
|
|
withEnv({ TELEGRAM_BOT_TOKEN: "tok-env" }, () => {
|
|
const cfg: OpenClawConfig = {
|
|
channels: {
|
|
telegram: { botToken: "tok-config" },
|
|
},
|
|
};
|
|
|
|
const account = resolveTelegramAccount({ cfg });
|
|
expect(account.accountId).toBe("default");
|
|
expect(account.token).toBe("tok-config");
|
|
expect(account.tokenSource).toBe("config");
|
|
});
|
|
});
|
|
|
|
it("does not fall back when accountId is explicitly provided", () => {
|
|
withEnv({ TELEGRAM_BOT_TOKEN: "" }, () => {
|
|
const cfg: OpenClawConfig = {
|
|
channels: {
|
|
telegram: { accounts: { work: { botToken: "tok-work" } } },
|
|
},
|
|
};
|
|
|
|
const account = resolveTelegramAccount({ cfg, accountId: "default" });
|
|
expect(account.accountId).toBe("default");
|
|
expect(account.tokenSource).toBe("none");
|
|
expect(account.token).toBe("");
|
|
});
|
|
});
|
|
|
|
it("formats debug logs with inspect-style output when debug env is enabled", () => {
|
|
withEnv({ TELEGRAM_BOT_TOKEN: "", OPENCLAW_DEBUG_TELEGRAM_ACCOUNTS: "1" }, () => {
|
|
const cfg: OpenClawConfig = {
|
|
channels: {
|
|
telegram: { accounts: { work: { botToken: "tok-work" } } },
|
|
},
|
|
};
|
|
|
|
expect(listTelegramAccountIds(cfg)).toEqual(["work"]);
|
|
resolveTelegramAccount({ cfg, accountId: "work" });
|
|
});
|
|
|
|
const lines = warnMock.mock.calls.map(([line]) => String(line));
|
|
expect(lines).toContain("listTelegramAccountIds [ 'work' ]");
|
|
expect(lines).toContain("resolve { accountId: 'work', enabled: true, tokenSource: 'config' }");
|
|
});
|
|
});
|
|
|
|
describe("resolveTelegramAccount allowFrom precedence", () => {
|
|
it("prefers accounts.default allowlists over top-level for default account", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
allowFrom: ["top"],
|
|
groupAllowFrom: ["top-group"],
|
|
accounts: {
|
|
default: {
|
|
botToken: "123:default",
|
|
allowFrom: ["default"],
|
|
groupAllowFrom: ["default-group"],
|
|
},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "default",
|
|
});
|
|
|
|
expect(resolved.config.allowFrom).toEqual(["default"]);
|
|
expect(resolved.config.groupAllowFrom).toEqual(["default-group"]);
|
|
});
|
|
|
|
it("falls back to top-level allowlists for named account without overrides", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
allowFrom: ["top"],
|
|
groupAllowFrom: ["top-group"],
|
|
accounts: {
|
|
work: { botToken: "123:work" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "work",
|
|
});
|
|
|
|
expect(resolved.config.allowFrom).toEqual(["top"]);
|
|
expect(resolved.config.groupAllowFrom).toEqual(["top-group"]);
|
|
});
|
|
|
|
it("does not inherit default account allowlists for named account when top-level is absent", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
accounts: {
|
|
default: {
|
|
botToken: "123:default",
|
|
allowFrom: ["default"],
|
|
groupAllowFrom: ["default-group"],
|
|
},
|
|
work: { botToken: "123:work" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "work",
|
|
});
|
|
|
|
expect(resolved.config.allowFrom).toBeUndefined();
|
|
expect(resolved.config.groupAllowFrom).toBeUndefined();
|
|
});
|
|
});
|
|
|
|
describe("resolveTelegramAccount groups inheritance (#30673)", () => {
|
|
it("inherits channel-level groups in single-account setup", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
groups: { "-100123": { requireMention: false } },
|
|
accounts: {
|
|
default: { botToken: "123:default" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "default",
|
|
});
|
|
|
|
expect(resolved.config.groups).toEqual({ "-100123": { requireMention: false } });
|
|
});
|
|
|
|
it("does NOT inherit channel-level groups to secondary account in multi-account setup", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
groups: { "-100123": { requireMention: false } },
|
|
accounts: {
|
|
default: { botToken: "123:default" },
|
|
dev: { botToken: "456:dev" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "dev",
|
|
});
|
|
|
|
expect(resolved.config.groups).toBeUndefined();
|
|
});
|
|
|
|
it("does NOT inherit channel-level groups to default account in multi-account setup", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
groups: { "-100123": { requireMention: false } },
|
|
accounts: {
|
|
default: { botToken: "123:default" },
|
|
dev: { botToken: "456:dev" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "default",
|
|
});
|
|
|
|
expect(resolved.config.groups).toBeUndefined();
|
|
});
|
|
|
|
it("uses account-level groups even in multi-account setup", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
groups: { "-100999": { requireMention: true } },
|
|
accounts: {
|
|
default: {
|
|
botToken: "123:default",
|
|
groups: { "-100123": { requireMention: false } },
|
|
},
|
|
dev: { botToken: "456:dev" },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "default",
|
|
});
|
|
|
|
expect(resolved.config.groups).toEqual({ "-100123": { requireMention: false } });
|
|
});
|
|
|
|
it("account-level groups takes priority over channel-level in single-account setup", () => {
|
|
const resolved = resolveTelegramAccount({
|
|
cfg: {
|
|
channels: {
|
|
telegram: {
|
|
groups: { "-100999": { requireMention: true } },
|
|
accounts: {
|
|
default: {
|
|
botToken: "123:default",
|
|
groups: { "-100123": { requireMention: false } },
|
|
},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
accountId: "default",
|
|
});
|
|
|
|
expect(resolved.config.groups).toEqual({ "-100123": { requireMention: false } });
|
|
});
|
|
});
|