diff --git a/src/plugin-handlers/agent-key-remapper.test.ts b/src/plugin-handlers/agent-key-remapper.test.ts index 179f54d10..34e1f08e7 100644 --- a/src/plugin-handlers/agent-key-remapper.test.ts +++ b/src/plugin-handlers/agent-key-remapper.test.ts @@ -2,7 +2,7 @@ import { describe, it, expect } from "bun:test" import { remapAgentKeysToDisplayNames } from "./agent-key-remapper" describe("remapAgentKeysToDisplayNames", () => { - it("remaps known agent keys to display names while preserving original keys", () => { + it("remaps known agent keys to display names", () => { // given agents with lowercase keys const agents = { sisyphus: { prompt: "test", mode: "primary" }, @@ -12,11 +12,10 @@ describe("remapAgentKeysToDisplayNames", () => { // when remapping const result = remapAgentKeysToDisplayNames(agents) - // then known agents get display name keys and original keys remain accessible + // then known agents get display name keys only expect(result["Sisyphus (Ultraworker)"]).toBeDefined() expect(result["oracle"]).toBeDefined() - expect(result["sisyphus"]).toBeDefined() - expect(result["Sisyphus (Ultraworker)"]).toBe(result["sisyphus"]) + expect(result["sisyphus"]).toBeUndefined() }) it("preserves unknown agent keys unchanged", () => { @@ -32,7 +31,7 @@ describe("remapAgentKeysToDisplayNames", () => { expect(result["custom-agent"]).toBeDefined() }) - it("remaps all core agents while preserving original keys", () => { + it("remaps all core agents to display names", () => { // given all core agents const agents = { sisyphus: {}, @@ -47,20 +46,20 @@ describe("remapAgentKeysToDisplayNames", () => { // when remapping const result = remapAgentKeysToDisplayNames(agents) - // then all get display name keys while original keys still work + // then all get display name keys without lowercase duplicates expect(result["Sisyphus (Ultraworker)"]).toBeDefined() - expect(result["sisyphus"]).toBeDefined() + expect(result["sisyphus"]).toBeUndefined() expect(result["Hephaestus (Deep Agent)"]).toBeDefined() - expect(result["hephaestus"]).toBeDefined() + expect(result["hephaestus"]).toBeUndefined() expect(result["Prometheus (Plan Builder)"]).toBeDefined() - expect(result["prometheus"]).toBeDefined() + expect(result["prometheus"]).toBeUndefined() expect(result["Atlas (Plan Executor)"]).toBeDefined() - expect(result["atlas"]).toBeDefined() + expect(result["atlas"]).toBeUndefined() expect(result["Metis (Plan Consultant)"]).toBeDefined() - expect(result["metis"]).toBeDefined() + expect(result["metis"]).toBeUndefined() expect(result["Momus (Plan Critic)"]).toBeDefined() - expect(result["momus"]).toBeDefined() + expect(result["momus"]).toBeUndefined() expect(result["Sisyphus-Junior"]).toBeDefined() - expect(result["sisyphus-junior"]).toBeDefined() + expect(result["sisyphus-junior"]).toBeUndefined() }) }) diff --git a/src/plugin-handlers/agent-key-remapper.ts b/src/plugin-handlers/agent-key-remapper.ts index c60bcfcb9..dd2a127e0 100644 --- a/src/plugin-handlers/agent-key-remapper.ts +++ b/src/plugin-handlers/agent-key-remapper.ts @@ -9,7 +9,6 @@ export function remapAgentKeysToDisplayNames( const displayName = AGENT_DISPLAY_NAMES[key] if (displayName && displayName !== key) { result[displayName] = value - result[key] = value } else { result[key] = value }