docs: restore manual quick start in README - #275
Merged
Conversation
Contributor
🟢 No documentation gap found — medium confidenceThe automated review found no documentation gap in the supplied changes. This review is advisory and does not block merging. |
ankitranjan7
pushed a commit
that referenced
this pull request
Aug 11, 2026
ankitranjan7
added a commit
that referenced
this pull request
Aug 11, 2026
* docs: route web work to Webcmd instead of blanket-disabling native tools Section 4 of start.md told every harness to disable its native browser tools. That is honest for narrow fetch/search tools, which Webcmd covers end to end, but wrong for full browser toolsets like Cursor's Browser, Hermes' browser_*, and OpenClaw's browser: those are wired into the local dev loop and do things Webcmd does not target. Split the table into Replace (disable) and Route (keep, with a routing rule), state the boundary as integration rather than capability, and keep full-disable as an explicit opt-in. Also resolves the contradictory "disable any web-search tool / do not disable builtin web search tool" sentence. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: ask before disabling tools, and cover user-installed ones Agents fetch start.md and follow it verbatim, so section 4 has to say who makes the call: the agent proposes, the user confirms. Retitle to "Override installed tools" and add a section for tools the user installed themselves (Playwright/Puppeteer MCPs, scraping or search MCPs, custom web-fetch skills), which compete with Webcmd the same way built-ins do. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: trim section 4 to a table and two lines Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: update tool disabling recommendations in start.md to include browser navigation and exclude search tools * docs: fill in the tool-shortlisting rubric in section 4 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: update tool disabling rubric and harness configuration table in start.md * docs: complete the keep-web-search rubric entry and correct harness tool names Researched the actual fetch/search/browser tools of all seven harnesses. Fixes: Hermes' web_search is in the `web` toolset, not bundled inside `browser`; Codex has no fetch or browser tool at all; Cursor's `Web` is the search tool and `Browser` is the navigation surface; OpenClaw's search tools move to the keep column. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: generalize the web_search entry instead of listing every harness Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: simplify descriptions for web search and scoped search tools in start.md * docs: align all seven harness pages with the researched tool inventory Keep web search everywhere; disable only fetch and open-web browser tools. Per-harness corrections from primary docs: Claude Code keeps WebSearch and gains the Browser-pane MCP tools; Codex has nothing to disable and should move off the stale "cached" default; OpenCode keeps websearch; Cursor's Web is its search tool while Browser is the overlap; Hermes toggles toolsets not tools, so web_extract cannot be dropped without losing web_search; OpenClaw keeps its three search tools and disables the browser via browser.enabled; Pi's competing surface is entirely user-installed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: document marketplace install on the Claude Code and Codex pages Both pages now lead with the plugin path and keep `webcmd skills add` as the manual alternative, with a warning against doing both. Commands were verified against the locally installed claude and codex CLIs. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * docs: restore manual quick start in README (#275) * feat: add Claude Code plugin marketplace manifest (#273) Claude Code discovers marketplaces via .claude-plugin/marketplace.json, so `claude plugin marketplace add agentrhq/webcmd` failed against the existing Codex-only manifests. Add the Claude Code marketplace and plugin manifests pointing at the repo root, so the seven bundled skills install as a plugin. Wire the new manifest version into release-please and extend the plugin metadata check to keep both manifests in sync with package.json. Co-authored-by: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Co-authored-by: ngaurav <nishant@agentr.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan