docs(shared): fix touchSession Frontend API link#9097
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: a6cf420 The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 packages
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 |
📝 WalkthroughWalkthroughThis PR updates the JSDoc documentation link for the ChangesDocumentation link fix
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
packages/shared/src/types/clerk.ts (1)
1404-1404: 🧹 Nitpick | 🔵 TrivialDocs team should sanity-check the generated reference output.
This is public-facing JSDoc under
packages/**, so the link change will flow into Clerk Docs. As per path instructions, “JSDoc on public/reference-facing APIs is customer-facing documentation, not ordinary internal comments.”🤖 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/clerk.ts` at line 1404, The JSDoc on the public API in clerk.ts is customer-facing and feeds Clerk Docs, so the generated reference output should be sanity-checked after this link change. Review the affected documentation comment and confirm the rendered docs text and link target are correct in the generated reference for the relevant public symbol(s), especially the session touch behavior description, before merging.Source: Path instructions
🤖 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.
Nitpick comments:
In `@packages/shared/src/types/clerk.ts`:
- Line 1404: The JSDoc on the public API in clerk.ts is customer-facing and
feeds Clerk Docs, so the generated reference output should be sanity-checked
after this link change. Review the affected documentation comment and confirm
the rendered docs text and link target are correct in the generated reference
for the relevant public symbol(s), especially the session touch behavior
description, before merging.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Repository UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 16430c28-8725-4086-984f-db13e3352a67
📒 Files selected for processing (2)
.changeset/fix-touch-session-doc-link.mdpackages/shared/src/types/clerk.ts
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
API Changes Report
Summary
No API Changes DetectedAll packages have stable APIs with no detected changes. Report generated by Break Check Last ran on |
Description
touchSessionoption JSDoc to link directly to the canonical Scalar Frontend API touch endpoint.@clerk/shared.Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
touchSessionoption so it now points directly to the correct Frontend API session endpoint.