Background
#1183 rewrote the ornn-agent-manual-cli frontmatter description. Two regressions slipped in and only surface when the skill is ingested (e.g. a GitHub refresh):
- Invalid YAML — the description is an unquoted plain scalar containing
SKILLSET request: bundle. A : (colon-space) inside a plain scalar makes the yaml parser read it as a nested mapping → INVALID_FRONTMATTER ("Nested mappings are not allowed in compact mappings at line 2, column 14"). skip_validation does not help — YAML parse precedes format validation.
- Over the length cap — the description was ~1400 chars;
skillFrontmatter.ts:225 caps description at .max(1024), so a proper validation would also reject it.
Fix
Rewrite the description to remove the embedded : (reword to a semicolon/comma clause) and trim to ≤1024 chars (now 993), preserving the trigger-oriented content. Verified with the API's own yaml.parse + validateSkillFrontmatter.
Definition of Done
GET/refresh of the skill parses + validates; description ≤1024; PR green + on develop.
Regression from #1183. Related to #1180.
Background
#1183 rewrote the
ornn-agent-manual-clifrontmatterdescription. Two regressions slipped in and only surface when the skill is ingested (e.g. a GitHub refresh):SKILLSET request: bundle. A:(colon-space) inside a plain scalar makes theyamlparser read it as a nested mapping →INVALID_FRONTMATTER("Nested mappings are not allowed in compact mappings at line 2, column 14").skip_validationdoes not help — YAML parse precedes format validation.skillFrontmatter.ts:225capsdescriptionat.max(1024), so a proper validation would also reject it.Fix
Rewrite the description to remove the embedded
:(reword to a semicolon/comma clause) and trim to ≤1024 chars (now 993), preserving the trigger-oriented content. Verified with the API's ownyaml.parse+validateSkillFrontmatter.Definition of Done
GET/refreshof the skill parses + validates; description ≤1024; PR green + ondevelop.Regression from #1183. Related to #1180.