Skip to content

Conversation

@stackloke2e
Copy link

Summary

This PR updates the ToolHive API models generated from the latest OpenAPI specification.

Changes

  • Updated Pydantic models in src/mcp_optimizer/toolhive/api_models/

Notes

  • This PR was automatically generated by the update-thv-models workflow
  • The models are generated using datamodel-codegen from ToolHive's OpenAPI endpoint

🤖 Generated with GitHub Actions

Automated update of ToolHive API models from OpenAPI specification.
@claude
Copy link

claude bot commented Nov 24, 2025

PR Review: Update ToolHive API models

✅ Code Quality: Excellent

This auto-generated PR cleanly updates Pydantic models from ToolHive's OpenAPI spec.

Key Changes

  • Refactoring: RemoteAuthConfig extracted to remote.py as remote.Config (non-breaking)
  • New module: tokenexchange.py for OAuth 2.0 token exchange (RFC 8749)
  • Enhanced OAuth: Added resource field support (RFC 8707)
  • Registry update: New api_url field in UpdateRegistryRequest

Analysis

✅ No breaking changes - backward compatible
✅ No codebase usage of affected models
✅ Properly excluded from linting (pyproject.toml:68)
✅ Consistent timestamp updates across all files
✅ All changes are additive

Security & Performance

  • No security concerns (declarative Pydantic models)
  • No performance impact (structure-only changes)

Verdict

Approve - Clean, well-structured auto-generated update with no issues.


Review completed by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants