From 58c92b63c6c970e478f138a5df0c8be135edf1c7 Mon Sep 17 00:00:00 2001 From: Kuzey Cimen Date: Thu, 29 Jan 2026 01:25:17 +0000 Subject: [PATCH] chore: remove outdated comments. --- src/discord/multi-account.test.ts | 10 +--------- src/routing/shared-channel.test.ts | 1 - 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/src/discord/multi-account.test.ts b/src/discord/multi-account.test.ts index 4020e072c..4c3cb066b 100644 --- a/src/discord/multi-account.test.ts +++ b/src/discord/multi-account.test.ts @@ -48,22 +48,14 @@ describe("discord multi-account logic", () => { discord: { token: "default_token", accounts: { - bot_a: {}, // no token + bot_a: {}, }, }, }, } as unknown as MoltbotConfig; const tokenA = resolveDiscordToken(config, { accountId: "bot_a" }); - // Current implementation might strictly require per-account token if it's not the default account. - // Let's verify behavior. - // Actually, looking at token.ts: - // const accountToken = ... - // if (accountToken) return ... - // const allowEnv = accountId === DEFAULT_ACCOUNT_ID; - // const configToken = allowEnv ? ... : undefined; - // So valid behavior: non-default accounts MUST have a specific token. expect(tokenA.token).toBe(""); }); diff --git a/src/routing/shared-channel.test.ts b/src/routing/shared-channel.test.ts index d94b5da09..a2e56f2b0 100644 --- a/src/routing/shared-channel.test.ts +++ b/src/routing/shared-channel.test.ts @@ -53,7 +53,6 @@ describe("resolveAgentRoute - Shared Channel", () => { const route = resolveAgentRoute(input); console.log("Channel Default:", route.sessionKey); - // Should NOT have :default expect(route.sessionKey).toBe("agent:main:discord:channel:channel_123"); }); });