Skip to content

#180 终审遗留:notes_processor 空切片守卫不可达且测试不约束;纯空白输入语义未锁 #185

Closed

Description

@zj1123581321

#179 修复(PR #180,合并 4e1be78)终审留下的两条非阻塞问题,本仓 risk-tier: personal,均低于 P1 红线。

  1. 不可达守卫 + 测试不约束:llm/processors/notes_processor.py 新增的 if not mapping.slices: 分支不可达——NotesMappingResult.is_valid 已要求 bool(self.slices),空切片在前面 if not mapping.is_valid 处就返回 FAILED。tests/unit/test_empty_input_processors.py::test_notes_processor_empty_slices_returns_failed_result 实际走 fingerprint mismatch 的旧校验,删掉新守卫仍通过(codex 终审红验控制实测)。修法:删掉该分支,修正测试 docstring 与 tests/README.md 中「锁住 max_workers=0」的表述。
  2. 纯空白输入语义未固定:plain_text_processor.py 的 [text] if text.strip() else [] 让 " \t\n" 这类输入交付为空串。不是缺陷,但没有测试固定该语义;补一条纯空白样本用例即可。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    优先级:P2质量债、待拍板的产品决策、可选优化落点:本仓修复代码在本仓 git 跟踪路径内

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions