From caddbc8d9a9226344addb795d15b63b4d623b845 Mon Sep 17 00:00:00 2001
From: Meta Vens <rwvens@hotmail.com>
Date: Wed, 28 Oct 2026 05:12:11 +0100
Subject: [PATCH] serve tests: pin the result bound's arithmetic and that
 in-bound results pass through untouched

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 crates/model-connector/src/serve.rs | 49 +++++++++++++++++++++++++++++
 1 file changed, 49 insertions(+)

diff --git a/crates/model-connector/src/serve.rs b/crates/model-connector/src/serve.rs
index 4158bdc..fec618f 100644
--- a/crates/model-connector/src/serve.rs
+++ b/crates/model-connector/src/serve.rs
@@ -485,6 +485,18 @@ mod tests {
             TransportDisposition::Transported { .. }
         ));
         assert_eq!(first, second);
+        let TransportDisposition::Transported { events, receipt } = &first.disposition else {
+            unreachable!("matched above");
+        };
+        assert_eq!(
+            events.len(),
+            1,
+            "a result within the bound is delivered as the provider made it"
+        );
+        assert!(matches!(
+            receipt.outcome,
+            TransportOutcome::Completed { .. }
+        ));
         assert_eq!(backend.executions.load(Ordering::SeqCst), 1);
 
         let mut conflicting_request = request();
@@ -802,6 +814,43 @@ mod tests {
         assert_eq!(backend.executions.load(Ordering::SeqCst), 1);
     }
 
+    #[test]
+    fn a_result_is_bounded_by_the_frame_less_its_overhead_inclusively() {
+        let invocation = fresh("request-bound");
+        let result_with = |text_len: usize| {
+            TransportResult::transported(
+                &invocation,
+                vec![TransportEvent {
+                    sequence: 0,
+                    payload: TransportEventPayload::TextDelta {
+                        text: "z".repeat(text_len),
+                    },
+                }],
+                receipt(&invocation),
+            )
+        };
+        let bounded = ServeLimits {
+            max_frame_bytes: 20_000,
+            ..limits()
+        };
+        let max = bounded.max_result_bytes();
+        assert_eq!(max, 20_000 - 4096);
+        // MessagePack headers are the same width across this range, so one probe fixes the size.
+        let probe = rmp_serde::to_vec(&result_with(10_000)).unwrap().len();
+        let exact = 10_000 + max - probe;
+        assert_eq!(rmp_serde::to_vec(&result_with(exact)).unwrap().len(), max);
+        assert_eq!(
+            bound_result(&invocation, result_with(exact), max),
+            result_with(exact)
+        );
+        let over = bound_result(&invocation, result_with(exact + 1), max);
+        let TransportDisposition::Transported { receipt, .. } = &over.disposition else {
+            panic!("expected a transported failure");
+        };
+        assert!(matches!(receipt.outcome, TransportOutcome::Failed { .. }));
+        over.validate_against(&invocation).unwrap();
+    }
+
     #[test]
     fn the_accept_loop_drops_a_peer_that_is_not_loopback() {
         let run = |reported: SocketAddr| {