refactor: centralize dependencies and feature-gate stacks-common

[Jiloc]

Description

This PR is the first of two (?) aiming at modularizing the clarity crate to enable a lightweight serialization/deserialization mode AND building on Wasm32 target, in two flavors:

• wasm-web, with js support for the entire VM
• wasm-deterministic for deterministc environments like CosmWasm, where rand is not supported.

This initial step focuses on preparing the workspace and refactoring stacks-common and feature-gate a couple of non-necessary features in stacks-common.

The key changes are:

• Centralizing common dependencies (like serde, slog, lazy_static..) in the root Cargo.toml under [workspace.dependencies]. Some of those dependencies are only used by the stacks-common create in this PR, but will be used by the clarity create in the following.
• Making nix optional in stacks-common via new ctrlc-handler feature flag.
• Removed time from the dependencies, as it's not currently used (directly at least).
• Remove default-features to chrono so to prevent pulling in additional dependencies that don't compile on CosmWasm and we don't use.

Extra, not mandatory (open to move in another PR):

• Upgraded chrono to 0.4.41 from 0.4.19 to fix a known (minor for us) vulnerability https://osv.dev/vulnerability/RUSTSEC-2020-0159. We only use chrono for adding the timestamp to the logs.
• Remove default-features for other dependencies we are not using.
• Fix a clippy::let_and_return warning

This change is fully backward compatible. The default features have been updated to preserve the existing behavior, so no changes are required for downstream crates.

Super open to suggestions for improvements/gating additional features!

Applicable issues

• first step of Refactor clarity to enable lightweight serialization crate #6236

Additional info (benefits, drawbacks, caveats)

Checklist

• Test coverage for new or modified code paths
• Changelog is updated
• Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)
• New clarity functions have corresponding PR in clarity-benchmarking repo
• New integration test(s) added to bitcoin-tests.yml

[fdefelici]

Looks good!

About the http-parser flag, I generally prefer keeping the number of feature flags to a minimum for simplicity 😊
That said, if we were to remove this flag, do you have a specific alternative in mind?

[kantai]

LGTM!

[Jiloc]

Looks good!

About the http-parser flag, I generally prefer keeping the number of feature flags to a minimum for simplicity 😊 That said, if we were to remove this flag, do you have a specific alternative in mind?

In the end the comment you made convinced me to remove it. The feature was small, and didn't add additional dependencies to the mix, no need to find a workaround :)

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 81.80%. Comparing base (381ee9b) to head (222096b).
Report is 16 commits behind head on develop.

Files with missing lines	Patch %	Lines
stacks-common/src/util/secp256k1/native.rs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #6238      +/-   ##
===========================================
- Coverage    81.91%   81.80%   -0.11%     
===========================================
  Files          546      545       -1     
  Lines       347235   347218      -17     
  Branches       323      323              
===========================================
- Hits        284441   284057     -384     
- Misses       62786    63153     +367     
  Partials         8        8              

Files with missing lines	Coverage Δ
stacks-common/src/deps_common/bitcoin/util/mod.rs	0.00% <ø> (ø)
stacks-common/src/libcommon.rs	75.00% <ø> (ø)
stacks-common/src/types/chainstate.rs	85.37% <100.00%> (ø)
stacks-common/src/util/chunked_encoding.rs	95.16% <ø> (ø)
stacks-common/src/util/mod.rs	88.88% <ø> (ø)
stacks-common/src/util/pipe.rs	86.25% <ø> (ø)
stacks-common/src/util/vrf.rs	88.15% <100.00%> (+0.52%)	⬆️
stackslib/src/burnchains/tests/burnchain.rs	99.86% <100.00%> (+<0.01%)	⬆️
stacks-common/src/util/secp256k1/native.rs	78.27% <0.00%> (+0.34%)	⬆️

... and 55 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 381ee9b...222096b. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.