feat: adjust pub fields

Signed-off-by: YUE Daian <[email protected]>

src/api/api.rs

@@ -13,7 +13,7 @@ use super::{
 lazy_static! {
     /// The singleton instance of [`OpenFeature`] struct.
     /// The client should always use this instance to access OpenFeature APIs.
-    pub static ref SINGLETON: RwLock<OpenFeature> = RwLock::new(OpenFeature::default());
+    static ref SINGLETON: RwLock<OpenFeature> = RwLock::new(OpenFeature::default());
 }
 
 /// THE struct of the OpenFeature API.
@@ -48,13 +48,18 @@ impl OpenFeature {
 
     /// Return the metadata of default (unnamed) provider.
     pub async fn provider_metadata(&self) -> ProviderMetadata {
-        self.provider_registry.get_default().await.get().metadata()
+        self.provider_registry
+            .get_default()
+            .await
+            .get()
+            .metadata()
+            .clone()
     }
 
     /// Return the metadata of named provider (a provider bound to clients with this name).
     pub async fn named_provider_metadata(&self, name: &str) -> Option<ProviderMetadata> {
         match self.provider_registry.get_named(name).await {
-            Some(provider) => Some(provider.get().metadata()),
+            Some(provider) => Some(provider.get().metadata().clone()),
             None => None,
         }
     }

src/api/client.rs

@@ -7,13 +7,7 @@ use super::{
 
 /// The metadata of OpenFeature client.
 pub struct ClientMetadata {
-    name: String,
-}
-
-impl ClientMetadata {
-    pub fn name(&self) -> &str {
-        &self.name.as_ref()
-    }
+    pub name: String,
 }
 
 /// The OpenFeature client.

src/api/global_evaluation_context.rs

@@ -5,7 +5,7 @@ use tokio::sync::{RwLock, RwLockReadGuard, RwLockWriteGuard};
 use crate::EvaluationContext;
 
 #[derive(Clone, Default)]
-pub struct GlobalEvaluationContext(pub Arc<RwLock<EvaluationContext>>);
+pub struct GlobalEvaluationContext(Arc<RwLock<EvaluationContext>>);
 
 impl GlobalEvaluationContext {
     pub async fn get(&self) -> RwLockReadGuard<EvaluationContext> {

src/api/mod.rs

@@ -1,5 +1,5 @@
 mod api;
-pub use api::*;
+pub use api::OpenFeature;
 
 mod client;
 pub use client::Client;

src/evaluation/details.rs

@@ -64,8 +64,8 @@ impl ToString for EvaluationReason {
 /// Struct representing error
 #[derive(Clone, Eq, PartialEq, Debug)]
 pub struct EvaluationError {
-    code: EvaluationErrorCode,
-    message: Option<String>,
+    pub code: EvaluationErrorCode,
+    pub message: Option<String>,
 }
 
 /// An enumerated error code represented idiomatically in the implementation language.

src/lib.rs

@@ -1,5 +1,5 @@
 mod api;
-pub use api::*;
+pub use api::{Client, OpenFeature};
 
 mod evaluation;
 pub use evaluation::*;

src/provider/feature_provider.rs

@@ -47,7 +47,7 @@ pub trait FeatureProvider: Send + Sync + 'static {
 
     /// The provider interface MUST define a metadata member or accessor, containing a name field
     /// or accessor of type string, which identifies the provider implementation.
-    fn metadata(&self) -> ProviderMetadata;
+    fn metadata(&self) -> &ProviderMetadata;
 
     /// Resolve given `flag_key` as a bool value.
     async fn resolve_bool_value(

src/provider/fixed_value_provider.rs

@@ -49,8 +49,8 @@ impl Default for FixedValueProvider {
 
 #[async_trait]
 impl FeatureProvider for FixedValueProvider {
-    fn metadata(&self) -> ProviderMetadata {
-        self.metadata.clone()
+    fn metadata(&self) -> &ProviderMetadata {
+        &self.metadata
     }
 
     async fn resolve_bool_value(

src/provider/no_op_provider.rs

@@ -30,8 +30,8 @@ impl Default for NoOpProvider {
 
 #[async_trait]
 impl FeatureProvider for NoOpProvider {
-    fn metadata(&self) -> ProviderMetadata {
-        self.metadata.clone()
+    fn metadata(&self) -> &ProviderMetadata {
+        &self.metadata
     }
 
     async fn resolve_bool_value(