From faa2d00cc6ef7153e00dea855cecde44331b6885 Mon Sep 17 00:00:00 2001 From: "autofix-ci[bot]" <114827586+autofix-ci[bot]@users.noreply.github.com> Date: Tue, 16 Sep 2025 09:12:31 +0000 Subject: [PATCH] [autofix.ci] apply automated fixes --- api/core/entities/mcp_provider.py | 8 ++++---- api/core/mcp/auth_client.py | 2 +- api/core/mcp/auth_client_comparison.md | 1 - api/services/tools/mcp_tools_manage_service.py | 2 +- 4 files changed, 6 insertions(+), 7 deletions(-) diff --git a/api/core/entities/mcp_provider.py b/api/core/entities/mcp_provider.py index 3ce95864f3..8ae39aabce 100644 --- a/api/core/entities/mcp_provider.py +++ b/api/core/entities/mcp_provider.py @@ -38,7 +38,7 @@ class MCPProviderEntity(BaseModel): # Authentication related authed: bool credentials: dict[str, Any] # encrypted credentials - code_verifier: Optional[str] = None # for OAuth + code_verifier: str | None = None # for OAuth # Tools and display info tools: list[dict[str, Any]] # parsed tools list @@ -98,7 +98,7 @@ class MCPProviderEntity(BaseModel): # If not JSON, assume it's a file path return file_helpers.get_signed_file_url(self.icon) - def to_api_response(self, user_name: Optional[str] = None) -> dict[str, Any]: + def to_api_response(self, user_name: str | None = None) -> dict[str, Any]: """Convert to API response format""" return { "id": self.id, @@ -117,14 +117,14 @@ class MCPProviderEntity(BaseModel): "description": I18nObject(en_US="", zh_Hans="").to_dict(), } - def retrieve_client_information(self) -> Optional[OAuthClientInformation]: + def retrieve_client_information(self) -> OAuthClientInformation | None: """OAuth client information if available""" client_info = self.decrypt_credentials().get("client_information", {}) if not client_info: return None return OAuthClientInformation.model_validate(client_info) - def retrieve_tokens(self) -> Optional[OAuthTokens]: + def retrieve_tokens(self) -> OAuthTokens | None: """OAuth tokens if available""" if not self.credentials: return None diff --git a/api/core/mcp/auth_client.py b/api/core/mcp/auth_client.py index dac0f3f3ec..839090fde9 100644 --- a/api/core/mcp/auth_client.py +++ b/api/core/mcp/auth_client.py @@ -37,7 +37,7 @@ class MCPClientWithAuthRetry(MCPClient): provider_entity: MCPProviderEntity | None = None, auth_callback: Callable[[MCPProviderEntity, "MCPToolManageService", Optional[str]], dict[str, str]] | None = None, - authorization_code: Optional[str] = None, + authorization_code: str | None = None, by_server_id: bool = False, mcp_service: Optional["MCPToolManageService"] = None, ): diff --git a/api/core/mcp/auth_client_comparison.md b/api/core/mcp/auth_client_comparison.md index 0519ecba6e..e69de29bb2 100644 --- a/api/core/mcp/auth_client_comparison.md +++ b/api/core/mcp/auth_client_comparison.md @@ -1 +0,0 @@ - \ No newline at end of file diff --git a/api/services/tools/mcp_tools_manage_service.py b/api/services/tools/mcp_tools_manage_service.py index 59a0e8c6cb..a886ba5614 100644 --- a/api/services/tools/mcp_tools_manage_service.py +++ b/api/services/tools/mcp_tools_manage_service.py @@ -33,7 +33,7 @@ class MCPToolManageService: # ========== Provider CRUD Operations ========== def get_provider( - self, *, provider_id: Optional[str] = None, server_identifier: Optional[str] = None, tenant_id: str + self, *, provider_id: str | None = None, server_identifier: str | None = None, tenant_id: str ) -> MCPToolProvider: """ Get MCP provider by ID or server identifier.