sycommon-python-lib 0.2.7a29__py3-none-any.whl → 0.2.7a30__py3-none-any.whl

sycommon/agent/deep_agent.py

@@ -35,6 +35,7 @@
 """
 
 import asyncio
+import hashlib
 import json
 import os
 import re
@@ -1366,6 +1367,79 @@ def _parse_skill_version(content: str) -> str:
     return ""
 
 
+def _decode_remote_skillmd(content: str) -> str:
+    """把沙箱 /read 返回的 SKILL.md 内容统一解成明文。
+
+    沙箱 /read 对大文件（如 skill-creator 的 34KB SKILL.md）返回 base64，
+    且 encoding 标志位可能漏标/误标。判断策略（不依赖 encoding 字段）：
+    - 明文 SKILL.md 一定含 frontmatter 标志 `---`；含则当明文直接返回。
+    - 不含 `---` 则当作 base64，解码后返回；解码失败或解码后仍无 `---` 返回原值。
+    """
+    if not content:
+        return content
+    if "---" in content:
+        return content  # 明文
+    try:
+        import base64 as _b64
+        decoded = _b64.b64decode(content).decode("utf-8", errors="replace")
+        return decoded
+    except Exception:
+        return content
+
+
+def _skill_signature(content: str) -> str:
+    """计算 SKILL.md 正文内容的指纹，用于检测「版本号一致但正文未同步」的脏数据。
+
+    取正文（去掉 frontmatter）的前 N 行规范化后的拼接 + md5，作为轻量指纹。
+    版本号相同时若指纹不同，说明沙箱里的正文是旧的，需强制重传。
+    frontmatter 跳过逻辑与 _parse_skill_version 一致（逐行数 ---）。
+    """
+    lines = content.split("\n")
+    in_frontmatter = False
+    body_start = 0
+    for i, line in enumerate(lines):
+        if line.strip() == "---":
+            if not in_frontmatter:
+                in_frontmatter = True  # 进入 frontmatter
+            else:
+                body_start = i + 1  # 离开 frontmatter，正文从这里开始
+                break
+    body = "\n".join(lines[body_start:])
+    # 规范化：去掉空白行和首尾空格，只取前 40 行做指纹（足够区分新旧大改）
+    norm_lines = [l.rstrip() for l in body.split("\n") if l.strip()]
+    head = "\n".join(norm_lines[:40])
+    return f"{len(norm_lines)}:{hashlib.md5(head.encode('utf-8')).hexdigest()[:12]}"
+
+
+async def _verify_skill_content_fresh(
+    backend, sandbox_dest: str, skill_name: str, local_src: str
+) -> bool:
+    """版本一致时，再校验沙箱 SKILL.md 正文指纹是否与本地一致。
+    一致返回 True（可跳过同步）；不一致返回 False（需强制重传）。
+    任何读取异常返回 False（保守起见触发重传，宁可多传不可漏传）。"""
+    local_md = os.path.join(local_src, "SKILL.md")
+    if not os.path.isfile(local_md):
+        return True  # 本地没 SKILL.md，无法校验，交给版本逻辑
+    try:
+        with open(local_md, "r", encoding="utf-8") as f:
+            local_sig = _skill_signature(f.read())
+    except Exception:
+        return True
+
+    try:
+        remote = await backend.aread(
+            f"{sandbox_dest}/{skill_name}/SKILL.md", _skip_lazy_sync=True)
+        if not remote or not remote.file_data:
+            return False  # 读不到 → 当作不一致，触发重传
+        raw = remote.file_data.get("content", "") or ""
+        # 统一用 _decode_remote_skillmd 处理明文/base64 两种返回
+        content = _decode_remote_skillmd(raw)
+        remote_sig = _skill_signature(content)
+        return bool(remote_sig) and local_sig == remote_sig
+    except Exception:
+        return False
+
+
 async def _sync_skills_to_sandbox(
     skills_dir: str,
     backend: HTTPSandboxBackend,
@@ -1424,21 +1498,11 @@ async def _sync_skills_to_sandbox(
                 if remote_content and remote_content.file_data:
                     # FileData 是 TypedDict(dict), 用键访问而非属性
                     content = remote_content.file_data.get("content", "") or ""
-                    # 优先直接解 version(明文小文件最快);
-                    # 解不出再兜底解码 base64——沙箱 /read 对大文件(如 skill-creator 的
-                    # 34KB SKILL.md)返回 base64, 但 encoding 标志位可能漏标/误标,
-                    # 若依赖该标志位,漏标时会因找不到 "version:" 行误判为"无版本"
-                    # → 每次新建 Agent 都全量重传。改为「先解 → 解不出再 b64 兜底」,
-                    # 不依赖 encoding 字段, 对明文/base64 两种返回都正确。
-                    version = _parse_skill_version(content)
-                    if version:
-                        return version
-                    try:
-                        import base64 as _b64
-                        decoded = _b64.b64decode(content).decode("utf-8", errors="replace")
-                        return _parse_skill_version(decoded)
-                    except Exception:
-                        return ""
+                    # 沙箱 /read 对大文件(如 skill-creator 的 34KB SKILL.md)返回 base64,
+                    # 且 encoding 标志位可能漏标/误标。统一用 _decode_remote_skillmd
+                    # 处理明文/base64 两种返回（不依赖 encoding 字段），再解 version。
+                    plain = _decode_remote_skillmd(content)
+                    return _parse_skill_version(plain)
             except Exception:
                 pass
             return ""
@@ -1456,25 +1520,45 @@ async def _sync_skills_to_sandbox(
             remote_version = remote_version_map.get(skill_name, "")
 
             if remote_version and remote_version == local_version:
-                SYLogger.debug(
-                    f"[DeepAgent] 技能版本一致,跳过: {skill_name} v{local_version}")
-                backend._synced_skills.add(skill_name)
-                skip_count += 1
-                continue
+                # 版本一致仍校验正文关键内容，防止"version 字段已更新但正文未同步"
+                # （历史上出现过沙箱 SKILL.md 版本号新、正文旧的脏数据，导致 agent
+                # 永远读不到新版指令）。校验失败则强制走删除+重传。
+                if not await _verify_skill_content_fresh(
+                    backend, sandbox_dest, skill_name, src
+                ):
+                    SYLogger.warning(
+                        f"[DeepAgent] 技能版本一致但正文不一致，强制重传: {skill_name} "
+                        f"v{local_version}")
+                    # 落到下方的删除+重传分支
+                else:
+                    SYLogger.debug(
+                        f"[DeepAgent] 技能版本一致,跳过: {skill_name} v{local_version}")
+                    backend._synced_skills.add(skill_name)
+                    skip_count += 1
+                    continue
 
-            # 版本不一致或无版本,删除旧目录(走 /delete,跳过懒同步,毫秒级)
+            # 版本不一致或无版本（或正文校验失败），删除旧目录(走 /delete,跳过懒同步,毫秒级)
             if skill_name in existing_dirs:
-                try:
-                    await backend.adelete(
-                        f"{sandbox_dest}/{skill_name}",
-                        recursive=True,
-                        _skip_lazy_sync=True)
+                deleted_ok = False
+                for _attempt in range(2):  # 最多重试一次，确保旧内容真删掉
+                    try:
+                        await backend.adelete(
+                            f"{sandbox_dest}/{skill_name}",
+                            recursive=True,
+                            _skip_lazy_sync=True)
+                        deleted_ok = True
+                        break
+                    except Exception as e:
+                        SYLogger.warning(
+                            f"[DeepAgent] 删除旧版技能失败(尝试{_attempt+1}): "
+                            f"{skill_name}, {e}")
+                if deleted_ok:
                     SYLogger.info(
-                        f"[DeepAgent] 技能版本不一致,覆盖: {skill_name} "
+                        f"[DeepAgent] 技能版本不一致,已删除旧目录: {skill_name} "
                         f"({remote_version or '无版本'} → {local_version})")
-                except Exception as e:
-                    SYLogger.warning(
-                        f"[DeepAgent] 删除旧版技能失败: {skill_name}, {e}")
+                else:
+                    SYLogger.error(
+                        f"[DeepAgent] 删除旧版技能彻底失败，本次跳过: {skill_name}")
 
             # 全量上传整个技能目录到沙箱(batch_upload 服务端无条件覆盖)
             await backend.async_sync_dirs([(src, f"{sandbox_dest}/{skill_name}")])

{sycommon_python_lib-0.2.7a29.dist-info → sycommon_python_lib-0.2.7a30.dist-info}/METADATA

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

{sycommon_python_lib-0.2.7a29.dist-info → sycommon_python_lib-0.2.7a30.dist-info}/RECORD

@@ -136,7 +136,7 @@ sycommon/services.py,sha256=iC73h5d5bpNqW2qjRUGODZFhUhcwvKVnKUFLKJlBSq0,25322
 sycommon/agent/__init__.py,sha256=mxceAeUifQ-DKvWp7ZEJIFlmOCb5wpYHPGQw3rwEN8I,4378
 sycommon/agent/agent_manager.py,sha256=UhhaekEumT7g4v_Z1UB4jTp13X0n8M8erYaQdkGGWkA,13620
 sycommon/agent/chat_events.py,sha256=t7qWa6OrIWLfqtd1AnqaVP67QWkn1JxpCBTZYQpoLuM,14226
-sycommon/agent/deep_agent.py,sha256=XdKpBC5ik-GHU2PyZ4KOeSF3EB2-A-r0SmAV3XJN7lI,74799
+sycommon/agent/deep_agent.py,sha256=uLqhiBLcNQlmaYcg0L6dreJGKHDJ4XFU7GWjuVioIkY,78708
 sycommon/agent/multi_agent_team.py,sha256=RFXHB9q5j1_2B-MDZVRTcbXHdEpRIXpAhrm1yroUDYI,30460
 sycommon/agent/summarization_utils.py,sha256=bqZRFqSekNGlv2mYQPaT_s2QNA5VDTKddi2Va_SkHWc,18376
 sycommon/agent/acp/__init__.py,sha256=vQGMQYAA5-ZaejYmDZ4r4Fklqs17qH3cDXQ5pZlwj-0,1844
@@ -308,8 +308,8 @@ sycommon/tools/syemail.py,sha256=BDFhgf7WDOQeTcjxJEQdu0dQhnHFPO_p3eI0-Ni3LhQ,561
 sycommon/tools/timing.py,sha256=OiiE7P07lRoMzX9kzb8sZU9cDb0zNnqIlY5pWqHcnkY,2064
 sycommon/xxljob/__init__.py,sha256=7eoBlQxv-B39IfRSCY2bkqdGYs1QRe1umAWd88VMEEM,86
 sycommon/xxljob/xxljob_service.py,sha256=1yifwIBNGsCIxLnQjHKiBlbsigc_zvPH-dMTZcNxe-Q,7649
-sycommon_python_lib-0.2.7a29.dist-info/METADATA,sha256=gq42NKV6-MSlGgQftI3YmU0DiUDErLKMZZOEbMtrOGQ,7962
-sycommon_python_lib-0.2.7a29.dist-info/WHEEL,sha256=K260EYznzXsJYBQGqmI8VTxEdiZYNvDZwW9cBh9-_MA,91
-sycommon_python_lib-0.2.7a29.dist-info/entry_points.txt,sha256=gsR4SssKxDWjRU8ggidzNcdMXDPRSKRS7UaGyNP84Qg,92
-sycommon_python_lib-0.2.7a29.dist-info/top_level.txt,sha256=RgphKrg7nJyZ7irJqbxFr-5H2LUYTvI7ivoWZH2hcD0,29
-sycommon_python_lib-0.2.7a29.dist-info/RECORD,,
+sycommon_python_lib-0.2.7a30.dist-info/METADATA,sha256=A_VXT1ygWDo4Hkvwonq65az8FeY0k_8ik6hkReVPKWY,7962
+sycommon_python_lib-0.2.7a30.dist-info/WHEEL,sha256=K260EYznzXsJYBQGqmI8VTxEdiZYNvDZwW9cBh9-_MA,91
+sycommon_python_lib-0.2.7a30.dist-info/entry_points.txt,sha256=gsR4SssKxDWjRU8ggidzNcdMXDPRSKRS7UaGyNP84Qg,92
+sycommon_python_lib-0.2.7a30.dist-info/top_level.txt,sha256=RgphKrg7nJyZ7irJqbxFr-5H2LUYTvI7ivoWZH2hcD0,29
+sycommon_python_lib-0.2.7a30.dist-info/RECORD,,