Skip to content

Add automatic Cloudflare skills installation for AI coding agents - #13846

Closed
dario-piotrowicz wants to merge 1 commit into
mainfrom
dario/DEVX-2607/wrangler-install-cloudflare-skills
Closed

Add automatic Cloudflare skills installation for AI coding agents#13846
dario-piotrowicz wants to merge 1 commit into
mainfrom
dario/DEVX-2607/wrangler-install-cloudflare-skills

Conversation

@dario-piotrowicz

@dario-piotrowicz dario-piotrowicz commented May 7, 2026

Copy link
Copy Markdown
Member

Fixes https://jira.cfdata.org/browse/DEVX-2607

Wrangler now detects AI coding agent configuration directories (e.g. Claude Code, Cursor, Cline, Gemini CLI, OpenCode) and offers to install Cloudflare skill files from the cloudflare/skills GitHub repository. Users are prompted once interactively; subsequent runs skip the prompt. Use --experimental-force-skills-install (alias --x-force-skills-install) to install without prompting.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: self-explanatory UX improvement

A picture of a cute animal (not mandatory, but encouraged)

@changeset-bot

changeset-bot Bot commented May 7, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 861eb4a

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

✅ All changesets look good

ask-bonk[bot]

This comment was marked as resolved.

@ask-bonk

ask-bonk Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

I'm Bonk, and I've done a quick review of your PR.

Review posted successfully on PR #13846 with two findings:

  1. Network call before CI/interactivity checks (high severity)downloadSkillsToTempDir() runs before the ci.isCI and isInteractive() guards. In CI or non-interactive environments where no metadata file exists yet but agent config dirs are present, this makes an unnecessary HTTP request to GitHub on every first invocation, adding latency before discarding the result. Suggestion posted to move the cheap checks before the download at agents-skills-install.ts:63-101.

  2. Missing hidden: true on the global flag (low severity) — The experimental-force-skills-install option is missing hidden: true unlike the other experimental flags, so it will appear in --help output for every wrangler command. Suggestion posted at index.ts:533.

github run

@pkg-pr-new

pkg-pr-new Bot commented May 7, 2026

Copy link
Copy Markdown
create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@13846

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@13846

miniflare

npm i https://pkg.pr.new/miniflare@13846

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@13846

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@13846

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@13846

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@13846

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@13846

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@13846

wrangler

npm i https://pkg.pr.new/wrangler@13846

commit: 861eb4a

@dario-piotrowicz
dario-piotrowicz force-pushed the dario/DEVX-2607/wrangler-install-cloudflare-skills branch 5 times, most recently from 0660560 to a669003 Compare May 7, 2026 14:58
@dario-piotrowicz
dario-piotrowicz marked this pull request as ready for review May 7, 2026 15:41
@workers-devprod
workers-devprod requested review from a team and petebacondarwin and removed request for a team May 7, 2026 15:41
@workers-devprod

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • .changeset/agents-skills-install.md: [@cloudflare/wrangler]
  • packages/wrangler/package.json: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/agents-skills-install.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/experimental-commands-api.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/register-yargs-command-skills.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/vitest.setup.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/agents-skills-install.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/core/register-yargs-command.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/dev.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/index.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/metrics/send-event.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/yargs-types.ts: [@cloudflare/wrangler]
  • pnpm-lock.yaml: [@cloudflare/wrangler]

@devin-ai-integration devin-ai-integration 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.

Devin Review found 2 potential issues.

View 5 additional findings in Devin Review.

Open in Devin Review

Comment thread packages/wrangler/src/agents-skills-install.ts Outdated
Comment thread packages/wrangler/src/agents-skills-install.ts
@dario-piotrowicz
dario-piotrowicz force-pushed the dario/DEVX-2607/wrangler-install-cloudflare-skills branch from 57e3f2d to cc94fa4 Compare May 7, 2026 17:49
@dario-piotrowicz
dario-piotrowicz force-pushed the dario/DEVX-2607/wrangler-install-cloudflare-skills branch from cc94fa4 to 861eb4a Compare May 7, 2026 18:13
@dario-piotrowicz
dario-piotrowicz requested a review from MattieTK May 11, 2026 12:37
@dario-piotrowicz
dario-piotrowicz marked this pull request as draft May 12, 2026 13:51
@dario-piotrowicz

Copy link
Copy Markdown
Member Author

This PR is probably to be replaced by #13897 🙂

@dario-piotrowicz

Copy link
Copy Markdown
Member Author

Closing in favour of #13897

@github-project-automation github-project-automation Bot moved this from Untriaged to Done in workers-sdk May 20, 2026
@dario-piotrowicz
dario-piotrowicz deleted the dario/DEVX-2607/wrangler-install-cloudflare-skills branch June 3, 2026 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants