mirror of
https://github.com/langbot-app/LangBot.git
synced 2026-06-14 01:36:03 +00:00
refactor(agent-runner): migrate deerflow and weknora to plugins
This commit is contained in:
@@ -18,14 +18,7 @@
|
||||
- **测试方式**: 需要 mock HTTP 响应或使用 fake LLM server
|
||||
- **状态**: 后续可补充 mock HTTP 测试
|
||||
|
||||
### 3. Agent Runner (`provider/runners/`)
|
||||
- **路径**: `src/langbot/pkg/provider/runners/`
|
||||
- **模块**: cozeapi, difysvapi, n8nsvapi, langflowapi, dashscopeapi, localagent, tboxapi
|
||||
- **排除原因**: 需要真实 Agent 平台(Coze、Dify、n8n 等)的 API 连接
|
||||
- **测试方式**: 需要 mock Agent 平台响应
|
||||
- **状态**: 后续可补充 mock 测试
|
||||
|
||||
### 4. 向量数据库 (`vector/vdbs/`)
|
||||
### 3. 向量数据库 (`vector/vdbs/`)
|
||||
- **路径**: `src/langbot/pkg/vector/vdbs/`
|
||||
- **模块**: chroma, milvus, pgvector, qdrant, seekdb
|
||||
- **排除原因**: 需要真实向量数据库实例运行
|
||||
@@ -42,7 +35,7 @@
|
||||
# 排除外部适配器后计算覆盖率
|
||||
pytest tests/unit_tests/ --cov=langbot.pkg \
|
||||
--cov-fail-under=0 \
|
||||
-o "cov_exclude_patterns=platform/sources/*,provider/modelmgr/requesters/*,provider/runners/*,vector/vdbs/*"
|
||||
-o "cov_exclude_patterns=platform/sources/*,provider/modelmgr/requesters/*,vector/vdbs/*"
|
||||
```
|
||||
|
||||
### 当前覆盖率(排除后)
|
||||
@@ -77,15 +70,11 @@ pytest tests/unit_tests/ --cov=langbot.pkg \
|
||||
- 使用 `httpx` mock 测试 API 响应解析
|
||||
- 测试重试逻辑、错误处理
|
||||
|
||||
2. **`provider/runners/`** (优先级:中)
|
||||
- Mock Agent 平台响应
|
||||
- 测试 session 管理、错误处理
|
||||
|
||||
3. **`platform/sources/`** (优先级:低)
|
||||
2. **`platform/sources/`** (优先级:低)
|
||||
- Mock 平台 webhook 事件
|
||||
- 测试消息解析、事件处理
|
||||
|
||||
4. **`vector/vdbs/`** (优先级:低)
|
||||
3. **`vector/vdbs/`** (优先级:低)
|
||||
- Mock 向量数据库操作
|
||||
- 测试 CRUD、查询逻辑
|
||||
|
||||
@@ -176,4 +165,4 @@ tests/unit_tests/
|
||||
| `core` | **28%** | 1289 | 🔄 需补充 app 启动 |
|
||||
| `persistence` | **24%** | 1099 | 🔄 需补充 mgr |
|
||||
|
||||
外部适配器测试需要 mock 环境或集成测试,不属于纯单元测试范畴。
|
||||
外部适配器测试需要 mock 环境或集成测试,不属于纯单元测试范畴。
|
||||
|
||||
@@ -32,6 +32,32 @@ class TestResolveRunnerId:
|
||||
runner_id = ConfigMigration.resolve_runner_id(pipeline_config)
|
||||
assert runner_id == 'plugin:langbot/local-agent/default'
|
||||
|
||||
def test_resolves_deerflow_and_weknora_legacy_runner_fields(self):
|
||||
assert (
|
||||
ConfigMigration.resolve_runner_id(
|
||||
{
|
||||
'ai': {
|
||||
'runner': {
|
||||
'runner': 'deerflow-api',
|
||||
},
|
||||
},
|
||||
}
|
||||
)
|
||||
== 'plugin:langbot/deerflow-agent/default'
|
||||
)
|
||||
assert (
|
||||
ConfigMigration.resolve_runner_id(
|
||||
{
|
||||
'ai': {
|
||||
'runner': {
|
||||
'runner': 'weknora-api',
|
||||
},
|
||||
},
|
||||
}
|
||||
)
|
||||
== 'plugin:langbot/weknora-agent/default'
|
||||
)
|
||||
|
||||
def test_resolve_no_runner_config(self):
|
||||
runner_id = ConfigMigration.resolve_runner_id({})
|
||||
assert runner_id is None
|
||||
@@ -73,6 +99,38 @@ class TestResolveRunnerConfig:
|
||||
)
|
||||
assert config == {'model': {'primary': 'uuid-123', 'fallbacks': []}}
|
||||
|
||||
def test_reads_deerflow_and_weknora_legacy_runner_blocks(self):
|
||||
pipeline_config = {
|
||||
'ai': {
|
||||
'deerflow-api': {
|
||||
'api-base': 'http://127.0.0.1:2026',
|
||||
'assistant-id': 'lead_agent',
|
||||
},
|
||||
'weknora-api': {
|
||||
'base-url': 'http://localhost:8080/api/v1',
|
||||
'app-type': 'agent',
|
||||
},
|
||||
},
|
||||
}
|
||||
|
||||
deerflow_config = ConfigMigration.resolve_runner_config(
|
||||
pipeline_config,
|
||||
'plugin:langbot/deerflow-agent/default',
|
||||
)
|
||||
weknora_config = ConfigMigration.resolve_runner_config(
|
||||
pipeline_config,
|
||||
'plugin:langbot/weknora-agent/default',
|
||||
)
|
||||
|
||||
assert deerflow_config == {
|
||||
'api-base': 'http://127.0.0.1:2026',
|
||||
'assistant-id': 'lead_agent',
|
||||
}
|
||||
assert weknora_config == {
|
||||
'base-url': 'http://localhost:8080/api/v1',
|
||||
'app-type': 'agent',
|
||||
}
|
||||
|
||||
def test_resolve_no_config(self):
|
||||
config = ConfigMigration.resolve_runner_config(
|
||||
{},
|
||||
@@ -155,3 +213,45 @@ class TestNormalizePipelineConfig:
|
||||
'model': {'primary': 'old-model', 'fallbacks': []},
|
||||
'knowledge-bases': ['kb_1'],
|
||||
}
|
||||
|
||||
def test_migrates_deerflow_legacy_runner_block(self):
|
||||
config = {
|
||||
'ai': {
|
||||
'runner': {'runner': 'deerflow-api'},
|
||||
'deerflow-api': {
|
||||
'api-base': 'http://127.0.0.1:2026',
|
||||
'assistant-id': 'lead_agent',
|
||||
},
|
||||
},
|
||||
}
|
||||
|
||||
migrated = ConfigMigration.migrate_pipeline_config(config)
|
||||
|
||||
assert migrated['ai']['runner']['id'] == 'plugin:langbot/deerflow-agent/default'
|
||||
assert 'runner' not in migrated['ai']['runner']
|
||||
assert 'deerflow-api' not in migrated['ai']
|
||||
assert migrated['ai']['runner_config']['plugin:langbot/deerflow-agent/default'] == {
|
||||
'api-base': 'http://127.0.0.1:2026',
|
||||
'assistant-id': 'lead_agent',
|
||||
}
|
||||
|
||||
def test_migrates_weknora_legacy_runner_block(self):
|
||||
config = {
|
||||
'ai': {
|
||||
'runner': {'runner': 'weknora-api'},
|
||||
'weknora-api': {
|
||||
'base-url': 'http://localhost:8080/api/v1',
|
||||
'app-type': 'agent',
|
||||
},
|
||||
},
|
||||
}
|
||||
|
||||
migrated = ConfigMigration.migrate_pipeline_config(config)
|
||||
|
||||
assert migrated['ai']['runner']['id'] == 'plugin:langbot/weknora-agent/default'
|
||||
assert 'runner' not in migrated['ai']['runner']
|
||||
assert 'weknora-api' not in migrated['ai']
|
||||
assert migrated['ai']['runner_config']['plugin:langbot/weknora-agent/default'] == {
|
||||
'base-url': 'http://localhost:8080/api/v1',
|
||||
'app-type': 'agent',
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user