Skip to content

fix(ApiLink): preserve enum member anchor case#41

Open
viktorkombov wants to merge 2 commits into
masterfrom
vkombov/preserve-enum-member-case
Open

fix(ApiLink): preserve enum member anchor case#41
viktorkombov wants to merge 2 commits into
masterfrom
vkombov/preserve-enum-member-case

Conversation

@viktorkombov
Copy link
Copy Markdown
Collaborator

TypeDoc emits PascalCase ids for enum members (e.g. id="Area"). The non-class branch was lowercasing the member, producing dead anchors like #area. Skip the lowercase step when kind === 'enum'.

TypeDoc emits PascalCase ids for enum members (e.g. id="Area"). The non-class branch was lowercasing the member, producing dead anchors like #area. Skip the lowercase step when kind === 'enum'.
@ChronosSF ChronosSF requested a review from Copilot May 21, 2026 08:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes ApiLink so enum member anchors remain case-correct (matching TypeDoc’s emitted enum member IDs), avoiding broken links like #area when the actual anchor is #Area.

Changes:

  • Preserve enum member anchor casing in the non-class URL branch by skipping the .toLowerCase() transformation when kind === 'enum'.
  • Add an inline comment documenting the TypeDoc enum member anchor casing behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +118 to +122
? member
: pkgConfig.pascalCaseMembers
? member.charAt(0).toUpperCase() + member.slice(1)
: member.toLowerCase()
}`
Comment on lines 115 to +122
const memberAnchorNonClass = member
? `#${pkgConfig.pascalCaseMembers ? member.charAt(0).toUpperCase() + member.slice(1) : member.toLowerCase()}`
? `#${
kind === 'enum'
? member
: pkgConfig.pascalCaseMembers
? member.charAt(0).toUpperCase() + member.slice(1)
: member.toLowerCase()
}`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants