Branch listings: Display the author information + refresh the listing when needed

Display the gravatar image in the branch listing if present.
Refresh the branch listing on every commit done to or undone from it.

Author: estib-vega

apps/desktop/src/lib/branches/branchListing.ts

@@ -63,6 +63,18 @@ export class BranchListingService {
 		return store;
 	}
 
+	/**
+	 * Refresh the information for a particular branch.
+	 *
+	 * Will only fetch the information if the branch is already being tracked.
+	 */
+	refreshBranchListingDetails(branchName: string) {
+		if (!this.branchListingDetails.has(branchName)) {
+			return;
+		}
+		this.updateBranchListing(branchName);
+	}
+
 	private accumulatedBranchListings: string[] = [];
 	private updateBranchListingTimeout: ReturnType<typeof setTimeout> | undefined;
 	// Accumulates multiple update calls
@@ -341,6 +353,8 @@ export class Author {
 	name?: string | undefined;
 	/** The email of the author as configured in the git config */
 	email?: string | undefined;
+	/** The gravatar id of the author */
+	gravatarUrl?: string | undefined;
 }
 
 /** Represents a fat struct with all the data associated with a branch */

apps/desktop/src/lib/commit/CommitCard.svelte

@@ -104,7 +104,7 @@
 			console.error('Unable to undo commit');
 			return;
 		}
-		branchController.undoCommit(branch.id, commit.id);
+		branchController.undoCommit(branch.id, branch.name, commit.id);
 	}
 
 	let isUndoable = commit instanceof DetailedCommit && type !== 'remote';

apps/desktop/src/lib/commit/CommitDialog.svelte

@@ -33,6 +33,7 @@
 		try {
 			await branchController.commitBranch(
 				$branch.id,
+				$branch.name,
 				message.trim(),
 				$selectedOwnership.toString(),
 				$runCommitHooks

apps/desktop/src/lib/commit/StackingCommitCard.svelte

@@ -104,7 +104,7 @@
 			console.error('Unable to undo commit');
 			return;
 		}
-		branchController.undoCommit(branch.id, commit.id);
+		branchController.undoCommit(branch.id, branch.name, commit.id);
 	}
 
 	let isUndoable = commit instanceof DetailedCommit && type !== 'remote';

apps/desktop/src/lib/navigation/BranchListingSidebarEntry.svelte

@@ -100,7 +100,7 @@
 				branchListingDetails.authors.map(async (author) => {
 					return {
 						name: author.name || unknownName,
-						srcUrl: await gravatarUrlFromEmail(author.email || unknownEmail)
+						srcUrl: author.gravatarUrl ?? (await gravatarUrlFromEmail(author.email || unknownEmail))
 					};
 				})
 			);

apps/desktop/src/lib/vbranches/branchController.ts

@@ -3,6 +3,7 @@ import { showError, showToast } from '$lib/notifications/toasts';
 import * as toasts from '$lib/utils/toasts';
 import posthog from 'posthog-js';
 import type { BaseBranchService } from '$lib/baseBranch/baseBranchService';
+import type { BranchListingService } from '$lib/branches/branchListing';
 import type { RemoteBranchService } from '$lib/stores/remoteBranches';
 import type { BranchPushResult, ForgeIdentifier, Hunk, LocalFile, StackOrder } from './types';
 import type { VirtualBranchService } from './virtualBranch';
@@ -14,7 +15,8 @@ export class BranchController {
 		readonly projectId: string,
 		readonly vbranchService: VirtualBranchService,
 		readonly remoteBranchService: RemoteBranchService,
-		readonly baseBranchService: BaseBranchService
+		readonly baseBranchService: BaseBranchService,
+		private readonly branchListingService: BranchListingService
 	) {}
 
 	async setTarget(branch: string, pushRemote: string | undefined = undefined) {
@@ -51,19 +53,21 @@ export class BranchController {
 	}
 
 	async commitBranch(
-		branch: string,
+		branchId: string,
+		branchName: string,
 		message: string,
 		ownership: string | undefined = undefined,
 		runHooks = false
 	) {
 		try {
 			await invoke<void>('commit_virtual_branch', {
 				projectId: this.projectId,
-				branch,
+				branch: branchId,
 				message,
 				ownership,
 				runHooks: runHooks
 			});
+			this.branchListingService.refreshBranchListingDetails(branchName);
 			posthog.capture('Commit Successful');
 		} catch (err: any) {
 			if (err.code === 'errors.commit.signing_failed') {
@@ -543,13 +547,14 @@ export class BranchController {
 		}
 	}
 
-	async undoCommit(branchId: string, commitOid: string) {
+	async undoCommit(branchId: string, branchName: string, commitOid: string) {
 		try {
 			await invoke<void>('undo_commit', {
 				projectId: this.projectId,
 				branchId,
 				commitOid
 			});
+			this.branchListingService.refreshBranchListingDetails(branchName);
 		} catch (err: any) {
 			showError('Failed to amend commit', err);
 		}

apps/desktop/src/routes/[projectId]/+layout.ts

@@ -79,7 +79,8 @@ export const load: LayoutLoad = async ({ params, parent }) => {
 		projectId,
 		vbranchService,
 		remoteBranchService,
-		baseBranchService
+		baseBranchService,
+		branchListingService
 	);
 
 	const branchDragActionsFactory = new BranchDragActionsFactory(branchController);