From dc58cc9d5c5c0c0c27def30d67c770020e9d86c3 Mon Sep 17 00:00:00 2001 From: Haileyesus Date: Mon, 23 Feb 2026 10:19:16 +0300 Subject: [PATCH] refactor(git-panel): centralize commit flow in handleCommit --- .../git-panel/view/changes/CommitComposer.tsx | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/src/components/git-panel/view/changes/CommitComposer.tsx b/src/components/git-panel/view/changes/CommitComposer.tsx index d31acc0..41d0d68 100644 --- a/src/components/git-panel/view/changes/CommitComposer.tsx +++ b/src/components/git-panel/view/changes/CommitComposer.tsx @@ -25,14 +25,15 @@ export default function CommitComposer({ const [isGeneratingMessage, setIsGeneratingMessage] = useState(false); const [isCollapsed, setIsCollapsed] = useState(isMobile); - const handleCommit = async () => { - if (!commitMessage.trim() || selectedFileCount === 0) { + const handleCommit = async (message = commitMessage) => { + const trimmedMessage = message.trim(); + if (!trimmedMessage || selectedFileCount === 0 || isCommitting) { return false; } setIsCommitting(true); try { - const success = await onCommit(commitMessage); + const success = await onCommit(trimmedMessage); if (success) { setCommitMessage(''); } @@ -68,15 +69,7 @@ export default function CommitComposer({ type: 'commit', message: `Commit ${selectedFileCount} file${selectedFileCount !== 1 ? 's' : ''} with message: "${trimmedMessage}"?`, onConfirm: async () => { - setIsCommitting(true); - try { - const success = await onCommit(commitMessage); - if (success) { - setCommitMessage(''); - } - } finally { - setIsCommitting(false); - } + await handleCommit(trimmedMessage); }, }); };