fix: prevent && command chaining in code reviewer

[dotuananh0712]

Summary

• Adds guidance to run git diff commands separately instead of chaining with &&
• Prevents consent prompts that occur when Claude chains commands

Fixes #476

Summary by CodeRabbit

Documentation

• Updated code review documentation with improved guidance on running diagnostic commands.
• Added explicit instructions emphasizing proper command execution best practices.
• Enhanced clarity through formatted examples demonstrating correct usage patterns.
• Includes important notes highlighting common pitfalls to avoid during workflow execution.

[coderabbitai]

📝 Walkthrough

Walkthrough

Documentation update to the code reviewer guide that emphasizes running git diff commands separately rather than chaining them with && operators, addressing command execution issues.

Changes

Cohort / File(s)	Summary
Documentation - Code Review Guide skills/requesting-code-review/code-reviewer.md	Added explicit guidance and formatting to instruct running git diff commands separately instead of chaining with &&, with repeated emphasis on avoiding command chaining. No functional code changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Commands were chaining, oh what a fright,
With && symbols causing prompts in the night,
But now we're clear, keep them apart,
One at a time, a documentation art! 🌟

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: preventing command chaining with && in code reviewer documentation to address the consent prompt issue.
Linked Issues check	✅ Passed	The pull request addresses issue #476 by adding explicit guidance in documentation to run git diff commands separately instead of chaining with &&, directly preventing the consent prompt issue.
Out of Scope Changes check	✅ Passed	All changes are scoped to the code-reviewer.md documentation file with guidance on command execution, directly related to fixing issue #476.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

No actionable comments were generated in the recent review. 🎉

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}