From de6500ff0770f7d8be566a08ffc911dc0742e0f4 Mon Sep 17 00:00:00 2001
From: Mattias Granlund <mtsgrd@gmail.com>
Date: Tue, 7 Jan 2025 00:49:22 +0100
Subject: [PATCH] Add button to run hooks ad-hoc

---
 .../src/lib/commit/CommitDialog.svelte        | 33 ++++++++++++++-----
 .../src/lib/vbranches/branchController.ts     | 14 +++-----
 2 files changed, 30 insertions(+), 17 deletions(-)

diff --git a/apps/desktop/src/lib/commit/CommitDialog.svelte b/apps/desktop/src/lib/commit/CommitDialog.svelte
index 0e66202a8f..08f84b0cb1 100644
--- a/apps/desktop/src/lib/commit/CommitDialog.svelte
+++ b/apps/desktop/src/lib/commit/CommitDialog.svelte
@@ -5,6 +5,7 @@
 	import { persistedCommitMessage, projectRunCommitHooks } from '$lib/config/config';
 	import { cloudCommunicationFunctionality } from '$lib/config/uiFeatureFlags';
 	import { SyncedSnapshotService } from '$lib/history/syncedSnapshotService';
+	import { showError } from '$lib/notifications/toasts';
 	import DropDownButton from '$lib/shared/DropDownButton.svelte';
 	import { intersectionObserver } from '$lib/utils/intersectionObserver';
 	import { BranchController } from '$lib/vbranches/branchController';
@@ -35,6 +36,7 @@
 
 	let commitMessageInput = $state<CommitMessageInput>();
 	let isCommitting = $state(false);
+	let isRunningHooks = $state(false);
 	let commitMessageValid = $state(false);
 	let isInViewport = $state(false);
 
@@ -60,7 +62,14 @@
 	}
 
 	async function runHooks() {
-		await branchController.runHooks(projectId, $selectedOwnership.toString());
+		isRunningHooks = true;
+		try {
+			await branchController.runHooks(projectId, $selectedOwnership.toString());
+		} catch (err: unknown) {
+			showError('Failed to run hooks', err);
+		} finally {
+			isRunningHooks = false;
+		}
 	}
 
 	function close() {
@@ -110,6 +119,20 @@
 		{#if $expanded && !isCommitting}
 			<div class="cancel-btn-wrapper" transition:slideFade={{ duration: 200, axis: 'x' }}>
 				<Button style="ghost" outline id="commit-to-branch" onclick={close}>Cancel</Button>
+				{#if $expanded}
+					<Button
+						onclick={() => {
+							runHooks();
+						}}
+						style="pop"
+						loading={isRunningHooks}
+						kind="solid"
+						disabled={isRunningHooks || $selectedOwnership.nothingSelected()}
+						id="run-hooks"
+					>
+						Run hooks
+					</Button>
+				{/if}
 			</div>
 		{/if}
 		{#if $expanded && canShowCommitAndPublish}
@@ -153,13 +176,6 @@
 				{/snippet}
 			</DropDownButton>
 		{:else}
-			<Button
-				onclick={() => {
-					runHooks();
-				}}
-				style="pop"
-				kind="solid">Run hooks</Button
-			>
 			<Button
 				style="pop"
 				kind="solid"
@@ -208,6 +224,7 @@
 	.cancel-btn-wrapper {
 		overflow: hidden;
 		margin-right: 6px;
+		white-space: nowrap;
 	}
 
 	/* MODIFIERS */
diff --git a/apps/desktop/src/lib/vbranches/branchController.ts b/apps/desktop/src/lib/vbranches/branchController.ts
index 2ae4961d00..8deca541f8 100644
--- a/apps/desktop/src/lib/vbranches/branchController.ts
+++ b/apps/desktop/src/lib/vbranches/branchController.ts
@@ -53,15 +53,11 @@ export class BranchController {
 	}
 
 	async runHooks(stackId: string, ownership: string) {
-		try {
-			await invoke<void>('run_hooks', {
-				projectId: this.projectId,
-				stackId,
-				ownership
-			});
-		} catch (err: any) {
-			showError('Failed to run hooks', err);
-		}
+		await invoke<void>('run_hooks', {
+			projectId: this.projectId,
+			stackId,
+			ownership
+		});
 	}
 
 	async commitBranch(