fix(skill): fix clerk skill install failing on published releases - #189

Merged
wyattjoh merged 2 commits into
mainfrom
fix/skill-install-frontmatter
Apr 20, 2026
Merged

fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom
fix/skill-install-frontmatter

Conversation

@wyattjoh

@wyattjohwyattjoh commented Apr 20, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted : as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray : fails locally instead of at install time.

Test plan

  • bun run test, bun run typecheck, bun run lint, bun run format:check
  • Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  • Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir

@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh

wyattjoh commented Apr 20, 2026

Copy link
Copy Markdown
ContributorAuthor

Stack: fix/skill-install-frontmatter

Part of a stacked PR chain. Do not merge manually.

@changeset-bot

changeset-botBot commented Apr 20, 2026•
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ebeb008

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

This PR includes changesets to release 1 package
NameType
clerkPatch

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

@coderabbitai

coderabbitaiBot commented Apr 20, 2026•
edited
Loading

Copy link
Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae

📥 Commits

Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.

📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough

Walkthrough

This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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

rafa-thayto
rafa-thayto reviewed Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md Outdated
@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh merged commit bc2e0f5 into mainApr 20, 2026
9 of 10 checks passed
@wyattjoh
wyattjoh deleted the fix/skill-install-frontmatter branch April 20, 2026 21:16
@github-actionsgithub-actionsBot mentioned this pull request Apr 20, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
fix(skill): fix clerk skill install failing on published releases - #189
Merged
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Apr 20, 2026
Merged
fix(skill): fix clerk skill install failing on published releases#189
wyattjoh merged 2 commits into
mainfrom 
fix/skill-install-frontmatter
Conversation
@wyattjoh
@wyattjohwyattjoh
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Summary
  • bunx clerk@canary skill install fails with No valid skills found because the bundled SKILL.md description contains raw HTTP: it handles. Strict YAML frontmatter parsers treat the unquoted :  as a nested mapping and reject the document, so the upstream skills CLI drops the skill as invalid.
  • Rewrite the description as a YAML folded block scalar (>-) so the whole value is a plain scalar regardless of internal punctuation, including future prose containing an apostrophe.
  • Add a regression test that parses the staged SKILL.md frontmatter via the already-present yaml dep, so a future stray :  fails locally instead of at install time.
Test plan
  • bun run test, bun run typecheck, bun run lint, bun run format:check
  •  Manual: bunx skills@1.5.1 add <staged-dir> against the fix now completes with Done! (reproduced in a temp dir, both with the CLI's own staging flow and a minimal stand-in)
  •  Re-verify once a canary publishes: bunx clerk@<canary> skill install in a fresh temp dir
@wyattjoh
fix(skill): fix clerk skill install failing on published releases
f6c424a
The bundled SKILL.md description contained a colon-space (`raw HTTP: it handles`)
which made strict YAML frontmatter parsers treat it as a nested mapping and
reject the skill as invalid, so `skills add` reported `No valid skills found`
and `clerk skill install` aborted. Wrap the description in single quotes so it
parses as a plain scalar regardless of internal punctuation, and add a
regression test that parses the staged frontmatter with the `yaml` dep.
@wyattjoh
wyattjoh
commented
Apr 20, 2026

Copy link


Copy Markdown

ContributorAuthor
Stack: fix/skill-install-frontmatter
Part of a stacked PR chain. Do not merge manually.
@changeset-bot
changeset-botBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: ebeb008
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
NameType
clerkPatch
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
@coderabbitai
coderabbitaiBot
commented
Apr 20, 2026•

edited

Loading

Copy link


Copy Markdown

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: 4cc3d45d-d582-4606-a062-72af9fe644ae
📥 Commits
Reviewing files that changed from the base of the PR and between f6c424a and ebeb008.
📒 Files selected for processing (1)
  • skills/clerk/SKILL.md
✅ Files skipped from review due to trivial changes (1)
  • skills/clerk/SKILL.md
📝 Walkthrough
Walkthrough
This PR adds a changeset marking clerk for a patch release and updates the clerk skill frontmatter so the bundled skills/clerk/SKILL.md frontmatter parses as strict YAML (the description is reformatted to a folded multiline block). It also adds a new test that reads the staged clerk/SKILL.md, extracts YAML frontmatter delimited by ---, parses it as strict YAML, and asserts name and description are non-empty strings.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and specifically describes the main fix: addressing clerk skill install failures on published releases related to frontmatter parsing issues.
Description check✅ PassedThe description thoroughly explains the problem (YAML parsing failures due to unquoted colons), the solution (YAML folded block scalar), and includes a comprehensive test plan with completed and pending steps.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
Comment @coderabbitai help to get the list of available commands and usage tips.
rafa-thayto
rafa-thayto
reviewed
Apr 20, 2026
View reviewed changes
Comment threadskills/clerk/SKILL.md
Outdated

@wyattjoh
refactor(skill): use folded block scalar for description frontmatter
ebeb008
Switch the SKILL.md description from a single-quoted scalar to a folded
block scalar (>-). Folded block scalars accept any characters in the
body, so future prose containing an apostrophe (e.g. "the user's app")
cannot silently reintroduce the parse failure this branch fixes. The
folded value is identical to the previous single-line string.
@wyattjoh
wyattjoh
merged commit bc2e0f5
into
mainApr 20, 2026

9 of 10 checks passed

@wyattjoh
wyattjoh
deleted the

fix/skill-install-frontmatter

branch
April 20, 2026 21:16
@github-actionsgithub-actionsBot
mentioned this pull request
Apr 20, 2026
 Merged

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


1 more reviewer

@rafa-thaytorafa-thaytorafa-thayto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@wyattjoh@rafa-thayto