|
|
e50bffa34f
|
fix(deepsearch): 只关自己拉起的 SearXNG;条数改由插件侧截断
两个都由「真调用/真测试」暴露,且都会让线上搜索表现为「后端不可用」。
一、归属:接管不等于拥有(例:E2E 测试把生产后端带走)
旧实现只要探活成功就认领关闭责任 → 同机第二个实例(测试拉起的插件、另一个 daemon)
退出时就 docker compose stop 掉**线上正在用的**后端。实测:跑一次
`go test ./internal/plugins/ -run TestRealPlugin_DeepSearch`,teardown 即关停
127.0.0.1:8888,用户看到的就是「搜索后端起不来」。
修:只有真正执行过 `docker compose up -d` 的实例才算「我们起的」;探到已在运行只接管。
二、条数:SearXNG 不认 count/limit(count/max_results 形同虚设)
实测 ?count=3、?limit=3、不带参数返回**完全相同的 35 条**,所以截断必须在插件里做。
旧实现把 count 当 limit 参数发给 SearXNG 就以为生效了 → 模型每次吞 35~58 条带摘要结果,
还会把「命中 N 条」当成「拿到了 N 条」报给用户(实测发生过)。
修:新增 limitResults(默认取 max_results,上限 20);输出改成
「命中 N 条,返回前 M 条」;不再发无意义的 limit 参数。
验证:22 项单测全过、-race 干净、vet/gofmt 干净;两条归属测试做过扰动(把旧语义放回
去后必红,并如实打出它执行的 `docker compose stop -t 2`);内核 E2E 三条通过且
**跑完 healthz 仍 200、容器未重启**;线上 1.1.2 实测 count=3 → 「命中 40 条,返回前 3 条」。
版本 1.1.0 → 1.1.2。
|
2026-09-13 09:55:27 +08:00 |
|