Skip to content

fix: 保留 uv 执行诊断并补齐失败日志 (T5.17) - #33

Merged
jinghero merged 1 commit into
AUTO-MAS-Project:mainfrom
jinghero:fix/cnb263-uv-diagnostics
Nov 10, 2026
Merged

jinghero merged 1 commit into
AUTO-MAS-Project:mainfrom
jinghero:fix/cnb263-uv-diagnostics

Conversation

@jinghero

@jinghero jinghero commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

摘要

  • uv 准备失败时保留底层失败类别和 Windows 错误号,便于从 Runtime 日志定位执行失败原因。
  • 修复命令关闭日志器后最终失败未落盘的问题,沿用同一命令和 operationId 追加终态记录;保持原错误码、退出码和清理语义。
  • 用真实日志器回读替换原有内存日志替身,补充缓存 uv 自检失败的回归覆盖。修改范围仅限 Runtime。

关联 AUTO-MAS CNB #263。

验证

  • 两项回归先红后绿:原实现丢失底层 details,且真实日志没有失败记录;修复后通过,并连同离线、取消、停采和日志关闭约束重复 10 轮通过。
  • 新编译 EXE 在隔离目录离线运行损坏的 uv 文件,关闭遥测;直接启动同一文件取得的 Windows 错误号 216 与 Runtime 结果、日志一致,退出码为 50,同一操作恰好一条失败记录。
  • go vet ./...、go build ./...、go run golang.org/x/vuln/cmd/govulncheck@v1.9.0 ./...、gofmt 和 diff 检查通过。
  • go test -p 1 ./... -count=1 在本机退出 1:runtime-state 临时目录清理失败、SUBST 夹具 W: 盘未就绪。未修改基线 116a045 使用相同命令也复现这两类失败;修复版 CLI、uv、日志、协议及遥测包通过,远端 Windows CI 的格式、vet、构建、完整测试和可达漏洞检查均已通过。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-10T08:40:57.395613Z c787ebe PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@jinghero
jinghero merged commit b7f5aeb into AUTO-MAS-Project:main Nov 10, 2026
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.

1 participant