Skip to content

Conversation

@m-y-mo
Copy link
Contributor

@m-y-mo m-y-mo commented Dec 11, 2025

The taskflow assumes that there is a method that fetches all the component issues for a repo, which doesn't exist. This PR adds that method.

@m-y-mo m-y-mo requested a review from p- as a code owner December 11, 2025 16:26
Copilot AI review requested due to automatic review settings December 11, 2025 16:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug in get_app_issues and adds a missing method to fetch all component issues for a repository. The original implementation incorrectly filtered by component_id without filtering by repo, making it impossible to fetch all issues for a repository.

Key changes:

  • Fixed get_app_issues to properly filter by repo and make component_id optional
  • Added get_component_issues_for_repo tool to expose repo-level issue querying
  • Added issue_id to the response dictionary for better issue tracking

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 11, 2025 16:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 11, 2025 17:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@m-y-mo m-y-mo merged commit 1a2eda2 into main Dec 12, 2025
15 checks passed
@m-y-mo m-y-mo deleted the fix_audit_iter branch December 12, 2025 08:59
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.

3 participants