agentic-dev-loop 1.0.0a12__tar.gz → 1.0.0a13__tar.gz

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-dev-loop
-Version: 1.0.0a12
+Version: 1.0.0a13
 Summary: Turn an LLM into a gated, sandboxed, resumable construction loop over your codebase.
 Author: Luke Wilson
 License-Expression: Apache-2.0

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "agentic-dev-loop"
-version = "1.0.0a12"
+version = "1.0.0a13"
 description = "Turn an LLM into a gated, sandboxed, resumable construction loop over your codebase."
 readme = "README.md"
 requires-python = ">=3.12"

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/src/agentic_dev_loop/engine/cli.py

@@ -12,6 +12,7 @@ from pathlib import Path
 from agentic_dev_loop.core.config import PipelineConfig, load_config
 from .dryrun import dry_run
 from .isolation import clean_stamps, isolation_state_dir
+from .loaders import mark_plan_completed
 from .registry import Registry, default_registry, load_extensions
 
 
@@ -112,6 +113,10 @@ def main(argv: list[str] | None = None) -> int:
     # dry run, --clean-stamps, and --help stay side-effect free.
     from .graph import run_pipeline
     success, _attempts = run_pipeline(args.pipeline, args, registry=registry)
+    if success and args.plan is not None:
+        # Only a completed run earns the stamp — failed runs leave the plan
+        # as-is. AI-DLC runs never pass --plan, so aidlc docs stay untouched.
+        mark_plan_completed(args.plan)
     if not success:
         print("agentic-dev-loop: pipeline failed — exiting with code 1",
               file=sys.stderr)

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/src/agentic_dev_loop/engine/isolation.py

@@ -274,11 +274,32 @@ def is_git_mutation(argv: list[str]) -> bool:
     return True
 
 
+def _expand_home(path: str) -> str:
+    """Expand a leading ``~`` against ``$HOME``, falling back to
+    ``USERPROFILE`` (Windows).
+
+    The engine is driven from bash on every platform (Windows via Git Bash),
+    where HOME is the authoritative home. CPython's Windows expanduser ignores
+    HOME in favour of USERPROFILE, so a literal ``~`` in a config file would
+    resolve inconsistently on Windows. Only a leading tilde (``~``, ``~/``,
+    ``~\\``) is handled here; ``~user`` forms and the no-HOME/no-USERPROFILE
+    case fall through to the stdlib below.
+    """
+    if path != "~" and not path.startswith(("~/", "~\\")):
+        return path
+    home = os.environ.get("HOME") or os.environ.get("USERPROFILE")
+    if home:
+        return home + path[1:]
+    return path
+
+
 def translate_mount_path(path: str) -> str:
     """expanduser + absolute resolution without symlink chasing. Identity on
     macOS/Windows/WSL2-with-Docker-Desktop (verbatim passthrough — Docker
-    Desktop translates). Idempotent: translate(translate(p)) == translate(p)."""
-    p = Path(path).expanduser()
+    Desktop translates). Idempotent: translate(translate(p)) == translate(p).
+    A leading ``~`` resolves against $HOME first (USERPROFILE fallback), then
+    the stdlib handles anything left (pwd on POSIX)."""
+    p = Path(_expand_home(path)).expanduser()
     if not p.is_absolute():
         p = p.absolute()
     return str(p)

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/src/agentic_dev_loop/engine/loaders.py

@@ -1,6 +1,7 @@
 """Built-in `plan` document loader + the loader-selection flow."""
 
 import argparse
+import re
 from pathlib import Path
 
 from agentic_dev_loop.core.config import PipelineConfig
@@ -18,6 +19,46 @@ def resolve_plan_docs(plan_path: Path | str) -> dict[str, str]:
     return {path.name: path.read_text()}
 
 
+# A status line at the top of a planning doc, in any recorded form:
+# `> **Status: COMPLETED** (date)`, `**Status:** ...`, `- **Status**: ...`,
+# or bare `Status: ...`.
+_STATUS_LINE = re.compile(r"^\s*(?:>\s*)?(?:[-*]\s+)?\*{0,2}Status\*{0,2}\s*:")
+# The status only counts as "recorded at the top" when it appears early —
+# buried history (e.g. `## Status history` or a later `Status: ...` line)
+# is not the doc's current status.
+_STATUS_SCAN_LINES = 10
+_STAMP = "> **Status: COMPLETED**"
+
+
+def mark_plan_completed(plan_path: Path | str) -> None:
+    """Stamp the plan doc as COMPLETED after a successful run.
+
+    Replaces the first status line found within the top of the doc, or
+    inserts the stamp right after the `# ` title (blank separators around
+    it) — or at the very top when there is no title. Trailing-newline
+    presence is preserved."""
+    path = Path(plan_path)
+    text = path.read_text(encoding="utf-8")
+    lines = text.splitlines()
+    trailing = "\n" if text.endswith("\n") else ""
+
+    for idx in range(min(len(lines), _STATUS_SCAN_LINES)):
+        if _STATUS_LINE.match(lines[idx]):
+            lines[idx] = _STAMP
+            break
+    else:
+        title_idx = next((i for i, line in enumerate(lines)
+                          if line.startswith("# ")), None)
+        insert_at = 0 if title_idx is None else title_idx + 1
+        stamp_block = [_STAMP]
+        if insert_at > 0 and lines[insert_at - 1] != "":
+            stamp_block.insert(0, "")   # blank separator above the stamp
+        if insert_at < len(lines) and lines[insert_at] != "":
+            stamp_block.append("")      # blank separator below the stamp
+        lines[insert_at:insert_at] = stamp_block
+    path.write_text("\n".join(lines) + trailing, encoding="utf-8")
+
+
 class PlanDocumentLoader:
     name = "plan"
     defaultable = False  # needs a path — never autodetects

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/src/agentic_dev_loop.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-dev-loop
-Version: 1.0.0a12
+Version: 1.0.0a13
 Summary: Turn an LLM into a gated, sandboxed, resumable construction loop over your codebase.
 Author: Luke Wilson
 License-Expression: Apache-2.0

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/tests/test_cli.py

@@ -202,6 +202,54 @@ class TestRun:
         # dev completed before the crash — only test re-ran on resume
         assert fake_agents.agents_called() == ["dev", "check", "check"]
 
+    def test_success_stamps_plan_completed(self, repo, fake_agents):
+        fake_agents.set("dev", [("write", "implementation-summary.md", "done")])
+
+        assert main(["pipeline.yaml", "--plan", "plan.md", "--no-viz"]) == 0
+
+        assert (repo / "plan.md").read_text() == (
+            "> **Status: COMPLETED**\n\nTHE PLAN")
+
+    def test_success_replaces_existing_status(self, repo, fake_agents):
+        (repo / "plan.md").write_text(
+            "# Plan\n\n> **Status: DONE**\n\nTHE PLAN\n")
+        git(repo, "add", "-A")
+        git(repo, "commit", "-m", "record status")
+        fake_agents.set("dev", [("write", "implementation-summary.md", "done")])
+
+        assert main(["pipeline.yaml", "--plan", "plan.md", "--no-viz"]) == 0
+
+        assert (repo / "plan.md").read_text() == (
+            "# Plan\n\n> **Status: COMPLETED**\n\nTHE PLAN\n")
+
+    def test_failed_run_does_not_stamp(self, repo, fake_agents):
+        write_pipeline(repo, "retry.yaml", RETRY_YAML)
+        fake_agents.set("dev", [("write", "implementation-summary.md", "done")])
+        fake_agents.set("check", [("write", "ci-fix.md", "VERDICT: FAIL")])
+
+        assert main(["retry.yaml", "--plan", "plan.md", "--no-viz"]) == 1
+
+        assert (repo / "plan.md").read_text() == "THE PLAN"
+
+    def test_aidlc_run_leaves_docs_untouched(self, repo, fake_agents):
+        write_pipeline(repo, "aidlc.yaml", with_extensions(
+            PLAIN_YAML, "agentic_dev_loop.engine.extensions.aidlc:AidlcExtension"))
+        unit_dir = repo / "aidlc-docs" / "construction" / "unit-3"
+        unit_dir.mkdir(parents=True)
+        (unit_dir / "design.md").write_text("# Design")
+        (repo / "aidlc-docs" / "aidlc-state.md").write_text(
+            "# State\n\n- **Current Stage**: CONSTRUCTION — Unit unit-3 (SCAN).\n")
+        git(repo, "add", "-A")
+        git(repo, "commit", "-m", "aidlc fixture")
+        fake_agents.set("dev", [("write", "implementation-summary.md", "done")])
+        state_before = (repo / "aidlc-docs" / "aidlc-state.md").read_text()
+
+        assert main(["aidlc.yaml", "--aidlc-unit", "unit-3", "--no-viz"]) == 0
+
+        assert (repo / "aidlc-docs" / "aidlc-state.md").read_text() == state_before
+        assert (repo / "aidlc-docs" / "construction" / "unit-3" / "design.md").read_text() == "# Design"
+        assert (repo / "plan.md").read_text() == "THE PLAN"
+
 
 class TestViz:
     def test_no_viz_flags_parse(self, repo):

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/tests/test_isolation.py

@@ -459,6 +459,21 @@ class TestResolveSandboxVenvDir:
         assert out == str(tmp_path / "venvs" / "engine")
         assert os.path.isabs(out)
 
+    def test_home_wins_over_userprofile(self, tmp_path, monkeypatch):
+        # The engine is driven from bash, where HOME is the authoritative home
+        # on every platform — CPython's Windows expanduser ignores HOME in
+        # favour of USERPROFILE, which would resolve `~` inconsistently.
+        monkeypatch.setenv("HOME", str(tmp_path / "home"))
+        monkeypatch.setenv("USERPROFILE", str(tmp_path / "prof"))
+        out = resolve_sandbox_venv_dir("~/venvs/engine", tmp_path / "ws")
+        assert out == str(tmp_path / "home" / "venvs" / "engine")
+
+    def test_userprofile_fallback_when_home_unset(self, tmp_path, monkeypatch):
+        monkeypatch.delenv("HOME", raising=False)
+        monkeypatch.setenv("USERPROFILE", str(tmp_path / "prof"))
+        out = resolve_sandbox_venv_dir("~/venvs/engine", tmp_path / "ws")
+        assert out == str(tmp_path / "prof" / "venvs" / "engine")
+
     def test_outside_workspace_accepted(self, tmp_path):
         ws = tmp_path / "ws"
         out = resolve_sandbox_venv_dir(str(tmp_path / "venvs"), ws)

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/tests/test_isolation_integration.py

@@ -3,6 +3,7 @@ reason when no daemon or integration image is present; never silently skipped
 in CI (the workflows pre-install Docker and the image)."""
 from __future__ import annotations
 
+import os
 import shutil
 
 import pytest
@@ -249,8 +250,14 @@ class TestRelocatedVenvE2E:
         try:
             b.provision("python -m venv .venv-sandbox", "k1",
                         str(tmp_path / "stamps"), force=False)
-            assert (venv_dir / "bin" / "python").exists()
-            assert not (ws / ".venv-sandbox").exists()
+            # 022: the venv materialized OUTSIDE the workspace. bin/python is
+            # a symlink to the CONTAINER's interpreter, which dangles on the
+            # host — existence must be checked without following links.
+            assert os.path.lexists(venv_dir / "bin" / "python")
+            assert (venv_dir / "pyvenv.cfg").exists()
+            # No venv content leaked into the workspace. Docker auto-creates
+            # the empty mountpoint dir, so its mere existence is not a leak.
+            assert not (ws / ".venv-sandbox" / "pyvenv.cfg").exists()
             req = SandboxExecutionRequest(
                 argv=[".venv-sandbox/bin/python", "-c", "import sys; print('ok')"],
                 cwd=ws,

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/tests/test_loaders.py

@@ -5,7 +5,9 @@ from pathlib import Path
 
 import pytest
 
-from agentic_dev_loop.engine.loaders import PlanDocumentLoader, resolve_plan_docs, select_loader
+from agentic_dev_loop.engine.loaders import (
+    PlanDocumentLoader, mark_plan_completed, resolve_plan_docs, select_loader,
+)
 from agentic_dev_loop.engine.registry import Registry, default_registry
 
 
@@ -116,3 +118,110 @@ class TestSelectLoader:
 
         with pytest.raises(ValueError, match="[Dd]efaultable"):
             select_loader(reg, _args(a=None, b=None))
+
+
+class TestMarkPlanCompleted:
+    def _write(self, tmp_path, text):
+        plan = tmp_path / "plan.md"
+        plan.write_text(text)
+        return plan
+
+    @pytest.mark.parametrize("status_line", [
+        "> **Status: COMPLETED** (2026-10-23)",
+        "> **Status: DONE**",
+        "Status: COMPLETED — shipped",
+        "**Status:** COMPLETED (2026-10-10)",
+        "- **Status**: COMPLETED — all stages done",
+    ])
+    def test_replaces_existing_status_line(self, tmp_path, status_line):
+        plan = self._write(tmp_path, f"# Plan\n\n{status_line}\n\nBody\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+
+    def test_replaces_inline_bold_status_line_entirely(self, tmp_path):
+        """The 013-style line with trailing prose is replaced wholesale."""
+        plan = self._write(
+            tmp_path,
+            "# Plan\n\n"
+            "**Status**: Completed — shipped: ... **Status: COMPLETED** (2026-10-23).\n\n"
+            "Body\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+
+    def test_inserts_after_title_when_no_status(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\n\nBody\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+
+    def test_inserts_at_top_when_no_title(self, tmp_path):
+        plan = self._write(tmp_path, "THE PLAN")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "> **Status: COMPLETED**\n\nTHE PLAN"
+
+    def test_title_without_blank_separator_still_gets_separators(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\nBody\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+
+    def test_status_heading_is_not_treated_as_status_line(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\n\n## Status history\n\nBody\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == (
+            "# Plan\n\n> **Status: COMPLETED**\n\n## Status history\n\nBody\n")
+
+    def test_deep_status_line_is_ignored_and_stamp_inserted_after_title(self, tmp_path):
+        """Only the top of the doc is scanned; a status buried beyond the
+        first ten lines is not the 'existing status recorded' — the stamp
+        goes in after the title and the deep line stays untouched."""
+        filler = "\n".join(f"line {i}" for i in range(1, 15))
+        plan = self._write(tmp_path,
+                           f"# Plan\n\n{filler}\n> **Status: IN PROGRESS**\n")
+
+        mark_plan_completed(plan)
+
+        text = plan.read_text()
+        assert text.startswith("# Plan\n\n> **Status: COMPLETED**\n\n")
+        assert "> **Status: IN PROGRESS**" in text
+
+    def test_idempotent_rerun(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\n\n> **Status: DONE**\n\nBody\n")
+
+        mark_plan_completed(plan)
+        once = plan.read_text()
+        mark_plan_completed(plan)
+
+        assert once == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+        assert plan.read_text() == once
+
+    def test_preserves_trailing_newline_absence(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\n\nBody")  # no trailing newline
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody"
+
+    def test_preserves_trailing_newline(self, tmp_path):
+        plan = self._write(tmp_path, "# Plan\n\nBody\n")
+
+        mark_plan_completed(plan)
+
+        assert plan.read_text() == "# Plan\n\n> **Status: COMPLETED**\n\nBody\n"
+
+    def test_accepts_string_path(self, tmp_path):
+        plan = self._write(tmp_path, "THE PLAN")
+
+        mark_plan_completed(str(plan))
+
+        assert plan.read_text() == "> **Status: COMPLETED**\n\nTHE PLAN"

{agentic_dev_loop-1.0.0a12 → agentic_dev_loop-1.0.0a13}/tests/test_trigger_script.py

@@ -109,3 +109,83 @@ def test_trigger_errors_when_cli_absent():
 
     assert result.returncode != 0
     assert "pipx install" in result.stderr
+
+
+def _stub_env(tmp_path, *, cli_body='echo "cli-ran: $*"', pipx_body="exit 0"):
+    bin_dir = tmp_path / "bin"
+    bin_dir.mkdir()
+    _make_stub(bin_dir, "agentic-dev-loop", cli_body)
+    _make_stub(bin_dir, "pipx", pipx_body)
+    return bin_dir
+
+
+def test_trigger_rejects_both_flags(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("--plan", "plan.md", "--aidlc-unit", "unit-3",
+                         env=_env_with(bin_dir))
+
+    assert result.returncode != 0
+    assert ("pass exactly one of --plan <path> or --aidlc-unit <slug>, "
+            "not both") in result.stderr
+    assert "cli-ran:" not in result.stdout
+
+
+def test_trigger_requires_exactly_one_flag(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script(env=_env_with(bin_dir))
+
+    assert result.returncode != 0
+    assert ("pass exactly one of --plan <path> or --aidlc-unit <slug>"
+            in result.stderr)
+    assert "cli-ran:" not in result.stdout
+
+
+def test_trigger_forwards_aidlc_unit(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("--aidlc-unit", "unit-3", env=_env_with(bin_dir))
+
+    assert result.returncode == 0, result.stderr
+    assert "cli-ran:" in result.stdout
+    assert "--aidlc-unit unit-3" in result.stdout
+
+
+def test_trigger_unit_positional_is_aidlc_shorthand(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("unit-3", "--dry-run", env=_env_with(bin_dir))
+
+    assert result.returncode == 0, result.stderr
+    assert "--aidlc-unit unit-3" in result.stdout
+    assert "--dry-run" in result.stdout
+
+
+def test_trigger_passes_other_flags_through(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("--plan", "plan.md", "--dry-run", "--max-attempts", "3",
+                         env=_env_with(bin_dir))
+
+    assert result.returncode == 0, result.stderr
+    assert "--plan plan.md --dry-run --max-attempts 3" in result.stdout
+
+
+def test_trigger_flag_missing_value_is_usage_error(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("--plan", env=_env_with(bin_dir))
+
+    assert result.returncode != 0
+    assert "--plan requires a path" in result.stderr
+    assert "cli-ran:" not in result.stdout
+
+
+def test_trigger_equals_forms_accepted(tmp_path):
+    bin_dir = _stub_env(tmp_path)
+
+    result = _run_script("--aidlc-unit=unit-3", env=_env_with(bin_dir))
+
+    assert result.returncode == 0, result.stderr
+    assert "--aidlc-unit unit-3" in result.stdout