Skip to content

Comments

Clarify scope and clean up limits docs#27597

Merged
elithrar merged 3 commits intoproductionfrom
docs/clarify-limits-scope
Feb 8, 2026
Merged

Clarify scope and clean up limits docs#27597
elithrar merged 3 commits intoproductionfrom
docs/clarify-limits-scope

Conversation

@elithrar
Copy link
Collaborator

@elithrar elithrar commented Jan 13, 2026

Adds explicit scope (per account) to ambiguous limits rows in D1, Durable Objects, and KV. Removes duplicate footnotes and standardizes footnote syntax across limits pages.

Addresses #25551.

@github-actions github-actions bot added product:d1 D1: https://developers.cloudflare.com/d1/ product:durable-objects Durable Objects: https://developers.cloudflare.com/workers/learning/using-durable-objects/ labels Jan 13, 2026
@github-actions github-actions bot added product:kv product:r2 R2 object storage: https://developers.cloudflare.com/r2 product:vectorize Vectorize: https://developers.cloudflare.com/vectorize/ labels Jan 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/d1/ @elithrar, @rozenmd, @vy-ton, @joshthoward, @oxyjun, @harshil1712, @cloudflare/pcx-technical-writing
/src/content/docs/durable-objects/ @elithrar, @vy-ton, @joshthoward, @oxyjun, @lambrospetrou, @mikenomitch, @cloudflare/pcx-technical-writing
/src/content/docs/kv/ @elithrar, @thomasgauvin, @oxyjun, @cloudflare/pcx-technical-writing
/src/content/docs/r2/ @oxyjun, @elithrar, @jonesphillip, @aninibread, @harshil1712, @cloudflare/workers-docs, @cloudflare/pcx-technical-writing
/src/content/docs/vectorize/ @elithrar, @vy-ton, @sejoker, @mchenco, @cloudflare/pcx-technical-writing

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026 •
edited
Loading

Preview URL: https://cec6e761.preview.developers.cloudflare.com
Preview Branch URL: https://docs-clarify-limits-scope.preview.developers.cloudflare.com

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/durable-objects/platform/limits/ https://docs-clarify-limits-scope.preview.developers.cloudflare.com/durable-objects/platform/limits/
https://developers.cloudflare.com/r2/platform/limits/ https://docs-clarify-limits-scope.preview.developers.cloudflare.com/r2/platform/limits/
https://developers.cloudflare.com/kv/platform/limits/ https://docs-clarify-limits-scope.preview.developers.cloudflare.com/kv/platform/limits/
https://developers.cloudflare.com/d1/platform/limits/ https://docs-clarify-limits-scope.preview.developers.cloudflare.com/d1/platform/limits/
https://developers.cloudflare.com/vectorize/platform/limits/ https://docs-clarify-limits-scope.preview.developers.cloudflare.com/vectorize/platform/limits/

@elithrar
Copy link
Collaborator Author

elithrar commented Feb 5, 2026

/bonk resolve merge conflicts and re-review.

@opencode-agent
Merge origin/production into docs/clarify-limits-scope
a4bd217 
Resolve conflict in kv/platform/limits.mdx: keep 'Namespaces per account'
scope clarification from this branch, accept cacheTtl minimum update
(60s -> 30s) from production.
@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 5, 2026

Summary

Merge Conflict Resolved

There was one conflict in src/content/docs/kv/platform/limits.mdx. The conflict was between:

  • This PR: Changed Namespaces to Namespaces per account (scope clarification) and formatted 1000 as 1,000
  • Production: Updated cacheTtl minimum from 60 seconds to 30 seconds

Resolution: Kept both changes -- the "per account" scope clarification from this PR and the corrected 30 seconds cacheTtl value from production.

Review of All 5 Files

All files look correct. The PR accomplishes its goals:

File Changes Status
D1 limits "Databases" -> "Databases per account"; removed duplicate <Details> footnotes; standardized [^N] syntax Good
DO limits "Maximum Durable Object classes" -> adds "(per account)"; removed duplicate <Details> footnotes Good
KV limits "Namespaces" -> "Namespaces per account"; fixed cacheTtl to 30s (from production) Good
R2 limits Migrated from <sup>N</sup> to [^N] footnote syntax; reordered footnotes logically Good
Vectorize limits Deprecated V1 section moved into collapsible <Details>; minor formatting fixes Good

Pre-existing Issues (not introduced by this PR)

  1. Broken anchor in DO limits: Footnotes [^4] and [^7] link to #increasing-durable-object-cpu-limits, but the actual heading in the rendered partial is ### Can I increase Durable Objects' CPU limit? (anchor: #can-i-increase-durable-objects-cpu-limit).
  2. Minor typo in D1 limits: "10 restores per 10 minute" should be "10 minutes" (plural).
  3. Minor inconsistency in D1 limits: (Workers Paid)[^2] vs (Workers Paid) [^1] -- inconsistent spacing before footnote reference.

github run

@elithrar elithrar enabled auto-merge (squash) February 6, 2026 20:16
@elithrar
Copy link
Collaborator Author

elithrar commented Feb 6, 2026

/bonk fix the outstanding issues too.

@opencode-agent @elithrar
Fix DO anchors, D1 typo & spacing
cec6e76 
Co-authored-by: elithrar <elithrar@users.noreply.github.com>
@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 6, 2026

All three fixes are applied correctly:

  1. DO limits - broken anchors (limits.mdx:32,68): Changed #increasing-durable-object-cpu-limits to #can-i-increase-durable-objects-cpu-limit in both [^4] and [^7] footnotes, matching the actual heading anchor from the rendered FAQ partial.

  2. D1 limits - typo (limits.mdx:16): Fixed 10 minute to 10 minutes.

  3. D1 limits - footnote spacing (limits.mdx:14): Added missing space before [^2] so it matches the style of [^1] on the line above ((Workers Paid) [^2]).

github run

@elithrar elithrar merged commit 39ffd2d into production Feb 8, 2026
9 checks passed
@elithrar elithrar deleted the docs/clarify-limits-scope branch February 8, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TownLake TownLake TownLake approved these changes

@Oxyjun Oxyjun Awaiting requested review from Oxyjun Oxyjun is a code owner

@jonesphillip jonesphillip Awaiting requested review from jonesphillip jonesphillip is a code owner

@aninibread aninibread Awaiting requested review from aninibread aninibread is a code owner

@harshil1712 harshil1712 Awaiting requested review from harshil1712 harshil1712 is a code owner

@thomasgauvin thomasgauvin Awaiting requested review from thomasgauvin thomasgauvin is a code owner

@vy-ton vy-ton Awaiting requested review from vy-ton vy-ton is a code owner

@joshthoward joshthoward Awaiting requested review from joshthoward joshthoward is a code owner

@lambrospetrou lambrospetrou Awaiting requested review from lambrospetrou lambrospetrou is a code owner

@mikenomitch mikenomitch Awaiting requested review from mikenomitch mikenomitch is a code owner

@rozenmd rozenmd Awaiting requested review from rozenmd

@sejoker sejoker Awaiting requested review from sejoker sejoker is a code owner

@mchenco mchenco Awaiting requested review from mchenco mchenco is a code owner

Assignees

@lambrospetrou lambrospetrou

@mikenomitch mikenomitch

@rozenmd rozenmd

@jonesphillip jonesphillip

@joshthoward joshthoward

@vy-ton vy-ton

@harshil1712 harshil1712

@thomasgauvin thomasgauvin

@aninibread aninibread

@Oxyjun Oxyjun

Labels

product:d1 D1: https://developers.cloudflare.com/d1/ product:durable-objects Durable Objects: https://developers.cloudflare.com/workers/learning/using-durable-objects/ product:kv product:r2 R2 object storage: https://developers.cloudflare.com/r2 product:vectorize Vectorize: https://developers.cloudflare.com/vectorize/ size/m

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.