docs(peek): skill tool count (14→16) + docs-deploy buildCommand fix#133
Conversation
…ent_detail Signed-off-by: harry-harish <22562634+harry-harish@users.noreply.github.com>
…ilter peek-docs...) Signed-off-by: harry-harish <22562634+harry-harish@users.noreply.github.com>
|
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 (3)
📝 WalkthroughWalkthroughUpdates ChangesPeek skill docs & build config
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 |
Two small documentation/content follow-ups.
1. Skill documents all 16 tools (was 14)
The bundled Claude Code skill (
packages/peek-cli/skills/peek-skill.md, installed bypeek init) claimed "14 tools" but the MCP server registers 16 — it was missing the two live-page read tools. Added a "Live-page read tools (2)" subsection forget_page_view+get_element_detail, corrected the count in all four places (frontmatter, body, heading, the "entire surface" note), relabeled the existing read group "Session-forensics read tools (8)", and updated the Level-1 description to "10 read tools (8 session-forensics + 2 live-page)". Documentation only — both tools already shipped. Changeset:@peekdev/clipatch.2. Docs-deploy buildCommand builds workspace deps first
CLAUDE.md's documented VercelbuildCommand(pnpm --filter peek-docs run build) only builds the app, so on a cache-cold runner@cubenest/docs-shared/distis missing and the Astro build fails (Failed to resolve entry for package "@cubenest/docs-shared"). Fixed both docs sites topnpm --filter peek-docs... run build/tracelane-docs... run build(the trailing...builds workspace dependencies in topological order first) + documented why. (This is the command the last successful peek-docs deploy actually used.)Test plan
pnpm lint— clean (513 files)pnpm changeset status—@peekdev/clirecognized🤖 Generated with Claude Code
Summary by CodeRabbit