Skip to content

fix(code-review): reduced permission prompts, prevented unnecessary cd calls #33397

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.

Sign up for GitHub

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

Open
sunnypatell wants to merge 1 commit into
base: main
Choose a base branch
from
+5 −3
Open

fix(code-review): reduced permission prompts, prevented unnecessary cd calls #33397

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions plugins/code-review/commands/code-review.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,15 @@
---
allowed-tools: Bash(gh issue view:*), Bash(gh search:*), Bash(gh issue list:*), Bash(gh pr comment:*), Bash(gh pr diff:*), Bash(gh pr view:*), Bash(gh pr list:*), mcp__github_inline_comment__create_inline_comment
allowed-tools: Bash(gh:*), Bash(git diff:*), Bash(git log:*), Bash(git blame:*), Bash(git show:*), Bash(git rev-parse:*), Read, Grep, Glob, mcp__github_inline_comment__create_inline_comment
description: Code review a pull request
---

Provide a code review for the given pull request.

**Agent assumptions (applies to all agents and subagents):**
- All tools are functional and will work without error. Do not test tools or make exploratory calls. Make sure this is clear to every subagent that is launched.
**Agent assumptions (applies to all agents and subagents). Relay these rules to EVERY subagent you launch:**
- All tools are functional and will work without error. Do not test tools or make exploratory calls.
- Only call a tool if it is required to complete the task. Every tool call should have a clear purpose.
- **NEVER prefix bash commands with `cd`.** The working directory is already correct. Run commands directly (e.g., `gh pr diff 123`, NOT `cd /path && gh pr diff 123`). This applies to all commands including `gh`, `git`, and any other CLI tools. Chaining `cd` with commands triggers unnecessary permission prompts and degrades the review experience.
- Use `Read` or `Grep` to inspect files instead of `cat` or `grep` bash commands.

To do this, follow these steps precisely:

Expand Down
Loading