agentic-dev-loop 0.2.0__tar.gz → 0.3.0__tar.gz

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-dev-loop
-Version: 0.2.0
+Version: 0.3.0
 Summary: Opinionated LangGraph engine for OpenHands agent pipelines
 Author: Luke Wilson
 License-Expression: Apache-2.0
@@ -65,7 +65,7 @@ Requires Python ≥ 3.12.
 ```bash
 export OPENAI_API_KEY=...   # whatever env var your pipeline's api_key_env names
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --plan plan.md
-agentic-dev-loop agentic-dev-loop/pipeline.yaml --ai-dlc-unit U2
+agentic-dev-loop agentic-dev-loop/pipeline.yaml --aidlc-unit unit-2
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --resume <thread-id> --max-attempts 5
 ./trigger-agentic-dev-loop.sh --plan plan.md   # same thing, from the repo root
 ```
@@ -84,10 +84,12 @@ expanded and live-tails its console. To eyeball the viz page without
 running a pipeline, `.venv/bin/python viz_demo.py` serves the real topology
 and replays a scripted run (every node state, verdict badges, retry cycle).
 
-`--ai-dlc-unit` accepts any unambiguous spelling of a unit number — `U2`,
-`2`, `U002`, `002` — or the full unit slug (`U002-custom-configuration`).
-If multiple unit directories match one id, the run fails loudly naming
-every candidate; pass the full slug to disambiguate.
+`--aidlc-unit` takes the unit directory name under
+`aidlc-docs/construction/` verbatim — `unit-2`,
+`unit-002-custom-configuration`, or any slug AI-DLC's own naming uses
+(AI-DLC imposes no naming convention on unit directories). No spelling
+tolerance, no numeric aliases: the slug must name the directory exactly,
+and a wrong slug fails loudly listing the directories that do exist.
 
 ## What a run looks like
 
@@ -258,10 +260,10 @@ Loader-selection flow: a loader's flag in argv selects it; flags from two
 loaders in one invocation error out; with no flag, the sole `defaultable`
 loader autoruns (the bundled `ai-dlc` loader autodetects the current unit
 from `aidlc-docs/aidlc-state.md`), otherwise the run proceeds with no
-injected docs and the pipeline `name:` as subject. Unit ids resolve by
-exact directory name or by numeric prefix (`unit-004-preset-management`
-answers to `4`, `004`, `U4`, `U004`, or its full slug); ambiguous ids fail
-loud listing all matches. A selected loader that
+injected docs and the pipeline `name:` as subject. A unit id must name
+its directory exactly (`--aidlc-unit unit-3`); there is no numeric
+fallback or spelling tolerance, and an unmatched slug fails loud listing
+the available directories. A selected loader that
 yields zero docs **fails fast** — context is never silently degraded.
 Duplicate CLI option strings across loaders are a load-time error naming both.
 
@@ -303,7 +305,7 @@ The development lifecycle workflow lives in
 phase details under `aws-aidlc-rule-details/`). Working docs — plans, user
 stories, units of work, state — live in `aidlc-docs/`, created as the
 workflow runs. The bundled `ai-dlc` document loader (see Extension points)
-wires the pipeline's `--ai-dlc-unit` flag to those docs.
+wires the pipeline's `--aidlc-unit` flag to those docs.
 
 OpenCode commands over the workflow: `/epic-status`, `/ticket-details`,
 `/unit-status`, `/update-subtask` (ticket boards over `aidlc-docs/`),

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/README.md

@@ -27,7 +27,7 @@ Requires Python ≥ 3.12.
 ```bash
 export OPENAI_API_KEY=...   # whatever env var your pipeline's api_key_env names
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --plan plan.md
-agentic-dev-loop agentic-dev-loop/pipeline.yaml --ai-dlc-unit U2
+agentic-dev-loop agentic-dev-loop/pipeline.yaml --aidlc-unit unit-2
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --resume <thread-id> --max-attempts 5
 ./trigger-agentic-dev-loop.sh --plan plan.md   # same thing, from the repo root
 ```
@@ -46,10 +46,12 @@ expanded and live-tails its console. To eyeball the viz page without
 running a pipeline, `.venv/bin/python viz_demo.py` serves the real topology
 and replays a scripted run (every node state, verdict badges, retry cycle).
 
-`--ai-dlc-unit` accepts any unambiguous spelling of a unit number — `U2`,
-`2`, `U002`, `002` — or the full unit slug (`U002-custom-configuration`).
-If multiple unit directories match one id, the run fails loudly naming
-every candidate; pass the full slug to disambiguate.
+`--aidlc-unit` takes the unit directory name under
+`aidlc-docs/construction/` verbatim — `unit-2`,
+`unit-002-custom-configuration`, or any slug AI-DLC's own naming uses
+(AI-DLC imposes no naming convention on unit directories). No spelling
+tolerance, no numeric aliases: the slug must name the directory exactly,
+and a wrong slug fails loudly listing the directories that do exist.
 
 ## What a run looks like
 
@@ -220,10 +222,10 @@ Loader-selection flow: a loader's flag in argv selects it; flags from two
 loaders in one invocation error out; with no flag, the sole `defaultable`
 loader autoruns (the bundled `ai-dlc` loader autodetects the current unit
 from `aidlc-docs/aidlc-state.md`), otherwise the run proceeds with no
-injected docs and the pipeline `name:` as subject. Unit ids resolve by
-exact directory name or by numeric prefix (`unit-004-preset-management`
-answers to `4`, `004`, `U4`, `U004`, or its full slug); ambiguous ids fail
-loud listing all matches. A selected loader that
+injected docs and the pipeline `name:` as subject. A unit id must name
+its directory exactly (`--aidlc-unit unit-3`); there is no numeric
+fallback or spelling tolerance, and an unmatched slug fails loud listing
+the available directories. A selected loader that
 yields zero docs **fails fast** — context is never silently degraded.
 Duplicate CLI option strings across loaders are a load-time error naming both.
 
@@ -265,7 +267,7 @@ The development lifecycle workflow lives in
 phase details under `aws-aidlc-rule-details/`). Working docs — plans, user
 stories, units of work, state — live in `aidlc-docs/`, created as the
 workflow runs. The bundled `ai-dlc` document loader (see Extension points)
-wires the pipeline's `--ai-dlc-unit` flag to those docs.
+wires the pipeline's `--aidlc-unit` flag to those docs.
 
 OpenCode commands over the workflow: `/epic-status`, `/ticket-details`,
 `/unit-status`, `/update-subtask` (ticket boards over `aidlc-docs/`),

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "agentic-dev-loop"
-version = "0.2.0"
+version = "0.3.0"
 description = "Opinionated LangGraph engine for OpenHands agent pipelines"
 readme = "README.md"
 requires-python = ">=3.12"

agentic_dev_loop-0.3.0/src/agentic_dev_loop/extensions/aidlc.py

@@ -0,0 +1,124 @@
+"""AI-DLC document-loader extension. Loading IS choosing: name
+`agentic_dev_loop.extensions.aidlc:AidlcExtension` in the pipeline's
+`extensions:` list and the pipeline gains `--aidlc-unit` — the unit
+directory name under `aidlc-docs/construction/` (e.g. `unit-3`,
+`support-for-windows`), matched verbatim — plus bare-run autodetect of
+the current unit from the Current Stage line of
+`aidlc-docs/aidlc-state.md`.
+
+No input mangling. AI-DLC's own ruleset imposes no naming convention on
+unit directories (`{unit-name}` is a free placeholder), so the extension
+imposes none either: the slug is the directory name, exactly. `U3`, `3`,
+`004` and friends are NOT accepted unless a directory is literally named
+that. The Current Stage line names the unit as its directory name
+("Unit unit-3", "Unit support-for-windows") for autodetect.
+"""
+
+import argparse
+import re
+from pathlib import Path
+
+_DOC_SUFFIXES = (".md", ".yaml", ".yml", ".txt")
+
+
+def resolve_unit_docs(docs_dir: Path | str, unit_slug: str) -> dict[str, str]:
+    docs: dict[str, str] = {}
+    docs_dir = Path(docs_dir)
+    unit_slug = unit_slug.strip()
+
+    construction_dir = docs_dir / "construction"
+    unit_dir: Path | None = None
+    if construction_dir.is_dir():
+        for entry in sorted(construction_dir.iterdir()):
+            if entry.is_dir() and entry.name == unit_slug:
+                unit_dir = entry
+                break
+
+    if unit_dir is None:
+        available = ""
+        if construction_dir.is_dir():
+            names = sorted(p.name for p in construction_dir.iterdir() if p.is_dir())
+            available = f" (available: {', '.join(names)})"
+        raise FileNotFoundError(
+            f"No unit directory named '{unit_slug}' under {construction_dir}"
+            f"{available or ' (no unit directories found)'}")
+
+    for file in sorted(unit_dir.rglob("*")):
+        if file.is_file() and file.suffix in _DOC_SUFFIXES:
+            key = str(file.relative_to(unit_dir))
+            docs[key] = file.read_text()
+
+    # At this point docs holds only unit-dir files — an empty load means the
+    # pipeline would build context from inception docs alone (the 2026-07-19
+    # U3 silent-degradation bug). Fail fast instead.
+    if not any(not k.startswith("inception/") for k in docs):
+        raise RuntimeError(
+            f"Unit dir {unit_dir} matched but yielded no docs — "
+            "refusing to build context from inception docs alone")
+
+    # Namespaced keys so an inception doc can never silently overwrite a
+    # unit doc with the same filename.
+    inception = docs_dir / "inception" / "application-design"
+    if inception.is_dir():
+        for file in sorted(inception.iterdir()):
+            if file.is_file() and file.suffix in (".md", ".txt"):
+                docs[f"inception/{file.name}"] = file.read_text()
+
+    # The unit's code-generation plans are the build spec — absent is fine,
+    # present is how the spec reaches agents.
+    plans_dir = docs_dir / "construction" / "plans"
+    if plans_dir.is_dir():
+        for file in sorted(plans_dir.glob(f"{unit_slug}-*-code-generation-plan.md")):
+            docs[f"plans/{file.name}"] = file.read_text()
+
+    return docs
+
+
+def get_current_unit(docs_dir: Path | str) -> str:
+    """Read the current unit slug from the aidlc-state.md Current Stage line.
+
+    The stage line must name the unit as its directory name ("Unit unit-3",
+    "Unit support-for-windows"); the slug is returned verbatim for a direct
+    directory match.
+    """
+    state_file = Path(docs_dir) / "aidlc-state.md"
+    if not state_file.is_file():
+        raise FileNotFoundError(f"aidlc-state file not found: {state_file}")
+    # Anchor to the Current Stage line — the file accumulates history, so an
+    # unanchored search could silently pick up an earlier "Unit N" mention.
+    match = re.search(r"Current Stage[^\n]*Unit ([\w-]+(?:\.[\w-]+)*)", state_file.read_text())
+    if not match:
+        raise RuntimeError(
+            f"Could not determine current unit from {state_file} "
+            "(no 'Unit <slug>' in the Current Stage line)")
+    return match.group(1)
+
+
+class AidlcDocumentLoader:
+    """`--aidlc-unit` (a unit directory name, matched verbatim), or bare-run
+    autodetect from the Current Stage line of aidlc-state.md."""
+
+    name = "ai-dlc"
+    defaultable = True
+
+    def __init__(self, docs_dir: Path | str = "aidlc-docs"):
+        self.docs_dir = Path(docs_dir)
+
+    def add_cli_args(self, parser: argparse.ArgumentParser) -> None:
+        parser.add_argument("--aidlc-unit", metavar="SLUG", default=None,
+                        help="AI-DLC unit: the unit directory name under "
+                             "aidlc-docs/construction/ (e.g. unit-3 or "
+                             "support-for-windows), matched verbatim; "
+                             "defaults to the current unit in "
+                             "aidlc-docs/aidlc-state.md")
+
+    def resolve(self, args: argparse.Namespace) -> tuple[dict[str, str], str]:
+        unit = args.aidlc_unit or get_current_unit(self.docs_dir)
+        return resolve_unit_docs(self.docs_dir, unit), f"unit {unit}"
+
+
+class AidlcExtension:
+    name = "ai-dlc"
+
+    def register(self, registry) -> None:
+        registry.add_document_loader(AidlcDocumentLoader())

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/src/agentic_dev_loop.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-dev-loop
-Version: 0.2.0
+Version: 0.3.0
 Summary: Opinionated LangGraph engine for OpenHands agent pipelines
 Author: Luke Wilson
 License-Expression: Apache-2.0
@@ -65,7 +65,7 @@ Requires Python ≥ 3.12.
 ```bash
 export OPENAI_API_KEY=...   # whatever env var your pipeline's api_key_env names
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --plan plan.md
-agentic-dev-loop agentic-dev-loop/pipeline.yaml --ai-dlc-unit U2
+agentic-dev-loop agentic-dev-loop/pipeline.yaml --aidlc-unit unit-2
 agentic-dev-loop agentic-dev-loop/pipeline.yaml --resume <thread-id> --max-attempts 5
 ./trigger-agentic-dev-loop.sh --plan plan.md   # same thing, from the repo root
 ```
@@ -84,10 +84,12 @@ expanded and live-tails its console. To eyeball the viz page without
 running a pipeline, `.venv/bin/python viz_demo.py` serves the real topology
 and replays a scripted run (every node state, verdict badges, retry cycle).
 
-`--ai-dlc-unit` accepts any unambiguous spelling of a unit number — `U2`,
-`2`, `U002`, `002` — or the full unit slug (`U002-custom-configuration`).
-If multiple unit directories match one id, the run fails loudly naming
-every candidate; pass the full slug to disambiguate.
+`--aidlc-unit` takes the unit directory name under
+`aidlc-docs/construction/` verbatim — `unit-2`,
+`unit-002-custom-configuration`, or any slug AI-DLC's own naming uses
+(AI-DLC imposes no naming convention on unit directories). No spelling
+tolerance, no numeric aliases: the slug must name the directory exactly,
+and a wrong slug fails loudly listing the directories that do exist.
 
 ## What a run looks like
 
@@ -258,10 +260,10 @@ Loader-selection flow: a loader's flag in argv selects it; flags from two
 loaders in one invocation error out; with no flag, the sole `defaultable`
 loader autoruns (the bundled `ai-dlc` loader autodetects the current unit
 from `aidlc-docs/aidlc-state.md`), otherwise the run proceeds with no
-injected docs and the pipeline `name:` as subject. Unit ids resolve by
-exact directory name or by numeric prefix (`unit-004-preset-management`
-answers to `4`, `004`, `U4`, `U004`, or its full slug); ambiguous ids fail
-loud listing all matches. A selected loader that
+injected docs and the pipeline `name:` as subject. A unit id must name
+its directory exactly (`--aidlc-unit unit-3`); there is no numeric
+fallback or spelling tolerance, and an unmatched slug fails loud listing
+the available directories. A selected loader that
 yields zero docs **fails fast** — context is never silently degraded.
 Duplicate CLI option strings across loaders are a load-time error naming both.
 
@@ -303,7 +305,7 @@ The development lifecycle workflow lives in
 phase details under `aws-aidlc-rule-details/`). Working docs — plans, user
 stories, units of work, state — live in `aidlc-docs/`, created as the
 workflow runs. The bundled `ai-dlc` document loader (see Extension points)
-wires the pipeline's `--ai-dlc-unit` flag to those docs.
+wires the pipeline's `--aidlc-unit` flag to those docs.
 
 OpenCode commands over the workflow: `/epic-status`, `/ticket-details`,
 `/unit-status`, `/update-subtask` (ticket boards over `aidlc-docs/`),

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/src/agentic_dev_loop.egg-info/SOURCES.txt

@@ -40,6 +40,7 @@ tests/test_publish.py
 tests/test_registry.py
 tests/test_results.py
 tests/test_sdk_compat.py
+tests/test_source_resolution.py
 tests/test_stages.py
 tests/test_verdict.py
 tests/test_viz.py

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/tests/test_aidlc.py

@@ -1,7 +1,12 @@
-"""AI-DLC document loader tests — ported from yosk's TestResolveUnitDocs /
-TestGetCurrentUnit (docs_dir parameterised instead of a monkeypatched module
-constant; coding-standards cases dropped — the pipeline's `additional_files:`
-now owns those)."""
+"""AI-DLC document loader tests.
+
+The loader contract: `--aidlc-unit` takes the unit DIRECTORY NAME verbatim
+(`unit-3`, `support-for-windows`) and matches it by exact string equality
+under `aidlc-docs/construction/`. No input mangling: `U3`, `3`, `004`,
+`unit_3` and friends are NOT accepted unless a directory is literally
+named that. Bare-run autodetect reads the slug from the Current Stage
+line of `aidlc-docs/aidlc-state.md` (`Unit <slug>`).
+"""
 
 import argparse
 
@@ -23,31 +28,40 @@ class TestResolveUnitDocs:
         other_dir.mkdir(parents=True)
         (other_dir / "other.md").write_text("# Other")
 
-        docs = resolve_unit_docs(tmp_path, "2")
+        docs = resolve_unit_docs(tmp_path, "unit-2")
 
         assert "design.md" in docs
         assert "tasks.md" in docs
         assert "other.md" not in docs
 
-    def test_accepts_u_prefixed_unit_id(self, tmp_path):
+    def test_u_prefixed_id_is_not_mangled(self, tmp_path):
+        """The slug is the directory name — `U2` must NOT resolve `unit-2`."""
         unit_dir = tmp_path / "construction" / "unit-2"
         unit_dir.mkdir(parents=True)
         (unit_dir / "design.md").write_text("# Design")
 
-        assert "design.md" in resolve_unit_docs(tmp_path, "U2")
+        with pytest.raises(FileNotFoundError, match="unit-2"):
+            resolve_unit_docs(tmp_path, "U2")
 
-    def test_accepts_lowercase_u_prefixed_unit_id(self, tmp_path):
+    def test_lowercase_u_prefixed_id_is_not_mangled(self, tmp_path):
         unit_dir = tmp_path / "construction" / "unit-3"
         unit_dir.mkdir(parents=True)
         (unit_dir / "design.md").write_text("# Design")
 
-        assert "design.md" in resolve_unit_docs(tmp_path, "u3")
+        with pytest.raises(FileNotFoundError, match="unit-3"):
+            resolve_unit_docs(tmp_path, "u3")
 
-    def test_raises_for_missing_unit(self, tmp_path):
+    def test_raises_for_missing_unit_and_lists_available(self, tmp_path):
         """Unknown unit must fail fast — silently running the pipeline with
-        zero design docs is a forbidden silent fallback."""
-        with pytest.raises(FileNotFoundError, match="unit"):
-            resolve_unit_docs(tmp_path, "99")
+        zero design docs is a forbidden silent fallback. The error names the
+        directories that DO exist."""
+        (tmp_path / "construction" / "unit-1").mkdir(parents=True)
+        (tmp_path / "construction" / "unit-2").mkdir(parents=True)
+
+        with pytest.raises(FileNotFoundError, match="unit-99") as exc:
+            resolve_unit_docs(tmp_path, "unit-99")
+        assert "unit-1" in str(exc.value)
+        assert "unit-2" in str(exc.value)
 
     def test_includes_inception_docs(self, tmp_path):
         unit_dir = tmp_path / "construction" / "unit-1"
@@ -57,7 +71,7 @@ class TestResolveUnitDocs:
         inception.mkdir(parents=True)
         (inception / "architecture.md").write_text("# Arch")
 
-        docs = resolve_unit_docs(tmp_path, "1")
+        docs = resolve_unit_docs(tmp_path, "unit-1")
 
         assert "inception/architecture.md" in docs
         assert "Arch" in docs["inception/architecture.md"]
@@ -72,7 +86,7 @@ class TestResolveUnitDocs:
         inception.mkdir(parents=True)
         (inception / "architecture.md").write_text("inception version")
 
-        docs = resolve_unit_docs(tmp_path, "1")
+        docs = resolve_unit_docs(tmp_path, "unit-1")
 
         assert docs["architecture.md"] == "unit version"
         assert docs["inception/architecture.md"] == "inception version"
@@ -85,7 +99,7 @@ class TestResolveUnitDocs:
         (unit_dir / "functional-design").mkdir(parents=True)
         (unit_dir / "functional-design" / "business-rules.md").write_text("# Rules")
 
-        docs = resolve_unit_docs(tmp_path, "U3")
+        docs = resolve_unit_docs(tmp_path, "unit-3")
 
         assert docs["functional-design/business-rules.md"] == "# Rules"
 
@@ -95,7 +109,7 @@ class TestResolveUnitDocs:
         (tmp_path / "construction" / "unit-3" / "functional-design").mkdir(parents=True)
 
         with pytest.raises(RuntimeError, match="no.*docs|zero"):
-            resolve_unit_docs(tmp_path, "U3")
+            resolve_unit_docs(tmp_path, "unit-3")
 
     def test_includes_code_generation_plans(self, tmp_path):
         """The unit's *-code-generation-plan.md files are the build spec."""
@@ -109,7 +123,7 @@ class TestResolveUnitDocs:
         (plans / "unit-3-scan-nfr-design-plan.md").write_text("# NOT a code-gen plan")
         (plans / "unit-2-onboarding-code-generation-plan.md").write_text("# Wrong unit")
 
-        docs = resolve_unit_docs(tmp_path, "U3")
+        docs = resolve_unit_docs(tmp_path, "unit-3")
 
         assert docs["plans/unit-3-scan-core-types-code-generation-plan.md"] == "# Core types"
         assert docs["plans/unit-3-scan-frontend-code-generation-plan.md"] == "# Frontend"
@@ -121,7 +135,44 @@ class TestResolveUnitDocs:
         unit_dir.mkdir(parents=True)
         (unit_dir / "design.md").write_text("# Design")
 
-        assert resolve_unit_docs(tmp_path, "U2")["design.md"] == "# Design"
+        assert resolve_unit_docs(tmp_path, "unit-2")["design.md"] == "# Design"
+
+
+class TestDirectSlugMatching:
+    """The slug IS the directory name — no spelling tolerance, no numeric
+    fallback, no prefix mangling."""
+
+    def _unit(self, tmp_path, name):
+        unit_dir = tmp_path / "construction" / name
+        unit_dir.mkdir(parents=True)
+        (unit_dir / "design.md").write_text("# Design")
+        return unit_dir
+
+    def test_slug_matches_directory_exactly(self, tmp_path):
+        self._unit(tmp_path, "unit-004-preset-management")
+        assert resolve_unit_docs(tmp_path, "unit-004-preset-management")["design.md"] == "# Design"
+
+    def test_bare_slug_directory(self, tmp_path):
+        self._unit(tmp_path, "support-for-windows")
+        assert resolve_unit_docs(tmp_path, "support-for-windows")["design.md"] == "# Design"
+
+    def test_number_alone_does_not_match_slugged_dir(self, tmp_path):
+        """The numeric fallback is gone: `004` must not resolve
+        `unit-004-preset-management`."""
+        self._unit(tmp_path, "unit-004-preset-management")
+        with pytest.raises(FileNotFoundError, match="004"):
+            resolve_unit_docs(tmp_path, "004")
+
+    def test_underscore_spelling_does_not_match(self, tmp_path):
+        """The `unit_<id>` variant is gone: the slug must spell the
+        directory exactly."""
+        self._unit(tmp_path, "unit-3")
+        with pytest.raises(FileNotFoundError, match="unit-3"):
+            resolve_unit_docs(tmp_path, "unit_3")
+
+    def test_non_numeric_slug(self, tmp_path):
+        self._unit(tmp_path, "unit-scan")
+        assert resolve_unit_docs(tmp_path, "unit-scan")["design.md"] == "# Design"
 
 
 class TestGetCurrentUnit:
@@ -130,16 +181,22 @@ class TestGetCurrentUnit:
         state.write_text(f"# AI-DLC State Tracking\n\n{stage_line}\n")
         return state
 
-    def test_parses_unit_from_current_stage(self, tmp_path):
+    def test_parses_slug_from_current_stage(self, tmp_path):
+        self._write_state(
+            tmp_path,
+            "- **Current Stage**: CONSTRUCTION — Unit unit-3 (SCAN) — Code Generation in progress.")
+        assert get_current_unit(tmp_path) == "unit-3"
+
+    def test_parses_bare_slug_from_current_stage(self, tmp_path):
         self._write_state(
             tmp_path,
-            "- **Current Stage**: CONSTRUCTION — Unit 3 (SCAN) — Code Generation in progress.")
-        assert get_current_unit(tmp_path) == "U3"
+            "- **Current Stage**: CONSTRUCTION — Unit support-for-windows.")
+        assert get_current_unit(tmp_path) == "support-for-windows"
 
-    def test_parses_different_unit_number(self, tmp_path):
+    def test_parses_different_slug(self, tmp_path):
         self._write_state(
-            tmp_path, "- **Current Stage**: CONSTRUCTION — Unit 12 (REPORT).")
-        assert get_current_unit(tmp_path) == "U12"
+            tmp_path, "- **Current Stage**: CONSTRUCTION — Unit unit-12 (REPORT).")
+        assert get_current_unit(tmp_path) == "unit-12"
 
     def test_ignores_unit_mentions_before_the_stage_line(self, tmp_path):
         """The state file accumulates history; only the Current Stage line
@@ -148,8 +205,8 @@ class TestGetCurrentUnit:
         state.write_text(
             "# AI-DLC State Tracking\n\n"
             "- Finished Unit 9 (REPORT) last week.\n"
-            "- **Current Stage**: CONSTRUCTION — Unit 3 (SCAN) — Code Generation.\n")
-        assert get_current_unit(tmp_path) == "U3"
+            "- **Current Stage**: CONSTRUCTION — Unit unit-3 (SCAN) — Code Generation.\n")
+        assert get_current_unit(tmp_path) == "unit-3"
 
     def test_stage_line_without_unit_raises_despite_earlier_mention(self, tmp_path):
         """A 'Unit N' somewhere else in the file is not a Current Stage —
@@ -182,108 +239,43 @@ class TestAidlcDocumentLoader:
     def test_is_defaultable(self):
         assert AidlcDocumentLoader.defaultable is True
 
-    def test_resolve_with_explicit_unit_normalises_subject(self, tmp_path):
+    def test_resolve_with_explicit_slug(self, tmp_path):
         self._unit_docs(tmp_path)
         loader = AidlcDocumentLoader(docs_dir=tmp_path)
 
-        docs, subject = loader.resolve(argparse.Namespace(ai_dlc_unit="u3"))
+        docs, subject = loader.resolve(argparse.Namespace(aidlc_unit="unit-3"))
 
         assert docs["design.md"] == "# Design"
-        assert subject == "unit U3"
+        assert subject == "unit unit-3"
 
     def test_resolve_without_unit_autodetects_current(self, tmp_path):
         self._unit_docs(tmp_path)
         (tmp_path / "aidlc-state.md").write_text(
-            "# State\n\n- **Current Stage**: CONSTRUCTION — Unit 3 (SCAN).\n")
+            "# State\n\n- **Current Stage**: CONSTRUCTION — Unit unit-3 (SCAN).\n")
         loader = AidlcDocumentLoader(docs_dir=tmp_path)
 
-        docs, subject = loader.resolve(argparse.Namespace(ai_dlc_unit=None))
+        docs, subject = loader.resolve(argparse.Namespace(aidlc_unit=None))
 
         assert docs["design.md"] == "# Design"
-        assert subject == "unit U3"
-
-    def test_extension_registers_loader_and_flag(self):
-        reg = Registry()
-        AidlcExtension().register(reg)
-
-        assert [l.name for l in reg.document_loaders] == ["ai-dlc"]
-        assert reg.cli_option_owners["--ai-dlc-unit"] == "ai-dlc"
-        assert reg.cli_dests["ai_dlc_unit"] == "ai-dlc"
-
+        assert subject == "unit unit-3"
 
-class TestResolveUnitDocsFlexibleUnitIds:
-    """Numeric tolerance: U004 / 004 / 4 / U4 must all resolve the same unit.
-
-    Real-world unit dirs are named unit-NNN-slug (e.g.
-    unit-004-preset-management); the loader must accept any unambiguous
-    spelling of the unit number and fail loud on ambiguity."""
-
-    def _unit(self, tmp_path, name):
-        unit_dir = tmp_path / "construction" / name
-        unit_dir.mkdir(parents=True)
-        (unit_dir / "design.md").write_text("# Design")
-        return unit_dir
-
-    @pytest.mark.parametrize("unit_id", [
-        "4", "U4", "u4", "004", "U004",
-        "004-preset-management", "U004-preset-management",
-    ])
-    def test_accepts_all_spellings_for_slugged_dir(self, tmp_path, unit_id):
-        self._unit(tmp_path, "unit-004-preset-management")
-        assert resolve_unit_docs(tmp_path, unit_id)["design.md"] == "# Design"
-
-    def test_plain_number_matches_underscore_dir_form(self, tmp_path):
-        self._unit(tmp_path, "unit_004_presets")
-        assert resolve_unit_docs(tmp_path, "4")["design.md"] == "# Design"
-
-    def test_higher_number_does_not_match(self, tmp_path):
-        self._unit(tmp_path, "unit-40")
-        with pytest.raises(FileNotFoundError, match="unit"):
-            resolve_unit_docs(tmp_path, "4")
-
-    def test_zero_padded_plain_dir(self, tmp_path):
-        self._unit(tmp_path, "unit-004")
-        assert resolve_unit_docs(tmp_path, "U4")["design.md"] == "# Design"
-
-    def test_ambiguous_number_raises_loud(self, tmp_path):
-        self._unit(tmp_path, "unit-4")
-        self._unit(tmp_path, "unit-004-preset-management")
-        with pytest.raises(RuntimeError, match="[Aa]mbiguous") as exc:
-            resolve_unit_docs(tmp_path, "4")
-        assert "unit-4" in str(exc.value)
-        assert "unit-004-preset-management" in str(exc.value)
-
-    def test_ambiguous_u_prefixed_number_raises_loud(self, tmp_path):
-        self._unit(tmp_path, "unit-004")
-        self._unit(tmp_path, "unit-004-preset-management")
-        with pytest.raises(RuntimeError, match="[Aa]mbiguous"):
-            resolve_unit_docs(tmp_path, "U004")
-
-    def test_non_numeric_id_still_uses_exact_match(self, tmp_path):
-        self._unit(tmp_path, "unit-scan")
-        assert resolve_unit_docs(tmp_path, "scan")["design.md"] == "# Design"
-
-
-class TestFlexibleIdsViaLoader:
-    def test_autodetect_resolves_slugged_zero_padded_dir(self, tmp_path):
+    def test_autodetect_resolves_slugged_dir(self, tmp_path):
         unit_dir = tmp_path / "construction" / "unit-004-preset-management"
         unit_dir.mkdir(parents=True)
         (unit_dir / "design.md").write_text("# Design")
         (tmp_path / "aidlc-state.md").write_text(
-            "# State\n\n- **Current Stage**: CONSTRUCTION — Unit 004 ADL-01.\n")
+            "# State\n\n- **Current Stage**: CONSTRUCTION — Unit unit-004-preset-management ADL-01.\n")
         loader = AidlcDocumentLoader(docs_dir=tmp_path)
 
-        docs, subject = loader.resolve(argparse.Namespace(ai_dlc_unit=None))
+        docs, subject = loader.resolve(argparse.Namespace(aidlc_unit=None))
 
         assert docs["design.md"] == "# Design"
-        assert subject == "unit U004"
-
-    def test_plain_number_subject(self, tmp_path):
-        unit_dir = tmp_path / "construction" / "unit-4"
-        unit_dir.mkdir(parents=True)
-        (unit_dir / "design.md").write_text("# Design")
-        loader = AidlcDocumentLoader(docs_dir=tmp_path)
+        assert subject == "unit unit-004-preset-management"
 
-        docs, subject = loader.resolve(argparse.Namespace(ai_dlc_unit="4"))
+    def test_extension_registers_loader_and_flag(self):
+        reg = Registry()
+        AidlcExtension().register(reg)
 
-        assert subject == "unit U4"
+        assert [l.name for l in reg.document_loaders] == ["ai-dlc"]
+        assert reg.cli_option_owners["--aidlc-unit"] == "ai-dlc"
+        assert reg.cli_dests["aidlc_unit"] == "ai-dlc"

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/tests/test_cli.py

@@ -120,11 +120,11 @@ class TestHelp:
         with pytest.raises(SystemExit) as e:
             main(["aidlc.yaml", "--help"])
         assert e.value.code == 0
-        assert "--ai-dlc-unit" in capsys.readouterr().out
+        assert "--aidlc-unit" in capsys.readouterr().out
 
         with pytest.raises(SystemExit):
             main(["pipeline.yaml", "--help"])
-        assert "--ai-dlc-unit" not in capsys.readouterr().out
+        assert "--aidlc-unit" not in capsys.readouterr().out
 
     def test_no_pipeline_shows_core_surface(self, repo, capsys):
         with pytest.raises(SystemExit) as e:

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/tests/test_e2e_fixtures.py

@@ -184,7 +184,7 @@ class TestHistoricalFixtures:
          / "unit-99").mkdir(parents=True)
         with pytest.raises(RuntimeError, match="yielded no docs"):
             main([str(consumer_repo / "agentic-dev-loop" / "pipeline.yaml"),
-                  "--ai-dlc-unit", "U99", "--no-viz"])
+                  "--aidlc-unit", "unit-99", "--no-viz"])
         assert fake_agents.calls == []
 
 

{agentic_dev_loop-0.2.0 → agentic_dev_loop-0.3.0}/tests/test_loaders.py

@@ -81,28 +81,28 @@ class TestSelectLoader:
 
     def test_flag_selects_its_loader(self):
         plan = _Loader("plan", "--plan", "plan")
-        aidlc = _Loader("ai-dlc", "--ai-dlc-unit", "ai_dlc_unit", defaultable=True)
+        aidlc = _Loader("ai-dlc", "--aidlc-unit", "aidlc_unit", defaultable=True)
         reg = self._registry_with(plan, aidlc)
 
-        assert select_loader(reg, _args(plan=Path("p.md"), ai_dlc_unit=None)) is plan
-        assert select_loader(reg, _args(plan=None, ai_dlc_unit="U3")) is aidlc
+        assert select_loader(reg, _args(plan=Path("p.md"), aidlc_unit=None)) is plan
+        assert select_loader(reg, _args(plan=None, aidlc_unit="U3")) is aidlc
 
     def test_flags_from_two_loaders_error_naming_both(self):
         reg = self._registry_with(
             _Loader("plan", "--plan", "plan"),
-            _Loader("ai-dlc", "--ai-dlc-unit", "ai_dlc_unit", defaultable=True))
+            _Loader("ai-dlc", "--aidlc-unit", "aidlc_unit", defaultable=True))
 
         with pytest.raises(ValueError) as excinfo:
-            select_loader(reg, _args(plan=Path("p.md"), ai_dlc_unit="U3"))
+            select_loader(reg, _args(plan=Path("p.md"), aidlc_unit="U3"))
 
         message = str(excinfo.value)
         assert "plan" in message and "ai-dlc" in message
 
     def test_sole_defaultable_autoruns_with_no_flag(self):
-        aidlc = _Loader("ai-dlc", "--ai-dlc-unit", "ai_dlc_unit", defaultable=True)
+        aidlc = _Loader("ai-dlc", "--aidlc-unit", "aidlc_unit", defaultable=True)
         reg = self._registry_with(_Loader("plan", "--plan", "plan"), aidlc)
 
-        assert select_loader(reg, _args(plan=None, ai_dlc_unit=None)) is aidlc
+        assert select_loader(reg, _args(plan=None, aidlc_unit=None)) is aidlc
 
     def test_none_defaultable_returns_none_with_no_flag(self):
         reg = self._registry_with(_Loader("plan", "--plan", "plan"))

agentic_dev_loop-0.3.0/tests/test_source_resolution.py

@@ -0,0 +1,23 @@
+"""The suite must exercise the working tree, not an installed wheel.
+
+This repo dogfoods its own published wheel in .venv (deliberate). Plain
+pytest would otherwise import the published code and silently test stale
+behaviour. This test pins the import to src/ — it fails the moment the
+conftest sys.path insertion stops working.
+"""
+
+from pathlib import Path
+
+import agentic_dev_loop
+
+REPO_SRC = Path(__file__).resolve().parents[1] / "src"
+
+
+def test_import_resolves_to_working_tree():
+    resolved = Path(agentic_dev_loop.__file__).resolve()
+    assert str(resolved).startswith(
+        str(REPO_SRC) + "/"
+    ), (
+        f"agentic_dev_loop imported from {resolved}, not the working tree "
+        f"({REPO_SRC}) — conftest's src/ sys.path insertion is not effective"
+    )

agentic_dev_loop-0.2.0/src/agentic_dev_loop/extensions/aidlc.py

@@ -1,152 +0,0 @@
-"""AI-DLC document-loader extension. Loading IS choosing: name
-`agentic_dev_loop.extensions.aidlc:AidlcExtension` in the pipeline's
-`extensions:` list and the pipeline gains `--ai-dlc-unit` (any
-unambiguous unit spelling: U4 / 4 / 004 / U004 / full slug) plus bare-run
-autodetect of the current unit from `aidlc-docs/aidlc-state.md`.
-
-Resolvers are verbatim ports of yosk's `resolve_unit_docs` /
-`get_current_unit`, with `docs_dir` parameterised (was the module-level
-AIDLC_DOCS_DIR) and the coding-standards merge dropped (the pipeline's
-`additional_files:` now owns shared files).
-"""
-
-import argparse
-import re
-from pathlib import Path
-
-_DOC_SUFFIXES = (".md", ".yaml", ".yml", ".txt")
-_UNIT_DIR_NUMBER = re.compile(r"unit[-_](\d+)")
-
-
-def _leading_number(text: str) -> int | None:
-    """Leading digits of a unit id (``"004-preset-management"`` -> 4), or
-    None for ids that do not start with a number."""
-    match = re.match(r"(\d+)", text)
-    return int(match.group(1)) if match else None
-
-
-def _dir_unit_number(dir_name: str) -> int | None:
-    """Numeric prefix of a unit dir name (``"unit-004-preset-management"``
-    -> 4, ``"unit-14"`` -> 14)."""
-    match = _UNIT_DIR_NUMBER.match(dir_name.lower())
-    return int(match.group(1)) if match else None
-
-
-def resolve_unit_docs(docs_dir: Path | str, unit_id: str) -> dict[str, str]:
-    docs: dict[str, str] = {}
-    docs_dir = Path(docs_dir)
-    unit_id = unit_id.strip()
-    if unit_id[:1] in ("U", "u"):
-        unit_id = unit_id[1:]
-
-    unit_patterns = [f"unit-{unit_id}", f"unit_{unit_id}"]
-    number = _leading_number(unit_id)
-    construction_dir = docs_dir / "construction"
-    candidates: list[Path] = []
-    if construction_dir.is_dir():
-        for entry in sorted(construction_dir.iterdir()):
-            if not entry.is_dir():
-                continue
-            name = entry.name.lower()
-            if name == unit_patterns[0].lower() or name == unit_patterns[1].lower():
-                candidates.append(entry)
-            elif number is not None and _dir_unit_number(entry.name) == number:
-                candidates.append(entry)
-
-    candidates = sorted(set(candidates), key=lambda p: p.name.lower())
-    if len(candidates) > 1:
-        raise RuntimeError(
-            f"Ambiguous unit id '{unit_id}': multiple unit directories match "
-            f"({', '.join(p.name for p in candidates)}). Pass the full unit "
-            "slug to disambiguate.")
-
-    unit_dir = candidates[0] if candidates else None
-    if unit_dir is None:
-        look_for = ", ".join(unit_patterns)
-        if number is not None:
-            look_for += f", or any unit dir numbered {number}"
-        raise FileNotFoundError(
-            f"No docs directory found for unit {unit_id} under {construction_dir} "
-            f"(looked for {look_for})")
-
-    for file in sorted(unit_dir.rglob("*")):
-        if file.is_file() and file.suffix in _DOC_SUFFIXES:
-            key = str(file.relative_to(unit_dir))
-            docs[key] = file.read_text()
-
-    # At this point docs holds only unit-dir files — an empty load means the
-    # pipeline would build context from inception docs alone (the 2026-07-19
-    # U3 silent-degradation bug). Fail fast instead.
-    if not any(not k.startswith("inception/") for k in docs):
-        raise RuntimeError(
-            f"Unit dir {unit_dir} matched but yielded no docs — "
-            "refusing to build context from inception docs alone")
-
-    # Namespaced keys so an inception doc can never silently overwrite a
-    # unit doc with the same filename.
-    inception = docs_dir / "inception" / "application-design"
-    if inception.is_dir():
-        for file in sorted(inception.iterdir()):
-            if file.is_file() and file.suffix in (".md", ".txt"):
-                docs[f"inception/{file.name}"] = file.read_text()
-
-    # The unit's code-generation plans are the build spec — absent is fine,
-    # present is how the spec reaches agents.
-    plans_dir = docs_dir / "construction" / "plans"
-    if plans_dir.is_dir():
-        for file in sorted(plans_dir.glob(f"unit-{unit_id}-*-code-generation-plan.md")):
-            docs[f"plans/{file.name}"] = file.read_text()
-
-    return docs
-
-
-def get_current_unit(docs_dir: Path | str) -> str:
-    """Read the current unit from the aidlc-state.md Current Stage line."""
-    state_file = Path(docs_dir) / "aidlc-state.md"
-    if not state_file.is_file():
-        raise FileNotFoundError(f"aidlc-state file not found: {state_file}")
-    # Anchor to the Current Stage line — the file accumulates history, so an
-    # unanchored search could silently pick up an earlier "Unit N" mention.
-    match = re.search(r"Current Stage[^\n]*Unit (\d+)", state_file.read_text())
-    if not match:
-        raise RuntimeError(
-            f"Could not determine current unit from {state_file} "
-            "(no 'Unit N' in the Current Stage line)")
-    return f"U{match.group(1)}"
-
-
-def _normalize_unit(unit: str) -> str:
-    unit = unit.strip()
-    if unit[:1] in ("U", "u"):
-        unit = unit[1:]
-    return f"U{unit}"
-
-
-class AidlcDocumentLoader:
-    """`--ai-dlc-unit` (any unambiguous spelling), or bare-run
-    autodetect from aidlc-state.md."""
-
-    name = "ai-dlc"
-    defaultable = True
-
-    def __init__(self, docs_dir: Path | str = "aidlc-docs"):
-        self.docs_dir = Path(docs_dir)
-
-    def add_cli_args(self, parser: argparse.ArgumentParser) -> None:
-        parser.add_argument("--ai-dlc-unit", metavar="U<N>", default=None,
-                        help="AI-DLC unit ID: any unambiguous spelling "
-                             "(U4, 4, 004, U004) or the full unit slug "
-                             "(U004-preset-management); defaults to the "
-                             "current unit in aidlc-docs/aidlc-state.md")
-
-    def resolve(self, args: argparse.Namespace) -> tuple[dict[str, str], str]:
-        unit = (_normalize_unit(args.ai_dlc_unit) if args.ai_dlc_unit
-                else get_current_unit(self.docs_dir))
-        return resolve_unit_docs(self.docs_dir, unit), f"unit {unit}"
-
-
-class AidlcExtension:
-    name = "ai-dlc"
-
-    def register(self, registry) -> None:
-        registry.add_document_loader(AidlcDocumentLoader())