Skip to content

fix: preserve structured MCP failure evidence and macOS App logs - #35

Merged
imgoose merged 1 commit into
mainfrom
codex/collector-host-evidence-20261028
Oct 29, 2026
Merged

imgoose merged 1 commit into
mainfrom
codex/collector-host-evidence-20261028

Conversation

@imgoose

@imgoose imgoose commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

本补丁补两处已确认的 Hosted MCP 采集链路遗漏:macOS 的 Codex App 日志存在却未自动选中;采集主命令仍示范纯文本输入,容易遗漏宿主已经提供的结构化错误上下文。

本 PR 叠在 #34 的已验登录体验候选上,供合并到 #34 后统一审查/发布,避免 Codex 版本相互覆盖。只向本分支推送,不直接修改 #34。若 #34 先合入 main,请正常调整本 PR 的目标分支并复核最终 diff。不能把本 PR 合入候选分支等同于正式 main 或商店已发布。

改动:

  • macOS 自动选取 ~/Library/Logs/com.openai.codex/YYYY/MM/DD;继续使用现有 ChatCut startup 事件、线程、3 文件/128 KiB 尾段及 30 条记录边界。显式路径优先,Windows/MXC 回退保留,Linux 不猜测目录。
  • Codex/Claude 两份指引改为优先使用已有 --failure-record,携带实际可用的原始 error/cause/stack、发生时间、本地调用/线程 ID 和原响应。纯文本入口保留为缺少结构化数据时的回退。没有新增或伪造原始字段。
  • collector 1.2.7;Codex 1.10.27、Claude 1.10.28,manifest 与请求头一致。修正原有 Claude 日志保留测试的 CRLF 比较,使 Windows 检查不因行结束符误报。

验证:

  • 改前同一受控 macOS 路径:默认 0 条、手动指定 1 条;改后默认 1 条。此为受控路径/事件验证,不是实际 macOS 宿主执行或原用户恢复。
  • 8 个相关测试文件 84/84 通过,覆盖原错保留、元数据缺失、原因链、响应、脱敏、Windows 回退和分片 SHA256。
  • Windows 原生 Codex CLI 与官方 Claude Code 2.1.310 在全新隔离配置目录完成完整本地 marketplace 安装:Codex Git 包 53 文件/18 Skills,Claude Git 包 57 文件/19 Skills,安装/缓存文件逐字节匹配。
  • 两宿主安装后各验证已知/未知元数据两种离线报告,共 4 份;原文件未变,原错/原因链/可用时间及 ID 保留,实际凭据脱敏,完整分片重组及 SHA256 一致。离线上报保持 skipped_offline。
  • 没有模型回合、生产上报、登录、用户全局配置改动或独立 Doctor 内容。

边界:
这不恢复旧报告已丢失的信息,不捕获宿主在解析前丢弃的原始 TLS/HTTP 字节,不把 App 启动通知当作原工具调用失败,也不证明旧 42 份报告的根因已经解决。正式源发布后仍需远端普通安装与新版自然报告验收。商店审核/上传由 Ruofan 按正常流程处理,现有审核包不动。


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@imgoose
imgoose requested a review from leowio October 28, 2026 12:08
Base automatically changed from codex/connection-onboarding-ux-pure-20261028 to main October 28, 2026 12:11
@imgoose
imgoose merged commit e8ebd99 into main Oct 29, 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