Don't prompt to fetch additional remotes when user explicitly configures them (#8219)

* Initial plan

* Initial analysis of remote fetching issue

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Fix: Don't prompt to fetch additional remotes when explicitly configured

When users explicitly configure githubPullRequests.remotes, the extension
should automatically fetch from all configured remotes instead of prompting
"Continue fetching from other remotes" after finding results in the first one.

This change modifies the fetchPagedData method to detect when remotes have
been explicitly configured (via global, workspace, or folder settings) and
skips the early break logic that was causing the prompt to appear.

Fixes microsoft/vscode-pull-request-github#XXXXX

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Address code review feedback: improve variable names and readability

- Renamed hasExplicitRemotesConfig to hasUserConfiguredRemotes for clarity
- Extracted complex conditional logic into well-named boolean variables
  (hasReceivedData, isFetchingNextPage, hasReachedPreviousFetchLimit, shouldBreakEarly)
  to improve readability and make the logic more self-documenting

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Revert unrelated change to vscode.proposed.chatParticipantAdditions.d.ts

* Fix: Preserve hasMorePages to show "Load more" button

When user configures remotes and we loop through all of them, we should
preserve itemData.hasMorePages so that the "Load more" button appears
if any repository has more pages. Previously, we were returning
hasMorePages: false which prevented the "Load more" button from showing.

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Remove unrelated change to vscode.proposed.chatParticipantAdditions.d.ts

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

Author: Copilot

src/github/folderRepositoryManager.ts

@@ -1086,6 +1086,10 @@ export class FolderRepositoryManager extends Disposable {
 
 		const activeGitHubRemotes = await this.getActiveGitHubRemotes(this._allGitHubRemotes);
 
+		// Check if user has explicitly configured remotes (not using defaults)
+		const remotesConfig = vscode.workspace.getConfiguration(PR_SETTINGS_NAMESPACE).inspect<string[]>(REMOTES);
+		const hasUserConfiguredRemotes = !!(remotesConfig?.globalValue || remotesConfig?.workspaceValue || remotesConfig?.workspaceFolderValue);
+
 		const githubRepositories = this._githubRepositories.filter(repo => {
 			if (!activeGitHubRemotes.find(r => r.equals(repo.remote))) {
 				return false;
@@ -1148,15 +1152,17 @@ export class FolderRepositoryManager extends Disposable {
 
 			pageInformation.hasMorePages = itemData.hasMorePages;
 
-			// Break early if
+			// Determine if we should break early from the loop:
 			// 1) we've received data AND
 			// 2) either we're fetching just the next page (case 2)
 			//    OR we're fetching all (cases 1&3), and we've fetched as far as we had previously (or further, in case 1).
-			if (
-				itemData.items.length &&
-				(options.fetchNextPage ||
-					((options.fetchNextPage === false) && !options.fetchOnePagePerRepo && (pagesFetched >= getTotalFetchedPages())))
-			) {
+			// 3) AND the user hasn't explicitly configured remotes (if they have, we should search all of them)
+			const hasReceivedData = itemData.items.length > 0;
+			const isFetchingNextPage = options.fetchNextPage;
+			const hasReachedPreviousFetchLimit = (options.fetchNextPage === false) && !options.fetchOnePagePerRepo && (pagesFetched >= getTotalFetchedPages());
+			const shouldBreakEarly = hasReceivedData && (isFetchingNextPage || hasReachedPreviousFetchLimit) && !hasUserConfiguredRemotes;
+
+			if (shouldBreakEarly) {
 				if (getTotalFetchedPages() === 0) {
 					// We're in case 1, manually set number of pages we looked through until we found first results.
 					setTotalFetchedPages(pagesFetched);
@@ -1173,7 +1179,7 @@ export class FolderRepositoryManager extends Disposable {
 
 		return {
 			items: itemData.items,
-			hasMorePages: false,
+			hasMorePages: itemData.hasMorePages,
 			hasUnsearchedRepositories: false,
 			totalCount: itemData.totalCount
 		};