fix(discord): prefer names in allowlist resolution logs

This commit is contained in:
Peter Steinberger
2026-03-02 04:31:04 +00:00
parent b7615e0ce3
commit 8b05e4091c
2 changed files with 5 additions and 3 deletions

View File

@@ -91,6 +91,6 @@ describe("resolveDiscordAllowlistConfig", () => {
expect(logs).toContain( expect(logs).toContain(
"discord channels unresolved: 145/c404 (guild:Ops; channel:missing-room)", "discord channels unresolved: 145/c404 (guild:Ops; channel:missing-room)",
); );
expect(logs).toContain("discord users resolved: 387→387 (name:Peter)"); expect(logs).toContain("discord users resolved: 387→Peter (id:387)");
}); });
}); });

View File

@@ -63,9 +63,11 @@ function formatDiscordChannelUnresolved(entry: DiscordChannelLogEntry): string {
} }
function formatDiscordUserResolved(entry: DiscordUserLogEntry): string { function formatDiscordUserResolved(entry: DiscordUserLogEntry): string {
const base = `${entry.input}${entry.id}`; const displayName = entry.name?.trim();
const target = displayName || entry.id;
const base = `${entry.input}${target}`;
return formatResolutionLogDetails(base, [ return formatResolutionLogDetails(base, [
entry.name ? `name:${entry.name}` : undefined, displayName && entry.id ? `id:${entry.id}` : undefined,
entry.guildName ? `guild:${entry.guildName}` : undefined, entry.guildName ? `guild:${entry.guildName}` : undefined,
entry.note, entry.note,
]); ]);