Add binary architecture compatibility check and better error handling

llm_clients/qwen_translator.py

@@ -1,6 +1,7 @@
 from typing import Generator, Any, Dict
 import os
 import sys
+import platform
 import subprocess
 import tempfile
 import zipfile
@@ -59,7 +60,19 @@ class QwenTranslatorClient(LlmClient):
             )
         
         if cls._binary_path and os.path.exists(cls._binary_path):
-            return cls._binary_path
+            # Verify it's still executable
+            if os.access(cls._binary_path, os.X_OK):
+                return cls._binary_path
+            else:
+                # Try to fix permissions
+                try:
+                    os.chmod(cls._binary_path, 0o755)
+                    if os.access(cls._binary_path, os.X_OK):
+                        return cls._binary_path
+                except Exception:
+                    pass
+                # If we can't fix it, re-download
+                cls._binary_path = None
         
         print("📥 Downloading pre-built llama.cpp binary...")
         
@@ -157,7 +170,7 @@ class QwenTranslatorClient(LlmClient):
             try:
                 os.chmod(found_binary, 0o755)
             except Exception as chmod_error:
-                print(f"   ⚠️  Warning: Could not set executable permissions: {chmod_error}")
+                print(f"   ⚠️  Warning: Could not set executable permissions: {chmod_error}", flush=True)
             
             # Move to expected location if needed (use 'main' as standard name)
             if found_binary != binary_path:
@@ -166,15 +179,68 @@ class QwenTranslatorClient(LlmClient):
                 shutil.move(str(found_binary), str(binary_path))
             
             cls._binary_path = str(binary_path)
-            print(f"   ✅ Binary extracted and ready at: {cls._binary_path}")
+            print(f"   ✅ Binary extracted and ready at: {cls._binary_path}", flush=True)
             
-            # Verify binary is executable
+            # Verify binary is executable and test it
             if not os.access(cls._binary_path, os.X_OK):
-                print(f"   ⚠️  Warning: Binary may not be executable. Attempting to fix...")
+                print(f"   ⚠️  Warning: Binary may not be executable. Attempting to fix...", flush=True)
                 try:
                     os.chmod(cls._binary_path, 0o755)
-                except Exception:
-                    pass
+                except Exception as e:
+                    print(f"   ⚠️  Could not set permissions: {e}", flush=True)
+            
+            # Test if binary can actually run (check architecture compatibility)
+            print(f"   🔍 Testing binary compatibility...", flush=True)
+            machine = platform.machine()
+            print(f"   System architecture: {machine}", flush=True)
+            
+            try:
+                # Try to run the binary with --help to verify it works
+                test_result = subprocess.run(
+                    [cls._binary_path, "--help"],
+                    capture_output=True,
+                    text=True,
+                    timeout=5
+                )
+                if test_result.returncode == 0 or "usage" in test_result.stdout.lower() or "options" in test_result.stdout.lower():
+                    print(f"   ✅ Binary is compatible and executable", flush=True)
+                else:
+                    print(f"   ⚠️  Binary test returned code {test_result.returncode}", flush=True)
+                    if test_result.stderr:
+                        print(f"   Stderr: {test_result.stderr[:200]}", flush=True)
+            except subprocess.TimeoutExpired:
+                print(f"   ⚠️  Binary test timed out", flush=True)
+            except OSError as os_error:
+                error_msg = str(os_error)
+                errno = getattr(os_error, 'errno', None)
+                if errno == 8 or "Exec format error" in error_msg or "cannot execute" in error_msg.lower():
+                    # Check what the binary actually is using 'file' command if available
+                    file_info = "unknown"
+                    try:
+                        file_result = subprocess.run(
+                            ["file", cls._binary_path],
+                            capture_output=True,
+                            text=True,
+                            timeout=2
+                        )
+                        if file_result.returncode == 0:
+                            file_info = file_result.stdout.strip()
+                    except (subprocess.TimeoutExpired, FileNotFoundError, Exception):
+                        pass
+                    
+                    raise RuntimeError(
+                        f"Binary architecture mismatch. The downloaded binary is not compatible with this system.\n"
+                        f"System architecture: {machine}\n"
+                        f"Binary info: {file_info}\n"
+                        f"Error: {error_msg}\n"
+                        f"The Ubuntu x64 binary may not be compatible with this system. "
+                        f"Translation feature requires a compatible llama.cpp binary for this architecture."
+                    ) from os_error
+                else:
+                    raise
+            except Exception as test_error:
+                error_msg = str(test_error)
+                print(f"   ⚠️  Binary test warning: {test_error}", flush=True)
             
             # Clean up zip file
             try: