Skip to content

修复导入异常空响应误报完成,保留合法 [] 成功语义(#134) - #135

Open
Iris3971 wants to merge 1 commit into
P0luz:testingfrom
Iris3971:codex/issue-134-testing
Open

Iris3971 wants to merge 1 commit into
P0luz:testingfrom
Iris3971:codex/issue-134-testing

Conversation

@Iris3971

@Iris3971 Iris3971 commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

改了什么

Fixes #134

Markdown 导入收到异常空响应时,现在计为失败并提供 empty_extraction 诊断;模型明确返回合法 JSON 空数组 [] 时,仍正常完成且创建 0 条记忆。部分分块失败不会丢失其他已成功保存的记忆。

本分支直接基于 testing 4ec562e4f3fa897e6b93fd09eb9f31e8e7f7cba5,仅包含 #134 最小修复、回归测试及配套文档、版本和 manifest 更新。版本为 4.0.1,无 main/testing 历史差异夹带。

为什么这么改

原实现将空正文直接转换为 [],导致“没有有效模型响应”和“模型明确判断没有值得记忆的内容”得到相同的成功状态。修复在导入提取边界抛出明确异常,复用既有失败计数、状态持久化和部分成功处理,保留合法空数组的成功语义。

不改共享 Dehydrator、提取提示词、独立导入预算、依赖或记忆存储规则,不记录模型响应正文或用户原文。

测过没有

  • 使用当前 testing 锁定依赖:定向测试 63 passed;全量测试 3273 passed、101 skipped,覆盖率 77.97%。
  • Ruff、manifest 校验和差异检查通过;与 testing 合并无冲突。
  • 推送前仅将提交身份及 Signed-off-by 改为 GitHub noreply,代码树与已审计提交完全一致;新 HEAD 下 manifest 相关测试再次运行,9 passed。
  • Bandit:testing 基线与本分支均报告相同的既有 10 条告警,位置和类型一致,本次无新增;并非安全扫描全绿。
  • GitHub Actions 当前在 Bandit 步骤因上述既有告警失败(运行日志);CI 后续全量测试和 Docker 集成尚未执行。
  • Docker 未验证:当前环境没有 Docker,未执行容器从零构建、启动及 Dashboard 验证。
  • 未调用真实模型供应商,不据此宣称 修复 Markdown 批量导入的提取 token budget 配置 #130 的真实输出 >2048 边界已覆盖。
  • 本 PR 不宣称完整发布验收通过。

仅提交审核,不部署、不合并。


  • 我的提交带有 Signed-off-by:,符合 DCO 与 CONTRIBUTING.md 的要求。
  • 没有把真实记忆内容、API Key、密码带进这个 PR。

Signed-off-by: Iris3971 <293864676+Iris3971@users.noreply.github.com>
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.

[Bug] Markdown 导入将空正文/无 choices 当成合法空提取,无告警显示 completed(main/testing 均可复现)

1 participant