move raw format envelope mismatch check to error path to avoid double JSON parse

Mrhs121 · Mrhs121 · commit eacaa533bdfa · 2026-03-18T19:27:33.000+08:00
diff --git a/crates/adapters/src/format/json/input.rs b/crates/adapters/src/format/json/input.rs
@@ -461,11 +461,17 @@ impl Parser for JsonParser {
                     self.apply_update::<WeightedUpdate<_>>(update, &metadata, &mut errors)
                 }
                 JsonUpdateFormat::Raw => {
-                    if let Some(error) = Self::raw_format_insert_delete_mismatch_error(update) {
-                        errors.push(error);
-                        self.last_event_number += 1;
-                    } else {
-                        self.apply_update::<&RawValue>(update, &metadata, &mut errors);
+                    let errors_before = errors.len();
+                    self.apply_update::<&RawValue>(update, &metadata, &mut errors);
+                    // On parse failure, check if the user sent insert/delete
+                    // envelopes with the raw format and provide a better hint.
+                    if errors.len() > errors_before {
+                        if let Some(error) =
+                            Self::raw_format_insert_delete_mismatch_error(update)
+                        {
+                            errors.truncate(errors_before);
+                            errors.push(error);
+                        }
                     }
                 }
                 JsonUpdateFormat::Redis | JsonUpdateFormat::Snowflake => {
