Skip to content

Conversation

@bfollington
Copy link
Contributor

@bfollington bfollington commented Oct 23, 2025

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

    • Renamed .claude/skills/recipe-dev → .claude/skills/pattern-dev and updated terminology (recipe → pattern, patch where relevant).
    • Added pattern-dev/references/workflow-guide.md and removed the old recipe-dev workflow guide.
    • Removed ct/references/commands.md; moved command usage to ct SKILL.md with a “self-documenting CLI” approach.
    • Streamlined ct/SKILL.md: clearer workflows, fewer hardcoded examples, and corrected link syntax to “[source]/[field] [target]/[input-field]”.
    • Reduced overlap between skills and docs; skills now point to docs/common/* for concepts and to the ct skill for commands.
  • Migration

    • Update internal links and references from recipe-dev to pattern-dev.
    • Replace “recipe” with “pattern” in related docs/examples where applicable.
    • For CT command details, use ct --help and the ct skill; the removed ct/references/commands.md is no longer available.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a 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 &#39;recipe&#39;, 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.

Copy link
Contributor

@ellyxir ellyxir left a 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?

@bfollington
Copy link
Contributor Author

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
@bfollington bfollington force-pushed the fix/2025-10-23-recipe-dev-to-pattern-de branch from e50cefd to 50a0b7b Compare October 24, 2025 00:08
@bfollington bfollington merged commit 8848aa6 into main Oct 24, 2025
8 checks passed
@bfollington bfollington deleted the fix/2025-10-23-recipe-dev-to-pattern-de branch October 24, 2025 22:14
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.

4 participants