fix(turbo-tasks): Remove T: Serialize bound from ResolveVc (#69995)

With the default derived implementation, serde would inject a `T: Serialize` or `T: Deserialize` bound.
    
However, like `Vc`, `ResolvedVc` is serialized as a couple numeric IDs (the type of `T` doesn't matter), so we shouldn't have this restriction on `ResolvedVc`.

Also, clean up `Vc<T>` using the same `bound = ""` trick with the derive macro.

turbopack/crates/turbo-tasks/src/vc/mod.rs

@@ -45,6 +45,8 @@ use crate::{
 /// some_ref.some_method_on_t();
 /// ```
 #[must_use]
+#[derive(Serialize, Deserialize)]
+#[serde(transparent, bound = "")]
 pub struct Vc<T>
 where
     T: ?Sized + Send,
@@ -231,27 +233,6 @@ where
 
 impl<T> Eq for Vc<T> where T: ?Sized + Send {}
 
-impl<T> Serialize for Vc<T>
-where
-    T: ?Sized + Send,
-{
-    fn serialize<S: serde::Serializer>(&self, serializer: S) -> Result<S::Ok, S::Error> {
-        self.node.serialize(serializer)
-    }
-}
-
-impl<'de, T> Deserialize<'de> for Vc<T>
-where
-    T: ?Sized + Send,
-{
-    fn deserialize<D: serde::Deserializer<'de>>(deserializer: D) -> Result<Self, D::Error> {
-        Ok(Vc {
-            node: RawVc::deserialize(deserializer)?,
-            _t: PhantomData,
-        })
-    }
-}
-
 // TODO(alexkirsz) This should not be implemented for Vc. Instead, users should
 // use the `ValueDebug` implementation to get a `D: Debug`.
 impl<T> std::fmt::Debug for Vc<T>

turbopack/crates/turbo-tasks/src/vc/resolved.rs

@@ -28,7 +28,7 @@ use crate::{
 };
 
 #[derive(Serialize, Deserialize)]
-#[serde(transparent)]
+#[serde(transparent, bound = "")]
 pub struct ResolvedVc<T>
 where
     T: ?Sized + Send,