lsm / HyperNeo / 30711462055

83%

Build Type

push

github

Committed by web-flow

Commit Message

fix: infer provider from model for long-horizon agent sessions (#2264)

* fix: infer provider from model for long-horizon agent sessions

A LH/worker agent whose settings set `model` but left `provider` null ran
fine (provider inferred at query time) but was hard-blocked from switching
models with "Session has no provider configured" — even to the same model.

buildLongHorizonAgentSessionConfig and the regular long-term-agent branch
now infer the provider from the model (mirroring custom-agent.ts) when
agent.provider is unset. model-switch-handler additionally infers the
previous provider from the stored model instead of throwing, so stranded
sessions can switch immediately instead of waiting for a wake/refresh.

* test: add provider-inference regression tests for LH agent model switching

Covers buildLongHorizonAgentSessionConfig inference (kimi + non-kimi +
explicit-provider-wins + default fallback), the regular worker-agent
branch, refreshLongHorizonAgentSessionConfig self-heal (undefined → kimi),
and the model-switch-handler guard inferring the provider from the stored
model — including the kimi → kimi same-model case from the bug report.

* test: clear Claude Code session env vars in unit test setup

provider-service.ts snapshots ANTHROPIC_MODEL, ENABLE_TOOL_SEARCH, and
eight sibling behavior vars into module-level "user configured" constants
at import time, and applyEnvVarsToProcess preserves any process.env value
matching that snapshot. When unit tests run from inside a Claude Code
session (which exports these vars), the snapshot captures the session's
values and the env-clearing assertions in provider-service.test.ts fail —
e.g. ENABLE_TOOL_SEARCH stays 'false' instead of being deleted.

Delete these vars in tests/unit/setup.ts so the module snapshot is
undefined and unit tests are hermetic to the runner's environment,
matching what CI's clean runners already see.

* fix: pre-route bare glm/minimax aliases in inferProviderForModel

Review feed... (continued)

Coverage Stats

11687 of 16768 branches covered (69.7%)

Branch coverage included in aggregate %.

123 of 126 new or added lines in 9 files covered. (97.62%)

3 existing lines in 1 file now uncovered.

100283 of 118655 relevant lines covered (84.52%)

302.58 hits per line