master-skill 0.10.0 → 0.10.1

package/scripts/validate-citation-contract.py

@@ -0,0 +1,193 @@
+#!/usr/bin/env python3
+"""Validate source-neutral citation contracts in persona metadata."""
+from __future__ import annotations
+
+import json
+import sys
+from pathlib import Path
+
+
+PREBUILT_DIR = Path(__file__).resolve().parent.parent / "prebuilt"
+
+EXPECTED_PERSONA_SLUGS = (
+    "ajahn-chah",
+    "atisha",
+    "buddhaghosa",
+    "fazang",
+    "huineng",
+    "kumarajiva",
+    "mahasi-sayadaw",
+    "milarepa",
+    "nagarjuna",
+    "ouyi",
+    "tsongkhapa",
+    "xuanzang",
+    "xuyun",
+    "yinguang",
+    "zhiyi",
+)
+
+EXPECTED_REQUIRED_FOR = [
+    "doctrinal_claim",
+    "practice_guidance",
+    "text_interpretation",
+]
+EXPECTED_KEYS = {
+    "version",
+    "claim_policy",
+    "required_for",
+    "allowed_source_types",
+    "minimum_claim_coverage",
+    "live_retrieval_allowed",
+}
+
+
+def _display_path(meta_path: Path) -> str:
+    return f"{meta_path.parent.name}/meta.json"
+
+
+def _persona_count(prebuilt_dir: Path) -> int:
+    count = 0
+    for meta_path in prebuilt_dir.glob("master-*/meta.json"):
+        try:
+            data = json.loads(meta_path.read_text(encoding="utf-8"))
+        except (OSError, json.JSONDecodeError):
+            continue
+        if data.get("kind") != "meta-skill":
+            count += 1
+    return count
+
+
+def validate(prebuilt_dir: Path) -> list[str]:
+    """Return deterministic citation-contract errors for persona metadata."""
+    errors: list[str] = []
+
+    for meta_path in sorted(prebuilt_dir.glob("master-*/meta.json")):
+        path = _display_path(meta_path)
+        try:
+            data = json.loads(meta_path.read_text(encoding="utf-8"))
+        except (OSError, json.JSONDecodeError) as exc:
+            errors.append(f"{path}: invalid metadata: {exc}")
+            continue
+
+        if data.get("kind") == "meta-skill":
+            if "citation_contract" in data:
+                errors.append(
+                    f"{path}: meta-skill must not declare citation_contract"
+                )
+            continue
+
+        sources = data.get("sources")
+        if not isinstance(sources, list) or not sources:
+            errors.append(f"{path}: persona sources must be a non-empty list")
+            continue
+
+        source_types: list[str] = []
+        for index, source in enumerate(sources):
+            source_type = source.get("type") if isinstance(source, dict) else None
+            if not isinstance(source_type, str) or not source_type.strip():
+                errors.append(
+                    f"{path}: sources[].type at index {index} must be a non-empty string"
+                )
+                continue
+            source_types.append(source_type)
+
+        contract = data.get("citation_contract")
+        if not isinstance(contract, dict):
+            errors.append(f"{path}: citation_contract must be an object")
+            continue
+
+        contract_keys = set(contract)
+        if contract_keys != EXPECTED_KEYS:
+            missing = sorted(EXPECTED_KEYS - contract_keys)
+            extra = sorted(contract_keys - EXPECTED_KEYS)
+            errors.append(
+                f"{path}: citation_contract keys must be exact; "
+                f"missing={missing}, extra={extra}"
+            )
+
+        version = contract.get("version")
+        if type(version) is not int or version != 1:
+            errors.append(f"{path}: version must be integer 1")
+
+        if contract.get("claim_policy") != "declared_sources_only":
+            errors.append(
+                f"{path}: claim_policy must be 'declared_sources_only'"
+            )
+
+        if contract.get("required_for") != EXPECTED_REQUIRED_FOR:
+            errors.append(
+                f"{path}: required_for must equal {EXPECTED_REQUIRED_FOR}"
+            )
+
+        allowed_source_types = contract.get("allowed_source_types")
+        expected_source_types = sorted(set(source_types))
+        if (
+            not isinstance(allowed_source_types, list)
+            or any(
+                not isinstance(source_type, str) or not source_type.strip()
+                for source_type in allowed_source_types
+            )
+            or allowed_source_types != expected_source_types
+        ):
+            errors.append(
+                f"{path}: allowed_source_types must equal {expected_source_types}"
+            )
+
+        coverage = contract.get("minimum_claim_coverage")
+        if (
+            isinstance(coverage, bool)
+            or not isinstance(coverage, (int, float))
+            or coverage != 0.9
+        ):
+            errors.append(f"{path}: minimum_claim_coverage must be numeric 0.9")
+
+        if contract.get("live_retrieval_allowed") is not True:
+            errors.append(f"{path}: live_retrieval_allowed must be true")
+
+    return errors
+
+
+def validate_repository(prebuilt_dir: Path) -> list[str]:
+    """Validate contracts plus the repository's fixed 15-persona roster."""
+    if not prebuilt_dir.is_dir():
+        return [f"{prebuilt_dir}: prebuilt directory not found"]
+
+    errors = validate(prebuilt_dir)
+    discovered: set[str] = set()
+    for meta_path in sorted(prebuilt_dir.glob("master-*/meta.json")):
+        try:
+            data = json.loads(meta_path.read_text(encoding="utf-8"))
+        except (OSError, json.JSONDecodeError):
+            continue
+        if data.get("kind") == "meta-skill":
+            continue
+        discovered.add(meta_path.parent.name.removeprefix("master-"))
+
+    expected = set(EXPECTED_PERSONA_SLUGS)
+    missing = sorted(expected - discovered)
+    unexpected = sorted(discovered - expected)
+    if missing or unexpected or len(discovered) != len(EXPECTED_PERSONA_SLUGS):
+        errors.append(
+            "persona roster must contain exactly "
+            f"{len(EXPECTED_PERSONA_SLUGS)} declared personas; "
+            f"missing={[f'master-{slug}' for slug in missing]}, "
+            f"unexpected={[f'master-{slug}' for slug in unexpected]}"
+        )
+
+    return errors
+
+
+def main() -> int:
+    errors = validate_repository(PREBUILT_DIR)
+    if errors:
+        for error in errors:
+            print(error, file=sys.stderr)
+        return 1
+
+    print(f"citation contracts OK ({_persona_count(PREBUILT_DIR)} personas)")
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/scripts/verify_citations.py

@@ -33,7 +33,14 @@ _SAFE_MASTER = re.compile(r"^[A-Za-z0-9_-]+$")
 
 # CBETA id 形态:T48n2008 / T08n0235(藏经卷+n+编号),及 API 返回的 X1218 / X0303
 # (无卷号)。无 `n` 的形态只认 T/X 两个集合,避免误吞 Wikidata 的 Q1234 / P5008。
-_CBETA_ID = re.compile(r"\b(?:[A-Z]{1,2}\d+n\d+|[TX]\d{3,})\b")
+#
+# 边界不能用 \b:Python 的 \w 覆盖 CJK,故「卷一T99n9999」的「一」与「T」之间
+# 没有 \b,整块引文会被 audit_answer 当作无 id 跳过 —— 而格式跑偏正是模型最可能
+# 编造经号的时候。改判「前后不是拉丁字母或数字」:汉字紧邻属真实引文形态,须命中;
+# 拉丁字母紧邻(FakeSutraT99n9999)通常意味着它只是更长 token 的一部分,不算引文。
+_CBETA_ID = re.compile(
+    r"(?<![0-9A-Za-z])(?:[A-Z]{1,2}\d+n\d+|[TX]\d{3,})(?![0-9A-Za-z])"
+)
 # 引文块 【…】
 _CITATION_BLOCK = re.compile(r"【([^】]*)】")
 # live 链接 fojin.app/texts/<数字>

package/skill-catalog.json

@@ -0,0 +1,147 @@
+{
+  "version": 1,
+  "skills": [
+    {
+      "name": "master-nagarjuna",
+      "kind": "persona",
+      "source": "prebuilt/master-nagarjuna",
+      "install_dir": "master-nagarjuna",
+      "aliases": ["nagarjuna", "master-nagarjuna"]
+    },
+    {
+      "name": "master-xuanzang",
+      "kind": "persona",
+      "source": "prebuilt/master-xuanzang",
+      "install_dir": "master-xuanzang",
+      "aliases": ["xuanzang", "master-xuanzang"]
+    },
+    {
+      "name": "master-kumarajiva",
+      "kind": "persona",
+      "source": "prebuilt/master-kumarajiva",
+      "install_dir": "master-kumarajiva",
+      "aliases": ["kumarajiva", "master-kumarajiva"]
+    },
+    {
+      "name": "master-huineng",
+      "kind": "persona",
+      "source": "prebuilt/master-huineng",
+      "install_dir": "master-huineng",
+      "aliases": ["huineng", "master-huineng"]
+    },
+    {
+      "name": "master-zhiyi",
+      "kind": "persona",
+      "source": "prebuilt/master-zhiyi",
+      "install_dir": "master-zhiyi",
+      "aliases": ["zhiyi", "master-zhiyi"]
+    },
+    {
+      "name": "master-fazang",
+      "kind": "persona",
+      "source": "prebuilt/master-fazang",
+      "install_dir": "master-fazang",
+      "aliases": ["fazang", "master-fazang"]
+    },
+    {
+      "name": "master-yinguang",
+      "kind": "persona",
+      "source": "prebuilt/master-yinguang",
+      "install_dir": "master-yinguang",
+      "aliases": ["yinguang", "master-yinguang"]
+    },
+    {
+      "name": "master-ouyi",
+      "kind": "persona",
+      "source": "prebuilt/master-ouyi",
+      "install_dir": "master-ouyi",
+      "aliases": ["ouyi", "master-ouyi"]
+    },
+    {
+      "name": "master-xuyun",
+      "kind": "persona",
+      "source": "prebuilt/master-xuyun",
+      "install_dir": "master-xuyun",
+      "aliases": ["xuyun", "master-xuyun"]
+    },
+    {
+      "name": "master-atisha",
+      "kind": "persona",
+      "source": "prebuilt/master-atisha",
+      "install_dir": "master-atisha",
+      "aliases": ["atisha", "master-atisha"]
+    },
+    {
+      "name": "master-tsongkhapa",
+      "kind": "persona",
+      "source": "prebuilt/master-tsongkhapa",
+      "install_dir": "master-tsongkhapa",
+      "aliases": ["tsongkhapa", "master-tsongkhapa"]
+    },
+    {
+      "name": "master-milarepa",
+      "kind": "persona",
+      "source": "prebuilt/master-milarepa",
+      "install_dir": "master-milarepa",
+      "aliases": ["milarepa", "master-milarepa"]
+    },
+    {
+      "name": "master-buddhaghosa",
+      "kind": "persona",
+      "source": "prebuilt/master-buddhaghosa",
+      "install_dir": "master-buddhaghosa",
+      "aliases": ["buddhaghosa", "master-buddhaghosa"]
+    },
+    {
+      "name": "master-mahasi-sayadaw",
+      "kind": "persona",
+      "source": "prebuilt/master-mahasi-sayadaw",
+      "install_dir": "master-mahasi-sayadaw",
+      "aliases": ["mahasi-sayadaw", "master-mahasi-sayadaw"]
+    },
+    {
+      "name": "master-ajahn-chah",
+      "kind": "persona",
+      "source": "prebuilt/master-ajahn-chah",
+      "install_dir": "master-ajahn-chah",
+      "aliases": ["ajahn-chah", "master-ajahn-chah"]
+    },
+    {
+      "name": "compare-masters",
+      "kind": "teaching-mode",
+      "source": "prebuilt/compare",
+      "install_dir": "compare-masters",
+      "aliases": ["compare-masters"]
+    },
+    {
+      "name": "master-debate",
+      "kind": "teaching-mode",
+      "source": "prebuilt/master-debate",
+      "install_dir": "master-debate",
+      "aliases": ["master-debate"]
+    },
+    {
+      "name": "master-curriculum",
+      "kind": "teaching-mode",
+      "source": "prebuilt/master-curriculum",
+      "install_dir": "master-curriculum",
+      "aliases": ["master-curriculum"]
+    },
+    {
+      "name": "create-master",
+      "kind": "generator",
+      "source": ".",
+      "install_dir": "create-master",
+      "aliases": ["create-master"],
+      "bundle_paths": [
+        "SKILL.md",
+        "tools",
+        "prompts",
+        "references",
+        "requirements.txt",
+        "ETHICS.md",
+        "masters"
+      ]
+    }
+  ]
+}