Skip to content

fix: UTF-8-safe truncation for WASM emitted messages#1015

Merged
zmanian merged 1 commit intostagingfrom
zmanian/pr362-utf8-truncation
Mar 12, 2026
Merged

fix: UTF-8-safe truncation for WASM emitted messages#1015
zmanian merged 1 commit intostagingfrom
zmanian/pr362-utf8-truncation

Conversation

@zmanian
Copy link
Copy Markdown
Collaborator

@zmanian zmanian commented Mar 12, 2026

Replacement for #362 on current staging.

This rescues the UTF-8 truncation safety fix from #362 onto current branch state:

  • truncate emitted WASM messages on character boundaries
  • add regression coverage for multibyte truncation

Validation:

  • cargo test channels::wasm::host::tests -- --nocapture

Original PR: #362

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added scope: channel/wasm WASM channel runtime size: S 10-49 changed lines risk: medium Business logic, config, or moderate-risk modules contributor: core 20+ merged PRs labels Mar 12, 2026
@zmanian zmanian enabled auto-merge (squash) March 12, 2026 03:26
@zmanian zmanian merged commit 0b81342 into staging Mar 12, 2026
10 checks passed
@zmanian zmanian deleted the zmanian/pr362-utf8-truncation branch March 12, 2026 18:10
bkutasi pushed a commit to bkutasi/ironclaw that referenced this pull request Mar 28, 2026
Co-authored-by: Lawyered <ethlawyer1@gmail.com>
drchirag1991 pushed a commit to drchirag1991/ironclaw that referenced this pull request Apr 8, 2026
Co-authored-by: Lawyered <ethlawyer1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: core 20+ merged PRs risk: medium Business logic, config, or moderate-risk modules scope: channel/wasm WASM channel runtime size: S 10-49 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants