Skip to content

Generate Claude-compatible skills#563

Open
michaelbaudino wants to merge 2 commits into
tirth8205:mainfrom
michaelbaudino:patch-1
Open

Generate Claude-compatible skills#563
michaelbaudino wants to merge 2 commits into
tirth8205:mainfrom
michaelbaudino:patch-1

Conversation

@michaelbaudino

Copy link
Copy Markdown

Pull Request

What & why

We currently generate Claude skills as a skill.md (lowercase) file in a sub-directory of .claude/skills/.

Unfortunately, Claude Code expects skills as a SKILL.md file (uppercase!), thus it simply ignores the skills we generate 😢

How it was tested

uvx pytest tests/test_skills.py --tb=short -q
uv run ruff check code_review_graph/
uv run --with mypy mypy code_review_graph/ --ignore-missing-imports --no-strict-optional

Note

I had to add --with mypy compared to the project instructions, because mypy is not declared as a dependency.

Checklist

  • Tests added for new functionality
  • All tests pass: uv run pytest tests/ --tb=short -q
  • Linting passes: uv run ruff check code_review_graph/
  • Type checking passes: uv run mypy code_review_graph/ --ignore-missing-imports --no-strict-optional
  • Lines are at most 100 characters
  • Docs updated where behavior changed (README, docs/, docstrings)

Claude Code expect skill files in a sub-directory name `SKILL.md` uppercase as [stated in the documentation](https://code.claude.com/docs/en/skills). We currently generate them as `skill.md` lowercase which makes Claude Code to ignore them.
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