Merge pull request #2142 from code-yeongyu/fix/issue-1922-retain-agent-keys

fix(agents): retain original agent keys in remapAgentKeysToDisplayNames to prevent crash

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", () => {
+  it("remaps known agent keys to display names while preserving original keys", () => {
     // given agents with lowercase keys
     const agents = {
       sisyphus: { prompt: "test", mode: "primary" },
@@ -12,10 +12,11 @@ describe("remapAgentKeysToDisplayNames", () => {
     // when remapping
     const result = remapAgentKeysToDisplayNames(agents)
 
-    // then known agents get display name keys
+    // then known agents get display name keys and original keys remain accessible
     expect(result["Sisyphus (Ultraworker)"]).toBeDefined()
     expect(result["oracle"]).toBeDefined()
-    expect(result["sisyphus"]).toBeUndefined()
+    expect(result["sisyphus"]).toBeDefined()
+    expect(result["Sisyphus (Ultraworker)"]).toBe(result["sisyphus"])
   })
 
   it("preserves unknown agent keys unchanged", () => {
@@ -31,7 +32,7 @@ describe("remapAgentKeysToDisplayNames", () => {
     expect(result["custom-agent"]).toBeDefined()
   })
 
-  it("remaps all core agents", () => {
+  it("remaps all core agents while preserving original keys", () => {
     // given all core agents
     const agents = {
       sisyphus: {},
@@ -46,15 +47,20 @@ describe("remapAgentKeysToDisplayNames", () => {
     // when remapping
     const result = remapAgentKeysToDisplayNames(agents)
 
-    // then all get display name keys
+    // then all get display name keys while original keys still work
-    expect(Object.keys(result)).toEqual([
+    expect(result["Sisyphus (Ultraworker)"]).toBeDefined()
-      "Sisyphus (Ultraworker)",
+    expect(result["sisyphus"]).toBeDefined()
-      "Hephaestus (Deep Agent)",
+    expect(result["Hephaestus (Deep Agent)"]).toBeDefined()
-      "Prometheus (Plan Builder)",
+    expect(result["hephaestus"]).toBeDefined()
-      "Atlas (Plan Executor)",
+    expect(result["Prometheus (Plan Builder)"]).toBeDefined()
-      "Metis (Plan Consultant)",
+    expect(result["prometheus"]).toBeDefined()
-      "Momus (Plan Critic)",
+    expect(result["Atlas (Plan Executor)"]).toBeDefined()
-      "Sisyphus-Junior",
+    expect(result["atlas"]).toBeDefined()
-    ])
+    expect(result["Metis (Plan Consultant)"]).toBeDefined()
+    expect(result["metis"]).toBeDefined()
+    expect(result["Momus (Plan Critic)"]).toBeDefined()
+    expect(result["momus"]).toBeDefined()
+    expect(result["Sisyphus-Junior"]).toBeDefined()
+    expect(result["sisyphus-junior"]).toBeDefined()
   })
 })

src/plugin-handlers/agent-key-remapper.ts

@@ -9,6 +9,7 @@ export function remapAgentKeysToDisplayNames(
     const displayName = AGENT_DISPLAY_NAMES[key]
     if (displayName && displayName !== key) {
       result[displayName] = value
+      result[key] = value
     } else {
       result[key] = value
     }