Bound the remote image prefetch of bun file.md by a deadline and a byte budget

mordant-baseline.toml

@@ -256,7 +256,6 @@
 
 [bun_threading]
 "unsafe_send_covers_more_than_pointers:src/threading/Semaphore.rs" = 1
-"unsafe_send_covers_more_than_pointers:src/threading/channel.rs" = 1
 
 [bun_uws_sys]
 "unused_pub:src/uws_sys/Loop.rs" = 2

src/http/AsyncHTTP.rs

@@ -549,8 +549,7 @@ impl<'a> AsyncHTTP<'a> {
 // ──────────────────────────────────────────────────────────────────────────
 
 // 32 pointers much cheaper than 1000 pointers
-// Note: `bun_threading::Channel` requires `T: Copy`, which
-// `HTTPClientResult` is not. `send_sync` is a one-shot blocking handoff, so a
+// `send_sync` is a one-shot blocking handoff, so a
 // Guarded<Option<T>>+Condvar is the exact semantics needed.
 pub(crate) struct SingleHTTPChannel {
     slot: bun_threading::Guarded<Option<HTTPClientResult<'static>>>,
@@ -864,11 +863,3 @@ impl<'a> AsyncHTTP<'a> {
         self.client.start(body);
     }
 }
-
-// ──────────────────────────────────────────────────────────────────────────
-// HTTPCallbackPair / HTTPChannel / HTTPChannelContext
-// ──────────────────────────────────────────────────────────────────────────
-
-// `bun_threading::Channel` requires `T: Copy`, which `HTTPClientResult` is not, so the
-// `HTTPChannel` here boxes the pair and ships the pointer (which IS `Copy`) through
-// a static-buffer channel. The receiver takes ownership of the Box.

src/http/lib.rs

@@ -464,8 +464,7 @@ pub struct HTTPClientResult<'a> {
     pub body: &'a [u8],
     /// Populated only on the terminal (`!has_more`) progress callback:
     /// `send_progress_update_*` moves the whole `decoded_body.list` here so
-    /// one-shot consumers (`send_sync`, `NetworkTask` manifest, S3 simple,
-    /// `RemoteImageDownload`) can `mem::take` it instead of
+    /// one-shot consumers (`send_sync`, `NetworkTask` manifest, S3 simple) can `mem::take` it instead of
     /// `extend_from_slice`ing the borrowed `body`. Streaming consumers read
     /// `body` on non-terminal callbacks and treat this as just the final
     /// chunk's bytes.