specsmith 0.11.3.dev421__tar.gz → 0.11.3.dev422__tar.gz

{specsmith-0.11.3.dev421/src/specsmith.egg-info → specsmith-0.11.3.dev422}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.11.3.dev421
+Version: 0.11.3.dev422
 Summary: Applied Epistemic Engineering toolkit — AEE agent sessions, execution profiles, FPGA/HDL governance, tool installer, 50+ CLI commands.
 Author: BitConcepts
 License-Expression: MIT

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "specsmith"
-version = "0.11.3.dev421"
+version = "0.11.3.dev422"
 description = "Applied Epistemic Engineering toolkit — AEE agent sessions, execution profiles, FPGA/HDL governance, tool installer, 50+ CLI commands."
 readme = "README.md"
 license = "MIT"

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422}/src/specsmith/agent/broker.py

@@ -141,7 +141,6 @@ def classify_intent(utterance: str) -> Intent:
 # ---------------------------------------------------------------------------
 
 
-_REQ_HEADING = re.compile(r"^##\s+\d+\.\s+(?P<title>.+)\s*$", re.MULTILINE)
 # Extended to match project-prefixed IDs e.g. REQ-NN-001, REQ-CLI-042
 _REQ_ID = re.compile(r"-\s*\*\*ID:\*\*\s*(REQ-(?:[A-Z][A-Z0-9_]*-)?\d+)")
 _REQ_DESC = re.compile(r"-\s*\*\*Description:\*\*\s*(.+)")

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422}/src/specsmith/auditor.py

@@ -238,7 +238,12 @@ _TEST_COVERS_PATTERN = re.compile(
 
 
 def check_req_test_consistency(root: Path) -> list[AuditResult]:
-    """Check that every REQ has at least one TEST and vice versa."""
+    """Check that every REQ has at least one TEST and vice versa.
+
+    In YAML-first mode, REQ IDs are loaded from .specsmith/requirements.json
+    (the synced machine state) rather than docs/REQUIREMENTS.md, which may be
+    a derived artifact and not the canonical source (#174).
+    """
     results: list[AuditResult] = []
 
     req_path = root / "docs" / "REQUIREMENTS.md"
@@ -246,23 +251,48 @@ def check_req_test_consistency(root: Path) -> list[AuditResult]:
     test_candidates = _get_test_spec_paths(root)
     test_path = next((p for p in test_candidates if p.exists()), None)
 
-    if not req_path.exists() or test_path is None:
+    if test_path is None:
         results.append(
             AuditResult(
                 name="req-test-consistency",
                 passed=True,
-                message="Skipped: REQUIREMENTS.md or test spec file not found",
+                message="Skipped: test spec file not found",
             )
         )
         return results
 
-    req_text = req_path.read_text(encoding="utf-8")
+    # In YAML-first mode load REQ IDs from the JSON machine state so we don't
+    # depend on docs/REQUIREMENTS.md being present or up to date (#174).
+    import contextlib as _contextlib
+    import json as _json_local
+
+    _reqs_json = root / ".specsmith" / "requirements.json"
+    _yaml_req_ids: set[str] | None = None
+    if _reqs_json.is_file():
+        with _contextlib.suppress(OSError, ValueError):
+            _records = _json_local.loads(_reqs_json.read_text(encoding="utf-8"))
+            _yaml_req_ids = {str(r["id"]) for r in _records if isinstance(r, dict) and r.get("id")}
+
+    if _yaml_req_ids is None and not req_path.exists():
+        results.append(
+            AuditResult(
+                name="req-test-consistency",
+                passed=True,
+                message="Skipped: REQUIREMENTS.md or requirements.json not found",
+            )
+        )
+        return results
+
+    req_text = req_path.read_text(encoding="utf-8") if req_path.exists() else ""
     test_text = test_path.read_text(encoding="utf-8")
 
     # Only check coverage for non-Draft requirements.
     # Draft requirements are stubs (e.g. auto-generated by import) and don't
     # need tests yet — checking them would produce noise on freshly imported projects.
-    all_req_ids = set(_REQ_PATTERN.findall(req_text))
+    # Prefer JSON machine state (covers YAML-first mode); fall back to MD parsing (#174).
+    all_req_ids: set[str] = (
+        _yaml_req_ids if _yaml_req_ids is not None else set(_REQ_PATTERN.findall(req_text))
+    )
     draft_req_ids: set[str] = set()
     for block in re.split(r"(?=^## REQ-)", req_text, flags=re.MULTILINE):
         ids_in_block = set(_REQ_PATTERN.findall(block))
@@ -993,9 +1023,9 @@ def check_industrial_artifacts(root: Path) -> list[AuditResult]:
     if not found_eds:
         return results  # No industrial artifacts found
 
-    undeclared = [
-        f for f in found_eds if str(f.relative_to(root)).replace("\\\\", "/") not in declared_paths
-    ]
+    # Use as_posix() for cross-platform comparison — avoids single-backslash
+    # mismatch on Windows where str(Path) uses '\' but declared paths use '/' (#173).
+    undeclared = [f for f in found_eds if f.relative_to(root).as_posix() not in declared_paths]
 
     if undeclared:
         results.append(

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422}/src/specsmith/governance_logic.py

@@ -85,7 +85,7 @@ def run_preflight(
 
     # Validate explicit REQ IDs against requirements.json and add any that match.
     if explicit_req_ids:
-        rq_json = (root / ".specsmith" / "requirements.json").resolve()
+        rq_json = (root / ".specsmith" / "requirements.json").resolve()  # lgtm[py/path-injection]
         if rq_json.is_file():
             try:
                 rq_records = _json.loads(rq_json.read_text(encoding="utf-8"))
@@ -100,7 +100,9 @@ def run_preflight(
     test_case_ids: list[str] = []
     # Include any explicitly named TEST-* IDs from the utterance.
     if explicit_test_ids:
-        tc_json_explicit = (root / ".specsmith" / "testcases.json").resolve()
+        tc_json_explicit = (
+            root / ".specsmith" / "testcases.json"
+        ).resolve()  # lgtm[py/path-injection]
         if tc_json_explicit.is_file():
             try:
                 tc_explicit = _json.loads(tc_json_explicit.read_text(encoding="utf-8"))
@@ -111,9 +113,9 @@ def run_preflight(
                 if eid in known_tc_ids:
                     test_case_ids.append(eid)
     if requirement_ids:
-        # .resolve() here clears taint for CodeQL py/path-injection; path is
-        # constructed from a validated root with a constant suffix.
-        tc_json = (root / ".specsmith" / "testcases.json").resolve()
+        # .resolve() clears CodeQL py/path-injection taint; path is built from
+        # validated root + a constant suffix containing no user data.
+        tc_json = (root / ".specsmith" / "testcases.json").resolve()  # lgtm[py/path-injection]
         if tc_json.is_file():
             try:
                 records = _json.loads(tc_json.read_text(encoding="utf-8"))

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422}/src/specsmith/validator.py

@@ -159,7 +159,12 @@ def _check_agents_md_refs(root: Path) -> list[ValidationResult]:
 
 
 def _check_req_ids_unique(root: Path) -> list[ValidationResult]:
-    """Check that requirement IDs are unique within REQUIREMENTS.md."""
+    """Check that requirement IDs are unique within REQUIREMENTS.md.
+
+    Uses only the canonical **ID:** field to count each REQ once (#171).
+    The generated REQUIREMENTS.md repeats each ID in both the section heading
+    and the '**ID:** REQ-XXX' field — a raw findall would double-count every ID.
+    """
     results: list[ValidationResult] = []
     req_path = root / "docs" / "REQUIREMENTS.md"
 
@@ -167,7 +172,12 @@ def _check_req_ids_unique(root: Path) -> list[ValidationResult]:
         return results
 
     text = req_path.read_text(encoding="utf-8")
-    req_ids = _REQ_PATTERN.findall(text)
+    # Match only canonical ID declarations, not heading or cross-references (#171).
+    # Pattern matches '- **ID:** REQ-XXX' or '**ID:** REQ-XXX' lines.
+    _ID_FIELD = re.compile(r"\*\*ID:\*\*\s*(REQ-(?:[A-Z]+-)*\d+)")
+    id_field_matches = _ID_FIELD.findall(text)
+    # Fall back to full scan if the markdown has no **ID:** fields (legacy format).
+    req_ids = id_field_matches if id_field_matches else _REQ_PATTERN.findall(text)
 
     seen: dict[str, int] = {}
     for rid in req_ids:

{specsmith-0.11.3.dev421 → specsmith-0.11.3.dev422/src/specsmith.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specsmith
-Version: 0.11.3.dev421
+Version: 0.11.3.dev422
 Summary: Applied Epistemic Engineering toolkit — AEE agent sessions, execution profiles, FPGA/HDL governance, tool installer, 50+ CLI commands.
 Author: BitConcepts
 License-Expression: MIT