Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
/bonk please review this PR for any issues |
|
Posted review on PR #28658. Here's the summary: One issue found — filename typo in AGENTS.md.
The rest of the PR looks solid:
|
|
Preview URL: https://ab5c6953.preview.developers.cloudflare.com |
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
|
this is great @mvvmm - thank you |
There was a problem hiding this comment.
Could you make use of this function perhaps: https://github.com/cloudflare/cloudflare-docs/blob/production/src/util/custom-loaders.ts#L20 for grabbing the file from middlecache?
Also, is there a way to re-use some of the content collection pattern for doing any of this? Or not really useful because all the changes are done before the tarball is generated?
No for a couple of reasons I think. This function saves to
I think the |
|
Used |
|
/bonk mind reviewing this one one more time after my new changes? |
|
UnknownError: "Overloaded" |
|
@mvvmm Bonk workflow failed. Check the logs for details. View workflow run · To retry, trigger Bonk again. |
Summary
Removes the committed copy of Cloudflare Skills from the repo and replaces it with a fetch-at-build-time approach sourcing from middlecache.
bin/fetch-skills.ts— fetchesskills.tar.gzfrom middlecache and extracts intoskills/. Skips ifskills/already exists;--forceoverrides.--softmode (used by predev) warns on failure instead of blocking.package.json— addsprebuildandpredevhooks to run the script automaticallyskills/— removed from git, added to.gitignoreIn dev, fetch runs once and is skipped on subsequent restarts; network failures warn but don't block. In CI,
skills/is always fetched fresh since it isn't committed — fetch failure fails the build.The well known skills can be verified here: https://11705275.preview.developers.cloudflare.com/.well-known/skills/index.json