Skip to content

docs: list use-siyuan in README - #3

Merged
vuon9 merged 1 commit into
mainfrom
docs/readme-use-siyuan
Sep 15, 2026
Merged

vuon9 merged 1 commit into
mainfrom
docs/readme-use-siyuan

Conversation

@vuon9

@vuon9 vuon9 commented Sep 15, 2026

Copy link
Copy Markdown
Owner

What

Adds the new use-siyuan skill to the root README's Written by me list.

Why

PR #2 merged the skill but the root README was not updated, so the skill listing was stale.

Testing

Docs-only change. setup-vstack/scripts/verify.py and the unittest suite are unaffected.

@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown

Skills review — PASS

  • PR docs: list use-siyuan in README #3 is docs-only (README.md:55): adds use-siyuan to Written by me to sync with PR feat: add use-siyuan CLI skill #2; no manifest add/remove/re-scope in setup-vstack/skills.json:1-186.
  • Required-set matches bidirectionally: setup-vstack/skills.json:17-168 has 7× required:true (brainstorming, wayfinder, test-driven-development, unslop, hunk-review, requesting-code-review, receiving-code-review) ↔ vmode/SKILL.md:33-44 same 7 bullets.
  • All 36 manifest entries have valid source (owner/repo or https://github.com/.../tree/... or local) and scope mine/external; every minevuon9/*.
  • Repo-local skills (vmode/SKILL.md:2, setup-vstack/SKILL.md:2, skills/*/SKILL.md:1-4) — name matches folder/manifest (^[a-z0-9-]+$), description starts Use when, README.md exists next to each SKILL.md.
  • No mismatch found; no setup-vstack/scripts/manage.py fix required (e.g. python3 setup-vstack/scripts/manage.py set-required <name> true not needed). Verdict: PASS.

New%20session%20-%202026-09-15T09%3A12%3A42.189Z
opencode session  |  github run

@vuon9
vuon9 merged commit 4a7f394 into main Sep 15, 2026
2 checks passed
@vuon9
vuon9 deleted the docs/readme-use-siyuan branch September 15, 2026 09:14
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