fix(legacy-html): prevent partial nav class matches

[moshams272]

Description

The previous .replace() logic used a partial match (class="nav-${head.api}). For APIs like stream, the partial string class="nav-stream would incorrectly match the first occurrence it found, which happens to be class="nav-stream_iter". This resulted in corrupted class names class="active_iter nav-stream" and left the actual stream link unmodified:

The same for Modules: node:module API:

The Fix:

Added closing double-quotes to the search and replace strings to enforce an exact match (class="nav-${head.api}").

Validation

• Generated the legacy-html docs locally.
• Verified that the Stream and Modules: node:module API sidebar links are now correctly bolded (active class applied).

Related Issues

#761

Check List

• I have read the Contributing Guidelines and made commit messages that follow the guideline.
• I have run node --run test and all tests passed.
• I have check code formatting with node --run format & node --run lint.
• I've covered new added functionality with unit tests if necessary.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
api-docs-tooling	Ready	Preview	Apr 11, 2026 11:25am

[cursor]

PR Summary

Low Risk
Low risk string-match adjustment limited to legacy HTML navigation generation; main risk is only if navigation markup differs from the expected class="nav-..." format.

Overview
Fixes the legacy-html generator’s sidebar highlighting by tightening the .replace() match from a partial class="nav-${head.api} prefix to the full class="nav-${head.api}", preventing APIs with shared prefixes (e.g., stream vs stream_iter) from corrupting class names when adding the active state.

^{Reviewed by Cursor Bugbot for commit b0ddc10. Bugbot is set up for automated code reviews on this repo. Configure here.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.43%. Comparing base (4242a79) to head (b0ddc10).

Files with missing lines	Patch %	Lines
src/generators/legacy-html/generate.mjs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #762   +/-   ##
=======================================
  Coverage   78.43%   78.43%           
=======================================
  Files         157      157           
  Lines       13963    13963           
  Branches     1152     1152           
=======================================
  Hits        10952    10952           
  Misses       3006     3006           
  Partials        5        5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ovflowd]

Nice, can you verify this doesn't break existing scenarios?

[moshams272]

Oh, I forgot to mention that! 🤦🏻‍♂️

Actually, I already verified it. After tracing, I found that the links contain a one class in the attribute (nav-${api}) and the only class that added to it is active:

So, It will replace it correctly, and verified the output locally for every link to ensure that it didn't collision.