执行摘要
- 一句话:将 issue 模板 model 字段改为自由输入
- 推荐动作:该 PR 是典型的基础设施改进,变更清晰、风险低,值得快速合并。对于参与 cookbook 添加流程的贡献者,建议阅读变更后的文档以了解新行为。
功能与动机
原有的 dropdown 字段需要硬编码选项列表,每添加一个新模型 cookbook 都需要编辑模板;同时 GitHub issue 表单的 dropdown prefill 有一个潜在 bug,只有当 URL 参数值完全匹配某个选项时才触发 prefill,否则静默失败。从非 deepseek-ai/deepseek-v4 模型页面提交的 issue 会导致该字段为空。详见 PR body。
实现拆解
- 调整 issue 模板字段类型:在
.github/ISSUE_TEMPLATE/3-playground-verified-cell.yml 中将 model 字段从 type: dropdown 改为 type: input,保留 deepseek-ai/deepseek-v4 作为 placeholder 示例。这样新模型 cookbook 无需再修改模板。
- 更新 cookbook-add-model 配置模板:在
.claude/skills/cookbook-add-model/templates/config.jsx.tmpl 中修改注释,说明 github.cookbookModel 用于预填自由文本输入框,并强调不要删除该 block,否则引擎会回退到 deepseek-ai/deepseek-v4 导致提交标签错误。
- 更新 cookbook-review-pr 检查清单:在
.claude/skills/cookbook-review-pr/SKILL.md 中修改相关检查项,说明 model 字段现在是自由输入,由 github.cookbookModel 预填。
- 更新 authoring-reference 文档:在
.claude/skills/cookbook-add-model/references/authoring-reference.md 中更新 github 配置项的说明,匹配新的自由输入行为。
关键文件:
.github/ISSUE_TEMPLATE/3-playground-verified-cell.yml(模块 Issue 模板;类别 infra;类型 infrastructure): 核心变更文件:将 model 字段从 dropdown 改为 input,移除硬编码选项,添加 placeholder。
.claude/skills/cookbook-add-model/templates/config.jsx.tmpl(模块 技巧模板;类别 other;类型 data-contract): 更新了注释,说明 cookbookModel 用于预填自由输入字段,并强调不要删除该 block。
.claude/skills/cookbook-review-pr/SKILL.md(模块 技巧文档;类别 docs;类型 documentation): 更新了 review 检查清单中关于 model 字段的描述。
.claude/skills/cookbook-add-model/references/authoring-reference.md(模块 技巧文档;类别 docs;类型 documentation): 更新了 github 配置项的表格说明,匹配新的自由输入行为。
关键符号:未识别
关键源码片段
.github/ISSUE_TEMPLATE/3-playground-verified-cell.yml
核心变更文件:将 model 字段从 dropdown 改为 input,移除硬编码选项,添加 placeholder。
# .github/ISSUE_TEMPLATE/3-playground-verified-cell.yml (partial)
- type: input # 原来是 dropdown
id: model
attributes:
label: Cookbook model
description: >
Which model's cookbook does this cell belong to?
Auto-filled by the playground — use the model id as it appears in the cookbook config.
placeholder: "deepseek-ai/deepseek-v4" # 保留默认值作为示例
validations:
required: true
.claude/skills/cookbook-add-model/templates/config.jsx.tmpl
更新了注释,说明 cookbookModel 用于预填自由输入字段,并强调不要删除该 block。
// .claude/skills/cookbook-add-model/templates/config.jsx.tmpl (partial)
// Prefills the issue template's free-form `model` field on "Submit verified cell".
// Use the HF id (`<hf-org>/<model-slug>`). Do NOT delete this block when pruning —
// without it the engine falls back to "deepseek-ai/deepseek-v4" and mislabels submissions.
github: {
cookbookModel: "__HF_ORG__/__MODEL_SLUG__",
},
评论区精华
该 PR 没有 review 评论。只有一个来自 Fridge003 的 APPROVED 审核,无额外讨论。
风险与影响
- 风险:风险极低。模板字段类型从 dropdown 改为 input 是向后兼容的,placeholder 保留了原有默认值。唯一潜在风险是如果外部工具或自动化脚本依赖 dropdown 的选项值,但该模板仅用于手动提交 issue,且 field id 保持不变,因此 prefill URL 不受影响。
- 影响:影响范围小,主要影响 playground verified-cell issue 的提交流程:用户现在可以自由输入任何模型 ID,不再受限于预设选项。同时简化了维护:新增模型 cookbook 时无需再更新该模板。关联的技能文档和模板注释也已同步更新。
- 风险标记:暂无
关联脉络
- PR #26885 (假设)添加 playground verified-cell issue 模板的原始 PR: 本 PR 修改的模板是在 PR#26885 中添加的,本 PR 是对其的改进。
参与讨论