-
Notifications
You must be signed in to change notification settings - Fork 9
Rename recipe-dev skill to pattern-dev
#1946
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8 issues found across 11 files
Prompt for AI agents (all 8 issues)
Understand the root cause of the following 8 issues and fix them.
<file name=".claude/skills/pattern-dev/SKILL.md">
<violation number="1" location=".claude/skills/pattern-dev/SKILL.md:30">
Please fix the duplicated wording so the PATTERNS.md description reads clearly (e.g., “Common patterns with examples”).</violation>
<violation number="2" location=".claude/skills/pattern-dev/SKILL.md:247">
Update this new bullet to refer to sub-patterns instead of sub-recipes so the terminology stays consistent with the pattern-dev skill.</violation>
</file>
<file name=".claude/agents/research-specialist.md">
<violation number="1" location=".claude/agents/research-specialist.md:10">
Instructing the agent to run `ct --help`/`ct charm --help` fails because the `ct` binary is not available on PATH by default; point the guidance to `./dist/ct` (or the supported invocation) so the first step is executable.</violation>
</file>
<file name=".claude/commands/onboarding.md">
<violation number="1" location=".claude/commands/onboarding.md:56">
Running `ct --help` fails because no `ct` binary is available; use the existing `deno task ct --help` entry point so the onboarding command can actually list CLI commands.</violation>
<violation number="2" location=".claude/commands/onboarding.md:56">
This file was partially updated, but an outdated reference to the `/recipe-dev` command remains on line 38. This should be updated to `/pattern-dev` to align with the skill renaming.</violation>
</file>
<file name=".claude/agents/recipe-dev-guide.md">
<violation number="1" location=".claude/agents/recipe-dev-guide.md:10">
The agent definition file `recipe-dev-guide.md` should have been renamed to `pattern-dev-guide.md` to match the `pattern-dev` skill. Its name and content are inconsistent with the new terminology in `.claude/skills/pattern-dev/SKILL.md`, creating a system-wide inconsistency.</violation>
</file>
<file name="docs/common/RECIPE_DEV_DEPLOY.md">
<violation number="1" location="docs/common/RECIPE_DEV_DEPLOY.md:212">
This documentation file was not renamed to `PATTERN_DEV_DEPLOY.md` and contains a broken link to `.claude/skills/recipe-dev/references/workflow-guide.md`, a file deleted in this PR. It also still uses the outdated term 'recipe', creating cross-file inconsistencies.</violation>
</file>
<file name="README.md">
<violation number="1" location="README.md:48">
This file was partially updated but still contains dangling references to the old `/recipe-dev` command on lines 57 and 67. These should be updated to `/pattern-dev` to reflect the skill renaming.</violation>
</file>
React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks good, we'll need to modify AGENTS.md , it has a reference to "recipe-dev" at the moment in it, and it should be updated. do you want to add that to this PR?
yep, will fix + action cubic bot's comments above and re-request review |
And reduce duplication in skills vs docs
also, fix file references
e50cefd to
50a0b7b
Compare
And reduce duplication in skills vs docs
Summary by cubic
Renamed the recipe-dev skill to pattern-dev and consolidated CLI docs to reduce duplication. CT docs now emphasize built-in --help for command discovery, and a focused workflow guide lives under pattern-dev.
Refactors
Migration