verifiers 0.3.2.dev169__tar.gz → 0.3.2.dev170__tar.gz

{verifiers-0.3.2.dev169 → verifiers-0.3.2.dev170}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: verifiers
-Version: 0.3.2.dev169
+Version: 0.3.2.dev170
 Summary: Verifiers: Environments for LLM Reinforcement Learning
 Project-URL: Homepage, https://github.com/primeintellect-ai/verifiers
 Project-URL: Documentation, https://github.com/primeintellect-ai/verifiers

{verifiers-0.3.2.dev169 → verifiers-0.3.2.dev170}/verifiers/v1/utils/artifacts.py

@@ -190,7 +190,17 @@ async def _tar_out(runtime: Runtime, artifact: Artifact, budget: int) -> bytes:
         )
         # The runtime enforces the remaining collection budget while transferring the
         # bytes, so replacing or growing the archive cannot race a separate size probe.
-        return await runtime.read(path, max_bytes=budget)
+        archive, dropped = _drop_special_files(
+            await runtime.read(path, max_bytes=budget)
+        )
+        if dropped:
+            logger.warning(
+                "artifact %r: dropped %d FIFO or device node(s), e.g. %s",
+                artifact.source,
+                len(dropped),
+                dropped[:3],
+            )
+        return archive
     finally:
         # Best-effort: the box is about to be destroyed and the name is unique per call.
         try:
@@ -199,6 +209,32 @@ async def _tar_out(runtime: Runtime, artifact: Artifact, budget: int) -> bytes:
             logger.debug("failed to remove %s", path, exc_info=True)
 
 
+def _drop_special_files(archive: bytes) -> tuple[bytes, list[str]]:
+    """Remove FIFOs and device nodes, plus hard links to them, from a collected archive.
+
+    They carry no content, and `restore` refuses them, so one an agent leaves in its
+    workspace (e.g. a named pipe it made to test streaming input) would otherwise fail
+    the whole restore. Returns the archive and the dropped member names."""
+    with tarfile.open(fileobj=io.BytesIO(archive), mode="r:") as source:
+        members = source.getmembers()
+        special = {m.name for m in members if m.isfifo() or m.ischr() or m.isblk()}
+        if not special:
+            return archive, []
+        dropped = [
+            m.name
+            for m in members
+            if m.name in special or (m.islnk() and m.linkname in special)
+        ]
+        out = io.BytesIO()
+        with tarfile.open(fileobj=out, mode="w:", format=source.format) as target:
+            for member in members:
+                if member.name in dropped:
+                    continue
+                content = source.extractfile(member) if member.isfile() else None
+                target.addfile(member, content)
+    return out.getvalue(), dropped
+
+
 def _validate_restore(root: str, archive: bytes | None) -> None:
     """Reject archive content that could restore outside its declared root."""
     root_path = PurePosixPath(root)