clean up

agixt/graphqlendpoints/Conversations.py

@@ -260,7 +260,7 @@ class NotificationEvent:
 @strawberry.type
 class ProviderSetting:
     name: str
-    value: Optional[str]
+    default_value: Optional[str]
 
 
 @strawberry.type
@@ -317,7 +317,7 @@ class EmbedderList:
 def convert_settings_to_type(settings_dict: Dict[str, str]) -> List[ProviderSetting]:
     """Convert settings dictionary to list of ProviderSetting objects"""
     return [
-        ProviderSetting(name=key, value=str(value))
+        ProviderSetting(name=key, default_value=str(value))
         for key, value in settings_dict.items()
     ]
 
@@ -532,28 +532,24 @@ async def notifications(
             page_info=page_info, edges=notifications[start_idx:end_idx]
         )
 
-    @strawberry.field
-    async def providers(self, info) -> List[ProviderWithSettings]:
-        """Get all providers with their settings"""
-        user = await get_user_from_context(info)
-        providers_settings = get_providers_with_settings()
-        return [
-            ProviderWithSettings(
-                provider=Provider(
-                    name=list(provider.keys())[0],
-                    settings=convert_settings_to_type(list(provider.values())[0]),
-                )
-            )
-            for provider in providers_settings
-        ]
-
     @strawberry.field
     async def providers_by_service(self, info, service: str) -> ProviderList:
         """Get providers that offer a specific service"""
         user = await get_user_from_context(info)
         providers = get_providers_by_service(service=service)
         return ProviderList(providers=providers)
 
+    @strawberry.field
+    async def providers(self, info) -> ProvidersWithDetails:
+        """Get comprehensive provider details"""
+        user = await get_user_from_context(info)
+        provider_details = get_providers_with_details()
+        providers = [
+            convert_provider_details({"name": name, **details})
+            for name, details in provider_details.items()
+        ]
+        return ProvidersWithDetails(providers=providers)
+
 
 # Response types for mutations
 @strawberry.type