Merge remote-tracking branch 'origin/develop' into fix/bitcoin-reorg-extend

Author: kantai

.cargo/config.toml

@@ -1,7 +1,7 @@
 [alias]
 stacks-node = "run --package stacks-node --"
 fmt-stacks = "fmt -- --config group_imports=StdExternalCrate,imports_granularity=Module"
-clippy-stacks = "clippy -p stx-genesis -p libstackerdb -p stacks-signer -p pox-locking -p clarity -p libsigner -p stacks-common --no-deps --tests --all-features -- -D warnings"
+clippy-stacks = "clippy -p stx-genesis -p libstackerdb -p stacks-signer -p pox-locking -p clarity -p libsigner -p stacks-common --no-deps --tests --all-features -- -D warnings -A clippy::uninlined-format-args"
 
 # Uncomment to improve performance slightly, at the cost of portability
 #   * Note that native binaries may not run on CPUs that are different from the build machine

.vscode/settings.json

@@ -1,12 +1,8 @@
 {
-    "lldb.launch.sourceLanguages": [
-        "rust"
-    ],
-    "rust-analyzer.runnables.extraEnv": {
-        "BITCOIND_TEST": "1"
-    },
-    "rust-analyzer.rustfmt.extraArgs": [
-        "+nightly"
-    ],
-    "files.eol": "\n"
-}
+  "lldb.launch.sourceLanguages": ["rust"],
+  "rust-analyzer.runnables.extraEnv": {
+    "BITCOIND_TEST": "1"
+  },
+  "rust-analyzer.rustfmt.extraArgs": ["+nightly"],
+  "files.eol": "\n"
+}

CHANGELOG.md

@@ -14,6 +14,7 @@ and this project adheres to the versioning scheme outlined in the [README.md](RE
   - `stacks_node_miner_stop_reason_total`: Counts the number of times the miner stopped mining due to various reasons.
   - Always report the number of transactions mined in the last attempt, even if there were 0
 
+- Added a new option `--hex-file <file_path>` to `blockstack-cli contract-call` command, that allows to pass a serialized Clarity value by file.
 - Added a new option `--postcondition-mode [allow, deny]` to `blockstack-cli publish` command, to set the post-condition mode to allow or deny on the transaction (default is deny)
 
 ### Changed

Cargo.lock

@@ -74,23 +74,22 @@ fn main() {
         "pub const GIT_COMMIT: Option<&'static str> = {git_commit:?};\n",
     ));
     if let Some(git_commit) = git_commit {
-        println!("cargo:rustc-env=GIT_COMMIT={}", git_commit);
+        println!("cargo:rustc-env=GIT_COMMIT={git_commit}");
     }
 
     let git_branch = current_git_branch();
     rust_code.push_str(&format!(
         "pub const GIT_BRANCH: Option<&'static str> = {git_branch:?};\n",
     ));
     if let Some(git_branch) = git_branch {
-        println!("cargo:rustc-env=GIT_BRANCH={}", git_branch);
+        println!("cargo:rustc-env=GIT_BRANCH={git_branch}");
     }
 
     let is_clean = if is_working_tree_clean() { "" } else { "+" };
     rust_code.push_str(&format!(
-        "pub const GIT_TREE_CLEAN: Option<&'static str> = Some(\"{}\");\n",
-        is_clean
+        "pub const GIT_TREE_CLEAN: Option<&'static str> = Some(\"{is_clean}\");\n",
     ));
-    println!("cargo:rustc-env=GIT_TREE_CLEAN={}", is_clean);
+    println!("cargo:rustc-env=GIT_TREE_CLEAN={is_clean}");
 
     let out_dir = env::var_os("OUT_DIR").unwrap();
     let dest_path = Path::new(&out_dir).join("versions.rs");

stacks-common/build.rs

@@ -369,7 +369,7 @@ pub fn c32_address_decode(c32_address_str: &str) -> Result<(u8, Vec<u8>), Error>
 
 pub fn c32_address(version: u8, data: &[u8]) -> Result<String, Error> {
     let c32_string = c32_check_encode(version, data)?;
-    Ok(format!("S{}", c32_string))
+    Ok(format!("S{c32_string}"))
 }
 
 #[cfg(test)]

stacks-common/src/address/c32.rs

@@ -56,15 +56,14 @@ impl fmt::Display for Error {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         match *self {
             Error::InvalidCrockford32 => write!(f, "Invalid crockford 32 string"),
-            Error::InvalidVersion(ref v) => write!(f, "Invalid version {}", v),
+            Error::InvalidVersion(ref v) => write!(f, "Invalid version {v}"),
             Error::EmptyData => f.write_str("Empty data"),
-            Error::BadByte(b) => write!(f, "invalid base58 character 0x{:x}", b),
+            Error::BadByte(b) => write!(f, "invalid base58 character 0x{b:x}"),
             Error::BadChecksum(exp, actual) => write!(
                 f,
-                "base58ck checksum 0x{:x} does not match expected 0x{:x}",
-                actual, exp
+                "base58ck checksum 0x{actual:x} does not match expected 0x{exp:x}"
             ),
-            Error::InvalidLength(ell) => write!(f, "length {} invalid for this base58 type", ell),
+            Error::InvalidLength(ell) => write!(f, "length {ell} invalid for this base58 type"),
             Error::TooShort(_) => write!(f, "base58ck data not even long enough for a checksum"),
             Error::Other(ref s) => f.write_str(s),
         }

stacks-common/src/address/mod.rs

@@ -246,7 +246,7 @@ impl<const MAX_SIZE: u16> BitVec<MAX_SIZE> {
             .data
             .into_iter()
             .fold(String::new(), |acc, byte| {
-                acc + &format!("{:08b}", byte).chars().rev().collect::<String>()
+                acc + &format!("{byte:08b}").chars().rev().collect::<String>()
             })
             .chars()
             .take(self.len() as usize)

stacks-common/src/bitvec.rs

@@ -139,15 +139,13 @@ fn read_next_vec<T: StacksMessageCodec + Sized, R: Read>(
         if len > max_items {
             // too many items
             return Err(Error::DeserializeError(format!(
-                "Array has too many items ({} > {}",
-                len, max_items
+                "Array has too many items ({len} > {max_items})"
             )));
         }
     } else if len != num_items {
         // inexact item count
         return Err(Error::DeserializeError(format!(
-            "Array has incorrect number of items ({} != {})",
-            len, num_items
+            "Array has incorrect number of items ({len} != {num_items})"
         )));
     }
 

stacks-common/src/codec/mod.rs

@@ -679,11 +679,11 @@ pub enum Error {
 impl fmt::Display for Error {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         match *self {
-            Error::MissingSeparator => write!(f, "missing human-readable separator, \"{}\"", SEP),
+            Error::MissingSeparator => write!(f, "missing human-readable separator, \"{SEP}\""),
             Error::InvalidChecksum => write!(f, "invalid checksum"),
             Error::InvalidLength => write!(f, "invalid length"),
-            Error::InvalidChar(n) => write!(f, "invalid character (code={})", n),
-            Error::InvalidData(n) => write!(f, "invalid data point ({})", n),
+            Error::InvalidChar(n) => write!(f, "invalid character (code={n})"),
+            Error::InvalidData(n) => write!(f, "invalid data point ({n})"),
             Error::InvalidPadding => write!(f, "invalid padding"),
             Error::MixedCase => write!(f, "mixed-case strings not allowed"),
         }