feat(skills): support MCP-first execution and clarify CLI fallback hierarchy

TT
2mo ago
+100-47

Summary

This PR updates the core Firecrawl skills to support MCP-first execution while keeping the CLI as a clean fallback, resolving an issue where coding agents in MCP-enabled environments falsely report Firecrawl as unavailable or attempt unauthorized npx installations when the local firecrawl binary is not on PATH.

Changes

  1. Execution Hierarchy: Added explicit guidance in skills/firecrawl/SKILL.md and skills/firecrawl/rules/install.md establishing the priority order:
    • MCP First: Use exposed native Firecrawl MCP tools (firecrawl_search, firecrawl_scrape, firecrawl_crawl, firecrawl_map, firecrawl_agent, etc.) directly when available.
    • CLI Fallback: Use firecrawl CLI commands when running in standard terminal / CLI environments.
    • Disambiguation: Clarified that a missing local CLI binary on PATH does not mean Firecrawl is unavailable if MCP tools are configured.
    • Safety Boundary: Explicitly instructed agents not to perform unauthorized background npx firecrawl-cli init or npm installs.
  2. Tool Mappings: Added dual MCP / CLI quick-start references to skills/firecrawl/SKILL.md overview table and individual skill documents (firecrawl-search, firecrawl-scrape, firecrawl-map, firecrawl-crawl, firecrawl-agent, firecrawl-interact, firecrawl-parse, firecrawl-monitor, firecrawl-download).
  3. Frontmatter Cleanup: Removed restrictive allowed-tools: [Bash(firecrawl *), Bash(npx firecrawl-cli *)] from skill frontmatter (matching the pattern previously adopted in firecrawl-developer-index and firecrawl-research-index) so agent harnesses can invoke native MCP tools without restriction.

Verification

  • Ran pnpm test (all 23 test suites / 426 unit tests passing).
  • Ran pnpm format:check (all TypeScript and Markdown files formatted via Prettier).

Summary by cubic

Determines Firecrawl execution priority as MCP-first with CLI fallback in the core skill docs, so coding agents in MCP-enabled environments no longer report Firecrawl as unavailable or run unauthorized installs when the local firecrawl binary isn't on PATH.

  • Adds an execution hierarchy across the skill docs: call native MCP tools first, then CLI, and never install packages unless the user explicitly requested it.
  • Removes allowed-tools Bash frontmatter from skill definitions so agent harnesses can invoke MCP tools directly.
  • Adds MCP tool references alongside CLI commands in the overview table and individual skill guides.