open-code-review

mirror of https://github.com/alibaba/open-code-review.git synced 2026-07-31 03:54:19 +00:00

History

Qiaochu Hu a13b27e5c9 fix(config): preserve hand-edited timeout_sec across config round-trips (#452 ) The LLM resolver honors providers.<name>.timeout_sec and llm.timeout_sec (and the docs tell users to hand-edit them into config.json), but the cmd-side ProviderEntry/LlmConfig structs lacked the field. Every loadOrCreateConfig + saveConfig cycle — any `ocr config set`, `ocr config model`, or interactive provider setup — rewrites the whole file from those structs, silently dropping the key and reverting the per-request timeout to the 300s default. Add the field to both structs so the value survives round-trips. This intentionally does not add `ocr config set` write support for the key, matching the documented behavior.	2026-07-22 19:12:54 +08:00
..
opencodereview	fix(config): preserve hand-edited timeout_sec across config round-trips (#452 )	2026-07-22 19:12:54 +08:00

fix(config): preserve hand-edited timeout_sec across config round-trips (#452 )

The LLM resolver honors providers.<name>.timeout_sec and llm.timeout_sec
(and the docs tell users to hand-edit them into config.json), but the
cmd-side ProviderEntry/LlmConfig structs lacked the field. Every
loadOrCreateConfig + saveConfig cycle — any `ocr config set`,
`ocr config model`, or interactive provider setup — rewrites the whole
file from those structs, silently dropping the key and reverting the
per-request timeout to the 300s default.

Add the field to both structs so the value survives round-trips. This
intentionally does not add `ocr config set` write support for the key,
matching the documented behavior.

2026-07-22 19:12:54 +08:00

opencodereview

fix(config): preserve hand-edited timeout_sec across config round-trips (#452 )

2026-07-22 19:12:54 +08:00