Fix review comments

Author: martinmoldrup

toolit/auto_loader.py

@@ -56,7 +56,7 @@ def load_tools_from_plugins() -> list[FunctionType]:
     """Discover and return plugin commands via entry points."""
     plugins = get_plugin_tools()
     for plugin in plugins:
-        register_command(plugin, rich_help_panel=RichHelpPanelNames.NAME_OF_THE_RICH_GROUP_HELP_PANEL_PLUGINS)
+        register_command(plugin, rich_help_panel=RichHelpPanelNames.PLUGINS_COMMANDS_PANEL)
     return plugins
 
 
@@ -83,7 +83,7 @@ def load_tools_from_folder(folder_path: pathlib.Path) -> list[FunctionType]:
     tool_groups: list[FunctionType] = get_items_from_folder(folder_path, tool_group_strategy)
     # Register each tool as a command
     for tool in tools:
-        register_command(tool, rich_help_panel=RichHelpPanelNames.NAME_OF_THE_RICH_GROUP_HELP_PANEL_PROJECT)
+        register_command(tool, rich_help_panel=RichHelpPanelNames.PROJECT_COMMANDS_PANEL)
     return tools + tool_groups
 
 

toolit/cli.py

@@ -7,7 +7,7 @@
 
 load_tools_from_folder(load_devtools_folder())
 load_tools_from_plugins()
-register_command(create_vscode_tasks_json, rich_help_panel=RichHelpPanelNames.NAME_OF_THE_RICH_GROUP_HELP_PANEL_PLUGINS)
+register_command(create_vscode_tasks_json, rich_help_panel=RichHelpPanelNames.PLUGINS_COMMANDS_PANEL)
 
 
 if __name__ == "__main__":

toolit/constants.py

@@ -8,8 +8,8 @@
 class RichHelpPanelNames:
     """Namespace for the different rich help panel names."""
 
-    NAME_OF_THE_RICH_GROUP_HELP_PANEL_PROJECT = "Commands from Project"
-    NAME_OF_THE_RICH_GROUP_HELP_PANEL_PLUGINS = "Commands from Plugins"
+    PROJECT_COMMANDS_PANEL = "Commands from Project"
+    PLUGINS_COMMANDS_PANEL = "Commands from Plugins"
 
 
 class ConfigFileKeys:

toolit/create_apps_and_register.py

@@ -29,7 +29,11 @@ def initialize() -> None:
     """Welcome to the Toolit CLI."""
 
 
-def register_command(command_func: Callable[..., Any], name: str | None = None, rich_help_panel: str|None = None) -> None:
+def register_command(
+    command_func: Callable[..., Any],
+    name: str | None = None,
+    rich_help_panel: str | None = None,
+) -> None:
     """Register an external command to the CLI and MCP server if available."""
     if not callable(command_func):
         msg = f"Command function {command_func} is not callable."