Skip to content

feat(skills): rename bundled clerk skill to clerk-cli#245

Open
kylemac wants to merge 1 commit intomainfrom
kylemac/rename-skill-clerk-cli
Open

feat(skills): rename bundled clerk skill to clerk-cli#245
kylemac wants to merge 1 commit intomainfrom
kylemac/rename-skill-clerk-cli

Conversation

@kylemac
Copy link
Copy Markdown
Contributor

@kylemac kylemac commented Apr 29, 2026

Summary

  • Renames the bundled agent skill from clerk to clerk-cli so its purpose is clear at install time, when it sits alongside other Clerk skills (clerk-setup, clerk-orgs, etc.).
  • clerk skill install and the install step in clerk init now write to <agent-dir>/skills/clerk-cli/. Updates the help-tip detection paths (incl. .github/prompts/clerk-cli.md) so the install nudge still disappears post-install.
  • The CLI binary, npm wrapper package, and clerk skill install command name are unchanged. Includes a minor changeset noting that prior skills/clerk/ installs are left in place and can be removed manually.

Test plan

  • bun run typecheck
  • bun run lint
  • bun run format
  • bun test packages/cli-core/src/commands/skill packages/cli-core/src/commands/init/skills.test.ts packages/cli-core/src/cli-program.test.ts (56/56 pass)
  • Verify clerk skill install writes to skills/clerk-cli/ in a real agent dir

Renames the bundled agent skill artifact so its purpose is clear when
installed alongside other skills. `clerk skill install` and the install
step in `clerk init` now write to `<agent-dir>/skills/clerk-cli/` instead
of `<agent-dir>/skills/clerk/`. The CLI binary, npm package, and command
names are unchanged.
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 29, 2026

🦋 Changeset detected

Latest commit: c5839a1

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

This PR includes changesets to release 1 package
Name Type
clerk Minor

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
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

📝 Walkthrough

Walkthrough

The change introduces a rename of the bundled agent skill from clerk to clerk-cli. All bundled markdown imports are updated to reference skills/clerk-cli/ instead of skills/clerk/, skill detection logic is modified to check for the renamed skill location, test cases are updated to verify the new path layout, and documentation is revised to reflect the change. The rename is applied consistently across implementation files, test cases, and documentation, including a changeset entry documenting this minor version update.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: renaming the bundled clerk skill to clerk-cli for clarity.
Description check ✅ Passed The description is directly related to the changeset, explaining the rationale, scope, and testing for the skill rename.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

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


Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.claude/skills/audit-clerk-skill/SKILL.md:
- Line 121: The template heading "clerk-cli skill audit — <YYYY-MM-DD>" uses an
em dash which violates the "no em-dashes" guardrail; replace the em dash (—)
with a plain hyphen (-) so the heading reads "clerk-cli skill audit -
<YYYY-MM-DD>" to keep the template self-consistent and conform to the rule.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 50032856-f9fd-42f7-a8fb-07eefad82ee1

📥 Commits

Reviewing files that changed from the base of the PR and between 17606be and c5839a1.

📒 Files selected for processing (12)
  • .changeset/rename-clerk-cli-skill.md
  • .claude/skills/audit-clerk-skill/SKILL.md
  • packages/cli-core/src/cli-program.test.ts
  • packages/cli-core/src/commands/init/README.md
  • packages/cli-core/src/commands/skill/README.md
  • packages/cli-core/src/commands/skill/install.test.ts
  • packages/cli-core/src/commands/skill/install.ts
  • packages/cli-core/src/lib/skill-detection.ts
  • skills/clerk-cli/SKILL.md
  • skills/clerk-cli/references/agent-mode.md
  • skills/clerk-cli/references/auth.md
  • skills/clerk-cli/references/recipes.md

Comment thread .claude/skills/audit-clerk-skill/SKILL.md
"clerk": minor
---

Rename the bundled agent skill from `clerk` to `clerk-cli` so its purpose is unambiguous during install. After upgrading, `clerk skill install` (and the install step in `clerk init`) writes the skill to `<agent-dir>/skills/clerk-cli/` instead of `<agent-dir>/skills/clerk/`. Existing `skills/clerk/` directories from prior installs are left in place; remove them manually if you want to avoid duplicate context.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
Rename the bundled agent skill from `clerk` to `clerk-cli` so its purpose is unambiguous during install. After upgrading, `clerk skill install` (and the install step in `clerk init`) writes the skill to `<agent-dir>/skills/clerk-cli/` instead of `<agent-dir>/skills/clerk/`. Existing `skills/clerk/` directories from prior installs are left in place; remove them manually if you want to avoid duplicate context.
Rename the bundled agent skill from `clerk` to `clerk-cli` for more clarity during install. After upgrading, `clerk skill install` (and the install step in `clerk init`) writes the skill to `<agent-dir>/skills/clerk-cli/` instead of `<agent-dir>/skills/clerk/`. Existing `skills/clerk/` directories from prior installs are left in place; remove them manually if you want to avoid duplicate context.

@kylemac kylemac requested a review from wyattjoh April 29, 2026 15:20
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