fix(tauri): correct agent_skills dedup sort and release SESSIONS during pty_write

Sort by (name, kind, source) so dedup_by on (name, kind) sees adjacent
duplicates. Clone the session writer Arc under SESSIONS then perform I/O
under only the per-session writer lock.

Author: cursoragent

native/macos/comux-tauri/src-tauri/src/lib.rs

@@ -190,11 +190,14 @@ fn pty_start(app: AppHandle, options: StartOptions) -> Result<(), String> {
 
 #[tauri::command]
 fn pty_write(thread_id: String, bytes: Vec<u8>) -> Result<(), String> {
-    let guard = SESSIONS.lock();
-    let session = guard
-        .get(&thread_id)
-        .ok_or_else(|| format!("thread '{}' not found", thread_id))?;
-    let mut writer = session.writer.lock();
+    let writer = {
+        let guard = SESSIONS.lock();
+        let session = guard
+            .get(&thread_id)
+            .ok_or_else(|| format!("thread '{}' not found", thread_id))?;
+        Arc::clone(&session.writer)
+    };
+    let mut writer = writer.lock();
     writer.write_all(&bytes).map_err(|e| e.to_string())?;
     writer.flush().map_err(|e| e.to_string())?;
     Ok(())
@@ -503,7 +506,7 @@ fn agent_skills(harness: Option<String>, project_root: Option<String>) -> Vec<Ag
         }
     }
 
-    out.sort_by(|a, b| a.name.cmp(&b.name).then(a.source.cmp(&b.source)));
+    out.sort_by(|a, b| a.name.cmp(&b.name).then(a.kind.cmp(&b.kind)).then(a.source.cmp(&b.source)));
     out.dedup_by(|a, b| a.name == b.name && a.kind == b.kind);
     out
 }