refactor: replace pub(crate) with ::new()

bitwarden · Oct 14, 2024 · fe72f34 · fe72f34
1 parent 5ea7645
commit fe72f34
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 6 deletions.
diff --git a/crates/bitwarden-wasm-internal/src/client.rs b/crates/bitwarden-wasm-internal/src/client.rs
@@ -58,10 +58,10 @@ impl BitwardenClient {
     }
 
     pub fn crypto(&self) -> ClientCrypto {
-        ClientCrypto(self.0.clone())
+        ClientCrypto::new(self.0.clone())
     }
 
     pub fn vault(&self) -> ClientVault {
-        ClientVault(self.0.clone())
+        ClientVault::new(self.0.clone())
     }
 }
diff --git a/crates/bitwarden-wasm-internal/src/crypto.rs b/crates/bitwarden-wasm-internal/src/crypto.rs
@@ -9,7 +9,13 @@ use wasm_bindgen::prelude::*;
 use crate::error::Result;
 
 #[wasm_bindgen]
-pub struct ClientCrypto(pub(crate) Rc<Client>);
+pub struct ClientCrypto(Rc<Client>);
+
+impl ClientCrypto {
+    pub fn new(client: Rc<Client>) -> Self {
+        Self(client)
+    }
+}
 
 #[wasm_bindgen]
 impl ClientCrypto {

diff --git a/crates/bitwarden-wasm-internal/src/vault/folders.rs b/crates/bitwarden-wasm-internal/src/vault/folders.rs
@@ -7,7 +7,13 @@ use wasm_bindgen::prelude::*;
 use crate::error::Result;
 
 #[wasm_bindgen]
-pub struct ClientFolders(pub(crate) Rc<Client>);
+pub struct ClientFolders(Rc<Client>);
+
+impl ClientFolders {
+    pub fn new(client: Rc<Client>) -> Self {
+        Self(client)
+    }
+}
 
 #[wasm_bindgen]
 impl ClientFolders {

diff --git a/crates/bitwarden-wasm-internal/src/vault/mod.rs b/crates/bitwarden-wasm-internal/src/vault/mod.rs
@@ -8,11 +8,17 @@ use wasm_bindgen::prelude::*;
 use crate::ClientFolders;
 
 #[wasm_bindgen]
-pub struct ClientVault(pub(crate) Rc<Client>);
+pub struct ClientVault(Rc<Client>);
+
+impl ClientVault {
+    pub fn new(client: Rc<Client>) -> Self {
+        Self(client)
+    }
+}
 
 #[wasm_bindgen]
 impl ClientVault {
     pub fn folders(&self) -> ClientFolders {
-        ClientFolders(self.0.clone())
+        ClientFolders::new(self.0.clone())
     }
 }