Reviews vMCP Go code changes for known anti-patterns and classifies findings by severity.
npx skills add https://github.com/stacklok/toolhive --skill vmcp-review
Please help me install this Agent Skill. Skill: vmcp-review Source: https://github.com/stacklok/toolhive/tree/main/.claude/skills/vmcp-review Command: npx skills add https://github.com/stacklok/toolhive --skill vmcp-review
Reviewing vMCP code changes manually risks missing recurring anti-patterns that increase cognitive load, create brittle dependencies, and undermine testability across the codebase.
pkg/vmcp/ and cmd/vmcp/, or audits all Go files in a target package..claude/rules/vmcp-anti-patterns.md, applying judgment about which checks are relevant.Review the vMCP changes in this pull request for known anti-patterns and report findings by severity.
High-intent search queries and answers about installing and using this skill.
Run the vmcp-review skill on the changed files under pkg/vmcp/ and cmd/vmcp/. It checks the code against the anti-patterns in .claude/rules/vmcp-anti-patterns.md and reports findings grouped by severity.
It checks for known anti-patterns that increase cognitive load, create brittle dependencies, or undermine testability. Findings are classified as Must Fix, Should Fix, or Note depending on whether the change introduces or expands the pattern.
Yes. When auditing a package, the skill examines all Go files in the target package rather than only changed files. It still skips anything outside the vMCP codebase.
No. General Go style is handled by golangci-lint, security issues by the security-advisor agent, and test quality by the unit-test-writer agent. This skill focuses only on vMCP-specific anti-patterns.
It only applies to code under pkg/vmcp/ and cmd/vmcp/, and it does not cover performance issues unless they stem from an anti-pattern like repeated body parsing. Non-vMCP code should go to the general code-reviewer agent.
brave-experimentsImplement code review feedback on pull requests with automated validation.
dmwin72015Configure and run golangci-lint v2 with justified //nolint suppressions.
nubivAutomate build, static analysis, tests, and security scans for Spring Boot projects.
LuigiAPCPereiraAutomates Go code linting with golangci-lint, reporting style, correctness, and performance issues.