Skip to content

fix: correct skill name and quote YAML description#38

Merged
rfgamaral merged 3 commits intomainfrom
fix/skill-name-and-yaml
Mar 25, 2026
Merged

fix: correct skill name and quote YAML description#38
rfgamaral merged 3 commits intomainfrom
fix/skill-name-and-yaml

Conversation

@rfgamaral
Copy link
Copy Markdown
Member

Overview

The skill YAML frontmatter had two issues: the name was outline instead of outline-cli (which didn't match the install directory, breaking skill detection), and the description contained an unquoted colon that broke YAML parsing. This renames the skill to outline-cli to match the convention used by todoist-cli and twist-cli, quotes the description, and uses SKILL_NAME in the install path for consistency.

Test plan

  • ol skill install installs without conflicts

Rename SKILL_NAME from "outline" to "outline-cli" to match the install
directory convention used by todoist-cli and twist-cli. Quote the
description in YAML frontmatter to prevent colons from breaking parsing.
Use SKILL_NAME in the install path for consistency.
@rfgamaral rfgamaral self-assigned this Mar 25, 2026
@rfgamaral rfgamaral added the 👀 Show PR PR must be reviewed before or after merging label Mar 25, 2026
@rfgamaral rfgamaral requested review from a team and craigcarlyle and removed request for a team March 25, 2026 15:23
Copy link
Copy Markdown

@doistbot-app doistbot-app Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR corrects the skill name to outline-cli and fixes YAML parsing by quoting the description, nicely aligning with established conventions and resolving skill detection for new installations. It is worth noting that existing local installations from previous versions will retain the broken frontmatter unless a migration or forced-reinstall path is introduced.

Share FeedbackReview Logs

Comment thread src/lib/skills/content.ts
@rfgamaral rfgamaral merged commit e0e4fa4 into main Mar 25, 2026
3 checks passed
github-actions Bot pushed a commit that referenced this pull request Mar 25, 2026
## [1.0.1](v1.0.0...v1.0.1) (2026-03-25)

### Bug Fixes

* correct skill name and quote YAML description ([#38](#38)) ([e0e4fa4](e0e4fa4))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@craigcarlyle
Copy link
Copy Markdown

@rfgamaral rfgamaral deleted the fix/skill-name-and-yaml branch March 26, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released 👀 Show PR PR must be reviewed before or after merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants