Skip to content

document local explorer via agent skill#13987

Merged
emily-shen merged 4 commits into
mainfrom
emily/explorer-skill
May 21, 2026
Merged

document local explorer via agent skill#13987
emily-shen merged 4 commits into
mainfrom
emily/explorer-skill

Conversation

@emily-shen
Copy link
Copy Markdown
Contributor

@emily-shen emily-shen commented May 20, 2026

Some docs and a diagram for the local explorer.

Is this a good place to put this, or should I put this closer to the local explorer code?
This current spot (.opencode/skills) might be more discoverable for agents, but less for humans... 🫠


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

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


Open in Devin Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 20, 2026

⚠️ No Changeset found

Latest commit: 15b3f83

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk May 20, 2026
@workers-devprod workers-devprod requested review from a team and dario-piotrowicz and removed request for a team May 20, 2026 14:58
@workers-devprod
Copy link
Copy Markdown
Contributor

workers-devprod commented May 20, 2026

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

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

This PR moves the local-explorer contributor guide from a short README into a detailed OpenCode agent skill (with diagram) and deletes the original README.

Issues (by severity):

  1. Stale reference in check-generate-api.ts — The error message at line 67 points to packages/miniflare/src/workers/local-explorer/README.md, which this PR deletes. Anyone hitting the regeneration check error will be sent to a nonexistent file.
  2. Self-referential dead link in SKILL.md — Line 14 of SKILL.md tells agents to read packages/miniflare/src/workers/local-explorer/README.md before editing, but this PR deletes that file.

Comment thread packages/miniflare/src/workers/local-explorer/README.md
Comment thread .opencode/skills/local-explorer/SKILL.md Outdated
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 20, 2026

I'm Bonk, and I've done a quick review of your PR. I posted a review on PR #13987 with two findings:

  1. Stale reference in check-generate-api.ts (line 67) — The error message still points users to the now-deleted packages/miniflare/src/workers/local-explorer/README.md. This needs updating to point to the new skill location.

  2. Self-referential dead link in SKILL.md (line 14) — The skill file tells agents to read packages/miniflare/src/workers/local-explorer/README.md as a prerequisite, but this PR deletes that file. Suggested replacing with a reference to the diagram file instead.

github run

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

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 bugs or issues to report.

Open in Devin Review

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 20, 2026

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 15b3f83

@emily-shen emily-shen force-pushed the emily/explorer-skill branch from 21eaac9 to 953cc9a Compare May 20, 2026 15:31
@dario-piotrowicz dario-piotrowicz added the no-changeset-required Skip PR check for valid changeset label May 21, 2026
Copy link
Copy Markdown
Member

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

LGTM, I just left some minor questions 🙂

Comment thread packages/miniflare/scripts/check-generate-api.ts Outdated
Comment thread .opencode/skills/local-explorer/assets/local-explorer-diagram.md
Copy link
Copy Markdown
Contributor

@workers-devprod workers-devprod left a comment

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 Untriaged to Approved in workers-sdk May 21, 2026
Co-authored-by: Dario Piotrowicz <dario@cloudflare.com>
@emily-shen emily-shen enabled auto-merge (squash) May 21, 2026 13:32
@emily-shen emily-shen merged commit 65a4b4e into main May 21, 2026
54 checks passed
@emily-shen emily-shen deleted the emily/explorer-skill branch May 21, 2026 13:49
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changeset-required Skip PR check for valid changeset

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants