Skip to content

chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling#8617

Open
jacekradko wants to merge 1 commit into
mainfrom
jacek/codeowners-hotload-boundary
Open

chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling#8617
jacekradko wants to merge 1 commit into
mainfrom
jacek/codeowners-hotload-boundary

Conversation

@jacekradko
Copy link
Copy Markdown
Member

@jacekradko jacekradko commented May 21, 2026

Adds a .github/CODEOWNERS (the repo has none today) covering the surfaces where an unreviewed change is most expensive, all owned by @clerk/team-sdk:

  • the hotload-boundary runtime files that ship to consumer apps without a version bump (clerk-js/src/core/clerk.ts, react/src/isomorphicClerk.ts, ui/src/contexts/), the surface AGENTS.md calls a contract;
  • the things that decide how every PR is built, gated, and published: .github/workflows/, the CODEOWNERS file itself, and the changeset config/changelog.

Per-PR changeset entries (.changeset/*.md) are deliberately left unowned so they don't force a review on every PR.

This only routes review requests until require_code_owner_reviews is enabled on main; that branch-protection flip is a separate follow-up after this lands. Empty changeset since it's repo-level config.

Summary by CodeRabbit

  • Chores
    • Updated code ownership configuration to clarify ownership for runtime surfaces, workflow/CI, and release-tooling areas.
    • Added a small changeset metadata file noting a hotload boundary for release tracking.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 21, 2026

🦋 Changeset detected

Latest commit: 7ebbd52

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

This PR includes changesets to release 0 packages

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

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

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

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

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 5, 2026 4:06pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

Adds CODEOWNERS entries assigning @clerk/team-sdk ownership for runtime-surface files and for CI/workflow and changeset/release tooling; adds a Changeset file documenting the hotload boundary.

Changes

Hotloaded boundary ownership

Layer / File(s) Summary
Hotloaded boundary ownership documentation
.github/CODEOWNERS, .changeset/codeowners-hotload-boundary.md
Adds CODEOWNERS entries assigning @clerk/team-sdk ownership for runtime-surface files (/packages/clerk-js/src/core/clerk.ts, /packages/react/src/isomorphicClerk.ts, /packages/ui/src/contexts/) and assigns @clerk/team-sdk for CI/workflows and changeset/release tooling (.github/workflows/, .github/CODEOWNERS, .changeset/config.json, .changeset/changelog.js); includes a new Changeset file.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • wobsoriano
  • alexcarpenter

"A rabbit peeks at ownership lines,
nudging tags where the boundary shines.
Small hops in code, tidy and spry—
guardians set beneath the sky. 🐇"

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling' accurately summarizes the main changes in the pull request, which adds a CODEOWNERS file to gate the runtime ABI surface and configure ownership for CI and release tooling.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

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

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 21, 2026

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 7ebbd52

Comment thread .github/CODEOWNERS
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.

TIL I'm a member of "Team User SDK" https://github.com/orgs/clerk/teams/team-user-sdk

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

TIL this team exists 😲

@clerk/team-sdk owns the hotload-boundary runtime surface that ships to consumer
apps without version bumps (clerk-js core clerk.ts, react isomorphicClerk, ui
contexts), plus the CI workflows, the CODEOWNERS policy itself, and the changeset
config/changelog. Per-PR changeset entries are left unowned so they don't gate
every PR. Empty changeset (repo-level config).
@jacekradko jacekradko force-pushed the jacek/codeowners-hotload-boundary branch from 8aa6361 to 7ebbd52 Compare June 5, 2026 16:05
@jacekradko jacekradko changed the title chore(repo): Add CODEOWNERS for hotload boundary files chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants