fix: feature fixes (#409)

## What ❔

- Make basic bootloader compilable with no_std not only on risc_v
- Remove testing feature on basic_bootloader when using api crate

## Why ❔

To have correct feature set and make it compilable in the sequencer

## Is this a breaking change?
- [ ] Yes
- [ ] No

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [ ] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
- [ ] Code has been formatted.

Author: AntonD3

api/Cargo.toml

@@ -11,15 +11,15 @@ categories.workspace = true
 
 [dependencies]
 
-forward_system = { path = "../forward_system"}
+forward_system = { path = "../forward_system", default-features = false}
 oracle_provider = { path = "../oracle_provider" }
 callable_oracles = { path = "../callable_oracles/" }
 zksync_os_runner = { path = "../zksync_os_runner"}
 risc_v_simulator = { workspace = true }
-zk_ee = { path = "../zk_ee" }
-crypto = { path = "../crypto" }
-basic_system = { path = "../basic_system" }
-evm_interpreter = { path = "../evm_interpreter" }
+zk_ee = { path = "../zk_ee", default-features = false }
+crypto = { path = "../crypto", default-features = false }
+basic_system = { path = "../basic_system", default-features = false }
+evm_interpreter = { path = "../evm_interpreter", default-features = false }
 ruint = { version = "1.12.3", default-features = false, features = ["alloc"] }
 alloy = { version = "=1", default-features = false, features = ["eip712", "consensus", "rpc-types", "signer-local", "dyn-abi", "network"] }
 zksync_os_interface = { workspace = true }

basic_bootloader/src/bootloader/mod.rs

@@ -332,9 +332,10 @@ where
 
         let l1_to_l2_tx_hash = Bytes32::from(l1_to_l2_txs_hasher.finalize());
 
+        // uses global alloc
         #[cfg(not(target_arch = "riscv32"))]
         cycle_marker::log_marker(
-            format!(
+            alloc::format!(
                 "Spent ergs for [run_prepared]: {}",
                 result_keeper.get_gas_used() * evm_interpreter::ERGS_PER_GAS
             )

basic_bootloader/src/bootloader/process_transaction.rs

@@ -754,17 +754,20 @@ where
             + intrinsic_computational_native_charged
             - main_native_used_for_pubdata;
 
+        // uses global alloc
         #[cfg(not(target_arch = "riscv32"))]
         cycle_marker::log_marker(
-            format!(
+            alloc::format!(
                 "Spent ergs for [process_transaction]: {}",
                 gas_used * ERGS_PER_GAS
             )
             .as_str(),
         );
+        // uses global alloc
         #[cfg(not(target_arch = "riscv32"))]
         cycle_marker::log_marker(
-            format!("Spent native for [process_transaction]: {computational_native_used}").as_str(),
+            alloc::format!("Spent native for [process_transaction]: {computational_native_used}")
+                .as_str(),
         );
 
         Ok(TxProcessingResult {

zk_ee/src/system/errors/context/nonempty.rs

@@ -9,33 +9,33 @@ use super::{
 
 #[derive(Debug, Default, Clone, PartialEq, Eq)]
 pub struct ErrorContext {
-    values: Vec<NamedContextElement>,
+    values: alloc::vec::Vec<NamedContextElement>,
 }
 
 impl IErrorContext for ErrorContext {
     #[inline(always)]
-    fn get(&self, name: &str) -> Option<&String> {
+    fn get(&self, name: &str) -> Option<&alloc::string::String> {
         self.values
             .iter()
             .find(|e| (e.name == name))
             .map(|e| &e.value)
     }
 
     #[inline(always)]
-    fn to_vec(&self) -> Option<Vec<NamedContextElement>> {
+    fn to_vec(&self) -> Option<alloc::vec::Vec<NamedContextElement>> {
         Some(self.values.clone())
     }
 
     #[inline(always)]
-    fn into_vec(self) -> Option<Vec<NamedContextElement>> {
+    fn into_vec(self) -> Option<alloc::vec::Vec<NamedContextElement>> {
         Some(self.values)
     }
 
     #[inline(always)]
     fn push(
         mut self,
         name: &'static str,
-        value: impl ToString,
+        value: impl alloc::string::ToString,
         visibility: ValueVisibility,
     ) -> Self {
         let mut perform_push_value = || {
@@ -57,7 +57,7 @@ impl IErrorContext for ErrorContext {
     #[inline(always)]
     fn push_lazy<F>(mut self, name: &'static str, f: F, visibility: ValueVisibility) -> Self
     where
-        F: FnOnce() -> String,
+        F: FnOnce() -> alloc::string::String,
     {
         let should_include = match visibility {
             ValueVisibility::AnyForwardRun => true,