Clean up review nits

Author: iSapozhnik

Sources/TextDiff/AppKit/NSTextDiffView.swift

@@ -369,11 +369,6 @@ public final class NSTextDiffView: NSView {
         scheduleHoverDismiss()
     }
 
-    private func clearHoverState() {
-        cancelPendingHoverDismiss()
-        clearHoverStateNow()
-    }
-
     private func clearHoverStateNow() {
         guard hoveredActionID != nil || hoveredIconRect != nil || isPointingHandCursorActive else {
             return
@@ -517,24 +512,6 @@ public final class NSTextDiffView: NSView {
         return false
     }
 
-    private func actionID(at point: CGPoint, layout: DiffLayout, context: DiffRevertInteractionContext) -> Int? {
-        for actionID in context.runIndicesByActionID.keys.sorted() {
-            guard let runIndices = context.runIndicesByActionID[actionID] else {
-                continue
-            }
-            for runIndex in runIndices {
-                guard layout.runs.indices.contains(runIndex),
-                      let chipRect = layout.runs[runIndex].chipRect else {
-                    continue
-                }
-                if chipRect.contains(point) {
-                    return actionID
-                }
-            }
-        }
-        return nil
-    }
-
     private func iconRect(for actionID: Int, context: DiffRevertInteractionContext) -> CGRect? {
         guard let unionRect = context.unionChipRectByActionID[actionID] else {
             return nil

Sources/TextDiff/TextDiffView.swift

@@ -134,7 +134,7 @@ public struct TextDiffView: View {
             )
         }
         Divider()
-        Text("Diff by words and revertable")
+        Text("Diff by words and revertible")
             .bold()
             TextDiffView(
                 original: "Add a diff view! Looks good!",

Tests/TextDiffTests/DiffLayouterPerformanceTests.swift

@@ -31,15 +31,16 @@ final class DiffLayouterPerformanceTests: XCTestCase {
         let updated = Self.replacingLastWord(in: original)
         let segments = TextDiffEngine.diff(original: original, updated: updated, mode: .character)
 
+        var lastLayout: DiffLayout?
         measure(metrics: [XCTClockMetric()]) {
-            let layout = DiffTokenLayouter.layout(
+            lastLayout = DiffTokenLayouter.layout(
                 segments: segments,
                 style: style,
                 availableWidth: availableWidth,
                 contentInsets: contentInsets
             )
-            XCTAssertFalse(layout.runs.isEmpty)
         }
+        XCTAssertFalse(lastLayout?.runs.isEmpty ?? true)
     }
 
     private func runLayoutWithRevertInteractionsPerformanceTest(wordCount: Int) {
@@ -52,6 +53,8 @@ final class DiffLayouterPerformanceTests: XCTestCase {
         let updated = Self.replacingLastWord(in: original)
         let segments = TextDiffEngine.diff(original: original, updated: updated, mode: .token)
 
+        var lastLayout: DiffLayout?
+        var lastContext: DiffRevertInteractionContext?
         measure(metrics: [XCTClockMetric()]) {
             let layout = DiffTokenLayouter.layout(
                 segments: segments,
@@ -66,9 +69,11 @@ final class DiffLayouterPerformanceTests: XCTestCase {
                 original: original,
                 updated: updated
             )
-            XCTAssertFalse(layout.runs.isEmpty)
-            XCTAssertNotNil(context)
+            lastLayout = layout
+            lastContext = context
         }
+        XCTAssertFalse(lastLayout?.runs.isEmpty ?? true)
+        XCTAssertNotNil(lastContext)
     }
 
     private static func largeText(wordCount: Int) -> String {

Tests/TextDiffTests/NSTextDiffSnapshotTests.swift

@@ -135,7 +135,7 @@ final class NSTextDiffSnapshotTests: XCTestCase {
             size: CGSize(width: 320, height: 110),
             configureView: { view in
                 view.isRevertActionsEnabled = true
-                _ = view._testingSetHoveredFirstRevertAction()
+                XCTAssertFalse(view._testingSetHoveredFirstRevertAction())
             },
             testName: "character_mode_no_affordance()"
         )

Tests/TextDiffTests/NSTextDiffViewTests.swift

@@ -272,6 +272,7 @@ func hoverLeaveSchedulesDismissNotImmediate() {
     let centers = view._testingActionCenters()
     #expect(centers.count == 1)
     guard let center = centers.first else {
+        Issue.record("Expected at least one action center")
         return
     }