Skip to content

feat: add use-siyuan CLI skill - #2

Merged
vuon9 merged 2 commits into
mainfrom
feat/use-siyuan-skill
Sep 15, 2026
Merged

vuon9 merged 2 commits into
mainfrom
feat/use-siyuan-skill

Conversation

@vuon9

@vuon9 vuon9 commented Sep 15, 2026

Copy link
Copy Markdown
Owner

What

Adds a new first-party skill, use-siyuan, documenting the siyuan kernel CLI (v3.8.3), plus a chore to make the local plugin manifest list all local skills.

Why

The siyuan CLI exposes the full SiYuan data model across 17 groups and ~90 leaf commands, but agents burn many --help calls discovering it and routinely hit the default-workspace trap.

Contents

  • skills/use-siyuan/SKILL.md - workspace resolution, mental model, core workflows, output/safety rules, gotchas
  • skills/use-siyuan/references/commands.md - full command catalog with signatures
  • skills/use-siyuan/README.md - house-style companion
  • setup-vstack/skills.json - registered as mine / vuon9/vstack
  • .claude-plugin/plugin.json - added use-siyuan and the previously missing automate-me

Key facts encoded

  • No running kernel needed; the CLI embeds it and works directly on workspace files
  • Data commands default to $HOME/SiYuan and fail with directory not found; resolve with siyuan workspace list and pass -w
  • -w <path> registers the path in ~/.config/siyuan/workspace.json as a side effect
  • -f json for parsing, global --dry-run, never hand-edit .sy files

Testing

RED then GREEN with a subagent on the same discovery task:

  • Without the skill: 10+ --help calls, hit the directory not found wall, confused by a second registered workspace
  • With the skill: 4 commands, no help spelunking, correct outline

Mutations were verified on a throwaway workspace copy (document/block/attr/dailynote/export/sql), then removed. setup-vstack/scripts/verify.py passes and the full unittest suite is green (26 tests).

@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown

Review: feat/use-siyuan-skill — PASS

  • Changed: Added skills/use-siyuan/ (SKILL.md:2 name use-siyuan, README.md, references/commands.md) + registered in setup-vstack/skills.json:150 as mine/vuon9/vstack; fixed .claude-plugin/plugin.json:8,14 to list automate-me + use-siyuan (now 10 locals).
  • Required sync: skills.json required:true set (7: brainstorming, hunk-review, receiving-code-review, requesting-code-review, test-driven-development, unslop, wayfinder) matches vmode/SKILL.md:33-44 exactly in both directions.
  • Manifest invariants: All 33 entries have name==SKILL.md frontmatter, source is owner/repo or tree URL or local, scope is mine/external, and every mine points at vuon9/*.
  • Repo-local checks: use-siyuan frontmatter name hyphen-case, description starts Use when (skills/use-siyuan/SKILL.md:3), README.md present; same holds for vmode, setup-vstack, automate-me, bro, etc.
  • No mismatch — no manage.py fix needed. If adding a required skill: python3 setup-vstack/scripts/manage.py set-required <name> true.
  • Verdict: PASS — ready to merge.

New%20session%20-%202026-09-15T07%3A09%3A35.987Z
opencode session  |  github run

@vuon9
vuon9 merged commit c39ab1a into main Sep 15, 2026
2 checks passed
@vuon9
vuon9 deleted the feat/use-siyuan-skill branch September 15, 2026 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant