From b5a94d274bb8a5a27bfc79e0d2b3d838d3c19064 Mon Sep 17 00:00:00 2001 From: Shakker Date: Thu, 5 Mar 2026 04:59:41 +0000 Subject: [PATCH] style(skills): align formatting cleanup for dedupe changes --- src/auto-reply/skill-commands.test.ts | 2 -- src/auto-reply/skill-commands.ts | 7 ------- src/discord/monitor/provider.ts | 1 - 3 files changed, 10 deletions(-) diff --git a/src/auto-reply/skill-commands.test.ts b/src/auto-reply/skill-commands.test.ts index 904d0faf7..b6f6e8639 100644 --- a/src/auto-reply/skill-commands.test.ts +++ b/src/auto-reply/skill-commands.test.ts @@ -147,10 +147,8 @@ describe("listSkillCommandsForAgents", () => { }, }); const names = commands.map((entry) => entry.name); - // demo-skill appears in both workspaces; only the first registration (demo_skill) survives. expect(names).toContain("demo_skill"); expect(names).not.toContain("demo_skill_2"); - // extra-skill is unique to the research workspace and should be present. expect(names).toContain("extra_skill"); }); diff --git a/src/auto-reply/skill-commands.ts b/src/auto-reply/skill-commands.ts index 458469e9a..4a184ecd3 100644 --- a/src/auto-reply/skill-commands.ts +++ b/src/auto-reply/skill-commands.ts @@ -46,10 +46,6 @@ export function listSkillCommandsForWorkspace(params: { }); } -// Deduplicate skill commands by skillName, keeping the first registration. -// When multiple agents have a skill with the same name (e.g. one with a -// workspace override and one from bundled), the suffix-renamed entries -// (github_2, github_3…) are dropped so every interface sees a clean list. function dedupeBySkillName(commands: SkillCommandSpec[]): SkillCommandSpec[] { const seen = new Set(); const out: SkillCommandSpec[] = []; @@ -129,9 +125,6 @@ export function listSkillCommandsForAgents(params: { entries.push(command); } } - // Dedupe by skillName across workspaces so every interface (Discord, TUI, - // Slack, text) sees a consistent command list without platform-specific - // workarounds. return dedupeBySkillName(entries); } diff --git a/src/discord/monitor/provider.ts b/src/discord/monitor/provider.ts index e6004af97..defa73d52 100644 --- a/src/discord/monitor/provider.ts +++ b/src/discord/monitor/provider.ts @@ -126,7 +126,6 @@ function formatThreadBindingDurationForConfigLabel(durationMs: number): string { return label === "disabled" ? "off" : label; } - function appendPluginCommandSpecs(params: { commandSpecs: NativeCommandSpec[]; runtime: RuntimeEnv;