Moves setAsDefault to remotes sub-provider

Author: eamodio

src/git/gitProvider.ts

@@ -484,6 +484,7 @@ export interface GitRemotesSubProvider {
 	addRemote?(repoPath: string, name: string, url: string, options?: { fetch?: boolean }): Promise<void>;
 	pruneRemote?(repoPath: string, name: string): Promise<void>;
 	removeRemote?(repoPath: string, name: string): Promise<void>;
+	setRemoteAsDefault(repoPath: string, name: string, value?: boolean): Promise<void>;
 }
 
 export interface GitStagingSubProvider {

src/git/models/remote.ts

@@ -86,8 +86,7 @@ export class GitRemote<TProvider extends RemoteProvider | undefined = RemoteProv
 	}
 
 	async setAsDefault(value: boolean = true) {
-		const repository = this.container.git.getRepository(this.repoPath);
-		await repository?.setRemoteAsDefault(this, value);
+		await this.container.git.remotes(this.repoPath).setRemoteAsDefault(this.name, value);
 	}
 }
 

src/git/models/repository.ts

@@ -838,12 +838,6 @@ export class Repository implements Disposable {
 		void this.runTerminalCommand('revert', ...args);
 	}
 
-	async setRemoteAsDefault(remote: GitRemote, value: boolean = true) {
-		await this.container.storage.storeWorkspace('remote:default', value ? remote.name : undefined);
-
-		this.fireChange(RepositoryChange.Remotes, RepositoryChange.RemoteProviders);
-	}
-
 	get starred() {
 		const starred = this.container.storage.getWorkspace('starred:repositories');
 		return starred != null && starred[this.id] === true;

src/git/sub-providers/remotes.ts

@@ -8,6 +8,7 @@ import type { GitCache } from '../cache';
 import type { GitProvider, GitRemotesSubProvider } from '../gitProvider';
 import type { GitRemote } from '../models/remote';
 import { getDefaultRemoteOrHighlander } from '../models/remote';
+import { RepositoryChange } from '../models/repository';
 import type { RemoteProvider } from '../remotes/remoteProvider';
 
 export abstract class RemotesGitProviderBase implements GitRemotesSubProvider {
@@ -181,4 +182,13 @@ export abstract class RemotesGitProviderBase implements GitRemotesSubProvider {
 
 		return undefined;
 	}
+
+	@log()
+	async setRemoteAsDefault(repoPath: string, name: string, value: boolean = true) {
+		await this.container.storage.storeWorkspace('remote:default', value ? name : undefined);
+		this.container.events.fire('git:repo:change', {
+			repoPath: repoPath,
+			changes: [RepositoryChange.Remotes, RepositoryChange.RemoteProviders],
+		});
+	}
 }