@xenonbyte/req-2-plan 0.4.2 → 0.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xenonbyte/req-2-plan",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Requirement-to-PLAN workflow CLI and agent integration installer.",
   "bin": {
     "r2p": "bin/r2p.js"

package/tools/workflow_cli/context_pack.py

@@ -6,6 +6,11 @@ import os
 from dataclasses import asdict, dataclass, field
 from pathlib import Path
 
+try:
+    import tomllib  # Python 3.11+
+except ImportError:  # pragma: no cover - older interpreters: pyproject parsing degrades to a no-op
+    tomllib = None
+
 from tools.workflow_cli.repo_baseline import SKIP_DIRS, scan_repo_baseline
 
 _CONFIG_NAMES = {
@@ -39,6 +44,41 @@ def _append_npm_dependencies(pack: ProjectContextPack, dependencies: object, *,
         pack.dependencies.append(dep)
 
 
+def _append_pyproject_facts(pack: ProjectContextPack, repo_path: Path) -> None:
+    """Collect PEP 621 [project] dependencies and a pytest signal from pyproject.toml.
+
+    Poetry/PDM private tables are out of scope; without tomllib (Python < 3.11)
+    this is a no-op, matching the pack's best-effort scan semantics."""
+    pyproject = repo_path / "pyproject.toml"
+    if tomllib is None or not pyproject.exists():
+        return
+    try:
+        data = tomllib.loads(pyproject.read_text(encoding="utf-8"))
+    except (tomllib.TOMLDecodeError, OSError, UnicodeDecodeError):
+        return
+    project = data.get("project")
+    project = project if isinstance(project, dict) else {}
+    raw_deps = project.get("dependencies")
+    specs = [s for s in raw_deps if isinstance(s, str)] if isinstance(raw_deps, list) else []
+    optional = project.get("optional-dependencies")
+    optional_specs: list[str] = []
+    if isinstance(optional, dict):
+        for group in optional.values():
+            if isinstance(group, list):
+                optional_specs.extend(s for s in group if isinstance(s, str))
+    if (specs or optional_specs) and "pip" not in pack.package_managers:
+        pack.package_managers.append("pip")
+    for spec in specs:
+        pack.dependencies.append({"name": spec, "version": "", "ecosystem": "pip"})
+    for spec in optional_specs:
+        pack.dependencies.append({"name": spec, "version": "", "ecosystem": "pip", "dev": True})
+    tool = data.get("tool")
+    has_pytest_config = isinstance(tool, dict) and isinstance(tool.get("pytest"), dict)
+    mentions_pytest = any(s.lower().startswith("pytest") for s in specs + optional_specs)
+    if (has_pytest_config or mentions_pytest) and "python -m pytest" not in pack.test_commands:
+        pack.test_commands.append("python -m pytest")
+
+
 def build_context_pack(repo_path: Path) -> ProjectContextPack:
     repo_path = Path(repo_path).resolve()
     baseline = scan_repo_baseline(repo_path)
@@ -68,6 +108,8 @@ def build_context_pack(repo_path: Path) -> ProjectContextPack:
         if not pack.test_commands:
             pack.test_commands.append("python -m pytest")
 
+    _append_pyproject_facts(pack, repo_path)
+
     for root, dirs, files in os.walk(repo_path):
         dirs[:] = [d for d in dirs if d not in SKIP_DIRS and not d.startswith(".")]
         rel_root = Path(root).relative_to(repo_path)

package/tools/workflow_cli/gates.py

@@ -431,13 +431,14 @@ def _check_plan_context_pack(run_dir: Path) -> list[str]:
 
 def _check_plan_file_refs(run_dir: Path, content: str) -> list[str]:
     """Hard-check Files paths against the Context Pack repo_root. create-type tasks
-    are exempt; the part after '::' (a symbol) is advisory and not checked (no AST pack yet)."""
+    must target paths that do not exist yet (R19); the part after '::' (a symbol) is
+    advisory and not checked (no AST pack yet)."""
     repo_root = _context_pack_repo_root(run_dir)
     if repo_root is None:
         return []  # no usable ground truth; standard tier blocks via _check_plan_context_pack
     issues: list[str] = []
     for body in _iter_plan_task_bodies(content):
-        skip_missing_path = _normalized_change_type(_task_change_type(body)) == "create"
+        is_create = _normalized_change_type(_task_change_type(body)) == "create"
         files_field = _plan_task_field_body(body, "Files")
         for path_part in _plan_task_file_paths(files_field):
             path = Path(path_part)
@@ -453,11 +454,18 @@ def _check_plan_file_refs(run_dir: Path, content: str) -> list[str]:
                 )
                 continue
             if not resolved.exists():
-                if not skip_missing_path:
+                if not is_create:
                     issues.append(
                         f"PLAN-TASK Files references missing path {path_part!r} "
                         "(mark the task 'Change Type: create' if it is a new file)."
                     )
+            elif is_create:
+                # R19: create must not silently mean "overwrite an existing file".
+                issues.append(
+                    f"PLAN-TASK Files references path {path_part!r} that already exists "
+                    "under 'Change Type: create'; use 'modify' for existing files or "
+                    "split the task."
+                )
     return issues
 
 

package/tools/workflow_cli/version.py

@@ -1 +1 @@
-R2P_VERSION = "0.4.2"
+R2P_VERSION = "0.4.3"