Refine Cargo machete checks and dependencies

- Run `cargo machete --with-metadata` in pre-commit
- Remove unused Betfair, IB, and system dev deps
- Add machete ignores for feature-gated build deps
- Gate serialization schema build deps behind `capnp`

Author: cjdsellers

.pre-commit-hooks/cargo_machete.sh

@@ -5,7 +5,7 @@
 # lists in each crate's Cargo.toml. Known categories:
 #   - Feature-graph plumbing (optional deps referenced only in [features])
 #   - Macro-expansion-only deps (brought into scope by a derive elsewhere)
-#   - Build-script deps (handled by default invocation; do not use --with-metadata)
+#   - Build-script deps (cargo-machete cannot always see feature-gated build.rs uses)
 
 set -euo pipefail
 
@@ -24,8 +24,8 @@ if [ "$installed_version" != "$PINNED_VERSION" ]; then
   echo "         cargo install --locked cargo-machete@${PINNED_VERSION}"
 fi
 
-echo "Running cargo machete..."
-if ! cargo machete; then
+echo "Running cargo machete --with-metadata..."
+if ! cargo machete --with-metadata; then
   echo ""
   echo "If a flagged dependency is a false positive (feature-gate plumbing,"
   echo "macro expansion, etc.), add it to the crate's Cargo.toml with a"

Cargo.lock

@@ -8,15 +8,15 @@ members = [
   "crates/adapters/bitmex",
   "crates/adapters/blockchain",
   "crates/adapters/bybit",
+  "crates/adapters/coinbase",
   "crates/adapters/databento",
   "crates/adapters/deribit",
   "crates/adapters/dydx",
-  "crates/adapters/coinbase",
   "crates/adapters/hyperliquid",
-  "crates/adapters/kraken",
-  "crates/adapters/polymarket",
   "crates/adapters/interactive_brokers",
+  "crates/adapters/kraken",
   "crates/adapters/okx",
+  "crates/adapters/polymarket",
   "crates/adapters/sandbox",
   "crates/adapters/tardis",
   "crates/analysis",
@@ -32,9 +32,9 @@ members = [
   "crates/infrastructure",
   "crates/live",
   "crates/model",
-  "crates/persistence/macros",
   "crates/network",
   "crates/persistence",
+  "crates/persistence/macros",
   "crates/plugin",
   "crates/portfolio",
   "crates/pyo3",
@@ -372,7 +372,6 @@ pretty_assertions = "1.4.1"
 proptest = { version = "1.11.0", default-features = false, features = ["std"] }
 rstest = "0.26.1"
 tempfile = "3.27.0"
-tracing-test = "0.2.6"
 turmoil = "0.7.2"
 
 # -----------------------------------------------------------------------------

Cargo.toml

@@ -76,7 +76,6 @@ pyo3-stub-gen = { workspace = true, optional = true }
 
 [dev-dependencies]
 nautilus-backtest = { workspace = true }
-nautilus-execution = { workspace = true }
 nautilus-testkit = { workspace = true }
 nautilus-trading = { workspace = true }
 

crates/adapters/betfair/Cargo.toml

@@ -100,7 +100,6 @@ nautilus-trading = { workspace = true }
 rstest = { workspace = true }
 rust_decimal_macros = { workspace = true }
 tempfile = { workspace = true }
-tracing-test = { workspace = true }
 
 # Run live smoke with `cargo run --example ib-data-tester --package nautilus-interactive-brokers --features examples`.
 # Run embedded config unit tests with `cargo test --example ib-data-tester --package nautilus-interactive-brokers --features examples`.

crates/adapters/interactive_brokers/Cargo.toml

@@ -57,6 +57,11 @@ python = [
   "streaming",
 ]
 
+[package.metadata.cargo-machete]
+ignored = [
+  "cbindgen", # Used by build.rs when the `ffi` feature is enabled
+]
+
 [package.metadata.docs.rs]
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]

crates/backtest/Cargo.toml

@@ -46,6 +46,11 @@ high-precision = ["nautilus-model/high-precision"]
 capnp = ["nautilus-serialization/capnp"]
 tracing-bridge = ["tracing", "tracing-subscriber"]
 
+[package.metadata.cargo-machete]
+ignored = [
+  "cbindgen", # Used by build.rs when the `ffi` feature is enabled
+]
+
 [package.metadata.docs.rs]
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]

crates/common/Cargo.toml

@@ -27,6 +27,12 @@ extension-module = ["python", "pyo3/extension-module"]
 ffi = ["cbindgen"]
 python = ["pyo3", "pyo3-stub-gen", "strum"]
 
+[package.metadata.cargo-machete]
+ignored = [
+  "cbindgen", # Used by build.rs when the `ffi` feature is enabled
+  "toml", # Used by build.rs to read pyproject.toml version metadata
+]
+
 [package.metadata.docs.rs]
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]

crates/core/Cargo.toml

@@ -37,6 +37,11 @@ high-precision = []
 defi = ["alloy-primitives", "high-precision"]
 cython-compat = []
 
+[package.metadata.cargo-machete]
+ignored = [
+  "cbindgen", # Used by build.rs when the `ffi` feature is enabled
+]
+
 [package.metadata.docs.rs]
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]

crates/model/Cargo.toml

@@ -34,7 +34,7 @@ crate-type = ["rlib"]
 [features]
 default = []
 arrow = ["dep:arrow", "nautilus-model/arrow"]
-capnp = ["dep:capnp"]
+capnp = ["dep:capnp", "dep:capnpc", "dep:walkdir"]
 display = ["arrow"]
 sbe = []
 extension-module = [
@@ -51,6 +51,12 @@ python = [
 ]
 high-precision = ["nautilus-model/high-precision"]
 
+[package.metadata.cargo-machete]
+ignored = [
+  "capnpc", # Used by build.rs when the `capnp` feature is enabled
+  "walkdir", # Used by build.rs when the `capnp` feature is enabled
+]
+
 [package.metadata.docs.rs]
 all-features = true
 rustdoc-args = ["--cfg", "docsrs"]
@@ -80,8 +86,8 @@ pretty_assertions = { workspace = true }
 rstest = { workspace = true }
 
 [build-dependencies]
-capnpc = { workspace = true }
-walkdir = { workspace = true }
+capnpc = { workspace = true, optional = true }
+walkdir = { workspace = true, optional = true }
 
 [[bench]]
 name = "capnp_serialization"