Update sui rust sdk / move binding & fix errors (#230)

* Update sui rust sdk to 0.0.7

* Remove move-binding dep

* fmt

* fix snapshot

* rerun CI

Author: manolisliolios

Cargo.lock

@@ -48,5 +48,6 @@ sui-storage = { git = "https://github.com/MystenLabs/sui.git", rev = "3b96ab72dd
 sui-field-count = { git = "https://github.com/MystenLabs/sui.git", rev = "3b96ab72dd5db2fb800837d6067bf45839178b62"}
 
 # New Rust SDK
-sui-sdk-types = { git = "https://github.com/mystenlabs/sui-rust-sdk", package = "sui-sdk-types", rev="86a9e06"}
+sui-sdk-types = { git = "https://github.com/mystenlabs/sui-rust-sdk", package = "sui-sdk-types", rev="048124e484f14b9bf2a402227c9bc255c7621bc1"}
+
 insta = "1.43.2"

Cargo.toml

@@ -7,7 +7,7 @@ expression: cmd_output
 Networks: mainnet
 
 - @testingafreeclaim/upgraded-demo
-# --
+# 
 Networks: mainnet
 
 Tip: You can find more information about a package by using: mvr resolve <name>

crates/ci-tests/tests/snapshots/use_case_tester__cmd-mvr_search_@testingafreeclaim__upgraded.snap

@@ -8,7 +8,7 @@ use diesel::{
 };
 use mvr_types::name::Name;
 use serde::{Deserialize, Serialize};
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::errors::ApiError;
 
@@ -21,12 +21,12 @@ use super::reader::Reader;
 /// The `NaiveDate` is the date of the analytics query, as we are caching the query on a daily basis
 /// per instance.
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct NameAnalyticsKey(pub Name, pub ObjectId, pub NaiveDate);
+pub struct NameAnalyticsKey(pub Name, pub Address, pub NaiveDate);
 
 /// Similar operation with analytics, this returns the "count" of
 /// dependents using a package, across versions.
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct NameDependentsCountKey(pub Name, pub ObjectId, pub NaiveDate);
+pub struct NameDependentsCountKey(pub Name, pub Address, pub NaiveDate);
 
 #[derive(Serialize, Deserialize, Clone, QueryableByName)]
 pub struct AnalyticsQueryResponse {
@@ -100,7 +100,7 @@ impl Loader<NameAnalyticsKey> for Reader {
 
         for res in result.into_iter() {
             // SAFETY: We should never have a malformed package id in the database.
-            let obj_id = ObjectId::from_str(&res.package_id).unwrap();
+            let obj_id = Address::from_str(&res.package_id).unwrap();
 
             if let Some(key) = addr_name_mapping.get(&obj_id) {
                 if let Some(v) = aggregated_values.get_mut(key) {

crates/mvr-api/src/data/name_analytics.rs

@@ -8,7 +8,7 @@ use diesel::{
     ExpressionMethods,
 };
 use serde::{Deserialize, Serialize};
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::errors::ApiError;
 
@@ -17,7 +17,7 @@ use super::reader::Reader;
 /// The `NaiveDate` is the date of the analytics query, as we are caching the query on a daily basis
 /// per instance.
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct PackageAnalyticsKey(pub ObjectId, pub NaiveDate);
+pub struct PackageAnalyticsKey(pub Address, pub NaiveDate);
 
 #[derive(Serialize, Deserialize, Clone, Queryable)]
 pub struct PackageAnalytics {

crates/mvr-api/src/data/package_analytics.rs

@@ -4,7 +4,7 @@ use async_graphql::dataloader::Loader;
 use diesel::{ExpressionMethods, QueryDsl};
 use mvr_schema::schema::package_dependencies;
 use serde::{Deserialize, Serialize};
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::errors::ApiError;
 
@@ -13,11 +13,11 @@ use super::reader::Reader;
 // This is a key to load all dependencies for a package.
 // Since there are system limits, we do not paginate this endpoint.
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct PackageDependenciesKey(pub ObjectId);
+pub struct PackageDependenciesKey(pub Address);
 
 #[derive(Serialize, Deserialize, Clone)]
 pub struct PackageDependencies {
-    pub dependencies: Vec<ObjectId>,
+    pub dependencies: Vec<Address>,
 }
 
 impl Default for PackageDependencies {
@@ -58,15 +58,15 @@ impl Loader<PackageDependenciesKey> for Reader {
             HashMap::new(),
             |mut acc, (package_id, dependency_package_id)| {
                 acc.entry(PackageDependenciesKey(
-                    // SAFETY: We know that the package_id is a valid ObjectId
-                    ObjectId::from_str(package_id).unwrap(),
+                    // SAFETY: We know that the package_id is a valid address
+                    Address::from_str(package_id).unwrap(),
                 ))
                 .or_insert_with(|| PackageDependencies {
                     dependencies: Vec::new(),
                 })
-                // SAFETY: We know that the dependency_package_id is a valid ObjectId
+                // SAFETY: We know that the dependency_package_id is a valid Address
                 .dependencies
-                .push(ObjectId::from_str(&dependency_package_id).unwrap());
+                .push(Address::from_str(&dependency_package_id).unwrap());
                 acc
             },
         );

crates/mvr-api/src/data/package_dependencies.rs

@@ -12,28 +12,28 @@ use diesel::{
 use futures::future::try_join_all;
 use mvr_schema::schema;
 use serde::{Deserialize, Serialize};
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::{errors::ApiError, utils::pagination::PaginationLimit};
 
 use super::reader::Reader;
 
 #[derive(Serialize, Deserialize, Debug, Default, Clone, Eq, PartialEq, Hash)]
 pub struct PackageDependentsCursor {
-    pub package_id: Option<ObjectId>,
+    pub package_id: Option<Address>,
     pub aggregated_total_calls: Option<i64>,
 }
 
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
 pub struct PackageDependentsKey(
-    pub ObjectId,
+    pub Address,
     pub PackageDependentsCursor,
     pub PaginationLimit,
     pub NaiveDate,
 );
 
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct PackageDependentsCountKey(pub ObjectId, pub NaiveDate);
+pub struct PackageDependentsCountKey(pub Address, pub NaiveDate);
 
 #[derive(Serialize, Deserialize, Clone, QueryableByName)]
 pub struct PackageDependent {

crates/mvr-api/src/data/package_dependents.rs

@@ -9,7 +9,7 @@ use diesel::{
     sql_types::{Array, BigInt, Text},
 };
 use mvr_types::name::VersionedName;
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::errors::ApiError;
 
@@ -30,7 +30,7 @@ pub struct NameResolution {
 
 #[derive(Clone, Eq, PartialEq, Debug)]
 pub struct ResolutionData {
-    pub id: ObjectId,
+    pub id: Address,
     pub version: i64,
 }
 
@@ -83,7 +83,7 @@ impl Loader<ResolutionKey> for Reader {
             .map(|name| {
                 let mut versioned_name = VersionedName::from_str(&name.name)?;
                 versioned_name.version = (name.version > 0).then_some(name.version as u64);
-                let object_id = ObjectId::from_str(&name.package_id).map_err(|e| {
+                let object_id = Address::from_str(&name.package_id).map_err(|e| {
                     ApiError::BadRequest(format!(
                         "Failed to parse package id: {e} {}",
                         name.package_id

crates/mvr-api/src/data/resolution_loader.rs

@@ -6,7 +6,7 @@ use diesel::{
     sql_types::{Array, Text},
 };
 use mvr_types::name::VersionedName;
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::errors::ApiError;
 
@@ -22,7 +22,7 @@ pub struct ReverseNameResolution {
 }
 
 #[derive(Clone, Eq, PartialEq, Hash, Debug)]
-pub struct ReverseResolutionKey(pub ObjectId);
+pub struct ReverseResolutionKey(pub Address);
 
 #[async_trait::async_trait]
 impl Loader<ReverseResolutionKey> for Reader {
@@ -72,7 +72,7 @@ impl Loader<ReverseResolutionKey> for Reader {
                 // SAFETY: We should never have a malformed name in the database.
                 let versioned_name = VersionedName::from_str(&name.name)?;
                 // SAFETY: We should never have a malformed package id in the database.
-                let object_id = ObjectId::from_str(&name.package_id).map_err(|e| {
+                let object_id = Address::from_str(&name.package_id).map_err(|e| {
                     ApiError::BadRequest(format!(
                         "Failed to parse package id: {e} {}",
                         name.package_id

crates/mvr-api/src/data/reverse_resolution_loader.rs

@@ -4,7 +4,7 @@
 use std::{collections::HashMap, sync::Arc};
 
 use axum::{extract::State, http::StatusCode};
-use sui_sdk_types::ObjectId;
+use sui_sdk_types::Address;
 
 use crate::{
     data::{app_state::AppState, resolution_loader::ResolutionData},
@@ -32,7 +32,7 @@ pub(crate) async fn health_check(
     Ok(StatusCode::OK)
 }
 
-fn into_object_id_map(resolution: &HashMap<String, ResolutionData>) -> HashMap<String, ObjectId> {
+fn into_object_id_map(resolution: &HashMap<String, ResolutionData>) -> HashMap<String, Address> {
     resolution
         .iter()
         .map(|(k, v)| (k.clone(), v.id))