feat: comprehensive marketplace management and CLI improvements

- Add --all option to marketplace install command for bulk installation
- Add new uninstall command for marketplaces with --all support
- Fix inconsistent output formatting in marketplace list commands across all app handlers
- Standardize marketplace list output to use numbered lists with ✓ indicators
- Fix import errors in server_config tests by adding missing wrapper functions
- Add new prompt management CLI commands (prompt_crud, prompt_defaults, etc.)
- Refactor and improve codebase health across multiple modules
- Enhance error handling and user feedback in CLI commands

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: zhujian0805

code_assistant_manager/agents/base.py

@@ -40,13 +40,11 @@ def __init__(
     @abstractmethod
     def app_name(self) -> str:
         """Return the name of the app (e.g., 'claude')."""
-        pass
 
     @property
     @abstractmethod
     def _default_agents_dir(self) -> Path:
         """Return the default agents directory for this app."""
-        pass
 
     @property
     def agents_dir(self) -> Path:

code_assistant_manager/cli/agents_commands.py

@@ -1,19 +1,56 @@
 """CLI commands for agent management."""
 
 import logging
-from pathlib import Path
 from typing import Optional
 
 import typer
 
-from code_assistant_manager.agents import (
-    VALID_APP_TYPES,
-    Agent,
-    AgentManager,
-    AgentRepo,
-)
+from code_assistant_manager.agents import VALID_APP_TYPES, AgentManager, AgentRepo
 from code_assistant_manager.cli.option_utils import resolve_app_targets
 from code_assistant_manager.menu.base import Colors
+
+# Module-level variables for typer defaults to avoid B008 violations
+URL_ARGUMENT = typer.Argument(
+    None,
+    help="GitHub URL or owner/repo to fetch and save (e.g., https://github.com/owner/repo or owner/repo)",
+)
+SAVE_OPTION = typer.Option(
+    False,
+    "--save",
+    "-s",
+    help="Save the fetched repo to config (only used with URL argument)",
+)
+AGENTS_PATH_OPTION = typer.Option(
+    None,
+    "--agents-path",
+    help="Agents subdirectory path in the repo (only used with URL argument)",
+)
+AGENT_KEY_ARGUMENT = typer.Argument(..., help="Agent identifier")
+APP_TYPE_OPTION = typer.Option(
+    "claude",
+    "--app",
+    "-a",
+    help="App type(s) to install to (claude, codex, gemini, droid, codebuddy, all). Comma-separated.",
+)
+FORCE_OPTION = typer.Option(False, "--force", "-f", help="Skip confirmation")
+OWNER_OPTION = typer.Option(..., "--owner", "-o", help="Repository owner")
+NAME_OPTION = typer.Option(..., "--name", "-n", help="Repository name")
+BRANCH_OPTION = typer.Option("main", "--branch", "-b", help="Repository branch")
+AGENTS_PATH_OPTION_REPO = typer.Option(
+    None, "--agents-path", help="Agents subdirectory path"
+)
+APP_TYPE_OPTION_ALL = typer.Option(
+    None,
+    "--app",
+    "-a",
+    help="App type(s) to show (claude, codex, gemini, droid, codebuddy, all). Default shows all.",
+)
+APP_TYPE_OPTION_UNINSTALL = typer.Option(
+    ...,
+    "--app",
+    "-a",
+    help="App type(s) to uninstall all agents from (claude, codex, gemini, droid, codebuddy, all). Comma-separated.",
+)
 from code_assistant_manager.plugins.fetch import parse_github_url
 
 logger = logging.getLogger(__name__)
@@ -67,21 +104,9 @@ def list_agents():
 
 @agent_app.command("fetch")
 def fetch_agents(
-    url: Optional[str] = typer.Argument(
-        None,
-        help="GitHub URL or owner/repo to fetch and save (e.g., https://github.com/owner/repo or owner/repo)",
-    ),
-    save: bool = typer.Option(
-        False,
-        "--save",
-        "-s",
-        help="Save the fetched repo to config (only used with URL argument)",
-    ),
-    agents_path: Optional[str] = typer.Option(
-        None,
-        "--agents-path",
-        help="Agents subdirectory path in the repo (only used with URL argument)",
-    ),
+    url: Optional[str] = URL_ARGUMENT,
+    save: bool = SAVE_OPTION,
+    agents_path: Optional[str] = AGENTS_PATH_OPTION,
 ):
     """Fetch agents from configured repositories or a specific GitHub URL.
 
@@ -227,13 +252,8 @@ def view_agent(agent_key: str):
 
 @agent_app.command("install")
 def install_agent(
-    agent_key: str = typer.Argument(..., help="Agent identifier"),
-    app_type: str = typer.Option(
-        "claude",
-        "--app",
-        "-a",
-        help="App type(s) to install to (claude, codex, gemini, droid, codebuddy, all). Comma-separated.",
-    ),
+    agent_key: str = AGENT_KEY_ARGUMENT,
+    app_type: str = APP_TYPE_OPTION,
 ):
     """Install an agent to one or more app's agents directories."""
     target_apps = resolve_app_targets(app_type, VALID_APP_TYPES, default="claude")
@@ -255,14 +275,9 @@ def install_agent(
 
 @agent_app.command("uninstall")
 def uninstall_agent(
-    agent_key: str = typer.Argument(..., help="Agent identifier"),
-    app_type: str = typer.Option(
-        "claude",
-        "--app",
-        "-a",
-        help="App type(s) to uninstall from (claude, codex, gemini, droid, codebuddy, all). Comma-separated.",
-    ),
-    force: bool = typer.Option(False, "--force", "-f", help="Skip confirmation"),
+    agent_key: str = AGENT_KEY_ARGUMENT,
+    app_type: str = APP_TYPE_OPTION,
+    force: bool = FORCE_OPTION,
 ):
     """Uninstall an agent from one or more app's agents directories."""
     target_apps = resolve_app_targets(app_type, VALID_APP_TYPES, default="claude")
@@ -316,12 +331,10 @@ def list_repos():
 
 @agent_app.command("add-repo")
 def add_repo(
-    owner: str = typer.Option(..., "--owner", "-o", help="Repository owner"),
-    name: str = typer.Option(..., "--name", "-n", help="Repository name"),
-    branch: str = typer.Option("main", "--branch", "-b", help="Repository branch"),
-    agents_path: Optional[str] = typer.Option(
-        None, "--agents-path", help="Agents subdirectory path"
-    ),
+    owner: str = OWNER_OPTION,
+    name: str = NAME_OPTION,
+    branch: str = BRANCH_OPTION,
+    agents_path: Optional[str] = AGENTS_PATH_OPTION_REPO,
 ):
     """Add an agent repository."""
     manager = _get_agent_manager()
@@ -343,9 +356,9 @@ def add_repo(
 
 @agent_app.command("remove-repo")
 def remove_repo(
-    owner: str = typer.Option(..., "--owner", "-o", help="Repository owner"),
-    name: str = typer.Option(..., "--name", "-n", help="Repository name"),
-    force: bool = typer.Option(False, "--force", "-f", help="Skip confirmation"),
+    owner: str = OWNER_OPTION,
+    name: str = NAME_OPTION,
+    force: bool = FORCE_OPTION,
 ):
     """Remove an agent repository."""
     manager = _get_agent_manager()
@@ -363,12 +376,7 @@ def remove_repo(
 
 @agent_app.command("installed")
 def list_installed_agents(
-    app_type: Optional[str] = typer.Option(
-        None,
-        "--app",
-        "-a",
-        help="App type(s) to show (claude, codex, gemini, droid, codebuddy, all). Default shows all.",
-    ),
+    app_type: Optional[str] = APP_TYPE_OPTION_ALL,
 ):
     """Show installed agents for each app."""
     manager = _get_agent_manager()
@@ -420,13 +428,8 @@ def list_installed_agents(
 
 @agent_app.command("uninstall-all")
 def uninstall_all_agents(
-    app_type: str = typer.Option(
-        ...,
-        "--app",
-        "-a",
-        help="App type(s) to uninstall all agents from (claude, codex, gemini, droid, codebuddy, all). Comma-separated.",
-    ),
-    force: bool = typer.Option(False, "--force", "-f", help="Skip confirmation"),
+    app_type: str = APP_TYPE_OPTION_UNINSTALL,
+    force: bool = FORCE_OPTION,
 ):
     """Uninstall all agents from one or more apps."""
     target_apps = resolve_app_targets(app_type, VALID_APP_TYPES)

code_assistant_manager/cli/base_commands.py

@@ -6,7 +6,7 @@
 
 import logging
 from abc import ABC, abstractmethod
-from typing import Any, Dict, List, Optional, Tuple, Union
+from typing import Any, List, Optional, Tuple
 
 import typer
 
@@ -84,7 +84,6 @@ def confirm_action(self, message: str, default: bool = False) -> bool:
     @abstractmethod
     def execute(self, *args, **kwargs) -> int:
         """Execute the command. Must be implemented by subclasses."""
-        pass
 
 
 class AppAwareCommand(BaseCommand):

code_assistant_manager/cli/plugins/plugin_discovery_commands.py

@@ -12,7 +12,6 @@
 from code_assistant_manager.plugins import (
     VALID_APP_TYPES,
     PluginManager,
-    fetch_repo_info_from_url,
     get_handler,
     parse_github_url,
 )

code_assistant_manager/cli/plugins/plugin_helpers.py

@@ -5,7 +5,7 @@
 
 import logging
 
-from code_assistant_manager.plugins import VALID_APP_TYPES, get_handler
+from code_assistant_manager.plugins import get_handler
 
 logger = logging.getLogger(__name__)
 

code_assistant_manager/cli/plugins/plugin_install_commands.py

@@ -4,23 +4,13 @@
 """
 
 import logging
-from pathlib import Path
 from typing import Optional
 
 import typer
 
 from code_assistant_manager.cli.option_utils import resolve_single_app
 from code_assistant_manager.menu.base import Colors
-from code_assistant_manager.plugins import (
-    BUILTIN_PLUGIN_REPOS,
-    PLUGIN_HANDLERS,
-    VALID_APP_TYPES,
-    PluginManager,
-    PluginRepo,
-    fetch_repo_info_from_url,
-    get_handler,
-    parse_github_url,
-)
+from code_assistant_manager.plugins import VALID_APP_TYPES, get_handler
 from code_assistant_manager.plugins.base import BasePluginHandler
 
 logger = logging.getLogger(__name__)