fix: address code review findings

- Use downcast_into to avoid unnecessary retain/release per attribute
- Suppress unused uses_language_correction warning on Windows
- Clarify macOS vs Windows attribute matching difference in AGENTS.md

Author: sh3ll3x3c

AGENTS.md

@@ -69,9 +69,9 @@ Fast-path to get coordinates without image analysis.
     ]
     ```
 *   **Platform behavior:**
-    *   **Both platforms:** Uses the **platform accessibility API** as the primary mechanism — searches the accessibility tree for elements whose name matches the query. This gives precise element-level coordinates (`confidence: 1.0`). Falls back to OCR automatically if accessibility finds no matches.
-    *   **macOS:** Accessibility API (primary), Vision OCR (fallback). Note: accessibility results use semantic names (e.g., "All Clear" instead of "AC", "Subtract" instead of "−"), so search by meaning rather than displayed symbol.
-    *   **Windows:** UI Automation (primary), WinRT OCR (fallback).
+    *   **Both platforms:** Uses the **platform accessibility API** as the primary mechanism — searches the accessibility tree for elements by name. This gives precise element-level coordinates (`confidence: 1.0`). Falls back to OCR automatically if accessibility finds no matches.
+    *   **macOS:** Accessibility API (primary), Vision OCR (fallback). Matches against element title, value, and description. Note: accessibility results use semantic names (e.g., "All Clear" instead of "AC", "Subtract" instead of "−"), so search by meaning rather than displayed symbol.
+    *   **Windows:** UI Automation (primary), WinRT OCR (fallback). Matches against element Name property only.
 
 ### 2. Input & Interaction (The "Hands")
 

src/macos/ax.rs

@@ -179,11 +179,9 @@ unsafe fn get_string_attribute(element: AXUIElementRef, attr_name: &str) -> Opti
         return None;
     }
 
-    // value_ref is owned (create rule) — wrap it so it gets released
-    let cf_value = CFType::wrap_under_create_rule(value_ref);
-
-    // Try to downcast to CFString
-    let cf_string = cf_value.downcast::<CFString>()?;
+    // value_ref is owned (create rule) — wrap it so it gets released.
+    // downcast_into consumes cf_value, avoiding an extra retain/release cycle.
+    let cf_string = CFType::wrap_under_create_rule(value_ref).downcast_into::<CFString>()?;
     Some(cf_string.to_string())
 }
 

src/tools/input.rs

@@ -497,7 +497,10 @@ fn find_text_in_window(
         uses_language_correction,
     )?;
     #[cfg(target_os = "windows")]
-    let mut matches = ocr::ocr_image(&screenshot.png_data, Some(screenshot.scale_factor))?;
+    let mut matches = {
+        let _ = uses_language_correction; // Windows OCR doesn't support this param
+        ocr::ocr_image(&screenshot.png_data, Some(screenshot.scale_factor))?
+    };
 
     // Offset OCR coordinates from image-relative to screen-absolute
     for m in &mut matches {