Function advisor_command_call

src/command_system/builtins.py:681–990

Handle /advisor — configure the reviewer model. Required argument shape: `` : `` — both halves explicit, separated by the first colon (so model strings containing ``/`` or further ``:`` are preserved verbatim). Provider must match a key in ``~/.clawcodex/config.json``

(args: str, context: CommandContext)

def advisor_command_call(args: str, context: CommandContext) -> LocalCommandResult:
    """Handle /advisor — configure the reviewer model.

    Required argument shape: ``<provider>:<model>`` — both halves
    explicit, separated by the first colon (so model strings
    containing ``/`` or further ``:`` are preserved verbatim).
    Provider must match a key in ``~/.clawcodex/config.json``&#x27;s
    ``providers`` map; clawcodex is multi-provider and the same
    model name (e.g. ``claude-opus-4-7``) can sit behind anthropic,
    openai (litellm), openrouter, bedrock, etc. Name-based inference
    was ambiguous and silently routed to the wrong endpoint.

    Branches (after parsing optional ``--client`` / ``--no-client``):
      * no args, no flags → status report (provider/model + mode).
      * ``unset`` | ``off`` → clear advisor_model, advisor_provider,
        and advisor_client_mode.
      * ``--no-client`` alone → keep model+provider, clear client-mode.
      * ``--client`` alone → keep model+provider, set client-mode.
      * ``<provider>:<model>`` → validate provider exists in config,
        persist both fields together. ``--client`` flag (if present)
        also persists advisor_client_mode.

    Examples:
      * ``/advisor anthropic:claude-opus-4-7``  (direct Anthropic API)
      * ``/advisor openai:claude-opus-4-7``     (litellm/proxy via openai provider)
      * ``/advisor openrouter:anthropic/claude-opus-4.1``
      * ``/advisor gemini:gemini-2.5-pro``
    """
    from ..models.model import canonical_model_name, resolve_model
    from ..models.validation import validate_model_name
    from ..utils.advisor import (
        ADVISOR_MODE_CLIENT_SIDE,
        ADVISOR_MODE_INACTIVE,
        ADVISOR_MODE_SERVER_SIDE,
        can_user_configure_advisor,
        decide_advisor_mode,
    )

    provider = getattr(context, "provider", None)

    # Hard-reject only when env-disabled (the user would silently
    # configure a value that no request can use).
    if not can_user_configure_advisor(provider):
        return LocalCommandResult(
            type="text",
            value=(
                "Advisor is disabled by the CLAUDE_CODE_DISABLE_ADVISOR_TOOL "
                "env var."
            ),
        )

    # Tokenize raw args so flag handling is order-insensitive. A
    # trailing or leading ``--client`` / ``--no-client`` should peel
    # off cleanly without breaking the model identifier.
    raw_tokens = (args or "").strip().split()
    force_client_flag: bool | None = None  # None = no flag passed
    rest_tokens: list[str] = []
    for tok in raw_tokens: