docs(repo): document changeset authoring in clerk-monorepo skill - #9212
Conversation
🦋 Changeset detectedLatest commit: 6744852 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen 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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-google-signin
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
📝 WalkthroughWalkthroughUpdates 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 Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/skills/clerk-monorepo/SKILL.md (1)
111-114: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick winKeep the
privatefield 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 checkingpackage.jsonfor 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
📒 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)
Audited 49 sessions in this repo from the last 7 days. Changeset authoring was the biggest source of repeated wasted work.
pnpm changesetis 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.