Fix factual errors in Workers limits, D1 pricing, Agent wrangler config, and troubleshooting - #34
Merged
dmmulroy merged 1 commit intoMar 27, 2026
Conversation
…ig, and troubleshooting A systematic audit found 11 factual errors across 4 files that contradict live Cloudflare documentation. Every correction is cited to a specific URL. - gotchas.md: Fix CPU time limits (was 10ms/30ms, now Free:10ms/Paid:30s-5min), subrequests (add Free:50 vs Paid:10,000), KV size (25 MB -> 25 MiB), env var size (5 MB -> 5 KB per variable) - building-ai-agent SKILL.md: TOML -> JSONC, new_classes -> new_sqlite_classes - d1/README.md: Remove fabricated $5/db fee, fix read rate denominator ($0.001/1K -> $0.001/1M rows) - troubleshooting.md: Fix 3 wrangler snippets (TOML+new_classes -> JSONC+new_sqlite_classes) - examples.md: wrangler.toml -> wrangler.jsonc in directory tree
adewale
pushed a commit
to adewale/skills
that referenced
this pull request
Apr 2, 2026
…limits, and docs Incorporates upstream fixes from cloudflare/skills PRs cloudflare#34 and cloudflare#35: - Fix Workers CPU time limits (ms → seconds across multiple files) - Fix D1 pricing errors (removed fabricated $5/db fee, corrected read rate) - Fix KV pricing (was wrong by 10x) - Fix wrangler config references (TOML → JSONC) - Fix Agents SDK configuration (new_classes → new_sqlite_classes) - Fix sandbox docs (runCode() API, return types, exec() timeout) - Fix secrets store CLI syntax - Add missing cloudflare-api MCP server to README - Fix vectorize batch limit (500 → 1000) - Various other factual corrections https://claude.ai/code/session_01VMYzryVgfa9JUTK5PKA4XU
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Fix 11 factual errors across 4 files: wrong Workers platform limits, wrong D1
pricing, wrong wrangler config format and migration keys for Agents, and wrong
wrangler config in agent troubleshooting examples.
Why
A systematic audit found numbers, config snippets, and pricing claims that
contradict the live Cloudflare documentation. These skills are consumed by AI
agents that will reproduce the errors verbatim in generated code and advice.
Changes
Fixes 1, 2, 3, 7, 8 — Workers limits table and error description (
gotchas.md)File:
skills/cloudflare/references/workers/gotchas.mdThe limits table and the "Too much CPU time used" error description contained
five errors, each contradicted by the live limits page:
What changed: Rewrote the two CPU time rows to show Free vs Paid. Split
subrequests into Free/Paid rows. Changed "25 MB" to "25 MiB". Changed "5 MB"
to "5 KB per variable". Updated the error description on line 7 to match.
Fixes 4, 6 — Agent wrangler config (
building-ai-agent-on-cloudflare/SKILL.md)File:
skills/building-ai-agent-on-cloudflare/SKILL.mdlines 177-191The wrangler config block used TOML format and the legacy
new_classesmigration key.
new_classescreates a non-SQLite Durable Object. Agents require SQLite.The correct key is
new_sqlite_classes.wrangler.jsoncfor new projects; some newer featuresare JSON-only.
What changed: Replaced the
wrangler.tomlTOML block with an equivalentwrangler.jsoncJSON block usingnew_sqlite_classes.References:
Fix 5 — D1 pricing (
d1/README.md)File:
skills/cloudflare/references/d1/README.mdline 83The pricing line stated "$5/month per database beyond free tier + $0.001 per
1K reads + $1 per 1M writes + $0.75/GB storage/month". Two errors:
(1,000x cheaper than stated).
What changed: Replaced with "$0.001 per million rows read + $1.00 per
million rows written + $0.75/GB storage/month (includes free monthly allowance;
no per-database fee)".
Reference: https://developers.cloudflare.com/d1/platform/pricing/
Fixes 9, 10, 11 — Agent troubleshooting wrangler snippets
File:
skills/building-ai-agent-on-cloudflare/references/troubleshooting.mdThree troubleshooting sections used
wrangler.tomlwithnew_classes:new_classes→new_sqlite_classesnew_classes→new_sqlite_classesSame justification as fixes 4 and 6. These are the same class of error in
the same skill's reference file.
References:
Fix 12 — Directory tree listing (
examples.md)File:
skills/building-ai-agent-on-cloudflare/references/examples.mdline 41The project structure diagram listed
wrangler.toml. Changed towrangler.jsoncfor consistency with all other config references in the skill.
Reference: https://developers.cloudflare.com/workers/wrangler/configuration/
Testing
building-ai-agent-on-cloudflare/use JSONC andnew_sqlite_classesgotchas.mdlimits table matches the live docs URL citedRisk
Documentation-only changes. No code, runtime behaviour, or bindings modified.
Every correction is cited to a specific live URL to make it auditable.