docs(readme): layer in spec-03 beats — npm badge, Tier-1/Tier-2 split, hyperlinked clients#72
Conversation
…, hyperlinked clients (closes W4) W4 from READINESS-AUDIT.md called for the launch-doc-spec-03 README beats. This adds the load-bearing pieces without depending on services we haven't stood up yet: - npm version badge wired to the (eventual) https://www.npmjs.com/package/openhop — shields.io renders "no published" until B4 publishes openhop@0.1.0. - "Works with" badge row split into two: * Tier-1 (one-step): Claude Code / Cursor / Windsurf / Cline / Continue.dev — each badge is a hyperlink to the client's official skills/rules docs URL (cited in packages/cli/src/init.ts:70-84). Continue.dev keeps its "advisory" mark per init.ts. * Tier-2 (via OpenSkills): Codex CLI, Gemini CLI, JetBrains Junie, GitHub Copilot, OpenCode, Goose, Antigravity — each linked to the client homepage. Aider sits next to them with a "CLI only*" footnote since it has no skill surface. - Install table grows a "Tier 1" / "everything else" row pair mirroring the badge split. The Tier-2 row is the universal-install line per 17:534 (`npx openskills install naorsabag/openhop`). Deferred until services exist (called out separately in the audit): - Discord link/CTA (no Discord registered yet) - npm version badge will only render usefully once B4 (npm publish) lands Verified: prettier --check clean. All hyperlink targets return 200 (including shields.io badge generators for the new "via OpenSkills" and "CLI only*" badges). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughUpdated README.md: added an npm version badge; pruned the “Works with” badges to Claude Code, Cursor, and OpenAI Codex; and restructured the Install section into separate rows for one-shot demo, long-lived server, client auto-detect install ( ChangesREADME: Badge and Install restructuring
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 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)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 84: Replace the non-executable placeholder "git clone …" in the README
table row labeled "Run from source (contributors)" with a concrete,
copy-pasteable git clone command that includes the repository HTTPS URL
(including the .git suffix) so contributors can directly run clone then install
then start dev; keep the existing sequence of commands (clone, npm install, npm
run dev) and ensure the table cell content is a single executable command
string.
- Line 82: Update the Tier 1 install table row that currently reads "| **Install
the skill — Tier 1** | `npx openhop init` — auto-detects Claude Code,
Cursor, Windsurf, Cline, Continue.dev and drops `SKILL.md` in place |" to make
Continue.dev's behavior advisory instead of implying full auto-detection parity;
change the phrasing after `npx openhop init` to list Continue.dev as "advisory"
or "recommended (advisory)" rather than "auto-detects", e.g., keep auto-detect
wording for Claude Code, Cursor, Windsurf, Cline and append a clause like
"Continue.dev — advisory (may require manual setup)" so readers are not misled
about automation, ensuring `SKILL.md` mention remains.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 3195e419-b409-4a63-81e3-6991ed763392
📒 Files selected for processing (1)
README.md
- Tier-1 install row: call out that Continue.dev support is advisory. The badge row already marks it that way; the install row was implying full auto-detect parity. Fixes that mismatch (CodeRabbit @ README:82). - Contributor row: replace `git clone …` placeholder with a real, copy-pasteable command including the repo URL and `cd OpenHop` (CodeRabbit @ README:84). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Both nits addressed in b7efb08:
|
…r, Codex) Drops the Tier-1/Tier-2 framing entirely. The badge bar now shows the three flagship clients in a single line; the longer list of supported clients (Windsurf, Cline, Codex, Gemini, Junie, Copilot, OpenCode, Goose, Antigravity, Aider) lives only in the install table. Install-table rows still distinguish the two install paths (`openhop init` vs `openskills install`), but the column labels lose the "Tier 1" language and just enumerate the clients in parentheses. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 66: Update the Tier-1 install row and badge bar to include Continue.dev
with the advisory note: modify the table cell containing the install text
"Claude Code, Cursor, Windsurf, Cline" (the string used with the `npx openhop
init` install guidance) to add "Cline (Continue.dev support is advisory)" or
append "Continue.dev (advisory)" next to Cline, and ensure the top badge bar
includes a Tier-1 mention for Continue.dev with the same advisory wording so the
README text and badges match the PR summary.
- Around line 28-30: Update the README badge bar so it matches the install-table
and feature descriptions: either replace the Codex badge with Windsurf and Cline
badges (so the badge bar lists "Windsurf" and "Cline" alongside Claude Code and
Cursor) or add a short clarifying sentence next to the badge bar explaining that
badges are not strictly the Tier-1 "npx openhop init" clients and that Codex is
shown for historical/third-party reasons; adjust the badge alt text and links
for the chosen badges and ensure consistency with the "npx openhop init" and
"npx openskills install" references elsewhere (search for "Codex", "Windsurf",
"Cline", "npx openhop init", and "npx openskills install" to find the badge bar
and install table to update).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: f7177af9-075a-4351-a946-635520080a91
📒 Files selected for processing (1)
README.md
The tier-collapse in f9aa3fd dropped Continue.dev from the install table's parenthesized client list. init.ts still detects Continue.dev (advisory), so the row should mention it for honesty. Addresses the second of two CodeRabbit nits on f9aa3fd. The first nit (badge selection should mirror the install-table tier structure) is intentionally not actioned — the user's request was to highlight three flagship clients (Claude Code, Cursor, Codex) regardless of install mechanism. Replied on the PR thread. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Round 2 review:
|
Per follow-up, take Continue.dev out of the Tier-1 install row's parenthesized client list. Reverts c436bd7. The Features bullet at line 44 still mentions Continue as part of the supported-client prose list — that's flavor copy, not a tag. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
W4 from
READINESS-AUDIT.mdcalled for the launch-doc-spec-03 README beats. This adds the load-bearing pieces without depending on services we haven't stood up yet.https://www.npmjs.com/package/openhop. Shields.io renders gracefully pre-publish; auto-flips to the live version once B4 lands.npx openhop init: Claude Code, Cursor, Windsurf, Cline, Continue.dev (advisory). Each badge hyperlinked to the client's official skills/rules docs URL (cited inpackages/cli/src/init.ts:70-84).npx openskills install naorsabag/openhop: Codex CLI, Gemini CLI, JetBrains Junie, GitHub Copilot, OpenCode, Goose, Antigravity. Each linked to the client homepage. Aider gets aCLI only*footnote since it has no skill surface.Deferred (still not in this PR): Discord link (no Discord registered yet).
Test plan
npm run format:checkclean.🤖 Generated with Claude Code
Summary by CodeRabbit