PR #14919: fix(model): only strip /v1 from OpenCode URLs on fresh provider resolution

• Repository: NousResearch/hermes-agent
• Author: aniruddhaadak80
• State: closed | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/14919

Description (problem / solution / changelog)

Summary

Fixes #14879

The /model command unconditionally strips /v1 from OpenCode base URLs when api_mode == "anthropic_messages", but this breaks models that need /v1 in the URL when the user switches models on the same provider (where the URL was already correct from a prior resolution).

Root Cause

In switch_model(), the /v1 stripping regex runs whenever:

1. api_mode == "anthropic_messages"
2. Provider is opencode-zen or opencode-go

But it didn't check whether the URL was freshly resolved or was carried over from the current session. When a user on opencode-go switches from one model to another (same provider), the base_url is already in the correct form and stripping /v1 corrupts it.

Changes

hermes_cli/model_switch.py — switch_model():

• Added (provider_changed or explicit_provider) to the /v1 strip condition
• /v1 is now only stripped when the URL was freshly resolved via resolve_runtime_provider() (i.e., the provider changed or was explicitly specified)
• When switching models on the same provider, the base_url is left as-is
• Updated the comment block to document this behavior

Testing

• Provider switch (e.g., openrouter → opencode-go): provider_changed=True → /v1 stripped as before
• Same-provider model switch (e.g., model A → model B on opencode-go): provider_changed=False, explicit_provider="" → /v1 preserved (bug fixed)
• Explicit provider (e.g., /model sonnet --provider opencode-go): explicit_provider="opencode-go" → /v1 stripped correctly
• Non-OpenCode providers: condition short-circuits at target_provider check → no change

Checklist

• Follows Conventional Commits format
• Single logical change
• No breaking changes
• Existing tests unaffected

Changed files

• hermes_cli/model_switch.py (modified, +6/-0)