agentic-dev-loop 0.2.0__py3-none-any.whl → 0.3.0__py3-none-any.whl

agentic_dev_loop/extensions/aidlc.py

@@ -1,13 +1,17 @@
 """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).
+`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
@@ -15,59 +19,29 @@ 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]:
+def resolve_unit_docs(docs_dir: Path | str, unit_slug: 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_slug = unit_slug.strip()
 
-    unit_patterns = [f"unit-{unit_id}", f"unit_{unit_id}"]
-    number = _leading_number(unit_id)
     construction_dir = docs_dir / "construction"
-    candidates: list[Path] = []
+    unit_dir: Path | None = None
     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.")
+            if entry.is_dir() and entry.name == unit_slug:
+                unit_dir = entry
+                break
 
-    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}"
+        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 docs directory found for unit {unit_id} under {construction_dir} "
-            f"(looked for {look_for})")
+            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:
@@ -94,37 +68,35 @@ def resolve_unit_docs(docs_dir: Path | str, unit_id: str) -> dict[str, str]:
     # 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")):
+        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 from the aidlc-state.md Current Stage line."""
+    """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 (\d+)", state_file.read_text())
+    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 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}"
+            "(no 'Unit <slug>' in the Current Stage line)")
+    return match.group(1)
 
 
 class AidlcDocumentLoader:
-    """`--ai-dlc-unit` (any unambiguous spelling), or bare-run
-    autodetect from aidlc-state.md."""
+    """`--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
@@ -133,15 +105,15 @@ class AidlcDocumentLoader:
         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")
+        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 = (_normalize_unit(args.ai_dlc_unit) if args.ai_dlc_unit
-                else get_current_unit(self.docs_dir))
+        unit = args.aidlc_unit or get_current_unit(self.docs_dir)
         return resolve_unit_docs(self.docs_dir, unit), f"unit {unit}"
 
 

{agentic_dev_loop-0.2.0.dist-info → agentic_dev_loop-0.3.0.dist-info}/METADATA

@@ -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.dist-info → agentic_dev_loop-0.3.0.dist-info}/RECORD

@@ -14,10 +14,10 @@ agentic_dev_loop/results.py,sha256=wVB8k9dDgiLbfGxR7N2gaeX4eAMz6bIj19JRjXfe84E,1
 agentic_dev_loop/stages.py,sha256=Ku9aku90T1S6MPn_wqgZNqLCswkGyQu5LQtwYTHmw1g,19109
 agentic_dev_loop/viz.py,sha256=eMgfLBEssorTygdzNDqy1VLVxqw4uWjyxLJtcBBQWSI,42633
 agentic_dev_loop/extensions/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-agentic_dev_loop/extensions/aidlc.py,sha256=sE1DqJjRxfnvkiefyMwWjn1VaYJoA6Ufab385sscHQg,6195
-agentic_dev_loop-0.2.0.dist-info/licenses/LICENSE,sha256=xx0jnfkXJvxRnG63LTGOxlggYnIysveWIZ6H3PNdCrQ,11357
-agentic_dev_loop-0.2.0.dist-info/METADATA,sha256=n3C_ChRzWVaL6Rgl0-NiGhcmhxBm2q0KZh9dgOw2i-8,14833
-agentic_dev_loop-0.2.0.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
-agentic_dev_loop-0.2.0.dist-info/entry_points.txt,sha256=9i9TjC5AHjEfv7nM6RFdf06Okr9cj9Cgdfrd7OWSg5g,63
-agentic_dev_loop-0.2.0.dist-info/top_level.txt,sha256=n1ktP1gXxGN6DJAnACR0mt276GRJB9PVDezf9DVPwaw,17
-agentic_dev_loop-0.2.0.dist-info/RECORD,,
+agentic_dev_loop/extensions/aidlc.py,sha256=S9sUISIwjGJ9HOC3JFnrnPVHcxvW6E2kYkjdkILePH8,5277
+agentic_dev_loop-0.3.0.dist-info/licenses/LICENSE,sha256=xx0jnfkXJvxRnG63LTGOxlggYnIysveWIZ6H3PNdCrQ,11357
+agentic_dev_loop-0.3.0.dist-info/METADATA,sha256=jUdGT4sIvnUWyhdRQdH-egLUASQX0TauvwsqZYPz3wY,14937
+agentic_dev_loop-0.3.0.dist-info/WHEEL,sha256=YVMoNqKzERt-wjUZwJ33xBGAwnFl-4cqbYkTtWa4itE,91
+agentic_dev_loop-0.3.0.dist-info/entry_points.txt,sha256=9i9TjC5AHjEfv7nM6RFdf06Okr9cj9Cgdfrd7OWSg5g,63
+agentic_dev_loop-0.3.0.dist-info/top_level.txt,sha256=n1ktP1gXxGN6DJAnACR0mt276GRJB9PVDezf9DVPwaw,17
+agentic_dev_loop-0.3.0.dist-info/RECORD,,