Skip to content

Remove skill runner - #2100

Open
aron-cf wants to merge 3 commits into
mainfrom
skill-runner-removal
Open

Remove skill runner#2100
aron-cf wants to merge 3 commits into
mainfrom
skill-runner-removal

Conversation

@aron-cf

@aron-cf aron-cf commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Agent Skills currently combine loading task instructions with optional script execution. This makes
SkillRegistry responsible for runtime permissions and adds a runner, build-time compilation, and
execution-specific dependencies to an otherwise read-only resource system.

This change removes skills.runner, run_skill_script, agents/skills/compile, execution-only
types, and the Think runner hook. Vite now preserves script files as inert resources. The
first-party examples use read_skill_resource or existing workspace tools instead. The Turndown
stub remains because Think's workspace Bash tool still imports just-bash.

The package builds and repository checks pass. The changeset, package README, examples, and Think guides document the breaking API removal and direct applications toward explicit tools when they need execution.

Keep first-party skills as instructions and readable resources. The
release-notes example now reads its style guide, while the assistant
workspace digest uses the existing workspace tools.
Keep Agent Skills focused on loading instructions and resources. Remove
the runner API, execution tool, compiler entry point, and Think hook.
Script files remain inert resources that models can read on demand.
Describe Agent Skills as on-demand instructions and inert resources.
Point applications that need execution to explicit tools with their own
permissions.
@changeset-bot

changeset-bot Bot commented Aug 12, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8ad7ab3

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

This PR includes changesets to release 3 packages
Name Type
agents Minor
@cloudflare/think Minor
@cloudflare/agent-think 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

@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: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@pkg-pr-new

pkg-pr-new Bot commented Aug 12, 2026

Copy link
Copy Markdown

Open in StackBlitz

agents

npm i https://pkg.pr.new/agents@2100

@cloudflare/ai-chat

npm i https://pkg.pr.new/@cloudflare/ai-chat@2100

@cloudflare/codemode

npm i https://pkg.pr.new/@cloudflare/codemode@2100

hono-agents

npm i https://pkg.pr.new/hono-agents@2100

@cloudflare/shell

npm i https://pkg.pr.new/@cloudflare/shell@2100

@cloudflare/think

npm i https://pkg.pr.new/@cloudflare/think@2100

@cloudflare/voice

npm i https://pkg.pr.new/@cloudflare/voice@2100

@cloudflare/worker-bundler

npm i https://pkg.pr.new/@cloudflare/worker-bundler@2100

commit: 8ad7ab3

@ben-reitz ben-reitz 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.

You think it's worth adding an example for how people can allow their agents to run scripts that are included in skills now?

(I don't know how commonly skills include scripts, but it is part of the skills spec...)

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.

2 participants