From eb3d45873f8d7ab5c05c4d50ca9b17ef67e561e5 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Fri, 14 Aug 2026 01:58:08 +0000
Subject: [PATCH 1/2] feat(ui): add window.confirm to table deletion

---
 .jules/palette.md                             |  3 ++
 .../components/modals/EditTableModal.test.tsx | 48 +++++++++++++++++++
 .../src/components/modals/EditTableModal.tsx  |  5 +-
 .../components/modals/ModalCoverage.test.tsx  |  1 +
 4 files changed, 56 insertions(+), 1 deletion(-)

diff --git a/.jules/palette.md b/.jules/palette.md
index bd0f73248..6ea1bb420 100644
--- a/.jules/palette.md
+++ b/.jules/palette.md
@@ -57,3 +57,6 @@
 ## 2026-07-30 - Add window.confirm for destructive actions
 **Learning:** Destructive actions like deleting groups and edge relationships previously occurred immediately without user confirmation.
 **Action:** Always wrap delete operations with window.confirm() dialogs and ensure corresponding tests successfully mock window.confirm.
+## 2026-06-20 - Confirm destructive actions like table deletion
+**Learning:** Destructive actions such as deleting a table require explicit user confirmation to prevent accidental data loss.
+**Action:** Always wrap delete operations with window.confirm() dialogs and ensure corresponding tests successfully mock window.confirm.
diff --git a/frontend/src/components/modals/EditTableModal.test.tsx b/frontend/src/components/modals/EditTableModal.test.tsx
index 1a8c9af5e..c492797b6 100644
--- a/frontend/src/components/modals/EditTableModal.test.tsx
+++ b/frontend/src/components/modals/EditTableModal.test.tsx
@@ -46,6 +46,54 @@ describe('EditTableModal', () => {
     expect(screen.getByRole('checkbox', { name: 'test_col NN 설정' })).toBeInTheDocument();
   });
 
+  it('deletes a table when 테이블 삭제 is clicked and confirmed', async () => {
+    const onDeleteTableMock = vi.fn();
+    const editingNode = {
+      id: 'table-1',
+      type: 'table',
+      position: { x: 0, y: 0 },
+      data: {
+        title: 'test_table',
+        comment: '',
+        columns: []
+      }
+    };
+
+    vi.spyOn(window, 'confirm').mockReturnValue(true);
+
+    render(<EditTableModal {...defaultProps} editingNode={editingNode as any} onDeleteTable={onDeleteTableMock} />);
+
+    const user = userEvent.setup();
+    await user.click(screen.getByRole('button', { name: '테이블 삭제' }));
+
+    expect(window.confirm).toHaveBeenCalledWith("'test_table' 테이블을 삭제하시겠습니까?");
+    expect(onDeleteTableMock).toHaveBeenCalled();
+  });
+
+  it('does not delete a table when 테이블 삭제 is clicked and canceled', async () => {
+    const onDeleteTableMock = vi.fn();
+    const editingNode = {
+      id: 'table-1',
+      type: 'table',
+      position: { x: 0, y: 0 },
+      data: {
+        title: 'test_table',
+        comment: '',
+        columns: []
+      }
+    };
+
+    vi.spyOn(window, 'confirm').mockReturnValue(false);
+
+    render(<EditTableModal {...defaultProps} editingNode={editingNode as any} onDeleteTable={onDeleteTableMock} />);
+
+    const user = userEvent.setup();
+    await user.click(screen.getByRole('button', { name: '테이블 삭제' }));
+
+    expect(window.confirm).toHaveBeenCalled();
+    expect(onDeleteTableMock).not.toHaveBeenCalled();
+  });
+
   it('returns null if not open or no editingNode', () => {
     const { container } = render(<EditTableModal {...defaultProps} isOpen={false} editingNode={null} />);
     expect(container.firstChild).toBeNull();
diff --git a/frontend/src/components/modals/EditTableModal.tsx b/frontend/src/components/modals/EditTableModal.tsx
index 998929d44..cfb61d1b1 100644
--- a/frontend/src/components/modals/EditTableModal.tsx
+++ b/frontend/src/components/modals/EditTableModal.tsx
@@ -189,7 +189,10 @@ export function EditTableModal({
           <div className="row" style={{ gap: 8 }}>
             <button
               type="button"
-              onClick={onDeleteTable}
+              onClick={() => {
+                if (!window.confirm(`'${editingNode.data.title}' 테이블을 삭제하시겠습니까?`)) return;
+                onDeleteTable();
+              }}
               style={{ color: "#b91c1c", borderColor: "#fca5a5" }}
             >
               테이블 삭제
diff --git a/frontend/src/components/modals/ModalCoverage.test.tsx b/frontend/src/components/modals/ModalCoverage.test.tsx
index 8732f7efa..8974d6089 100644
--- a/frontend/src/components/modals/ModalCoverage.test.tsx
+++ b/frontend/src/components/modals/ModalCoverage.test.tsx
@@ -183,6 +183,7 @@ describe('modal behavior coverage', () => {
     expect(deleteEditing(tableNode)?.data.columns).toHaveLength(1)
 
     fireEvent.submit(document.getElementById('editTableForm')!)
+    vi.spyOn(window, 'confirm').mockReturnValueOnce(true)
     fireEvent.click(screen.getByRole('button', { name: '테이블 삭제' }))
     fireEvent.click(screen.getByRole('button', { name: '복제' }))
     const duplicate = setNodes.mock.calls[2]?.[0] as (nodes: Node<TableNodeData>[]) => Node<TableNodeData>[]

From 50c4da9f4b08947bf421c815fe466f5680998b9f Mon Sep 17 00:00:00 2001
From: Seongho Bae <me@seonghobae.me>
Date: Fri, 14 Aug 2026 12:44:20 +0900
Subject: [PATCH 2/2] docs(ui): record table deletion safeguard

---
 .jules/palette.md | 2 +-
 CHANGELOG.md      | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.jules/palette.md b/.jules/palette.md
index 6ea1bb420..ab0630a44 100644
--- a/.jules/palette.md
+++ b/.jules/palette.md
@@ -57,6 +57,6 @@
 ## 2026-07-30 - Add window.confirm for destructive actions
 **Learning:** Destructive actions like deleting groups and edge relationships previously occurred immediately without user confirmation.
 **Action:** Always wrap delete operations with window.confirm() dialogs and ensure corresponding tests successfully mock window.confirm.
-## 2026-06-20 - Confirm destructive actions like table deletion
+## 2026-08-14 - Confirm destructive actions like table deletion
 **Learning:** Destructive actions such as deleting a table require explicit user confirmation to prevent accidental data loss.
 **Action:** Always wrap delete operations with window.confirm() dialogs and ensure corresponding tests successfully mock window.confirm.
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3c999061c..0f907baa5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,6 +1,7 @@
 # Changelog
 
 ## Unreleased
+- [FE] 🛡️ **테이블 삭제 확인**: 편집 모달에서 테이블 이름을 포함한 확인 대화상자를 표시하고, 사용자가 명시적으로 확인한 경우에만 기존 삭제 경로를 실행합니다. 취소하면 테이블을 그대로 보존합니다.
 - [FE] ⚡ **검색 노드 참조 안정화 및 순차 스냅샷 폴링**: 같은 정규화 검색어와 원본 테이블 데이터에는 장식된 `node.data` 참조를 재사용하여 드래그 중 불필요한 하위 렌더링과 할당을 줄입니다. 스냅샷 폴링은 이전 요청이 끝난 뒤에만 다음 요청을 예약하며, 선택 변경·언마운트 후 도착한 오래된 성공 또는 실패 응답을 무시합니다.
 - [BE] 🔒 **공유 export 전 경로 redaction**: 공개 share의 SQL / index-design / reversing-spec export에서 코멘트·`example_value`를 제거합니다. 단위 테스트로 누출을 차단합니다.
 - [BE] 🛠️ **함수 인덱스 중복 오탐 수정**: `lower(email)` 등 expression index를 평문 컬럼 인덱스의 중복으로 잘못 판단하지 않도록 괄호 파서를 강화했습니다.