aboutsummaryrefslogtreecommitdiff
path: root/autogpts/autogpt/autogpt/plugins/plugins_config.py
diff options
context:
space:
mode:
Diffstat (limited to 'autogpts/autogpt/autogpt/plugins/plugins_config.py')
-rw-r--r--autogpts/autogpt/autogpt/plugins/plugins_config.py18
1 files changed, 11 insertions, 7 deletions
diff --git a/autogpts/autogpt/autogpt/plugins/plugins_config.py b/autogpts/autogpt/autogpt/plugins/plugins_config.py
index cc255035a..0494b6e38 100644
--- a/autogpts/autogpt/autogpt/plugins/plugins_config.py
+++ b/autogpts/autogpt/autogpt/plugins/plugins_config.py
@@ -42,16 +42,17 @@ class PluginsConfig(BaseModel):
plugins_denylist,
plugins_allowlist,
)
- if type(config_data) != dict:
+ if type(config_data) is not dict:
logger.error(
- f"Expected plugins config to be a dict, got {type(config_data)}, continuing without plugins"
+ f"Expected plugins config to be a dict, got {type(config_data)}."
+ " Continuing without plugins."
)
return empty_config
return cls(plugins=config_data)
except BaseException as e:
logger.error(
- f"Plugin config is invalid, continuing without plugins. Error: {e}"
+ f"Plugin config is invalid. Continuing without plugins. Error: {e}"
)
return empty_config
@@ -63,7 +64,7 @@ class PluginsConfig(BaseModel):
plugins_allowlist: list[str],
) -> dict[str, PluginConfig]:
if not plugins_config_file.is_file():
- logger.warn("plugins_config.yaml does not exist, creating base config.")
+ logger.warning("plugins_config.yaml does not exist, creating base config.")
cls.create_empty_plugins_config(
plugins_config_file,
plugins_denylist,
@@ -75,13 +76,13 @@ class PluginsConfig(BaseModel):
plugins = {}
for name, plugin in plugins_config.items():
- if type(plugin) == dict:
+ if type(plugin) is dict:
plugins[name] = PluginConfig(
name=name,
enabled=plugin.get("enabled", False),
config=plugin.get("config", {}),
)
- elif type(plugin) == PluginConfig:
+ elif isinstance(plugin, PluginConfig):
plugins[name] = plugin
else:
raise ValueError(f"Invalid plugin config data type: {type(plugin)}")
@@ -93,7 +94,10 @@ class PluginsConfig(BaseModel):
plugins_denylist: list[str],
plugins_allowlist: list[str],
):
- """Create an empty plugins_config.yaml file. Fill it with values from old env variables."""
+ """
+ Create an empty plugins_config.yaml file.
+ Fill it with values from old env variables.
+ """
base_config = {}
logger.debug(f"Legacy plugin denylist: {plugins_denylist}")