Skip to content

fix: 修复仓库审查问题并统一预览权限和发行来源 - #100

Merged
uiliugang merged 4 commits into
mainfrom
audit/repository-review-2026-11-11
Nov 11, 2026
Merged

uiliugang merged 4 commits into
mainfrom
audit/repository-review-2026-11-11

Conversation

@uiliugang

@uiliugang uiliugang commented Nov 11, 2026 •

Copy link
Copy Markdown
Owner

概述

附件中的十项已有补丁经逐项审查后保留。本次继续修复八项有明确代码路径的问题,并按维护者确认的产品决定,默认拒绝桌面预览网页定位,将官网桌面和 CLI 下载统一到 uiliugang/t3code。

修复内容

  • 原十项:MCode 仓库身份、秘密临时文件中断清理、需求源失败缓存、MEP 状态码复核、图标清空竞态、配置特殊自有键、移动端图标查找、合法双点目录、expo-widgets 依赖扩展与失效文档链接。
  • SSH:stderr 和 stdout 使用相同的 JSON 凭据脱敏与诊断长度限制;收集阶段设置上限,超大成功输出明确失败,尾部模式避免保留从凭据中途开始的残行。
  • 项目导入:重试与 forget 只清理路径归属核验通过、尚未完整克隆的临时目录;保留目标目录、符号链接目标和完整但移动失败的克隆。
  • GitHub 指派人与 nightly:每页 100 条,最多顺序读取 10 页,短页停止,避免无限请求。
  • 标签同步:增加受信任的手动入口,不改变 PR 事件的执行权限。
  • Windows:生成运行路径前将 T3CODE_HOME 绝对化。
  • 命令输出:以单调时钟计算节流间隔,避免系统时间回拨导致长时间等待。
  • IndexedDB:失败后可重试,关闭阻塞后迟到的连接,并在版本变更或关闭后失效缓存连接。
  • 桌面预览:从自动允许集合移除 geolocation,请求与检查处理器均默认拒绝。
  • 官网来源:发行 API、首页与下载页链接、仓库链接和安装命令统一使用 fork;发行缓存按仓库隔离,避免继续返回旧的上游下载。移除上游 npm 安装指引,保留上游移动商店入口并在 llms.txt 中区分来源。缺少的平台安装包回退到 fork 发行页。

验证

  • vp run check:changed origin/main --concurrency 2 退出码 0:格式、lint、未使用补丁、两轮 knip、GraphQL 与 42 个受影响包类型检查全部通过。
  • 17 个测试文件,118 项通过,8 项因平台不适用而跳过;React Compiler 定向检查 7 项通过。
  • SSH、克隆重试、指派人分页、时钟回拨、IndexedDB 与 nightly 均有修复前失败、修复后通过的回归证据。
  • T3 内置浏览器完成配对、设置页、免费 Replay 审批与完成流程,并检查 1280x800、430x932 两个视口。
  • 浏览器实际制造 IndexedDB 版本阻塞,验证解除后持久写入、版本升级释放连接、重建连接与清空成功。
  • 两项产品决定均有失败到成功回归:两个定位处理器拒绝请求,旧的上游发行缓存不再命中;内置浏览器核对真实 fork v0.0.8 下载、首页两个入口、Stable/Nightly 切换及安装命令,桌面与 430px 窄屏均通过。
  • 开发服务使用工作树隔离数据;测试数据由只读提取生成,没有写入真实数据库。
  • 截图两次返回 Preview automation snapshot timed out after 15000ms,浏览器脚本和点击正常。按用户要求跳过 PR 图片上传。

限制与保留事项

  • Windows 相对路径已补充原生回归,但当前机器为 macOS,该用例未运行;未宣称 Windows、Electron 或原生移动端实机通过。
  • 分页最多覆盖 1,000 条,超出该预算的组织成员或发行仍有限制。
  • MEP 元数据探测、统一 IPC 来源校验与 Rust stdin 上限仍属需进一步验证的风险;当前服务端按项目绑定仓库读取统计,未证实附件所述跨主机结果碰撞场景可达。这四项未按已证实缺陷修改。
  • 历史上已永久缓存的需求源失败仍需手动刷新一次。
  • 未执行真实 MEP/MCode 写入、远端 SSH 安装或 GitHub 标签同步;完整 CI 由新提交触发,不能用本地检查代替。

模型:gpt-6.1-sol;Harness:Codex(T3 Code)。

修复问题来源失败缓存、密钥临时文件清理、资源目录判断、仓库身份匹配和事项状态验证。
补齐 favicon 失效竞态及客户端特殊键处理的回归测试,校正 expo-widgets 依赖扩展和失效文档链接。

基于最新 main 保留现有变更;19 个文件,10 项已验证修复。
验证:74 项回归测试通过,42 个包直接类型检查通过。check:changed 其余阶段通过,缓存类型检查运行器受当前环境的 seccomp 限制。
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Nov 11, 2026
@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 21.5 KiB 21.5 KiB 0 B (0.0%) 24.4 KiB ✅
Codex Live turn messages 1 1 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.9 KiB 21.9 KiB 0 B (0.0%) 24.4 KiB ✅
Claude Live turn messages 1 1 0 (0.0%) 8 ✅

Baseline: 9324cfa · PR result: d47d051 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

统一 SSH 错误输出脱敏和有界收集,清理核验过的失败导入临时目录,补齐 GitHub 指派人与 nightly 发行分页。

修复时钟回拨导致的输出延迟、IndexedDB 打开失败后的恢复、Windows 相对安装目录和标签同步入口,并补充回归验证。
@uiliugang uiliugang changed the title fix: 修复仓库审查确认的缓存、状态和边界问题 fix: 修复仓库审查确认的缓存、恢复和边界问题 Nov 11, 2026
liugang59 added 2 commits November 11, 2026 12:27
桌面预览的请求与检查处理器均拒绝定位。官网桌面与 CLI 下载、安装指令和发行缓存统一使用 uiliugang/t3code,并保留上游移动服务的来源说明。

新增旧上游缓存隔离回归,更新定位权限回归,并完成内置浏览器下载验收。
@uiliugang uiliugang changed the title fix: 修复仓库审查确认的缓存、恢复和边界问题 fix: 修复仓库审查问题并统一预览权限和发行来源 Nov 11, 2026
@uiliugang
uiliugang marked this pull request as ready for review November 11, 2026 06:06
@uiliugang
uiliugang merged commit 3626582 into main Nov 11, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant