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

fix: [bug] IR Layer A 仍失败: rules[56] 空 sources + Layer C QE Audit 100% 不合格 - 来自 #18 - Closes #64 #65

Merged
pzhang_zywl merged 1 commits from dev/issue-64-fix-empty-sources into main 2026-06-02 17:26:02 +08:00
Conversation 0 Commits 1 Files Changed 2
+33 -15
pzhang_zywl commented 2026-06-02 17:25:24 +08:00
Owner
Copy Link
Copy Source

Summary

  • step3 _normalize_rule: 空 sources 的 rule 自动添加最小 text source
    • 从 rule path 推断 section
    • 提供 text_snippet 占位符

Root Cause

LLM 在 step2 生成的 rule 可能包含空 sources 数组，违反 Layer A schema 要求
(sources: must have at least one source)。

Test

  • pytest: 109 passed, 15 skipped
  • 新增 UT: test_normalize_empty_sources

Closes #64

[da-0602-1321]

## Summary - step3 `_normalize_rule`: 空 sources 的 rule 自动添加最小 text source - 从 rule path 推断 section - 提供 text_snippet 占位符 ## Root Cause LLM 在 step2 生成的 rule 可能包含空 sources 数组，违反 Layer A schema 要求 (sources: must have at least one source)。 ## Test - [x] pytest: 109 passed, 15 skipped - [x] 新增 UT: test_normalize_empty_sources Closes #64 --- [da-0602-1321]
pzhang_zywl added 1 commit 2026-06-02 17:25:27 +08:00
fix: step3 _normalize_rule 为空 sources 的 rule 添加最小 text source - Closes #64
CI / test (pull_request) Successful in 11s
Details
a24408521c 
防御性处理 LLM 输出中 sources 为空数组的情况，避免 Layer A schema 失败。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pzhang_zywl merged commit d682f64c01 into main 2026-06-02 17:26:02 +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#65
Delete Branch "dev/issue-64-fix-empty-sources"

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?