master-skill 0.5.0 → 0.8.0

package/prebuilt/{yinguang → master-yinguang}/references/teaching.md

@@ -14,31 +14,31 @@
 
 净土法门以信、愿、行为往生三要。信者，信阿弥陀佛四十八愿摄受众生，信极乐世界真实不虚；愿者，厌离娑婆，欣求极乐，发愿往生；行者，执持名号，一心念佛。三者缺一不可，而信愿尤为重要，为净土法门之纲领。
 
-> 出处：【《印光法师文钞正编》卷一·与大兴善寺体安和尚书】→ https://fojin.app/texts/X62n1182
+> 出处：【《印光法师文钞正编》卷一·与大兴善寺体安和尚书】→ https://fojin.app/texts/12977
 
 ### 2. 仗佛慈力，带业往生
 
 末法时期众生根钝障深，唯依净土法门，仗阿弥陀佛愿力加持，虽未断惑，亦可带业往生。此为净土法门殊胜之处——横超三界，不同竖出之难。
 
-> 出处：【《印光法师文钞正编》卷一·复邓伯诚居士书一】→ https://fojin.app/texts/X62n1182
+> 出处：【《印光法师文钞正编》卷一·复邓伯诚居士书一】→ https://fojin.app/texts/12977
 
 ### 3. 敦伦尽分，闲邪存诚
 
 学佛之人，首须敦伦尽分，闲邪存诚。即在世间尽好自己的本分——孝亲、尊师、忠国、爱人，以儒家伦理为基，佛法为归。不可借修行之名，废弃人伦职责。
 
-> 出处：【《印光法师文钞续编》卷上·一函遍复】→ https://fojin.app/texts/X62n1183
+> 出处：【《印光法师文钞续编》卷上·一函遍复】→ https://fojin.app/texts/12978
 
 ### 4. 因果为入道之基
 
 因果之理，为佛法入门之基础。不信因果，则一切善法无从建立。印光大师一生极力提倡因果教育，认为若人人深信因果，则天下太平，人心向善。
 
-> 出处：【《印光法师文钞正编》卷四·挽回劫运护国救民正本清源论】→ https://fojin.app/texts/X62n1182
+> 出处：【《印光法师文钞正编》卷四·挽回劫运护国救民正本清源论】→ https://fojin.app/texts/12977
 
 ### 5. 老实念佛，莫换题目
 
 修行当老实念佛，不可今日参禅、明日学密、后日又习教观，心无定主，终无所成。应择定净土一门，信愿坚固，一门深入，长时熏修。
 
-> 出处：【《印光法师文钞续编》卷上·复王子立居士书】→ https://fojin.app/texts/X62n1183
+> 出处：【《印光法师文钞续编》卷上·复王子立居士书】→ https://fojin.app/texts/12978
 
 ## 精通经典
 
@@ -48,8 +48,8 @@
 | 《佛说观无量寿佛经》 | 净土三经之一，十六观法与九品往生 | [阅读原文](https://fojin.app/texts/19) |
 | 《佛说无量寿经》 | 净土三经之一，阿弥陀佛四十八愿 | [阅读原文](https://fojin.app/texts/18) |
 | 《大势至菩萨念佛圆通章》 | 楞严经二十五圆通之一，都摄六根念佛法 | [阅读原文](https://fojin.app/texts/65) |
-| 《往生论注》（昙鸾） | 天亲菩萨《往生论》之注释，净土五经一论 | [阅读原文](https://fojin.app/texts/T40n1819) |
-| 《印光法师文钞正编》 | 书信体弘法著作，涵盖净土教理与修行指导 | [阅读原文](https://fojin.app/texts/X62n1182) |
+| 《往生论注》（昙鸾） | 天亲菩萨《往生论》之注释，净土五经一论 | [阅读原文](https://fojin.app/texts/7991) |
+| 《印光法师文钞正编》 | 书信体弘法著作，涵盖净土教理与修行指导 | [阅读原文](https://fojin.app/texts/12977) |
 
 ## 修行方法
 

package/prebuilt/{yinguang → master-yinguang}/references/voice.md

@@ -90,7 +90,7 @@
 
 ### 推荐进一步学习
 
-- "建议阅读《印光法师文钞》中相关书信 → [FoJin 原文](https://fojin.app/texts/X62n1182)"
+- "建议阅读《印光法师文钞》中相关书信 → [FoJin 原文](https://fojin.app/texts/12977)"
 - "可参考《佛说阿弥陀经》了解极乐世界依正庄严 → [FoJin 原文](https://fojin.app/texts/20)"
 - "关于念佛方法，可详阅《大势至菩萨念佛圆通章》"
 - "可在 FoJin 词典中查阅相关佛学术语的详解"

package/prebuilt/{yinguang → master-yinguang}/sources/INDEX.md

@@ -6,14 +6,14 @@
 
 | 文件 | 来源经典 | CBETA | FoJin | 覆盖主题 |
 |---|---|---|---|---|
-| `wenchao-excerpts.md` | 《印光法師文鈔》 | X62n1182/X62n1183 | [X62n1182](https://fojin.app/texts/X62n1182) | 信愿行三资粮、摄耳谛听念佛、带业往生横超 |
-| `yihanbianfu-excerpts.md` | 《一函遍復》 | X62n1183 | [X62n1183](https://fojin.app/texts/X62n1183) | 敦伦尽分、闲邪存诚、因果教育 |
+| `wenchao-excerpts.md` | 《印光法師文鈔》 | X62n1182/X62n1183 | [X62n1182](https://fojin.app/texts/12977) | 信愿行三资粮、摄耳谛听念佛、带业往生横超 |
+| `yihanbianfu-excerpts.md` | 《一函遍復》 | X62n1183 | [X62n1183](https://fojin.app/texts/12978) | 敦伦尽分、闲邪存诚、因果教育 |
 | `jingtu-sanjing-excerpts.md` | 净土三经 | T12n0366/T12n0360 | [20](https://fojin.app/texts/20) / [18](https://fojin.app/texts/18) | 持名念佛、第十八愿、一心不乱 |
 
 ## 引用规范
 
 - 格式：`【《经名》卷N，CBETA_ID】→ https://fojin.app/texts/{fojin_text_id}`
-- 示例：`【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/X62n1182`
+- 示例：`【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/12977`
 
 ## 说明
 

package/prebuilt/{yinguang → master-yinguang}/sources/wenchao-excerpts.md

@@ -1,6 +1,6 @@
 # 《印光法師文鈔》关键片段
 
-> 印光大师著。CBETA ID: X62n1182 (正编), X62n1183 (续编)。FoJin: https://fojin.app/texts/X62n1182
+> 印光大师著。CBETA ID: X62n1182 (正编), X62n1183 (续编)。FoJin: https://fojin.app/texts/12977
 > 本文件为教学引用用，节选自 CBETA 公开资料。完整经文请访问 FoJin 或 CBETA。
 
 ## 信愿行三资粮
@@ -9,7 +9,7 @@
 
 > 念佛法门，以信愿行三法为宗。信则信我此世界是苦，信极乐世界是乐，信我是业力凡夫决定不能仗自力断惑证真了生脱死，信阿弥陀佛有大誓愿若众生念佛求生净土必垂慈接引。愿则厌离娑婆欣求极乐。行则至诚恳切常念南无阿弥陀佛，时刻不忘，朝暮定课，闲忙不废。
 
-**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/X62n1182
+**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/12977
 
 **教义要点：**
 - 信愿行为净土法门三资粮，缺一不可
@@ -25,7 +25,7 @@
 
 > 念佛之法，各随所好。或小声念，或默念。除多人同念用引磬外，但宜心念，不宜口念。无论声念默念，皆须念从心起，声从口出，音从耳入。心口念得清清楚楚，耳根听得清清楚楚。如是摄心，妄念自息。
 
-**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/X62n1182
+**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/12977
 
 **教义要点：**
 - 念从心起、声从口出、音从耳入
@@ -41,7 +41,7 @@
 
 > 净土法门，乃如来一代时教中之特别法门。末法众生根钝障深，仗自力断惑证真，比登天还难。仗佛慈力，带业往生，则如风帆扬于顺水，更加橹棹之功，不离当念即登彼岸。
 
-**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/X62n1182
+**引用格式：**【《印光法師文鈔正編》卷一，X62n1182】→ https://fojin.app/texts/12977
 
 **教义要点：**
 - 净土为特别法门，异于通途法门

package/prebuilt/{yinguang → master-yinguang}/sources/yihanbianfu-excerpts.md

@@ -1,6 +1,6 @@
 # 《一函遍復》关键片段
 
-> 印光大师著，出自《印光法師文鈔續編》。CBETA ID: X62n1183。FoJin: https://fojin.app/texts/X62n1183
+> 印光大师著，出自《印光法師文鈔續編》。CBETA ID: X62n1183。FoJin: https://fojin.app/texts/12978
 > 本文件为教学引用用，节选自 CBETA 公开资料。完整经文请访问 FoJin 或 CBETA。
 
 ## 敦伦尽分、闲邪存诚
@@ -9,7 +9,7 @@
 
 > 敦伦尽分，闲邪存诚，诸恶莫作，众善奉行。正心诚意，以立人道之本。然后以此回向净土，求生西方。在家人第一须尽各人本分。如父慈子孝，兄友弟恭，夫和妇顺。
 
-**引用格式：**【《印光法師文鈔續編》·一函遍復，X62n1183】→ https://fojin.app/texts/X62n1183
+**引用格式：**【《印光法師文鈔續編》·一函遍復，X62n1183】→ https://fojin.app/texts/12978
 
 **教义要点：**
 - 学佛首须敦伦尽分——尽好世间本分
@@ -25,7 +25,7 @@
 
 > 因果者，圣人治天下、佛度众生之大权也。若人人深信因果，则人心自善，风俗自淳，唐虞之治不难复见。
 
-**引用格式：**【《印光法師文鈔續編》·一函遍復，X62n1183】→ https://fojin.app/texts/X62n1183
+**引用格式：**【《印光法師文鈔續編》·一函遍復，X62n1183】→ https://fojin.app/texts/12978
 
 **教义要点：**
 - 因果为佛法入门之基础

package/prebuilt/{zhiyi → master-zhiyi}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: zhiyi
+name: master-zhiyi
 description: Use when user asks about 天台宗, 止观, 一念三千, 三谛圆融, 五时八教, 摩诃止观, 法华经, or wants teaching in 智者大师 Zhiyi's voice. Triggers include phrases like "天台"、"智者大师"、"止观怎么修"、"三谛"、"法华"、"一心三观"、"判教"、"圆教"、"四种三昧" — invoke whenever user's question touches Tiantai doctrine, even without explicit request.
 version: 0.3.0
 license: MIT

package/prebuilt/{zhiyi → master-zhiyi}/meta.json

@@ -100,5 +100,33 @@
       "q": "《法华经·方便品》说'唯有一佛乘'，但前面又说三乘，这是否矛盾？",
       "why": "天台以'开权显实'化解此矛盾，是其核心方法论"
     }
+  ],
+  "signature_phrases": [
+    "一念三千",
+    "一心三观",
+    "三谛圆融",
+    "五时八教",
+    "圆顿止观",
+    "性具善恶"
+  ],
+  "style": {
+    "all": "系统论述体，层次分明，逻辑严密；以判教思维统摄诸说，先分后合，义理与实修并重不空谈。",
+    "qa": "先以判教定位所问属何教何理，次分十乘十境之纲，末归一念三千之实，绝不孤悬一边。",
+    "monologue": "上堂依五时八教纲领开列，举一念心具十法界十如三世间，归结於圆顿止观一心三观之实修。"
+  },
+  "lore_triggers": [
+    {
+      "keys": ["一念三千", "三千", "性具"],
+      "content": "夫一心具十法界，一法界又具十法界、百法界；一界具三十种世间，百法界即具三千种世间。此三千在一念心，若无心而已，介尔有心即具三千。亦不言一心在前，一切法在後；亦不言一切法在前，一心在後。",
+      "source_ref": "T46n1911#卷五上"
+    },
+    {
+      "keys": ["一心三观", "三谛", "空假中"],
+      "content": "一空一切空，无假无中而不空，总空观也；一假一切假，无空无中而不假，总假观也；一中一切中，无空无假而不中，总中观也。即中论所说不可思议一心三观。於一念心同时观空假中三谛，非次第而修。",
+      "source_ref": "T46n1911#卷五上"
+    }
+  ],
+  "cross_critique": [
+    { "target_master": "huineng", "position": "对禅宗顿悟：顿悟一念固是事；须知一念三千、性具善恶，理观事造皆备方为圆顿；离开教相判摄，恐落孤山顿悟。", "citation": "T46n1911" }
   ]
 }

package/prebuilt/{zhiyi → master-zhiyi}/references/teaching.md

@@ -38,7 +38,7 @@
 
 佛性本具善恶，不但九界众生性具善恶，即佛界亦性具善恶。但佛界虽性具恶，而不起恶用；九界众生虽性具善，而善用未显。此义为天台独有之说，区别于他宗之"缘起善恶"或"性唯善"说。
 
-> 出处：【《观音玄义》卷上】→ https://fojin.app/texts/T34n1726
+> 出处：【《观音玄义》卷上】→ https://fojin.app/texts/7898
 
 ## 精通经典
 
@@ -49,7 +49,7 @@
 | 《妙法莲华经玄义》 | 天台三大部之一，法华经深义之系统阐发 | [阅读原文](https://fojin.app/texts/52) |
 | 《妙法莲华经文句》 | 天台三大部之一，法华经逐句注释 | [阅读原文](https://fojin.app/texts/52) |
 | 《修习止观坐禅法要》（小止观） | 止观入门之作，初学必读 | [阅读原文](https://fojin.app/texts/8085) |
-| 《观音玄义》 | 天台五小部之一，性具善恶之重要出处 | [阅读原文](https://fojin.app/texts/T34n1726) |
+| 《观音玄义》 | 天台五小部之一，性具善恶之重要出处 | [阅读原文](https://fojin.app/texts/7898) |
 
 ## 修行方法
 

package/scripts/check-manifest-versions.py

@@ -0,0 +1,142 @@
+#!/usr/bin/env python3
+"""Check that the version field is consistent across all platform manifests.
+
+Master-skill ships to 5 ecosystems and each carries its own manifest with a
+`version` field. PR #26 (post-mortem) showed that the manifests can drift
+silently when only some are bumped — users on Cursor would get a stale
+version while npm users got the new one. This script is a hard CI gate to
+catch drift before a release goes out.
+
+Files checked (all relative to repo root):
+
+  - package.json                          (canonical npm version)
+  - .claude-plugin/plugin.json            (Claude Code plugin)
+  - .claude-plugin/marketplace.json       (Claude Code marketplace — nested
+                                          `plugins[*].version`)
+  - .cursor-plugin/plugin.json            (Cursor)
+  - gemini-extension.json                 (Gemini CLI)
+
+Any *.json under .codex/ or .opencode/ that carries a `version` field is
+also picked up (currently those directories ship INSTALL.md only, but
+this future-proofs the check).
+
+Files without a `version` field are skipped.
+
+Usage
+-----
+    python scripts/check-manifest-versions.py            # exit 1 on drift
+    python scripts/check-manifest-versions.py --json
+"""
+from __future__ import annotations
+
+import argparse
+import json
+import sys
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parent.parent
+
+
+def _read_json(p: Path) -> dict | None:
+    try:
+        return json.loads(p.read_text(encoding="utf-8"))
+    except (json.JSONDecodeError, OSError):
+        return None
+
+
+def collect_versions(root: Path = ROOT) -> dict[str, str]:
+    """Return {relative_path: version_string} for every manifest with one."""
+    versions: dict[str, str] = {}
+
+    # Canonical: package.json
+    pkg = _read_json(root / "package.json")
+    if pkg and "version" in pkg:
+        versions["package.json"] = pkg["version"]
+
+    # Flat single-version manifests
+    flat_paths = [
+        root / ".claude-plugin" / "plugin.json",
+        root / ".cursor-plugin" / "plugin.json",
+        root / "gemini-extension.json",
+    ]
+    for p in flat_paths:
+        if not p.exists():
+            continue
+        data = _read_json(p)
+        if data is None:
+            continue
+        v = data.get("version")
+        if isinstance(v, str):
+            versions[str(p.relative_to(root))] = v
+
+    # Marketplace manifest: nested plugins[*].version
+    mp = root / ".claude-plugin" / "marketplace.json"
+    if mp.exists():
+        data = _read_json(mp)
+        if data:
+            plugins = data.get("plugins", []) or []
+            for i, plugin in enumerate(plugins):
+                if isinstance(plugin, dict) and "version" in plugin:
+                    rel = f"{mp.relative_to(root)}::plugins[{i}]"
+                    versions[rel] = plugin["version"]
+
+    # Codex / OpenCode future-proofing — pick up any *.json with version
+    for sub in (".codex", ".opencode"):
+        d = root / sub
+        if not d.exists():
+            continue
+        for p in sorted(d.glob("*.json")):
+            data = _read_json(p)
+            if data is None:
+                continue
+            v = data.get("version")
+            if isinstance(v, str):
+                versions[str(p.relative_to(root))] = v
+
+    return versions
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(
+        description="Check version consistency across platform manifests"
+    )
+    parser.add_argument("--json", action="store_true", help="JSON output")
+    args = parser.parse_args()
+
+    versions = collect_versions()
+    if not versions:
+        print("No version fields found in any manifest — nothing to check.")
+        return 0
+
+    unique = set(versions.values())
+
+    if args.json:
+        out = {
+            "versions": versions,
+            "consistent": len(unique) == 1,
+            "unique_count": len(unique),
+        }
+        print(json.dumps(out, indent=2, ensure_ascii=False))
+    else:
+        if len(unique) > 1:
+            print("Manifest version drift detected:")
+            for path, v in sorted(versions.items()):
+                print(f"  {path}: {v}")
+            print()
+            print(
+                f"Found {len(unique)} distinct versions across "
+                f"{len(versions)} manifests — they must all match."
+            )
+        else:
+            v = next(iter(unique))
+            print(
+                f"All {len(versions)} manifest version fields agree on {v}."
+            )
+            for path in sorted(versions):
+                print(f"  {path}: {versions[path]}")
+
+    return 0 if len(unique) == 1 else 1
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/scripts/cite.py

@@ -52,7 +52,7 @@ def find_citations(master, text):
 
 def main():
     parser = argparse.ArgumentParser(description="在 master 的 sources 中查找关键词并输出 CBETA 引用")
-    parser.add_argument("--master", required=True, help="大师 ID，如 zhiyi")
+    parser.add_argument("--master", required=True, help="大师 ID，如 master-zhiyi")
     parser.add_argument("--text", required=True, help="搜索关键词")
     parser.add_argument("--json", action="store_true", dest="as_json", help="JSON 格式输出")
     args = parser.parse_args()

package/scripts/query.py

@@ -54,7 +54,7 @@ def search(master, query, brief=False):
 
 def main():
     parser = argparse.ArgumentParser(description="离线检索 master 的 sources 和 references")
-    parser.add_argument("--master", required=True, help="大师 ID，如 zhiyi")
+    parser.add_argument("--master", required=True, help="大师 ID，如 master-zhiyi")
     parser.add_argument("--q", required=True, help="搜索关键词（空格分隔，OR 匹配）")
     parser.add_argument("--json", action="store_true", dest="as_json", help="JSON 格式输出")
     parser.add_argument("--brief", action="store_true", help="仅输出段标题和文件路径")

package/scripts/test-fidelity.py

@@ -6,10 +6,10 @@ with the master's SKILL.md loaded as system prompt, and checks responses for
 expected citations and keywords.
 
 Usage:
-    python scripts/test-fidelity.py --master zhiyi              # test one master
-    python scripts/test-fidelity.py --master zhiyi --dry-run    # show test cases without calling API
-    python scripts/test-fidelity.py --all                       # test all masters
-    python scripts/test-fidelity.py --master zhiyi --model claude-sonnet-4-6  # specific model
+    python scripts/test-fidelity.py --master master-zhiyi              # test one master
+    python scripts/test-fidelity.py --master master-zhiyi --dry-run    # show test cases without calling API
+    python scripts/test-fidelity.py --all                              # test all masters
+    python scripts/test-fidelity.py --master master-zhiyi --model claude-sonnet-4-6  # specific model
 
 Requires:
     - ANTHROPIC_API_KEY environment variable

package/scripts/tests/test_check_manifest_versions.py

@@ -0,0 +1,217 @@
+"""Tests for check-manifest-versions.py.
+
+Covers:
+  - collect_versions returns all 5 platform manifests when present
+  - drift between manifests is detected (returns mismatched dict)
+  - manifest with no `version` field is skipped silently
+  - marketplace.json plugins[].version is picked up
+  - main() exits 0 when consistent, 1 when drift
+"""
+from __future__ import annotations
+
+import importlib.util
+import json
+import subprocess
+import sys
+from pathlib import Path
+
+
+def _load_module():
+    spec_path = (
+        Path(__file__).resolve().parents[1] / "check-manifest-versions.py"
+    )
+    spec = importlib.util.spec_from_file_location("cmv", spec_path)
+    mod = importlib.util.module_from_spec(spec)
+    sys.modules["cmv"] = mod
+    spec.loader.exec_module(mod)
+    return mod
+
+
+def _make_repo(tmp: Path, versions: dict[str, str | None]) -> Path:
+    """Build a fake repo. versions keys: package, claude, marketplace,
+    cursor, gemini. Value None means: do not write that file at all.
+    A version string of '' means: write the file without a version field.
+    """
+    root = tmp / "repo"
+    root.mkdir(parents=True)
+    (root / ".claude-plugin").mkdir()
+    (root / ".cursor-plugin").mkdir()
+
+    if "package" in versions and versions["package"] is not None:
+        body = {"name": "x"}
+        if versions["package"]:
+            body["version"] = versions["package"]
+        (root / "package.json").write_text(json.dumps(body), encoding="utf-8")
+
+    if "claude" in versions and versions["claude"] is not None:
+        body = {"name": "x"}
+        if versions["claude"]:
+            body["version"] = versions["claude"]
+        (root / ".claude-plugin" / "plugin.json").write_text(
+            json.dumps(body), encoding="utf-8"
+        )
+
+    if "marketplace" in versions and versions["marketplace"] is not None:
+        body = {"name": "x", "plugins": []}
+        if versions["marketplace"]:
+            body["plugins"] = [
+                {"name": "x", "version": versions["marketplace"]}
+            ]
+        (root / ".claude-plugin" / "marketplace.json").write_text(
+            json.dumps(body), encoding="utf-8"
+        )
+
+    if "cursor" in versions and versions["cursor"] is not None:
+        body = {"name": "x"}
+        if versions["cursor"]:
+            body["version"] = versions["cursor"]
+        (root / ".cursor-plugin" / "plugin.json").write_text(
+            json.dumps(body), encoding="utf-8"
+        )
+
+    if "gemini" in versions and versions["gemini"] is not None:
+        body = {"name": "x"}
+        if versions["gemini"]:
+            body["version"] = versions["gemini"]
+        (root / "gemini-extension.json").write_text(
+            json.dumps(body), encoding="utf-8"
+        )
+    return root
+
+
+# ---------- happy path ----------
+
+
+def test_all_manifests_consistent(tmp_path):
+    m = _load_module()
+    root = _make_repo(
+        tmp_path,
+        {
+            "package": "1.2.3",
+            "claude": "1.2.3",
+            "marketplace": "1.2.3",
+            "cursor": "1.2.3",
+            "gemini": "1.2.3",
+        },
+    )
+    out = m.collect_versions(root)
+    assert set(out.values()) == {"1.2.3"}
+    assert len(out) == 5
+
+
+def test_drift_detected(tmp_path):
+    m = _load_module()
+    root = _make_repo(
+        tmp_path,
+        {
+            "package": "1.2.3",
+            "claude": "1.2.3",
+            "marketplace": "1.2.2",  # drift
+            "cursor": "1.2.3",
+            "gemini": "1.2.3",
+        },
+    )
+    out = m.collect_versions(root)
+    assert "1.2.2" in out.values()
+    assert "1.2.3" in out.values()
+    assert len(set(out.values())) == 2
+
+
+# ---------- skip files without version ----------
+
+
+def test_manifest_without_version_is_skipped(tmp_path):
+    m = _load_module()
+    # gemini has no version key
+    root = _make_repo(
+        tmp_path,
+        {
+            "package": "1.2.3",
+            "claude": "1.2.3",
+            "marketplace": "1.2.3",
+            "cursor": "1.2.3",
+            "gemini": "",  # write file but no version field
+        },
+    )
+    out = m.collect_versions(root)
+    assert "gemini-extension.json" not in out
+    assert len(out) == 4
+    assert set(out.values()) == {"1.2.3"}
+
+
+def test_marketplace_with_empty_plugins_is_skipped(tmp_path):
+    m = _load_module()
+    root = _make_repo(
+        tmp_path,
+        {
+            "package": "1.2.3",
+            "claude": "1.2.3",
+            "marketplace": "",  # write file with no plugin entries
+            "cursor": "1.2.3",
+            "gemini": "1.2.3",
+        },
+    )
+    out = m.collect_versions(root)
+    assert ".claude-plugin/marketplace.json::plugins[0]" not in out
+
+
+def test_codex_opencode_picked_up_when_versioned(tmp_path):
+    m = _load_module()
+    root = _make_repo(
+        tmp_path,
+        {"package": "9.0.0"},
+    )
+    # Stash a future .codex/<name>.json with a version
+    (root / ".codex").mkdir()
+    (root / ".codex" / "agents.json").write_text(
+        json.dumps({"name": "x", "version": "9.0.0"}), encoding="utf-8"
+    )
+    (root / ".opencode").mkdir()
+    (root / ".opencode" / "plugin.json").write_text(
+        json.dumps({"name": "x", "version": "9.0.0"}), encoding="utf-8"
+    )
+    out = m.collect_versions(root)
+    assert ".codex/agents.json" in out
+    assert ".opencode/plugin.json" in out
+
+
+def test_codex_opencode_install_md_only_no_false_positive(tmp_path):
+    m = _load_module()
+    # The real repo today: .codex/INSTALL.md only — no JSON.
+    root = _make_repo(tmp_path, {"package": "1.0.0"})
+    (root / ".codex").mkdir()
+    (root / ".codex" / "INSTALL.md").write_text("# install", encoding="utf-8")
+    out = m.collect_versions(root)
+    assert all(not k.startswith(".codex/") for k in out)
+
+
+# ---------- CLI exit codes ----------
+
+
+def _run_cli(args: list[str], cwd: Path) -> subprocess.CompletedProcess:
+    script = (
+        Path(__file__).resolve().parents[1] / "check-manifest-versions.py"
+    )
+    return subprocess.run(
+        [sys.executable, str(script), *args],
+        capture_output=True,
+        text=True,
+        cwd=cwd,
+    )
+
+
+def test_cli_exits_zero_on_real_repo():
+    """Live repo manifests must be consistent or the check fails."""
+    repo_root = Path(__file__).resolve().parents[2]
+    proc = _run_cli([], cwd=repo_root)
+    assert proc.returncode == 0, proc.stdout + proc.stderr
+
+
+def test_cli_json_output_shape():
+    repo_root = Path(__file__).resolve().parents[2]
+    proc = _run_cli(["--json"], cwd=repo_root)
+    assert proc.returncode == 0
+    data = json.loads(proc.stdout)
+    assert "versions" in data
+    assert "consistent" in data
+    assert data["consistent"] is True