Skip to content

docs(shared): fix broken Clerk docs links in organization JSDoc#8825

Merged
alexisintech merged 4 commits into
mainfrom
aa/fix-broken-links
Jun 11, 2026
Merged

docs(shared): fix broken Clerk docs links in organization JSDoc#8825
alexisintech merged 4 commits into
mainfrom
aa/fix-broken-links

Conversation

@alexisintech

@alexisintech alexisintech commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

  • Fix four JSDoc-linked URLs that 404'd: types/organizationinvitationtypes/organization-invitation, types/organizationsuggestiontypes/organization-suggestion, and types/organizationobjects/organization.
  • Affects OrganizationDomainResource, OrganizationMembershipResource, OrganizationSettingsResource, and UpdateEnrollmentModeParams.

Test plan

  • Hover any of the touched fields in an IDE; confirm the link target resolves on clerk.com instead of 404ing.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Fixed and standardized reference links in JSDoc for organization and billing resources across the codebase.
  • Chores
    • Added a changeset recording the documentation fixes.

The JSDoc-linked URLs for `organization-invitation`, `organization-suggestion`, and the `Organization` reference page all 404'd because they used the old un-hyphenated slugs and `types/organization` instead of `objects/organization`. Hover docs in IDEs now resolve to the correct pages.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 13d81f2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Jun 11, 2026 1:25am
swingset Ready Ready Preview, Comment Jun 11, 2026 1:25am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 4d4c149a-e527-4124-ac58-b83d223fbc8e

📥 Commits

Reviewing files that changed from the base of the PR and between 5a9ddd3 and 13d81f2.

📒 Files selected for processing (1)
  • packages/shared/src/types/billing.ts
✅ Files skipped from review due to trivial changes (1)
  • packages/shared/src/types/billing.ts

📝 Walkthrough

Walkthrough

This PR updates JSDoc links in organization-related type definitions to hyphenated documentation paths, adds a changeset recording those edits, and reformats a few billing JSDoc comment lines without changing any types or runtime behavior.

Changes

Organization JSDoc Link Corrections

Layer / File(s) Summary
JSDoc link corrections and changeset
.changeset/fix-organization-jsdoc-links.md, packages/shared/src/types/organizationDomain.ts, packages/shared/src/types/organizationMembership.ts, packages/shared/src/types/organizationSettings.ts
JSDoc links updated from legacy slugs (organizationinvitation, organizationsuggestion, /reference/types/organization) to hyphenated paths (organization-invitation, organization-suggestion, /reference/objects/organization); changeset added.
Billing JSDoc formatting
packages/shared/src/types/billing.ts
Adjusted JSDoc wording/wrapping for GetPlansParams.orgId and reformatted comments for the ConfirmCheckoutParams discriminated union; no type or signature changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Links hopped back in line, with hyphens neat and spry,
Docs point where they should, beneath a clearer sky,
Four notes and billing's trim—no code was changed today,
A tiny changeset nibbles errors all away.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing broken Clerk docs links in organization JSDoc comments across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comments-only change; no package patch needed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@pkg-pr-new

pkg-pr-new Bot commented Jun 11, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8825

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8825

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8825

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8825

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8825

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8825

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8825

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8825

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8825

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8825

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8825

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8825

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8825

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8825

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8825

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8825

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8825

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8825

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8825

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8825

commit: 13d81f2

@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-06-11T01:27:18.572Z

Summary

Metric Count
Packages analyzed 19
Packages with changes 0
🔴 Breaking changes 0
🟡 Non-breaking changes 0
🟢 Additions 0

Note
Break Check could not snapshot 3 subpaths; the diff below excludes them.

  • @clerk/astro ./env: Internal Error: Unable to determine module for: /home/runner/_work/javascript/javascript/packages/astro/env.d.ts You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
  • @clerk/shared ./cookie: Internal Error: Unable to follow symbol for "Cookies" You have encountered a software defect. Please consider reporting the issue to the maintainers of this application.
  • @clerk/testing ./cypress: Symbol not found for identifier: Cypress

No API Changes Detected

All packages have stable APIs with no detected changes.


Report generated by Break Check

Last ran on 13d81f2.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/shared/src/types/organizationDomain.ts (1)

83-83: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Missed broken link: Line 83 still uses the old organizationsuggestion slug.

Line 83 references https://clerk.com/docs/reference/types/organizationsuggestion, which is the old non-hyphenated slug. This should be updated to organization-suggestion to match the fix applied at line 135 and align with the existing hyphenated format used in organizationSuggestion.ts.

📝 Proposed fix
  /**
-  * The total number of pending [suggestions](https://clerk.com/docs/reference/types/organizationsuggestion) associated with this domain.
+  * The total number of pending [suggestions](https://clerk.com/docs/reference/types/organization-suggestion) associated with this domain.
   */

Based on coding guidelines, JSDoc on public APIs is customer-facing documentation that may render in generated Clerk Docs. This link correction aligns with the PR's objective to fix broken documentation URLs.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/shared/src/types/organizationDomain.ts` at line 83, Update the
broken JSDoc URL in the organization domain type: locate the JSDoc comment in
packages/shared/src/types/organizationDomain.ts that contains the link
"https://clerk.com/docs/reference/types/organizationsuggestion" (the doc comment
describing pending suggestions) and replace that slug with the hyphenated form
"https://clerk.com/docs/reference/types/organization-suggestion" so it matches
the fixed link used elsewhere (e.g., organizationSuggestion.ts).

Source: Coding guidelines

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@packages/shared/src/types/organizationDomain.ts`:
- Line 83: Update the broken JSDoc URL in the organization domain type: locate
the JSDoc comment in packages/shared/src/types/organizationDomain.ts that
contains the link
"https://clerk.com/docs/reference/types/organizationsuggestion" (the doc comment
describing pending suggestions) and replace that slug with the hyphenated form
"https://clerk.com/docs/reference/types/organization-suggestion" so it matches
the fixed link used elsewhere (e.g., organizationSuggestion.ts).

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: f5d2b045-1600-4c35-8f87-4563c8244e7b

📥 Commits

Reviewing files that changed from the base of the PR and between b7cac0a and dd9d6c8.

📒 Files selected for processing (4)
  • .changeset/fix-organization-jsdoc-links.md
  • packages/shared/src/types/organizationDomain.ts
  • packages/shared/src/types/organizationMembership.ts
  • packages/shared/src/types/organizationSettings.ts

CI format:check was failing on union type member indentation.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@alexisintech alexisintech enabled auto-merge (squash) June 11, 2026 01:24
@alexisintech alexisintech merged commit 07e90c1 into main Jun 11, 2026
49 checks passed
@alexisintech alexisintech deleted the aa/fix-broken-links branch June 11, 2026 01:31
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