feat(install-mcp): add openclaw host adapter#242
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Openclaw is not urgently needed and I will take a look after checking other PRs |
52b2a6e to
7afc1d0
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7c11ad5844
ℹ️ 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".
What changed
Adds
adapters/openclaw/— a ninth host in thevouch install-mcpadaptercatalogue, alongside a generic-discovery test for its full T1-T4 install,
idempotency, and that its T3 slash commands are byte-identical to
claude-code's (proving reuse, not a fork).
Closes #230
Why
vouch install-mcp openclawcurrently fails — only eight hosts areregistered. OpenClaw needs the same manifest-driven adapter experience
Claude Code users get: a one-command writer that drops plugin-enablement,
docs, slash commands, and policy into a managed project. Closes #230.
What might break
Nothing. No
.vouch/on-disk shape changes, nokb.*method changes —this is purely a new
adapters/<host>/template directory plus itsmanifest, discovered generically by the existing
install_adapter.pywriter. Existing hosts are untouched.
VEP
Not a surface change — no VEP needed.
Tests
make checkpasses locally (lint + mypy + pytest)CHANGELOG.mdupdated under## [Unreleased]