chore(all): remove once_cell (#1576)

## Summary
Migrated from usage of `once_cell` to `LazyLock`

## Background
Rust 1.81 stabilized `LazyLock`, we can now remove our dependency on
`once_cell`.

## Changes
- Removed `once_cell` as a dependency
- Replaced all implementations of `once_cell::sync::Lazy` to
`std::sync::LazyLock`

## Testing
Passing all tests

## Related Issues
closes #1520

Cargo.toml

@@ -73,7 +73,6 @@ insta = "1.36.1"
 itertools = "0.12.1"
 itoa = "1.0.10"
 jsonrpsee = { version = "0.20" }
-once_cell = "1.17.1"
 pbjson-types = "0.6"
 # Note that when updating the penumbra versions, vendored types in `proto/sequencerapis/astria_vendored` may need to be updated as well.
 # can update to a tagged release when https://github.com/penumbra-zone/penumbra/pull/4822 is included

crates/astria-bridge-withdrawer/Cargo.toml

@@ -21,7 +21,6 @@ ethers = { workspace = true, features = ["ws"] }
 hyper = { workspace = true }
 humantime = { workspace = true }
 ibc-types = { workspace = true }
-once_cell = { workspace = true }
 pin-project-lite = { workspace = true }
 prost = { workspace = true }
 serde = { workspace = true, features = ["derive"] }

crates/astria-bridge-withdrawer/tests/blackbox/helpers/test_bridge_withdrawer.rs

@@ -2,6 +2,7 @@ use std::{
     io::Write as _,
     mem,
     net::SocketAddr,
+    sync::LazyLock,
     time::Duration,
 };
 
@@ -37,7 +38,6 @@ use ibc_types::core::{
     channel::ChannelId,
     client::Height as IbcHeight,
 };
-use once_cell::sync::Lazy;
 use sequencer_client::{
     Address,
     NonceResponse,
@@ -74,7 +74,7 @@ pub(crate) const DEFAULT_IBC_DENOM: &str = "transfer/channel-0/utia";
 pub(crate) const SEQUENCER_CHAIN_ID: &str = "test-sequencer";
 const ASTRIA_ADDRESS_PREFIX: &str = "astria";
 
-static TELEMETRY: Lazy<()> = Lazy::new(|| {
+static TELEMETRY: LazyLock<()> = LazyLock::new(|| {
     if std::env::var_os("TEST_LOG").is_some() {
         let filter_directives = std::env::var("RUST_LOG").unwrap_or_else(|_| "info".into());
         telemetry::configure()
@@ -251,7 +251,7 @@ impl TestBridgeWithdrawerConfig {
             ethereum_config,
             asset_denom,
         } = self;
-        Lazy::force(&TELEMETRY);
+        LazyLock::force(&TELEMETRY);
 
         // sequencer signer key
         let keyfile = NamedTempFile::new().unwrap();

crates/astria-composer/Cargo.toml

@@ -30,7 +30,6 @@ humantime = { workspace = true }
 hyper = { workspace = true }
 hex = { workspace = true }
 itertools = { workspace = true }
-once_cell = { workspace = true }
 pin-project-lite = { workspace = true }
 prost = { workspace = true }
 reqwest = { workspace = true, features = ["json"] }

crates/astria-composer/src/executor/tests.rs

@@ -4,6 +4,7 @@ use std::{
         IpAddr,
         SocketAddr,
     },
+    sync::LazyLock,
     time::Duration,
 };
 
@@ -20,7 +21,6 @@ use astria_core::{
     protocol::transaction::v1alpha1::action::SequenceAction,
 };
 use astria_eyre::eyre;
-use once_cell::sync::Lazy;
 use prost::{
     bytes::Bytes,
     Message as _,
@@ -72,7 +72,7 @@ use crate::{
     Config,
 };
 
-static TELEMETRY: Lazy<()> = Lazy::new(|| {
+static TELEMETRY: LazyLock<()> = LazyLock::new(|| {
     // This config can be meaningless - it's only used inside `try_init` to init the metrics, but we
     // haven't configured telemetry to provide metrics here.
     let config = Config {
@@ -121,7 +121,7 @@ fn sequence_action() -> SequenceAction {
 
 /// Start a mock sequencer server and mount a mock for the `accounts/nonce` query.
 async fn setup() -> (MockServer, Config, NamedTempFile) {
-    Lazy::force(&TELEMETRY);
+    LazyLock::force(&TELEMETRY);
     let server = MockServer::start().await;
 
     let keyfile = NamedTempFile::new().unwrap();

crates/astria-composer/src/rollup.rs

@@ -1,8 +1,10 @@
 //! Parsing strings of the form `<rollup_name>::<url>`
 
-use std::fmt;
+use std::{
+    fmt,
+    sync::LazyLock,
+};
 
-use once_cell::sync::Lazy;
 use regex::Regex;
 
 #[derive(Debug)]
@@ -33,7 +35,7 @@ impl std::error::Error for ParseError {}
 
 impl Rollup {
     pub(super) fn parse(from: &str) -> Result<Self, ParseError> {
-        static ROLLUP_RE: Lazy<Regex> = Lazy::new(|| {
+        static ROLLUP_RE: LazyLock<Regex> = LazyLock::new(|| {
             Regex::new(
                 r"(?x)
                 ^(?P<rollup_name>[[:alnum:]-]+?)

crates/astria-composer/tests/blackbox/helper/mod.rs

@@ -5,6 +5,7 @@ use std::{
         IpAddr,
         SocketAddr,
     },
+    sync::LazyLock,
     time::Duration,
 };
 
@@ -28,7 +29,6 @@ use astria_core::{
 };
 use astria_eyre::eyre;
 use ethers::prelude::Transaction;
-use once_cell::sync::Lazy;
 use telemetry::metrics;
 use tempfile::NamedTempFile;
 use tendermint_rpc::{
@@ -50,7 +50,7 @@ use wiremock::{
 
 pub mod mock_sequencer;
 
-static TELEMETRY: Lazy<()> = Lazy::new(|| {
+static TELEMETRY: LazyLock<()> = LazyLock::new(|| {
     // This config can be meaningless - it's only used inside `try_init` to init the metrics, but we
     // haven't configured telemetry to provide metrics here.
     let config = Config {
@@ -107,7 +107,7 @@ pub struct TestComposer {
 /// There is no explicit error handling in favour of panicking loudly
 /// and early.
 pub async fn spawn_composer(rollup_ids: &[&str]) -> TestComposer {
-    Lazy::force(&TELEMETRY);
+    LazyLock::force(&TELEMETRY);
 
     let mut rollup_nodes = HashMap::new();
     let mut rollups = String::new();

crates/astria-conductor/Cargo.toml

@@ -83,7 +83,6 @@ config = { package = "astria-config", path = "../astria-config", features = [
 
 celestia-tendermint = { workspace = true }
 insta = { workspace = true, features = ["json"] }
-once_cell = { workspace = true }
 wiremock = { workspace = true }
 
 chrono = "0.4.35"

crates/astria-conductor/tests/blackbox/helpers/mod.rs

@@ -1,4 +1,7 @@
-use std::time::Duration;
+use std::{
+    sync::LazyLock,
+    time::Duration,
+};
 
 use astria_conductor::{
     conductor,
@@ -25,7 +28,6 @@ use celestia_types::{
     nmt::Namespace,
     Blob,
 };
-use once_cell::sync::Lazy;
 use prost::Message;
 use sequencer_client::{
     tendermint,
@@ -52,7 +54,7 @@ pub const SEQUENCER_CHAIN_ID: &str = "test_sequencer-1000";
 pub const INITIAL_SOFT_HASH: [u8; 64] = [1; 64];
 pub const INITIAL_FIRM_HASH: [u8; 64] = [1; 64];
 
-static TELEMETRY: Lazy<()> = Lazy::new(|| {
+static TELEMETRY: LazyLock<()> = LazyLock::new(|| {
     astria_eyre::install().unwrap();
     if std::env::var_os("TEST_LOG").is_some() {
         let filter_directives = std::env::var("RUST_LOG").unwrap_or_else(|_| "info".into());
@@ -82,7 +84,7 @@ pub async fn spawn_conductor(execution_commit_level: CommitLevel) -> TestConduct
          environment does not stall the runtime: the test could be configured using \
          `#[tokio::test(flavor = \"multi_thread\", worker_threads = 1)]`"
     );
-    Lazy::force(&TELEMETRY);
+    LazyLock::force(&TELEMETRY);
 
     let mock_grpc = MockGrpc::spawn().await;
     let mock_http = wiremock::MockServer::start().await;

crates/astria-config/Cargo.toml

@@ -14,12 +14,11 @@ homepage = "https://astria.org"
 figment = { version = "0.10.8", features = ["env"] }
 
 names = { version = "0.14", optional = true, default-features = false }
-once_cell = { workspace = true, optional = true }
 regex = { workspace = true, optional = true }
 serde = { workspace = true }
 
 [dev-dependencies]
 serde = { workspace = true, features = ["derive"] }
 
 [features]
-tests = ["figment/test", "dep:names", "dep:once_cell", "dep:regex"]
+tests = ["figment/test", "dep:names", "dep:regex"]

crates/astria-config/src/tests.rs

@@ -29,16 +29,17 @@
 //! }
 //! ```
 
+use std::sync::LazyLock;
+
 use figment::Jail;
-use once_cell::sync::Lazy;
 use regex::Regex;
 
 use crate::{
     Config,
     _internal,
 };
 
-static TEST_PREFIX: Lazy<String> = Lazy::new(|| {
+static TEST_PREFIX: LazyLock<String> = LazyLock::new(|| {
     use names::{
         Generator,
         Name,
@@ -47,8 +48,8 @@ static TEST_PREFIX: Lazy<String> = Lazy::new(|| {
 });
 
 fn populate_environment_from_example(jail: &mut Jail, unique_test_prefix: &str, example_env: &str) {
-    static RE_START: Lazy<Regex> = Lazy::new(|| Regex::new(r"^[[:space:]]+").unwrap());
-    static RE_END: Lazy<Regex> = Lazy::new(|| Regex::new(r"[[:space:]]+$").unwrap());
+    static RE_START: LazyLock<Regex> = LazyLock::new(|| Regex::new(r"^[[:space:]]+").unwrap());
+    static RE_END: LazyLock<Regex> = LazyLock::new(|| Regex::new(r"[[:space:]]+$").unwrap());
 
     for line in example_env.lines() {
         if let Some((key, val)) = line.trim().split_once('=') {
@@ -71,7 +72,7 @@ fn populate_environment_from_example(jail: &mut Jail, unique_test_prefix: &str,
 /// Panics if a config `C` could not be created from `example_env`.
 #[track_caller]
 pub fn example_env_config_is_up_to_date<C: Config>(example_env: &str) {
-    let unique_test_prefix = Lazy::force(&TEST_PREFIX);
+    let unique_test_prefix = LazyLock::force(&TEST_PREFIX);
     let full_test_prefix = format!("{unique_test_prefix}_{}", C::PREFIX);
 
     Jail::expect_with(|jail| {

crates/astria-sequencer-relayer/Cargo.toml

@@ -77,7 +77,6 @@ celestia-tendermint = { workspace = true }
 celestia-types = { workspace = true }
 hyper = { workspace = true }
 itertools = { workspace = true }
-once_cell = { workspace = true }
 rand_core = { version = "0.6", features = ["getrandom"] }
 tempfile = { workspace = true }
 tendermint-rpc = { workspace = true, features = ["http-client"] }

crates/astria-sequencer-relayer/tests/blackbox/helpers/test_sequencer_relayer.rs

@@ -10,6 +10,7 @@ use std::{
     io::Write,
     mem,
     net::SocketAddr,
+    sync::LazyLock,
     time::Duration,
 };
 
@@ -28,7 +29,6 @@ use astria_sequencer_relayer::{
 use http::StatusCode;
 use isahc::AsyncReadResponseExt;
 use itertools::Itertools;
-use once_cell::sync::Lazy;
 use serde::Deserialize;
 use serde_json::json;
 use telemetry::metrics;
@@ -123,7 +123,7 @@ const STATUS_RESPONSE: &str = r#"
   }
 }"#;
 
-static TELEMETRY: Lazy<()> = Lazy::new(|| {
+static TELEMETRY: LazyLock<()> = LazyLock::new(|| {
     astria_eyre::install().unwrap();
     if std::env::var_os("TEST_LOG").is_some() {
         let filter_directives = std::env::var("RUST_LOG")
@@ -675,7 +675,7 @@ impl TestSequencerRelayerConfig {
             "the sequencer relayer must be run on a multi-threaded runtime, e.g. the test could \
              be configured using `#[tokio::test(flavor = \"multi_thread\", worker_threads = 1)]`"
         );
-        Lazy::force(&TELEMETRY);
+        LazyLock::force(&TELEMETRY);
 
         let celestia_app = MockCelestiaAppServer::spawn(self.celestia_chain_id.clone()).await;
         let celestia_app_grpc_endpoint = format!("http://{}", celestia_app.local_addr);