-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
feat(autopilot): Improve issue creation #106353
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
feat(autopilot): Improve issue creation #106353
Conversation
| # Hide certain categories from the default issue stream | ||
| group_categories.discard(GroupCategory.FEEDBACK.value) | ||
| group_categories.discard(GroupCategory.INSTRUMENTATION.value) |
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.
When registering the issue category, I defined in_default_search = False though it seems like this parameter is not used respected.
Here I am mirroring what is being done for feedback issues.
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.
@wedamija maybe you know if I missed something here?
| description=f"The following SDK integrations are available for your project but not configured: {integrations_list}. " | ||
| f"Adding these integrations can improve error tracking and provide better insights into your application's behavior. " | ||
| title=f"Missing SDK Integration: {integration}", | ||
| # TODO: Generate subtitle and description using AI |
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.
Is this TODO still relevant? Do you want to generate this one fly? I'm not sure why we don't just "hardcode" this
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.
Yes, I would love explain why this was identified as missing and what features it enables.
E.g. You are using the anthropic ai package on the next-js edge runtime which requires manual steps to activate the SDK integration. By enabling it you get ....
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.
Ah, gotcha! Makes sense!
shellmayr
left a comment
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.
LGTM 👍
Create one issue for each missing integration.
Add evidence displays to display additional data.
Exclude instrumentation issues from default search.