-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add code reference links and backticks to JSDoc comments #1484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add code reference links and backticks to JSDoc comments #1484
Conversation
|
@modelcontextprotocol/client
@modelcontextprotocol/server
@modelcontextprotocol/express
@modelcontextprotocol/hono
@modelcontextprotocol/node
commit: |
067ff0a to
3e75363
Compare
Replace plain text and backtick-quoted symbol references in JSDoc
comments with proper `{@linkcode}` tags for TypeDoc. This makes class
names, function names, and other symbols clickable in the generated API
documentation.
Also adds type-only imports with ESLint disable comments for symbols
that are only referenced in JSDoc `{@link}` tags.
Co-Authored-By: Claude Opus 4.5 <[email protected]>
Wrap code identifiers in backticks throughout JSDoc documentation for consistency. This includes parameter names, variable names, HTTP headers, OAuth grant types, URL paths, and string literals. Co-Authored-By: Claude Opus 4.5 <[email protected]>
Use `index.` prefix in `{@linkcode}` tags to reference symbols via the
package index re-exports, eliminating the need for unused imports and
their eslint-disable comments.
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
3e75363 to
53a9bca
Compare
This PR contains 2 somewhat overlapping commits which add the following to JSDoc comments across the code base:
{@linkcode}tags to cross-link code references, both for ease of reference and to catch docs drift (viapnpm run docs:check)The diff is somewhat large, but essentially uniform. The additional lines (404 lines vs 414 lines) are due to extra import statements and
// eslint-disable-next-line @typescript-eslint/no-unused-varscomments to allow TypeDoc to reference previously-unimported types in some{@linkcode}tags (i.e., the imports are used by TypeDoc, but eslint doesn't recognize that).