fix: keep --telemetry as a hidden no-op so existing start commands still parse

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
mateo 2026-09-20 02:36:59 +00:00
parent f820472488
commit 8e530cf819
2 changed files with 18 additions and 0 deletions

View file

@ -754,6 +754,14 @@ class ProxyInitializationHelpers:
type=float,
help="Set max budget for API calls - works for hosted models like OpenAI, TogetherAI, Anthropic, etc.`",
)
@click.option(
"--telemetry",
default=None,
type=bool,
hidden=True,
expose_value=False,
help="Deprecated no-op kept so existing start commands still parse",
)
@click.option(
"--log_config",
default=None,

View file

@ -617,6 +617,16 @@ class TestProxyInitializationHelpers:
assert "Skipping server startup" in result.output
mock_uvicorn_run.assert_not_called()
# --- legacy --telemetry flag is accepted, ignored and hidden ---
result = runner.invoke(
run_server, ["--local", "--skip_server_startup", "--telemetry", "False"]
)
assert (
result.exit_code == 0
), f"exit_code={result.exit_code}, output={result.output}"
assert "Skipping server startup" in result.output
assert "telemetry" not in runner.invoke(run_server, ["--help"]).output
# --- normal startup ---
mock_uvicorn_run.reset_mock()