docs(peek): harden llms-install.md for Windows + Cline agents#128
Conversation
From a real Cline-on-Windows install run: - Step 3: split `npm install -g @peekdev/cli && peek init` into two commands — `&&` chaining is invalid in Windows PowerShell 5.1 (the stock Windows shell). - Step 1: state explicitly it's a config-FILE edit (not a shell command) and name each client's config path, so an agent doesn't route it to execute_command. - Step 2: make verification reload-aware (peek's tools appear only after the client reloads) and note list_recent_sessions is an MCP tool, not a CLI. Docs only — no changeset, no published-package change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: harry-harish <22562634+harry-harish@users.noreply.github.com>
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
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)
📝 WalkthroughWalkthrough
ChangesInstallation Guide Clarifications
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
What
Hardens
llms-install.md(peek's Cline-Marketplace install script) for Windows + Cline agents — from a real Cline-on-Windows test run that hit a transient malformed-tool-call while following it. Docs only, no changeset.npm install -g @peekdev/cli && peek initinto two separate commands;&&chaining is invalid in Windows PowerShell 5.1 (the stock Windows default shell). Also flagspeek initas interactive.cline_mcp_settings.json,~/.cursor/mcp.json,~/.claude.json), so an agent doesn't wrongly route it toexecute_command.list_recent_sessionsis an MCP tool, not a CLI.Net effect: fewer ways for an agent to emit a malformed/empty shell command while following the doc → higher first-try success for Cline Marketplace installs (submitted as cline/mcp-marketplace#1911).
🤖 Generated with Claude Code
Summary by CodeRabbit
list_recent_sessionsis an MCP tool, not a command-line command.