Skip to content

mcp-rs: rename the cargo package and binary to rustwright-mcp#159

Merged
suchintan merged 6 commits into
mainfrom
repo-sync/rustwright-cloud-136
Jul 24, 2026
Merged

mcp-rs: rename the cargo package and binary to rustwright-mcp#159
suchintan merged 6 commits into
mainfrom
repo-sync/rustwright-cloud-136

Conversation

@suchintan

Copy link
Copy Markdown
Member

@suchintan suchintan added the sync Automated cross-repository sync label Jul 24, 2026
@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Summary

Clean, self-contained rename of the mcp-rs cargo package/binary to rustwright-mcp (matching the npm distribution name). Docs, tests, and the CLI's binary-lookup fallback were all updated consistently, and I couldn't find any stale mcp-rs references left behind.

🔴 Critical Issues (0)

None found.

🟡 Suggestions (1)
  • python/rustwright/cli.py:1480 correctly falls back to shutil.which("mcp-rs") if rustwright-mcp isn't found, and tests/test_cli_mcp_integration.py checks both binary names in the local target dir — good backward-compat handling for anyone with an old build lying around. Worth double-checking this fallback has an explicit removal date/issue tracked (e.g. a follow-up cleanup once the rename has been out for a release or two), since CONTRIBUTING.md doesn't otherwise track deprecation windows for internal tooling like this.
📝 Minor / Style (1)
  • The new CODE_ARCHITECTURE.md row for mcp/ (deprecated Python MCP server) is a good addition for orientation, but it's a bit of a scope surprise in an otherwise pure-rename PR — consider calling it out in the PR description (it currently just links to the internal companion PR) so reviewers know it's intentional documentation-only scope creep, not an accidental artifact of the rename.

@suchintan
suchintan merged commit 7d692a2 into main Jul 24, 2026
14 of 15 checks passed
@suchintan
suchintan deleted the repo-sync/rustwright-cloud-136 branch July 24, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sync Automated cross-repository sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant