Skip to content

Feat/openapi2mcp composition#290

Merged
harshadixit12 merged 6 commits intomainfrom
feat/openapi2mcp-composition
Mar 25, 2026
Merged

Feat/openapi2mcp composition#290
harshadixit12 merged 6 commits intomainfrom
feat/openapi2mcp-composition

Conversation

@harshadixit12
Copy link
Contributor

@harshadixit12 harshadixit12 commented Mar 24, 2026

Summary

This PR makes refactoring on top of #286
Refactoring done using gemini, and copilot.

Notable changes include:

  1. Use of openapi2kong.Convert inside openapi2mcp before the MCP tools are created - instead of duplication
  2. Introduction of openapitools package containing the common code used in openapi2kong, and openapi2mcp
  3. Test for kebab case has been moved into openapitools package - utils_test.go

Diff between the two branches in question - https://github.com/Kong/go-apiops/compare/openapi2mcp..feat/openapi2mcp-composition

@codecov-commenter
Copy link

codecov-commenter commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 58.20000% with 209 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.98%. Comparing base (7284f01) to head (e494f0b).

Files with missing lines Patch % Lines
openapi2mcp/openapi2mcp.go 69.20% 65 Missing and 44 partials ⚠️
openapitools/utils.go 17.94% 95 Missing and 1 partial ⚠️
openapi2kong/openapi2kong.go 85.71% 0 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #290      +/-   ##
==========================================
- Coverage   69.82%   64.98%   -4.85%     
==========================================
  Files          24       25       +1     
  Lines        2751     3119     +368     
==========================================
+ Hits         1921     2027     +106     
- Misses        653      881     +228     
- Partials      177      211      +34     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@harshadixit12 harshadixit12 marked this pull request as ready for review March 25, 2026 11:28
@harshadixit12 harshadixit12 requested a review from mheap March 25, 2026 11:29
@harshadixit12 harshadixit12 merged commit 96899b9 into main Mar 25, 2026
7 checks passed
@harshadixit12 harshadixit12 deleted the feat/openapi2mcp-composition branch March 25, 2026 14:41
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.

3 participants