Skip to content

Refactor and update Gemini CLI version#328

Open
RissyRan wants to merge 1 commit intomainfrom
update_gemini_version
Open

Refactor and update Gemini CLI version#328
RissyRan wants to merge 1 commit intomainfrom
update_gemini_version

Conversation

@RissyRan
Copy link
Collaborator

@RissyRan RissyRan commented Feb 6, 2026

Description:

  • Onboard Gemini CLI assistant
  • Refactor Gemini CLI to achieve parity with MaxText for support and maintenance.
  • Updated the developer doc here

How does it work now?

PR review

Option 1 - same as before, add a gemini-review label
Option 2 - add any inline comment or comment summary starting with @gemini-cli /review with your context

General invocations & requests

Add any inline comment or comment summary starting with @gemini-cli with your context

Test:

Manual tests in PR

@RissyRan RissyRan requested a review from entrpn as a code owner February 6, 2026 22:25
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

RissyRan

This comment was marked as outdated.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

🤖 Hi @RissyRan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

## 📋 Review Summary

This Pull Request refactors and updates the Gemini CLI version by reorganizing its GitHub Actions workflows and defining new TOML configuration files for the gemini-invoke and gemini-review commands. The changes significantly improve the modularity and clarity of the Gemini CLI's integration with GitHub Actions.

🔍 General Feedback

  • The new workflow structure using workflow_call for gemini-invoke.yml and gemini-review.yml is a good improvement for modularity and reusability.
  • The security considerations regarding command substitution and untrusted input are clearly stated in the TOML prompt files, which is good for the agent's internal understanding.
  • The use of actions/create-github-app-token@v2 for minting identity tokens is a secure way to handle permissions.

RissyRan

This comment was marked as outdated.

Copy link
Collaborator Author

@RissyRan RissyRan left a comment

Choose a reason for hiding this comment

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

@gemini-cli /review

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

🤖 Hi @RissyRan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

📋 Review Summary

This Pull Request refactors the Gemini CLI integration, introducing new workflows and command definitions. The changes streamline the bot's operation and bring it to parity with MaxText.

🔍 General Feedback

  • The refactoring into separate dispatch, invoke, and review workflows is a positive change for modularity.
  • Several suggestions have been made regarding action version pinning and tool permissions to enhance stability and security.
  • Consider externalizing lengthy prompts for improved maintainability in the future.

@AI-Hypercomputer AI-Hypercomputer deleted a comment from github-actions bot Feb 6, 2026
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

🤖 Hi @RissyRan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

🤖 Hi @RissyRan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

📋 Review Summary

This Pull Request refactors and updates the Gemini CLI version by introducing new workflow files and configuration files. The changes aim to streamline the Gemini CLI integration and align it with MaxText for better support.

🔍 General Feedback

  • The new workflow setup is clear and modular, which is a good improvement for maintainability.
  • There are a few areas identified for improvement related to consistency in tool naming within prompts and pinning action versions for stability.

@github-actions
Copy link

github-actions bot commented Feb 7, 2026

🤖 Hi @RissyRan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

## 📋 Review Summary

This pull request refactors and updates the Gemini CLI version, including new GitHub Actions workflows for dispatching, invoking, and reviewing. The changes aim to onboard the Gemini CLI assistant and achieve parity with MaxText for support and maintenance.

🔍 General Feedback

  • The refactoring into separate workflows for dispatch, invoke, and review is well-structured and improves modularity.
  • The TOML configurations for invoke and review commands are detailed and adhere to security protocols.
  • All necessary permissions and environment variables appear to be correctly configured across the new workflows.
  • A minor improvement would be to pin the google-github-actions/run-gemini-cli action to a specific version or commit SHA instead of @main for better stability in production.

@AI-Hypercomputer AI-Hypercomputer deleted a comment from github-actions bot Feb 7, 2026
@RissyRan RissyRan changed the title [WIP] Refactor and update Gemini CLI version Refactor and update Gemini CLI version Feb 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant