Skip to content

fix(sync): parse skill descriptions with yaml - #67

Merged
dreyfus92 merged 3 commits into
mainfrom
fix/sync-description
Sep 24, 2026
Merged

dreyfus92 merged 3 commits into
mainfrom
fix/sync-description

Conversation

@natemoo-re

Copy link
Copy Markdown
Member

bsh sync wrote a garbled skill index to AGENTS.md:

- **test** — [skills/test/SKILL.md](skills/test/SKILL.md) - > Vitest test runner with colocated

Two bugs:

  • Block scalars leak their indicator. Every bundled skill uses description: >, and ultramatter returns the > as part of the value. It also mangles |-, >-, and |2 (e.g. |- → "- Lint it."). This PR replaces ultramatter, which only sync used, with yaml, a zero-dependency YAML parser.
  • The first-sentence split cut at any period, so .test.ts truncated the test description. It now splits only on a period followed by whitespace or end of string.

Output against the bundled skills now:

- **test** — [skills/test/SKILL.md](skills/test/SKILL.md) - Vitest test runner with colocated .test.ts files and test utilities from @bomb.sh/tools/test-utils

updateAgentsMd is now exported so it can be tested directly.

ultramatter returns YAML block scalar indicators (`>`, `|-`) as part of
the value, and the first-sentence split broke on any period, e.g. `.test.ts`.
@changeset-bot

changeset-bot Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5fc3caf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bomb.sh/tools Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@socket-security

socket-security Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedyaml@​2.8.3 ⏵ 2.9.1100 +110010090100

View full report

@pkg-pr-new

pkg-pr-new Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@bomb.sh/tools@67

commit: 5fc3caf

@dreyfus92 dreyfus92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thank you @natemoo-re 💜

@dreyfus92
dreyfus92 merged commit 1071d34 into main Sep 24, 2026
6 checks passed
@dreyfus92
dreyfus92 deleted the fix/sync-description branch September 24, 2026 02:05
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