fix(datagrid): crash adding a row while a cell overlay is open (#1378)

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- Fixed a crash when adding a row while a cell editor or value viewer was open (#1378)
+
 ## [0.43.2] - 2026-05-22
 
 ### Changed

TablePro/Views/Results/KeyHandlingTableView.swift

@@ -3,30 +3,29 @@ import AppKit
 final class KeyHandlingTableView: NSTableView {
     weak var coordinator: TableViewCoordinator?
 
-    private var isRaisingOverlayEditor = false
-    private var isRaisingOverlayViewer = false
+    private var isRaisingOverlay = false
 
     override var acceptsFirstResponder: Bool {
         true
     }
 
     override func didAddSubview(_ subview: NSView) {
         super.didAddSubview(subview)
-        raiseOverlayIfNeeded(coordinator?.overlayEditor, subview: subview, flag: &isRaisingOverlayEditor)
-        raiseOverlayIfNeeded(coordinator?.overlayViewer, subview: subview, flag: &isRaisingOverlayViewer)
+        guard !isRaisingOverlay else { return }
+        isRaisingOverlay = true
+        defer { isRaisingOverlay = false }
+        raiseOverlayIfNeeded(coordinator?.overlayEditor, subview: subview)
+        raiseOverlayIfNeeded(coordinator?.overlayViewer, subview: subview)
     }
 
-    private func raiseOverlayIfNeeded(_ overlay: CellOverlayBase?, subview: NSView, flag: inout Bool) {
-        guard !flag else { return }
+    private func raiseOverlayIfNeeded(_ overlay: CellOverlayBase?, subview: NSView) {
         guard let overlay,
               overlay.isActive,
               let container = overlay.containerView,
               container !== subview,
               container.superview === self,
               subviews.last !== container else { return }
-        flag = true
         overlay.raiseToFront()
-        flag = false
     }
 
     var selection = TableSelection() {

TableProTests/Views/Results/KeyHandlingTableViewOverlayTests.swift

@@ -0,0 +1,51 @@
+//
+//  KeyHandlingTableViewOverlayTests.swift
+//  TableProTests
+//
+
+import AppKit
+import Foundation
+import SwiftUI
+import TableProPluginKit
+import Testing
+
+@testable import TablePro
+
+@MainActor
+private final class StubColumnLayoutPersister: ColumnLayoutPersisting {
+    func load(for tableName: String, connectionId: UUID) -> ColumnLayoutState? { nil }
+    func save(_ layout: ColumnLayoutState, for tableName: String, connectionId: UUID) {}
+    func clear(for tableName: String, connectionId: UUID) {}
+}
+
+@Suite("KeyHandlingTableView overlay raise")
+@MainActor
+struct KeyHandlingTableViewOverlayTests {
+    private func makeCoordinator() -> TableViewCoordinator {
+        TableViewCoordinator(
+            changeManager: AnyChangeManager(DataChangeManager()),
+            isEditable: true,
+            selectedRowIndices: .constant([]),
+            delegate: nil,
+            layoutPersister: StubColumnLayoutPersister()
+        )
+    }
+
+    @Test("adding a subview while an overlay is active raises it to front without trapping")
+    func addingSubviewRaisesActiveOverlay() {
+        let tableView = KeyHandlingTableView()
+        let coordinator = makeCoordinator()
+        tableView.coordinator = coordinator
+
+        let editor = CellOverlayEditor()
+        coordinator.overlayEditor = editor
+        let container = CellOverlayContainerView(frame: NSRect(x: 0, y: 0, width: 80, height: 24))
+        editor.install(in: tableView, row: 0, column: 0, columnIndex: 0, container: container)
+
+        tableView.addSubview(NSView(frame: NSRect(x: 0, y: 0, width: 10, height: 10)))
+
+        #expect(tableView.subviews.last === container)
+
+        editor.removeOverlay()
+    }
+}