Repository navigation
Conversation
There was a problem hiding this comment.
All reported issues were addressed across 6 files
Shadow auto-approve: would not auto-approve because issues were found.
View guided diff | Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would require human review. Adds an official-docs reference to the developer-index skill and registers two new workflow skills, with tests and a version bump. Merge safety depends on an external catalog sync not visible in the diff; the installer silently skips missing skills.
View guided diff | Turn on auto-fix | Re-trigger cubic
Name the situations where model knowledge goes stale (a named version, an upgrade, a pasted library error, an unfamiliar API) so agents look up current documentation instead of answering from memory, and state what the skill is not for.
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would require human review. Adds an official-docs reference to the developer-index skill, registers two workflow skills in the CLI installer, adds tests, and bumps the version. Merge safety depends on an external catalog sync (firecrawl/firecrawl-workflows#11) that is not visible in this diff.
View guided diff | Turn on auto-fix | Re-trigger cubic
Why
Official documentation questions should get version-aware guidance through the existing developer-index skill, and the CLI needs to install the two new workflow skills. This resubmits #318, reverted in #319, with one correction.
Summary
firecrawl-developer-index, guidance that distinguishes a merged fix from released behavior and names the release check (release notes, changelog entry or tag),firecrawl-job-postingsandfirecrawl-fantasy-sportsregistered in the workflow selection (18 workflows), and the init and setup tests.firecrawl/skills. Until then,firecrawl setup job-postingsandfirecrawl setup fantasy-sportsexit with an error because the catalog does not have those names, andfirecrawl setup workflowsinstalls the other 16 workflows without them.Test Plan
pnpm run build,pnpm run type-check,pnpm run format:checkpnpm test: 38 files and 689 tests passed withFIRECRAWL_API_KEYunset--help;firecrawl developerandfirecrawl search --categories developer --sources webrun against productionfirecrawl-developer-indexfrom this branch into an isolated project with the standard skills installer; the reference file is packaged