Skip to content

Conversation

jerop
Copy link
Contributor

@jerop jerop commented Aug 13, 2025

This commit improves the gemini issue triage workflow by:

  • Passing the available labels as an environment variable to the Gemini CLI.
  • Using the output of the Gemini CLI to apply labels to the issue.
  • Adding a step to get all repository labels and pass them to the gemini-cli.
  • Updating the prompt to classify the issue and output the labels in JSON format.
  • Adding a step to apply the labels to the issue using the github-script.
image

@jerop jerop requested review from a team as code owners August 13, 2025 03:42
@jerop jerop force-pushed the feat/gemini-issue-triage-improvements branch 5 times, most recently from 0c4a4d2 to c2eb38c Compare August 13, 2025 04:13
@jerop jerop force-pushed the feat/gemini-issue-triage-improvements branch 2 times, most recently from ebb153e to 7c76b68 Compare August 13, 2025 09:19
Copy link
Member

@sethvargo sethvargo left a comment

Choose a reason for hiding this comment

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

Perhaps a silly question, but is there a reason we don't use github.rest.issues.setLabels instead of bifurcating between "add" and "remove"? This only runs when issues are (re)opened, so I'm not sure we need to take on the additional complexity and API calls.

I only commented on the first file, but consider the feedback throughout all files.

@jerop jerop force-pushed the feat/gemini-issue-triage-improvements branch 5 times, most recently from 2df9eb6 to 3916c88 Compare August 14, 2025 02:30
This commit improves the gemini issue triage workflow by:

- Passing the available labels as an environment variable to the Gemini CLI.
- Using the output of the Gemini CLI to apply labels to the issue.
- Adding a step to get all repository labels and pass them to the gemini-cli.
- Updating the prompt to classify the issue and output the labels in JSON format.
- Adding a step to apply the labels to the issue using the github-script.
@jerop jerop force-pushed the feat/gemini-issue-triage-improvements branch from 3916c88 to d5d2051 Compare August 14, 2025 02:45
@jerop
Copy link
Contributor Author

jerop commented Aug 14, 2025

@sethvargo addressed all the feedback, thank you!

is there a reason we don't use github.rest.issues.setLabels instead of bifurcating between "add" and "remove"? This only runs when issues are (re)opened, so I'm not sure we need to take on the additional complexity and API calls.

was originally taking a more conservative approach to add new labels and remove some but leave others untouched, but it seems too complex for little benefit, and the user can always implement this if needed -- so switched to the simpler approach using github.rest.issues.setLabels

Example runs in a private repo I shared with you:

@jerop jerop requested a review from sethvargo August 14, 2025 02:47
@sethvargo sethvargo merged commit b60b57b into main Aug 14, 2025
12 checks passed
@sethvargo sethvargo deleted the feat/gemini-issue-triage-improvements branch August 14, 2025 14:34
google-github-actions-bot added a commit that referenced this pull request Aug 14, 2025
## What's Changed
* Highlight core tools and mcp servers in configuration docs by @jerop
in #160
* feat(/review): support pull_request_review &
pull_request_review_comment events by @leehagoodjames in
#161
* Github-workflow to automate validation of new PRs based on
Contribution Guidelines of the repository by @jasmeetsb in
#162
* fix: Update gemini-issue-automated-triage.yml by @teolemon in
#165
* Add instruction for code review limit. by @aliciatang07 in
#168
* feat(logging): Enable debug logging across all workflows and
gemini-cli by @leehagoodjames in
#178
* feat(workflows): reduce restrictions for invoking workflows for
private repos by @leehagoodjames in
#177
* fix(runner): capture stderr and stdout after command finishes by
@sethvargo in
#183
* chore(logging): only enable Gemini debug when DEBUG is set by
@sethvargo in
#180
* feat: improve gemini issue triage workflow by @jerop in
#171

## New Contributors
* @jasmeetsb made their first contribution in
#162
* @teolemon made their first contribution in
#165

**Full Changelog**:
v0.1.10...v0.1.11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants