diff --git a/src/crewai/knowledge/source/base_file_knowledge_source.py b/src/crewai/knowledge/source/base_file_knowledge_source.py index e086cbf65..3318b62cc 100644 --- a/src/crewai/knowledge/source/base_file_knowledge_source.py +++ b/src/crewai/knowledge/source/base_file_knowledge_source.py @@ -71,28 +71,24 @@ class BaseFileKnowledgeSource(BaseKnowledgeSource, ABC): def _process_file_paths(self) -> List[Path]: """Convert file_path to a list of Path objects.""" - # Check if old file_path is being used + paths = None if hasattr(self, "file_path") and self.file_path is not None: self._logger.log( "warning", "The 'file_path' attribute is deprecated and will be removed in a future version. Please use 'file_paths' instead.", color="yellow", ) - paths = ( - [self.file_path] - if isinstance(self.file_path, (str, Path)) - else self.file_path - ) + paths = self.file_path else: if self.file_paths is None: raise ValueError("Your source must be provided with a file_paths: []") - elif isinstance(self.file_paths, list) and len(self.file_paths) == 0: - raise ValueError("Empty file_paths are not allowed") - else: - paths = ( - [self.file_paths] - if isinstance(self.file_paths, (str, Path)) - else self.file_paths - ) + paths = self.file_paths + + if isinstance(paths, (str, Path)): + paths = [paths] + elif not isinstance(paths, list): + raise ValueError( + "file_path/file_paths must be a Path, str, or a list of these types" + ) return [self.convert_to_path(path) for path in paths]