Skip to content

Commit 4e2b19a

Browse files
NathanFlurryclaude
andcommitted
chore: update progress for US-071
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent a1e283e commit 4e2b19a

2 files changed

Lines changed: 14 additions & 2 deletions

File tree

progress.txt

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2791,3 +2791,15 @@ PRD: ralph/kernel-hardening (46 stories)
27912791
- encode_body was infallible (returned nothing) so callers like encode_frame_into had to handle errors only from the size check — now encode_body itself can fail on field overflow
27922792
- All u16/u8 truncation points in encode_body now use the guarded helper functions
27932793
---
2794+
2795+
## 2026-03-19 - US-071
2796+
- Replaced unbounded IPC writer channel in main.rs with bounded(1024)
2797+
- Replaced unbounded session command channel in session.rs with bounded(256)
2798+
- Session threads now block naturally when IPC writer buffer is full (backpressure)
2799+
- Connection handler blocks when session command buffer is full
2800+
- All 79 cargo tests pass, all 28 typecheck tasks pass
2801+
- Files changed: crates/v8-runtime/src/main.rs, crates/v8-runtime/src/session.rs
2802+
- **Learnings for future iterations:**
2803+
- Only main.rs and session.rs have production unbounded channels; execution.rs, host_call.rs, snapshot.rs unbounded channels are all in test code
2804+
- crossbeam_channel::bounded API is drop-in replacement for unbounded — same Sender/Receiver types
2805+
---

scripts/ralph/prd.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1040,8 +1040,8 @@
10401040
"Typecheck passes"
10411041
],
10421042
"priority": 61,
1043-
"passes": false,
1044-
"notes": "PR review finding: CLAUDE.md explicitly flags unbounded buffering as critical risk."
1043+
"passes": true,
1044+
"notes": "Changed IPC writer channel to bounded(1024) in main.rs and session command channel to bounded(256) in session.rs. All 79 cargo tests pass, all 28 typecheck tasks pass."
10451045
},
10461046
{
10471047
"id": "US-072",

0 commit comments

Comments
 (0)