fix(discord): add missing type exports and fix unused imports
- Re-export DirectoryConfigParams and ChannelDirectoryEntry from channels/targets
- Remove unused ChannelDirectoryEntry and resolveDiscordAccount imports
- Fix parseDiscordTarget calls to not pass incompatible options type
- Fix unused catch parameter
Fixes CI build failures on main.
🤖 Generated with Claude Code
This commit is contained in:
parent
6fc3ca4996
commit
e59e6aa757
@ -1,3 +1,6 @@
|
|||||||
|
export type { DirectoryConfigParams } from "./plugins/directory-config.js";
|
||||||
|
export type { ChannelDirectoryEntry } from "./plugins/types.js";
|
||||||
|
|
||||||
export type MessagingTargetKind = "user" | "channel";
|
export type MessagingTargetKind = "user" | "channel";
|
||||||
|
|
||||||
export type MessagingTarget = {
|
export type MessagingTarget = {
|
||||||
|
|||||||
@ -10,7 +10,6 @@ import {
|
|||||||
import type { DirectoryConfigParams } from "../channels/plugins/directory-config.js";
|
import type { DirectoryConfigParams } from "../channels/plugins/directory-config.js";
|
||||||
|
|
||||||
import { listDiscordDirectoryPeersLive } from "./directory-live.js";
|
import { listDiscordDirectoryPeersLive } from "./directory-live.js";
|
||||||
import { resolveDiscordAccount } from "./accounts.js";
|
|
||||||
|
|
||||||
export type DiscordTargetKind = MessagingTargetKind;
|
export type DiscordTargetKind = MessagingTargetKind;
|
||||||
|
|
||||||
@ -104,7 +103,7 @@ export async function resolveDiscordTarget(
|
|||||||
const userId = match.id.replace(/^user:/, "");
|
const userId = match.id.replace(/^user:/, "");
|
||||||
return buildMessagingTarget("user", userId, trimmed);
|
return buildMessagingTarget("user", userId, trimmed);
|
||||||
}
|
}
|
||||||
} catch (error) {
|
} catch {
|
||||||
// Directory lookup failed - fall through to parse as-is
|
// Directory lookup failed - fall through to parse as-is
|
||||||
// This preserves existing behavior for channel names
|
// This preserves existing behavior for channel names
|
||||||
}
|
}
|
||||||
|
|||||||
@ -13,8 +13,12 @@ describe("resolveTelegramForumThreadId", () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
it("returns undefined for non-forum groups without messageThreadId", () => {
|
it("returns undefined for non-forum groups without messageThreadId", () => {
|
||||||
expect(resolveTelegramForumThreadId({ isForum: false, messageThreadId: undefined })).toBeUndefined();
|
expect(
|
||||||
expect(resolveTelegramForumThreadId({ isForum: undefined, messageThreadId: 99 })).toBeUndefined();
|
resolveTelegramForumThreadId({ isForum: false, messageThreadId: undefined }),
|
||||||
|
).toBeUndefined();
|
||||||
|
expect(
|
||||||
|
resolveTelegramForumThreadId({ isForum: undefined, messageThreadId: 99 }),
|
||||||
|
).toBeUndefined();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("returns General topic (1) for forum groups without messageThreadId", () => {
|
it("returns General topic (1) for forum groups without messageThreadId", () => {
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user