Skip to content

Commit 76e0fb1

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

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
@@ -2785,3 +2785,15 @@ PRD: ralph/kernel-hardening (46 stories)
27852785
- 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
27862786
- All u16/u8 truncation points in encode_body now use the guarded helper functions
27872787
---
2788+
2789+
## 2026-03-19 - US-071
2790+
- Replaced unbounded IPC writer channel in main.rs with bounded(1024)
2791+
- Replaced unbounded session command channel in session.rs with bounded(256)
2792+
- Session threads now block naturally when IPC writer buffer is full (backpressure)
2793+
- Connection handler blocks when session command buffer is full
2794+
- All 79 cargo tests pass, all 28 typecheck tasks pass
2795+
- Files changed: crates/v8-runtime/src/main.rs, crates/v8-runtime/src/session.rs
2796+
- **Learnings for future iterations:**
2797+
- Only main.rs and session.rs have production unbounded channels; execution.rs, host_call.rs, snapshot.rs unbounded channels are all in test code
2798+
- crossbeam_channel::bounded API is drop-in replacement for unbounded — same Sender/Receiver types
2799+
---

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)