openreward 0.1.159.dev1__tar.gz → 0.1.159.dev2__tar.gz

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openreward
-Version: 0.1.159.dev1
+Version: 0.1.159.dev2
 Summary: Python SDK for the OpenReward platform.
 Author-email: GR Inc <hello@gr.inc>
 License-Expression: MIT

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/openreward/environments/_sandbox_tools.py

@@ -15,7 +15,6 @@ Public exports:
 """
 from __future__ import annotations
 
-import base64
 import os
 import shlex
 from typing import Any, Dict, List, Optional, overload
@@ -62,15 +61,13 @@ async def upload_text(
 ) -> None:
     """Write text content to a file in the sandbox.
 
-    Uses ``check_run`` with a base64-encoded payload so binary-safe
-    transfer works without depending on ``sandbox.upload`` (which expects
-    a local-FS path, not in-memory bytes).
+    Goes through ``sandbox.upload_bytes`` (the exec-agent's ``/files``
+    endpoint), so no shell and no ``base64`` binary are involved and the
+    path needs no quoting. Parent directories are not created.
     """
     if ensure_trailing_newline and not content.endswith("\n"):
         content = content + "\n"
-    encoded = base64.b64encode(content.encode("utf-8")).decode("ascii")
-    # Quote the destination; the base64 payload is [A-Za-z0-9+/=] and needs none.
-    await sandbox.check_run(f"echo '{encoded}' | base64 -d > {shlex.quote(path)}")
+    await sandbox.upload_bytes(content.encode("utf-8"), path)
 
 
 # ── Workspace rooting ───────────────────────────────────────────────────────

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/openreward/toolsets/gemini_cli.py

@@ -10,7 +10,6 @@ source (``google-gemini/gemini-cli/packages/core/src/tools``).
 """
 from __future__ import annotations
 
-import base64
 import os
 import shlex
 from typing import Any, Dict, List, Literal, Optional
@@ -18,6 +17,7 @@ from typing import Any, Dict, List, Literal, Optional
 from pydantic import BaseModel, Field
 
 from openreward.environments import TextBlock, Toolset, ToolOutput, tool
+from openreward.environments._sandbox_tools import upload_text
 
 
 # ── Pydantic parameter models ──
@@ -87,19 +87,6 @@ async def _download_text(sandbox: Any, path: str) -> str:
     return data.decode("utf-8")
 
 
-async def _upload_text(
-    sandbox: Any,
-    path: str,
-    content: str,
-    ensure_trailing_newline: bool = True,
-) -> None:
-    if ensure_trailing_newline and not content.endswith("\n"):
-        content = content + "\n"
-    encoded = base64.b64encode(content.encode("utf-8")).decode("ascii")
-    # Quote the destination; the base64 payload is [A-Za-z0-9+/=] and needs none.
-    await sandbox.check_run(f"echo '{encoded}' | base64 -d > {shlex.quote(path)}")
-
-
 # ── Tool descriptions (verbatim from upstream Gemini CLI source) ──
 # Source: packages/core/src/tools/definitions/model-family-sets/default-legacy.ts
 # Shell description: packages/core/src/tools/definitions/dynamic-declaration-helpers.ts
@@ -272,7 +259,7 @@ class GeminiCliToolset(Toolset):
             dir_name = os.path.dirname(params.file_path)
             if dir_name:
                 await self.sandbox.run(f"mkdir -p {shlex.quote(dir_name)}")
-            await _upload_text(
+            await upload_text(
                 self.sandbox,
                 params.file_path,
                 params.content,
@@ -329,7 +316,7 @@ class GeminiCliToolset(Toolset):
             else:
                 new_content = content.replace(params.old_string, params.new_string, 1)
 
-            await _upload_text(
+            await upload_text(
                 self.sandbox, params.file_path, new_content, ensure_trailing_newline=True
             )
 

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/openreward/toolsets/hermes.py

@@ -7,7 +7,6 @@ upstream registry definitions (``nousresearch/hermes-agent``).
 """
 from __future__ import annotations
 
-import base64
 import os
 import shlex
 from typing import Any, Optional
@@ -17,6 +16,7 @@ from pydantic import BaseModel
 from openreward.environments.environment import tool
 from openreward.environments.toolset import Toolset
 from openreward.environments.types import TextBlock, ToolOutput
+from openreward.environments._sandbox_tools import upload_text
 
 
 async def _download_text(sandbox: Any, path: str) -> str:
@@ -24,19 +24,6 @@ async def _download_text(sandbox: Any, path: str) -> str:
     return data.decode("utf-8")
 
 
-async def _upload_text(
-    sandbox: Any,
-    path: str,
-    content: str,
-    ensure_trailing_newline: bool = True,
-) -> None:
-    if ensure_trailing_newline and not content.endswith("\n"):
-        content = content + "\n"
-    encoded = base64.b64encode(content.encode("utf-8")).decode("ascii")
-    # Quote the destination; the base64 payload is [A-Za-z0-9+/=] and needs none.
-    await sandbox.check_run(f"echo '{encoded}' | base64 -d > {shlex.quote(path)}")
-
-
 # ── Pydantic parameter models ──
 
 class TerminalParams(BaseModel, extra="forbid"):
@@ -188,7 +175,7 @@ class HermesToolset(Toolset):
             dir_name = os.path.dirname(params.path)
             if dir_name:
                 await self.sandbox.run(f"mkdir -p {shlex.quote(dir_name)}")
-            await _upload_text(
+            await upload_text(
                 self.sandbox,
                 params.path,
                 params.content,
@@ -319,7 +306,7 @@ class HermesToolset(Toolset):
             else:
                 new_content = content.replace(params.old_string, params.new_string, 1)
 
-            await _upload_text(self.sandbox, params.path, new_content, ensure_trailing_newline=True)
+            await upload_text(self.sandbox, params.path, new_content, ensure_trailing_newline=True)
 
             return ToolOutput(
                 metadata={"output": "", "exit_code": 0},
@@ -344,7 +331,7 @@ class HermesToolset(Toolset):
                 )
 
             patch_tmp = "/tmp/_hermes_patch.diff"
-            await _upload_text(self.sandbox, patch_tmp, params.patch, ensure_trailing_newline=True)
+            await upload_text(self.sandbox, patch_tmp, params.patch, ensure_trailing_newline=True)
 
             output, code = await self.sandbox.run(f"patch -p1 < {shlex.quote(patch_tmp)}")
             if code != 0:

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/openreward/toolsets/openclaw.py

@@ -7,7 +7,6 @@ upstream definitions.
 """
 from __future__ import annotations
 
-import base64
 import os
 import shlex
 from typing import Any, List, Optional
@@ -17,6 +16,7 @@ from pydantic import BaseModel
 from openreward.environments.environment import tool
 from openreward.environments.toolset import Toolset
 from openreward.environments.types import TextBlock, ToolOutput
+from openreward.environments._sandbox_tools import upload_text
 
 
 def _proc_path(sid: str, ext: str) -> str:
@@ -29,19 +29,6 @@ async def _download_text(sandbox: Any, path: str) -> str:
     return data.decode("utf-8")
 
 
-async def _upload_text(
-    sandbox: Any,
-    path: str,
-    content: str,
-    ensure_trailing_newline: bool = True,
-) -> None:
-    if ensure_trailing_newline and not content.endswith("\n"):
-        content = content + "\n"
-    encoded = base64.b64encode(content.encode("utf-8")).decode("ascii")
-    # Quote the destination; the base64 payload is [A-Za-z0-9+/=] and needs none.
-    await sandbox.check_run(f"echo '{encoded}' | base64 -d > {shlex.quote(path)}")
-
-
 # ── Pydantic parameter models ──
 
 class ExecParams(BaseModel, extra="forbid"):
@@ -277,7 +264,7 @@ class OpenClawToolset(Toolset):
             dir_name = os.path.dirname(params.path)
             if dir_name:
                 await self.sandbox.run(f"mkdir -p {shlex.quote(dir_name)}")
-            await _upload_text(
+            await upload_text(
                 self.sandbox,
                 params.path,
                 params.content,
@@ -317,7 +304,7 @@ class OpenClawToolset(Toolset):
                     )
                 content = content.replace(item.oldText, item.newText, 1)
 
-            await _upload_text(self.sandbox, params.path, content, ensure_trailing_newline=True)
+            await upload_text(self.sandbox, params.path, content, ensure_trailing_newline=True)
 
             return ToolOutput(
                 metadata={"output": "", "exit_code": 0},
@@ -336,7 +323,7 @@ class OpenClawToolset(Toolset):
     async def apply_patch(self, params: ApplyPatchParams) -> ToolOutput:
         try:
             patch_tmp = "/tmp/_openclaw_patch.diff"
-            await _upload_text(self.sandbox, patch_tmp, params.input, ensure_trailing_newline=True)
+            await upload_text(self.sandbox, patch_tmp, params.input, ensure_trailing_newline=True)
 
             output, code = await self.sandbox.run(f"patch -p1 < {shlex.quote(patch_tmp)}")
             if code != 0:

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/openreward.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openreward
-Version: 0.1.159.dev1
+Version: 0.1.159.dev2
 Summary: Python SDK for the OpenReward platform.
 Author-email: GR Inc <hello@gr.inc>
 License-Expression: MIT

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "openreward"
-version = "0.1.159.dev1"
+version = "0.1.159.dev2"
 description = "Python SDK for the OpenReward platform."
 readme = "README.md"
 license = "MIT"

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_cli_toolset_root_and_timeout.py

@@ -24,6 +24,9 @@ class RecordingSandbox:
         self.uploads.append(cmd)
         return RunResult(output="", return_code=0)
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        self.uploads.append(f"upload:{path}")
+
     async def download(self, path: str) -> bytes:
         self.uploads.append(f"download:{path}")
         return self.file_content.encode()

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_gemini_cli_toolset.py

@@ -6,7 +6,6 @@ FastAPI server to verify end-to-end HTTP routing through the session layer.
 """
 
 import asyncio
-import base64
 from threading import Thread
 from typing import Generator
 
@@ -37,12 +36,11 @@ class _MockSandbox:
 
     async def check_run(self, cmd: str, **kwargs) -> str:
         self.commands.append(cmd)
-        if " | base64 -d > " in cmd:
-            encoded = cmd.split("'", 2)[1]
-            path = cmd.split(" > ", 1)[1]
-            self.files[path] = base64.b64decode(encoded)
         return ""
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        self.files[path] = data
+
     async def download(self, path: str) -> bytes:
         return self.files.get(path, b"")
 

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_sandbox_tool_shell_safety.py

@@ -69,6 +69,11 @@ class RealShellSandbox:
     async def download(self, path: str) -> bytes:
         return Path(path).read_bytes()
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        # The agent writes the path verbatim: no shell, so no quoting to test here,
+        # but a missing parent directory still fails like the agent's 404 would.
+        Path(path).write_bytes(data)
+
 
 class RaisingSandbox:
     """Every sandbox call raises the given exception."""

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_sandbox_tools.py

@@ -1,7 +1,5 @@
 """``_sandbox_tools`` outputs carry no reward."""
 
-import base64
-import re
 from functools import partial
 
 import pytest
@@ -39,11 +37,11 @@ class FakeSandbox:
         return (f"ran: {cmd}", 0)
 
     async def check_run(self, cmd: str, **kwargs) -> str:
-        m = re.fullmatch(r"echo '([A-Za-z0-9+/=]+)' \| base64 -d > (.+)", cmd)
-        if m:
-            self.files[m.group(2)] = base64.b64decode(m.group(1).encode("ascii"))
         return ""
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        self.files[path] = data
+
     async def download(self, path: str) -> bytes:
         if path not in self.files:
             raise FileNotFoundError(path)

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_session_toolset.py

@@ -6,7 +6,6 @@ through the HTTP layer (POST /create, GET /task_tools, POST /call).
 """
 
 import asyncio
-import base64
 from threading import Thread
 from typing import Generator
 
@@ -37,13 +36,11 @@ class _MockSandbox:
 
     async def check_run(self, cmd: str, **kwargs) -> str:
         self.commands.append(cmd)
-        # Emulate "echo BASE64 | base64 -d > path" used by _upload_text.
-        if " | base64 -d > " in cmd:
-            encoded = cmd.split("'", 2)[1]
-            path = cmd.split(" > ", 1)[1]
-            self.files[path] = base64.b64decode(encoded)
         return ""
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        self.files[path] = data
+
     async def download(self, path: str) -> bytes:
         return self.files.get(path, b"")
 

{openreward-0.1.159.dev1 → openreward-0.1.159.dev2}/tests/test_toolsets.py

@@ -532,7 +532,6 @@ def test_list_task_tools_override():
 # A FakeFsSandbox in-memory file dict is enough — none of the tools talk
 # to a real container.
 
-import base64
 import re
 
 from openreward.toolsets import CLIToolset
@@ -555,7 +554,7 @@ class FakeFsSandbox:
     Supports the three calls the SDK helpers make:
       * ``await sandbox.download(path)`` -> bytes (from ``self.files``)
       * ``await sandbox.check_run(cmd)`` -> recognises the base64 upload
-        pattern emitted by ``upload_text`` and updates ``self.files``;
+        ``upload_bytes`` call made by ``upload_text`` and updates ``self.files``;
         otherwise echoes the command.
       * ``await sandbox.run(cmd)`` -> echoes ``(cmd, return_code)`` with a
         configurable default return code.
@@ -582,13 +581,11 @@ class FakeFsSandbox:
 
     async def check_run(self, cmd: str):
         self.check_run_log.append(cmd)
-        m = re.fullmatch(r"echo '([A-Za-z0-9+/=]+)' \| base64 -d > (.+)", cmd)
-        if m:
-            payload = base64.b64decode(m.group(1).encode("ascii"))
-            self.files[m.group(2)] = payload
-            return ("", 0)
         return ("", 0)
 
+    async def upload_bytes(self, data: bytes, path: str) -> None:
+        self.files[path] = data
+
     async def run(self, cmd: str, **kwargs):
         self.run_log.append(cmd)
         for pat, (output, code) in self.run_responses: