@xenonbyte/req-2-plan 0.4.1 → 0.4.2

package/package.json

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

package/tools/workflow_cli/agent_templates/claude/commands/r2p-start.md

@@ -3,7 +3,7 @@ description: Start a new requirement-to-PLAN workflow run
 ---
 Run `{{R2P_BIN_DIR}}/r2p-start` with the requirement as a positional argument.
 
-Usage: `{{R2P_BIN_DIR}}/r2p-start [--separate] ("<raw requirement>" | --file <path>)`
+Usage: `{{R2P_BIN_DIR}}/r2p-start [--separate] [--repo-path <dir>] ("<raw requirement>" | --file <path>)`
 
 To start from a requirement document, pass `--file <path>` instead of inline text — r2p reads the file contents as the requirement (the path itself is never stored as the requirement): `{{R2P_BIN_DIR}}/r2p-start [--separate] --file ./requirement.md`. `--file` and a positional requirement are mutually exclusive.
 

package/tools/workflow_cli/agent_templates/codex/skills/r2p-start/SKILL.md

@@ -7,7 +7,7 @@ description: Start a new requirement-to-PLAN workflow run from a raw requirement
 
 Run `{{R2P_BIN_DIR}}/r2p-start` with the requirement as a positional argument.
 
-Usage: `{{R2P_BIN_DIR}}/r2p-start [--separate] ("<raw requirement>" | --file <path>)`
+Usage: `{{R2P_BIN_DIR}}/r2p-start [--separate] [--repo-path <dir>] ("<raw requirement>" | --file <path>)`
 
 To start from a requirement document, pass `--file <path>` instead of inline text — r2p reads the file contents as the requirement (the path itself is never stored as the requirement): `{{R2P_BIN_DIR}}/r2p-start [--separate] --file ./requirement.md`. `--file` and a positional requirement are mutually exclusive.
 

package/tools/workflow_cli/context_pack.py

@@ -27,6 +27,18 @@ class ProjectContextPack:
     source_dirs: list = field(default_factory=list)
 
 
+def _append_npm_dependencies(pack: ProjectContextPack, dependencies: object, *, dev: bool = False) -> None:
+    if not isinstance(dependencies, dict):
+        return
+    for name, version in dependencies.items():
+        if not isinstance(name, str) or not isinstance(version, str):
+            continue
+        dep = {"name": name, "version": version, "ecosystem": "npm"}
+        if dev:
+            dep["dev"] = True
+        pack.dependencies.append(dep)
+
+
 def build_context_pack(repo_path: Path) -> ProjectContextPack:
     repo_path = Path(repo_path).resolve()
     baseline = scan_repo_baseline(repo_path)
@@ -41,15 +53,8 @@ def build_context_pack(repo_path: Path) -> ProjectContextPack:
             scripts = data.get("scripts")
             if isinstance(scripts, dict) and scripts.get("test"):
                 pack.test_commands.append("npm test")
-            dependencies = data.get("dependencies")
-            if isinstance(dependencies, dict):
-                for name, ver in dependencies.items():
-                    pack.dependencies.append({"name": name, "version": ver, "ecosystem": "npm"})
-            dev_dependencies = data.get("devDependencies")
-            if isinstance(dev_dependencies, dict):
-                for name, ver in dev_dependencies.items():
-                    pack.dependencies.append(
-                        {"name": name, "version": ver, "ecosystem": "npm", "dev": True})
+            _append_npm_dependencies(pack, data.get("dependencies"))
+            _append_npm_dependencies(pack, data.get("devDependencies"), dev=True)
         except (ValueError, OSError):
             pass
 
@@ -84,6 +89,25 @@ def to_json(pack: ProjectContextPack) -> str:
     return json.dumps(asdict(pack), indent=2, ensure_ascii=False)
 
 
+_DEP_DISPLAY_CAP = 20
+
+
+def _dependencies_markdown(dependencies: list) -> str:
+    """Itemize dependencies (name, version, ecosystem, dev flag), capped to keep the seed small."""
+    if not dependencies:
+        return "- dependencies (0): none\n"
+    lines = [f"- dependencies ({len(dependencies)}):\n"]
+    for dep in dependencies[:_DEP_DISPLAY_CAP]:
+        label = " ".join(part for part in (dep.get("name", ""), dep.get("version", "")) if part)
+        ecosystem = dep.get("ecosystem", "")
+        suffix = f"{ecosystem}, dev" if dep.get("dev") else ecosystem
+        lines.append(f"  - {label} ({suffix})\n")
+    hidden = len(dependencies) - _DEP_DISPLAY_CAP
+    if hidden > 0:
+        lines.append(f"  - … and {hidden} more\n")
+    return "".join(lines)
+
+
 def to_markdown(pack: ProjectContextPack) -> str:
     return (
         "# Project Context Pack\n\n"
@@ -93,7 +117,7 @@ def to_markdown(pack: ProjectContextPack) -> str:
         f"- test_commands: {pack.test_commands or 'none'}\n"
         f"- entrypoints: {pack.entrypoints or 'none'}\n"
         f"- config_files: {pack.config_files or 'none'}\n"
-        f"- dependencies: {len(pack.dependencies)} found\n"
+        f"{_dependencies_markdown(pack.dependencies)}"
         f"- source_dirs: {pack.source_dirs}\n"
     )
 

package/tools/workflow_cli/gates.py

@@ -290,14 +290,6 @@ def _find_next_plan_task_field_start(task_body: str, after: int) -> int | None:
     return None
 
 
-def _plan_task_field_value(task_body: str, field: str) -> str:
-    found = _find_plan_task_field(task_body, field)
-    if found is None:
-        return ""
-    match, _ = found
-    return match.group(1).strip()
-
-
 def _iter_plan_task_bodies(content: str):
     return heading_bounded_bodies(content, _PLAN_TASK_RE.match)
 
@@ -496,8 +488,10 @@ def _check_spec_refs_valid(run_dir: Path, content: str) -> list[str]:
 
 
 # R10: Change Type is a closed operation-kind enum; 'new' is a legacy alias.
-_CHANGE_TYPE_VALUES = frozenset({"create", "modify", "delete"})
+_FILE_CHANGE_TYPES = frozenset({"create", "modify", "delete"})
+_CHANGE_TYPE_VALUES = _FILE_CHANGE_TYPES | {"non_code"}
 _CHANGE_TYPE_ALIASES = {"new": "create"}
+_TDD_APPLICABLE_VALUES = frozenset({"yes", "no"})
 
 
 def _normalized_change_type(raw: str) -> str:
@@ -510,6 +504,11 @@ def _task_change_type(body: str) -> str:
     return " ".join(_plan_task_field_body(body, "Change Type").split())
 
 
+def _task_tdd_applicable(body: str) -> str:
+    """Whitespace-normalized TDD Applicable field body (same line + continuation lines)."""
+    return " ".join(_plan_task_field_body(body, "TDD Applicable").split())
+
+
 def _check_plan_task_fields(content: str) -> list[str]:
     issues: list[str] = []
     numbers: list[int] = []
@@ -523,10 +522,32 @@ def _check_plan_task_fields(content: str) -> list[str]:
             if not _plan_task_field_body(body, field).strip():
                 issues.append(f"{label} is missing a non-empty '{field}:' field.")
         raw_change_type = _task_change_type(body)
-        if raw_change_type and _normalized_change_type(raw_change_type) not in _CHANGE_TYPE_VALUES:
+        change_type = _normalized_change_type(raw_change_type)
+        if raw_change_type and change_type not in _CHANGE_TYPE_VALUES:
             issues.append(
                 f"{label} has invalid 'Change Type: {raw_change_type}'; "
-                "allowed: create|modify|delete (alias: new = create)."
+                "allowed: create|modify|delete|non_code (alias: new = create)."
+            )
+        # R16: Change Type and Files must agree — file-op types need a real
+        # path; non_code is the only legal shape for no-file tasks.
+        files_body = _plan_task_field_body(body, "Files")
+        if files_body.strip():
+            file_paths = _plan_task_file_paths(files_body)
+            if change_type in _FILE_CHANGE_TYPES and not file_paths:
+                issues.append(
+                    f"{label} has 'Change Type: {raw_change_type}' but 'Files:' lists "
+                    "no real file path; use 'Change Type: non_code' for tasks that "
+                    "touch no files."
+                )
+            elif change_type == "non_code" and file_paths:
+                issues.append(
+                    f"{label} has 'Change Type: non_code' but 'Files:' lists a file path; "
+                    "use create|modify|delete for file-touching tasks."
+                )
+        raw_tdd = _task_tdd_applicable(body)
+        if raw_tdd and raw_tdd.lower() not in _TDD_APPLICABLE_VALUES:
+            issues.append(
+                f"{label} has invalid 'TDD Applicable: {raw_tdd}'; allowed: yes|no."
             )
     if numbers:
         if len(set(numbers)) != len(numbers):
@@ -575,8 +596,7 @@ def _plan_tasks_missing_code(content: str) -> bool:
         return False
     for body in bodies:
         skeleton = _plan_task_field_body(body, "Skeleton")
-        tdd_applicable = _plan_task_field_value(body, "TDD Applicable")
-        if tdd_applicable.lower() == "yes" and not _has_complete_code_fence(skeleton):
+        if _task_tdd_applicable(body).lower() == "yes" and not _has_complete_code_fence(skeleton):
             return True
     return False
 

package/tools/workflow_cli/version.py

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