-
Notifications
You must be signed in to change notification settings - Fork 0
remcp: install claude code: remove mention of session hooks #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Visit the preview URL for this PR (updated for commit b2d406d): https://cerebral-af--pr23-remcp-docs-x3q3pmyg.web.app (expires Tue, 02 Dec 2025 16:18:53 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 8a2d9ca8dd85813c8df331b6bffa6a9b1abecf05 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review by RecurseML
🔍 Review performed on 1b876ce..adf05d7
✨ No bugs found, your code is sparkling clean
✅ Files analyzed, no issues (1)
• docs/remcp/installation/install-claude-code.mdx
recursiveAF
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar check on ln22.
Proposed solutions:
"This installs a global rule in your CLAUDE.md file that ensures Claude Code uses remcp automatically."
"If you want to remove the global rule and prefer to call remcp directly, you can run this command:"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review by RecurseML
🔍 Review performed on adf05d7..b2d406d
✨ No bugs found, your code is sparkling clean
✅ Files analyzed, no issues (1)
• docs/remcp/installation/install-claude-code.mdx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
High-level PR Summary
Updates documentation to clarify that the
remcp configure claudecommand installs instructions via CLAUDE.md rather than session hooks. The changes replace all mentions of "hooks" and "SessionStart hook" with more accurate terminology describing the instruction-based configuration approach.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
docs/remcp/installation/install-claude-code.mdx