-
Notifications
You must be signed in to change notification settings - Fork 2.7k
fix: Prevent stale bot from flagging internal maintainer discussions #3938
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.
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
Conversation
Summary of ChangesHello @ryanaiagent, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a crucial improvement to the stale bot's behavior, ensuring that active internal discussions among maintainers do not lead to issues being prematurely marked as stale. By providing the bot with more context about who is commenting and who is being addressed, it can now intelligently differentiate between a question awaiting an author's response and an ongoing team collaboration, thereby enhancing workflow efficiency and reducing false positives. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Response from ADK Triaging Agent Hello @ryanaiagent, thank you for creating this PR! I noticed that the checklist in your PR description indicates that tests have not been added. To ensure the quality of the codebase, our contribution guidelines require that all code changes include tests. Could you please add tests that prove your fix is effective? You can find more details in the Testing Requirements section of our contribution guide. This will help reviewers to verify your changes more efficiently. Thanks! |
There was a problem hiding this comment.
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 effectively addresses the issue of the stale bot incorrectly flagging issues with active internal maintainer discussions. The changes in agent.py correctly provide the necessary context (maintainers list, last actor) to the LLM. The updated prompt in PROMPT_INSTRUCTION.txt introduces a new, well-placed check for maintainer-to-maintainer communication, which should prevent the erroneous flagging. The overall solution is well-designed. I have one suggestion to improve the clarity of a tool-calling instruction in the prompt to make it more robust.
|
Hi @xuanyang15 , can you review this and get this merged. |
|
Thank you @ryanaiagent for your contribution! 🎉 Your changes have been successfully imported and merged via Copybara in commit f51b9b7. Closing this PR as the changes are now in the main branch. |
Problem
The stale bot was incorrectly marking issues as
stalewhen the last comment was a question from one maintainer to another (e.g.,@maintainerB what do you think?). The bot interpreted this as a question waiting for the author, rather than an active internal discussion.Solution
agent.py: Modifiedget_issue_stateto expose the full list ofmaintainersand thelast_actor_nameto the LLM context.Changes
adk_stale_agent/agent.py: Passmaintainerslist andlast_actor_nameto the LLM.adk_stale_agent/PROMPT_INSTRUCTION.txt: Added instruction to check for maintainer-to-maintainer mentions.Checklist