xrefkit 0.6.0__tar.gz → 0.6.2__tar.gz

{xrefkit-0.6.0 → xrefkit-0.6.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xrefkit
-Version: 0.6.0
+Version: 0.6.2
 Summary: Portable XID, Skill, Knowledge, workflow, and MCP runtime for XRefKit
 Author: synthaicode
 License: MIT License
@@ -156,6 +156,20 @@ This command starts the optional catalog and runtime MCP adapter. This checkout
 does not implement management upload, staging, sealing, review, or adoption of
 uploaded SkillDefinition bytes into the active catalog.
 
+To connect VS Code/Copilot, run from your target workspace:
+
+```powershell
+python -m pip install "xrefkit[mcp]"
+python -m xrefkit mcp setup --repo . --output .xrefkit/setup-review
+python -m xrefkit mcp setup apply --repo . --source .xrefkit/setup-review
+```
+
+Review the generated files before applying them, open the workspace in VS Code,
+and start the `xrefkit` MCP server. In Copilot chat, ask: "Call XRefKit's
+get_startup_context and help me get started." The AI receives any needed legacy
+migration guidance; you can describe your goal without specifying Skill names.
+Opening VS Code alone does not guarantee an AI conversation starts.
+
 ## Where to go next
 
 - [Install XRefKit and register Skill Packages](docs/guides/089_xrefkit_package_first_registration.md#xid-4F8C2A7D1E90)
@@ -172,3 +186,13 @@ XRefKit does not require provider API keys to explore or install the package.
 Do not commit secrets, API keys, access tokens, `.env` files, or provider
 credentials. Authenticate external AI tools through their official provider
 mechanisms.
+
+## Repository Skill adoption and scoped source-analysis knowledge
+
+This repository uses the audited canonical Skill sources listed in
+`skills/repository_adoption.json`. Legacy source identities remain aliases;
+external legacy split Skills and YAML Skill Packages remain supported. See
+[SkillDefinition adoption](docs/core/contracts/096_skill_definition_contract.md#xid-E6A19D4B72C3).
+The source-analysis common XID remains stable and routes coherent criteria
+fragments on demand. Narrow tasks avoid unrelated bodies; full reviews retain
+every required axis. No similarity-based knowledge merge or deletion occurs.

{xrefkit-0.6.0 → xrefkit-0.6.2}/README.md

@@ -117,6 +117,20 @@ This command starts the optional catalog and runtime MCP adapter. This checkout
 does not implement management upload, staging, sealing, review, or adoption of
 uploaded SkillDefinition bytes into the active catalog.
 
+To connect VS Code/Copilot, run from your target workspace:
+
+```powershell
+python -m pip install "xrefkit[mcp]"
+python -m xrefkit mcp setup --repo . --output .xrefkit/setup-review
+python -m xrefkit mcp setup apply --repo . --source .xrefkit/setup-review
+```
+
+Review the generated files before applying them, open the workspace in VS Code,
+and start the `xrefkit` MCP server. In Copilot chat, ask: "Call XRefKit's
+get_startup_context and help me get started." The AI receives any needed legacy
+migration guidance; you can describe your goal without specifying Skill names.
+Opening VS Code alone does not guarantee an AI conversation starts.
+
 ## Where to go next
 
 - [Install XRefKit and register Skill Packages](docs/guides/089_xrefkit_package_first_registration.md#xid-4F8C2A7D1E90)
@@ -133,3 +147,13 @@ XRefKit does not require provider API keys to explore or install the package.
 Do not commit secrets, API keys, access tokens, `.env` files, or provider
 credentials. Authenticate external AI tools through their official provider
 mechanisms.
+
+## Repository Skill adoption and scoped source-analysis knowledge
+
+This repository uses the audited canonical Skill sources listed in
+`skills/repository_adoption.json`. Legacy source identities remain aliases;
+external legacy split Skills and YAML Skill Packages remain supported. See
+[SkillDefinition adoption](docs/core/contracts/096_skill_definition_contract.md#xid-E6A19D4B72C3).
+The source-analysis common XID remains stable and routes coherent criteria
+fragments on demand. Narrow tasks avoid unrelated bodies; full reviews retain
+every required axis. No similarity-based knowledge merge or deletion occurs.

{xrefkit-0.6.0 → xrefkit-0.6.2}/pyproject.toml

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

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_artifact_selected_family.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -25,9 +27,8 @@ def test_artifact_v1_parse_concrete_metadata_and_legacy(directory, skill_id, xid
     assert set(metadata["aliases"]) == {body_xid, meta_xid}
     assert metadata["inputs"] and metadata["outputs"]
     assert metadata["control_refs"] == []
-    assert "CAP-" not in definition["method"]
-    assert "## Reporting Contract" not in definition["method"]
-    assert "skill_doc: `./SKILL.md`" in (repo / relative.parent / "meta.md").read_text()
+    assert_preserved_source_boundary(definition)
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_artifact_v1_explicit_batch_selection_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_brownfield_csharp_structure_definitions.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -24,9 +26,9 @@ def test_tracked_trio_parse_aliases_metadata_and_legacy_readability():
         for forbidden in ("capability", "tuning", "responsibility", "execution_mode", "model", "maturity"):
             assert forbidden not in metadata
         for common in ("## Reporting Contract", "## Logging", "## Context Direction Guard"):
-            assert common not in definition["method"]
-        assert f"<!-- xid: {body_xid} -->" in (repo / "skills" / directory / "SKILL.md").read_text(encoding="utf-8")
-        assert f"<!-- xid: {meta_xid} -->" in (repo / "skills" / directory / "meta.md").read_text(encoding="utf-8")
+            assert_preserved_source_boundary(definition)
+        assert_legacy_receipt(repo / 'skills' / directory / 'SKILL.md')
+        assert_legacy_receipt(repo / 'skills' / directory / 'meta.md')
 
 
 def test_tracked_trio_combined_catalog_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_brownfield_file_editing_protocol.py

@@ -5,7 +5,7 @@ ROOT = Path(__file__).resolve().parents[1]
 
 
 def test_brownfield_file_editing_protocol_is_in_canonical_and_packaged_skill() -> None:
-    canonical = (ROOT / "skills" / "brownfield-workflow" / "SKILL.md").read_text(encoding="utf-8")
+    canonical = (ROOT / "skills" / "brownfield-workflow" / "SKILL.v1.md").read_text(encoding="utf-8")
     packaged = (
         ROOT / "packages" / "xrefkit-skills-brownfield" / "src" / "xrefkit_skills_brownfield"
         / "skills" / "brownfield_workflow" / "entry.md"

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_business_intake_conversation_definitions.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -21,10 +23,9 @@ def test_business_intake_v1_definitions_parse_aliases_and_preserve_legacy():
         assert metadata["control_refs"] == []
         for forbidden in ("capability", "tuning", "responsibility", "execution_mode", "model", "maturity"):
             assert forbidden not in metadata
-        assert "## Context Direction Guard" not in definition["method"]
-        assert "## Reporting Contract" not in definition["method"]
-        assert f"<!-- xid: {body_xid} -->" in (repo / relative.parent / "SKILL.md").read_text(encoding="utf-8")
-        assert f"skill_doc: `./SKILL.md`" in (repo / relative.parent / "meta.md").read_text(encoding="utf-8")
+        assert_preserved_source_boundary(definition)
+        assert_legacy_receipt(repo / relative.parent / 'SKILL.md')
+        assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_business_intake_v1_explicit_catalog_selection_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_business_intake_definition.py

@@ -1,3 +1,4 @@
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -43,13 +44,11 @@ def test_business_intake_v1_definitions_parse_select_resolve_and_preserve_legacy
         assert resolved["unsatisfied_required"] == []
         assert resolved["needs"][0]["satisfied"] is True
 
-        legacy = (directory / "SKILL.md").read_text(encoding="utf-8")
-        meta = (directory / "meta.md").read_text(encoding="utf-8")
-        assert f"<!-- xid: {body_xid} -->" in legacy
-        assert f"<!-- xid: {meta_xid} -->" in meta
-        assert "skill_doc: `./SKILL.md`" in meta
+        assert_legacy_receipt(directory / 'SKILL.md')
+        assert_legacy_receipt(directory / 'meta.md')
 
     catalog = XRefCatalog.build(repo, skill_definition_paths=paths)
-    assert {entry.skill_id for entry in catalog.skills if entry.definition_format == "skill_definition_v1"} == {
-        "business_learning_interview", "business_intake_scoping"
+    assert {"business_learning_interview", "business_intake_scoping"} <= {
+        entry.skill_id for entry in catalog.skills if entry.definition_format == "skill_definition_v1"
     }
+    assert len(catalog.skills) == 62

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_catalog_preparation_definition.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -25,11 +27,9 @@ def test_v1_definitions_parse_with_aliases_and_concrete_metadata(skill_id, folde
     assert metadata["inputs"] and metadata["outputs"]
     assert metadata["control_refs"] == []
     assert not any(key in metadata for key in ("capability", "tuning", "responsibility", "execution_mode", "model", "model_tier", "maturity"))
-    assert "CAP-" not in definition["method"]
-    assert "## Reporting Contract" not in definition["method"]
-    assert "## Required Knowledge (XID)" not in definition["method"]
-    assert (repo / folder / "SKILL.md").read_text(encoding="utf-8").startswith("<!-- xid: ")
-    assert (repo / folder / "meta.md").read_text(encoding="utf-8").startswith("<!-- xid: ")
+    assert_preserved_source_boundary(definition)
+    assert_legacy_receipt(repo / folder / 'SKILL.md')
+    assert_legacy_receipt(repo / folder / 'meta.md')
 
 
 def test_explicit_catalog_and_knowledge_resolution_cover_the_trio():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_code_constraint_definition.py

@@ -1,3 +1,4 @@
+from repository_skill_receipts import assert_preserved_source_boundary
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -38,10 +39,8 @@ def test_code_constraint_v1_resolves_selected_knowledge_and_legacy_files_are_unc
     assert all(need["activation_state"] == "active" for need in result["needs"])
     assert all(need["satisfied"] is True for need in result["needs"])
 
-    meta = (repo / "skills/packs/constraint-derivation/code_constraint_derivation/meta.md").read_text(encoding="utf-8")
-    legacy = (repo / "skills/packs/constraint-derivation/code_constraint_derivation/SKILL.md").read_text(encoding="utf-8")
-    assert "skill_doc: `./SKILL.md`" in meta
-    assert "<!-- xid: D4701BFC6EA4 -->" in meta
-    assert "<!-- xid: D4701BFC6EA5 -->" in legacy
-    assert "## Reporting Contract" not in definition["method"]
+    from repository_skill_receipts import assert_legacy_receipt
+    assert assert_legacy_receipt(repo / "skills/packs/constraint-derivation/code_constraint_derivation/meta.md")["xid"] == "D4701BFC6EA4"
+    assert assert_legacy_receipt(repo / "skills/packs/constraint-derivation/code_constraint_derivation/SKILL.md")["xid"] == "D4701BFC6EA5"
+    assert_preserved_source_boundary(definition)
     assert "111D282CA0EA" in definition["metadata"]["control_refs"]

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_constraint_derivation_family_definition.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -24,13 +26,10 @@ def test_constraint_derivation_v1_migration(skill_id, xid, body_xid, meta_xid, s
     assert metadata["xid"] == xid
     assert set(metadata["aliases"]) == {body_xid, meta_xid}
     assert metadata["control_refs"] == ["111D282CA0EA"]
-    assert "## Reporting Contract" not in definition["method"]
-    assert "## Required Knowledge (XID)" not in definition["method"]
-    assert "Context direction guard" not in definition["method"]
+    assert_preserved_source_boundary(definition)
 
-    legacy = (repo / relative.parent / "SKILL.md").read_bytes()
-    assert f"<!-- xid: {body_xid} -->".encode() in legacy
-    assert f"skill_doc: `./SKILL.md`".encode() in (repo / relative.parent / "meta.md").read_bytes()
+    assert_legacy_receipt(repo / relative.parent / 'SKILL.md')
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_constraint_derivation_v1_explicit_selection_and_knowledge_resolution():

xrefkit-0.6.2/tests/test_constraint_derivation_selected_family.py

@@ -0,0 +1,103 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
+from pathlib import Path
+
+import pytest
+
+from xrefkit.mcp.catalog import XRefCatalog
+from xrefkit.skill_definition import load_skill_definition
+
+
+CASES = [
+    ("async_constraint_derivation", "9A4C7E1D2B60", "F6580A1C2346", "F6580A1C2345"),
+    ("auth_constraint_derivation", "C8F2A6D1E430", "A7691B2D3457", "A7691B2D3456"),
+    ("commonality_derivation", "E7B3C951A2D0", "B87A2C3E4568", "B87A2C3E4567"),
+    ("cross_constraint_derivation", "F4A8D2C6B190", "E5812C0D7FB7", "E5812C0D7FB6"),
+    ("design_constraint_derivation", "A6D9F3B1C720", "B214C6D8E012", "B214C6D8E011"),
+]
+
+
+@pytest.mark.parametrize("skill_id,xid,body_xid,meta_xid", CASES)
+def test_selected_constraint_derivation_v1(skill_id, xid, body_xid, meta_xid):
+    repo = Path(__file__).resolve().parents[1]
+    relative = Path(f"skills/packs/constraint-derivation/{skill_id}/SKILL.v1.md")
+    definition = load_skill_definition(repo / relative)
+    metadata = definition["metadata"]
+    assert metadata["xid"] == xid
+    expected_aliases = {body_xid, meta_xid}
+    if skill_id == "async_constraint_derivation":
+        expected_aliases.add("F6580A1C2344")  # Previously published candidate alias.
+    assert set(metadata["aliases"]) == expected_aliases
+    assert metadata["control_refs"] == ["111D282CA0EA"]
+    assert_preserved_source_boundary(definition)
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
+
+
+def test_selected_constraint_derivation_catalog_and_knowledge_resolution():
+    repo = Path(__file__).resolve().parents[1]
+    paths = [Path(f"skills/packs/constraint-derivation/{skill}/SKILL.v1.md") for skill, *_ in CASES]
+    catalog = XRefCatalog.build(repo, skill_definition_paths=paths)
+    entries = {entry.skill_id: entry for entry in catalog.skills if entry.skill_id in {case[0] for case in CASES}}
+    assert set(entries) == {case[0] for case in CASES}
+    assert all(entry.definition_format == "skill_definition_v1" for entry in entries.values())
+    for skill_id, _, _, _ in CASES:
+        definition = load_skill_definition(repo / f"skills/packs/constraint-derivation/{skill_id}/SKILL.v1.md")
+        need_ids = [need["id"] for need in definition["metadata"]["knowledge_needs"]]
+        result = catalog.resolve_skill_knowledge(skill_id, need_ids)
+        assert result["unresolved_activation"] == []
+        assert result["unsatisfied_required"] == []
+
+
+def test_async_definition_resolves_actual_legacy_body_identity_after_retirement(tmp_path):
+    repo = Path(__file__).resolve().parents[1]
+    path = repo / "skills/packs/constraint-derivation/async_constraint_derivation/SKILL.v1.md"
+    definition = load_skill_definition(path)
+    # A retired split source must resolve through the configured successor.
+    successor = tmp_path / "SKILL.v1.md"
+    successor.write_bytes(path.read_bytes())
+    catalog = XRefCatalog.build(tmp_path, skill_definition_paths=[successor])
+    resolved = catalog.get_document_by_xid("F6580A1C2346")
+    assert resolved["content_hash"] == definition["content_hash"]
+    assert resolved["content"].encode("utf-8") == path.read_bytes()
+    assert resolved["xid"] == definition["metadata"]["xid"]
+    assert resolved["requested_xid"] == "F6580A1C2346"
+    assert resolved["resolved_via"] == "definition_alias"
+    cached = catalog.get_document_by_xid("F6580A1C2346", resolved["content_hash"])
+    assert cached["cache_status"] == "not_modified"
+
+
+def test_alias_does_not_replace_present_source_or_hide_conflicts(tmp_path):
+    repo = Path(__file__).resolve().parents[1]
+    source = repo / "skills/packs/constraint-derivation/async_constraint_derivation"
+    successor = tmp_path / "SKILL.v1.md"
+    successor.write_bytes((source / "SKILL.v1.md").read_bytes())
+    documents = tmp_path / "docs"
+    documents.mkdir()
+    original = documents / "original.md"
+    original.write_bytes(b"<!-- xid: F6580A1C2346 -->\n# External original source\n")
+    catalog = XRefCatalog.build(tmp_path, skill_definition_paths=[successor])
+    resolved = catalog.get_document_by_xid("F6580A1C2346")
+    assert resolved["xid"] == "F6580A1C2346"
+    assert "resolved_via" not in resolved
+    (documents / "conflicting.md").write_bytes(original.read_bytes())
+    conflict = catalog.get_document_by_xid("F6580A1C2346")
+    assert conflict["error"] == "xid_conflict"
+    assert len(conflict["matches"]) == 2
+
+
+def test_managed_definition_alias_and_canonical_share_raw_identity(tmp_path):
+    repo = Path(__file__).resolve().parents[1]
+    source = repo / "skills/packs/constraint-derivation/async_constraint_derivation/SKILL.v1.md"
+    directory = tmp_path / "skills" / "async_constraint_derivation"
+    directory.mkdir(parents=True)
+    successor = directory / "SKILL.v1.md"
+    successor.write_bytes(source.read_bytes())
+    definition = load_skill_definition(successor)
+    catalog = XRefCatalog.build(tmp_path, skill_definition_paths=[successor])
+    canonical = catalog.get_document_by_xid(definition["metadata"]["xid"])
+    alias = catalog.get_document_by_xid("F6580A1C2346")
+    assert canonical["content"].encode("utf-8") == successor.read_bytes()
+    assert canonical["content_hash"] == alias["content_hash"] == definition["content_hash"]
+    assert canonical["xid"] == alias["xid"]
+    cached = catalog.get_document_by_xid(canonical["xid"], alias["content_hash"])
+    assert cached["cache_status"] == "not_modified"

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_database_batch_definition_migration.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -34,12 +36,10 @@ def test_database_batch_v1_definitions_parse_with_aliases_and_preserve_legacy():
         assert set(metadata["aliases"]) == {body_xid, meta_xid}
         assert metadata["control_refs"] == []
         assert not {"capability", "tuning", "responsibility", "execution_mode", "model", "maturity"} & metadata.keys()
-        assert "CAP-" not in definition["method"]
-        assert "Skill operating contract" not in definition["method"]
-        assert "Context direction guard" not in definition["method"]
+        assert_preserved_source_boundary(definition)
         legacy_dir = repo / relative.parent
-        assert f"<!-- xid: {body_xid} -->" in (legacy_dir / "SKILL.md").read_text(encoding="utf-8")
-        assert f"<!-- xid: {meta_xid} -->" in (legacy_dir / "meta.md").read_text(encoding="utf-8")
+        assert_legacy_receipt(legacy_dir / 'SKILL.md')
+        assert_legacy_receipt(legacy_dir / 'meta.md')
 
 
 def test_database_batch_combined_explicit_catalog_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_design_business_intake_selected_family.py

@@ -1,3 +1,4 @@
+from repository_skill_receipts import assert_preserved_source_boundary
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -26,7 +27,7 @@ def test_design_and_business_intake_batch_selects_without_identity_collision():
 
     assert len(xids) == len(set(xids))
     assert set(xids).isdisjoint(aliases)
-    assert all("CAP-" not in definition["method"] for definition in definitions)
+    assert all(assert_preserved_source_boundary(definition) for definition in definitions)
 
     catalog = XRefCatalog.build(repo, skill_definition_paths=PATHS)
     expected = {definition["metadata"]["skill_id"] for definition in definitions}

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_design_planning_definition_migration.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -36,7 +38,7 @@ def test_pair_omits_runtime_and_common_control_metadata(case):
     definition = load_skill_definition(repo / "skills" / skill_id / "SKILL.v1.md")
     forbidden = {"capability", "tuning", "responsibility", "execution_mode", "model", "maturity"}
     assert forbidden.isdisjoint(definition["metadata"])
-    assert "## Reporting Contract" not in definition["method"]
+    assert_preserved_source_boundary(definition)
 
 
 def test_pair_is_explicitly_catalog_selected_and_knowledge_resolves():
@@ -63,10 +65,6 @@ def test_pair_is_explicitly_catalog_selected_and_knowledge_resolves():
 def test_legacy_files_remain_readable_and_unchanged_identity(case):
     skill_id = case[0]
     repo = Path(__file__).resolve().parents[1]
-    body = (repo / "skills" / skill_id / "SKILL.md").read_text(encoding="utf-8")
-    meta = (repo / "skills" / skill_id / "meta.md").read_text(encoding="utf-8")
+    assert_legacy_receipt(repo / 'skills' / skill_id / 'SKILL.md')
+    assert_legacy_receipt(repo / 'skills' / skill_id / 'meta.md')
     expected = next(item for item in CASES if item[0] == skill_id)
-    assert f"xid-{expected[2]}" in body
-    assert f"xid-{expected[3]}" in meta
-    assert f"# Skill: {skill_id}" in body
-    assert f"# Skill Meta: {skill_id}" in meta

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_dotnet_definition_candidate.py

@@ -1,6 +1,7 @@
 import json
 import shutil
 from pathlib import Path
+from repository_skill_receipts import assert_preserved_source_boundary
 
 import pytest
 
@@ -12,7 +13,7 @@ from xrefkit.skillmeta import _parse_meta_lines
 
 def test_representative_conversion_is_lossless_and_nonactivating(tmp_path):
     repo = Path(__file__).resolve().parents[1]
-    source = repo / "skills/dotnet_change_analysis"
+    source = repo / "tests/fixtures/legacy_dotnet_skill"
     dest = tmp_path / "skills/dotnet_change_analysis"
     shutil.copytree(source, dest)
     before = {p.name: p.read_bytes() for p in [dest / "SKILL.md", dest / "meta.md"]}
@@ -85,7 +86,8 @@ def test_tracked_v1_definition_resolves_knowledge_and_explicit_selection_is_sing
     assert definition["metadata"]["xid"] == "9883EF4E8CA9"
     assert set(definition["metadata"]["aliases"]) == {"D94E3B3A7C11", "1F4A6D20B8E1"}
     migrated_method = definition["method"].replace("9883EF4E8CA9", "D94E3B3A7C11")
-    assert migrated_method.splitlines() == candidate["method"].splitlines()
+    assert migrated_method.split("## Preserved source obligations", 1)[0].rstrip().splitlines() == candidate["method"].rstrip().splitlines()
+    assert_preserved_source_boundary(definition)
     catalog = XRefCatalog.build(repo, skill_definition_paths=[relative])
     entries = [entry for entry in catalog.skills if entry.skill_id == "dotnet_change_analysis"]
     assert len(entries) == 1
@@ -99,5 +101,5 @@ def test_tracked_v1_definition_resolves_knowledge_and_explicit_selection_is_sing
 
 def test_legacy_meta_still_points_to_legacy_body():
     repo = Path(__file__).resolve().parents[1]
-    meta = (repo / "skills/dotnet_change_analysis/meta.md").read_text(encoding="utf-8")
-    assert "skill_doc: `./SKILL.md`" in meta
+    from repository_skill_receipts import assert_legacy_receipt
+    assert_legacy_receipt(repo / "skills/dotnet_change_analysis/meta.md")

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_editorial_family_definition.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -19,12 +21,10 @@ def test_editorial_v1_migration(skill_id, xid, body_xid, meta_xid):
     assert definition["metadata"]["xid"] == xid
     assert set(definition["metadata"]["aliases"]) == {body_xid, meta_xid}
     assert definition["metadata"]["control_refs"] == []
-    assert "## Reporting Contract" not in definition["method"]
-    assert "## Required Knowledge (XID)" not in definition["method"]
-    assert "Context direction guard" not in definition["method"]
+    assert_preserved_source_boundary(definition)
     assert "human" in definition["method"].lower()
-    assert f"<!-- xid: {body_xid} -->".encode() in (repo / relative.parent / "SKILL.md").read_bytes()
-    assert b"skill_doc: `./SKILL.md`" in (repo / relative.parent / "meta.md").read_bytes()
+    assert_legacy_receipt(repo / relative.parent / 'SKILL.md')
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_editorial_v1_explicit_selection_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_editorial_intake_definition.py

@@ -1,3 +1,4 @@
+from repository_skill_receipts import assert_preserved_source_boundary
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -12,8 +13,7 @@ def test_editorial_intake_v1_aliases_catalog_knowledge_and_legacy_body():
     assert metadata["xid"] == "7C4E9A2D6F81"
     assert set(metadata["aliases"]) == {"77F7D4CB9F99", "54437A84B3D0"}
     assert "capability" not in metadata
-    assert "## Context Direction Guard" not in definition["method"]
-    assert "## Reporting Contract" not in definition["method"]
+    assert_preserved_source_boundary(definition)
     catalog = XRefCatalog.build(repo, skill_definition_paths=[relative])
     entries = [entry for entry in catalog.skills if entry.skill_id == "editorial_intake"]
     assert len(entries) == 1
@@ -23,6 +23,6 @@ def test_editorial_intake_v1_aliases_catalog_knowledge_and_legacy_body():
     assert resolved["unresolved_activation"] == []
     assert resolved["unsatisfied_required"] == []
     assert all(need["satisfied"] is True for need in resolved["needs"])
-    legacy = (repo / "skills/packs/editorial-ops/editorial_intake/SKILL.md").read_text(encoding="utf-8")
-    assert "<!-- xid: 77F7D4CB9F99 -->" in legacy
-    assert "skill_doc: `./SKILL.md`" in (repo / "skills/packs/editorial-ops/editorial_intake/meta.md").read_text(encoding="utf-8")
+    from repository_skill_receipts import assert_legacy_receipt
+    assert assert_legacy_receipt(repo / "skills/packs/editorial-ops/editorial_intake/SKILL.md")["xid"] == "77F7D4CB9F99"
+    assert_legacy_receipt(repo / "skills/packs/editorial-ops/editorial_intake/meta.md")

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_editorial_selected_family.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -22,9 +24,8 @@ def test_editorial_v1_parse_aliases_and_legacy(skill_id, xid, body_xid, meta_xid
     assert metadata["xid"] == xid
     assert set(metadata["aliases"]) == {body_xid, meta_xid}
     assert metadata["control_refs"] == []
-    assert "## Reporting Contract" not in definition["method"]
-    assert "## Required Knowledge (XID)" not in definition["method"]
-    assert "skill_doc: `./SKILL.md`" in (repo / relative.parent / "meta.md").read_text()
+    assert_preserved_source_boundary(definition)
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_editorial_v1_explicit_selection_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_final_skilldefinition_batch.py

@@ -1,3 +1,4 @@
+from repository_skill_receipts import assert_preserved_source_boundary
 from pathlib import Path
 
 from xrefkit.mcp.catalog import XRefCatalog
@@ -29,7 +30,7 @@ def test_final_batch_selects_together_without_identity_or_common_control_leaks()
 
     assert len(xids) == len(set(xids))
     assert set(xids).isdisjoint(aliases)
-    assert all("CAP-" not in definition["method"] for definition in definitions)
+    assert all(assert_preserved_source_boundary(definition) for definition in definitions)
     assert all(
         "7A2F4C8D1601"
         not in [xid for need in definition["metadata"]["knowledge_needs"] for xid in need["seed_xids"]]

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_flow_selected_family.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -23,10 +25,8 @@ def test_flow_v1_parse_aliases_and_boundaries(skill_id, xid, body_xid, meta_xid)
     assert metadata["xid"] == xid
     assert set(metadata["aliases"]) == {body_xid, meta_xid}
     assert metadata["control_refs"] == []
-    assert "CAP-" not in definition["method"]
-    assert "## Reporting Contract" not in definition["method"]
-    assert "## Required Knowledge (XID)" not in definition["method"]
-    assert "skill_doc: `./SKILL.md`" in (repo / relative.parent / "meta.md").read_text()
+    assert_preserved_source_boundary(definition)
+    assert_legacy_receipt(repo / relative.parent / 'meta.md')
 
 
 def test_flow_v1_batch_selection_and_knowledge_resolution():

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_implementation_flow_definition.py

@@ -1,3 +1,5 @@
+from repository_skill_receipts import assert_preserved_source_boundary
+from repository_skill_receipts import assert_legacy_receipt
 from pathlib import Path
 
 import pytest
@@ -43,7 +45,7 @@ def test_flow_definition_omits_runtime_and_common_control_metadata(case) -> None
     metadata = definition["metadata"]
     forbidden = {"capability", "tuning", "responsibility", "execution_mode", "model", "maturity"}
     assert forbidden.isdisjoint(metadata)
-    assert "## Reporting Contract" not in definition["method"]
+    assert_preserved_source_boundary(definition)
 
 
 def test_flow_definitions_are_explicitly_catalog_selected_and_knowledge_resolves() -> None:
@@ -67,9 +69,5 @@ def test_legacy_files_remain_readable_and_retain_original_xids(
     skill_id: str, xid: str, body_xid: str, meta_xid: str
 ) -> None:
     repo = Path(__file__).resolve().parents[1]
-    body = (repo / "skills" / skill_id / "SKILL.md").read_text(encoding="utf-8")
-    meta = (repo / "skills" / skill_id / "meta.md").read_text(encoding="utf-8")
-    assert f"xid-{body_xid}" in body
-    assert f"xid-{meta_xid}" in meta
-    assert f"# Skill: {skill_id}" in body
-    assert f"# Skill Meta: {skill_id}" in meta
+    assert_legacy_receipt(repo / 'skills' / skill_id / 'SKILL.md')
+    assert_legacy_receipt(repo / 'skills' / skill_id / 'meta.md')

{xrefkit-0.6.0 → xrefkit-0.6.2}/tests/test_knowledge_relations_validator.py

@@ -60,6 +60,24 @@ class KnowledgeRelationsValidatorTests(unittest.TestCase):
                 any("missing from knowledge/000_index.md" in error for error in errors)
             )
 
+    def test_resolves_relationship_to_repository_contract(self) -> None:
+        with tempfile.TemporaryDirectory() as tmp:
+            root = Path(tmp)
+            self._write(root, "knowledge/000_index.md", "<!-- xid: INDEX -->\n# Index\n- [A](a.md#xid-A)\n")
+            self._write(root, "knowledge/a.md", "<!-- xid: A -->\n# A\n\n## Knowledge Relations\n- applies_to: [Contract](../docs/contract.md#xid-CONTRACT)\n")
+            self._write(root, "docs/contract.md", "<!-- xid: CONTRACT -->\n# Contract\n")
+            self.assertEqual([], VALIDATOR.validate(root))
+
+    def test_duplicate_contract_owners_are_ambiguous_but_same_file_repeat_is_allowed(self) -> None:
+        with tempfile.TemporaryDirectory() as tmp:
+            root = Path(tmp)
+            self._write(root, "knowledge/000_index.md", "<!-- xid: INDEX -->\n# Index\n- [A](a.md#xid-A)\n")
+            self._write(root, "knowledge/a.md", "<!-- xid: A -->\n# A\n\n## Knowledge Relations\n- applies_to: [Contract](../docs/c1.md#xid-CONTRACT)\n")
+            self._write(root, "docs/c1.md", "<!-- xid: CONTRACT -->\n# Contract\n<!-- xid: CONTRACT -->\n")
+            self.assertEqual([], VALIDATOR.validate(root))
+            self._write(root, "docs/c2.md", "<!-- xid: CONTRACT -->\n# Other Contract\n")
+            self.assertTrue(any("ambiguous target XID 'CONTRACT'" in e for e in VALIDATOR.validate(root)))
+
     def test_reports_duplicate_primary_titles(self) -> None:
         with tempfile.TemporaryDirectory() as tmp:
             root = Path(tmp)