Skip to content

docs: remove uv installation instructions from README - #15

Merged
chouzz merged 3 commits into
mainfrom
docs/readme-install-cleanup
Aug 19, 2026
Merged

docs: remove uv installation instructions from README#15
chouzz merged 3 commits into
mainfrom
docs/readme-install-cleanup

Conversation

@chouzz

@chouzz chouzz commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Changes

  • Remove the curl | sh uv install script and the "make sure uv is installed" instruction from the Installation section — this repo is not a tutorial for installing uv. The uvx doc link remains for anyone who needs it.
  • Claude Code and Codex sections are already collapsed by default (their <details> blocks have no open attribute, from docs: add collapsible per-agent MCP install instructions #13), so no change was needed there.

The Development section keeps its uv run usage since that is the actual dev workflow for this repo.

chouzz added 2 commits August 19, 2026 03:45
This repo is not a tutorial for installing uv; users who need it can
refer to the linked uvx docs.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8806ae43ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
```

Then register the server with your AI coding agent. Click yours to expand 👇
RemoteShell is a stdio MCP server distributed on PyPI. Run it directly with [`uvx`](https://docs.astral.sh/uv/) — no install step required. Then register the server with your AI coding agent. Click yours to expand 👇

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Document the uvx prerequisite before setup

When installing on a fresh machine that does not already have uv/uvx on PATH, every registration example below stores or invokes uvx remoteshell-mcp, so the MCP client will fail with a command-not-found error. This edit removed the only prerequisite/installation note while still saying “no install step required,” leaving first-time users without a step to make the documented commands runnable; consider keeping a short uvx prerequisite note even if the curl installer snippet is omitted.

Useful? React with 👍 / 👎.

@chouzz
chouzz merged commit 7451b46 into main Aug 19, 2026
2 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.

1 participant