[Bug] skill frontmatter 含「冒号+空格」的 description 未加引号时被静默忽略，catalog 完全无感知 / Skills with colon-containing unquoted description are silently dropped from the catalog

[yuyukosama2004]

[Bug] skill frontmatter 含「冒号+空格」的 description 未加引号时被静默忽略，catalog 完全无感知 / Skills with colon-containing unquoted description are silently dropped from the catalog

现象 / Symptom

~/.dsh/skills/<name>/SKILL.md 的 frontmatter 中，description 值如果包含「冒号+空格」（如 Priority order: check the curated list first）且未用引号包裹，YAML 解析会在 parseFrontmatter 抛 Nested mappings are not allowed in compact mappings，skill 被静默移除：

• 会话 skill catalog（<available_skills>）中完全不出现该 skill；
• 用户与模型侧无任何提示，仅后端 logger.warn（skill file ... ignored: invalid YAML frontmatter）；
• 排查只能靠逐文件比对 catalog 差异，极难发现（今日实测，三个新 skill 全部中招后才发现）。

This is a concrete trigger of the same defect class described in #936 (malformed frontmatter entries are dropped with only a logger.warn, and the catalog cannot distinguish an absent skill from an invalid one). Here the trigger is a YAML parse error rather than a name/description validation failure — same silent removal, same undiagnosable shape.

复现 / Reproduce

mkdir -p ~/.dsh/skills/foo
cat > ~/.dsh/skills/foo/SKILL.md <<'EOF'
---
name: foo
description: Use when the user asks to find plugins. Priority order: check the curated list first.
---
EOF
# 启动 dsh web 后，foo 不会出现在 skill 工具的 <available_skills> 目录中

修复方式：description: "Use when ... Priority order: check ..."（双引号包裹）后立即生效。

环境 / Environment

• @deepseek-ai/dsh 0.1.0-rc.6（CLI），deepseek-harness-desktop 0.3.4
• Windows 11 / PowerShell 5.1
• 相关包：@deepseek-ai/dsh-skill-filesystem（parseSkillFile → parseFrontmatter 抛错路径，仅 ctx.logger.warn 后 return）

建议 / Suggestion

与 #936 的修复建议一致（rejected 计数 / failOnInvalidSkill / 启动摘要均适用），此处补充一个真实触发形态供确认修复优先级。

/cc #936

[Electricitysheep]

frontmatter 里 description 含「冒号+空格」未加引号被静默忽略——YAML 解析陷阱，插件开发常见坑。规避：YAML 值里含冒号/特殊字符必加引号。

插件/技能开发规范见第 4 章：https://github.com/Electricitysheep/dsh-handbook/blob/main/docs/04-plugin-dev.md

[yuyukosama2004]

感谢指导

[Electricitysheep]

不客气，能帮上忙就好。手册欢迎随时来提意见或补充（frontmatter 这类 YAML 坑我们也会陆续收进第 4 章）。

[yuyukosama2004]

补充：同类问题的第二个触发形态 + 一个可复现的 modlens 联动缺陷

1. 已解决：切换到 (modlens vision) 变体后粘贴识图正常

服务端排查确认一切正常（插件挂载、/modlens/paste 路由、read_image 工具、模型变体均已生效），问题的表象（「模型不支持图片」）来自未切换模型变体。解法：在模型选择器选择 DeepSeek-V4-Flash (modlens vision) 变体后粘贴即可，请求时由 modlens 自动转换图片为证据文本。

2. 发现的联动缺陷：verdict 遍历未排除自身 wrapper provider，导致 paste-to-path（玩法①）永远不触发

排查中发现 pasteTakeoverVerdict()（dsh/index.js）的一个逻辑问题，可能值得单独反馈给 modlens 作者：

• verdict 遍历所有 provider 的模型，按 label 匹配；匹配到 inputModalities 含 image 的模型即 veto（return false）；
• 但 modlens 自己注册的 wrapper provider（deepseek-modlens）复用原始模型 id（如 deepseek-v4-flash），仅改 name 为 DeepSeek-V4-Flash (modlens vision)、inputModalities: ['text','image']；
• 于是对任一 label（如 deepseek-v4-flash）：
  1. 先匹配 deepseek-official 的纯文本模型 → 本应返回接管（takeover: true）；
  2. 继续遍历到同 id 的 wrapper 模型 → 含 image → veto，整体返回 false；
• 结果：GET /modlens/paste?model=... 对任何 label 都返回 {"takeover":false}，client.js 的 paste-to-path 永不触发（实测确认）。

建议：verdict 遍历时应跳过自身注册的 wrapper provider（或在匹配时排除 provider === providerId 的条目），否则玩法①（纯文本模型直接粘贴转路径）在存在 wrapper 变体的配置下不可用。当前仅玩法②（切换变体）可用。

环境备注

• @deepseek-ai/dsh 0.1.0-rc.6 / deepseek-harness-desktop 0.3.4 / Windows 11
• modlens 3.16.1（~/.dsh/profiles/web），provider 为 dashscope 千问（openai 兼容）
• 以上两个问题与本帖主题（skill frontmatter 静默失败）无直接关系，仅因排查过程关联记录于此；如需独立讨论可以拆帖。

[boyin111-1]

Very reproducible - and the silent part is the worst part: no catalog entry, no user-facing error, just a logger.warn. I added an offline scan for exactly this to https://github.com/boyin111-1/dsh-doctor (check 15): it walks ~/.dsh/skills and every preset's skills/ directory, and flags any SKILL.md whose unquoted description contains an ASCII ': ' (the YAML compact-mapping trigger) with the exact file path and a suggested quoted form. It also covers #1450's variant (the ': ' inside longer descriptions) and relates to #936's rejected-count idea - the scanner is the 'tell me which skills are being dropped and why' half.

The real fix should make parseFrontmatter failures visible (count rejected skills, surface them in the startup summary or a --dump-skills debug command) - the scan is just the offline tripwire until then.