From e3a77e639808f0c11ede6fd2ae9ca1279e5af005 Mon Sep 17 00:00:00 2001 From: Xiaohan-Tian <157918347+Xiaohan-Tian@users.noreply.github.com> Date: Thu, 16 Apr 2026 20:29:10 -0700 Subject: [PATCH] fix: removed unnecessary override confirmation when user save project with project name unchanged --- src/util/saveUtil.ts | 31 ++++++------------------------- 1 file changed, 6 insertions(+), 25 deletions(-) diff --git a/src/util/saveUtil.ts b/src/util/saveUtil.ts index b7075e8..465a318 100644 --- a/src/util/saveUtil.ts +++ b/src/util/saveUtil.ts @@ -1,4 +1,4 @@ -import { KGProjectStorage, DuplicateEntryError } from '../core/io/KGProjectStorage'; +import { KGProjectStorage } from '../core/io/KGProjectStorage'; import { KGCore } from '../core/KGCore'; import { RESERVED_PROJECT_NAME } from './projectNameUtil'; @@ -64,34 +64,15 @@ export const saveProject = async ( } } - // Same name — existing overwrite logic + // Same name — overwrite silently (user hasn't changed the name, so no confirmation needed) try { - await storage.save(projectName, KGCore.instance().getCurrentProject(), false); + await storage.save(projectName, KGCore.instance().getCurrentProject(), true); setStatus(`Project "${projectName}" has been saved`); onSaveSuccess(projectName); return true; } catch (error) { - if (error instanceof DuplicateEntryError) { - const confirmed = window.confirm( - `Project "${projectName}" already exists. Do you want to overwrite it?`, - ); - if (confirmed) { - try { - await storage.save(projectName, KGCore.instance().getCurrentProject(), true); - setStatus(`Project "${projectName}" has been saved`); - onSaveSuccess(projectName); - return true; - } catch (overwriteError) { - console.error('Error overwriting project:', overwriteError); - window.alert(`An error occurred while overwriting the project: ${overwriteError}`); - return false; - } - } - return false; - } else { - console.error('Error saving project:', error); - window.alert(`An unknown error ${error} occurred. Please try again.`); - return false; - } + console.error('Error saving project:', error); + window.alert(`An error occurred while saving: ${error}`); + return false; } };