optio-opencode 0.1.0__tar.gz → 0.1.2__tar.gz

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: optio-opencode
-Version: 0.1.0
+Version: 0.1.2
 Summary: Run opencode web as an optio task; local subprocess or remote via SSH.
 Author-email: Kristof Csillag <kristof.csillag@deai-labs.com>
 License-Expression: Apache-2.0

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "optio-opencode"
-version = "0.1.0"
+version = "0.1.2"
 description = "Run opencode web as an optio task; local subprocess or remote via SSH."
 readme = "README.md"
 license = "Apache-2.0"
@@ -27,6 +27,8 @@ classifiers = [
 ]
 dependencies = [
     "optio-core>=0.1,<0.2",
+    # 0.1.1 introduces the bind_addr kwarg on Host.establish_tunnel —
+    # required for OPTIO_WIDGET_TUNNEL_BIND to work.
     "optio-host>=0.1,<0.2",
     "asyncssh>=2.14",
 ]

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/src/optio_opencode/host_actions.py

@@ -328,6 +328,7 @@ async def launch_opencode(
     *,
     ready_timeout_s: float = 30.0,
     opencode_executable: str = "opencode",
+    hostname: str = "127.0.0.1",
 ) -> tuple[ProcessHandle, int]:
     """Launch ``opencode web`` on ``host``; wait for the listening URL.
 
@@ -340,6 +341,12 @@ async def launch_opencode(
     directory to PATH so opencode's automatic browser-launch is
     suppressed.
 
+    ``hostname`` is passed to ``opencode web --hostname=`` so callers
+    can bind to a non-loopback interface when consumers reach the server
+    across a network boundary (e.g. LocalHost inside a docker container
+    serving a sibling API-proxy container). Defaults to ``127.0.0.1`` to
+    keep RemoteHost-over-SSH and single-host deployments unchanged.
+
     Returns ``(handle, opencode_port)``. Caller is responsible for
     eventually terminating the handle via ``host.terminate_subprocess``.
     """
@@ -374,7 +381,7 @@ async def launch_opencode(
         f"exec env "
         f"OPENCODE_SERVER_PASSWORD=\"$(cat {shlex.quote(host.workdir + '/' + pw_file)})\" "
         f"BROWSER=true "
-        f"{opencode_executable} web --port=0 --hostname=127.0.0.1"
+        f"{opencode_executable} web --port=0 --hostname={shlex.quote(hostname)}"
     )
 
     # Prepend the noop-browsers bin dir to PATH via env on launch_subprocess.

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/src/optio_opencode/session.py

@@ -210,16 +210,35 @@ async def run_opencode_session(ctx: ProcessContext, config: OpencodeTaskConfig)
             host, opencode_executable=opencode_exec,
         )
         version_suffix = f" {version}" if version else ""
+        # --- tunnel + widget registration --------------------------------
+        # By default the SSH tunnel listens on 127.0.0.1 — only the worker
+        # process (this engine) can reach it.  For multi-container deploys
+        # where the API proxy lives in a different container than the
+        # engine but on the same Docker network, set
+        # OPTIO_WIDGET_TUNNEL_BIND=0.0.0.0 so sibling containers can
+        # connect to the engine's port, and OPTIO_WIDGET_TUNNEL_HOST to
+        # the Docker DNS name other containers resolve to reach this
+        # engine (e.g. the compose service name). Both default to
+        # 127.0.0.1 so single-host deploys are unchanged.
+        bind_addr = os.environ.get("OPTIO_WIDGET_TUNNEL_BIND", "127.0.0.1")
+        upstream_host = os.environ.get("OPTIO_WIDGET_TUNNEL_HOST", "127.0.0.1")
+
+        # LocalHost has no SSH tunnel — establish_tunnel is a no-op — so
+        # opencode itself must bind to ``bind_addr`` for sibling containers
+        # to reach it. RemoteHost keeps opencode bound to the remote's
+        # loopback; the SSH tunnel on the engine side handles exposure.
+        opencode_hostname = bind_addr if isinstance(host, LocalHost) else "127.0.0.1"
+
         ctx.report_progress(None, f"Launching opencode{version_suffix}…")
         handle, opencode_port = await host_actions.launch_opencode(
             host, password,
             ready_timeout_s=READY_TIMEOUT_S,
             opencode_executable=opencode_exec,
+            hostname=opencode_hostname,
         )
         launched_handle = handle
 
-        # --- tunnel + widget registration --------------------------------
-        worker_port = await host.establish_tunnel(opencode_port)
+        worker_port = await host.establish_tunnel(opencode_port, bind_addr=bind_addr)
 
         if preserved_session_id is not None:
             session_id = preserved_session_id
@@ -235,7 +254,7 @@ async def run_opencode_session(ctx: ProcessContext, config: OpencodeTaskConfig)
             )
 
         await ctx.set_widget_upstream(
-            f"http://127.0.0.1:{worker_port}",
+            f"http://{upstream_host}:{worker_port}",
             inner_auth=BasicAuth(username="opencode", password=password),
         )
         # Point the iframe directly at the pre-created session so viewers

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/src/optio_opencode.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: optio-opencode
-Version: 0.1.0
+Version: 0.1.2
 Summary: Run opencode web as an optio task; local subprocess or remote via SSH.
 Author-email: Kristof Csillag <kristof.csillag@deai-labs.com>
 License-Expression: Apache-2.0

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/tests/test_host_local.py

@@ -64,6 +64,50 @@ async def test_launch_times_out_on_no_url(tmp_path):
     pass
 
 
+async def test_launch_opencode_passes_hostname_into_cmd(local_host, monkeypatch):
+    """Multi-container deploys need opencode bound to a non-loopback
+    interface so a sibling API-proxy container can reach it. The
+    ``hostname`` kwarg must propagate into the ``opencode web
+    --hostname=`` argument."""
+    captured: dict[str, str] = {}
+
+    async def fake_launch_subprocess(self, cmd, *, env=None, cwd=None):
+        captured["cmd"] = cmd
+        raise RuntimeError("stop before waiting on stdout")
+
+    monkeypatch.setattr(LocalHost, "launch_subprocess", fake_launch_subprocess)
+
+    await local_host.setup_workdir()
+    with pytest.raises(RuntimeError, match="stop before"):
+        await host_actions.launch_opencode(
+            local_host, password="pw",
+            ready_timeout_s=1.0,
+            opencode_executable="opencode",
+            hostname="0.0.0.0",
+        )
+    assert "--hostname=0.0.0.0" in captured["cmd"]
+
+
+async def test_launch_opencode_default_hostname_is_loopback(local_host, monkeypatch):
+    """Default keeps single-host / RemoteHost-over-SSH behaviour intact."""
+    captured: dict[str, str] = {}
+
+    async def fake_launch_subprocess(self, cmd, *, env=None, cwd=None):
+        captured["cmd"] = cmd
+        raise RuntimeError("stop")
+
+    monkeypatch.setattr(LocalHost, "launch_subprocess", fake_launch_subprocess)
+
+    await local_host.setup_workdir()
+    with pytest.raises(RuntimeError, match="stop"):
+        await host_actions.launch_opencode(
+            local_host, password="pw",
+            ready_timeout_s=1.0,
+            opencode_executable="opencode",
+        )
+    assert "--hostname=127.0.0.1" in captured["cmd"]
+
+
 async def test_tail_file_yields_appended_lines(local_host):
     await local_host.setup_workdir()
     log_path = os.path.join(local_host.workdir, "optio.log")

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/tests/test_session_hooks.py

@@ -118,7 +118,7 @@ def _patch_host_actions(monkeypatch, host):
     async def _version(_host, *, opencode_executable="opencode"):
         return None
 
-    async def _launch(_host, _password, *, ready_timeout_s=30.0, opencode_executable="opencode"):
+    async def _launch(_host, _password, *, ready_timeout_s=30.0, opencode_executable="opencode", hostname="127.0.0.1"):
         host.timeline.append("launch_opencode")
         raise RuntimeError("test never gets past launch")
 

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/tests/test_session_local.py

@@ -105,7 +105,7 @@ def _supply_scenario(monkeypatch):
     orig_launch = host_actions.launch_opencode
     scenario_holder: dict = {"name": "happy"}
 
-    async def _launch(host, password, *, ready_timeout_s=30.0, opencode_executable="opencode"):
+    async def _launch(host, password, *, ready_timeout_s=30.0, opencode_executable="opencode", hostname="127.0.0.1"):
         del opencode_executable  # we substitute fully
         return await orig_launch(
             host, password,
@@ -114,6 +114,7 @@ def _supply_scenario(monkeypatch):
                 f"{sys.executable} {FAKE_OPENCODE} "
                 f"--scenario {scenario_holder['name']}"
             ),
+            hostname=hostname,
         )
     monkeypatch.setattr(host_actions, "launch_opencode", _launch)
 

{optio_opencode-0.1.0 → optio_opencode-0.1.2}/tests/test_session_resume.py

@@ -53,7 +53,7 @@ def _supply_scenario(monkeypatch):
     orig_launch = host_actions.launch_opencode
     holder = {"name": "happy"}
 
-    async def _launch(host, password, *, ready_timeout_s=30.0, opencode_executable="opencode"):
+    async def _launch(host, password, *, ready_timeout_s=30.0, opencode_executable="opencode", hostname="127.0.0.1"):
         del opencode_executable
         return await orig_launch(
             host, password,
@@ -61,6 +61,7 @@ def _supply_scenario(monkeypatch):
             opencode_executable=(
                 f"{sys.executable} {FAKE_OPENCODE} --scenario {holder['name']}"
             ),
+            hostname=hostname,
         )
     monkeypatch.setattr(host_actions, "launch_opencode", _launch)