fix(articles): actually enforce the limits the editor declares

[aquie00t]

Two more from continuing the review, and they share a root: the editor knows the server's limits and was not using them.

A constant declared and never read

ARTICLE_LIMITS.requestBytesMax — the 256 KB cap on the whole request body — was defined when the editor was written and never referenced anywhere.

It matters because it is independent of the character count. A body comfortably inside the 100.000 character limit still breaches it once it carries emoji or Turkish letters, and the server answers 413, not a validation error, so nothing downstream could explain what happened. Turkish writers reach it first, which makes it a poor thing to leave unchecked in a Turkish-language product.

The character limit was checked in the wrong place

bodyTooLong disabled the publish button. canSave — which gates autosave — only looked for emptiness. So an over-length body kept being sent on every typing pause, each attempt rejected with a bare 400, spending the sixty-a-minute update budget and telling the writer nothing.

Both checks now live in one function the hook and the page share, and save re-runs it rather than trusting its caller — it is also reached from the retry button and from publish. The indicator names the limit that is breached instead of saying only that something is unsaved.

While in there

The draft was re-serialised on every render for the dirty check, over a body that can run to a hundred thousand characters — so every keystroke in a long article stringified the whole thing. It is serialised once per change now, and both the dirty check and the byte count read that.

Reviewing

The three tests that matter fail on main: an over-length body is sent, a 90.000-emoji body (well under the character limit, well over 256 KB) is sent, and neither is reported to the writer.

The page test's mock now keeps problem in step with canSave, since the two always agree in the real hook — canSave is problem === null.

Verification

pnpm build, tsc -b, pnpm lint clean. 479 unit tests across 66 files and 29 Playwright tests pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Hu1QuLxS84vdf1gmzoGtWP

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	tdn-client	9876914	Commit Preview URL Branch Preview URL	Aug 26 2026, 03:24 AM