Skip to content

feat: make code review extension available for pr reviews#18

Open
cynthialong0-0 wants to merge 3 commits intogemini-cli-extensions:mainfrom
cynthialong0-0:main
Open

feat: make code review extension available for pr reviews#18
cynthialong0-0 wants to merge 3 commits intogemini-cli-extensions:mainfrom
cynthialong0-0:main

Conversation

@cynthialong0-0
Copy link

@cynthialong0-0 cynthialong0-0 commented Feb 27, 2026

Summary

This pull request add ability for code-review extension to do PR reviews with provided github mcp server and env variables. It will be triggered with user prompt /code-review pr-review.

The pr-review integration will be used in the gemini cli github action pr review later.

Changes

  • Update code-review.toml with shell commands to check if injected args equal to pr-review, then display the corresponding instructions. For the pr-review workflow, it will read env variables for the pull request information, and look for tools from github mcp server to leave comments in the PR.

related google-github-actions/run-gemini-cli#468

Copy link
Contributor

@mctavish mctavish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the README.md with instructions on how to use this form of the extension. While this is intended for the Gemini CLI Github Action, the functionality is generally available and may address the needs of other users of the extension.

Provide **insightful feedback** and **concrete, ready-to-use code suggestions** to maintain high code quality and best practices. Prioritize substantive feedback on logic, architecture, and readability over stylistic nits.
</OBJECTIVE>

<CONTEXT>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<CONTEXT> tags should be inside the conditional, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants