fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk
, 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports - #9434

Merged
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426
Aug 13, 2026
Merged

fix(astro,chrome-extension): Add missing Astro and Chrome Extension exports#9434
wobsoriano merged 1 commit into
mainfrom
rob/fix-9425-9426

Conversation

@wobsoriano

Copy link
Copy Markdown
Member

Description

Fixes#9425 and #9426.

Corrects the @clerk/astro/types export so it exposes the documented Clerk types.

Adds the documented @clerk/chrome-extension/legacy entry point for the legacy useSignIn() and useSignUp() hooks, including ESM, CommonJS, and TypeScript declarations.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07a9c3e

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

This PR includes changesets to release 2 packages
NameType
@clerk/chrome-extensionPatch
@clerk/astroPatch

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

vercelBot commented Aug 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
clerk-js-sandboxReadyReadyPreviewAug 13, 2026 4:17am
swingsetReadyReadyPreviewAug 13, 2026 4:17am

Request Review

@wobsorianowobsoriano changed the title fix(astro,chrome-extension): Add missing Astro and Chrome Extension e…fix(astro,chrome-extension): Add missing Astro and Chrome Extension exportsAug 13, 2026
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9434

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9434

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9434

@clerk/expo

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9434

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 07a9c3e

@github-actions

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-08-13T04:18:28.303Z

Summary

MetricCount
Packages analyzed19
Packages with changes2
🔴 Breaking changes1
🟡 Non-breaking changes0
🟢 Additions13

Warning
1 breaking change(s) detected - Major version bump required

🤖 This report was reviewed by claude-sonnet-4-6.

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

PackageSubpathChange
@clerk/astro./types_default

@clerk/astro

Current version: 4.0.11
Recommended bump: MAJOR → 5.0.0

Subpath ./types

🔴 Breaking Changes (1)

Changed: _default
- _default: (params?: (import("./types").AstroClerkIntegrationParams & {- enableEnvSchema?: boolean;- }) | undefined) => import("astro").AstroIntegration

Static analyzer: Removed function _default

🤖 AI review (confirmed) (95%): The _default export (the main integration factory function) has been removed entirely, so any consumer calling it will fail to compile.

Migration: Replace the removed _default import/call with the updated export name or alternative integration entry point provided in the new version of @clerk/astro.

🟢 Additions (12)

Click to expand 12 changes
Added: AstroClerkCreateInstanceParams
+ type AstroClerkCreateInstanceParams<TUi extends Ui = Ui> = AstroClerkIntegrationParams<TUi> & InternalClerkScriptProps & {+ publishableKey: string;+ };

Added type alias AstroClerkCreateInstanceParams

Added: AstroClerkIntegrationParams
+ type AstroClerkIntegrationParams<TUi extends Ui = Ui> = Without<ClerkOptions, 'isSatellite' | 'sdkMetadata' | 'standardBrowser' | 'selectInitialSession' | 'routerReplace' | 'routerDebug' | 'routerPush' | 'polling' | 'touchSession' | 'appearance'> & MultiDomainAndOrProxyPrimitives & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ prefetchUI?: boolean;+ };

Added type alias AstroClerkIntegrationParams

Added: AstroClerkUpdateOptions
+ type AstroClerkUpdateOptions<TUi extends Ui = Ui> = Pick<ClerkOptions, 'localization'> & {+ appearance?: ExtractAppearanceType<TUi, Appearance>;+ };

Added type alias AstroClerkUpdateOptions

Added: AstroMiddleware
+ export type AstroMiddleware = MiddlewareHandler;

Added type alias AstroMiddleware

Added: AstroMiddlewareContextParam
+ export type AstroMiddlewareContextParam = APIContext;

Added type alias AstroMiddlewareContextParam

Added: AstroMiddlewareNextParam
+ export type AstroMiddlewareNextParam = MiddlewareNext;

Added type alias AstroMiddlewareNextParam

Added: AstroMiddlewareReturn
+ export type AstroMiddlewareReturn = Response | Promise<Response>;

Added type alias AstroMiddlewareReturn

Added: AuthFn
+ export type AuthFn = GetAuthFnNoRequest<SessionAuthObjectWithRedirect>;

Added type alias AuthFn

Added: ButtonProps
+ export type ButtonProps = {+ asChild?: boolean;+ };

Added type alias ButtonProps

Added: InternalUIComponentId
+ export type InternalUIComponentId = 'sign-in' | 'sign-up' | 'create-organization' | 'organization-list' | 'organization-profile' | 'organization-switcher' | 'user-avatar' | 'user-button' | 'user-profile' | 'google-one-tap' | 'waitlist' | 'pricing-table' | 'api-keys';

Added type alias InternalUIComponentId

Added: ProtectProps
+ export type ProtectProps = ProtectParams;

Added type alias ProtectProps

Added: SessionAuthObjectWithRedirect
+ export type SessionAuthObjectWithRedirect = SessionAuthObject & {+ redirectToSignIn: RedirectFun<Response>;+ };

Added type alias SessionAuthObjectWithRedirect


@clerk/chrome-extension

Current version: 3.1.69
Recommended bump: MINOR → 3.2.0

Subpath ./legacy

🟢 Additions (1)

Added: ./legacy

New subpath export ./legacy


Report generated by Break Check

Last ran on 07a9c3e.

@coderabbitai

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The changes correct the @clerk/astro/types declaration path. They add a public @clerk/chrome-extension/legacy entry point that re-exports useSignIn and useSignUp. The legacy entry point is included in the build and published files, with CommonJS, ESM, and TypeScript declaration metadata. Tests verify its exported names. Changesets document patch releases for both packages.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score:⚪ Minimal · up to 07a9c

The PR adds documented Astro and Chrome Extension exports. No actionable merge-blocking risk remains; the published subpath and generated declaration artifacts should receive routine follow-up coverage.

Possibly related issues

Suggested reviewers:dstaley

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly identifies the Astro and Chrome Extension export fixes.
Description check✅ PassedThe description directly explains both export fixes and identifies the related issues.
Linked Issues check✅ PassedThe changes correct the @clerk/astro/types declaration path required by issue [#9425].
Out of Scope Changes check✅ PassedAll changes support the documented Astro and Chrome Extension export fixes, including tests and release changesets.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitaicoderabbitaiBot 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.

🧹 Nitpick comments (1)
packages/chrome-extension/src/__tests__/exports.test.ts (1)

12-16: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Cover the published subpath and emitted artifacts.

This test imports ../legacy, so it validates only the source re-export. It does not exercise @clerk/chrome-extension/legacy, the import and require conditions, or dist/types/legacy.d.ts. Add a package-level smoke and type-resolution test after the build.

As per coding guidelines, test files must include comprehensive unit, integration, and E2E coverage, and unit tests are required for all new functionality.

🤖 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/chrome-extension/src/__tests__/exports.test.ts` around lines 12 -
16, Extend the legacy exports coverage beyond the source import by adding a
post-build package-level smoke test for `@clerk/chrome-extension/legacy` under
both import and require conditions, and verify that dist/types/legacy.d.ts
resolves the published type surface. Keep the existing legacyExports key
assertion, and run these checks against emitted artifacts rather than ../legacy.

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.
Nitpick comments:
In `@packages/chrome-extension/src/__tests__/exports.test.ts`:
- Around line 12-16: Extend the legacy exports coverage beyond the source import
by adding a post-build package-level smoke test for
`@clerk/chrome-extension/legacy` under both import and require conditions, and
verify that dist/types/legacy.d.ts resolves the published type surface. Keep the
existing legacyExports key assertion, and run these checks against emitted
artifacts rather than ../legacy.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 491a1ff1-efc2-417d-a38e-b64d12b5a21c

📥 Commits

Reviewing files that changed from the base of the PR and between 1447302 and 07a9c3e.

📒 Files selected for processing (8)
  • .changeset/add-chrome-extension-legacy.md
  • .changeset/fix-astro-types-export.md
  • packages/astro/package.json
  • packages/chrome-extension/legacy/package.json
  • packages/chrome-extension/package.json
  • packages/chrome-extension/src/__tests__/exports.test.ts
  • packages/chrome-extension/src/legacy.ts
  • packages/chrome-extension/tsup.config.ts
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go(manual)
  • clerk/dashboard(manual)
  • clerk/accounts(manual)
  • clerk/backoffice(manual)
  • clerk/clerk(manual)
  • clerk/clerk-docs(manual)
  • clerk/cloudflare-workers(manual)
  • clerk/clerk-ios(auto-detected)
  • clerk/cli(auto-detected)
  • clerk/clerk-android(auto-detected)

@jeremy-clerkjeremy-clerk 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.

👍

@wobsoriano
wobsoriano merged commit a088c94 into mainAug 13, 2026
54 checks passed
@wobsoriano
wobsoriano deleted the rob/fix-9425-9426 branch August 13, 2026 18:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@clerk/astro/types does not export documented Clerk types

2 participants

@wobsoriano@jeremy-clerk