Skip to content

[WIP] Fix GitHub search tool returning zero matching issues - #8938

Draft
Alex Ross (alexr00) with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-search-tool-issue
Draft

[WIP] Fix GitHub search tool returning zero matching issues#8938
Alex Ross (alexr00) with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-search-tool-issue

Conversation

Copilot AI commented Sep 7, 2026

Copy link
Copy Markdown
Contributor
  • Trace explicit-repository search resolution and paging behavior
  • Add a focused regression test for searching a non-workspace repository
  • Apply the smallest production fix to search the resolved repository
  • Run targeted tests, compile, lint, and hygiene checks
  • Scan changed files and run final code/security validation

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI and others added 4 commits September 7, 2026 16:05
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

@alexr00 Alex Ross (alexr00) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copilot can you give an explanation for the change?

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

Copilot can you give an explanation for the change?

Previously, the tool resolved the explicit owner/repo but then called FolderRepositoryManager.getIssues(), whose paging logic only searches active workspace remotes. A requested non-workspace repository was therefore filtered out and could produce zero results.

The change queries the resolved GitHubRepository directly and makes the structured repository input authoritative. It also normalizes any existing repo: qualifier and appends the explicit qualifier outside the Boolean expression, so every branch of the query—and the confirmation link—is scoped to the requested repository. The regression test verifies this path and confirms the workspace-wide search is not used.

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.

GitHub search tool returns zero matching GitHub Issues despite there being many

2 participants