Skip to content

ci(repo): Scope core-2 release dispatch to sdk-infra-workers - #8741

Merged
jacekradko merged 1 commit into
release/core-2from
jacek/core-2-release-dispatch-scope
Jun 3, 2026
Merged

ci(repo): Scope core-2 release dispatch to sdk-infra-workers#8741
jacekradko merged 1 commit into
release/core-2from
jacek/core-2-release-dispatch-scope

Conversation

@jacekradko

Copy link
Copy Markdown
Contributor

The Release (Core 2) job was failing on "Trigger workflows on related repos". release/core-2 is a maintenance branch, but it was still dispatching prepare-nextjs-sdk-update.yml to clerk/dashboard and typedoc-core-2.yml to clerk/clerk-docs, both of which have moved on to Core 3. Those dispatches now return 404 / 403, and since the calls were fire-and-forget promises with no error handling, the rejection bubbled up as an unhandled error and failed the whole job (which main doesn't hit because its version of this step was already hardened).

This keeps only the sdk-infra-workers clerk-js CDN version bump, awaits it instead of firing-and-forgetting, and marks the step continue-on-error: true so a downstream-notification hiccup can't fail a release that already published to npm.

Failing run: https://github.com/clerk/javascript/actions/runs/26906104288

The release/core-2 maintenance branch was dispatching workflows to clerk/dashboard and clerk/clerk-docs, which have moved on to Core 3. Those dispatches now 404/403 and, because the calls were fire-and-forget promises with no continue-on-error, the rejection failed the whole Release job.

Keep only the sdk-infra-workers clerk-js CDN version bump, await it, and mark the step continue-on-error so a post-publish notification failure can't fail a release that already shipped to npm.
@vercel

vercel Bot commented Jun 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Jun 3, 2026 7:07pm

Request Review

@changeset-bot

changeset-bot Bot commented Jun 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9882feb

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

This PR includes changesets to release 0 packages

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

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

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

@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8e108e19-3300-4e4e-b1d3-75f229360f65

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jacek/core-2-release-dispatch-scope

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

@pkg-pr-new

pkg-pr-new Bot commented Jun 3, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@8741

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@8741

@clerk/elements

npm i https://pkg.pr.new/@clerk/elements@8741

@clerk/clerk-expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/clerk-react

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

@clerk/react-router

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

@clerk/remix

npm i https://pkg.pr.new/@clerk/remix@8741

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/themes

npm i https://pkg.pr.new/@clerk/themes@8741

@clerk/types

npm i https://pkg.pr.new/@clerk/types@8741

@clerk/upgrade

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

@clerk/vue

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

commit: 9882feb

@wobsoriano wobsoriano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

niceee

@jacekradko
jacekradko merged commit 0351aad into release/core-2 Jun 3, 2026
43 of 45 checks passed
@jacekradko
jacekradko deleted the jacek/core-2-release-dispatch-scope branch June 3, 2026 20:39
Sign up for free to 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.

2 participants