Commit Graph

87 Commits

Author SHA1 Message Date
xianren 6ba49597e0 fix(ci): use context.repo.repo instead of context.repo.name
github-script's context.repo object has {owner, repo}, not {owner, name}.
The typo caused the weekly Verify FoJin Links workflow to POST to
repos/xr843//issues (double slash, empty repo name) and fail with 404.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-13 16:13:23 +08:00
Tim Ren 3deec3ef68 Merge pull request #10 from xr843/feat/non-cli-onboarding
feat(onboarding): browser-first README + starter_questions for 8 masters
2026-04-11 10:55:18 +08:00
Tim Ren e02921da99 Merge pull request #9 from xr843/feat/compare-divergence-radar
feat(compare): divergence radar + labeled differences + classic debate templates
2026-04-11 10:55:15 +08:00
xianren 7e2070318d feat(onboarding): browser-first README + starter_questions for 8 masters
Shift the README narrative so that non-developer users — the 99% of the
actual audience — find the browser entry point first and the CLI install
second. The old structure buried fojin.app/chat at line 48 under a wall
of developer badges.

Changes:

- README.md / README_EN.md: new "立即体验 / Try It Now" hero section
  right after the title block, with a big call-to-action linking to
  fojin.app/chat, a "which master should I ask" recommendation table,
  and an explicit redirect for CLI users to the developer section.
- Rename "快速开始 / Quick Start" → "开发者安装 / Developer Installation"
  with a clear "if you just want to try it, use the browser instead"
  callout at the top.
- Update TOC anchors in both languages to match.

- prebuilt/<master>/meta.json × 8: new `starter_questions` field with
  4 Q&A entries per master. Each entry has `q` (the question) and `why`
  (why this question is suited for this specific master's voice and
  source corpus). Intended consumer: fojin.app/chat can fetch the raw
  meta.json on master selection and render these as cold-start prompts
  — solving the empty-input-box problem for first-time users.

Rationale: the biggest onboarding gap for non-CLI users is "I don't
know what to ask this master." A curated 4-question starter per master
provides both a safe entry point and a teaching signal about what this
particular master is actually good at answering.

Validated: scripts/validate.py --strict passes (9/9 masters).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-11 10:46:32 +08:00
xianren 28e5436ff9 feat(compare): divergence radar + labeled differences + classic debate templates
Upgrade /compare-masters from a loose comparison table to a structured
divergence analysis framework. Core changes:

- 5-dim radar (否定/安立/入手处/根器/终极表达): force the LLM to locate
  exactly which layer the disagreement sits on, not just "A thinks X,
  B thinks Y" without axis.
- Mandatory divergence labels: every concrete difference must be tagged
  [宗派性] / [侧重性] / [表达性] / [根器性]. This prevents the most
  common mistake — treating a 根器性 difference as a 宗派性 conflict.
- Classic debate templates: 禅净之争 / 性相之辩 / 空有之争 / 顿渐之辩.
  Pre-load the real historical disputes so the LLM does not invent
  fake controversy and knows the actual cruxes.
- Meta-question follow-ups (Step 4): generate "why are they different"
  prompts to turn a single comparison into a learning session.
- HARD-GATE additions: NO UNLABELED DIVERGENCE, NO EMPTY RADAR CELL,
  with rationalization defenses and red flags.

Rationale: the loose 共通点/差异点 template was too permissive — it
let the LLM produce shallow "each has their own merits" summaries.
Forcing a 5-axis radar + typed labels surfaces the actual structural
divergence, which is the real educational value of multi-master
comparison.

Validated: scripts/validate.py --strict passes (9/9 masters).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-11 10:42:58 +08:00
Tim Ren 6b0ce62b20 Merge pull request #8 from xr843/fix/compare-meta-skill-lint
fix(lint): exempt meta-skills from lineage/source checks
2026-04-11 09:20:43 +08:00
xr843 a0c367c058 fix(lint): exempt meta-skills from lineage/source checks
The compare-masters skill is an aggregate meta-skill that borrows from
other masters and has no single lineage, dates, sources, or own corpus
directories. Treating it as a regular master made --strict CI fail on
all branches since 2026-04-08.

Introduce a kind field (default master) and skip the lineage/dates/
sources/citation_format and references/sources directory checks when
kind == meta-skill. compare/SKILL.md gains kind: meta-skill.

Required fields (name, description) and version/license are still
enforced for all skills.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-11 09:18:59 +08:00
Tim Ren 38e7eb09dd Merge pull request #7 from xr843/docs/sync-readme-v03
docs(readme): sync README to v0.3 reality, bring EN in lockstep with CN
2026-04-10 14:36:06 +08:00
Tim Ren 3f51fc7ee7 Merge pull request #6 from xr843/chore/sync-plugin-metadata
chore(plugin): sync Cursor plugin metadata with Claude plugin
2026-04-10 14:36:02 +08:00
xianren 4eb92ab048 docs(readme): sync README to v0.3 reality, bring EN in lockstep with CN
README.md and README_EN.md had drifted from the code. The Chinese README
still claimed "SKILL.md < 100 行" (actual: 386 lines at repo root,
124–159 lines in prebuilt/*), and the English README still advertised
the v0.2 release — completely missing NPX install, provenance,
fidelity tests, HARD-GATE, two-stage review, and multi-platform
plugins.

Changes in both languages:

- Remove the "SKILL.md < 100 lines" claim (it was never true of the
  rebuilt v0.3 files and made the whole doc feel aspirational)
- Add v0.3 post-iteration banner covering two-stage review, HARD-GATE,
  multi-platform plugin support, session-start hook, pressure tests,
  and CI validation
- Expand Features list with HARD-GATE, two-stage review, unified
  multi-platform plugin bullets
- Rewrite the architecture diagram so it shows the session-start hook,
  HARD-GATE placement, two-stage review step (Step 3.5), and the
  five-platform manifest layout
- Add a Troubleshooting / 常见问题 section covering: FoJin API
  unreachability and offline fallback, CBETA citation format and
  verification, `npx master-skill install` failure recovery,
  correction-mode for historical accuracy, and how to contribute
  a new prebuilt master
- Update Contributing to reference the actual v0.3 layout (references/,
  sources/, tests/fidelity.jsonl) and CI fidelity dry-run gate

Chinese-specific:

- Fix the "OpenClaw" typo in the usage environment list
  (Claude Code / Codex CLI / OpenClaw → Claude Code / Cursor /
  Codex CLI / OpenCode / Gemini CLI)

English-specific (larger catch-up):

- Replace the stale "v0.2 Update" banner with the v0.3 banner
- Replace the v0.2 Features list (Dual-mode output, Incremental
  evolution, Version management) with the current v0.3 feature set
- Add the missing "Compare Masters" section (the English README never
  documented /compare-masters at all)
- Rewrite the architecture diagram — it still showed teaching.md and
  voice.md at the top level of prebuilt/{slug}/ and referenced
  tools/version_manager.py, which is not the v0.3 layout
- Fix the same "OpenClaw" typo

No behavioral changes — docs only.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-10 14:27:32 +08:00
xianren 1af80522dc chore(plugin): sync Cursor plugin metadata with Claude plugin
Keeps the two per-platform plugin manifests in lockstep so marketplace
listings don't diverge. Cursor now matches Claude on:

- description: adds "RAG-grounded in FoJin knowledge graph" tail so
  discovery text is identical across platforms
- keywords: adds chinese-buddhism, cbeta, fojin (8 keywords total,
  matching .claude-plugin/plugin.json)

No behavioral change — manifest metadata only.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-10 14:22:34 +08:00
Tim Ren 7b2af14c7f Merge pull request #5 from xr843/feat/two-stage-review
feat: two-stage review for create-master generation pipeline
2026-04-08 21:45:05 +08:00
xianren f22204a75e feat: add two-stage review for create-master generation pipeline
Inspired by superpowers' spec-reviewer + code-quality-reviewer pattern,
add mandatory two-stage review between generation and preview:

Stage 1 — Doctrine Review (doctrine_reviewer.md):
  - Citation coverage check (target >= 90%)
  - CBETA ID attribution accuracy
  - Sectarian boundary validation
  - Auto-fix with max 2 retry rounds

Stage 2 — Voice Review (voice_reviewer.md):
  - Layer 0 hard rules completeness
  - Style-to-tradition matching
  - Layer structure clarity check
  - Historical plausibility

Both stages must PASS before entering Step 4 (preview).
Added as Step 3.5 in create-master SKILL.md workflow.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-08 21:43:46 +08:00
Tim Ren a8513910cb Merge pull request #4 from xr843/feat/multi-platform-plugin
feat: multi-platform plugin support (Claude Code, Cursor, Codex, OpenCode, Gemini)
2026-04-08 21:41:56 +08:00
Tim Ren 12020444e0 Merge pull request #3 from xr843/feat/hard-gate-and-pressure-tests
feat: HARD-GATE discipline enforcement, pressure tests, and CI pipeline
2026-04-08 21:41:53 +08:00
xianren f3472d6254 feat: add multi-platform plugin support and session-start hook
Add support for 5 platforms beyond NPX:

- Claude Code: .claude-plugin/plugin.json + marketplace.json
- Cursor: .cursor-plugin/plugin.json + hooks-cursor.json
- Codex CLI: .codex/INSTALL.md with symlink setup
- OpenCode: .opencode/INSTALL.md with plugin config
- Gemini CLI: gemini-extension.json + GEMINI.md

Infrastructure:
- SessionStart hook with platform detection (Claude Code,
  Cursor, Copilot CLI) that injects available masters list
- Cross-platform run-hook.cmd polyglot wrapper (Windows + Unix)
- .version-bump.json for unified version management across
  all platform config files
- Update README/README_EN with multi-platform install instructions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-08 21:38:08 +08:00
xianren 45c0416bc3 feat: add HARD-GATE to compare and create-master skills
- compare: add NO COMPARATIVE RANKING and NO FABRICATED DIALOGUE
  iron laws with rationalization defense table
- create-master: add NO FABRICATED SOURCES iron law for the
  generation pipeline
- Add 2 boundary tests for compare (sectarian ranking, fabricated
  dialogue)
- Fix validate-fidelity.py to support compare's assertion fields
  and no_fabricated_dialogue boundary type

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-08 21:32:13 +08:00
xianren c654e7440f feat: add CI validation pipeline and boundary test support
- Update test-fidelity.py to support must_not_contain and
  must_not_contain_first_turn fields for boundary/pressure tests
- Add validate-fidelity.py for structural validation of all
  fidelity.jsonl files (no API needed)
- Add GitHub Actions workflow: runs validate + dry-run on every
  push/PR touching prebuilt/scripts/prompts/tools

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-08 21:30:23 +08:00
xianren da80665fa5 feat: add HARD-GATE discipline enforcement and pressure tests for all 8 masters
Inspired by obra/superpowers project patterns, add three-layer defense
to prevent AI hallucination and rule violation:

- Iron Laws: NO DOCTRINAL CLAIM WITHOUT CBETA CITATION,
  NO PERSONA BEFORE CONTEXT, NO SECTARIAN JUDGMENT
- Rationalization Defense Table: 6 common AI excuses with rebuttals
- Red Flag List: 5 observable violation signals

Also add 40 pressure test cases (5 per master) covering:
- Sectarian judgment boundary
- No prophecy boundary
- Neutral first turn boundary
- Citation bypass pressure
- Master-specific challenges (meta, hostile, misunderstanding, etc.)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-08 21:28:41 +08:00
xianren 29a17ea281 docs: remove redundant "(手动)" from Claude Code install heading
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-07 14:16:06 +08:00
xianren 7f26cb3239 docs: add online demo section linking to fojin.app/chat
Users can now try all pre-built masters directly in the browser
without installing Claude Code or Codex CLI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-07 14:10:38 +08:00
xianren 59767e96b9 docs: update README for v0.3 — NPX install, architecture diagram, provenance features 2026-04-06 07:24:01 +08:00
xianren 623d88aa3e feat: v0.3 architecture rebuild — progressive disclosure, provenance, fidelity tests, NPX installer v0.3.0 2026-04-06 07:21:43 +08:00
xianren 88debde58d refactor(4-masters): rebuild fazang, huineng, kumarajiva, xuanzang to v0.3 architecture
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-06 07:18:52 +08:00
xianren d88da7b091 feat(scripts): add cite.py and query.py offline tools
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-06 07:13:56 +08:00
xianren 936ce09069 feat: add NPX installer CLI (install/list/uninstall)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-06 07:13:51 +08:00
xianren 937b642da8 refactor(zhiyi): v0.3 architecture rebuild — progressive disclosure + provenance + fidelity tests
Sample master (zhiyi) rebuilt to new architecture:
- SKILL.md slimmed from 225→94 lines with decision tree + Quick Ref
- Provenance frontmatter: CBETA IDs, FoJin text IDs, citation_format
- voice.md/teaching.md moved to references/ (loaded on demand)
- sources/ with canonical excerpts (offline-capable)
- tests/fidelity.jsonl: 5 Q&A pairs with expected citations/keywords
- scripts/validate.py: cross-master frontmatter linter
- scripts/test-fidelity.py: Claude API-based fidelity test runner

Follows Anthropic Agent Skills patterns:
  - Progressive disclosure (metadata→body→references)
  - Decision tree for branching workflows
  - Task-gated reference loading
  - Quick Reference table
  - Scripts as black boxes (--help, never Read source)
2026-04-06 07:05:43 +08:00
xianren 5b30b081c5 docs: add Codex CLI installation instructions
Codex CLI natively supports AgentSkills standard via ~/.codex/skills/.
Verified with codex-cli 0.118.0 — 10 skills registered successfully.
2026-04-05 13:33:50 +08:00
xianren 1de3bb9388 fix: correct FoJin URL format for juan paths
FoJin frontend route is '/texts/:id/read?juan=N' (query param),
not '/texts/{id}/juan/{N}' (path segment). Old format fell through
to React router 404.

Verified against FoJin frontend App.tsx route definitions:
- /texts/:id         → TextDetailPage
- /texts/:id/read    → TextReaderPage (uses ?juan= query)
2026-04-05 12:11:36 +08:00
xianren eb87794773 docs: simplify Community section wording 2026-04-05 10:50:45 +08:00
xianren 799beb6ba6 docs: add Community section linking to LINUX DO 2026-04-05 10:46:01 +08:00
xianren 1e0947acc4 docs: simplify v0.2 update banner 2026-04-05 10:16:10 +08:00
xianren 1b8554242d docs: add hero section with Diamond Sutra epigraph, badges, and navigation
Inspired by colleague-skill's README layout, adapted with dignified
Buddhist tone:
- Diamond Sutra verse as epigraph (in quotation marks, with source)
- Badges row (License, Python, Claude Code, AgentSkills)
- Scholar-oriented hook paragraphs (studying Yogacara, Chan, etc.)
- Serious tagline (no marketing flourish)
- Navigation quick-links
- v0.2 update banner
2026-04-05 10:13:39 +08:00
xianren 25aea07f6a refactor: rename Buddha-skill → Master-skill
Matches internal architecture (create-master, master_xuanzang, etc.)
and accurately reflects content (historical Chinese Buddhist masters,
not the Buddha himself).

Changes:
- 32 text references across 9 files updated
- Sanskrit terms (buddha-dhātu, buddhatā) preserved in dictionary context
- GitHub repo renamed (preserves stars/forks/history, auto-redirects)
- Internal naming already uses master_* (unchanged)

All 79 tests pass.
2026-04-05 10:03:42 +08:00
xianren 9dffcf3379 fix: enforce per-master RAG queries in /compare-masters with tradition-specific terms
Previously LLM might run ONE combined semantic query, causing the non-dominant
master to get fallback search URLs (fojin.app/search?q=...) instead of precise
text_id links.

Now the skill:
- Explicitly requires N separate queries for N masters (emphasized with ⚠️)
- Provides a query-rewriting table: each master's query uses its tradition's
  terminology (唯识/中观/禅宗/天台/华严/净土 terms)
- Example: user asks '因缘果' → xuanzang queries '因缘果 六因 四缘 五果 种子',
  kumarajiva queries '因缘所生法 空 中道 无自性 四句'

This ensures both masters get precise text_id citations, not degraded search URLs.
2026-04-05 09:43:31 +08:00
xianren 9873d0514d feat: add --brief flag to rag_query.py to reduce output verbosity
- New --brief flag for semantic/search subcommands outputs one-line-per-result
  (title + link + 80-char snippet) instead of full 500-char excerpts
- Reduces output from ~60 lines to ~7 lines per query (3 results)
- /compare-masters now uses --brief by default (multiple queries no longer
  flood the terminal with long excerpts)
- Full output (default) preserved for single-master /ask and debugging
2026-04-05 09:32:55 +08:00
xianren 16cbda8e50 fix: robust tool path resolution + precise selection feedback in /compare-masters
- Add PROJECT_ROOT discovery with 3 fallback paths (dev/manual/OpenClaw)
- Replace ${CLAUDE_SKILL_DIR}/tools/... with $PROJECT_ROOT/tools/...
  (CLAUDE_SKILL_DIR in subskills points to skill dir, not project root)
- Add graceful degradation when rag_query.py unreachable
- Output selection reasoning explicitly (e.g. 'xuanzang: 强关键词匹配(5项)
  + kumarajiva: 主题互补视角') instead of vague '主题映射'
2026-04-05 09:17:11 +08:00
xianren 6446903daf feat: smart master selection + expanded keywords (6→24 per master)
- /compare-masters now uses keyword matching from meta.json search_scope.keywords
- 3-tier selection: user-specified → keyword matching → topic fallback
- Expanded each master's keywords to 15-25 core concepts (avg 24)
- Document --masters manual override in README

Selection improvements verified:
- '遍行因' → xuanzang (5 matches)
- '明心见性' → huineng (3 matches)
- '空性/中观' → kumarajiva + huineng
- '念佛' → yinguang + ouyi + xuyun
2026-04-05 09:07:24 +08:00
xianren 02df9344b5 feat: 首轮身份中立原则 — masters no longer assume user identity on first turn
- Add Layer 0 hard rule to all 8 masters' voice.md: first turn must use
  neutral address (您/汝/你/问者), forbidden terms include 居士/行者/学人/
  善男子/善女人/出家人/师父/大众/道友/善信/道友
- From turn 2+: masters adapt to user's self-disclosed or question-inferred
  identity, restoring each master's historical address style
- Layer 2 开场方式/称呼方式 reorganized into 首轮中立 / 身份已知后 tiers
- Update voice_builder.md and voice_analyzer.md templates so future
  /create-master runs inherit this rule
- Add tools/sync_skill_from_voice.py to keep SKILL.md PART B in sync
- Add 48 regression tests in test_voice_rules.py (all 79 tests pass)
2026-04-05 08:44:35 +08:00
xianren e0afab5c9f fix: slugify now correctly lowercases English names and handles spaces 2026-04-05 08:11:31 +08:00
xianren 2a63a50ed3 test: add basic test suite for fojin_bridge, skill_writer, verify_sources (P1)
31 tests across 3 modules; all HTTP calls mocked, no real API dependencies.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-04-05 08:07:34 +08:00
xianren 6c6e19f2bc ci: add weekly GitHub Actions workflow for FoJin link verification (P2)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-04-05 08:06:40 +08:00
xianren 02d1069c46 feat: add /compare-masters skill for multi-master comparison (P1) 2026-04-05 08:05:46 +08:00
xianren 6f34e98508 docs: add complete FoJin API reference for LLM ad-hoc queries (P0)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-04-05 08:05:25 +08:00
xianren 5688f40c1a feat: graceful degradation when FoJin API is unavailable (P0)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-04-05 08:04:24 +08:00
xianren da2a7f2dc8 refactor: remove 南传/藏传 masters, focus exclusively on 汉传祖师大德 2026-04-05 07:42:00 +08:00
xianren 8da9b47dda docs: refocus PRD on 汉传佛教 — remove 南传/藏传 sections 2026-04-05 07:41:38 +08:00
xianren ce8024ac92 docs: simplify analyzer prompts — remove 南传/藏传 sections, focus on 汉传 2026-04-05 07:41:23 +08:00
xianren 4ef0c6fd70 docs: simplify acknowledgments — remove FoJin and colleague-skill 2026-04-05 07:29:07 +08:00
xianren d881797005 docs: expand SKILL.md with detailed flow control, error handling, and priority rules (P1)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-04 22:41:24 +08:00