Skip to content

Suggest Cloudflare skills installation after commands instead of before#14264

Merged
dario-piotrowicz merged 3 commits into
mainfrom
dario/install-skills-scoped
Jun 12, 2026
Merged

Suggest Cloudflare skills installation after commands instead of before#14264
dario-piotrowicz merged 3 commits into
mainfrom
dario/install-skills-scoped

Conversation

@dario-piotrowicz

@dario-piotrowicz dario-piotrowicz commented Jun 11, 2026

Copy link
Copy Markdown
Member

Fixes #14116

Before wrangler on every command would trigger the skills installation flow when it checks for agents configurations and after an interactive prompt potentially installs them for the user (at most once).

This had many various edge cases and downsides, like:

  • disturbing/surprising users with this interactive prompt unrelated to their current task
  • interrupting the user's flow
  • not working well for parallel wrangler executions (where they could each hid the prompt of the other, effectively blocking them)

Now instead the question is presented after the successful execution of non-long lived commands (such as wrangler login), this doesn't insert the prompt upfront for every command so it should address all the problematic edge case scenarios we've seen.

Additionally this PR adds extra logic to save the skills metadata file before asking the prompt (and potentially overriding it after), this should ensure that user should not see this prompt more than once.


Example of new flow:

$ npx https://pkg.pr.new/wrangler@14264 whoami 

 ⛅️ wrangler 4.100.0 (dario/install-skills-scoped)
──────────────────────────────────────────────────
Getting User settings...
👋 You are logged in with an OAuth Token, associated with the email dario@cloudflare.com.
┌──────────────┬──────────────────────────────────┐
│ Account Name │ Account ID                       │
├──────────────┼──────────────────────────────────┤
│ ...          │ ...                              │
└──────────────┴──────────────────────────────────┘
🔓 Token Permissions:
Scope (Access)
- ...

? Before you go, Wrangler detected potential configurations for the following AI coding agents on your system likely without Cloudflare skills: Claude Code, OpenCode, GitHub Copilot. Would you like Wrangler to automatically install the Cloudflare skills for you? › (Y/n)

To try the flow just run one of the Wrangler commands that trigger this using the prerelease, like for example:

$ npx https://pkg.pr.new/wrangler@14264 whoami 

(To trigger this make sure not to have a agents-skills-install.jsonc file in your global .wrangler directory)


  • 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 feature / behavior

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


Open in Devin Review

@changeset-bot

changeset-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c78b74f

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

This PR includes changesets to release 3 packages
Name Type
wrangler Minor
@cloudflare/vite-plugin Patch
@cloudflare/vitest-pool-workers Patch

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

@workers-devprod workers-devprod requested review from a team and jamesopstad and removed request for a team June 11, 2026 15:34
@workers-devprod

workers-devprod commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/cloudchamber
  • @cloudflare/d1
  • @cloudflare/workers-kv
  • ✅ @cloudflare/wrangler
Show detailed file reviewers
  • packages/wrangler/src/tests/containers/delete.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/images.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/info.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/instances.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/list.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/push.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/registries.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/containers/ssh.test.ts: [@cloudflare/cloudchamber]
  • packages/wrangler/src/tests/d1/d1.test.ts: [@cloudflare/d1]
  • packages/wrangler/src/tests/kv/help.test.ts: [@cloudflare/workers-kv]
  • packages/wrangler/src/tests/kv/key.test.ts: [@cloudflare/workers-kv]
  • packages/wrangler/src/tests/kv/namespace.test.ts: [@cloudflare/workers-kv]

@dario-piotrowicz dario-piotrowicz marked this pull request as draft June 11, 2026 15:36
ask-bonk[bot]

This comment was marked as resolved.

@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

✅ All changesets look good

@ask-bonk

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Jun 11, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jun 11, 2026

Copy link
Copy Markdown
create-cloudflare

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

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14264

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14264

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: c78b74f

@dario-piotrowicz dario-piotrowicz force-pushed the dario/install-skills-scoped branch 4 times, most recently from cb90570 to 9663ed3 Compare June 11, 2026 18:09
@dario-piotrowicz dario-piotrowicz marked this pull request as ready for review June 11, 2026 18:19
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@dario-piotrowicz dario-piotrowicz force-pushed the dario/install-skills-scoped branch from dbe7564 to bc545c5 Compare June 11, 2026 19:45
devin-ai-integration[bot]

This comment was marked as resolved.

Comment thread packages/wrangler/src/core/register-yargs-command.ts Outdated
@github-project-automation github-project-automation Bot moved this from Untriaged to In Review in workers-sdk Jun 12, 2026

@NuroDev NuroDev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Codeowners Bypass

@workers-devprod workers-devprod 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.

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from In Review to Approved in workers-sdk Jun 12, 2026
@dario-piotrowicz dario-piotrowicz merged commit 21dbc12 into main Jun 12, 2026
63 of 64 checks passed
@dario-piotrowicz dario-piotrowicz deleted the dario/install-skills-scoped branch June 12, 2026 14:49
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Jun 12, 2026
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.

Interruptive interactive prompts for unrelated commands for ai skills installs with wrangler@>=4.94.0

4 participants