Repository navigation
Conversation
- Add hero with 1.1K+ downloads badge linking to ClawHub skill page - Reorganize intro: tagline + 'Why it matters' for non-technical readers - Add 'Traction' table with ClawHub metrics (downloads, version, date) - Add badges (downloads, license, ACP/JSON-RPC, OpenCode, Python, CI) - Document real JSON-RPC method flow (initialize -> session/new -> prompt -> update -> cancel -> load) - Add author block with hakke.cl / ClawHub / GitHub links - Quick start now shows ClawHub install as recommended path
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe README was reorganized into structured sections covering the skill description, capabilities, install and demo steps, ACP/JSON-RPC method mapping, repository layout, CI checks, contributing guidance, author links, and license reference. ChangesREADME restructure
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7aaf317506
ℹ️ 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".
| # Load as skill (Hermes Agent) | ||
| # Option A — copy into the active profile's skills dir | ||
| # Hermes Agent — copy into the active profile's skills dir | ||
| cp SKILL.md ~/.hermes/profiles/<profile>/skills/opencode-acp-control.md |
There was a problem hiding this comment.
Restore a checkout step before copying SKILL.md
For the Hermes install path, a user following the README from GitHub no longer has a local checkout before this cp SKILL.md ... command—the clone/cd lines that previously created it were removed—so the quick start fails with cp: cannot stat 'SKILL.md'. Add a clone/download step, or show a raw download command, before the copy instructions.
Useful? React with 👍 / 👎.
Why
The README was technically correct but did not surface the main social proof: 1.1K+ downloads on ClawHub. A recruiter or agent author landing on the repo sees a plain technical intro with no signal of traction.
What changed
Scope
Risk
Low. README only. No functional change to the skill or the demo script.
Links
Summary by CodeRabbit
Documentation
New Features