From 1704812f50442f04007f8bf208a0a2d8a8c10708 Mon Sep 17 00:00:00 2001 From: Bentlybro Date: Fri, 13 Feb 2026 16:37:54 +0000 Subject: [PATCH] fix: address CodeRabbit review comments - Fix cache initialization inconsistency in BlockSchema.__pydantic_init_subclass__ (use None instead of {} to match clear_schema_cache behavior) - Change logger.error to logger.debug in stagehand blocks (not an error condition) --- autogpt_platform/backend/backend/blocks/_base.py | 3 ++- autogpt_platform/backend/backend/blocks/stagehand/blocks.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/autogpt_platform/backend/backend/blocks/_base.py b/autogpt_platform/backend/backend/blocks/_base.py index 2fff25f369..2d6fd7a764 100644 --- a/autogpt_platform/backend/backend/blocks/_base.py +++ b/autogpt_platform/backend/backend/blocks/_base.py @@ -243,7 +243,8 @@ class BlockSchema(BaseModel): super().__pydantic_init_subclass__(**kwargs) # Reset cached JSON schema to prevent inheriting it from parent class - cls.cached_jsonschema = {} + # Use None instead of {} because {} is truthy and would prevent regeneration + cls.cached_jsonschema = None credentials_fields = cls.get_credentials_fields() diff --git a/autogpt_platform/backend/backend/blocks/stagehand/blocks.py b/autogpt_platform/backend/backend/blocks/stagehand/blocks.py index 78cc47ce96..0562064dbc 100644 --- a/autogpt_platform/backend/backend/blocks/stagehand/blocks.py +++ b/autogpt_platform/backend/backend/blocks/stagehand/blocks.py @@ -102,7 +102,7 @@ class StagehandRecommendedLlmModel(str, Enum): ), "Logic failed and open_router provider attempted to be prepended to model name! in stagehand/_config.py" model_name = f"{model_metadata.provider}/{model_name}" - logger.error(f"Model name: {model_name}") + logger.debug(f"Model name: {model_name}") return model_name @property