SCM - use sequencer for tree operations (microsoft#255979)

Author: lszomoru

src/vs/workbench/contrib/scm/browser/scmRepositoriesViewPane.ts

@@ -28,6 +28,7 @@ import { MenuId } from '../../../../platform/actions/common/actions.js';
 import { IHoverService } from '../../../../platform/hover/browser/hover.js';
 import { observableConfigValue } from '../../../../platform/observable/common/platformObservableUtils.js';
 import { autorun, IObservable, IObservableSignal, observableSignal, observableSignalFromEvent, runOnChange } from '../../../../base/common/observable.js';
+import { Sequencer } from '../../../../base/common/async.js';
 
 class ListDelegate implements IListVirtualDelegate<ISCMRepository> {
 
@@ -101,6 +102,7 @@ export class SCMRepositoriesViewPane extends ViewPane {
 	private treeViewModel!: SCMRepositoriesViewModel;
 	private treeDataSource!: RepositoryTreeDataSource;
 	private treeIdentityProvider!: RepositoryTreeIdentityProvider;
+	private readonly treeOperationSequencer = new Sequencer();
 
 	private readonly visibleCountObs: IObservable<number>;
 	private readonly providerCountBadgeObs: IObservable<'hidden' | 'auto' | 'visible'>;
@@ -149,32 +151,32 @@ export class SCMRepositoriesViewPane extends ViewPane {
 			this.treeViewModel = this.instantiationService.createInstance(SCMRepositoriesViewModel);
 			this._register(this.treeViewModel);
 
-			// Initial rendering
-			await this.tree.setInput(this.treeViewModel);
-
-			// scm.repositories.visible setting
-			this.visibilityDisposables.add(autorun(reader => {
-				const visibleCount = this.visibleCountObs.read(reader);
-				this.updateBodySize(visibleCount);
-			}));
-
-			// onDidChangeRepositoriesSignal
-			this.visibilityDisposables.add(autorun(async reader => {
-				this.treeViewModel.onDidChangeRepositoriesSignal.read(reader);
-				await this.updateChildren();
-			}));
-
-			// onDidChangeRepositoryContextValueSignal
-			this.visibilityDisposables.add(autorun(async reader => {
-				this.treeViewModel.onDidChangeRepositoryContextValueSignal.read(reader);
-				await this.updateChildren();
-			}));
-
-			// onDidChangeVisibleRepositoriesSignal
-			this.visibilityDisposables.add(autorun(async reader => {
-				this.treeViewModel.onDidChangeVisibleRepositoriesSignal.read(reader);
-				this.updateTreeSelection();
-			}));
+			this.treeOperationSequencer.queue(async () => {
+				// Initial rendering
+				await this.tree.setInput(this.treeViewModel);
+
+				// scm.repositories.visible setting
+				this.visibilityDisposables.add(autorun(reader => {
+					const visibleCount = this.visibleCountObs.read(reader);
+					this.updateBodySize(visibleCount);
+				}));
+
+				// onDidChangeRepositoriesSignal
+				// onDidChangeRepositoryContextValueSignal
+				this.visibilityDisposables.add(autorun(async reader => {
+					this.treeViewModel.onDidChangeRepositoriesSignal.read(reader);
+					this.treeViewModel.onDidChangeRepositoryContextValueSignal.read(reader);
+
+					await this.treeOperationSequencer.queue(() => this.updateChildren());
+				}));
+
+				// onDidChangeVisibleRepositoriesSignal
+				this.visibilityDisposables.add(autorun(async reader => {
+					this.treeViewModel.onDidChangeVisibleRepositoriesSignal.read(reader);
+
+					await this.treeOperationSequencer.queue(async () => this.updateTreeSelection());
+				}));
+			});
 		}, this, this._store);
 	}