Skip to content

Conversation

@GuanyiLi-Craig
Copy link
Contributor

  1. improve function docs and test
  2. add skills to the category
  3. expose category in openapi
  4. add output schema for getting tools.

Copilot AI review requested due to automatic review settings January 2, 2026 14:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the HuMCP server with improved documentation, structured skill metadata, OpenAPI schema integration, and comprehensive testing coverage.

Key Changes:

  • Added skill discovery system with SKILL.md files containing frontmatter metadata for each tool category
  • Implemented Pydantic response schemas for all API endpoints with typed responses
  • Integrated skill metadata and OpenAPI tags into category-based documentation
  • Added extensive test coverage with pytest-cov and new test suites for skills, server, routes, and decorator modules
  • Enhanced security in filesystem and data processing tools with allowlists and validation

Reviewed changes

Copilot reviewed 44 out of 45 changed files in this pull request and generated no comments.

Show a summary per file
File Description
uv.lock Added coverage 7.13.1 and pytest-cov dependencies; updated grafi version
tests/humcp/* New comprehensive test suites for skills, server, routes, and decorator
tests/conftest.py Shared fixtures for test isolation and sample tools
tests/tools/* Formatting improvements for better readability
src/tools/*/SKILL.md New skill documentation with YAML frontmatter for each category
src/tools/local/local_file_system.py Added path validation security with HUMCP_ALLOW_ABSOLUTE_PATHS env var
src/tools/data/pandas.py Added allowlists for safe pandas operations
src/tools/data/csv.py Added SQL query validation to prevent injection
src/tools/google/* Migrated from FastMCP to @tool decorator, improved docstrings
src/humcp/skills.py New module for discovering and parsing SKILL.md files
src/humcp/schemas.py New Pydantic models for typed API responses
src/humcp/routes.py Updated to use response models and integrate skill metadata
src/humcp/server.py Added OpenAPI tags generation from discovered categories
pyproject.toml Added pytest-cov, coverage configuration, updated grafi
.pre-commit-config.yaml Added pytest pre-push hook, Python 3.13 version, additional checks
.github/workflows/ci.yaml Restructured CI with separate lint, type-check, test jobs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GuanyiLi-Craig GuanyiLi-Craig merged commit 92b21b8 into main Jan 2, 2026
10 checks passed
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.

2 participants