Skip to content

docs(skills): fix npx allowed-tools pattern and broken cross-skill links - #189

Open
chenxin-yan wants to merge 2 commits into
firecrawl:mainfrom
chenxin-yan:skills/hygiene
Open

docs(skills): fix npx allowed-tools pattern and broken cross-skill links#189
chenxin-yan wants to merge 2 commits into
firecrawl:mainfrom
chenxin-yan:skills/hygiene

Conversation

@chenxin-yan

@chenxin-yan chenxin-yan commented Aug 17, 2026

Copy link
Copy Markdown

What

  • allowed-tools: Bash(npx firecrawl *)Bash(npx firecrawl-cli *) across all skills. The npm package firecrawl is the JS SDK and has no bin, so npx firecrawl fails on a clean machine (could not determine executable to run). It only works by accident when a local node_modules/.bin/firecrawl shim exists. npx firecrawl-cli resolves this package and runs its firecrawl bin reliably everywhere.
  • Fix broken relative links in the workflow escalation pattern: [...](firecrawl-cli)[...](../firecrawl-cli/SKILL.md).
  • Drop hardcoded v1.8.0 from an example output so it doesn't drift with releases.

Verification

$ cd /tmp && bunx firecrawl --version
error: could not determine executable to run for package firecrawl
$ bunx firecrawl-cli --version
1.20.0

Docs-only change; no source touched.

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 10 files

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