fix(server): recover claude skill scalar frontmatter

[maria-rcks]

claude accepts descriptions containing an unquoted : , but t3's scanner drops those skills from the $ picker. when strict yaml parsing fails, this change repairs plain scalar values and reparses the full document, preserving invocation metadata and rejecting malformed structures.

closes #7757. closes #7814.

verification:

• blacksmith: all 21 focused discovery tests, scoped lint/formatting, and server typecheck passed. the new discovery case failed on the baseline.
• actual claude cli 2.1.288 in an isolated config: initialization inventory accepted the exact issue description. the baseline $ picker omitted kane; after provider refresh, filtering and selecting the real row inserted its skill chip.
• invocation flags remained intact; hidden skills stayed out of the picker and an unclosed tool list stayed undiscovered. external kane tool execution is unverified.

before:

after:

7-second recording of actual filtering and selection (verified uploaded playback):

https://uploads-production-47e4.up.railway.app/files/c6e6abf2-e1a1-407c-9532-c047f3334fb5/skill-selection.mp4

model: unknown-model. harness: codex.

[macroscopeapp]

Approvability

Verdict: Approved at 017de12

Macroscope's review found this PR approvable — This narrowly scoped server fix restores discovery of Claude skills with valid Claude-style scalar descriptions while preserving invocation metadata and rejecting unrelated malformed frontmatter. Focused tests cover the recovery cases and confirm malformed documents remain excluded.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: bc21c9b3-f325-4699-a495-11803cfbcfde

📥 Commits

Reviewing files that changed from the base of the PR and between 7379933 and 017de12.

📒 Files selected for processing (2)

• apps/server/src/provider/Drivers/ClaudeSkills.test.ts
• apps/server/src/provider/Drivers/ClaudeSkills.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.

---

📝 Walkthrough

Walkthrough

parseSkillFrontmatter now retries YAML parsing after repairing eligible plain scalar values containing a colon followed by a space. Tests cover skill discovery metadata and malformed frontmatter cases.

Changes

Claude skill frontmatter

Layer / File(s)	Summary
Frontmatter repair and validation apps/server/src/provider/Drivers/ClaudeSkills.ts, apps/server/src/provider/Drivers/ClaudeSkills.test.ts	The parser retries after quoting eligible colon-bearing plain scalar values. Tests check discovery metadata and malformed YAML cases.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 017de

Skills whose descriptions contain a colon followed by a space should now appear in the picker, and malformed frontmatter is still skipped. No merge-blocking risk was identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Issue #7757 requires T3 to discover Claude skills with plain-scalar descriptions containing : , or to log rejected files if that mismatch remains. ClaudeSkills.ts retries parsing after quoting eli…
Out of Scope Changes check	✅ Passed	The PR changes only Claude skill frontmatter parsing and its tests. Both changes directly support the parser-strictness mismatch in issue #7757. No unrelated changes appear in the whole-PR diff.
Title check	✅ Passed	The title clearly and concisely describes the main change: recovering Claude skill frontmatter with scalar descriptions that strict YAML parsing rejects.
Description check	✅ Passed	The description explains the problem and change, references the related issues, and gives detailed test and CLI verification results. It does not include explicit maintainer approval or explain why th…

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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