Skip to content

Merge local changes: add local provider continuous monitoring, fix Cl…#359

Open
jackyxiwang wants to merge 2 commits intoRightNow-AI:mainfrom
jackyxiwang:feature/merge-local-changes
Open

Merge local changes: add local provider continuous monitoring, fix Cl…#359
jackyxiwang wants to merge 2 commits intoRightNow-AI:mainfrom
jackyxiwang:feature/merge-local-changes

Conversation

@jackyxiwang
Copy link

…awHub timeout, and update config

Summary

Changes

  • 合并本地修改 :

  • 实现了本地提供者的持续监控功能,使用配置文件中指定的 check_interval_secs 间隔

  • 修复了 ClawHub 超时问题,将超时时间从 30 秒增加到 120 秒

  • 更新了 Marketplace 客户端的超时设置为 120 秒

  • 在 KernelConfig 结构体中添加了 local_providers 字段

  • 修复了所有编译错误

  • 代码修改详情 :

  • crates/openfang-kernel/src/kernel.rs :添加了本地提供者持续监控的逻辑

  • crates/openfang-skills/src/clawhub.rs :增加了 ClawHub 客户端超时

  • crates/openfang-skills/src/marketplace.rs :添加了 Marketplace 客户端超时

  • crates/openfang-types/src/config.rs :添加了 LocalProvidersConfig 结构体和相关字段

  • 构建和提交 :

  • 成功构建了代码,确保所有修改都能正常编译

  • 将修改提交到了本地仓库

  • 将修改推送到了远程仓库的 feature/merge-local-changes 分支

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (if applicable)

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant