Skip to content

Commit 6b028d6

Browse files
committed
Fix inconsistent displayName field lookup in parseCodexSkillsResult
Read displayName from both the top-level skill object and the nested interface sub-object, consistent with how shortDescription is handled. Previously displayName was only read from skill.interface.displayName, silently dropping any top-level skill.displayName value.
1 parent 2d6e8fb commit 6b028d6

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

apps/server/src/provider/codexAppServer.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,11 @@ function parseCodexSkillsResult(result: unknown, cwd: string): ReadonlyArray<Ser
6464
? { description: nonEmptyTrimmed(skill?.description) }
6565
: {}),
6666
...(nonEmptyTrimmed(skill?.scope) ? { scope: nonEmptyTrimmed(skill?.scope) } : {}),
67-
...(nonEmptyTrimmed(display?.displayName)
68-
? { displayName: nonEmptyTrimmed(display?.displayName) }
67+
...(nonEmptyTrimmed(skill?.displayName) || nonEmptyTrimmed(display?.displayName)
68+
? {
69+
displayName:
70+
nonEmptyTrimmed(skill?.displayName) ?? nonEmptyTrimmed(display?.displayName),
71+
}
6972
: {}),
7073
...(nonEmptyTrimmed(skill?.shortDescription) || nonEmptyTrimmed(display?.shortDescription)
7174
? {

0 commit comments

Comments
 (0)