Skip to content

docs: refresh stale auth-API references in JSDoc#84

Merged
kallebysantos merged 1 commit into
mainfrom
tomas/fix-stale-auth-jsdoc
Jun 30, 2026
Merged

docs: refresh stale auth-API references in JSDoc#84
kallebysantos merged 1 commit into
mainfrom
tomas/fix-stale-auth-jsdoc

Conversation

@tomaspozo

Copy link
Copy Markdown
Member

What

The allowauth / 'always''none' / 'public''publishable' / authTypeauthMode rename (landed in #48) was already applied across the prose docs and SKILL.md, but a few JSDoc reference comments — which feed the typedoc-generated API reference — still used the old names.

Changes

  • src/types.tsAuthResult doc: "public""publishable"; keyName doc: "always""none"
  • src/adapters/elysia/plugin.ts — both @example blocks: allow: 'user'auth: 'user'

Not touched (intentional)

  • Deprecation-coverage tests (verify-credentials.test.ts, with-supabase.test.ts) that exercise the legacy allow option on purpose
  • The allow backward-compat wiring (create-supabase-context.ts, deprecation.ts) and MIGRATION.md

JSDoc-comment-only edits — no compilation impact.

🤖 Generated with Claude Code

The allow→auth / 'always'→'none' / 'public'→'publishable' /
authType→authMode rename was already applied across the prose docs,
but a few JSDoc reference comments (which feed the typedoc API
reference) still used the old names:

- AuthResult doc: "public" → "publishable"
- keyName doc: "always" → "none"
- Elysia plugin @example blocks: allow: 'user' → auth: 'user'
@tomaspozo tomaspozo requested review from a team as code owners June 19, 2026 20:50
@pkg-pr-new

pkg-pr-new Bot commented Jun 19, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@supabase/server@84

commit: 6d9af7f

@kallebysantos kallebysantos merged commit 69ec52d into main Jun 30, 2026
7 checks passed
@kallebysantos kallebysantos deleted the tomas/fix-stale-auth-jsdoc branch June 30, 2026 11:17
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