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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: optio-opencode
-Version: 0.1.2
+Version: 0.1.3
 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.2 → optio_opencode-0.1.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "optio-opencode"
-version = "0.1.2"
+version = "0.1.3"
 description = "Run opencode web as an optio task; local subprocess or remote via SSH."
 readme = "README.md"
 license = "Apache-2.0"

{optio_opencode-0.1.2 → optio_opencode-0.1.3}/src/optio_opencode/prompt.py

@@ -79,9 +79,18 @@ caveats:**
 
 ### Detecting a resume: `resume.log`
 
-Each session start (fresh or resumed) appends one ISO 8601 timestamp
-to `./resume.log`. The very first line is the original launch
-timestamp; each subsequent line is a resume.
+Each session start (fresh or resumed) appends one line to
+`./resume.log`. Line format:
+
+```
+<ISO 8601 UTC timestamp>[ REFRESHED:<comma-separated filenames>]
+```
+
+The very first line is the original launch timestamp; each subsequent
+line is a resume. The optional `REFRESHED:` suffix signals that the
+harness rewrote the listed files on that resume (e.g.
+`2026-05-28T13:15:42Z REFRESHED:AGENTS.md`) — your in-memory copy of
+those files is stale and must be re-read before continuing.
 
 **At the start of every new incoming user message, read
 `./resume.log` first.** Compare the latest line to the value you
@@ -92,6 +101,10 @@ the situation as a resume:
   outside the workdir are still where you left them.
 - Re-establish anything that's gone (re-launch a server, re-fetch a
   file, etc.) before continuing.
+- **If the latest line carries a `REFRESHED:` suffix, re-read each
+  listed file** (e.g. `cat ./AGENTS.md`) — the harness updated it
+  since your last context snapshot and the version you remember is
+  out of date.
 - Then resume the work you were doing.
 
 If a resume slips past unnoticed, a failing tool call is the

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

@@ -160,6 +160,7 @@ async def run_opencode_session(ctx: ProcessContext, config: OpencodeTaskConfig)
         """
         nonlocal launched_handle, opencode_exec, session_id, preserved_session_id
 
+        refreshed_files: list[str] = []
         if not resuming:
             # Fresh start: the protocol driver has already created the
             # workdir, deliverables/ subdir, and empty optio.log. Ensure
@@ -189,9 +190,14 @@ async def run_opencode_session(ctx: ProcessContext, config: OpencodeTaskConfig)
             # self-healing (snapshot lookup returns None → fresh-start
             # fallback) handles the rare case where the flag is true but
             # no snapshot exists.
+        else:
+            # Resume: when on_resume_refresh is wired, recompute AGENTS.md
+            # from the refreshed config and overwrite the workdir copy if
+            # the rendered text differs from the snapshot-restored file.
+            refreshed_files = await _maybe_refresh_on_resume(host, hook_ctx, config)
 
         if config.supports_resume:
-            await _append_resume_log_entry(host)
+            await _append_resume_log_entry(host, refreshed=refreshed_files)
 
         # opencode is already installed by run_opencode_session before
         # this body runs (so resume restore can call opencode_import
@@ -469,16 +475,26 @@ async def _rotate_optio_log(host: Host) -> None:
     await host.write_text("optio.log", "")
 
 
-async def _append_resume_log_entry(host) -> None:
-    """Append one ISO 8601 UTC timestamp line to <workdir>/resume.log.
+async def _append_resume_log_entry(
+    host, *, refreshed: list[str] | None = None,
+) -> None:
+    """Append one line to <workdir>/resume.log.
+
+    Line format: ``<ISO 8601 UTC timestamp>[ REFRESHED:<comma-separated names>]``.
+    The optional ``REFRESHED:`` suffix signals that the harness rewrote
+    the listed files on this session start. Agents are instructed (via
+    the resume section of AGENTS.md) to re-read tagged files.
 
     Creates the file if missing (via shell `>>`). Caller is responsible
     for gating this on config.supports_resume.
     """
     ts = datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ")
+    line = ts
+    if refreshed:
+        line = f"{ts} REFRESHED:{','.join(refreshed)}"
     target = f"{host.workdir}/resume.log"
     result = await host.run_command(
-        f"echo {shlex.quote(ts)} >> {shlex.quote(target)}"
+        f"echo {shlex.quote(line)} >> {shlex.quote(target)}"
     )
     if result.exit_code != 0:
         raise RuntimeError(
@@ -487,6 +503,46 @@ async def _append_resume_log_entry(host) -> None:
         )
 
 
+async def _maybe_refresh_on_resume(
+    host, hook_ctx, config: OpencodeTaskConfig,
+) -> list[str]:
+    """Run the on_resume_refresh hook (if any) and rewrite AGENTS.md when
+    the rendered content differs from the workdir copy.
+
+    Returns the list of filenames the harness rewrote (currently at most
+    ``["AGENTS.md"]``), suitable for tagging the next ``resume.log`` line.
+    A hook that raises is logged and ignored — the resumed session keeps
+    whatever AGENTS.md the snapshot restored.
+    """
+    if config.on_resume_refresh is None:
+        return []
+    try:
+        new_config = config.on_resume_refresh(config)
+    except Exception:
+        _LOG.exception(
+            "on_resume_refresh raised; keeping existing AGENTS.md from snapshot",
+        )
+        return []
+    new_agents_md = compose_agents_md(
+        new_config.consumer_instructions,
+        workdir_exclude=new_config.workdir_exclude,
+        supports_resume=new_config.supports_resume,
+    )
+    try:
+        existing = await hook_ctx.read_text_from_host("AGENTS.md", silent=True)
+    except FileNotFoundError:
+        existing = None
+    except Exception:
+        _LOG.exception(
+            "failed to read existing AGENTS.md on resume; rewriting unconditionally",
+        )
+        existing = None
+    if existing == new_agents_md:
+        return []
+    await host.write_text("AGENTS.md", new_agents_md)
+    return ["AGENTS.md"]
+
+
 def _pick_local_workdir() -> str:
     return tempfile.mkdtemp(prefix="optio-opencode-")
 

{optio_opencode-0.1.2 → optio_opencode-0.1.3}/src/optio_opencode/types.py

@@ -43,6 +43,14 @@ class OpencodeTaskConfig:
     # raises a config error: asymmetric usage is always a mistake.
     session_blob_encrypt: Callable[[bytes], bytes] | None = None
     session_blob_decrypt: Callable[[bytes], bytes] | None = None
+    # Optional hook fired on resume only (never on fresh start). Receives
+    # the original task config; returns a (possibly mutated/replaced) config.
+    # The harness re-renders AGENTS.md from the returned config and writes
+    # it back to the workdir only when it differs from the file on disk.
+    # When written, the harness tags the new line in resume.log with
+    # `REFRESHED:AGENTS.md` so the agent knows to re-read. None (default)
+    # → no refresh; the resumed session keeps its original AGENTS.md.
+    on_resume_refresh: Callable[["OpencodeTaskConfig"], "OpencodeTaskConfig"] | None = None
 
     def __post_init__(self) -> None:
         e = self.session_blob_encrypt is not None

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: optio-opencode
-Version: 0.1.2
+Version: 0.1.3
 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.2 → optio_opencode-0.1.3}/tests/test_session_local.py

@@ -348,6 +348,185 @@ async def test_append_resume_log_entry_appends_on_repeat_call(tmp_workdir):
     assert lines[0] != lines[1]
 
 
+async def test_append_resume_log_entry_with_refreshed_tag(tmp_workdir):
+    """Passing refreshed=[...] adds ` REFRESHED:<files>` suffix to the line."""
+    import os
+    import re
+    from optio_host.host import LocalHost
+    from optio_opencode.session import _append_resume_log_entry
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+
+    await _append_resume_log_entry(host, refreshed=["AGENTS.md"])
+
+    resume_log = os.path.join(host.workdir, "resume.log")
+    with open(resume_log) as f:
+        lines = [line for line in f.read().splitlines() if line]
+    assert len(lines) == 1
+    assert re.match(
+        r"^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z REFRESHED:AGENTS\.md$",
+        lines[0],
+    )
+
+
+async def test_append_resume_log_entry_with_multiple_refreshed_files(tmp_workdir):
+    """refreshed=[a, b] → REFRESHED:a,b (comma-separated, no spaces)."""
+    import os
+    import re
+    from optio_host.host import LocalHost
+    from optio_opencode.session import _append_resume_log_entry
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+
+    await _append_resume_log_entry(host, refreshed=["AGENTS.md", "opencode.json"])
+
+    resume_log = os.path.join(host.workdir, "resume.log")
+    with open(resume_log) as f:
+        lines = [line for line in f.read().splitlines() if line]
+    assert len(lines) == 1
+    assert re.match(
+        r"^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z REFRESHED:AGENTS\.md,opencode\.json$",
+        lines[0],
+    )
+
+
+async def test_append_resume_log_entry_empty_refreshed_omits_tag(tmp_workdir):
+    """refreshed=[] → bare timestamp, no REFRESHED: suffix (same as None)."""
+    import os
+    import re
+    from optio_host.host import LocalHost
+    from optio_opencode.session import _append_resume_log_entry
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+
+    await _append_resume_log_entry(host, refreshed=[])
+
+    resume_log = os.path.join(host.workdir, "resume.log")
+    with open(resume_log) as f:
+        lines = [line for line in f.read().splitlines() if line]
+    assert len(lines) == 1
+    assert re.match(r"^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z$", lines[0])
+
+
+# ---- resume refresh -------------------------------------------------------
+
+
+async def test_maybe_refresh_on_resume_no_hook(tmp_workdir):
+    """on_resume_refresh=None → returns [] and does not write AGENTS.md."""
+    import os
+    from optio_host.host import LocalHost
+    from optio_opencode.session import _maybe_refresh_on_resume
+    from optio_opencode.types import OpencodeTaskConfig
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+    config = OpencodeTaskConfig(consumer_instructions="x")
+
+    refreshed = await _maybe_refresh_on_resume(host, None, config)
+
+    assert refreshed == []
+    assert not os.path.exists(os.path.join(host.workdir, "AGENTS.md"))
+
+
+async def test_maybe_refresh_on_resume_unchanged_content_skips_write(tmp_workdir):
+    """Hook return identical to existing AGENTS.md → no write, [] returned."""
+    import os
+    from optio_host.host import LocalHost
+    from optio_opencode.prompt import compose_agents_md
+    from optio_opencode.session import _maybe_refresh_on_resume
+    from optio_opencode.types import OpencodeTaskConfig
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+    config = OpencodeTaskConfig(
+        consumer_instructions="task X",
+        on_resume_refresh=lambda c: c,
+    )
+    expected = compose_agents_md(
+        config.consumer_instructions,
+        workdir_exclude=config.workdir_exclude,
+        supports_resume=config.supports_resume,
+    )
+    await host.write_text("AGENTS.md", expected)
+    mtime_before = os.path.getmtime(os.path.join(host.workdir, "AGENTS.md"))
+
+    class _FakeHookCtx:
+        async def read_text_from_host(self, path, *, silent=False):
+            full = os.path.join(host.workdir, path)
+            with open(full) as f:
+                return f.read()
+
+    refreshed = await _maybe_refresh_on_resume(host, _FakeHookCtx(), config)
+
+    assert refreshed == []
+    mtime_after = os.path.getmtime(os.path.join(host.workdir, "AGENTS.md"))
+    assert mtime_after == mtime_before  # write was skipped
+
+
+async def test_maybe_refresh_on_resume_changed_content_writes(tmp_workdir):
+    """Hook returns new instructions → AGENTS.md rewritten, ['AGENTS.md']."""
+    import os
+    from dataclasses import replace
+    from optio_host.host import LocalHost
+    from optio_opencode.prompt import compose_agents_md
+    from optio_opencode.session import _maybe_refresh_on_resume
+    from optio_opencode.types import OpencodeTaskConfig
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+    config = OpencodeTaskConfig(
+        consumer_instructions="old task",
+        on_resume_refresh=lambda c: replace(c, consumer_instructions="new task"),
+    )
+    old_agents = compose_agents_md(
+        "old task",
+        workdir_exclude=config.workdir_exclude,
+        supports_resume=config.supports_resume,
+    )
+    await host.write_text("AGENTS.md", old_agents)
+
+    class _FakeHookCtx:
+        async def read_text_from_host(self, path, *, silent=False):
+            full = os.path.join(host.workdir, path)
+            with open(full) as f:
+                return f.read()
+
+    refreshed = await _maybe_refresh_on_resume(host, _FakeHookCtx(), config)
+
+    assert refreshed == ["AGENTS.md"]
+    with open(os.path.join(host.workdir, "AGENTS.md")) as f:
+        content = f.read()
+    assert "new task" in content
+    assert "old task" not in content
+
+
+async def test_maybe_refresh_on_resume_hook_raises_keeps_existing(tmp_workdir):
+    """Hook that raises → returns [] and leaves existing AGENTS.md alone."""
+    import os
+    from optio_host.host import LocalHost
+    from optio_opencode.session import _maybe_refresh_on_resume
+    from optio_opencode.types import OpencodeTaskConfig
+
+    def _boom(c):
+        raise RuntimeError("hook is broken")
+
+    host = LocalHost(taskdir=tmp_workdir)
+    await host.setup_workdir()
+    await host.write_text("AGENTS.md", "existing content")
+    config = OpencodeTaskConfig(
+        consumer_instructions="x", on_resume_refresh=_boom,
+    )
+
+    refreshed = await _maybe_refresh_on_resume(host, None, config)
+
+    assert refreshed == []
+    with open(os.path.join(host.workdir, "AGENTS.md")) as f:
+        assert f.read() == "existing content"
+
+
 async def test_session_local_supports_resume_false_skips_resume_log(
     ctx_and_captures, _supply_scenario, tmp_workdir, monkeypatch,
 ):

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

@@ -184,7 +184,15 @@ async def test_resume_appends_second_line_to_resume_log(mongo_db, task_root):
     lines = [line for line in contents.splitlines() if line]
     assert len(lines) == 2, f"expected 2 lines, got {len(lines)}: {contents!r}"
 
-    iso_re = re.compile(r"^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z$")
+    # Line format: `<ISO 8601 timestamp>[ REFRESHED:<comma-separated names>]`.
+    # This test exercises the no-refresh path (no on_resume_refresh hook),
+    # so every line is a bare timestamp; the regex still accepts the
+    # extended form to stay valid as the format evolves.
+    line_re = re.compile(
+        r"^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z(?: REFRESHED:\S+)?$"
+    )
     for line in lines:
-        assert iso_re.match(line), f"non-ISO-8601 line: {line!r}"
-    assert lines[0] <= lines[1], f"timestamps not monotonic: {lines!r}"
+        assert line_re.match(line), f"unrecognized resume.log line: {line!r}"
+    # Timestamps (the leading token of each line) are monotonic.
+    timestamps = [line.split()[0] for line in lines]
+    assert timestamps[0] <= timestamps[1], f"timestamps not monotonic: {timestamps!r}"

{optio_opencode-0.1.2 → optio_opencode-0.1.3}/tests/test_types.py

@@ -95,3 +95,16 @@ def test_opencode_task_config_supports_resume_default_true():
 def test_opencode_task_config_supports_resume_can_be_disabled():
     cfg = OpencodeTaskConfig(consumer_instructions="x", supports_resume=False)
     assert cfg.supports_resume is False
+
+
+def test_opencode_task_config_on_resume_refresh_default_none():
+    cfg = OpencodeTaskConfig(consumer_instructions="x")
+    assert cfg.on_resume_refresh is None
+
+
+def test_opencode_task_config_on_resume_refresh_accepts_callable():
+    def _refresh(c):
+        return c
+
+    cfg = OpencodeTaskConfig(consumer_instructions="x", on_resume_refresh=_refresh)
+    assert cfg.on_resume_refresh is _refresh