remote-exec-grpc 0.2.1__tar.gz → 0.2.2__tar.gz

remote_exec_grpc-0.2.2/.github/workflows/ci.yml

@@ -0,0 +1,49 @@
+name: CI
+
+on:
+  push:
+    branches: [main, dev]
+
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  test:
+    name: test (Python ${{ matrix.python-version }})
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.11", "3.12", "3.13"]
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up uv
+        uses: astral-sh/setup-uv@v6
+        with:
+          python-version: ${{ matrix.python-version }}
+          enable-cache: true
+
+      - name: Install dependencies
+        run: uv sync --locked
+
+      - name: Run tests
+        run: uv run pytest -q
+
+  build:
+    name: build package
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up uv
+        uses: astral-sh/setup-uv@v6
+
+      - name: Build sdist and wheel
+        run: uv build
+
+      - name: Check package metadata
+        run: uvx twine check dist/*

{remote_exec_grpc-0.2.1 → remote_exec_grpc-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: remote-exec-grpc
-Version: 0.2.1
+Version: 0.2.2
 Summary: Lightweight gRPC remote command execution with real-time stdout/stderr streaming
 License: MIT
 Requires-Python: >=3.11

{remote_exec_grpc-0.2.1 → remote_exec_grpc-0.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "remote-exec-grpc"
-version = "0.2.1"
+version = "0.2.2"
 description = "Lightweight gRPC remote command execution with real-time stdout/stderr streaming"
 readme = "README.md"
 requires-python = ">=3.11"

{remote_exec_grpc-0.2.1 → remote_exec_grpc-0.2.2}/src/remote_exec/executor.py

@@ -122,6 +122,35 @@ async def terminate_process_group(pgid: int, grace: float = DEFAULT_KILL_GRACE)
     _signal_process_group(pgid, signal.SIGKILL)
 
 
+class _ExitNotifyingProtocol(asyncio.subprocess.SubprocessStreamProtocol):
+    """Stream protocol that resolves ``exited`` as soon as the process exits.
+
+    Before Python 3.13, ``Process.wait()`` only returns once the process has
+    exited *and* all its pipes are closed. A background child that inherited
+    stdout (``sleep 300 &``) would therefore block ``wait()`` until it exits,
+    so we could never notice the main process exiting and clean up its group.
+    """
+
+    def __init__(self, limit: int, loop: asyncio.AbstractEventLoop):
+        super().__init__(limit=limit, loop=loop)
+        self.exited: asyncio.Future[int] = loop.create_future()
+
+    def process_exited(self) -> None:
+        # Read the code first: the base class may drop _transport (3.13+).
+        returncode = self._transport.get_returncode()
+        super().process_exited()
+        if not self.exited.done():
+            self.exited.set_result(returncode)
+
+
+async def _spawn(program: str, *args: str, **kwds) -> tuple[asyncio.subprocess.Process, asyncio.Future[int]]:
+    """Like ``asyncio.create_subprocess_exec`` but also returns an exit future."""
+    loop = asyncio.get_running_loop()
+    protocol_factory = lambda: _ExitNotifyingProtocol(limit=2**16, loop=loop)  # noqa: E731
+    transport, protocol = await loop.subprocess_exec(protocol_factory, program, *args, **kwds)
+    return asyncio.subprocess.Process(transport, protocol, loop), protocol.exited
+
+
 class Executor:
     """Runs commands; tracks background cleanup tasks so shutdown can await them."""
 
@@ -151,7 +180,7 @@ class Executor:
         start_time = utcnow()
 
         try:
-            proc = await asyncio.create_subprocess_exec(
+            proc, exited = await _spawn(
                 self.shell,
                 "-c",
                 spec.command,
@@ -187,7 +216,8 @@ class Executor:
                     pass
 
         async def wait_and_reap() -> int:
-            rc = await proc.wait()
+            # Not proc.wait(): before 3.13 it also waits for the pipes to close.
+            rc = await exited
             # Kill whatever the command left behind in its process group.
             await terminate_process_group(pgid, self.kill_grace)
             return rc

{remote_exec_grpc-0.2.1 → remote_exec_grpc-0.2.2}/uv.lock

@@ -272,7 +272,7 @@ wheels = [
 
 [[package]]
 name = "remote-exec-grpc"
-version = "0.2.1"
+version = "0.2.2"
 source = { editable = "." }
 dependencies = [
     { name = "grpcio" },