Skip to content

fix: forward maxSize when parsing nested messages#700

Merged
anthony-swirldslabs merged 1 commit intomainfrom
699-forwardMaxSize
Dec 1, 2025
Merged

fix: forward maxSize when parsing nested messages#700
anthony-swirldslabs merged 1 commit intomainfrom
699-forwardMaxSize

Conversation

@anthony-swirldslabs
Copy link
Copy Markdown
Contributor

Description:
Forwarding a custom maxSize value to parse() calls for nested messages, so that the custom value is respected when parsing them.

Related issue(s):

Fixes #699

Notes for reviewer:
A new integration test is added to verify the behavior.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Anthony Petrov <anthony@swirldslabs.com>
@anthony-swirldslabs anthony-swirldslabs self-assigned this Dec 1, 2025
@anthony-swirldslabs anthony-swirldslabs requested review from a team as code owners December 1, 2025 21:20
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 1, 2025

JUnit Test Report

   78 files  ±0     78 suites  ±0   3m 46s ⏱️ ±0s
1 350 tests ±0  1 346 ✅ ±0   4 💤 ±0  0 ❌ ±0 
7 227 runs  ±0  7 207 ✅ ±0  20 💤 ±0  0 ❌ ±0 

Results for commit 86e2883. ± Comparison against base commit e4b7805.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 1, 2025

Integration Test Report

    411 files  ±0      411 suites  ±0   15m 55s ⏱️ -3s
114 871 tests +1  114 871 ✅ +1  0 💤 ±0  0 ❌ ±0 
115 112 runs  +1  115 112 ✅ +1  0 💤 ±0  0 ❌ ±0 

Results for commit 86e2883. ± Comparison against base commit e4b7805.

@anthony-swirldslabs anthony-swirldslabs merged commit 75500bd into main Dec 1, 2025
12 checks passed
@anthony-swirldslabs anthony-swirldslabs deleted the 699-forwardMaxSize branch December 1, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Forward maxSize to nested messages parsing

2 participants