Skip to content

fix(skills): 裁剪 firecrawl-research-index 的 description(1006 → 561 字符) - #289

Open
qikairo7 wants to merge 1 commit into
firecrawl:mainfrom
qikairo7:chore/trim-research-index-description
Open

qikairo7 wants to merge 1 commit into
firecrawl:mainfrom
qikairo7:chore/trim-research-index-description

Conversation

@qikairo7

@qikairo7 qikairo7 commented Sep 29, 2026 •

Copy link
Copy Markdown

改了什么

skills/firecrawl-research-index/SKILL.md 的 frontmatter description 从 1006 字符裁到 561 字符,只动这一个文件的这一行(1 file changed, +1/-1)。

为什么

这段 description 会注入每个加载该技能的 agent 会话。1006 字符的描述已贴近 agentskills.io 规范的 1024 上限,余量近乎为零。它还超出 GitHub Copilot 对 skill description 的截断限制(见 #165 对 firecrawl-monitor 的同类修复)。agent 每次会话都要为这段文字付出上下文成本,其中大部分是内部工具机制枚举,对触发判断没有帮助。

保留了什么

裁剪只删机制枚举(内部工具名、缓存细节),以下信息原样保留:

  • 语料库范围:arXiv、PubMed、bioRxiv 等学术源
  • 路由守卫:需要跨源学术检索时用 firecrawl_search 的 categories=["research"]
  • 核心触发词与 NOT-for 边界

验证

  • 改动前后 frontmatter 均通过 YAML 解析(yaml.safe_load),description 长度 561
  • git diff --check 干净,git diff --name-only 仅此一文件

Summary by cubic

修剪 skills/firecrawl-research-index/SKILL.md 的 frontmatter description,从 1006 字符减至 561 字符。

该描述会注入每个加载此技能的 agent 会话,原长度已贴近 agentskills.io 的 1024 上限,并超出 GitHub Copilot 的截断限制。裁剪只删除了内部工具机制枚举,语料库范围、firecrawl_search 的 categories=["research"] 路由守卫、核心触发词与 NOT-for 边界均原样保留。改动前后 frontmatter 均通过 YAML 解析。

Written for commit 7af2bef. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Shadow auto-approve: would auto-approve. Trims the skill frontmatter description from 1006 to 561 characters, lowering per-session context cost and staying under length limits while preserving corpus scope, routing guard, and core trigger boundaries.

Re-trigger cubic

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