feat(client): ovcli.conf 支持 timeout 配置 + 修复 Rust CLI agent_id#308
Merged
feat(client): ovcli.conf 支持 timeout 配置 + 修复 Rust CLI agent_id#308
Conversation
…lignment - Python HTTP client reads `timeout` from ovcli.conf when using default value, with priority: SDK explicit param > ovcli.conf > default 60.0 - Rust CLI: replace unused `user` field with `agent_id`, send X-OpenViking-Agent header to align with Python client behavior - Rust CLI: read `timeout` from ovcli.conf and pass to reqwest client - Move timeout documentation from configuration guide to API overview - Update examples and ovcli.conf.example with timeout field Closes #306 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
|
zhoujh01
approved these changes
Feb 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
为 Python 和 Rust CLI 客户端添加通过
ovcli.conf配置 HTTP 超时的支持,同时修复 Rust CLI 未正确读取agent_id的问题。Related Issue
Closes #306
Type of Change
Changes Made
openviking_cli/client/http.py): 从ovcli.conf读取timeout, 优先级: SDK 显式传参 > ovcli.conf > 默认值 60.0openviking_cli/client/sync_http.py): 透传timeout参数到AsyncHTTPClientcrates/ov_cli/src/config.rs): 将未使用的user字段替换为agent_id, 新增timeout字段 (serde 默认 60.0)crates/ov_cli/src/client.rs): 接收agent_id和timeout_secs参数, 发送X-OpenViking-Agent请求头, 与 Python 客户端对齐crates/ov_cli/src/main.rs): 将配置中的agent_id和timeout传递给HttpClientovcli.conf.example加入timeout字段, 更新代码示例Testing
Checklist
Additional Notes
Rust CLI 此前有一个从未使用的
user字段, 且不支持agent_id和可配置超时。本 PR 将 Rust CLI 与 Python 客户端对齐:ovcli.conf读取agent_id并作为X-OpenViking-Agent请求头发送ovcli.conf读取timeout并应用到 reqwest HTTP 客户端