Skip to content

chore: fix claude skills format#297

Merged
ioncache merged 5 commits into
mainfrom
chore/fix-claude-skills-format
May 21, 2026
Merged

chore: fix claude skills format#297
ioncache merged 5 commits into
mainfrom
chore/fix-claude-skills-format

Conversation

@ioncache
Copy link
Copy Markdown
Owner

@ioncache ioncache commented May 21, 2026

Overview

Converts the flat Claude Code skill stubs in .claude/skills/ to the subdirectory format (<name>/SKILL.md) required by Claude Code, and updates the .ai/ skill-creation instructions to document the correct format going forward.

Details

  • Replaced five single-line @<path> stub files with proper <name>/SKILL.md files, each including YAML frontmatter (name, description, disable-model-invocation) and $ARGUMENTS passthrough.
  • Updated .ai/skills/ai-add-skill.md and .ai/skills/ai-add-prompt.md to instruct future skill creation to use the subdirectory format rather than the old flat-file format.
  • No production code, tests, or public API changed.

Summary by CodeRabbit

  • Documentation

    • Updated skill and prompt docs to reflect a per-skill directory layout and revised configuration metadata and forwarding behavior.
  • Refactor

    • Reorganized AI skills into per-skill subdirectories with dedicated configuration files and explicit argument/forwarding sections.
    • Updated skill definitions for prompts, instructions, skill creation, planning, and unresolved-PR comment verification to follow the new layout.

Review Change Stack

ioncache and others added 4 commits May 21, 2026 00:45
Claude Code reads custom slash commands from .claude/commands/, not
.claude/skills/. Update maintenance skill docs and README to match.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…rrect paths

Migrates flat `.claude/skills/<name>.md` files to the required
`.claude/skills/<name>/SKILL.md` subdirectory format for Claude Code.

Adds `$ARGUMENTS` passthrough and fixes `@` import paths to use
file-relative paths (`../../../.ai/...`) instead of project-root paths.

Updates the `ai-add-prompt` and `ai-add-skill` maintenance skills to
generate SKILL.md files in the correct format.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 49bb3bde-4b07-4a9b-8cf1-01411245157a

📥 Commits

Reviewing files that changed from the base of the PR and between 1f32953 and 28ff606.

📒 Files selected for processing (1)
  • .claude/skills/verify-unresolved-pr-comments/SKILL.md

📝 Walkthrough

Walkthrough

The PR reorganizes Claude skill wrappers from single-file structures under .claude/skills/ to per-skill subdirectories with SKILL.md manifests. Documentation for shared skill generators is updated to reflect this pattern, and multiple Claude wrapper SKILL.md files are added to delegate to shared .ai implementations via relative $ARGUMENTS includes.

Changes

Claude skill directory restructuring and wrapper pattern

Layer / File(s) Summary
Shared skill documentation updates
.ai/skills/ai-add-prompt.md, .ai/skills/ai-add-skill.md
Documentation for ai-add-prompt and ai-add-skill generators updated to target .claude/skills/<name>/SKILL.md, describing disable-model-invocation: true, an $ARGUMENTS block, and relative includes to @../../../.ai/skills/<name>.md or @../../../.ai/prompts/<name>.md.
Claude skill wrapper implementations
.claude/skills/ai-add-instruction/SKILL.md, .claude/skills/ai-add-prompt/SKILL.md, .claude/skills/ai-add-skill/SKILL.md, .claude/skills/plan-writing/SKILL.md, .claude/skills/verify-unresolved-pr-comments/SKILL.md
New SKILL.md manifests added for several Claude skills; each manifest contains name, description, disable-model-invocation: true, and an $ARGUMENTS directive that forwards to the corresponding shared .ai skill or prompt via a relative path.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • ioncache/data-sanitization#296: Related updates to the AI maintenance-skill layer and shared .ai definitions that intersect with the per-skill SKILL.md wrapper layout and relative include patterns.

Poem

🐰 I hopped through .claude/ neat and spry,

Each SKILL.md found its cozy sty,
Arguments forwarded, paths aligned,
Shared .ai wisdom now well-defined,
A tiny rabbit cheers the tidy sky.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: converting Claude skills from flat files to a subdirectory SKILL.md format.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/fix-claude-skills-format

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 100% (🎯 100%) 127 / 127
🔵 Statements 100% (🎯 100%) 129 / 129
🔵 Functions 100% (🎯 100%) 14 / 14
🔵 Branches 100% (🎯 100%) 72 / 72
File CoverageNo changed files found.
Generated in workflow #182 for commit 28ff606 by the Vitest Coverage Report Action

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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.claude/skills/verify-unresolved-pr-comments/SKILL.md:
- Around line 1-4: The YAML frontmatter in the skill file is missing the
disable-model-invocation: true field; open
.claude/skills/verify-unresolved-pr-comments/SKILL.md and add a top-level line
disable-model-invocation: true in the YAML block alongside name:
verify-unresolved-pr-comments and description so the frontmatter matches other
skill wrappers and the templates (.ai/skills/ai-add-prompt.md and
.ai/skills/ai-add-skill.md).
🪄 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: ASSERTIVE

Plan: Pro Plus

Run ID: 3237112c-7528-46e7-9b32-8ddb4f6cccd9

📥 Commits

Reviewing files that changed from the base of the PR and between 791d29e and 1f32953.

📒 Files selected for processing (12)
  • .ai/skills/ai-add-prompt.md
  • .ai/skills/ai-add-skill.md
  • .claude/skills/ai-add-instruction.md
  • .claude/skills/ai-add-instruction/SKILL.md
  • .claude/skills/ai-add-prompt.md
  • .claude/skills/ai-add-prompt/SKILL.md
  • .claude/skills/ai-add-skill.md
  • .claude/skills/ai-add-skill/SKILL.md
  • .claude/skills/plan-writing.md
  • .claude/skills/plan-writing/SKILL.md
  • .claude/skills/verify-unresolved-pr-comments.md
  • .claude/skills/verify-unresolved-pr-comments/SKILL.md
💤 Files with no reviewable changes (5)
  • .claude/skills/plan-writing.md
  • .claude/skills/ai-add-skill.md
  • .claude/skills/ai-add-instruction.md
  • .claude/skills/verify-unresolved-pr-comments.md
  • .claude/skills/ai-add-prompt.md

Comment thread .claude/skills/verify-unresolved-pr-comments/SKILL.md
@ioncache ioncache merged commit ef5604a into main May 21, 2026
5 checks passed
@ioncache ioncache deleted the chore/fix-claude-skills-format branch May 21, 2026 11:25
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.

1 participant