Skip to content

chore(reviewbot): fix antigravity_review execution and configuration#3921

Merged
rickeylev merged 4 commits into
bazel-contrib:mainfrom
rickeylev:fix-reviewbot-tool-execution
Jul 11, 2026
Merged

chore(reviewbot): fix antigravity_review execution and configuration#3921
rickeylev merged 4 commits into
bazel-contrib:mainfrom
rickeylev:fix-reviewbot-tool-execution

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

Currently, running antigravity_review.py with uv run fails because it lacks PEP 723 dependency metadata and passes system_instructions to Agent instead of LocalAgentConfig. Also, skills_paths points to the skill directory rather than the SKILL.md file.

To fix, add inline PEP 723 script metadata declaring dependencies, pass system_instructions to LocalAgentConfig, and target SKILL.md directly in skills_paths.

Currently, running antigravity_review.py with uv run fails because
it lacks PEP 723 dependency metadata and passes system_instructions
to Agent instead of LocalAgentConfig. Also, skills_paths points to
the skill directory rather than the SKILL.md file.

To fix, add inline PEP 723 script metadata declaring dependencies,
pass system_instructions to LocalAgentConfig, and target SKILL.md
directly in skills_paths.
@rickeylev rickeylev requested a review from aignas as a code owner July 11, 2026 18:36

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds PEP 723 inline script metadata to antigravity_review.py and refactors the initialization of LocalAgentConfig and Agent by passing system_instructions directly into the configuration and updating the skill path. The review feedback recommends dynamically resolving the path to SKILL.md relative to the script's location using Path(__file__) to prevent execution failures when run from outside the repository root.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread tools/private/reviewbot/antigravity_review.py Outdated
When antigravity_review.py is executed outside the repository root,
hardcoded relative paths fail to locate the SKILL.md file.

To fix, compute skills_paths dynamically relative to Path(__file__).
Because antigravity_review.py now includes PEP 723 script metadata,
uv run automatically manages dependencies and Python versions.

To clean up the workflow, remove the explicit venv setup and pip
install steps from automated_pr_review.yaml.
@rickeylev rickeylev merged commit cf152cd into bazel-contrib:main Jul 11, 2026
4 of 6 checks passed
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.

1 participant