refactor: Simplify helper function and update schema description per review feedback

Co-Authored-By: unknown <>

Author: devin-ai-integration[bot]

airbyte_cdk/legacy/sources/declarative/manifest_declarative_source.py

@@ -350,13 +350,9 @@ def _initialize_cache_for_parent_streams(
         """
         parent_streams = set()
 
-        def _should_enable_cache(requester: Dict[str, Any]) -> bool:
-            """Return False only if use_cache is explicitly set to False."""
-            return requester.get("use_cache") is not False
-
         def _set_cache_if_not_disabled(requester: Dict[str, Any]) -> None:
             """Set use_cache to True only if not explicitly disabled."""
-            if _should_enable_cache(requester):
+            if requester.get("use_cache") is not False:
                 requester["use_cache"] = True
 
         def update_with_cache_parent_configs(

airbyte_cdk/sources/declarative/concurrent_declarative_source.py

@@ -433,13 +433,9 @@ def _initialize_cache_for_parent_streams(
         """
         parent_streams = set()
 
-        def _should_enable_cache(requester: Dict[str, Any]) -> bool:
-            """Return False only if use_cache is explicitly set to False."""
-            return requester.get("use_cache") is not False
-
         def _set_cache_if_not_disabled(requester: Dict[str, Any]) -> None:
             """Set use_cache to True only if not explicitly disabled."""
-            if _should_enable_cache(requester):
+            if requester.get("use_cache") is not False:
                 requester["use_cache"] = True
 
         def update_with_cache_parent_configs(

airbyte_cdk/sources/declarative/declarative_component_schema.yaml

@@ -2286,7 +2286,7 @@ definitions:
           - "$ref": "#/definitions/CustomErrorHandler"
       use_cache:
         title: Use Cache
-        description: Enables stream requests caching. This field is automatically set by the CDK.
+        description: Enables stream requests caching. When set to true, repeated requests to the same URL will return cached responses. Parent streams automatically have caching enabled. Only set this to false if you are certain that caching should be disabled, as it may negatively impact performance when the same data is needed multiple times (e.g., for scroll-based pagination APIs where caching causes duplicate records).
         type: boolean
         default: false
       $parameters: