Fixes a regression, we could trigger a link open while dragging/copying

Implemented the fix.

  - Sources/SwiftTerm/Mac/MacTerminalView.swift:2672 now opens a link only when didSelectionDrag is false.
  - Tests/SwiftTermTests/MouseTrackingTests.swift:331 now expects a selection drag to open no link. It also confirms that the
    drag state resets.

Author: migueldeicaza

Sources/SwiftTerm/Mac/MacTerminalView.swift

@@ -2669,7 +2669,8 @@ open class TerminalView: NSView, NSTextInputClient, NSUserInterfaceValidations,
         lastSelectionDragPoint = nil
         let hit = calculateMouseHit(with: event).grid
         updateHoverLink(at: hit, commandOverride: commandActive || event.modifierFlags.contains(.command))
-        if let result = linkForClick(at: hit, hasCommandModifier: event.modifierFlags.contains(.command)) {
+        if !didSelectionDrag,
+           let result = linkForClick(at: hit, hasCommandModifier: event.modifierFlags.contains(.command)) {
             terminalDelegate?.requestOpenLink(source: self, link: result.link, params: result.params)
             return
         }

Tests/SwiftTermTests/MouseTrackingTests.swift

@@ -328,7 +328,7 @@ struct MouseTrackingTests {
         }
     }
 
-    @Test @MainActor func linkClickClearsStaleSelectionDragState() async {
+    @Test @MainActor func selectionDragDoesNotOpenLinkAndClearsDragState() async {
         let view = TerminalView(frame: CGRect(x: 0, y: 0, width: 320, height: 160))
         view.semanticClickCoalescingDelay = 0.01
         let window = NSWindow(
@@ -364,7 +364,7 @@ struct MouseTrackingTests {
         view.didSelectionDrag = true
         view.mouseUp(with: linkUp)
 
-        #expect(delegate.openedLinks == ["https://example.com"])
+        #expect(delegate.openedLinks.isEmpty)
         #expect(!view.didSelectionDrag)
 
         let promptPoint = CGPoint(