feat: add Claude Code skills and expand agent documentation#453
Draft
Jazzcort wants to merge 1 commit into
Draft
feat: add Claude Code skills and expand agent documentation#453Jazzcort wants to merge 1 commit into
Jazzcort wants to merge 1 commit into
Conversation
|
For team members: test commit |
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Add three Claude Code skills (generate-commit-message, prepare-for-pull-request, review-pr) for workflow automation. Create CLAUDE.md with core rules. Expand AGENTS.md with architecture, project layout, tools overview, configuration reference, and tech stack details.
|
For team members: test commit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
The project lacked structured guidance for Claude Code agents — there was no
CLAUDE.md, theAGENTS.mdwas a minimal stub, and there were no reusable skills for common contributorworkflows (committing, creating PRs, reviewing code). Contributors and agents had to rediscover project conventions each time.
Summary
generate-commit-message,prepare-for-pull-request,review-pr) to standardize contributor workflowsCLAUDE.mdwith core operating rules and a pointer toAGENTS.mdAGENTS.mdfrom a brief stub into a comprehensive agent reference covering architecture, project layout, tools, configuration, tech stack, and securityChanges
Docs:
CLAUDE.md— four core rules for agent behavior plus project details referenceAGENTS.md— added architecture diagram, detailed project layout, tools overview table (fixed + guarded), configuration reference for allLINUX_MCP_*env vars, tech stacksection, and expanded security guidance (SSH key-only auth, host key verification, audit log redaction)
docs/security.md) to the Docs sectionClaude Code Skills:
.claude/skills/generate-commit-message/SKILL.md— analyzes staged changes and generates conventional commit messages using project-specific type/scope conventions.claude/skills/prepare-for-pull-request/SKILL.md— runsmake verify, gathers changes since a base ref, and generates a structured PR description.claude/skills/review-pr/SKILL.md— six-dimension PR review (format, types, semantics, readability, redundancy, bugs/security) with verdict criteria🤖 Generated with Claude Code
Just some experiments with skills. Feel free to try it out! we can merge this if it really helps.
Usage:
/generate-commit-message:Stage your changes, then invoke the skill
git add src/linux_mcp_server/tools/network.py tests/tools/test_network.py
Invoke /generate-commit-message in claude code
It inspects git diff --cached, checks recent commit style, and produces a conventional commit message like:
feat(network): add protocol filter to get_listening_ports
/prepare-for-pull-request:Invoke it with a base ref (or let it auto-detect from reflog):
/prepare-for-pull-request main
It runs make verify first (lint, types, tests). If everything passes, it analyzes all commits since main, then generates a structured PR description with Motivation, Summary, Changes,
and Test plan sections. You can then create the PR directly or edit first.
Without arguments it auto-detects the branch point:
/prepare-for-pull-request
/review-pr:Pass a GitHub PR URL:
/review-pr https://github.com/yourusername/linux-mcp-server/pull/42
It fetches the PR, checks out the branch, and runs a 6-dimension review: format/lint, type check, semantic correctness, readability/redundancy, bugs/security, and tests. The output is a
structured report ending with a verdict (APPROVE, REQUEST CHANGES, or NEEDS DISCUSSION) and actionable findings with severity levels.