From 22ae9ef78e11ceb90c179f3b6435162269fb8511 Mon Sep 17 00:00:00 2001 From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Date: Fri, 30 Jan 2026 20:14:03 +0000 Subject: [PATCH] fix: Address PR review comments MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Change Crew.verbose field type from bool to bool | None to allow env var check - Remove unused imports (StringIO, pytest) from test file Co-Authored-By: João --- lib/crewai/src/crewai/crew.py | 8 +++++++- lib/crewai/tests/test_verbose_control.py | 3 --- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/lib/crewai/src/crewai/crew.py b/lib/crewai/src/crewai/crew.py index 02900834d..56bd7a679 100644 --- a/lib/crewai/src/crewai/crew.py +++ b/lib/crewai/src/crewai/crew.py @@ -193,7 +193,13 @@ class Crew(FlowTrackable, BaseModel): tasks: list[Task] = Field(default_factory=list) agents: list[BaseAgent] = Field(default_factory=list) process: Process = Field(default=Process.sequential) - verbose: bool = Field(default=False) + verbose: bool | None = Field( + default=None, + description=( + "Whether to enable verbose logging output. True=always enable, " + "False=always disable, None=check CREWAI_VERBOSE env var (defaults to True if not set)." + ), + ) memory: bool = Field( default=False, description="If crew should use memory to store memories of it's execution", diff --git a/lib/crewai/tests/test_verbose_control.py b/lib/crewai/tests/test_verbose_control.py index 017339be8..d2b899403 100644 --- a/lib/crewai/tests/test_verbose_control.py +++ b/lib/crewai/tests/test_verbose_control.py @@ -1,11 +1,8 @@ """Test verbose control for Flow and Crew.""" import os -from io import StringIO from unittest.mock import patch -import pytest - from crewai.events.event_listener import EventListener from crewai.flow.flow import Flow, start, listen from crewai.utilities.logger_utils import should_enable_verbose