feat(ui): refine app rules tab, result card preview, and icon scaling

Author: ganeshmshetty

Sources/Core/Rules/AppRule.swift

@@ -5,6 +5,10 @@
 import Foundation
 
 public struct AppPolicyContext: Sendable {
+    /// Disables OpenClip entirely for the matching application (no popup, hotkeys ignored).
+    public let disabled: Bool
+    /// Suppresses the automatic popup on text selection; OpenClip will only trigger via explicit hotkey.
+    public let hotkeyOnly: Bool
     /// Force a text result to be delivered as a copy instead of a paste, even when the app lists a
     /// Paste command. The explicit escape hatch for apps (e.g. Terminal) that advertise Paste but
     /// cannot reliably replace a selection.
@@ -19,11 +23,15 @@ public struct AppPolicyContext: Sendable {
     public static let `default` = AppPolicyContext()
 
     public init(
+        disabled: Bool = false,
+        hotkeyOnly: Bool = false,
         denyPaste: Bool = false,
         useMenuCopy: Bool = false,
         retrievalMode: SelectionRetrievalMode = .axTextControl,
         gate: SelectionGatePolicy = .default
     ) {
+        self.disabled = disabled
+        self.hotkeyOnly = hotkeyOnly
         self.denyPaste = denyPaste
         self.useMenuCopy = useMenuCopy
         self.retrievalMode = retrievalMode
@@ -34,19 +42,25 @@ public struct AppPolicyContext: Sendable {
 public struct AppRule: Codable, Sendable, Equatable, Identifiable {
     public var id: String { bundleIdentifiers.first ?? UUID().uuidString }
     public let bundleIdentifiers: [String]
+    public let disabled: Bool?
+    public let hotkeyOnly: Bool?
     public let useMenuCopy: Bool?
     public let denyPaste: Bool?
     public let retrievalMode: SelectionRetrievalMode?
     public let gate: SelectionGatePolicy?
 
     public init(
         bundleIdentifiers: [String],
+        disabled: Bool? = nil,
+        hotkeyOnly: Bool? = nil,
         useMenuCopy: Bool? = nil,
         denyPaste: Bool? = nil,
         retrievalMode: SelectionRetrievalMode? = nil,
         gate: SelectionGatePolicy? = nil
     ) {
         self.bundleIdentifiers = bundleIdentifiers
+        self.disabled = disabled
+        self.hotkeyOnly = hotkeyOnly
         self.useMenuCopy = useMenuCopy
         self.denyPaste = denyPaste
         self.retrievalMode = retrievalMode
@@ -55,6 +69,8 @@ public struct AppRule: Codable, Sendable, Equatable, Identifiable {
 
     public enum CodingKeys: String, CodingKey {
         case bundleIdentifiers = "bundle-identifiers"
+        case disabled = "disabled"
+        case hotkeyOnly = "hotkey-only"
         case useMenuCopy = "use-menu-copy"
         case denyPaste = "deny-paste"
         case retrievalMode = "retrieval-mode"

Sources/Core/Rules/RuleEngine.swift

@@ -83,6 +83,8 @@ public final class RuleEngine: ObservableObject, Sendable {
             }
             return AppRule(
                 bundleIdentifiers: expandedIdentifiers,
+                disabled: rule.disabled,
+                hotkeyOnly: rule.hotkeyOnly,
                 useMenuCopy: rule.useMenuCopy,
                 denyPaste: rule.denyPaste,
                 retrievalMode: rule.retrievalMode,
@@ -101,6 +103,8 @@ public final class RuleEngine: ObservableObject, Sendable {
                     explicitRetrievalMode = true
                 }
                 context = AppPolicyContext(
+                    disabled: rule.disabled ?? context.disabled,
+                    hotkeyOnly: rule.hotkeyOnly ?? context.hotkeyOnly,
                     denyPaste: rule.denyPaste ?? context.denyPaste,
                     useMenuCopy: rule.useMenuCopy ?? context.useMenuCopy,
                     retrievalMode: rule.retrievalMode ?? context.retrievalMode,
@@ -114,6 +118,8 @@ public final class RuleEngine: ObservableObject, Sendable {
         // explicitly supplied `retrieval-mode` (even `.axTextControl`) opts out of the conversion.
         if context.useMenuCopy && context.retrievalMode == .axTextControl && !explicitRetrievalMode {
             context = AppPolicyContext(
+                disabled: context.disabled,
+                hotkeyOnly: context.hotkeyOnly,
                 denyPaste: context.denyPaste,
                 useMenuCopy: context.useMenuCopy,
                 retrievalMode: .menuCopy,

Sources/OpenClip/Platform/Effects/ActionResultHandler.swift

@@ -147,15 +147,19 @@ public final class DefaultActionResultHandler: ActionResultHandler, Sendable {
             pendingRestoreTask = nil
             let pasteboard = self.pasteboard
             deliverPaste(to: pasteboard) {
-                pasteboard.setString(text, forType: .string)
+                let item = NSPasteboardItem()
+                item.setString(text, forType: .string)
+                item.setData(Data(), forType: PasteboardSnapshot.transientType)
+                item.setData(Data(), forType: PasteboardSnapshot.autoGeneratedType)
+                pasteboard.writeObjects([item])
             }
 
         case .pasteContent(let payload):
             pendingRestoreTask?.cancel()
             pendingRestoreTask = nil
             let pasteboard = self.pasteboard
             deliverPaste(to: pasteboard) {
-                writePayload(payload, to: pasteboard)
+                writePayload(payload, to: pasteboard, transient: true)
             }
 
         case .openURL(let url):
@@ -233,16 +237,22 @@ public final class DefaultActionResultHandler: ActionResultHandler, Sendable {
         try await center.add(request)
     }
 
-    private func writePayload(_ payload: RichPasteboardPayload, to pasteboard: NSPasteboard) {
+    private func writePayload(_ payload: RichPasteboardPayload, to pasteboard: NSPasteboard, transient: Bool = false) {
+        let item = NSPasteboardItem()
         if let rtf = payload.rtf, let rtfData = rtf.data(using: .utf8) {
-            pasteboard.setData(rtfData, forType: .rtf)
+            item.setData(rtfData, forType: .rtf)
         }
         if let html = payload.html {
-            pasteboard.setString(html, forType: .html)
+            item.setString(html, forType: .html)
         }
         if let text = payload.plainText {
-            pasteboard.setString(text, forType: .string)
+            item.setString(text, forType: .string)
+        }
+        if transient {
+            item.setData(Data(), forType: PasteboardSnapshot.transientType)
+            item.setData(Data(), forType: PasteboardSnapshot.autoGeneratedType)
         }
+        pasteboard.writeObjects([item])
     }
 
     /// Writes `write` onto the pasteboard then synthesizes ⌘V, honoring the per-click copy

Sources/OpenClip/Platform/HotkeyManager.swift

@@ -16,15 +16,15 @@ public final class HotkeyManager {
     public static let shared = HotkeyManager()
     private var lastFallbackClipboard: (changeCount: Int, text: String)?
 
-    /// Trigger gate for the hotkey path (pure, unit-tested). Mirrors every monitor trigger site:
-    /// respect the master enable switch, never target OpenClip itself, and never fire inside
-    /// excluded apps — the default retrieval cascade ends in `.keyboardCopy`, which would inject
-    /// a synthetic ⌘C into apps that must never be touched.
     internal static func triggerAllowed(isAppEnabled: Bool, frontmost: NSRunningApplication?) -> Bool {
         guard isAppEnabled,
               let frontmost,
               let bundleID = frontmost.bundleIdentifier else { return false }
-        return !AppFilter.isExcluded(bundleID: bundleID)
+        if AppFilter.isExcluded(bundleID: bundleID) {
+            return false
+        }
+        let policy = RuleEngine.shared.resolvePolicies(for: bundleID)
+        return !policy.disabled
     }
 
     public func setup(popupController: PopupWindowController) {

Sources/OpenClip/Platform/MacSelectionMonitor.swift

@@ -216,6 +216,9 @@ internal final class MacSelectionMonitor: SelectionMonitoring {
             defer { if !delivered { self.triggeredByHold = false } }
 
             let policy = self.policyResolver(app.bundleIdentifier)
+            if policy.disabled || policy.hotkeyOnly {
+                return
+            }
             let appIdentity = AppIdentity(app)
             let probeTask = self.preparePasteProbe?(app, policy)
 
@@ -314,6 +317,9 @@ internal final class MacSelectionMonitor: SelectionMonitoring {
             }
 
             let policy = self.policyResolver(app.bundleIdentifier)
+            if policy.disabled || policy.hotkeyOnly {
+                return
+            }
 
             // Measure drag distance for click filtering
             var isDragOrMultiClick = clickCount >= 2
@@ -365,6 +371,9 @@ internal final class MacSelectionMonitor: SelectionMonitoring {
             }
 
             let policy = self.policyResolver(app.bundleIdentifier)
+            if policy.disabled || policy.hotkeyOnly {
+                return
+            }
             let appIdentity = AppIdentity(app)
             let probeTask = self.preparePasteProbe?(app, policy)
             let result = await retriever.retrieve(

Sources/OpenClip/UI/Icons/ActionIconView.swift

@@ -150,8 +150,9 @@ public struct ActionIconView: View {
             switch icon {
             case .symbol(let name):
                 if name.contains(":") {
+                    // Iconify SVGs usually have internal padding in their viewBox; scale up so optical weight matches SF Symbols.
                     let isBrand = name.hasPrefix("simple-icons:") || name.hasPrefix("logos:") || name.hasPrefix("cib:") || name.contains("brand")
-                    let opticalFactor: CGFloat = isBrand ? 0.88 : 1.0
+                    let opticalFactor: CGFloat = isBrand ? 1.08 : 1.20
                     let dim = targetDimension * opticalFactor
                     IconifySVGView(iconId: name)
                         .frame(width: dim, height: dim)
@@ -180,7 +181,7 @@ public struct ActionIconView: View {
                         image
                             .resizable()
                             .aspectRatio(contentMode: .fit)
-                            .frame(maxWidth: targetDimension, maxHeight: targetDimension)
+                            .frame(maxWidth: targetDimension * 1.15, maxHeight: targetDimension * 1.15)
                     } else if phase.error != nil {
                         Image(systemName: "exclamationmark.triangle")
                             .font(.system(size: targetDimension * 0.9, weight: .regular))
@@ -197,7 +198,7 @@ public struct ActionIconView: View {
                         .resizable()
                         .renderingMode(.template)
                         .aspectRatio(contentMode: .fit)
-                        .frame(maxWidth: targetDimension, maxHeight: targetDimension)
+                        .frame(maxWidth: targetDimension * 1.18, maxHeight: targetDimension * 1.18)
                 } else {
                     Image(systemName: "questionmark.square")
                         .font(.system(size: targetDimension, weight: .regular))

Sources/OpenClip/UI/Popup/PopupMetrics.swift

@@ -41,9 +41,8 @@ public enum PopupMetrics {
     /// Fraction of an extra result row shown beyond `searchMaxRows` so the next action peeks,
     /// hinting that the list scrolls.
     public static let searchPeekRowFraction: CGFloat = 0.5
-    /// Shared height cap for the popup panel (search palette field + result rows). The code value
-    /// 240 wins over any stale comment.
-    public static let popupMaxHeight: CGFloat = 240
+    /// Shared height cap for the popup panel (search palette field + result rows and content cards).
+    public static let popupMaxHeight: CGFloat = 300
     /// Native AI result card sizing: width clamped to the shared popup column and a max body
     /// height so a long response scrolls instead of growing the panel without bound.
     public static let aiCardMinWidth: CGFloat = 220
@@ -53,7 +52,7 @@ public enum PopupMetrics {
     /// reports a starved ideal height, so the panel would never grow to fit the response (the body
     /// collapsed to nothing while the header/footer rendered). A concrete height gives the card a
     /// deterministic preferred size and keeps the whole card under `popupMaxHeight`.
-    public static let aiCardBodyHeight: CGFloat = 120
+    public static let aiCardBodyHeight: CGFloat = 160
     /// How long an info/error toast stays up before auto-dismissing (1.2 s — long enough to read
     /// "Copied"-style feedback). Loading toasts have no timer — they live until the action's
     /// result lands.