@dashevo/dapi-grpc 4.1.1 → 4.2.0-dev.10

package/Cargo.toml

@@ -26,12 +26,30 @@ tenderdash-proto = []
 # Client support.
 client = ["platform"]
 
+# Networked tonic client: `connect()` on generated clients, TLS roots.
+# Deliberately NOT in the default set: cargo features are not target-scoped,
+# so a default-on transport would force tonic's transport stack onto wasm32
+# consumers riding defaults, where it does not build. Without this feature
+# the crate provides message types and transport-generic client stubs only.
+# Tonic's codegen graph still includes tokio-stream and a minimal Tokio
+# slice, but its hyper/rustls transport stack is not enabled. Native
+# networked consumers enable this explicitly (rs-dapi-client does so through
+# its non-wasm dependency); the wasm codegen path never emits `connect()`.
+transport = [
+  "tonic/channel",
+  "tonic/transport",
+  "tonic/tls-native-roots",
+  "tonic/tls-webpki-roots",
+  "tonic/tls-ring",
+]
+
 # Build tonic server code. Includes all client features and adds server-specific dependencies.
 server = [
   "platform",
   "tenderdash-proto/server",
   "client",
   "drive",
+  "transport",
   "tonic/router",
 ]
 
@@ -55,14 +73,7 @@ tonic = { version = "0.14.2", features = ["codegen"], default-features = false }
 getrandom = { version = "0.2", features = ["js"] }
 
 [target.'cfg(not(target_arch = "wasm32"))'.dependencies]
-tonic = { version = "0.14.2", features = [
-  "codegen",
-  "channel",
-  "transport",
-  "tls-native-roots",
-  "tls-webpki-roots",
-  "tls-ring",
-], default-features = false }
+tonic = { version = "0.14.2", features = ["codegen"], default-features = false }
 
 [build-dependencies]
 tonic-prost-build = { version = "0.14.2" }

package/build.rs

@@ -10,6 +10,7 @@ use tonic_prost_build::Builder;
 const SERDE_WITH_BYTES: &str = r#"#[cfg_attr(feature = "serde", serde(with = "serde_bytes"))]"#;
 const SERDE_WITH_BASE64: &str =
     r#"#[cfg_attr(feature = "serde", serde(with = "crate::deserialization::vec_base64string"))]"#;
+const SERDE_DEFAULT: &str = r#"#[cfg_attr(feature = "serde", serde(default))]"#;
 const SERDE_WITH_STRING: &str =
     r#"#[cfg_attr(feature = "serde", serde(with = "crate::deserialization::from_to_string"))]"#;
 
@@ -69,6 +70,7 @@ fn generate_code(typ: ImplType, output_base: &Path) {
 
     println!("cargo:rerun-if-changed=./protos");
     println!("cargo:rerun-if-env-changed=CARGO_FEATURE_SERDE");
+    println!("cargo:rerun-if-env-changed=CARGO_FEATURE_TRANSPORT");
     println!("cargo:rerun-if-env-changed=CARGO_CFG_TARGET_ARCH");
     println!("cargo:rerun-if-env-changed=DAPI_GRPC_OUT_DIR");
 }
@@ -334,6 +336,16 @@ fn configure_platform(mut platform: MappingConfig) -> MappingConfig {
         .field_attribute("id", SERDE_WITH_BYTES)
         .field_attribute("identity_id", SERDE_WITH_BYTES)
         .field_attribute("ids", SERDE_WITH_BASE64)
+        // Wire-format compat for mock vectors captured before the chained
+        // surface existed: the field deserializes to its default when
+        // absent (same pattern DocumentQuery's own serde defaults follow
+        // for pre-SQL-surface fixtures).
+        .field_attribute("GetDocumentsRequestV1.chained", SERDE_DEFAULT)
+        .field_attribute("GetDocumentsRequestV1.sub_queries", SERDE_DEFAULT)
+        // Same compat rule for the typed IN_TIME_RANGE operand: mock
+        // vectors captured while the operand still rode `value` carry no
+        // `time_range` key.
+        .field_attribute("GetDocumentsRequest.WhereClause.time_range", SERDE_DEFAULT)
         .field_attribute("ResponseMetadata.height", SERDE_WITH_STRING)
         .field_attribute("ResponseMetadata.time_ms", SERDE_WITH_STRING)
         .field_attribute("start_at_ms", SERDE_WITH_STRING)
@@ -416,15 +428,23 @@ enum ImplType {
 impl ImplType {
     // Configure the builder based on the implementation type.
     pub fn configure(&self, builder: Builder) -> Builder {
+        // The `transport` cargo feature controls whether generated clients get
+        // the `connect()` convenience impls over tonic's own channel. Without
+        // it, clients are still generated but stay generic over the caller's
+        // transport. Never enabled for wasm32, where tonic transport does not
+        // build. Note: cfg!(target_arch) in a build script reflects the HOST,
+        // so the target must be read from CARGO_CFG_TARGET_ARCH.
+        let transport = std::env::var("CARGO_FEATURE_TRANSPORT").is_ok()
+            && std::env::var("CARGO_CFG_TARGET_ARCH").map(|arch| arch != "wasm32") == Ok(true);
         match self {
             Self::Server => builder
                 .build_client(true)
                 .build_server(true)
-                .build_transport(true),
+                .build_transport(transport),
             Self::Client => builder
                 .build_client(true)
                 .build_server(false)
-                .build_transport(true),
+                .build_transport(transport),
             Self::Wasm => builder
                 .build_client(true)
                 .build_server(false)