-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce mcp server cli adapter #166
Conversation
I am currently working on the failing test. |
...main/java/io/quarkiverse/mcp/server/cli/adapter/deployment/McpServerCliAdapterProcessor.java
Outdated
Show resolved
Hide resolved
23faf61
to
cba828a
Compare
@mkouba applied your feedback. I now have to fix the broken test. For some reason it's behaving pretty random (e.g. most of the time no response is received). Any insight? |
I can try to take a look.. |
aa5cf2f
to
673b071
Compare
I think we should move the |
Added an abstract command that contains the I tried to use byte code transformers to get the |
5bcb33a
to
9db5b6c
Compare
Currently, the user just needs to drop the dependency into a project that contains a @topcommand. |
This is so cool! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks pretty cool. I've added a few minor comments.
cli-adapter/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
cli-adapter/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
@mkouba applied feedback |
@iocanel i will definitely need this I have a CLI i want to test this with! |
cli-adapter/runtime/src/main/resources/META-INF/quarkus-extension.yaml
Outdated
Show resolved
Hide resolved
@iocanel i built this locally and trying it with my Picocli to get it running and I can't My CLI: https://github.com/ksmpartners/domino-cli When I added <dependency>
<groupId>io.quarkiverse.mcp</groupId>
<artifactId>quarkus-mcp-server-cli-adapter</artifactId>
<version>999-SNAPSHOT</version>
</dependency> Then run Then run |
Will have a look first thing in the morning |
Pls file a new issue. |
@melloware: You should be using: |
@iocanel when I do that I get this error...
Also it seems the |
Resolves: #157