Skip to content

Bound batch decode by item limit (PLT-820) - #98

Merged
amir-deris merged 2 commits into
mainfrom
amir/plt-820-parse-message-allocation-check
Sep 4, 2026
Merged

amir-deris merged 2 commits into
mainfrom
amir/plt-820-parse-message-allocation-check

Prove the batch decode truncation end-to-end (PLT-820)

4634ec8
Select commit
Loading
Failed to load commit list.
Seidroid / AI Review succeeded Sep 4, 2026 in 0s

Claude + Codex + Cursor Review: success

The fix is sound: parseMessage now stops one element past the item limit, which correctly preserves handleBatch's strict len(msgs) > limit rejection while bounding per-element allocations, and it's wired into every built-in codec path (WS, IPC/stdio via ServeCodec, and HTTP via the new attachHandler in serveSingleRequest). No blocking issues found; a few non-blocking notes on latent budget coupling on the HTTP path, allocation-test robustness, and the unexported setHandler extension point.