refactor(skills): rewrite and optimize agent skills - #196
Open
chenxin-yan wants to merge 6 commits into
Open
Conversation
Descriptions are always-loaded context; trigger-first wording cuts token load while keeping every distinct trigger branch.
… content - Router SKILL.md 329 -> 141 lines: pure routing; monitor details, install/auth, and search-feedback specifics now live only in their sub-skills, with routing pointers left behind. - Moved two unique monitor facts into firecrawl-monitor (markdown-only diff.json parse-diff AST note; goal bullet about user-named noise). - Resolved keyless contradiction: keyless free tier is real for search/scrape/interact (src/utils/client.ts isKeylessMode; scrape and search absent from AUTH_REQUIRED_COMMANDS in src/index.ts). Single correct statement kept in rules/install.md, including which commands prompt interactive login.
…positive phrasing, completion criteria - Replace cached CLI option tables with 'firecrawl <cmd> --help' pointers, keeping only options the recipes actually use (all verified against src/) - Delete 'When to use' sections that restated frontmatter descriptions - Split firecrawl-monitor: lean recipe in SKILL.md, goal/query authoring in goals.md, JSON change tracking in json-tracking.md, each behind a one-line 'read when' pointer - Rewrite never/don't phrasing as positive instructions; keep genuine opt-out and data guardrails - Add exactly one 'Done when' completion criterion per skill skills/**/*.md drops from 1045 to 837 lines (SKILL.md files: 1045 -> 723).
…d auth terminal rules EXP-046 forensics: claude SKOP-NEU-02 skipped Firecrawl in optimized arms (lost web-research trigger); monitor traces looped on 429 to 600s timeouts; bad-key traces retried identical failing calls.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 13 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
…hen accuracy, valid JSON example
chenxin-yan
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
rewrite agent skills to follow best practices, reducing context bloat, better skills invocation, eliminate duplication among skill files.
problems
<command> --helpto get up to date optionsfirecrawl/rules/install.md, monitor detail → the monitor skill, search feedback → the search skill) and the router points to ittoken impact (~4 chars/token estimate)
Skill bodies (loaded on invocation):
Frontmatter descriptions (always loaded in every agent's skill catalog):
eval results