Adds discovered/found repos to local path map (#3633)

* Adds discovered/found repos to local path map

* Move to function, add gating and queue as microtask

* Tightens output typing

* Removes unneeded variable

* Improvements

* Tweaks

---------

Co-authored-by: Eric Amodio <[email protected]>

Author: axosoft-ramint

src/git/gitProviderService.ts

@@ -653,8 +653,11 @@ export class GitProviderService implements Disposable {
 			this.updateContext();
 
 			if (added.length) {
-				// Defer the event trigger enough to let everything unwind
-				queueMicrotask(() => this.fireRepositoriesChanged(added));
+				queueMicrotask(() => {
+					void this.addRepositoriesToPathMap(added);
+					// Defer the event trigger enough to let everything unwind
+					this.fireRepositoriesChanged(added);
+				});
 			}
 		} finally {
 			queueMicrotask(() => {
@@ -2425,8 +2428,11 @@ export class GitProviderService implements Disposable {
 				this.updateContext();
 
 				if (added.length) {
-					// Send a notification that the repositories changed
-					queueMicrotask(() => this.fireRepositoriesChanged(added));
+					queueMicrotask(() => {
+						void this.addRepositoriesToPathMap(added);
+						// Send a notification that the repositories changed
+						this.fireRepositoriesChanged(added);
+					});
 				}
 
 				repository = repositories.length === 1 ? repositories[0] : this.getRepository(uri);
@@ -2440,6 +2446,19 @@ export class GitProviderService implements Disposable {
 		return promise;
 	}
 
+	@gate()
+	@log()
+	async addRepositoriesToPathMap(repos: Repository[]): Promise<void> {
+		const scope = getLogScope();
+		for (const repo of repos) {
+			try {
+				await this.container.repositoryIdentity.addRepositoryToPathMap(repo);
+			} catch (ex) {
+				Logger.error(ex, scope);
+			}
+		}
+	}
+
 	@log()
 	async getOrOpenRepositoryForEditor(editor?: TextEditor): Promise<Repository | undefined> {
 		editor = editor ?? window.activeTextEditor;

src/plus/repos/repositoryIdentityService.ts

@@ -4,9 +4,14 @@ import type { Container } from '../../container';
 import { RemoteResourceType } from '../../git/models/remoteResource';
 import type { Repository } from '../../git/models/repository';
 import { parseGitRemoteUrl } from '../../git/parsers/remoteParser';
-import type { GkProviderId, RepositoryIdentityDescriptor } from '../../gk/models/repositoryIdentities';
+import type {
+	GkProviderId,
+	RepositoryIdentityDescriptor,
+	RepositoryIdentityProviderDescriptor,
+} from '../../gk/models/repositoryIdentities';
 import { missingRepositoryId } from '../../gk/models/repositoryIdentities';
 import { log } from '../../system/decorators/log';
+import { getSettledValue } from '../../system/promise';
 import type { ServerConnection } from '../gk/serverConnection';
 
 export class RepositoryIdentityService implements Disposable {
@@ -25,6 +30,23 @@ export class RepositoryIdentityService implements Disposable {
 		return this.locateRepository(identity, options);
 	}
 
+	async getRepositoryIdentity<T extends string | GkProviderId>(
+		repository: Repository,
+	): Promise<RepositoryIdentityDescriptor<T>> {
+		const [bestRemotePromise, initialCommitShaPromise] = await Promise.allSettled([
+			this.container.git.getBestRemoteWithProvider(repository.uri),
+			this.container.git.getFirstCommitSha(repository.uri),
+		]);
+		const bestRemote = getSettledValue(bestRemotePromise);
+
+		return {
+			name: repository.name,
+			initialCommitSha: getSettledValue(initialCommitShaPromise),
+			remote: bestRemote,
+			provider: bestRemote?.provider?.providerDesc as RepositoryIdentityProviderDescriptor<T>,
+		};
+	}
+
 	@log()
 	private async locateRepository<T extends string | GkProviderId>(
 		identity: RepositoryIdentityDescriptor<T>,
@@ -132,20 +154,29 @@ export class RepositoryIdentityService implements Disposable {
 				(await this.container.git.validateReference(locatedRepo.uri, identity.initialCommitSha))
 			) {
 				foundRepo = locatedRepo;
-				await this.addFoundRepositoryToMap(foundRepo, identity);
+				await this.addRepositoryToPathMap(foundRepo, identity);
 			}
 		}
 
 		return foundRepo;
 	}
 
-	private async addFoundRepositoryToMap<T extends string | GkProviderId>(
+	async addRepositoryToPathMap<T extends string | GkProviderId>(
 		repo: Repository,
 		identity?: RepositoryIdentityDescriptor<T>,
 	) {
+		if (repo.virtual) return;
+
+		const [identityResult, remotesResult] = await Promise.allSettled([
+			identity == null ? this.getRepositoryIdentity<T>(repo) : undefined,
+			repo.git.getRemotes(),
+		]);
+
+		identity ??= getSettledValue(identityResult);
+		const remotes = getSettledValue(remotesResult) ?? [];
+
 		const repoPath = repo.uri.fsPath;
 
-		const remotes = await repo.git.getRemotes();
 		for (const remote of remotes) {
 			const remoteUrl = remote.provider?.url({ type: RemoteResourceType.Repo });
 			if (remoteUrl != null) {