Skip to content
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

repo-gardening: Update AI prompt for labelling #39558

Draft
wants to merge 2 commits into
base: trunk
Choose a base branch
from
Draft
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -66,11 +66,21 @@ ${ repoLabels
.join( '\n' ) }

Analyze the issue and suggest relevant labels. Rules:
# Updated prompt: Ensures 'Product', 'Platform' and 'Type' issues are always added.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the AI supposed to ignore this line because it looks like a comment?

- Use only existing labels provided.
- Include 1 '[Product]' label.
Copy link
Contributor

@anomiex anomiex Sep 27, 2024

Choose a reason for hiding this comment

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

This seems like it will confuse the AI, since no "Product" labels are included in the list above and the AI was instructed to only use labels from the list above.

- Include 1 '[Feature Group]' label.
- Include 1 to 3 '[Feature]' labels.
- Include the "[Platform] Simple" AND/OR "[Platform] Atomic" labels as appropriate.
- Only if NOT present: Add 1 of the following labels based on your assessment of the issue type: '[Type] Bug', '[Type] Enhancement', or '[Type] Feature'.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, neither "Platform" nor "Type" are included in the list above.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, I wonder whether the AI will have any clue about what either of the platforms are and when they might apply.

- Briefly explain each label choice in 1 sentence.
- Format your response as a JSON object, with each suggested label as a key, and your explanation of the label choice as the value.
- Format your response as a JSON object, with 'labels' and 'explanations' keys.- Use only existing labels provided.
Copy link
Contributor

Choose a reason for hiding this comment

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

This changed instruction does not match the example response format below.


# Original prompt
#- Include 1 '[Feature Group]' label.
#- Include 1 to 3 '[Feature]' labels.
#- Briefly explain each label choice in 1 sentence.
#- Format your response as a JSON object, with each suggested label as a key, and your explanation of the label choice as the value.
Copy link
Contributor

Choose a reason for hiding this comment

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

More comments the AI is supposed to figure out to ignore?


Example response format:
{
Expand Down
Loading