feat(skills): add opencli-skill and document Codex/Cursor/Claude Code integration#442
feat(skills): add opencli-skill and document Codex/Cursor/Claude Code integration#442shouldnotappearcalm wants to merge 27 commits intojackwener:mainfrom
Conversation
…odules - Add getErrorMessage() to errors.ts (used in 5 files) - Add DEFAULT_DAEMON_PORT to constants.ts (used in 5 files) - Reduces code duplication and improves maintainability
|
add douyin, jd, linkedin, pixiv, web. |
…jd, linkedin, pixiv, web and improve SKILL.md description Made-with: Cursor
Astro-Han
left a comment
There was a problem hiding this comment.
A few structural concerns before diving into individual files:
1. Path mismatch in root SKILL.md
The diff adds a "Skill Package in Repo" section to SKILL.md pointing to skills/opencli-platform-router/, but the actual directory created is skills/opencli-skill/. These don't match.
2. README Claude Code section overwrites existing CLAUDE.md
The suggested cat > CLAUDE.md <<'EOF' command would overwrite the repo's existing CLAUDE.md (which contains adapter development rules, the CLI-EXPLORER reference, etc.). Should be append or a separate file.
3. Duplicate command reference with no auto-generation
The root SKILL.md already has a comprehensive command reference. This PR adds 63 hand-written markdown files that duplicate that information in a different format. Without a generation script, these will drift out of sync every time a command is added or changed. A script that generates from adapter source (or --help output) would make this sustainable.
4. Source file changes in a docs-only PR
src/build-manifest.ts and src/validate.ts have blank-line removals. Minor, but the PR description says "No runtime behavior changes to OpenCLI core execution paths", so source files shouldn't be touched.
|
Follow-up to my earlier review: this is not just a future drift risk, the new skill command reference is already incomplete in this PR. I compared the new
So the problem here is stronger than "these docs may drift later": they already cannot be treated as the canonical command reference for agents, because an agent following these docs today would conclude that some real commands do not exist. |
- add generator/check scripts for command refs and SKILL.md\n- add unified skill:sync / skill:check and CI workflow\n- regenerate skills/opencli-skill references from src/clis\n- fix SKILL.md path references and README CLAUDE.md append behavior\n- restore formatting-only blank lines in src/build-manifest.ts and src/validate.ts
1. Path Mismatch in Root
|
|
Hi, thanks for the thorough review! All 4 structural concerns have been addressed in the latest update:
Beyond the fixes, I've also built out the auto-generation logic for skill content — command references and SKILL.md descriptions are now derived directly from source adapters, so they stay in sync automatically. Would you mind taking a look and letting me know if the approach seems reasonable? Happy to adjust if needed. |
Regenerate opencli-skill docs from updated src/clis after conflict resolution to keep coverage and commands aligned.
Overview
This PR packages OpenCLI platform knowledge into a reusable skill layout and documents how to wire it into common agent environments.
Included in this PR
Why
OpenCLI already exposes broad platform coverage, but agents need a deterministic, loadable interface.
opencli-skill provides that interface and standardizes how agents discover commands, required args, and expected usage patterns.
Scope