Skip to content

docs(repo): document changeset authoring in clerk-monorepo skill - #9212

Merged
alexcarpenter merged 1 commit into
mainfrom
carp/clerk-monorepo-skill
Jul 22, 2026
Merged

docs(repo): document changeset authoring in clerk-monorepo skill#9212
alexcarpenter merged 1 commit into
mainfrom
carp/clerk-monorepo-skill

Conversation

@alexcarpenter

@alexcarpenter alexcarpenter commented Jul 22, 2026

Copy link
Copy Markdown
Member

Audited 49 sessions in this repo from the last 7 days. Changeset authoring was the biggest source of repeated wasted work. pnpm changeset is an interactive prompt that agents can't accomplish. This PR adds a new reference doc in monorepo skill to outline how to author the changeset.

@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6744852

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

vercel Bot commented Jul 22, 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 Jul 22, 2026 1:43pm
swingset Ready Ready Preview, Comment Jul 22, 2026 1:43pm

Request Review

@pkg-pr-new

pkg-pr-new Bot commented Jul 22, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

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

@clerk/electron-passkeys

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

@clerk/eslint-plugin

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

@clerk/expo

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

@clerk/expo-google-signin

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 6744852

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updates monorepo skill guidance to have contributors write Changeset files directly, stage them, and verify them with the CI status command. Adds a detailed Changesets reference covering file formats, package inclusion, version bump rules, changelog content, validation, and amendments. Adds initial Changeset authoring guidance and repositions the @clerk/eslint-plugin entry in the package map.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title accurately summarizes the main change: adding changeset authoring documentation to the clerk-monorepo skill.
Description check ✅ Passed The description is directly related to the changeset authoring guidance added in this PR.

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

@coderabbitai coderabbitai Bot 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)
.claude/skills/clerk-monorepo/SKILL.md (1)

111-114: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick win

Keep the private field as a source-of-truth check.

The repository publishing contract excludes any package with private: true, but these instructions say every package outside the current three-item list is published. If another private package is added without updating this list, agents may author invalid changesets. State that the list is current and instruct checking package.json for packages not covered by it.

🤖 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 @.claude/skills/clerk-monorepo/SKILL.md around lines 111 - 114, Update the
package-publishing guidance in .claude/skills/clerk-monorepo/SKILL.md at lines
111-114 to state that the three-package private list is current but
package.json’s private field is the source of truth; instruct agents to check
package.json for any additional private packages before authoring changesets.
Apply the same clarification to
.claude/skills/clerk-monorepo/references/changesets.md at lines 70-79,
preserving the existing publishing and changeset rules.
🤖 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 @.claude/skills/clerk-monorepo/SKILL.md:
- Around line 111-114: Update the package-publishing guidance in
.claude/skills/clerk-monorepo/SKILL.md at lines 111-114 to state that the
three-package private list is current but package.json’s private field is the
source of truth; instruct agents to check package.json for any additional
private packages before authoring changesets. Apply the same clarification to
.claude/skills/clerk-monorepo/references/changesets.md at lines 70-79,
preserving the existing publishing and changeset rules.

ℹ️ Review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: 6076ba7c-0e9f-45e2-9ffe-a30a8369a94a

📥 Commits

Reviewing files that changed from the base of the PR and between 018d950 and 6744852.

📒 Files selected for processing (4)
  • .changeset/changeset-authoring-guidance.md
  • .claude/skills/clerk-monorepo/SKILL.md
  • .claude/skills/clerk-monorepo/references/changesets.md
  • .claude/skills/clerk-monorepo/references/package-map.md
🔗 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)

@alexcarpenter
alexcarpenter merged commit 706bae2 into main Jul 22, 2026
51 checks passed
@alexcarpenter
alexcarpenter deleted the carp/clerk-monorepo-skill branch July 22, 2026 14:19
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