feat(init): install all core and feature skills by default by rafa-thayto · Pull Request #185 · clerk/cli

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto · 2026-04-20T16:08:50Z

Summary

Expands DEFAULT_SKILLS from 2 (clerk, clerk-setup) to 7 by also installing clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks on every clerk init run. Framework-specific skills still layer on top, one per detected dep.
Drops express and fastify from FRAMEWORK_SKILL_MAP — they no longer need a framework-specific skill because clerk-backend-api is now a default for all projects.
Summarizes the confirm prompt so 8 slugs don't wrap a narrow terminal: Install agent skills? (clerk core + features + nextjs-patterns). Pass --no-skills to opt out, same as before.

Test plan

bun run format / bun run lint / bun run typecheck — pass
bun test packages/cli-core/src/commands/init/skills.test.ts — 10/10 pass (5 new resolveSkills cases, 2 new formatSkillsPromptMessage cases, 3 existing buildSkillsArgs cases)
bun changeset status --since=origin/main — exit 0 (patch bump for clerk)
Manual smoke: run clerk init in a throwaway Next.js app and confirm the prompt reads Install agent skills? (clerk core + features + nextjs-patterns) and the post-accept log line lists all 8 skills
Manual smoke: run clerk init in an Express project and confirm only the 7 defaults install (no duplicate clerk-backend-api)
Manual smoke: run clerk init --no-skills and confirm the install step is skipped entirely

coderabbitai · 2026-04-20T16:08:58Z

Warning

Rate limit exceeded

@rafa-thayto has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 50 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc175dca-4853-4155-9ff2-f9634edf7971

📥 Commits

Reviewing files that changed from the base of the PR and between 198903d and 21f9b57.

📒 Files selected for processing (4)

.changeset/required-skills.md
packages/cli-core/src/commands/init/README.md
packages/cli-core/src/commands/init/skills.test.ts
packages/cli-core/src/commands/init/skills.ts

📝 Walkthrough

Walkthrough

This PR changes the default agent skills that clerk init stages: the default upstream set is now a core+features collection (including clerk-setup, clerk-custom-ui, clerk-backend-api, clerk-orgs, clerk-testing, and clerk-webhooks). A single framework-specific skill is still added when applicable; express and fastify no longer map to distinct framework skills and rely on clerk-backend-api instead. The skills resolution logic was refactored (deduplication via Set), new helpers (resolveUpstreamSkills, getFrameworkSkill, formatSkillsPromptMessage) were exported, prompt/log formatting was updated, documentation was updated, and tests were added.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately describes the main change: expanding the default skill set to include all core and feature skills during initialization.
Description check	✅ Passed	The PR description accurately describes the changeset: expanding default skills from 2 to 7, removing express/fastify from the framework map, and updating the prompt summary.

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

changeset-bot · 2026-04-20T16:09:00Z

🦋 Changeset detected

Latest commit: 21f9b57

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

This PR includes changesets to release 1 package

Name	Type
clerk	Patch

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

wyattjoh

Code review

Small, focused change. Expands the default skill set, drops the now-redundant Express/Fastify entries, and shortens the confirm prompt. Tests are tight. Two small consistency suggestions below.

Expand the default skill set from 2 to 7 so agents get context for orgs, webhooks, testing, custom UI, and the backend API in addition to clerk/clerk-setup. Framework-specific skills still layer on top, one per detected dep. Express and Fastify no longer need a framework-specific skill because clerk-backend-api is now a default. The confirm prompt is summarized so 8 skill slugs don't wrap the terminal: "Install agent skills? (clerk core + features + nextjs-patterns)". Pass --no-skills to opt out as before.

rafa-thayto requested a review from wyattjoh April 20, 2026 16:36

wyattjoh reviewed Apr 20, 2026
View reviewed changes

Comment threadpackages/cli-core/src/commands/init/skills.ts Outdated
Comment threadpackages/cli-core/src/commands/init/skills.ts

rafa-thayto requested a review from wyattjoh April 20, 2026 18:30

rafa-thayto force-pushed the rafa-thayto/required-skills branch from 52ff381 to 198903dCompare April 20, 2026 20:50

rafa-thayto requested a review from Railly April 20, 2026 21:40

rafa-thayto force-pushed the rafa-thayto/required-skills branch from 198903d to fc62a8eCompare April 20, 2026 21:42

rafa-thayto force-pushed the rafa-thayto/required-skills branch from fc62a8e to 21f9b57Compare April 21, 2026 14:35

wyattjoh approved these changes Apr 21, 2026
View reviewed changes

rafa-thayto merged commit 5a8bbd2 into mainApr 21, 2026
10 checks passed

rafa-thayto deleted the rafa-thayto/required-skills branch April 21, 2026 16:47

github-actionsBot mentioned this pull request Apr 21, 2026
ci(repo): Version Packages #181
Merged