fix(read): avoid search focus for chat_id opens

chat_id lookups already come from the chat list registry, so prefer the visible chat-list row before falling back to KakaoTalk global search. This lets read and send avoid the brittle search-field focus path when the target room is already visible in the chat list.

Constraint: KakaoTalk search fields can remain focused in unrelated small windows and fail AX focus verification.

Rejected: Only document FOCUS_FAIL recovery | leaves chat_id reads dependent on the same failing search path.

Confidence: medium

Scope-risk: moderate

Tested: swift build; .build/debug/kmsg read --help

Not-tested: live read --chat-id completion; local debug binary hung in auth/window bootstrap before returning output. Python tests unavailable because pytest is not installed.

Author: MadKangYu

Sources/kmsg/Commands/ReadCommand.swift

@@ -19,11 +19,20 @@ struct ReadCommand: ParsableCommand {
     static let configuration = CommandConfiguration(
         commandName: "read",
         abstract: "Read messages from a chat",
-        discussion: "When author is \"(me)\", the message was sent by you."
+        discussion: """
+            Use either:
+              kmsg read <chat>
+              kmsg read --chat-id <chat-id>
+
+            When author is "(me)", the message was sent by you.
+            """
     )
 
+    @Option(name: .long, help: "Read using a chat_id from 'kmsg chats'")
+    var chatID: String?
+
     @Argument(help: "Name of the chat to read from (partial match supported)")
-    var chat: String
+    var chat: String?
 
     @Option(name: .shortAndLong, help: "Maximum number of messages to show")
     var limit: Int = 20
@@ -49,6 +58,19 @@ struct ReadCommand: ParsableCommand {
     @Flag(name: .long, help: "Output in JSON format")
     var json: Bool = false
 
+    func validate() throws {
+        if let chatID, !chatID.isEmpty {
+            guard chat == nil else {
+                throw ValidationError("Chat name cannot be provided together with --chat-id.")
+            }
+            return
+        }
+
+        guard let chat, !chat.isEmpty else {
+            throw ValidationError("Chat name is required unless --chat-id is provided.")
+        }
+    }
+
     func run() throws {
         guard AccessibilityPermission.ensureGranted() else {
             AccessibilityPermission.printInstructions()
@@ -65,10 +87,18 @@ struct ReadCommand: ParsableCommand {
         let transcriptReader = KakaoTalkTranscriptReader(kakao: kakao, runner: runner)
 
         let resolution: ChatWindowResolution
+        let requestedChat: String
         do {
-            resolution = try chatWindowResolver.resolve(query: chat)
+            if let chatID {
+                requestedChat = chatID
+                resolution = try chatWindowResolver.resolve(chatID: chatID)
+            } else {
+                let chat = chat ?? ""
+                requestedChat = chat
+                resolution = try chatWindowResolver.resolve(query: chat)
+            }
         } catch {
-            print("No chat window found for '\(chat)'")
+            print("No chat window found for '\(requestedChat)'")
             print("Reason: \(error)")
             print("\nAvailable windows:")
             for (index, window) in kakao.windows.enumerated() {
@@ -80,6 +110,11 @@ struct ReadCommand: ParsableCommand {
         let window = resolution.window
         if resolution.openedViaSearch {
             runner.log("read: opening chat via search")
+        } else if resolution.method == .openedViaChatList {
+            runner.log("read: opening chat via chat list")
+        }
+
+        if resolution.openedTransiently {
             if keepWindow {
                 runner.log("read: keep-window enabled; auto-opened window will be kept")
             } else {
@@ -90,16 +125,16 @@ struct ReadCommand: ParsableCommand {
         }
 
         defer {
-            if resolution.openedViaSearch && !keepWindow {
+            if resolution.openedTransiently && !keepWindow {
                 let resolvedTitle = window.title ?? ""
-                if !resolvedTitle.isEmpty && !resolvedTitle.localizedCaseInsensitiveContains(chat) {
+                if chatID == nil && !resolvedTitle.isEmpty && !resolvedTitle.localizedCaseInsensitiveContains(requestedChat) {
                     runner.log("read: skipped auto-close because resolved title '\(resolvedTitle)' did not match query")
                 } else if chatWindowResolver.closeWindow(window) {
                     runner.log("read: auto-opened chat window closed")
                 } else {
                     runner.log("read: failed to close auto-opened chat window")
                 }
-            } else if resolution.openedViaSearch && keepWindow {
+            } else if resolution.openedTransiently && keepWindow {
                 runner.log("read: auto-opened chat window kept by --keep-window")
             }
         }
@@ -108,7 +143,7 @@ struct ReadCommand: ParsableCommand {
         do {
             snapshot = try transcriptReader.readSnapshot(
                 from: window,
-                fallbackChatTitle: chat,
+                fallbackChatTitle: window.title ?? requestedChat,
                 limit: limit
             )
         } catch TranscriptReadError.transcriptContextUnavailable {

Sources/kmsg/Commands/SendCommand.swift

@@ -129,22 +129,18 @@ struct SendCommand: ParsableCommand {
             runner.log("window strategy: focusedWindow -> mainWindow -> windows.first")
             let resolution: ChatWindowResolution
             if let chatID {
-                guard let record = ChatIdentityRegistryStore.shared.record(for: chatID) else {
-                    throw KakaoTalkError.elementNotFound("Unknown chat_id '\(chatID)'. Run 'kmsg chats' first to refresh the local registry.")
-                }
                 print("Looking for chat with \(targetDescription)...")
-                print("Resolved \(targetDescription) to '\(record.displayName)'.")
-                resolution = try chatWindowResolver.resolve(query: record.displayName)
-                if resolution.openedViaSearch {
-                    print("No existing chat window. Opening via search...")
+                resolution = try chatWindowResolver.resolve(chatID: chatID)
+                if resolution.openedTransiently {
+                    print("No existing chat window. Opening via chat list or search...")
                 } else {
                     print("Found existing chat window.")
                 }
             } else {
                 let recipient = recipient ?? ""
                 print("Looking for chat with \(targetDescription)...")
                 resolution = try chatWindowResolver.resolve(query: recipient)
-                if resolution.openedViaSearch {
+                if resolution.openedTransiently {
                     print("No existing chat window. Opening via search...")
                 } else {
                     print("Found existing chat window.")

Sources/kmsg/Commands/WatchCommand.swift

@@ -100,7 +100,7 @@ struct WatchCommand: ParsableCommand {
 
         var currentWindow = resolution.window
         var currentChatTitle = currentWindow.title ?? chat
-        var autoOpenedWindow: UIElement? = resolution.openedViaSearch ? currentWindow : nil
+        var autoOpenedWindow: UIElement? = resolution.openedTransiently ? currentWindow : nil
         var cachedContext: MessageTranscriptContext?
 
         defer {
@@ -224,7 +224,7 @@ struct WatchCommand: ParsableCommand {
         currentWindow = resolution.window
         currentChatTitle = currentWindow.title ?? chat
         cachedContext = nil
-        if resolution.openedViaSearch {
+        if resolution.openedTransiently {
             autoOpenedWindow = currentWindow
         }
         return try stabilizeBaseline(

Sources/kmsg/KakaoTalk/ChatWindowResolver.swift

@@ -3,6 +3,7 @@ import Foundation
 
 enum ChatWindowResolutionMethod {
     case existingWindow
+    case openedViaChatList
     case openedViaSearch
 }
 
@@ -13,6 +14,10 @@ struct ChatWindowResolution {
     var openedViaSearch: Bool {
         method == .openedViaSearch
     }
+
+    var openedTransiently: Bool {
+        method != .existingWindow
+    }
 }
 
 private enum ChatWindowFailureCode: String {
@@ -72,6 +77,30 @@ struct ChatWindowResolver {
         return ChatWindowResolution(window: chatWindow, method: .openedViaSearch)
     }
 
+    func resolve(chatID: String) throws -> ChatWindowResolution {
+        guard let record = ChatIdentityRegistryStore.shared.record(for: chatID) else {
+            throw KakaoTalkError.elementNotFound("Unknown chat_id '\(chatID)'. Run 'kmsg chats' first to refresh the local registry.")
+        }
+
+        let usableWindow = try requireUsableWindow()
+        let query = record.displayName
+
+        if let existingWindow = findMatchingChatWindow(in: kakao.windows, query: query) {
+            return ChatWindowResolution(window: existingWindow, method: .existingWindow)
+        }
+
+        if let chatListWindow = kakao.chatListWindow,
+           let chatWindow = openChatListRow(chatID: chatID, query: query, in: chatListWindow, fallbackWindow: usableWindow)
+        {
+            return ChatWindowResolution(window: chatWindow, method: .openedViaChatList)
+        }
+
+        runner.log("chat_id: falling back to search for '\(query)'")
+        let searchWindow = selectSearchWindow(fallback: usableWindow)
+        let chatWindow = try openChatViaSearch(query: query, in: searchWindow, fallbackWindow: usableWindow)
+        return ChatWindowResolution(window: chatWindow, method: .openedViaSearch)
+    }
+
     @discardableResult
     func closeWindow(_ window: UIElement) -> Bool {
         let closeAction = "AXClose"
@@ -229,6 +258,58 @@ struct ChatWindowResolver {
         throw KakaoTalkError.windowNotFound("[\(ChatWindowFailureCode.windowNotReady.rawValue)] Chat window for '\(query)' did not open")
     }
 
+    private func openChatListRow(chatID: String, query: String, in chatListWindow: UIElement, fallbackWindow: UIElement) -> UIElement? {
+        runner.log("chat_id: scanning chat list rows")
+        let scanner = ChatListScanner()
+        let snapshots = scanner.scan(in: chatListWindow, limit: 200, trace: { message in
+            runner.log(message)
+        })
+        guard !snapshots.isEmpty else {
+            runner.log("chat_id: chat list scan returned no rows")
+            return nil
+        }
+
+        let registry = ChatIdentityRegistryStore.shared
+        let assignedIDs = registry.assignChatIDs(for: snapshots.map(\.discovery))
+        guard let matchIndex = assignedIDs.firstIndex(of: chatID) else {
+            runner.log("chat_id: no visible chat row matched \(chatID)")
+            return nil
+        }
+
+        let row = snapshots[matchIndex].element
+        runner.log("chat_id: matched row title='\(snapshots[matchIndex].discovery.title)'")
+        kakao.activate()
+        _ = tryRaiseWindow(chatListWindow)
+
+        if triggerChatListRowOpen(row) {
+            if let window = waitForOpenedChatWindow(query: query, fallbackWindow: fallbackWindow) {
+                return window
+            }
+        }
+
+        runner.log("chat_id: matched row did not open a chat window")
+        return nil
+    }
+
+    private func triggerChatListRowOpen(_ row: UIElement) -> Bool {
+        if tryActivateSearchResult(row, label: "chat list row") {
+            return true
+        }
+
+        let selected = trySelectSearchResult(row, label: "chat list row")
+        if !selected, let parent = row.parent, trySelectSearchResult(parent, label: "chat list row.parent") {
+            runner.pressEnterKey()
+            return true
+        }
+
+        if selected {
+            runner.pressEnterKey()
+            return true
+        }
+
+        return false
+    }
+
     private func resolveCachedElement(
         slot: AXPathSlot,
         root: UIElement,