From ddf5bbf2b91da033ca22326d157bda71585dfb48 Mon Sep 17 00:00:00 2001
From: "autocraft-bot[bot]"
 <354215646+autocraft-bot[bot]@users.noreply.github.com>
Date: Wed, 11 Nov 2026 02:26:54 +0000
Subject: [PATCH] Apply review fixes

- crates/metafile/src/canvas.rs
- crates/metafile/src/tests.rs
---
 crates/metafile/src/canvas.rs | 2 +-
 crates/metafile/src/tests.rs  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/crates/metafile/src/canvas.rs b/crates/metafile/src/canvas.rs
index 4cc4777d7..dc9d855f6 100644
--- a/crates/metafile/src/canvas.rs
+++ b/crates/metafile/src/canvas.rs
@@ -684,7 +684,7 @@ impl Canvas {
         // A pattern brush's pixels count against the byte budget like drawn items.
         if let Obj::Brush(Brush { pattern: Some(p), .. }) = &obj {
             let cost = p.px.len();
-            if self.used.saturating_add(cost) > MAX_BYTES {
+            if self.used.saturating_add(self.held).saturating_add(cost) > MAX_BYTES {
                 self.full = true;
                 return;
             }
diff --git a/crates/metafile/src/tests.rs b/crates/metafile/src/tests.rs
index fa2b48617..df238683e 100644
--- a/crates/metafile/src/tests.rs
+++ b/crates/metafile/src/tests.rs
@@ -899,10 +899,10 @@ fn place_fits_the_frame_scales_strokes_and_drops_far_paths() {
 
 #[test]
 fn place_counts_the_items_own_clip_region_before_keeping_it() {
-    // A path of 499,999 elements nearly fills the 500,000 budget; a second one whose new clip
+    // A path of 499,998 elements nearly fills the 500,000 budget; a second one whose new clip
     // region adds elements goes past it and is left out.
     let mut segs = vec![Seg::Move(0.0, 0.0)];
-    segs.extend((0..499_998).map(|i| Seg::Line(1.0 + (i % 50) as f32, 1.0)));
+    segs.extend((0..499_997).map(|i| Seg::Line(1.0 + (i % 50) as f32, 1.0)));
     let big = Item::Path { segs, fill: None, stroke: None, even_odd: false, region: None };
     let small = Item::Path {
         segs: vec![Seg::Move(0.0, 0.0), Seg::Line(1.0, 1.0)],