Skip to content

[codex] Add required skill frontmatter#1

Open
liqiorg wants to merge 2 commits into
aresbit:mainfrom
liqiorg:codex/add-skill-frontmatter
Open

[codex] Add required skill frontmatter#1
liqiorg wants to merge 2 commits into
aresbit:mainfrom
liqiorg:codex/add-skill-frontmatter

Conversation

@liqiorg
Copy link
Copy Markdown

@liqiorg liqiorg commented May 29, 2026

Summary

  • Add required YAML frontmatter to SKILL.md.
  • Clarify the skill description for URL fetching, conversion, and archiving workflows.

Why

SKILL.md was missing frontmatter delimited by triple dashes, so skill loaders that require name and description metadata may fail to discover it.

Validation

  • Parsed the frontmatter with Ruby YAML and confirmed name and description are present.

@cr-gpt
Copy link
Copy Markdown

cr-gpt Bot commented May 29, 2026

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds frontmatter metadata, including a name and description, to the top of the SKILL.md file. The review feedback suggests formatting the long description using a folded block scalar (>-) to improve readability and maintainability, and points out an inconsistency with the description in the main body of the document.

Comment thread SKILL.md Outdated
@@ -1,3 +1,8 @@
---
name: fetch-skill
description: 用于抓取、转换或归档 URL 内容的技能。自动识别普通网页、X/Twitter 单条推文、回复、用户时间线、X Article 和微信公众号文章,并路由到合适后端,输出干净的 Markdown、JSON 或纯文本;默认零依赖,按需使用 Camofox 或 wechat-article-exporter 增强。
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The description in the frontmatter is very long (over 200 characters) on a single line, which can make it difficult to read and maintain. It is recommended to use a folded block scalar (>-) to break the text into multiple lines for better readability.

Additionally, please note that this description is inconsistent with the **Description:** section in the main body of the document (lines 10-11). Consider updating the main body description to match this new, more detailed version to ensure consistency.

Suggested change
description: 用于抓取、转换或归档 URL 内容的技能。自动识别普通网页、X/Twitter 单条推文、回复、用户时间线、X Article 和微信公众号文章,并路由到合适后端,输出干净的 Markdown、JSON 或纯文本;默认零依赖,按需使用 Camofox 或 wechat-article-exporter 增强。
description: >-
用于抓取、转换或归档 URL 内容的技能。自动识别普通网页、X/Twitter 单条推文、回复、用户时间线、X Article 和微信公众号文章,并路由到合适后端,输出干净的 Markdown、JSON 或纯文本;默认零依赖,按需使用 Camofox 或 wechat-article-exporter 增强。

@liqiorg liqiorg marked this pull request as ready for review May 29, 2026 07:43
@cr-gpt
Copy link
Copy Markdown

cr-gpt Bot commented May 29, 2026

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

@liqiorg
Copy link
Copy Markdown
Author

liqiorg commented May 29, 2026

Updated the frontmatter description to use a folded block scalar (>-) and synced the body description to match it.

The remaining cr-gpt comments appear to be caused by a missing OPENAI_API_KEY in the repository bot configuration, not by this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant