Skip to content

add MCP server badge #1

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add MCP server badge #1

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ Node.js server implementing the Model Context Protocol (MCP) for running arbitra

![Website Preview](https://raw.githubusercontent.com/alfonsograziano/node-code-sandbox-mcp/master/assets/images/website_homepage.png)

<a href="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp">
<img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox Server MCP server" />
Copy link

@gemini-code-assist gemini-code-assist bot Nov 5, 2025

Choose a reason for hiding this comment

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

medium

For improved accessibility, the alt text for the badge image should be more descriptive. The current text, "Node.js Sandbox Server MCP server", is slightly repetitive. I suggest changing it to more clearly state the purpose of the badge.

Suggested change
<img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox Server MCP server" />
<img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Status badge for Node.js Sandbox MCP server" />

</a>
Comment on lines +7 to +9
Copy link

@coderabbitai coderabbitai bot Nov 5, 2025

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

Minor: Remove redundant "server" in badge alt text.

The alt text "Node.js Sandbox Server MCP server" repeats "server" twice, which is redundant. Simplify to avoid confusion for screen readers.

-  <img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox Server MCP server" />
+  <img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox MCP server badge" />
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<a href="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp">
<img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox Server MCP server" />
</a>
<a href="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp">
<img width="380" height="200" src="https://glama.ai/mcp/servers/@ssdeanx/node-code-sandbox-mcp/badge" alt="Node.js Sandbox MCP server badge" />
</a>
🤖 Prompt for AI Agents 
In README.md around lines 7 to 9, the image badge alt text reads "Node.js
Sandbox Server MCP server" which redundantly repeats "server"; update the img
alt attribute to a concise, non-redundant phrase such as "Node.js Sandbox MCP"
(or "Node.js Sandbox Server MCP" if you want to keep the single word "Server")
to remove the duplicate word and maintain clear, accessible alt text for screen
readers.


👉 [Look at the official website](https://jsdevai.com/)

## Features
Expand Down Expand Up @@ -262,7 +266,7 @@ Terminate and remove the sandbox container.
- Incrementally install and reuse dependencies.
- **One-shot execution** with `run_js_ephemeral` is perfect for:
- Quick experiments or simple scripts.
- Cases where you don’t need to maintain state or cache dependencies.
- Cases where you don't need to maintain state or cache dependencies.
- Clean, atomic runs without worrying about manual teardown.
- **Detached mode** is useful when you want to:
- Spin up servers or long-lived services on-the-fly
Expand Down Expand Up @@ -290,4 +294,4 @@ The above copyright notice and this permission notice shall be included in all c
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
Copy link

@gemini-code-assist gemini-code-assist bot Nov 5, 2025

Choose a reason for hiding this comment

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

medium

This change removes the final newline character from the file. It is a widely-followed convention to end text files with a single newline character. This practice avoids unexpected behavior with common command-line tools (like cat) and helps version control systems represent file changes more clearly. Please re-add the newline at the end of the file.

Copy link

@coderabbitai coderabbitai bot Nov 5, 2025

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

Add trailing newline to file.

The file should end with a single newline character per markdown best practices (MD047).

 OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

297-297: Files should end with a single newline character

(MD047, single-trailing-newline)

🤖 Prompt for AI Agents 
In README.md around line 297, the file is missing a trailing newline; add a
single newline character at the end of the file so it ends with exactly one
newline (per MD047) and ensure no extra blank lines are appended.