Skip to content

fix(copilot): 保留清理失败首错和进程诊断 - #520

Merged
Danceiny merged 1 commit into
mainfrom
fix/copilot-cleanup-diagnostics-518
Sep 19, 2026
Merged

Danceiny merged 1 commit into
mainfrom
fix/copilot-cleanup-diagnostics-518

Conversation

@Danceiny

Copy link
Copy Markdown
Owner

普通 planner 关闭时偶发超时,原日志既缺少进程归属,也可能被 finally 的清理异常盖住首错。本 PR 补齐诊断和失败现场保留,便于定位 #518;原始超时本轮没有复现,不宣称根因修复,也不关闭 #518

  • 区分 task/warmer,保留退出状态、既有时限、PID 和有界进程组快照;不记录参数、环境、请求或 provider 错误文本。
  • 多层错误日志仍能显示完整安全诊断;清理失败保持失败,默认 500 ms grace/2000 ms deadline 不变。
  • core proof 保留首错并依次清理,失败时保留隔离 stateRoot;独立审查的现场丢失、平台范围和日志可见性问题均已处理。

最终提交 7f5f86893899fdf083bf1b261bf54e07cebf16f8,macOS arm64,Node 22.23.2;临时 HOME/stateRoot,全部 live 开关为 0。

本地命令 退出码 关键结果
(cd ts && npx tsc --noEmit) 0 无类型错误,8.8 秒
node scripts/build-dist-compat-tests.mjs 0 dist 兼容检查通过,3.48 秒
GOTRY_SESSION_LIVE=0 GOTRY_HBCLI_LIVE=0 GOTRY_HOTELBYTE_SKILLS_LIVE=0 GOTRY_LAVISH_LIVE=0 bash scripts/run-all-tests.sh 0 ALL SUITES GREEN(明细见各节;真 LLM 巡检:replay-real.ts / time-eval-tests.ts --real,ADR-11 层),944.42 秒

最终完整日志 SHA-256:14e44ef87aadf4939b84157d290aaaaaa626a0400a8c1f7086df58a575b6ecbc;完成于 2026-09-19T14:48:08.831216+00:00。提交前完整回归也通过(873.54 秒);源代码冻结摘要及两轮日志均已归档。

E2E:完整回归包含真实安装的 SDK → worker → CLI/GoTry 插件 → 本地 HTTP/SSE provider → concluded turn → shutdown。末行 BOOKING COPILOT DSH CORE PROOF: real subprocess/sdk/plugin/tool-call/concluded-turn/session event/idle OK。Node 24.16.0 定向 core 与诊断 proof 也通过;旧 wrapper 对照按预期在缺少诊断的断言处失败。

边界:观察失败注入是诊断契约测试,不是历史超时复现;本地 provider 不代表真实供应商 UAT。进程组采集仅覆盖 Darwin detached fallback,Linux/Windows 明确返回未知或不支持,不能据此声称跨平台范围验收。没有新增依赖或架构/交易边界变化;回滚诊断协议和对应测试即可恢复原面。

双语专业测试报告。Related to #518#510#511#513 各自的验收仍保留。

@Danceiny
Danceiny merged commit 8da0efe into main Sep 19, 2026
8 checks passed
@Danceiny
Danceiny deleted the fix/copilot-cleanup-diagnostics-518 branch September 19, 2026 14:50
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