refactor!: converge the repo to skills + website (Codex-executed work packages)#75
Conversation
Executed by Codex (gpt-5.6-sol), designed and reviewed by Claude. - Delete plugins/, .claude-plugin/, shared/mcp-config/, sync-skills.mjs, and the empty CHANGELOG shell + conventional-changelog dep - Claude Code installs like every other client now: claude mcp add + copy skills/ into ~/.claude/skills (the marketplace channel is retired) - Inline the Cursor / VS Code / Windsurf config templates into AGENT-INSTALL.md; drop the sync CI step Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…-rich descriptions Executed by Codex, designed and reviewed by Claude. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Executed by Codex (gpt-5.6-sol), designed and reviewed by Claude; review fixup: graceful error when skills/ is missing. Guards against the two past incident classes: frontmatter drift and skills referencing tools that do not exist in yuque-mcp (the whitelist lives in scripts/known-tools.json). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… everywhere Executed by Codex (gpt-5.6-sol), designed and reviewed by Claude; review fixup: mkdir -p before the skills copy one-liner. - Install section: configure yuque-mcp + copy skills/ (marketplace flow is gone); other clients pointed at AGENT-INSTALL.md - Upgrade: git pull + re-copy; npx picks up MCP server updates itself - Remove EditionTabs/ComingSoon; team edition is one hint line under Skills until the group tools ship - Hero/Modules/Footer copy updated to MCP Server + Skills narrative Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 75da562a17
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # 2. Install the plugin — required step | ||
| claude plugin install yuque-personal@yuque | ||
| # 1. MCP server | ||
| claude mcp add yuque-mcp -- npx -y yuque-mcp --token=$YUQUE_TOKEN |
There was a problem hiding this comment.
With this new two-step Claude Code path, the skills are copied to ~/.claude/skills (user-wide), but this claude mcp add uses Claude Code's default local scope. The official docs say “By default it registers the server at local scope” and to pass --scope user for all projects (https://code.claude.com/docs/en/mcp-quickstart), so anyone running the install from the cloned checkout gets globally discoverable Yuque skills that fail in other projects because the MCP tools are only registered locally; add --scope user here or make the MCP and skills both project-scoped.
Useful? React with 👍 / 👎.
1 participant
概要
按确认的改造计划执行：仓库收敛为 skills + 官网。四个工作包由 Codex (gpt-5.6-sol) 编码、Claude 设计任务卡并三段验收（边界核查 → 功能验收 → code review）。
WP-A
92686d7— 拆除 plugin 体系（breaking）plugins/、.claude-plugin/、shared/mcp-config/、同步脚本、CHANGELOG 空壳及 conventional-changelog 依赖claude mcp add+ 复制skills/（marketplace 渠道退役，已装用户需按新方式重装）WP-B
3236e85— skills 资产质量统一WP-C
520d018— CI 护栏scripts/validate-skills.mjs（零依赖）：frontmatter 规则 + 工具引用白名单校验（scripts/known-tools.json，防"skills 引用不存在工具上架"事故复发）WP-D
75da562— 官网对齐git pull+ 覆盖复制；说明 npx 自动更新 MCP servermkdir -p验证
npm run check-skills8/8 ✅（红绿注入测试通过）tsc -b && vite build✅ /eslint✅ / 浏览器 DOM 断言 10/10 ✅🤖 Generated with Claude Code