Skip to content

修复权限审批等待继承 Run 截止时间 - #166

Merged
rojim666 merged 1 commit into
rojim666:mainfrom
z2Ace0107:codex/issue-69-pr05-permission-run-deadline
Sep 20, 2026
Merged

rojim666 merged 1 commit into
rojim666:mainfrom
z2Ace0107:codex/issue-69-pr05-permission-run-deadline

Conversation

@z2Ace0107

Copy link
Copy Markdown
Contributor

变更内容

  • 让权限审批等待共享父 Run 的绝对 wall-clock deadline,而不是在 Run 结束后继续等待独立的权限超时。
  • 区分 Run deadline、权限 timeout、用户拒绝和事件发送器自身的 TimeoutError,保留各自的错误语义。
  • 为权限事件发送、Future 等待和 pending 清理增加 deadline 边界,防止挂起请求无限等待。
  • 防止旧 Run 的迟到响应或旧请求清理误伤同 ID 的新请求。
  • 将工具层结果映射为 deadline_exceeded,不误发布 permission.denied,也不对 deadline 结果自动重试。
  • 增加权限管理器、工具工作流和 S5 权限生命周期回归测试。

验证结果

  • PR-05 定向测试:63 passed
  • 权限生命周期集成测试:1 passed
  • Runner 测试:28 passed
  • Ruff:通过
  • 定向 mypy:通过
  • compileall:通过
  • CRLF-aware diff check:通过

已知限制

  • 当前 origin/main 缺少 sztu_code.core.prompts.Sztubuddy 模块/目录,原生测试环境会因此出现导入失败。
  • 定向验证使用了一次性进程内 stub 绕过该既有基线导入缺失;stub 未写入仓库,也没有在本 PR 中修复该基线问题。
  • 更大范围测试中还观察到一个与本 PR 无关的既有浮点边界抖动:0.05000000000291038 <= 0.05;本 PR 未修改该测试。

关联

Refs #69

@rojim666
rojim666 marked this pull request as ready for review September 20, 2026 18:51
Copilot AI lite review requested due to automatic review settings September 20, 2026 18:51
@rojim666
rojim666 self-requested a review as a code owner September 20, 2026 18:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@rojim666
rojim666 merged commit 9507485 into rojim666:main Sep 20, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants