Skip to content

MCP Server - #48

Open
majst01 wants to merge 9 commits into
mainfrom
mcp-server
Open

MCP Server#48
majst01 wants to merge 9 commits into
mainfrom
mcp-server

Conversation

@majst01

@majst01 majst01 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

This adds a MCP Server to the cli and replaces my work here: metal-stack/metal-apiserver#277

depends on:

TODO:

  • solve the interface differences between connect-go-simple and the interfaces generated by the protoc-gen-go-mcp. Support connect-simple redpanda-data/protoc-gen-go-mcp#51.
    • now "fixed" in the api by modifying the generated client adapters
  • implement remaining api methods
  • Write a small docu
  • figure out why this endpoint does not inherit connection credentials from existing config.yaml

Used AI-Tools ✨

  • none used for generation

@metal-robot metal-robot Bot added this to Development Aug 3, 2026
@majst01
majst01 force-pushed the mcp-server branch 5 times, most recently from 87ca65e to 23241b3 Compare August 5, 2026 05:11
@majst01
majst01 marked this pull request as ready for review August 13, 2026 13:55
@majst01
majst01 requested a review from a team as a code owner August 13, 2026 13:55
@majst01
majst01 requested a review from iljarotar August 13, 2026 13:55
Comment thread cmd/mcp/mcp.go Outdated
RunE: func(cmd *cobra.Command, args []string) error {
raw, mcps := gosdk.NewServer("metal-stack.io mcp server", v.V.String())

forwardToClients(c, mcps)

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.

Disable logging to prevent that the AI agent don't get disurbed

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.

done

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant