Skip to content

Update mcp server based on "github.com/mark3labs/mcp-go/server"#3

Merged
maximilien merged 17 commits intoAI4quantum:mainfrom
akihikokuroda:mcp-go
Oct 28, 2025
Merged

Update mcp server based on "github.com/mark3labs/mcp-go/server"#3
maximilien merged 17 commits intoAI4quantum:mainfrom
akihikokuroda:mcp-go

Conversation

@akihikokuroda
Copy link
Member

@akihikokuroda akihikokuroda commented Oct 21, 2025

There are 2 major changes:

  1. Change the mcp server based on "github.com/mark3labs/mcp-go/server"
  2. merge maestro mcp server from PR Port maestro MCP server #2

fix: #1

@akihikokuroda akihikokuroda marked this pull request as draft October 21, 2025 19:37
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda akihikokuroda marked this pull request as ready for review October 22, 2025 19:18
@akihikokuroda
Copy link
Member Author

All CI tests passed.
Please review.

I'll move some tests from maestro repo.
Thanks!

Copy link
Contributor

@maximilien maximilien left a comment

Choose a reason for hiding this comment

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

Is this ready for review @akihikokuroda ?

@akihikokuroda
Copy link
Member Author

No, not yet. I found some mock code by Bob and fixing them now. Please hold review for now.

Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda akihikokuroda marked this pull request as draft October 27, 2025 00:55
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
@akihikokuroda akihikokuroda marked this pull request as ready for review October 28, 2025 15:52
@akihikokuroda
Copy link
Member Author

@maximilien This is ready for review except workflow deploy. The deploy still needs a lot of work like building the maestro container image with the go executable, update the http request handling to use the go maestro.

Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Copy link
Contributor

@maximilien maximilien left a comment

Choose a reason for hiding this comment

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

Looks good. Let’s merge and test this more widely

@maximilien maximilien merged commit ecb10e1 into AI4quantum:main Oct 28, 2025
11 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.

question: mcp server

2 participants