xrefkit 0.4.2__tar.gz → 0.4.3__tar.gz

{xrefkit-0.4.2 → xrefkit-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xrefkit
-Version: 0.4.2
+Version: 0.4.3
 Summary: Portable XID, Skill, Knowledge, workflow, and MCP runtime for XRefKit
 Author: synthaicode
 License: MIT License

{xrefkit-0.4.2 → xrefkit-0.4.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "xrefkit"
-version = "0.4.2"
+version = "0.4.3"
 description = "Portable XID, Skill, Knowledge, workflow, and MCP runtime for XRefKit"
 readme = "README.md"
 requires-python = ">=3.11"

{xrefkit-0.4.2 → xrefkit-0.4.3}/tests/test_instruction_workflow.py

@@ -150,3 +150,23 @@ def test_workitem_requires_criterion_or_explicit_unknown_reason(tmp_path: Path)
     assert "criterion=`` reason=`The business owner has not defined the acceptance outcome`" in text
     assert _run(tmp_path, "skill", "phase", "--log", str(out), "--phase", "execution", "--status", "done", "--role", "instruction:executor") == 0
     assert _run(tmp_path, "skill", "verify", "--log", str(out)) == 1
+
+
+def test_workitem_criterion_is_immutable_and_changes_use_supersedes(tmp_path: Path) -> None:
+    out = tmp_path / "work" / "sessions" / "run.md"
+    assert _run(tmp_path, "workflow", "run", "--task", "Do work", "--out", str(out), "--use-default-completion-conditions") == 0
+    base = [
+        "skill", "workitem", "--log", str(out), "--item", "WI-001",
+        "--text", "Implement original outcome", "--completion-criterion", "original outcome is verified",
+        "--status", "pending", "--role", "instruction:executor",
+    ]
+    assert _run(tmp_path, *base) == 0
+    assert _run(tmp_path, *base[:-6], "--completion-criterion", "different outcome is verified", "--status", "pending", "--role", "instruction:executor") == 1
+    assert _run(
+        tmp_path,
+        "skill", "workitem", "--log", str(out), "--item", "WI-002", "--supersedes", "WI-001",
+        "--text", "Implement revised outcome", "--completion-criterion", "revised outcome is verified",
+        "--status", "pending", "--role", "instruction:executor",
+    ) == 0
+    text = out.read_text(encoding="utf-8")
+    assert "WI-002" in text and "supersedes=`WI-001`" in text

{xrefkit-0.4.2 → xrefkit-0.4.3}/xrefkit/__init__.py

@@ -2,4 +2,4 @@
 
 __all__ = ["__version__"]
 
-__version__ = "0.4.2"
+__version__ = "0.4.3"

{xrefkit-0.4.2 → xrefkit-0.4.3}/xrefkit/operations_cli.py

@@ -499,6 +499,7 @@ def _build_parser() -> argparse.ArgumentParser:
     p_skill_workitem.add_argument("--text", default=None, help="Work item text; required when adding a new item")
     p_skill_workitem.add_argument("--completion-criterion", default=None, help="Observable procedural condition for this work item")
     p_skill_workitem.add_argument("--criterion-unknown-reason", default=None, help="Why the completion criterion cannot yet be defined for unknown/blocked/escalated work")
+    p_skill_workitem.add_argument("--supersedes", default=None, help="Existing work item whose criterion is being replaced by this new item")
     p_skill_workitem.add_argument(
         "--status",
         required=True,

{xrefkit-0.4.2 → xrefkit-0.4.3}/xrefkit/skillrun.py

@@ -179,6 +179,12 @@ WORKLIST_ROWS = [
     ("Handoff", "Record outputs, unresolved items, next owner, and human decision points."),
 ]
 WORKITEM_RE = re.compile(
+    r"^- \[(?P<checkbox>[ x!])\] (?P<item_id>[A-Za-z0-9_.-]+) "
+    r"status=`(?P<status>[^`]+)` role=`(?P<role>[^`]+)` "
+    r"criterion=`(?P<criterion>[^`]*)` reason=`(?P<reason>[^`]*)` "
+    r"supersedes=`(?P<supersedes>[^`]*)`: (?P<text>.*)$"
+)
+WORKITEM_V2_RE = re.compile(
     r"^- \[(?P<checkbox>[ x!])\] (?P<item_id>[A-Za-z0-9_.-]+) "
     r"status=`(?P<status>[^`]+)` role=`(?P<role>[^`]+)` "
     r"criterion=`(?P<criterion>[^`]*)` reason=`(?P<reason>[^`]*)`: (?P<text>.*)$"
@@ -838,6 +844,21 @@ def _parse_work_items(text: str) -> list[dict[str, str]]:
                     "role": match.group("role"),
                     "criterion": match.group("criterion"),
                     "reason": match.group("reason"),
+                    "supersedes": match.group("supersedes"),
+                    "text": match.group("text"),
+                }
+            )
+            continue
+        match = WORKITEM_V2_RE.match(line)
+        if match:
+            items.append(
+                {
+                    "item_id": match.group("item_id"),
+                    "status": match.group("status"),
+                    "role": match.group("role"),
+                    "criterion": match.group("criterion"),
+                    "reason": match.group("reason"),
+                    "supersedes": "",
                     "text": match.group("text"),
                 }
             )
@@ -852,16 +873,17 @@ def _parse_work_items(text: str) -> list[dict[str, str]]:
                 "role": match.group("role"),
                 "criterion": "",
                 "reason": "legacy work item has no recorded completion criterion",
+                "supersedes": "",
                 "text": match.group("text"),
             }
         )
     return items
 
 
-def _render_workitem_line(*, item_id: str, status: str, role: str, criterion: str, reason: str, text: str) -> str:
+def _render_workitem_line(*, item_id: str, status: str, role: str, criterion: str, reason: str, supersedes: str, text: str) -> str:
     return (
         f"- [{_workitem_checkbox(status)}] {item_id} status=`{status}` role=`{role}` "
-        f"criterion=`{criterion}` reason=`{reason}`: {text}"
+        f"criterion=`{criterion}` reason=`{reason}` supersedes=`{supersedes}`: {text}"
     )
 
 
@@ -913,6 +935,7 @@ def update_work_item(args) -> SkillRunResult:
     item_text = str(args.text or "").strip()
     criterion = str(getattr(args, "completion_criterion", None) or "").strip().replace("`", "'").replace("\n", " ")
     reason = str(getattr(args, "criterion_unknown_reason", None) or "").strip().replace("`", "'").replace("\n", " ")
+    supersedes = str(getattr(args, "supersedes", None) or "").strip().replace("`", "'").replace("\n", " ")
     if not item_id:
         return SkillRunResult(ok=False, skill_id=None, skill_doc=None, run_log=str(log_path), errors=["missing --item"])
     if status not in VALID_WORKITEM_STATUSES:
@@ -931,10 +954,21 @@ def update_work_item(args) -> SkillRunResult:
     items = _parse_work_items(text)
     existing = next((item for item in items if item["item_id"] == item_id), None)
     if existing:
+        existing_criterion = existing.get("criterion", "")
+        if criterion and criterion != existing_criterion:
+            return SkillRunResult(
+                ok=False,
+                skill_id=None,
+                skill_doc=None,
+                run_log=str(log_path),
+                errors=[
+                    f"completion criterion for {item_id} is immutable; create a new work item with --supersedes {item_id}"
+                ],
+            )
+        if supersedes:
+            return SkillRunResult(ok=False, skill_id=None, skill_doc=None, run_log=str(log_path), errors=[f"--supersedes is only valid when creating a new work item, not updating {item_id}"])
         existing["status"] = status
         existing["role"] = role
-        if criterion:
-            existing["criterion"] = criterion
         if reason:
             existing["reason"] = reason
         if item_text:
@@ -942,7 +976,9 @@ def update_work_item(args) -> SkillRunResult:
     else:
         if not item_text:
             return SkillRunResult(ok=False, skill_id=None, skill_doc=None, run_log=str(log_path), errors=["new work item requires --text"])
-        items.append({"item_id": item_id, "status": status, "role": role, "criterion": criterion, "reason": reason, "text": item_text})
+        if supersedes and not any(item["item_id"] == supersedes for item in items):
+            return SkillRunResult(ok=False, skill_id=None, skill_doc=None, run_log=str(log_path), errors=[f"superseded work item not found: {supersedes}"])
+        items.append({"item_id": item_id, "status": status, "role": role, "criterion": criterion, "reason": reason, "supersedes": supersedes, "text": item_text})
 
     text = _replace_concrete_work_items_section(text, items)
     text = _append_phase_event(text, phase=f"workitem:{item_id}", status=status, role=role, note=item_text or None)
@@ -1469,6 +1505,10 @@ def _progression_record_errors(
     if not work_items:
         errors.append("at least one concrete work item is required before closure")
     for item in work_items:
+        if item.get("supersedes") and not any(previous["item_id"] == item["supersedes"] for previous in work_items):
+            errors.append(f"work item {item['item_id']} supersedes missing work item {item['supersedes']}")
+        if item.get("supersedes") == item["item_id"]:
+            errors.append(f"work item {item['item_id']} cannot supersede itself")
         if not item.get("criterion"):
             if item["status"] in {"unknown", "blocked", "escalated"} and item.get("reason"):
                 pass

{xrefkit-0.4.2 → xrefkit-0.4.3}/xrefkit.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xrefkit
-Version: 0.4.2
+Version: 0.4.3
 Summary: Portable XID, Skill, Knowledge, workflow, and MCP runtime for XRefKit
 Author: synthaicode
 License: MIT License