Skip to content

fix: reject oversized rich-text edits without truncation - #192

Merged
dazzatronus merged 1 commit into
mainfrom
fix/clamp-rich-text-max-length
Sep 25, 2026
Merged

dazzatronus merged 1 commit into
mainfrom
fix/clamp-rich-text-max-length

Conversation

@dazzatronus

@dazzatronus dazzatronus commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Oversized text must not crash editing or be silently cut through a Unicode character or variable token. Reject edits and variable insertions when either the template or resolved text exceeds 5,000 characters, using the textarea's native validation message. Preserve the draft and saved bindings so the user can correct the input.

Cancel pending saves on variable insertion and keep the popup open when validation fails. Regression tests stay in the public SDK suite.

Verify: npm run verify:ci passes on Node 22, including 2,088 tests. Four integration cases cover Unicode boundaries, template and resolved-value limits, and binding preservation.

@dazzatronus
dazzatronus force-pushed the fix/clamp-rich-text-max-length branch from bc0e894 to 86806f0 Compare September 25, 2026 01:50
@dazzatronus dazzatronus changed the title fix: clamp rich-text edits to schema max before updateClip fix: reject oversized rich-text edits without truncation Sep 25, 2026
@dazzatronus dazzatronus self-assigned this Sep 25, 2026
@dazzatronus
dazzatronus merged commit 21dcdf1 into main Sep 25, 2026
1 check passed
@dazzatronus
dazzatronus deleted the fix/clamp-rich-text-max-length branch September 25, 2026 01:53
github-actions Bot pushed a commit that referenced this pull request Sep 25, 2026
## [2.22.6](v2.22.5...v2.22.6) (2026-09-25)

### Bug Fixes

* reject oversized rich-text edits without truncation ([#192](#192)) ([21dcdf1](21dcdf1))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.22.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant