Skip to content

docs(example): show command to run MCP Inspector #42

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

Merged
merged 3 commits into from
May 27, 2025

Conversation

cablehead
Copy link
Contributor

No description provided.

Copy link
Member

@hashemix hashemix left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, @cablehead!

Requested a small change ☝️

Also, for consistency, would you mind adding a similar command to the README files of the other server examples as well?

  • examples/hello-world-mcp-server-core
  • examples/hello-world-server-sse
  • examples/hello-world-server-core-sse

@@ -28,6 +28,19 @@ cargo build -p hello-world-mcp-server --release
3. After building the project, the binary will be located at `target/release/hello-world-mcp-server`
You can test it with [MCP Inspector](https://modelcontextprotocol.io/docs/tools/inspector), or alternatively, use it with any MCP client you prefer.

```bash
npx @modelcontextprotocol/inspector ./target/release/hello-world-mcp-server
Copy link
Member

@hashemix hashemix May 26, 2025

Choose a reason for hiding this comment

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

Just a small suggestion , I think it is better to also add -y to it to avoid prompting the user
( terminal output needs to be updated accordingly too)

npx -y @modelcontextprotocol/inspector ./target/release/hello-world-mcp-server

@cablehead cablehead requested a review from hashemix May 27, 2025 15:03
Copy link
Member

@hashemix hashemix left a comment

Choose a reason for hiding this comment

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

Looks great! thanks for the updates.

@hashemix hashemix merged commit fb7f441 into rust-mcp-stack:main May 27, 2025
3 checks passed
@cablehead
Copy link
Contributor Author

thanks!

@cablehead cablehead deleted the patch-1 branch May 28, 2025 12:53
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.

2 participants