Instructions #365
Instructions #365
Conversation
It was disturbing my sense of aesthetics
|
Tests pass locally, and I cannot see anything wrong with the one that fails in CI |
There was a problem hiding this comment.
Hey @zmeggyesi , thanks for the effort. However the instructions are already included in the InitializeResponse. You do not need a getter for this.
The public getter still might be useful for consistency and debugging.
I've left comments as you made some unnecessary changes.
Alos tests are missing
| <dependency> | ||
| <groupId>io.modelcontextprotocol.sdk</groupId> | ||
| <artifactId>mcp</artifactId> | ||
| <version>0.11.0-SNAPSHOT</version> |
There was a problem hiding this comment.
Please revert this change back to 0.11.0-SNAPSHOT
| <groupId>io.modelcontextprotocol.sdk</groupId> | ||
| <artifactId>mcp</artifactId> | ||
| <version>0.11.0-SNAPSHOT</version> | ||
| <version>0.12.0-SNAPSHOT</version> |
There was a problem hiding this comment.
Please revert this change back to 0.11.0-SNAPSHOT
| <groupId>io.modelcontextprotocol.sdk</groupId> | ||
| <artifactId>mcp</artifactId> | ||
| <version>0.11.0-SNAPSHOT</version> | ||
| <version>0.12.0-SNAPSHOT</version> |
There was a problem hiding this comment.
Please revert this change back to 0.11.0-SNAPSHOT
| </parent> | ||
| <artifactId>mcp</artifactId> | ||
| <packaging>jar</packaging> | ||
| <version>0.12.0-SNAPSHOT</version> |
There was a problem hiding this comment.
Please revert this change back to 0.11.0-SNAPSHOT
| import org.slf4j.LoggerFactory; | ||
| import reactor.core.publisher.Flux; | ||
| import reactor.core.publisher.Mono; | ||
| import reactor.util.annotation.Nullable; |
There was a problem hiding this comment.
Please remove the reactor Nullable.
| * Get the server instructions if available | ||
| * @return The preset instructions for communication with the server | ||
| */ | ||
| @Nullable |
There was a problem hiding this comment.
Please remove the reactor Nullable.
| import io.modelcontextprotocol.spec.McpSchema; | ||
| import io.modelcontextprotocol.spec.McpSchema.LoggingMessageNotification; | ||
| import io.modelcontextprotocol.util.Assert; | ||
| import reactor.util.annotation.Nullable; |
There was a problem hiding this comment.
Please remove the reactor Nullable.
| * Get the server instructions if available | ||
| * @return The preset instructions for communication with the server | ||
| */ | ||
| @Nullable |
There was a problem hiding this comment.
Please remove the reactor Nullable.
|
Closing as nothing to do. |
2 participants
Motivation and Context
As a follow-up of #99 this PR provides access to the instructions as a readable field so they can be returned to the client during initialization
How Has This Been Tested?
Breaking Changes
No.
Types of changes
Checklist
Additional context
closes #364