document local explorer via agent skill#13987
Conversation
|
|
Codeowners approval required for this PR:
Show detailed file reviewers |
There was a problem hiding this comment.
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):
- Stale reference in
check-generate-api.ts— The error message at line 67 points topackages/miniflare/src/workers/local-explorer/README.md, which this PR deletes. Anyone hitting the regeneration check error will be sent to a nonexistent file. - Self-referential dead link in
SKILL.md— Line 14 ofSKILL.mdtells agents to readpackages/miniflare/src/workers/local-explorer/README.mdbefore editing, but this PR deletes that file.
|
I'm Bonk, and I've done a quick review of your PR. I posted a review on PR #13987 with two findings:
|
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
21eaac9 to
953cc9a
Compare
dario-piotrowicz
left a comment
There was a problem hiding this comment.
LGTM, I just left some minor questions 🙂
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
Co-authored-by: Dario Piotrowicz <dario@cloudflare.com>
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... 🫠
A picture of a cute animal (not mandatory, but encouraged)