luna/oh-my-opencode
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #3944 from survivor998/fix/ultraworker-display-name-zwsp

Browse Source 
fix(display): lowercase ultraworker to avoid ZWSP rendering glitch
This commit is contained in:
YeonGyu-Kim
2026-05-21 00:45:07 +09:00
committed by GitHub
parent 10ad8d87e9 cd39f8858e
commit 7651c856aa
2 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/shared/agent-display-names.test.ts
+10 -10
View File
@@ -9,8 +9,8 @@ describe("getAgentDisplayName", () => {
// when getAgentDisplayName called // when getAgentDisplayName called
const result = getAgentDisplayName(configKey) const result = getAgentDisplayName(configKey)
// then returns "Sisyphus - Ultraworker" // then returns "Sisyphus - ultraworker"
expect(result).toBe("Sisyphus - Ultraworker") expect(result).toBe("Sisyphus - ultraworker")
}) })
it("returns display name for uppercase config key (old format - case-insensitive)", () => { it("returns display name for uppercase config key (old format - case-insensitive)", () => {
@@ -20,8 +20,8 @@ describe("getAgentDisplayName", () => {
// when getAgentDisplayName called // when getAgentDisplayName called
const result = getAgentDisplayName(configKey) const result = getAgentDisplayName(configKey)
// then returns "Sisyphus - Ultraworker" (case-insensitive lookup) // then returns "Sisyphus - ultraworker" (case-insensitive lookup)
expect(result).toBe("Sisyphus - Ultraworker") expect(result).toBe("Sisyphus - ultraworker")
}) })
it("returns original key for unknown agents (fallback)", () => { it("returns original key for unknown agents (fallback)", () => {
@@ -137,10 +137,10 @@ describe("getAgentDisplayName", () => {
describe("getAgentConfigKey", () => { describe("getAgentConfigKey", () => {
it("resolves display name to config key", () => { it("resolves display name to config key", () => {
// given display name "Sisyphus - Ultraworker" // given display name "Sisyphus - ultraworker"
// when getAgentConfigKey called // when getAgentConfigKey called
// then returns "sisyphus" // then returns "sisyphus"
expect(getAgentConfigKey("Sisyphus - Ultraworker")).toBe("sisyphus") expect(getAgentConfigKey("Sisyphus - ultraworker")).toBe("sisyphus")
}) })
it("resolves display name case-insensitively", () => { it("resolves display name case-insensitively", () => {
@@ -195,7 +195,7 @@ describe("getAgentConfigKey", () => {
describe("getAgentListDisplayName", () => { describe("getAgentListDisplayName", () => {
it("returns the canonical display name for the core agent list", () => { it("returns the canonical display name for the core agent list", () => {
expect(getAgentListDisplayName("sisyphus")).toBe("Sisyphus - Ultraworker") expect(getAgentListDisplayName("sisyphus")).toBe("Sisyphus - ultraworker")
expect(getAgentListDisplayName("hephaestus")).toBe("Hephaestus - Deep Agent") expect(getAgentListDisplayName("hephaestus")).toBe("Hephaestus - Deep Agent")
expect(getAgentListDisplayName("prometheus")).toBe("Prometheus - Plan Builder") expect(getAgentListDisplayName("prometheus")).toBe("Prometheus - Plan Builder")
expect(getAgentListDisplayName("atlas")).toBe("Atlas - Plan Executor") expect(getAgentListDisplayName("atlas")).toBe("Atlas - Plan Executor")
@@ -222,14 +222,14 @@ describe("stripAgentListSortPrefix", () => {
describe("normalizeAgentForPrompt", () => { describe("normalizeAgentForPrompt", () => {
it("strips core UI ordering prefixes back to canonical display names", () => { it("strips core UI ordering prefixes back to canonical display names", () => {
expect(normalizeAgentForPrompt(getAgentListDisplayName("sisyphus"))).toBe("Sisyphus - Ultraworker") expect(normalizeAgentForPrompt(getAgentListDisplayName("sisyphus"))).toBe("Sisyphus - ultraworker")
expect(normalizeAgentForPrompt(getAgentListDisplayName("hephaestus"))).toBe("Hephaestus - Deep Agent") expect(normalizeAgentForPrompt(getAgentListDisplayName("hephaestus"))).toBe("Hephaestus - Deep Agent")
expect(normalizeAgentForPrompt(getAgentListDisplayName("prometheus"))).toBe("Prometheus - Plan Builder") expect(normalizeAgentForPrompt(getAgentListDisplayName("prometheus"))).toBe("Prometheus - Plan Builder")
expect(normalizeAgentForPrompt(getAgentListDisplayName("atlas"))).toBe("Atlas - Plan Executor") expect(normalizeAgentForPrompt(getAgentListDisplayName("atlas"))).toBe("Atlas - Plan Executor")
}) })
it("removes zero-width characters before returning canonical names", () => { it("removes zero-width characters before returning canonical names", () => {
expect(normalizeAgentForPrompt("Sisyphus\u200B - Ultraworker")).toBe("Sisyphus - Ultraworker") expect(normalizeAgentForPrompt("Sisyphus\u200B - Ultraworker")).toBe("Sisyphus - ultraworker")
}) })
it("converts legacy parenthesized names to canonical display names", () => { it("converts legacy parenthesized names to canonical display names", () => {
@@ -255,7 +255,7 @@ describe("AGENT_DISPLAY_NAMES", () => {
it("contains all expected agent mappings", () => { it("contains all expected agent mappings", () => {
// given expected mappings // given expected mappings
const expectedMappings = { const expectedMappings = {
sisyphus: "Sisyphus - Ultraworker", sisyphus: "Sisyphus - ultraworker",
hephaestus: "Hephaestus - Deep Agent", hephaestus: "Hephaestus - Deep Agent",
prometheus: "Prometheus - Plan Builder", prometheus: "Prometheus - Plan Builder",
atlas: "Atlas - Plan Executor", atlas: "Atlas - Plan Executor",
src/shared/agent-display-names.ts
+1 -1
View File
@@ -10,7 +10,7 @@
* type selector dropdown. Use ` - ` (space-dash-space) instead of `(...)`. * type selector dropdown. Use ` - ` (space-dash-space) instead of `(...)`.
*/ */
export const AGENT_DISPLAY_NAMES: Record<string, string> = { export const AGENT_DISPLAY_NAMES: Record<string, string> = {
sisyphus: "Sisyphus - Ultraworker", sisyphus: "Sisyphus - ultraworker",
hephaestus: "Hephaestus - Deep Agent", hephaestus: "Hephaestus - Deep Agent",
prometheus: "Prometheus - Plan Builder", prometheus: "Prometheus - Plan Builder",
atlas: "Atlas - Plan Executor", atlas: "Atlas - Plan Executor",