Tactical change for refreshing PRs tree (#7357)

* Tactical change for refreshing PRs tree

* Change poll interval, add event

* small diff

Author: alexr00

src/github/copilotPrWatcher.ts

@@ -117,7 +117,7 @@ export class CopilotPRWatcher extends Disposable {
 		if (shouldContinue) {
 			this._timeout = setTimeout(() => {
 				this._pollForChanges();
-			}, 60 * 1000); // Poll every minute
+			}, 3 * 60 * 1000); // Poll every 3 minute
 		}
 	}
 

src/github/folderRepositoryManager.ts

@@ -1165,6 +1165,12 @@ export class FolderRepositoryManager extends Disposable {
 			Logger.debug(`Fetch pull request category ${categoryQuery} - enter`, this.id);
 			const { octokit, query, schema } = await githubRepository.ensure();
 
+			/* __GDPR__
+				"pr.search.category" : {
+				}
+			*/
+			this.telemetry.sendTelemetryEvent('pr.search.category');
+
 			const user = (await githubRepository.getAuthenticatedUser()).login;
 			// Search api will not try to resolve repo that redirects, so get full name first
 			repo = await githubRepository.getMetadata();
@@ -1202,7 +1208,7 @@ export class FolderRepositoryManager extends Disposable {
 					// Pull requests fetched with a query can be from any repo.
 					// We need to use the correct GitHubRepository for this PR.
 					return response.repo.createOrUpdatePullRequestModel(
-						await parseGraphQLPullRequest(response.data.repository.pullRequest, response.repo),
+						await parseGraphQLPullRequest(response.data.repository.pullRequest, response.repo)
 					);
 				})))
 				.filter(item => item !== null) as PullRequestModel[];

src/github/githubRepository.ts

@@ -1511,7 +1511,7 @@ export class GitHubRepository extends Disposable {
 				}
 				const oldTimeline = issueModel.timelineEvents;
 				issueModel.timelineEvents = allEvents;
-				if (!oldLastEvent || (allEvents.length !== oldTimeline.length)) {
+				if (oldTimeline.length > 0 && allEvents.length !== oldTimeline.length) {
 					this._onDidChangePullRequests.fire();
 				}
 			}
@@ -1632,7 +1632,7 @@ export class GitHubRepository extends Disposable {
 		Logger.debug(`Fetch timeline events of PR #${pullRequestModel.number} - done`, PullRequestModel.ID);
 		const oldEvents = pullRequestModel.timelineEvents;
 		pullRequestModel.timelineEvents = events;
-		if (oldEvents.length !== events.length) {
+		if ((oldEvents.length > 0) && oldEvents.length !== events.length) {
 			this._onDidChangePullRequests.fire();
 		}
 		return events;

src/view/prsTreeModel.ts

@@ -50,17 +50,11 @@ export class PrsTreeModel extends Disposable {
 			}
 
 			this._repoEvents.get(manager)!.push(manager.onDidChangeActivePullRequest(() => {
-				this.clearRepo(manager);
+				this._onDidChangeData.fire(manager);
 				if (this._activePRDisposables.has(manager)) {
 					disposeAll(this._activePRDisposables.get(manager)!);
 					this._activePRDisposables.delete(manager);
 				}
-				if (manager.activePullRequest) {
-					this._activePRDisposables.set(manager, [
-						manager.activePullRequest.onDidChangeComments(() => {
-							this.clearRepo(manager);
-						})]);
-				}
 			}));
 			this._repoEvents.get(manager)!.push(manager.onDidChangeAnyPullRequests(() => {
 				this._onDidChangeData.fire(manager);