Skip to content

docs(pr-review-toolkit): zero-prompt redesign proposal #87

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

Closed
cblecker wants to merge 1 commit into from
Closed

docs(pr-review-toolkit): zero-prompt redesign proposal #87

Show file tree
Hide file tree
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
7 changes: 5 additions & 2 deletions pr-review-toolkit/docs/PR_REVIEW_REQUIREMENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,9 @@ optimized first as separate user-facing commands.

### Safe And Predictable Tooling

- GitHub data should be fetched through GitHub MCP tools.
- GitHub API data (metadata, review threads) and review posting should
go through GitHub MCP tools; repository content may come from
read-only local git.
- GitHub read operations and GitHub write operations should be clearly separated.
- Posting should only happen after explicit user approval.
- The toolkit should avoid ad-hoc, model-generated parsing scripts for GitHub
Expand Down Expand Up @@ -276,7 +278,8 @@ The user should be able to edit or remove drafted comments before posting.

- Analysis should avoid tools capable of modifying the repository unless the
user explicitly requests a different workflow.
- GitHub data access should use GitHub MCP read tools.
- GitHub API data access should use GitHub MCP read tools; repository
content may come from read-only local git.
- GitHub posting should use GitHub MCP write tools.
- Generated parsing scripts should not be used for GitHub data.
- Reusable helper scripts, if used, should be bundled with the plugin and
Expand Down
Loading
Loading