From f40e2cf98a2c82e107580d79259f7f67f258e1e4 Mon Sep 17 00:00:00 2001 From: Yongtao Huang Date: Thu, 14 Aug 2025 19:52:07 +0800 Subject: [PATCH] Fix: remove redundant allowed_keys check in jsonable_encoder (#23931) Signed-off-by: Yongtao Huang Co-authored-by: Yongtao Huang <99629139+hyongtao-db@users.noreply.github.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> --- api/core/model_runtime/utils/encoders.py | 7 ++----- api/core/provider_manager.py | 2 +- api/core/workflow/workflow_engine_manager.py | 0 3 files changed, 3 insertions(+), 6 deletions(-) delete mode 100644 api/core/workflow/workflow_engine_manager.py diff --git a/api/core/model_runtime/utils/encoders.py b/api/core/model_runtime/utils/encoders.py index a5c11aeeba..f65339fbfc 100644 --- a/api/core/model_runtime/utils/encoders.py +++ b/api/core/model_runtime/utils/encoders.py @@ -151,12 +151,9 @@ def jsonable_encoder( return format(obj, "f") if isinstance(obj, dict): encoded_dict = {} - allowed_keys = set(obj.keys()) for key, value in obj.items(): - if ( - (not sqlalchemy_safe or (not isinstance(key, str)) or (not key.startswith("_sa"))) - and (value is not None or not exclude_none) - and key in allowed_keys + if (not sqlalchemy_safe or (not isinstance(key, str)) or (not key.startswith("_sa"))) and ( + value is not None or not exclude_none ): encoded_key = jsonable_encoder( key, diff --git a/api/core/provider_manager.py b/api/core/provider_manager.py index 6de4f3a303..9250497d29 100644 --- a/api/core/provider_manager.py +++ b/api/core/provider_manager.py @@ -523,7 +523,7 @@ class ProviderManager: # Init trial provider records if not exists if ProviderQuotaType.TRIAL not in provider_quota_to_provider_record_dict: try: - # FIXME ignore the type errork, onyl TrialHostingQuota has limit need to change the logic + # FIXME ignore the type error, only TrialHostingQuota has limit need to change the logic new_provider_record = Provider( tenant_id=tenant_id, # TODO: Use provider name with prefix after the data migration. diff --git a/api/core/workflow/workflow_engine_manager.py b/api/core/workflow/workflow_engine_manager.py deleted file mode 100644 index e69de29bb2..0000000000