Ensure we can always build bitwarden-sm (#333)

We need to ensure `bitwarden-sm` is always buildable to avoid breaking
sdk-sm.

Author: Hinton

.github/workflows/build-rust-crates.yml

@@ -10,6 +10,9 @@ on:
 env:
   CARGO_TERM_COLOR: always
 
+permissions:
+  contents: read
+
 jobs:
   build:
     name: Building ${{matrix.package}} for - ${{ matrix.os }}
@@ -27,8 +30,10 @@ jobs:
         package:
           - bitwarden-api-api
           - bitwarden-api-identity
+          - bitwarden-core
           - bitwarden-crypto
           - bitwarden-generators
+          - bitwarden-sm
 
     steps:
       - name: Checkout

crates/bitwarden-core/src/client/internal.rs

@@ -10,9 +10,10 @@ use bitwarden_state::registry::StateRegistry;
 use chrono::Utc;
 use uuid::Uuid;
 
-use super::encryption_settings::EncryptionSettings;
+#[cfg(any(feature = "internal", feature = "secrets"))]
+use crate::client::encryption_settings::EncryptionSettings;
 #[cfg(feature = "secrets")]
-use super::login_method::ServiceAccountLoginMethod;
+use crate::client::login_method::ServiceAccountLoginMethod;
 use crate::{
     auth::renew::renew_token, client::login_method::LoginMethod, error::UserIdAlreadySetError,
     key_management::KeyIds, DeviceType,

crates/bitwarden-core/src/key_management/mod.rs

@@ -9,9 +9,11 @@
 //!   [Encryptable](bitwarden_crypto::Encryptable) and [Decryptable](bitwarden_crypto::Encryptable).
 use bitwarden_crypto::{key_ids, KeyStore, SymmetricCryptoKey};
 
+#[cfg(feature = "internal")]
 pub mod crypto;
+#[cfg(feature = "internal")]
 mod crypto_client;
-
+#[cfg(feature = "internal")]
 pub use crypto_client::CryptoClient;
 
 key_ids! {