chore: add sdk::PublicKey to workspaces::PublicKey conversion

.gitignore

@@ -1,5 +1,5 @@
 # Build files
-/target
+**/target
 Cargo.lock
 
 # Code Editor related files

CHANGELOG.md

@@ -6,10 +6,13 @@
 
 - [Import a couple functions over from near_crypto for PublicKey](https://github.com/near/workspaces-rs/pull/265)
   - Impl `Ord`, `PartialOrd`, `Hash`, `BorshSerialize`, `BorshDeserialize`, `Display`, and `FromStr` for `PublicKey`
+    - NOTE: Borsh bytes format is the same as near-sdk, where it is in the form of [bytes_len, key_type, key_data..]
   - Added `PublicKey::{empty, len, key_data}`
   - Impl `Display` for `SecretKey`.
   - more docs were added to both `SecretKey` and `PublicKey`.
   - Impl `Display`, `FromStr`, `TryFrom<u8>` for `KeyType`.
+- [Added `TryFrom<near_sdk::PublicKey>` for `workspaces::PublicKey`](https://github.com/near/workspaces-rs/pull/267)
+  - Added `KeyType::len` and `PublicKey::try_from_bytes`
 
 ### Changed
 

workspaces/Cargo.toml

@@ -32,6 +32,7 @@ tokio-retry = "0.3"
 tracing = "0.1"
 url = { version = "2.2.2", features = ["serde"] }
 
+near-sdk = { version = "4.1", optional = true }
 near-account-id = "0.15.0"
 near-crypto = "0.15.0"
 near-primitives = "0.15.0"
@@ -55,8 +56,9 @@ test-log = { version = "0.2.8", default-features = false, features = ["trace"] }
 tracing-subscriber = { version = "0.3.5", features = ["env-filter"] }
 
 [features]
-default = ["install"]
+default = ["install", "interop_sdk"]
 install = []  # Install the sandbox binary during compile time
+interop_sdk = ["near-sdk"]
 unstable = ["cargo_metadata"]
 
 [package.metadata.docs.rs]

workspaces/src/types/mod.rs

@@ -6,8 +6,12 @@ pub(crate) mod account;
 pub(crate) mod block;
 pub(crate) mod chunk;
 
+#[cfg(feature = "interop_sdk")]
+mod sdk;
+
 use std::convert::TryFrom;
 use std::fmt::{self, Debug, Display};
+use std::io;
 use std::path::Path;
 use std::str::FromStr;
 
@@ -65,6 +69,15 @@ impl KeyType {
             near_crypto::KeyType::SECP256K1 => Self::SECP256K1,
         }
     }
+
+    /// Length of the bytes of the public key associated with this key type.
+    #[allow(clippy::len_without_is_empty)] // This is an associated length.
+    pub const fn len(&self) -> usize {
+        match self {
+            Self::ED25519 => 32,
+            Self::SECP256K1 => 64,
+        }
+    }
 }
 
 impl Display for KeyType {
@@ -105,19 +118,7 @@ impl From<PublicKey> for near_crypto::PublicKey {
 
 /// Public key of an account on chain. Usually created along with a [`SecretKey`]
 /// to form a keypair associated to the account.
-#[derive(
-    Debug,
-    Clone,
-    Hash,
-    Eq,
-    PartialEq,
-    Ord,
-    PartialOrd,
-    Serialize,
-    Deserialize,
-    BorshSerialize,
-    BorshDeserialize,
-)]
+#[derive(Debug, Clone, Hash, Eq, PartialEq, Ord, PartialOrd, Serialize, Deserialize)]
 pub struct PublicKey(pub(crate) near_crypto::PublicKey);
 
 #[allow(clippy::len_without_is_empty)] // PublicKey is guaranteed to never be empty due to KeyType restrictions.
@@ -128,6 +129,19 @@ impl PublicKey {
         Self(near_crypto::PublicKey::empty(key_type.into_near_keytype()))
     }
 
+    /// Create a new [`PublicKey`] from the given bytes. This will return an error if the bytes are not in the
+    /// correct format. Expected to have key type be the first byte encoded, with the remaining bytes being the
+    /// key data.
+    pub fn try_from_bytes(bytes: &[u8]) -> Result<Self> {
+        let key_type = KeyType::try_from(bytes[0])?;
+        Ok(Self(
+            near_crypto::PublicKey::try_from_slice(bytes).map_err(|e| {
+                ErrorKind::DataConversion
+                    .full(format!("Invalid key data for key type: {key_type}"), e)
+            })?,
+        ))
+    }
+
     /// Get the number of bytes this key uses. This will differ depending on the [`KeyType`]. i.e. for
     /// ED25519 keys, this will return 32 + 1, while for SECP256K1 keys, this will return 64 + 1. The +1
     /// is used to store the key type, and will appear at the start of the serialized key.
@@ -164,6 +178,36 @@ impl FromStr for PublicKey {
     }
 }
 
+impl BorshSerialize for PublicKey {
+    fn serialize<W: io::Write>(&self, writer: &mut W) -> io::Result<()> {
+        // NOTE: sdk::PublicKey requires that we serialize the length of the key first, then the key itself.
+        // Casted usize to u32 since the length in WASM is only 4 bytes long.
+        BorshSerialize::serialize(&(self.len() as u32), writer)?;
+        // Serialize key type and key data:
+        BorshSerialize::serialize(&self.0, writer)
+    }
+}
+
+impl BorshDeserialize for PublicKey {
+    fn deserialize(buf: &mut &[u8]) -> io::Result<Self> {
+        let len: u32 = BorshDeserialize::deserialize(buf)?;
+        let pk: near_crypto::PublicKey = BorshDeserialize::deserialize(buf)?;
+
+        // Check that the length of the key matches the length we read from the buffer:
+        if pk.len() != len as usize {
+            return Err(io::Error::new(
+                io::ErrorKind::InvalidData,
+                format!(
+                    "Key length of {} does not match length of {} read from buffer",
+                    pk.len(),
+                    len
+                ),
+            ));
+        }
+        Ok(Self(pk))
+    }
+}
+
 /// Secret key of an account on chain. Usually created along with a [`PublicKey`]
 /// to form a keypair associated to the account. To generate a new keypair, use
 /// one of the creation methods found here, such as [`SecretKey::from_seed`]

workspaces/src/types/sdk.rs

@@ -0,0 +1,18 @@
+use std::convert::TryFrom;
+
+use crate::error::{Error, ErrorKind};
+
+use super::PublicKey;
+
+impl TryFrom<near_sdk::PublicKey> for PublicKey {
+    type Error = Error;
+
+    fn try_from(pk: near_sdk::PublicKey) -> Result<Self, Self::Error> {
+        Self::try_from_bytes(pk.as_bytes()).map_err(|e| {
+            ErrorKind::DataConversion.full(
+                "Could not convert sdk::PublicKey into workspaces::PublicKey",
+                e,
+            )
+        })
+    }
+}

workspaces/tests/test-contracts/type-serialize/Cargo.toml

@@ -0,0 +1,24 @@
+[package]
+name = "test-contract-type-serialization"
+version = "0.0.0"
+authors = ["Near Inc <[email protected]>"]
+edition = "2018"
+
+[lib]
+crate-type = ["cdylib"]
+
+[dependencies]
+anyhow = "1.0"
+bs58 = "0.4"
+near-sdk = "4.1"
+
+[profile.release]
+codegen-units = 1
+# Tell `rustc` to optimize for small code size.
+opt-level = "z"
+lto = true
+debug = false
+panic = "abort"
+overflow-checks = true
+
+[workspace]

workspaces/tests/test-contracts/type-serialize/build.sh

@@ -0,0 +1,4 @@
+#!/bin/sh
+
+cargo build --target wasm32-unknown-unknown --release
+cp target/wasm32-unknown-unknown/release/test_contract_type_serialization.wasm ./res/

workspaces/tests/test-contracts/type-serialize/src/lib.rs

@@ -0,0 +1,41 @@
+use near_sdk::borsh::{self, BorshDeserialize, BorshSerialize};
+use near_sdk::near_bindgen;
+use near_sdk::{CurveType, PublicKey};
+
+use std::convert::TryFrom;
+
+#[derive(Default, BorshSerialize, BorshDeserialize)]
+#[near_bindgen]
+struct Contract {}
+
+#[near_bindgen]
+impl Contract {
+    pub fn pass_pk_back_and_forth(&self, pk: PublicKey) -> PublicKey {
+        let mut data = vec![CurveType::ED25519 as u8];
+        data.extend(
+            bs58::decode("6E8sCci9badyRkXb3JoRpBj5p8C6Tw41ELDZoiihKEtp")
+                .into_vec()
+                .expect("could not convert bs58 to vec"),
+        );
+        let pk_expected =
+            PublicKey::try_from(data).expect("could not create public key from parts");
+
+        assert_eq!(pk, pk_expected);
+        pk
+    }
+
+    #[result_serializer(borsh)]
+    pub fn pass_borsh_pk_back_and_forth(&self, #[serializer(borsh)] pk: PublicKey) -> PublicKey {
+        let mut data = vec![CurveType::ED25519 as u8];
+        data.extend(
+            bs58::decode("6E8sCci9badyRkXb3JoRpBj5p8C6Tw41ELDZoiihKEtp")
+                .into_vec()
+                .expect("could not convert bs58 to vec"),
+        );
+        let pk_expected =
+            PublicKey::try_from(data).expect("could not create public key from parts");
+
+        assert_eq!(pk, pk_expected);
+        pk
+    }
+}

workspaces/tests/types.rs

@@ -1,10 +1,18 @@
+use std::convert::TryFrom;
 use std::str::FromStr;
 
 use borsh::{BorshDeserialize, BorshSerialize};
 
 use workspaces::types::{KeyType, PublicKey, SecretKey};
 use workspaces::AccountId;
 
+use near_sdk as sdk;
+
+fn default_workspaces_pubkey() -> anyhow::Result<PublicKey> {
+    let data = bs58::decode("279Zpep9MBBg4nKsVmTQE7NbXZkWdxti6HS1yzhp8qnc1ExS7gU").into_vec()?;
+    Ok(PublicKey::try_from_slice(data.as_slice())?)
+}
+
 #[test]
 fn test_keypair_ed25519() -> anyhow::Result<()> {
     let pk_expected = "\"ed25519:DcA2MzgpJbrUATQLLceocVckhhAqrkingax4oJ9kZ847\"";
@@ -52,15 +60,42 @@ fn test_pubkey_serialization() -> anyhow::Result<()> {
     Ok(())
 }
 
+#[cfg(feature = "interop_sdk")]
+#[tokio::test]
+async fn test_pubkey_from_sdk_ser() -> anyhow::Result<()> {
+    const TYPE_SER_BYTES: &[u8] =
+        include_bytes!("test-contracts/type-serialize/res/test_contract_type_serialization.wasm");
+    let worker = workspaces::sandbox().await?;
+    let contract = worker.dev_deploy(TYPE_SER_BYTES).await?;
+
+    // Test out serde serialization and deserialization for PublicKey
+    let ws_pk = default_workspaces_pubkey()?;
+    let sdk_pk: sdk::PublicKey = contract
+        .call("pass_pk_back_and_forth")
+        .args_json(serde_json::json!({ "pk": ws_pk }))
+        .transact()
+        .await?
+        .json()?;
+    assert_eq!(ws_pk, PublicKey::try_from(sdk_pk)?);
+
+    // Test out borsh serialization and deserialization for PublicKey
+    let sdk_pk: sdk::PublicKey = contract
+        .call("pass_borsh_pk_back_and_forth")
+        .args_borsh(&ws_pk)
+        .transact()
+        .await?
+        .borsh()?;
+    assert_eq!(ws_pk, PublicKey::try_from(sdk_pk)?);
+
+    Ok(())
+}
+
 #[test]
 fn test_pubkey_borsh_format_change() -> anyhow::Result<()> {
-    let mut data = vec![KeyType::ED25519 as u8];
-    data.extend(bs58::decode("6E8sCci9badyRkXb3JoRpBj5p8C6Tw41ELDZoiihKEtp").into_vec()?);
-
-    let pk = PublicKey::try_from_slice(data.as_slice())?;
+    let pk = default_workspaces_pubkey()?;
     assert_eq!(
         pk.try_to_vec()?,
-        bs58::decode("16E8sCci9badyRkXb3JoRpBj5p8C6Tw41ELDZoiihKEtp").into_vec()?
+        bs58::decode("279Zpep9MBBg4nKsVmTQE7NbXZkWdxti6HS1yzhp8qnc1ExS7gU").into_vec()?
     );
 
     Ok(())