Skip to content

feat: add a new command - deck file openapi2mcp#1954

Merged
harshadixit12 merged 1 commit into
mainfrom
feat/file-openapi2mcp
Mar 26, 2026
Merged

feat: add a new command - deck file openapi2mcp#1954
harshadixit12 merged 1 commit into
mainfrom
feat/file-openapi2mcp

Conversation

@harshadixit12
Copy link
Copy Markdown
Contributor

@harshadixit12 harshadixit12 commented Mar 25, 2026

This PR adds three changes -

  1. New command - deck file openapi2mcp
  2. openapi2kong - Fix header based routing to only consider required headers, and add a fallback (through go-apiops bump)
  3. openapikong - Fix duplicate plugin IDs generated for the routes when header based routing is used

Issues resolved:
#1856
#1838

@harshadixit12 harshadixit12 force-pushed the feat/file-openapi2mcp branch from 2c69e54 to 41f5bdc Compare March 25, 2026 16:34
Comment thread cmd/file_openapi2mcp.go
//

func newOpenapi2MCPCmd() *cobra.Command {
openapi2mcpCmd := &cobra.Command{
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 86 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.04%. Comparing base (d9d642e) to head (41f5bdc).

Files with missing lines Patch % Lines
cmd/file_openapi2mcp.go 0.00% 85 Missing ⚠️
cmd/root.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1954      +/-   ##
==========================================
- Coverage   33.51%   33.04%   -0.47%     
==========================================
  Files          76       77       +1     
  Lines        6726     6812      +86     
==========================================
- Hits         2254     2251       -3     
- Misses       4293     4381      +88     
- Partials      179      180       +1     

☔ 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 26, 2026 05:10
@harshadixit12 harshadixit12 merged commit 4a293ab into main Mar 26, 2026
40 checks passed
@harshadixit12 harshadixit12 deleted the feat/file-openapi2mcp branch March 26, 2026 06:55
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