fix(proxy): remove ghost team BYOK models from /models on delete

[aanchal22]

Relevant issues

Fixes #22594

Related PRs: #22595 (dibyom, stale), #28833 (oss-agent-shin, has blockers), #29526 (companion create-side fix)

Pre-Submission checklist

• I have added meaningful tests
• My PR passes all unit tests on make test-unit
• My PR's scope is as isolated as possible; it only solves 1 specific problem
• I have requested a Greptile review by commenting @greptileai and received a Confidence Score of at least 4/5 before requesting a maintainer review

Screenshots / Proof of Fix

Reproduction script creates a team with 2 BYOK models sequentially, then deletes them. Before the fix, both models persist as ghosts in team.models and /models indefinitely. After the fix, deleted models are removed immediately.

Before (v1.85.0):
  After deleting both models:
  team.models in DB: ['all-proxy-models', 'bug2-model-1', 'bug2-model-2']
  Ghosts: both models still visible via /models

After (this PR):
  After deleting model-2:
  team.models in DB: ['all-proxy-models', 'bug2-model-1']
  After deleting model-1:
  team.models in DB: ['all-proxy-models']
  No ghosts.

Also tested the alias-overwrite scenario: create model-1, create model-2 (overwrites model-1's alias entry), delete model-1. Before the fix, model-1 persists as a ghost. After the fix, model-1 is cleaned up and model-2 is preserved.

Type

Bug Fix

Changes

delete_model passed model_params.model_name (the internal name, e.g. model_name_{team_id}_{uuid}) to delete_team_model_alias, which searched alias map values for a match. For BYOK models the alias map is typically empty or was overwritten by a later model create, so the lookup always failed and the public name was never removed from team.models.

This PR adds two helpers and rewires the delete path:

• _resolve_team_public_model_name: extracts the user-visible name from model_info.team_public_model_name, falling back to model_params.model_name for non-BYOK models.
• _cleanup_team_model_references: builds a names_to_remove set starting with the public name (unconditionally), then adds any alias keys whose values match the internal name. Strips all matched names from team.models and cleans up the alias map in the same pass.

After the DB cleanup, calls _refresh_cached_team (already exists on main) so /models reflects the deletion without waiting for cache TTL. The cache refresh is wrapped in try/except so a Redis failure does not undo the DB delete.

delete_team_model_alias is left in place; other code paths may still reference it.

[greptile-apps]

Too many files changed for review. (1166 files found, 100 file limit)