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

fix: step1 空章节过滤 + step3 rule_signature None-safe - Closes #21 #31

Merged
pzhang_zywl merged 1 commits from dev/issue-21-fix-empty-section-coverage into main 2026-06-01 13:19:21 +08:00
Conversation 0 Commits 1 Files Changed 2
+21 -3
pzhang_zywl commented 2026-06-01 13:17:46 +08:00
Owner
Copy Link
Copy Source

Summary

  • step1 _quick_validate: 添加 _has_section_content() 过滤空内容章节（如仅含"无"字的图片章节），修复覆盖率误报
  • step3 rule_signature: 使用 or {} 防御 trigger=None 导致的 AttributeError

Root Cause

  • 4 个未覆盖章节在 parsed JSON 中仅 1 个字符"无"，无图无表——来自 doc_parser 无法解析的图片内容
  • step3 rule_signature 在 _normalize_rule 之前调用，LLM 产出的 trigger: null 会崩溃

Test

  • pytest 全量通过 (79 passed, 13 skipped)
  • step1 _has_section_content 与 QE tests/acceptance 逻辑一致

Closes #21

[da-0601-0914]

## Summary - step1 _quick_validate: 添加 _has_section_content() 过滤空内容章节（如仅含"无"字的图片章节），修复覆盖率误报 - step3 rule_signature: 使用 or {} 防御 trigger=None 导致的 AttributeError ## Root Cause - 4 个未覆盖章节在 parsed JSON 中仅 1 个字符"无"，无图无表——来自 doc_parser 无法解析的图片内容 - step3 rule_signature 在 _normalize_rule 之前调用，LLM 产出的 trigger: null 会崩溃 ## Test - [x] pytest 全量通过 (79 passed, 13 skipped) - [x] step1 _has_section_content 与 QE tests/acceptance 逻辑一致 Closes #21 [da-0601-0914]
pzhang_zywl added 1 commit 2026-06-01 13:17:48 +08:00
fix: step1 空章节过滤 + step3 rule_signature None-safe - Closes #21
CI / test (pull_request) Successful in 9s
Details
ebda8e37d1 
- step1 _quick_validate 添加 _has_section_content() 过滤空内容章节
  (如仅含"无"字的图片章节)，避免误报低覆盖率警告
- step3 rule_signature 使用 `or {}` 防御 trigger=None 场景
  修复 QE 报告的 step3 AttributeError

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pzhang_zywl merged commit 50eb37094a into main 2026-06-01 13:19:21 +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#31
Delete Branch "dev/issue-21-fix-empty-section-coverage"

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?