sycommon-python-lib 0.2.7a21__tar.gz → 0.2.7a22__tar.gz

{sycommon_python_lib-0.2.7a21 → sycommon_python_lib-0.2.7a22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sycommon-python-lib
-Version: 0.2.7a21
+Version: 0.2.7a22
 Summary: Add your description here
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{sycommon_python_lib-0.2.7a21 → sycommon_python_lib-0.2.7a22}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sycommon-python-lib"
-version = "0.2.7a21"
+version = "0.2.7a22"
 description = "Add your description here"
 readme = "README.md"
 requires-python = ">=3.11"

{sycommon_python_lib-0.2.7a21 → sycommon_python_lib-0.2.7a22}/src/sycommon/agent/middleware/sandbox_path_guard.py

@@ -61,7 +61,13 @@ _PATH_TOOLS = {
 
 # 宿主沙箱根前缀(用于 execute 命令串扫描;shell 里无法安全改写,只能拦)。
 # 命中即视为「引用了宿主沙箱真实路径」,一律拦。
-_HOST_SANDBOX_RE = re.compile(re.escape(_WORKSPACE_ROOT) + r'(?:/|\\)')
+# 注意：要拦到「根目录本身」和「根下子路径」两种形式。原来要求尾斜杠
+# (_WORKSPACE_ROOT/)，会漏过 `find /data/sycommon_sandbox -name x`（后跟空格，
+# 没有尾斜杠）这种「拿沙箱根当 find 起点」的越权扫描。改为允许「尾斜杠」或
+# 「词边界(后跟空白/行尾/引号)」，根目录本身与任意子路径都能命中。
+_HOST_SANDBOX_RE = re.compile(
+    re.escape(_WORKSPACE_ROOT) + r'(?:[/\\]|(?=\s|$|["\']))'
+)
 
 _DENIED_MSG = (
     "Error: 不允许使用宿主机绝对路径访问沙箱文件。"

{sycommon_python_lib-0.2.7a21 → sycommon_python_lib-0.2.7a22}/src/sycommon/middleware/sandbox.py

@@ -136,7 +136,6 @@ def _build_init_script(workspace: str) -> str:
         safe_workspace = workspace.replace("'", "''")
         return f'''
 $env:SANDBOX_ROOT = '{safe_workspace}'
-$env:SANDBOX_OUTPUT = Join-Path '{safe_workspace}' 'output'
 Set-Location '{safe_workspace}'
 '''
 
@@ -144,9 +143,6 @@ Set-Location '{safe_workspace}'
     return f'''
 # 沙箱环境初始化
 export SANDBOX_ROOT={safe_workspace}
-# 产物文件统一输出目录：技能脚本写产物时优先用 $SANDBOX_OUTPUT（等价 $SANDBOX_ROOT/output），
-# 避免落技能目录等不同步 MinIO 的位置。脚本回退默认值用相对 "output"（本地开发无此变量时）。
-export SANDBOX_OUTPUT="$SANDBOX_ROOT/output"
 
 # 资源限制（防止失控进程）
 ulimit -f 1048576 2>/dev/null     # 单文件最大 1GB
@@ -355,6 +351,46 @@ def _kill_process_tree(process):
             pass
 
 
+def _translate_virtual_prefixes(command: str) -> str:
+    """把 execute 命令里的沙箱虚拟路径前缀翻译成 $SANDBOX_ROOT 前缀。
+
+    根因：read_file/ls 等文件工具认虚拟路径（/skills/...、/output/...，由服务端
+    resolve_sandbox_path 映射）；但 execute 里的 shell（cat/ls/cp/python 等）把
+    /skills/ 当真·文件系统绝对路径，沙箱根 / 下没有 /skills → "No such file"。
+    agent 因此误判「execute 受限」改走 read_file。
+
+    这里在命令交给 shell 前，把"作为路径参数出现"的虚拟前缀替换成 $SANDBOX_ROOT。
+    关键是**只替换 token 边界**（前导是空格/引号/行首/重定向符/`=`），不替换
+    字符串中间出现的，避免误伤（如命令里恰好有形如片段的普通字符串）。
+
+    仅替换这些已知虚拟前缀；其余路径（用户工作目录下的相对路径、真绝对路径）不动。
+    对已是 $SANDBOX_ROOT/... 形式的不重复处理（无 /skills/ token）。
+    """
+    if not command or "$SANDBOX_ROOT" in command:
+        # 用户已显式用 $SANDBOX_ROOT 时，尊重其写法，不再二次翻译
+        # （避免把命令里独立出现的 /skills/ 等误改，多数情况一次替换就够）
+        return command
+
+    import re
+    # 虚拟前缀 → 沙箱子目录（与 resolve_sandbox_path 的映射保持一致）
+    # 顺序无关紧要，前缀互不互为子串（skills/ vs memory/ vs output/ ...）。
+    prefixes = [
+        "skills", "memory", "output", "projects", "data",
+        "conversation_history", "large_tool_results", "tmp",
+    ]
+
+    # token 边界：前导是 空白/引号/重定向(|<>)/行首/=，后接虚拟前缀名再接 /
+    # 用负向零宽断言匹配边界，避免命中 "my-skills/"、"some/output/" 这种子串。
+    out = command
+    for pfx in prefixes:
+        pattern = re.compile(
+            r'(?:(?<=\s)|(?<=\")|(?<=\')|(?<=^)|(?<==)|(?<=\|)|(?<=<)|(?<=>))'
+            + re.escape("/" + pfx + "/")
+        )
+        out = pattern.sub("$SANDBOX_ROOT/" + pfx + "/", out)
+    return out
+
+
 def _build_execute_command(init_script: str, user_command: str) -> tuple[str, dict]:
     """构建跨平台的执行命令。
 
@@ -462,8 +498,15 @@ def setup_sandbox_handler(app: FastAPI, config: dict = None):
         # 使用统一初始化脚本
         init_script = _build_init_script(workspace)
 
+        # 把命令里的虚拟路径前缀(/skills/、/output/...)翻译成 $SANDBOX_ROOT，
+        # 让 cat/ls/cp/python 等 shell 命令能访问到技能/产物文件（否则 No such file）。
+        translated_command = _translate_virtual_prefixes(req.command)
+        if translated_command != req.command:
+            SYLogger.info(
+                f"[Sandbox Server] 虚拟前缀翻译: {req.command!r} -> {translated_command!r}")
+
         # 构建跨平台执行命令
-        full_command, _ = _build_execute_command(init_script, req.command)
+        full_command, _ = _build_execute_command(init_script, translated_command)
 
         # 设置环境变量
         env = os.environ.copy()
@@ -471,7 +514,6 @@ def setup_sandbox_handler(app: FastAPI, config: dict = None):
         env["TEMP"] = os.path.join(workspace, "tmp")
         env["TMP"] = os.path.join(workspace, "tmp")
         env["SANDBOX_ROOT"] = workspace
-        env["SANDBOX_OUTPUT"] = os.path.join(workspace, "output")
 
         try:
             # 使用 asyncio.create_subprocess_shell 异步执行命令，避免阻塞事件循环
@@ -593,6 +635,27 @@ def setup_sandbox_handler(app: FastAPI, config: dict = None):
         SYLogger.info(f"[Sandbox Server] 目录内容: {len(result)} 项")
         return result
 
+    def _looks_like_text(raw_prefix: bytes) -> bool:
+        """粗判一段字节是否为文本（用于 /read 决定按文本还是二进制返回）。
+
+        对齐 deepagents BaseSandbox.read 的检测协议（backends/sandbox.py:315-325）：
+        用增量 UTF-8 解码器，8192 字节前缀若正好截断在多字节字符（中文 3 字节、
+        emoji 4 字节）中间，增量解码器会缓存尾部半截序列、不报错；只有真遇到
+        非法字节序列才抛 UnicodeDecodeError。
+
+        旧实现 raw_prefix.decode('utf-8')（strict）会在「中文被 8192 腰斩」时
+        必抛 UnicodeDecodeError，把正常的中文文本文件误判成「二进制」，导致
+        /read 返回 base64 乱码、agent 只能退而用 grep 兜底。
+        """
+        if not raw_prefix:
+            return True
+        import codecs
+        try:
+            codecs.getincrementaldecoder('utf-8')().decode(raw_prefix, final=False)
+            return True
+        except UnicodeDecodeError:
+            return False
+
     @sandbox_router.post("/read", response_model=ReadResponse)
     async def read_file(req: ReadRequest):
         """读取文件内容"""
@@ -633,11 +696,10 @@ def setup_sandbox_handler(app: FastAPI, config: dict = None):
             with open(file_path, 'rb') as f:
                 raw_prefix = f.read(8192)
 
-            try:
-                raw_prefix.decode('utf-8')
-                is_text = True
-            except UnicodeDecodeError:
-                is_text = False
+            # 文本/二进制判定：旧版整段 strict decode 会在「中文被 8192 截断在
+            # 多字节边界」时误判为二进制。改用 _looks_like_text（errors='ignore' +
+            # 控制字符占比），既修截断 bug 又能挡住真二进制。
+            is_text = _looks_like_text(raw_prefix)
 
             if is_text:
                 lines = []

{sycommon_python_lib-0.2.7a21 → sycommon_python_lib-0.2.7a22}/src/sycommon_python_lib.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sycommon-python-lib
-Version: 0.2.7a21
+Version: 0.2.7a22
 Summary: Add your description here
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown