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

fix: [test] blocked-check 将 API 错误误判为阻塞已解除 - Closes #58 #60

Merged
pzhang_zywl merged 2 commits from test/issue-58 into main 2026-06-02 16:21:05 +08:00
Conversation 0 Commits 2 Files Changed 1
+42 -29
pzhang_zywl commented 2026-06-02 16:20:23 +08:00
Owner
Copy Link
Copy Source

Summary

  • 新增 _req_safe() 函数：API 错误返回 None 而非崩溃
  • blocked_check / _unblock_issues_blocked_by / _get_blocking_refs 改用 _req_safe
  • API 失败时保守处理：保持 blocked（不误解除）

Closes #58

[qe-agent: qa-0602-1358]

## Summary - 新增 _req_safe() 函数：API 错误返回 None 而非崩溃 - blocked_check / _unblock_issues_blocked_by / _get_blocking_refs 改用 _req_safe - API 失败时保守处理：保持 blocked（不误解除） Closes #58 --- [qe-agent: qa-0602-1358]
pzhang_zywl added 2 commits 2026-06-02 16:20:26 +08:00
test: blocked-check 用 _req_safe 替代 _req 避免 API 错误误判 - Closes #58 d73da7cda9 
- 新增 _req_safe()：API 错误返回 None 而非 sys.exit(1)
- blocked_check / _unblock_issues_blocked_by / _get_blocking_refs 改用 _req_safe
- API 失败时保守处理：保持 blocked 状态（不误解除）
- 验证：#18 正确识别被 #57 阻塞

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
test: blocked-check 用 _req_safe 替代 _req 避免 API 错误误判 - Closes #58
CI / test (pull_request) Successful in 12s
Details
eccaa28b1d 
- 新增 _req_safe()：API 错误返回 None 而非 sys.exit(1)
- blocked_check / _unblock_issues_blocked_by / _get_blocking_refs 改用 _req_safe
- API 失败时保守处理：保持 blocked 状态

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
pzhang_zywl merged commit eb2af77c90 into main 2026-06-02 16:21:05 +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#60
Delete Branch "test/issue-58"

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?