fix: address all 6 PR #4 review issues

- Remove duplicate "remote" key in target JSON output
- Check output_status for error in handle_exec
- Fix remote type annotation from Optional[int] to Optional[str]
- Add missing mi-async on for target mode non-stop
- Handle queue.Empty timeout explicitly with 60s timeout
- Add remote address format validation (host:port)

Constraint: Must validate remote format before passing to GDB
Confidence: high
Scope-risk: narrow

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: Cerdore

src/gdb_cli/cli.py

@@ -14,8 +14,8 @@
 
 import json
 import os
+import re
 import time
-from pathlib import Path
 from typing import Optional
 
 import click
@@ -204,6 +204,10 @@ def target(
     gdb_path: str,
 ) -> None:
     """Connect to remote GDB server"""
+    if not re.match(r'^[\w.-]+:\d+$', remote):
+        print_error(f"Invalid remote format: {remote}")
+        raise click.exceptions.Exit(1)
+
     existing = find_session_by_remote(remote)
     if existing:
         print_json({
@@ -234,7 +238,6 @@ def target(
             "mode": session.mode,
             "remote": session.remote,
             "binary": session.binary,
-            "remote": session.remote,
             "sock_path": session.sock_path,
             "gdb_pid": gdb_process.pid,
             "safety_level": session.safety_level,

src/gdb_cli/gdb_server/handlers.py

@@ -590,7 +590,14 @@ def run_command():
         # where it will run, and this thread waits to receive the output
         # from the queue.
         gdb.post_event(run_command)
-        output_status, output = result_queue.get(timeout=30.0)
+
+        try:
+            output_status, output = result_queue.get(timeout=60.0)
+        except queue.Empty:
+            return {"command": command, "error": "Command timeout after 60s"}
+
+        if output_status == "error":
+            return {"command": command, "error": output}
 
         return {
             "command": command,

src/gdb_cli/launcher.py

@@ -265,6 +265,7 @@ def launch_target(
     if non_stop:
         gdb_commands.append("set non-stop on")
         gdb_commands.append("set target-async on")
+        gdb_commands.append("set mi-async on")
 
     # 加载 binary (可选)
     if binary:

src/gdb_cli/session.py

@@ -50,7 +50,7 @@ def create_session(
     binary: Optional[str] = None,
     core: Optional[str] = None,
     pid: Optional[int] = None,
-    remote: Optional[int] = None,
+    remote: Optional[str] = None,
     timeout: int = 600,
     safety_level: str = "readonly"
 ) -> SessionMeta: