@layerzerolabs/oapp-stellar-contracts 0.2.122 → 0.2.124

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@layerzerolabs/oapp-stellar-contracts",
-  "version": "0.2.122",
+  "version": "0.2.124",
   "private": false,
   "license": "MIT",
   "files": [
@@ -11,14 +11,14 @@
     "clippy.toml"
   ],
   "dependencies": {
-    "@layerzerolabs/common-utils-macros-stellar-contracts": "0.2.122",
-    "@layerzerolabs/protocol-stellar-v2": "0.2.122",
-    "@layerzerolabs/common-utils-stellar-contracts": "0.2.122",
-    "@layerzerolabs/oapp-macros-stellar-contracts": "0.2.122"
+    "@layerzerolabs/common-utils-macros-stellar-contracts": "0.2.124",
+    "@layerzerolabs/common-utils-stellar-contracts": "0.2.124",
+    "@layerzerolabs/oapp-macros-stellar-contracts": "0.2.124",
+    "@layerzerolabs/protocol-stellar-v2": "0.2.124"
   },
   "devDependencies": {
-    "@layerzerolabs/build-utils-rust": "0.2.122",
-    "@layerzerolabs/vm-tooling-stellar": "0.2.122"
+    "@layerzerolabs/build-utils-rust": "0.2.124",
+    "@layerzerolabs/vm-tooling-stellar": "0.2.124"
   },
   "publishConfig": {
     "access": "public",
@@ -42,10 +42,12 @@
     "build": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo build --release\" stellar",
     "clean": "rm -rf ./dependencies ./target ./node_modules",
     "format": "pnpm exec lz-tool --script \"cargo fmt\" stellar",
-    "lint": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo clippy -- -D warnings\" stellar",
-    "lint:fix": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo clippy --fix --allow-dirty --allow-staged && cargo fmt\" stellar",
+    "format:check": "pnpm exec lz-tool --script \"cargo fmt -- --check\" stellar",
+    "lint": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo fmt -- --check && cargo clippy --all-targets -- -D warnings\" stellar",
+    "lint:fix": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo clippy --all-targets --fix --allow-dirty --allow-staged && cargo fmt\" stellar",
     "resolve-dependencies": "pnpm exec build-utils-rust resolve",
-    "test": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo nextest run -E 'not test(/^ui_/)'\" stellar",
+    "test": "pnpm test:non-ui && pnpm test:ui",
+    "test:non-ui": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo nextest run -E 'not test(/^ui_/)'\" stellar",
     "test:snapshot:check": "pnpm exec turbo-snapshot check",
     "test:snapshot:update": "pnpm exec turbo-snapshot update",
     "test:ui": "pnpm resolve-dependencies && pnpm exec lz-tool --script \"cargo nextest run -E 'test(/^ui_/)'\" stellar"

package/rustfmt.toml

@@ -1,15 +1,12 @@
-format_macro_bodies = true
+# Only stable rustfmt options are kept here: the Stellar docker toolchain runs
+# rustfmt on the stable channel (rust 1.90.0), which silently drops unstable
+# options (they only apply on nightly). Do not re-add unstable options unless the
+# tooling is switched to a pinned nightly (see the Solana packages for that setup).
 max_width = 120
-format_macro_matchers = true
-format_strings = true
-imports_granularity = "Crate"
-match_arm_blocks = false
-reorder_impl_items = true
-group_imports = "StdExternalCrate"
 use_field_init_shorthand = true
 use_small_heuristics = "Max"
-wrap_comments = true
-format_code_in_doc_comments = true
 
-# most of these are unstable, so we enable them
-unstable_features = true
+# style_edition is a stable option honored by the docker toolchain. Pin it to
+# 2021 to match every crate's `edition = "2021"` so formatting stays stable
+# across rustfmt versions (setting it to 2024 would reformat the code).
+style_edition = "2021"

package/src/oapp_options_type3.rs

@@ -1,7 +1,10 @@
 use crate::{self as oapp, errors::OAppError};
 use common_macros::{contract_trait, only_role, storage};
 use soroban_sdk::{assert_with_error, contractevent, contracttype, panic_with_error, Bytes, Env, Vec};
-use utils::{buffer_reader::BufferReader, rbac::{RoleBasedAccessControl, AUTHORIZER}};
+use utils::{
+    buffer_reader::BufferReader,
+    rbac::{RoleBasedAccessControl, AUTHORIZER},
+};
 
 pub const OPTION_TYPE3: u16 = 3;
 

package/src/tests/oapp_receiver.rs

@@ -146,11 +146,11 @@ fn test_is_compose_msg_sender() {
 
     // Test with same contract address (should return true)
     let result = oapp_client.is_compose_msg_sender(&origin, &message, &oapp_client.address);
-    assert_eq!(result, true);
+    assert!(result);
     let different_sender = Address::generate(&env);
 
     // Test with different sender address (should return false)
-    assert_eq!(oapp_client.is_compose_msg_sender(&origin, &message, &different_sender), false);
+    assert!(!oapp_client.is_compose_msg_sender(&origin, &message, &different_sender));
 }
 
 #[test]
@@ -159,7 +159,7 @@ fn test_allow_initialize_path_cases() {
 
     // no peer set -> false
     let origin_no_peer = Origin { src_eid: UNSET_EID, sender: BytesN::from_array(&env, &[5; 32]), nonce: 1 };
-    assert_eq!(oapp_client.allow_initialize_path(&origin_no_peer), false);
+    assert!(!oapp_client.allow_initialize_path(&origin_no_peer));
 
     // set peer for REMOTE_EID
     let peer_bytes: BytesN<32> = BytesN::from_array(&env, &[5; 32]);
@@ -167,11 +167,11 @@ fn test_allow_initialize_path_cases() {
 
     // matching -> true
     let origin_match = Origin { src_eid: REMOTE_EID, sender: peer_bytes.clone(), nonce: 1 };
-    assert_eq!(oapp_client.allow_initialize_path(&origin_match), true);
+    assert!(oapp_client.allow_initialize_path(&origin_match));
 
     // non-matching -> false
     let origin_non_match = Origin { src_eid: REMOTE_EID, sender: BytesN::from_array(&env, &[6; 32]), nonce: 1 };
-    assert_eq!(oapp_client.allow_initialize_path(&origin_non_match), false);
+    assert!(!oapp_client.allow_initialize_path(&origin_non_match));
 }
 
 #[test]

package/src/tests/oapp_sender.rs

@@ -347,7 +347,7 @@ fn test_lz_send_native_only() {
     assert_eq!(params.receiver, peer);
     assert_eq!(params.message, message);
     assert_eq!(params.options, options);
-    assert_eq!(params.pay_in_zro, false);
+    assert!(!params.pay_in_zro);
     assert_eq!(refund, refund_address);
 
     assert_eq!(native_token_client.balance(&sender), 0);
@@ -411,7 +411,7 @@ fn test_lz_send_with_zro() {
     assert_eq!(params.receiver, peer);
     assert_eq!(params.message, message);
     assert_eq!(params.options, options);
-    assert_eq!(params.pay_in_zro, true);
+    assert!(params.pay_in_zro);
     assert_eq!(refund, refund_address);
 
     assert_eq!(native_token_client.balance(&sender), 0);

package/src/tests/test_macros.rs

@@ -429,6 +429,6 @@ mod test_full_manual_all {
 
 #[test]
 fn test_macros_compile() {
-    // This test verifies that all macro combinations compile successfully
-    assert!(true);
+    // This test verifies that all macro combinations compile successfully.
+    // Reaching this point means every macro invocation above compiled.
 }