Fix refresh button not fetching latest data due to double-fire race condition

When the refresh button (pr.refreshList) was clicked, two separate
_onDidChangeTreeData events fired on the same CategoryTreeNode instances:

1. forceClearCache() triggered _onDidChangeData → refreshAllQueryResults(true)
2. The pr.refreshList handler itself also called refreshAllQueryResults(true)

This caused VS Code to call getChildren() twice concurrently on the same
nodes. The second call's super.getChildren(true) would dispose the PR
children that the first call had just populated, resulting in the tree
failing to display the fresh data from GitHub.

Fix: Remove _onDidChangeData.fire() from forceClearCache() to eliminate
the duplicate event source, and update pr.refreshList to call
refreshAllQueryResults() without the reset flag (which is a no-op anyway
since forceClearCache() already empties the cache).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: wankun-tcj

src/view/prsTreeDataProvider.ts

@@ -65,7 +65,7 @@ export class PullRequestsTreeDataProvider extends Disposable implements vscode.T
 		}));
 		this._register(vscode.commands.registerCommand('pr.refreshList', _ => {
 			this.prsTreeModel.forceClearCache();
-			this.refreshAllQueryResults(true);
+			this.refreshAllQueryResults();
 		}));
 
 		this._register(vscode.commands.registerCommand('pr.loadMore', (node: CategoryTreeNode) => {

src/view/prsTreeModel.ts

@@ -181,7 +181,6 @@ export class PrsTreeModel extends Disposable {
 	public forceClearCache() {
 		this._cachedPRs.clear();
 		this._allCachedPRs.clear();
-		this._onDidChangeData.fire();
 	}
 
 	public hasPullRequest(pr: PullRequestModel): boolean {