docs: fix stale commands and references - #7205
Conversation
The docs had drifted from the code: - `RPC_REQUIRED_SCOPE` was renamed to `RPC_REQUIRED_SCOPES` in apps/server/src/auth/RpcAuthorization.ts. - The Ghostty upstream pin was attributed to mobile's VERSION file; it lives at native/libghostty-vt/VERSION and is shared by Android and web. - Several pages still used `pnpm` invocations from before the `vp` migration, so the commands fail when pasted. - The mobile test skill pointed at `scripts/pair-client.sh`, which does not resolve from the repository root. - docs/user/composer.md and docs/architecture/terminal-renderers.md existed but were not linked from the docs index. - terminal-renderers.md was missing the "For maintainers" banner that the other internals and operations pages carry. Docs only; no source or behavior changes.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
ApprovabilityVerdict: Approved 6b81280 Documentation-only changes updating stale CLI commands ( You can customize Macroscope's approvability policy. Learn more. |
The docs had drifted from the code:
RPC_REQUIRED_SCOPEwas renamed toRPC_REQUIRED_SCOPESin apps/server/src/auth/RpcAuthorization.ts.pnpminvocations from before thevpmigration, so the commands fail when pasted.scripts/pair-client.sh, which does not resolve from the repository root.Docs only; no source or behavior changes.
What Changed
Why
UI Changes
Checklist
Note
Low Risk
Markdown and agent-skill documentation only; no runtime, API, or build logic changes.
Overview
Docs-only alignment so pasted commands and cross-references match the repo today—no application or server behavior changes.
Command and path fixes: Maintainer docs that still showed
pnpmfor screenshot capture, Ghostty WASM builds, desktop dev web, and Expo now usevp run/vp exec. The mobile test skill’s troubleshooting line points at.agents/skills/test-t3-mobile/scripts/pair-client.shinstead of a root-relativescripts/path that does not exist.Accuracy: Architecture overview renames
RPC_REQUIRED_SCOPEtoRPC_REQUIRED_SCOPES. Ghostty docs and the web ghostty README attribute the upstream pin tonative/libghostty-vt/VERSION(shared by Android and web), not mobile’s VERSION file.Navigation and polish:
docs/README.mdadds links to Message composer and Terminal renderers;terminal-renderers.mdgets the standard maintainer banner.updating.mdgets a small grammar tweak on post-update wording; the screenshot runbook backticks--skip-build.Reviewed by Cursor Bugbot for commit 958c498. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix stale commands and references across docs
pnpmcommands tovp runequivalents in several docs, including t3-connect.md, mobile-app-store-screenshots.md, and terminal-renderers.md.RPC_REQUIRED_SCOPEtoRPC_REQUIRED_SCOPESin overview.md.Macroscope summarized 958c498.