skills: parse SKILL.md frontmatter with yaml.safe_load

Replaces the hand-rolled block-scalar walker (added one commit ago) with
PyYAML's safe_load. PyYAML's default SafeLoader is pure-Python — no C
extension required — and handles every YAML edge case for free instead
of reimplementing them.

Side-benefit: also fixes a second latent bug. The regex parser stripped
the outer YAML quotes but left inner `\"` escapes intact as literal
backslash-quote characters, so descriptions like
`"... mentions \"switch workspace\"..."` ended up in manifest.json
with the backslashes preserved. yaml.safe_load resolves these
correctly. Regenerated manifest reflects the fix for databricks-config.

Co-authored-by: Isaac

Author: jamesbroadhead

manifest.json

@@ -1,6 +1,6 @@
 {
   "version": "2",
-  "updated_at": "2026-05-12T21:53:00Z",
+  "updated_at": "2026-05-12T21:58:09Z",
   "skills": {
     "databricks-apps": {
       "version": "0.1.1",
@@ -253,7 +253,7 @@
     },
     "databricks-config": {
       "version": "0.0.1",
-      "description": "Manage Databricks workspace connections: check current workspace, switch profiles, list available workspaces, or authenticate to a new workspace. Use when the user mentions \\\"switch workspace\\\", \\\"which workspace\\\", \\\"current profile\\\", \\\"databrickscfg\\\", \\\"connect to workspace\\\", or \\\"databricks auth\\\".",
+      "description": "Manage Databricks workspace connections: check current workspace, switch profiles, list available workspaces, or authenticate to a new workspace. Use when the user mentions \"switch workspace\", \"which workspace\", \"current profile\", \"databrickscfg\", \"connect to workspace\", or \"databricks auth\".",
       "experimental": true,
       "updated_at": "2026-05-12T21:15:40Z",
       "files": [

scripts/skills.py

@@ -9,6 +9,8 @@
 from datetime import datetime, timezone
 from pathlib import Path
 
+import yaml
+
 
 SHARED_ASSETS = [
     "assets/databricks.svg",
@@ -194,17 +196,8 @@ def check_assets_synced(repo_root: Path) -> list[str]:
 # Manifest generation
 # ---------------------------------------------------------------------------
 
-_BLOCK_SCALAR_INDICATORS = {"|", "|-", "|+", ">", ">-", ">+"}
-
-
 def extract_description_from_skill(skill_path: Path) -> str:
-    """Best-effort extraction of `description:` from SKILL.md frontmatter.
-
-    Handles plain (`description: foo`), quoted (`description: "foo"`), and
-    block-scalar (`description: >-` followed by indented lines) values. The
-    regex-only version captured the block-scalar indicator verbatim, which
-    corrupted manifest entries and Codex marketplace metadata.
-    """
+    """Best-effort extraction of `description:` from SKILL.md frontmatter."""
     skill_md = skill_path / "SKILL.md"
     if not skill_md.exists():
         return ""
@@ -214,24 +207,9 @@ def extract_description_from_skill(skill_path: Path) -> str:
     end_idx = content.find("---", 3)
     if end_idx == -1:
         return ""
-    lines = content[3:end_idx].splitlines()
-    for i, line in enumerate(lines):
-        m = re.match(r'^description:\s*(.*?)\s*$', line)
-        if not m:
-            continue
-        value = m.group(1)
-        if value in _BLOCK_SCALAR_INDICATORS:
-            collected = []
-            for cont in lines[i + 1:]:
-                if cont and not cont[0].isspace():
-                    break
-                stripped = cont.strip()
-                if stripped:
-                    collected.append(stripped)
-            joiner = " " if value.startswith(">") else "\n"
-            return joiner.join(collected)
-        return value.strip().strip('"').strip("'")
-    return ""
+    parsed = yaml.safe_load(content[3:end_idx]) or {}
+    description = parsed.get("description", "")
+    return description.strip() if isinstance(description, str) else ""
 
 
 # Markers that separate the "what this skill does" lead-in from the