Tributary Framework Cleanup and Testing

.github/actions/cargo-install/action.yml

@@ -48,13 +48,13 @@ runs:
 
           # Use `musl` for the `libc`
           export RUSTTARGET="$(cargo --version --verbose | grep '^host: ' | cut -f2 -d' ' | sed s/gnu/musl/)"
-          rustup +1.99 target add "$RUSTTARGET"
+          rustup +1.100 target add "$RUSTTARGET"
           export RUSTTARGET="--target $RUSTTARGET"
 
           export CC=clang
         fi
 
-        CARGO_JSON=$(cargo +1.99 $CARGOFLAGS install "$RUSTTARGET" --locked "${CRATE}" "${GIT}" "${VERSION}" --message-format=json)
+        CARGO_JSON=$(cargo +1.100 $CARGOFLAGS install $RUSTTARGET --locked "${CRATE}" ${GIT} ${VERSION} --message-format=json)
         BINARY=$(echo "$CARGO_JSON" | jq -r '.executable | select(. != null)')
         # Check this installed to the directory we expect for caching purposes
         if [ $CACHE -eq 1 ]; then

.github/nightly-version

@@ -1 +1 @@
-2026-10-01
+2026-11-01

.github/workflows/libraries-on-redox.yml

@@ -9,13 +9,13 @@ on:
     paths:
       - "common/**"
       - "crypto/**"
-      - "coordinator/tributary-sdk/tendermint/*"
+      - "coordinator/tributary-framework/tendermint/*"
 
   pull_request:
     paths:
       - "common/**"
       - "crypto/**"
-      - "coordinator/tributary-sdk/tendermint/*"
+      - "coordinator/tributary-framework/tendermint/*"
 
   workflow_dispatch:
 
@@ -61,7 +61,7 @@ jobs:
 
           redoxer toolchain
 
-          CRATES=$(find ./common ./crypto ./coordinator/tributary-sdk/tendermint -name "Cargo.toml")
+          CRATES=$(find ./common ./crypto ./coordinator/tributary-framework/tendermint -name "Cargo.toml")
           echo "$CRATES" | while IFS="\n" read -r manifest; do
             FEATURES="--all-features"
             if [ "$manifest" = "./common/db/Cargo.toml" ]; then

.github/workflows/lint.yml

@@ -51,15 +51,11 @@ jobs:
           cargo clippy --locked --all-targets -p serai-abi -- -D warnings
 
           # nightly Rust
-          cp ./Cargo.toml ./Cargo.toml.bak
-          sed s/'# TODO: `-Zcargo-lints`: '//g ./Cargo.toml > ./Cargo.toml.cargo-lints
-          mv ./Cargo.toml.cargo-lints ./Cargo.toml
-          cargo +"${NIGHTLY_VERSION}" clippy --locked -Zcargo-lints --all-features --all-targets -- -D warnings
-          cargo +"${NIGHTLY_VERSION}" clippy --locked -Zcargo-lints --no-default-features --all-targets -p serai-primitives -- -D warnings
-          cargo +"${NIGHTLY_VERSION}" clippy --locked -Zcargo-lints --all-targets -p serai-primitives -- -D warnings
-          cargo +"${NIGHTLY_VERSION}" clippy --locked -Zcargo-lints --no-default-features --all-targets -p serai-abi -- -D warnings
-          cargo +"${NIGHTLY_VERSION}" clippy --locked -Zcargo-lints --all-targets -p serai-abi -- -D warnings
-          mv ./Cargo.toml.bak ./Cargo.toml
+          cargo +"${NIGHTLY_VERSION}" clippy --locked --all-features --all-targets -- -D warnings
+          cargo +"${NIGHTLY_VERSION}" clippy --locked --no-default-features --all-targets -p serai-primitives -- -D warnings
+          cargo +"${NIGHTLY_VERSION}" clippy --locked --all-targets -p serai-primitives -- -D warnings
+          cargo +"${NIGHTLY_VERSION}" clippy --locked --no-default-features --all-targets -p serai-abi -- -D warnings
+          cargo +"${NIGHTLY_VERSION}" clippy --locked --all-targets -p serai-abi -- -D warnings
 
       - name: Verify `pallet::hooks` is unused
         shell: bash
@@ -300,7 +296,7 @@ jobs:
             cargo +"${LOCKFILE_VERSION}" update -p fs-err@2.0.0 --precise 2.4.0 || true
 
             # Check this lockfile is actually adequate to build this crate
-            cargo +1.99 check
+            cargo +1.100 check
 
             # We then find the actual `rust-version` using `cargo-msrv` (again stripping to just the
             # major and minor version).

Cargo.toml

@@ -71,8 +71,8 @@ members = [
   "processor/ethereum",
   "processor/monero",
 
-  "coordinator/tributary-sdk/tendermint",
-  "coordinator/tributary-sdk",
+  "coordinator/tributary-framework/tendermint",
+  "coordinator/tributary-framework",
   "coordinator/cosign/types",
   "coordinator/cosign",
   "coordinator/substrate",
@@ -280,6 +280,20 @@ unused_import_braces = "warn"
 unused_lifetimes = "warn"
 unused_macro_rules = "warn"
 
+[workspace.lints.cargo]
+manual_readme = "warn"
+missing_lints_inheritance = "warn"
+non_kebab_case_bins = "warn"
+non_kebab_case_features = "warn"
+non_kebab_case_packages = "warn"
+redundant_homepage = "warn"
+redundant_readme = "warn"
+unknown_lints = "warn"
+unused_dependencies = "warn"
+unused_workspace_dependencies = "warn"
+unused_workspace_package_fields = "warn"
+# TODO: Lint for implicit features?
+
 [workspace.lints.clippy]
 # Manually verified with a GitHub workflow
 incompatible_msrv = "allow"
@@ -351,16 +365,3 @@ suspicious_xor_used_as_pow = "warn"
 unnecessary_self_imports = "warn"
 unused_trait_names = "warn"
 wildcard_enum_match_arm = "warn"
-
-# TODO: `-Zcargo-lints`: [workspace.lints.cargo]
-# TODO: `-Zcargo-lints`: missing_lints_inheritance = "warn"
-# TODO: `-Zcargo-lints`: non_kebab_case_bins = "warn"
-# TODO: `-Zcargo-lints`: non_kebab_case_features = "warn"
-# TODO: `-Zcargo-lints`: non_kebab_case_packages = "warn"
-# TODO: `-Zcargo-lints`: redundant_homepage = "warn"
-# TODO: `-Zcargo-lints`: redundant_readme = "warn"
-# TODO: `-Zcargo-lints`: unknown_lints = "warn"
-# TODO: `-Zcargo-lints`: unused_dependencies = "warn"
-# TODO: `-Zcargo-lints`: unused_workspace_dependencies = "warn"
-# TODO: `-Zcargo-lints`: unused_workspace_package_fields = "warn"
-# TODO: Lint for implicit features?

LICENSE.md

@@ -44,12 +44,12 @@ Each portion of the Serai codebase (such as a "crate", in Rust terminology, but
 generally as organized within a folder) is expected to declare its license
 within its manifest and within its folder. For portions of the Serai codebase
 which may be _nested_ within each other, such as seen with
-`coordinator/tributary-sdk/tendermint`, the associated license is expected to
-be the one of closest proximity
+`coordinator/tributary-framework/tendermint`, the associated license is
+expected to be the one of closest proximity
 (e.g. `coordinator/LICENSE` is not expected to cover
-`coordinator/tributary-sdk/tendermint` as
-`coordinator/tributary-sdk/tendermint/LICENSE` exists and is immediate to
-`coordinator/tributary-sdk/tendermint`).
+`coordinator/tributary-framework/tendermint` as
+`coordinator/tributary-framework/tendermint/LICENSE` exists and is immediate to
+`coordinator/tributary-framework/tendermint`).
 
 If any files are of unclear licensing terms, or incorrectly licensed, please
 reach out as this is considered an error to be immediately corrected.

common/db/src/lib.rs

@@ -77,7 +77,7 @@ pub trait Transaction: Get {
   /// If no value is stored to this key, this operation is effectively a NOP.
   ///
   /// This is considered as a write to this key, with properties as documented by
-  /// [`Transaction::put`].
+  /// [`Transaction::set`].
   ///
   /// The implementation MAY panic if the database is corrupt or would become corrupted. If the
   /// length of the key exceeds an internal limit on the length of keys, this may be considered as

common/request/Cargo.toml

@@ -7,7 +7,7 @@ repository = "https://github.com/serai-dex/serai/tree/develop/common/request"
 authors = ["Luke Parker <lukeparker5132@gmail.com>"]
 keywords = ["http", "https", "async", "request", "ssl"]
 edition = "2021"
-rust-version = "1.81"
+rust-version = "1.85"
 
 [package.metadata.docs.rs]
 all-features = true
@@ -19,7 +19,7 @@ workspace = true
 [dependencies]
 tower-service = { version = "0.3", default-features = false }
 hyper = { version = "1", default-features = false, features = ["http1", "client"] }
-hyper-util = { version = "0.1.9", default-features = false, features = ["http1", "client-legacy"] }
+hyper-util = { version = "0.1.21", default-features = false, features = ["http1", "client-legacy"] }
 http-body-util = { version = "0.1.2", default-features = false }
 futures-util = { version = "0.3", default-features = false, features = ["std"] }
 tokio = { version = "1.13", default-features = false, features = ["sync"] }

common/zalloc/Cargo.toml

@@ -7,7 +7,7 @@ repository = "https://github.com/serai-dex/serai/tree/develop/common/zalloc"
 authors = ["Luke Parker <lukeparker5132@gmail.com>"]
 keywords = []
 edition = "2021"
-rust-version = "1.81"
+rust-version = "1.100"
 
 [package.metadata.docs.rs]
 all-features = true
@@ -23,4 +23,3 @@ zeroize = { version = "1.6", default-features = false }
 alloc = ["zeroize/alloc"]
 std = ["alloc", "zeroize/std"]
 default = ["std"]
-allocator = []

common/zalloc/README.md

@@ -9,13 +9,13 @@ A Rust allocator which zeroizes all memory on deallocation.
 
 ### Allocator
 
-On nightly, with the `allocator` feature enabled, [`ZeroizingAlloc`] also
-implements
-[`Allocator`](https://doc.rust-lang.org/1.96.0/alloc/alloc/trait.Allocator.html)
+[`ZeroizingAlloc`] implements
+[`Allocator`](https://doc.rust-lang.org/alloc/alloc/trait.Allocator.html)
 and can be used for individual values with
-[`Box`](https://doc.rust-lang.org/1.96.0/alloc/boxed/struct.Box.html). This is
-notable for effecting zeroize even for types which don't implement
-[`Zeroize`](https://docs.rs/zeroize/1.9.0/zeroize/trait.Zeroize.html).
+[`Box`](https://doc.rust-lang.org/alloc/boxed/struct.Box.html). This is
+notable for effecting zeroization even for types which don't implement
+[`Zeroize`](https://docs.rs/zeroize/1.9.0/zeroize/trait.Zeroize.html)
+_assuming in-place construction_.
 
 ### Safety