auto-coder 3.0.2__tar.gz → 3.0.3__tar.gz

{auto_coder-3.0.2/src/auto_coder.egg-info → auto_coder-3.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: auto-coder
-Version: 3.0.2
+Version: 3.0.3
 Summary: AutoCoder: AutoCoder
 Author: allwefantasy
 Author-email: allwefantasy <allwefantasy@gmail.com>

{auto_coder-3.0.2 → auto_coder-3.0.3}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "auto-coder"
-version = "3.0.2"
+version = "3.0.3"
 description = "AutoCoder: AutoCoder"
 authors = [
     {name = "allwefantasy", email = "allwefantasy@gmail.com"}

{auto_coder-3.0.2 → auto_coder-3.0.3/src/auto_coder.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: auto-coder
-Version: 3.0.2
+Version: 3.0.3
 Summary: AutoCoder: AutoCoder
 Author: allwefantasy
 Author-email: allwefantasy <allwefantasy@gmail.com>

{auto_coder-3.0.2 → auto_coder-3.0.3}/src/autocoder/common/git_utils.py

@@ -1,4 +1,6 @@
 import os
+import locale
+import subprocess
 from loguru import logger
 from typing import List, Optional, Dict, Any
 from pydantic import BaseModel
@@ -52,6 +54,58 @@ class CommitResult(BaseModel):
     error_message: Optional[str] = None
 
 
+def _safe_decode_output(data: Any) -> str:
+    """
+    Safely decode subprocess output bytes to string.
+
+    - Prefer UTF-8 when possible (common for git output).
+    - Fallback to system preferred encoding.
+    - Never raise UnicodeDecodeError (use replacement as last resort).
+    """
+    if data is None:
+        return ""
+    if isinstance(data, str):
+        return data
+    if isinstance(data, memoryview):
+        data = data.tobytes()
+    if not isinstance(data, (bytes, bytearray)):
+        return str(data)
+
+    preferred = locale.getpreferredencoding(False) or "utf-8"
+    for enc in ("utf-8", preferred):
+        try:
+            return bytes(data).decode(enc)
+        except UnicodeDecodeError:
+            continue
+    return bytes(data).decode(preferred, errors="replace")
+
+
+def _run_git_capture(repo_path: str, args: List[str]) -> tuple[int, str, str]:
+    """
+    Run `git` with binary capture to avoid Windows locale decode crashes.
+    Returns: (exit_code, stdout_text, stderr_text)
+    """
+    try:
+        completed = subprocess.run(
+            ["git", *args],
+            cwd=repo_path,
+            capture_output=True,
+            text=False,   # critical: avoid implicit decoding (gbk/cp936) in reader threads
+            check=False,
+        )
+        return (
+            completed.returncode,
+            _safe_decode_output(completed.stdout),
+            _safe_decode_output(completed.stderr),
+        )
+    except FileNotFoundError as e:
+        # GitPython requires git executable anyway, but keep this error readable.
+        msg = str(e)
+        return 127, "", msg
+    except Exception as e:
+        return 1, "", str(e)
+
+
 def init(repo_path: str) -> bool:
     if not GIT_AVAILABLE:
         logger.warning("Git is not available. Skipping Git initialization.")
@@ -111,16 +165,45 @@ def commit_changes(repo_path: str, message: str) -> CommitResult:
             )
 
             if commit.parents:
-                changed_files = repo.git.diff(
-                    commit.parents[0].hexsha, commit.hexsha, name_only=True
-                ).split("\n")
-                result.changed_files = [file for file in changed_files if file.strip()]
+                parent_hexsha = commit.parents[0].hexsha
+
+                # Use `git` CLI in binary mode to avoid UnicodeDecodeError on Windows GBK locale.
+                # GitPython's internal decoding can crash in subprocess reader threads when output
+                # contains non-GBK bytes (e.g., UTF-8 diff content).
+                rc, stdout, stderr = _run_git_capture(
+                    repo_path,
+                    [
+                        "-c",
+                        "core.quotepath=false",
+                        "diff",
+                        "--name-only",
+                        parent_hexsha,
+                        commit.hexsha,
+                    ],
+                )
+                if rc != 0 and stderr:
+                    logger.warning(f"git diff --name-only failed: {stderr}")
+
+                changed_files = [line.strip() for line in stdout.splitlines() if line.strip()]
+                result.changed_files = changed_files
 
                 for file in result.changed_files:
-                    diff = repo.git.diff(
-                        commit.parents[0].hexsha, commit.hexsha, "--", file
+                    rc, diff_out, diff_err = _run_git_capture(
+                        repo_path,
+                        [
+                            "-c",
+                            "core.quotepath=false",
+                            "diff",
+                            "--no-color",
+                            parent_hexsha,
+                            commit.hexsha,
+                            "--",
+                            file,
+                        ],
                     )
-                    result.diffs[file] = diff
+                    if rc != 0 and diff_err:
+                        logger.warning(f"git diff for {file} failed: {diff_err}")
+                    result.diffs[file] = diff_out
             else:
                 result.error_message = (
                     "This is the initial commit, no parent to compare against."

{auto_coder-3.0.2 → auto_coder-3.0.3}/src/autocoder/common/tools_manager/utils.py

@@ -5,12 +5,39 @@ Tools Manager Utility Functions
 """
 
 import os
+import locale
 import subprocess
 from pathlib import Path
 from typing import Optional, List
 from loguru import logger
 
 
+def _safe_decode_output(data: object) -> str:
+    """
+    Safely decode subprocess output to string.
+
+    - Prefer UTF-8 when possible.
+    - Fallback to system preferred encoding.
+    - Never raise UnicodeDecodeError.
+    """
+    if data is None:
+        return ""
+    if isinstance(data, str):
+        return data
+    if isinstance(data, memoryview):
+        data = data.tobytes()
+    if not isinstance(data, (bytes, bytearray)):
+        return str(data)
+
+    preferred = locale.getpreferredencoding(False) or "utf-8"
+    for enc in ("utf-8", preferred):
+        try:
+            return bytes(data).decode(enc)
+        except UnicodeDecodeError:
+            continue
+    return bytes(data).decode(preferred, errors="replace")
+
+
 def is_tool_command_file(file_path: str) -> bool:
     """
     检查文件是否是有效的工具命令文件
@@ -93,12 +120,14 @@ def _try_command_line_help(file_path: str) -> Optional[str]:
             result = subprocess.run(
                 cmd, 
                 capture_output=True, 
-                text=True, 
+                text=False,  # Avoid Windows locale decode crashes (gbk/cp936 strict)
                 timeout=5, 
                 cwd=Path(file_path).parent
             )
-            if result.returncode == 0 and (result.stdout.strip() or result.stderr.strip()):
-                return result.stdout.strip() or result.stderr.strip()
+            stdout = _safe_decode_output(result.stdout).strip()
+            stderr = _safe_decode_output(result.stderr).strip()
+            if result.returncode == 0 and (stdout or stderr):
+                return stdout or stderr
         except Exception:
             continue
     return None

{auto_coder-3.0.2 → auto_coder-3.0.3}/src/autocoder/version.py

@@ -1,4 +1,4 @@
 # This file is auto-generated by Hatchling. As such, do not:
 #   - modify
 #   - track in version control e.g. be sure to add to .gitignore
-__version__ = "3.0.2"
+__version__ = "3.0.3"