zeekrAI/document_analyzer
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: [product] _normalize_rule 增加 screen_type 默认值防御 + step2 test 降级 warn - Closes #86 #87

Merged
pzhang_zywl merged 1 commits from dev/issue-86-screen-type-defense into main 2026-06-03 14:44:50 +08:00
Conversation 0 Commits 1 Files Changed 3
+56 -2
pzhang_zywl commented 2026-06-03 14:44:30 +08:00
Owner
Copy Link
Copy Source

Summary

  • _normalize_rule 新增 precondition 防御层：screen_type 缺失→"any"，geo 缺失→"global"，precondition=None→{}
  • test_step2_precondition_fields 降级为 warn（取决于 LLM 输出质量，step3 兜底）
  • 新增 4 个 normalize UT：missing screen_type / missing geo / None precondition / missing precondition

修复类型

  • 代码级修复（UT 可验证）

Test

  • pytest 全量通过 (115 passed, 16 skipped)
  • 新增 4 个 normalize UT

Closes #86

[da-0603-1426]

## Summary - `_normalize_rule` 新增 precondition 防御层：screen_type 缺失→"any"，geo 缺失→"global"，precondition=None→{} - `test_step2_precondition_fields` 降级为 warn（取决于 LLM 输出质量，step3 兜底） - 新增 4 个 normalize UT：missing screen_type / missing geo / None precondition / missing precondition ## 修复类型 - [x] 代码级修复（UT 可验证） ## Test - [x] pytest 全量通过 (115 passed, 16 skipped) - [x] 新增 4 个 normalize UT Closes #86 --- [da-0603-1426]
pzhang_zywl added 1 commit 2026-06-03 14:44:33 +08:00
fix: _normalize_rule adds screen_type/geo defaults + step2 test downgrades to warn - Closes #86
CI / test (pull_request) Successful in 10s
Details
f7f00091a6 
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pzhang_zywl merged commit bdef679c2b into main 2026-06-03 14:44:50 +08:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
blocked
被其他 issue 阻塞，暂不处理
 product-code
产品功能相关
 test-code
QE-Agent 测试代码相关
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dzhou_dev_agent_01 dzhou_qe_agent_01 hshen_dev_agent_01 hshen_qe_agent_01 phong_dev_agent_01 phong_qe_agent_01 pzhang_dev_agent_01 pzhang_qe_agent_01 pzhang_zywl vlu_dev_agent_01 vlu_qe_agent_01
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: zeekrAI/document_analyzer#87
Delete Branch "dev/issue-86-screen-type-defense"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?