Commit 2fb11e1
committed
fix(ui): 修复模型连接测试的关键问题并增强代码质量
修复问题:
1. 修复 API 参数调用错误 - addModel 方法缺少 key 参数导致验证失败
2. 修复 API Key 脱敏问题 - 测试连接时使用脱敏后的 Key 导致认证失败
3. 修复状态管理时序问题 - 使用 nextTick 替代不可靠的 setTimeout
增强功能:
1. 强化连接测试的前置校验逻辑,增加配置完整性检查
2. 优化并发控制,测试期间自动禁用按钮,防止重复请求
3. 统一守卫逻辑到单一数据源,遵循 DRY 原则
4. 改进 paramOverrides 更新逻辑,支持参数删除操作
技术改进:
- 使用 Vue nextTick API 替代 setTimeout,提升可靠性
- 实现临时模型测试模式,避免污染原始配置
- 增强 canTestFormConnection 校验,包含配置完整性检查
- 正确处理 API Key 脱敏/恢复逻辑
影响范围:
- packages/core/src/services/model/manager.ts
- packages/ui/src/composables/useTextModelManager.ts
- packages/ui/src/components/TextModelEditModal.vue
- packages/ui/src/components/TextModelManager.vue1 parent 935c6e3 commit 2fb11e1
File tree
4 files changed
+95
-16
lines changed- packages
- core/src/services/model
- ui/src
- components
- composables
4 files changed
+95
-16
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
332 | 332 | | |
333 | 333 | | |
334 | 334 | | |
335 | | - | |
336 | | - | |
337 | | - | |
338 | | - | |
339 | | - | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
340 | 340 | | |
341 | 341 | | |
342 | 342 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
331 | 331 | | |
332 | 332 | | |
333 | 333 | | |
334 | | - | |
| 334 | + | |
335 | 335 | | |
336 | 336 | | |
337 | 337 | | |
| |||
403 | 403 | | |
404 | 404 | | |
405 | 405 | | |
406 | | - | |
| 406 | + | |
| 407 | + | |
| 408 | + | |
| 409 | + | |
407 | 410 | | |
| 411 | + | |
| 412 | + | |
408 | 413 | | |
409 | 414 | | |
410 | | - | |
411 | 415 | | |
412 | 416 | | |
413 | 417 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
44 | 49 | | |
45 | 50 | | |
46 | 51 | | |
47 | 52 | | |
48 | 53 | | |
49 | 54 | | |
50 | 55 | | |
51 | | - | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
52 | 68 | | |
53 | 69 | | |
54 | 70 | | |
55 | 71 | | |
56 | 72 | | |
57 | 73 | | |
| 74 | + | |
58 | 75 | | |
59 | 76 | | |
60 | | - | |
61 | 77 | | |
62 | 78 | | |
63 | 79 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
139 | 139 | | |
140 | 140 | | |
141 | 141 | | |
142 | | - | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
143 | 154 | | |
144 | 155 | | |
145 | 156 | | |
| |||
334 | 345 | | |
335 | 346 | | |
336 | 347 | | |
337 | | - | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
338 | 354 | | |
339 | 355 | | |
340 | 356 | | |
| |||
546 | 562 | | |
547 | 563 | | |
548 | 564 | | |
549 | | - | |
| 565 | + | |
| 566 | + | |
| 567 | + | |
550 | 568 | | |
551 | 569 | | |
| 570 | + | |
552 | 571 | | |
553 | | - | |
554 | | - | |
555 | | - | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
| 581 | + | |
| 582 | + | |
| 583 | + | |
| 584 | + | |
| 585 | + | |
| 586 | + | |
| 587 | + | |
| 588 | + | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
| 592 | + | |
| 593 | + | |
| 594 | + | |
| 595 | + | |
| 596 | + | |
| 597 | + | |
| 598 | + | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
| 609 | + | |
| 610 | + | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
| 614 | + | |
556 | 615 | | |
557 | 616 | | |
558 | 617 | | |
| |||
0 commit comments