Skip to content

fix(skill): use block-style YAML for SKILL.md metadata#197

Merged
senamakel merged 1 commit into
tinyhumansai:mainfrom
Giri-Aayush:fix/skill-frontmatter-yaml
Jun 26, 2026
Merged

fix(skill): use block-style YAML for SKILL.md metadata#197
senamakel merged 1 commit into
tinyhumansai:mainfrom
Giri-Aayush:fix/skill-frontmatter-yaml

Conversation

@Giri-Aayush

@Giri-Aayush Giri-Aayush commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What

The metadata block in website/public/SKILL.md (the canonical skill file, served at tiny.place/SKILL.md and symlinked from sdk/SKILL.md) was written as a JSON-style flow mapping with trailing commas. The Agent Skills reference validator rejects it:

$ skills-ref validate ./tinyplace
Invalid YAML in frontmatter: Found ugly disallowed JSONesque flow mapping

This rewrites the block as plain block-style YAML. The data is unchanged.

Why

agentskills.io is the open SKILL.md standard, and several downstream skill registries validate submissions with skills-ref. The flow-mapping frontmatter fails that check even though looser parsers (the skills CLI, ClawHub) accept it today, so the skill could not pass automated validation.

Verification

  • skills-ref validate passes after the change:
    Valid skill: ./tinyplace
    
  • Parsed the old and new frontmatter and confirmed every field is equal in meaning, including the nested metadata.openclaw install spec (deep-equality check passed). Only the YAML serialization changed.
  • sdk/SKILL.md and the served tiny.place/SKILL.md both resolve to this file, so they pick up the change with no extra edits.

Summary by CodeRabbit

  • Documentation
    • Updated the skill’s setup metadata to a clearer YAML format.
    • Added package, author, version, and installation details, including required command-line tools and install instructions.

The metadata block was a JSON-style flow mapping with trailing commas. The
Agent Skills reference validator (skills-ref) rejects that as invalid YAML
frontmatter, so the skill fails the agentskills.io standard that downstream
registries check against. Rewrite it as block-style YAML. The data is the
same; only the serialization changes.
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

@Giri-Aayush is attempting to deploy a commit to the Vezures Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0b9d07f3-f1bb-4c33-8c2f-c9f22370522a

📥 Commits

Reviewing files that changed from the base of the PR and between b34d00e and 98f701a.

📒 Files selected for processing (1)
  • website/public/SKILL.md

📝 Walkthrough

Walkthrough

Updated website/public/SKILL.md to replace JSON-style frontmatter metadata with YAML fields for the package, author, version, and openclaw install requirements.

Changes

SKILL frontmatter metadata update

Layer / File(s) Summary
YAML frontmatter rewrite
website/public/SKILL.md
The frontmatter metadata block is replaced with YAML fields for author, version, package, and an openclaw section with the CLI emoji, tinyplace binary requirement, and npm install entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 I hop through YAML, neat and bright,
Tinyplace fields now feel just right.
A carrot of metadata, fresh and sweet,
Makes my little bunny ears twitch in beat.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: converting SKILL.md metadata to block-style YAML.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@senamakel senamakel merged commit 9d1648a into tinyhumansai:main Jun 26, 2026
7 of 10 checks passed
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.

2 participants