Use JsonPackedValue::from_network (#35918)

GitOrigin-RevId: d8d274bc5faba70a360f2e4e6ef422aafe4e7b7d

Author: goffrie

crates/udf/src/action_outcome.rs

@@ -24,11 +24,7 @@ use pb::{
 #[cfg(any(test, feature = "testing"))]
 use proptest::prelude::*;
 use semver::Version;
-use serde_json::Value as JsonValue;
-use value::{
-    ConvexValue,
-    JsonPackedValue,
-};
+use value::JsonPackedValue;
 
 #[cfg(any(test, feature = "testing"))]
 use crate::HttpActionRequest;
@@ -97,9 +93,7 @@ impl ActionOutcome {
         let result = result.ok_or_else(|| anyhow::anyhow!("Missing result"))?;
         let result = match result.result {
             Some(FunctionResultTypeProto::JsonPackedValue(value)) => {
-                let json: JsonValue = serde_json::from_str(&value)?;
-                let value = ConvexValue::try_from(json)?;
-                Ok(JsonPackedValue::pack(value))
+                Ok(JsonPackedValue::from_network(value)?)
             },
             Some(FunctionResultTypeProto::JsError(js_error)) => Err(js_error.try_into()?),
             None => anyhow::bail!("Missing result"),

crates/udf/src/udf_outcome.rs

@@ -26,10 +26,8 @@ use proptest::prelude::Arbitrary;
 #[cfg(any(test, feature = "testing"))]
 use proptest::prelude::Strategy;
 use rand::Rng;
-use serde_json::Value as JsonValue;
 use value::{
     heap_size::HeapSize,
-    ConvexValue,
     JsonPackedValue,
 };
 
@@ -223,9 +221,7 @@ impl UdfOutcome {
         let result = result.ok_or_else(|| anyhow::anyhow!("Missing result"))?;
         let result = match result.result {
             Some(FunctionResultTypeProto::JsonPackedValue(value)) => {
-                let json: JsonValue = serde_json::from_str(&value)?;
-                let value = ConvexValue::try_from(json)?;
-                Ok(JsonPackedValue::pack(value))
+                Ok(JsonPackedValue::from_network(value)?)
             },
             Some(FunctionResultTypeProto::JsError(js_error)) => Err(js_error.try_into()?),
             None => anyhow::bail!("Missing result"),