Skip to content

[Maintainer handoff] PR review/merge order and integration evidence / 审阅合并顺序 #407

Open

Description

@Haoming9527

本 issue 汇总这批修复的审阅入口、依赖和合并顺序。2026-10-25 更新:21 个分片已按下列顺序全部合入 main(merge commit,均为 T-Auto 署名/共同作者),最终内容与 #406 已验证 head 等价(仅 docs/CHANGELOG.md 条目顺序不同)。各 PR 正文中的 Closes/Fixes 已按声明关闭对应 issue;保留项见第 4 节。落地明细见本 issue 评论。

建议先按下表审阅分片,再选择一种落地方式。#406 是完整组合的集成草稿,已验证组合后的行为;它不是需要与所有分片重复合并的一份额外实现。

1. 建议审阅及逐项合并顺序

同一行的逗号表示可独立审阅;箭头表示已经核实的提交祖先依赖。跨行次序是便于控制风险的建议,不代表所有行之间都有代码依赖。

顺序 PR 内容与对应 issue
1 #401 所有目标分支的 PR 都触发 CI,附管理员 ruleset 交接。对应 #303 的代码部分;实际启用必需检查仍未完成。
2 #344、#389 审计存储路径保护(#329);Windows 受限令牌下的 MSYS 用户共享段访问(#363)。
3 #383 → #390 → #405 增量会话协调和低成本工具视图(#333/#369/#382);会话追加窗口内延后发布、兼容纯文本 prompt hook、首轮就绪回归(#386/#387/#375);独立根会话的阻塞探针隔离(#359,并为 #346 提供运行时修复路径)。
4 #392;#391 → #396 stdin/PATH 与 Bash 脚本契约(#364/#372);终端展示(#32);有界 JSON spill 摘要(#297)。
5 #393、#394、#395 有界 grep 与完整性口径(#384);catalogue/settings/manifest 一致性(#315);已验证的宿主版本范围及组件图标导出(#200)。
6 #398 macOS Intel 构建、发布与 CI 支持(#371)。
7 #388、#397、#400 校验和对象说明(#300);升级、卸载及迁移文档(#175/#335/#372);Bash 自有工具限制和 discovery/宿主边界说明(#380/#346)。
8 #381 → #399 → #402 → #404 benchmark 报告/夹具/默认值(#379);跨平台准备及比较保护(#295);实际模型与思考档位路由(#294);candidate 组件开关、就绪检查及比较保护(#296)。
9 #403 PR 审阅快照 CLI,覆盖 #356 当前的 CLI-first P0;后续工具簇、时间线和收益评估未声称完成。

这 21 个分片目前均为 ready for review。各自正文保留实现说明、红绿回归和实际 CI 链接。

堆叠 PR 的具体处理

2. #406 的定位:另一种落地方式

Draft 集成 PR #406 包含上表全部 21 个分片的原始历史及冲突处理,目标为 main。可用于查看最终组合、核对交叉影响和复用已验证的冲突处理。

默认仍按分片审阅/合并,以符合仓库约 1,000 行实现变更的分片约定。 若维护者明确选择整体集成方式,应先完成上表分片审阅、确认接受聚合 diff,再将 #406 转为 ready 并合并这一入口;随后按实际提交/差异核对各分片状态,不再把同一批内容逐项重复合入。若已按分片落地,则重新评估 #406 剩余 diff,不把它当作仍需合并的整批新功能。

#406 目前保持 Draft。没有执行任何 PR merge,也没有批量关闭现有 issue。

3. 当前验证证据(2026-10-25)

核验的 main:662a38871f2241886343090adda0a1107a50e274。集成 head:2e261035f88780015a7e132d36e7614e6cf4a8de。

  • #406 自己的 pull_request CI:40116812179:completed / success。9 个执行的 job 全通过,包括 Linux、Windows、macOS arm64、macOS Intel、Node floor 和 CI required;条件化 benchmark smoke 为预期跳过。
  • 同一 head 的完整集成 + benchmark smoke:40115118596:workflow_dispatch,12/12 job 通过,包含 Linux/Windows/macOS 三个平台的 model-free smoke。此运行补充组合证据,不替代上一条 PR-event 运行。
  • 所列 21 个分片的当前 head 均有通过的执行检查,GitHub 当前均报告 MERGEABLE。chore(integration): combine reviewed issue fixes #406 也为 MERGEABLE;这是当前快照,不是对未来提交无冲突的保证。
  • 本地组合验证:build/typecheck 通过;workflow 89 checks、manifest 133 checks、426 个代码文件大小检查通过;23 个 benchmark/session/review/manifest 套件通过,另有 5 个 metadata/settings/policy/presentation 套件通过。catalogue 套件因独立集成目录未构建 native worker 而跳过,未计为本地通过;完整平台 CI 提供运行时覆盖。

4. 保留的验收边界

English summary

Review the 21 slices in the table, preserving the three dependency chains and the additional #401 → #404 dependency. After #402 lands, retarget #404 from its feature-branch base to main before deleting that base branch, refresh the diff and obtain new PR-event CI. Prefer the repository's small-PR route; #406 is an alternative aggregate landing path after explicit maintainer review, not another copy to merge after all slices. Its own PR-event CI is now green, and the same head also passed all 12 integration/smoke jobs. Keep the existing issues open pending actual landing/acceptance; #261/#207/#303/#360 and Draft/NO-MERGE #362 retain their specific exceptions.

Activity

  1. T-Auto commented on Oct 25, 2026

    @T-Auto
    Owner

    落地结果(2026-10-25)

    21 个分片已按本 issue 第 1 节的顺序全部合入 main,最终 main = dfafddc941695090df2c6e2e323193785aec4eb6。

    顺序 PR merge commit 标题
    1 #401 446552be95 fix(ci): validate stacked PRs and prepare required-check enforcement
    2 #344 7549377c9f fix(file): reserve audit-store components in absolute paths
    3 #389 f870aee18c fix(bash-ipc): preserve access to user shared sections
    4 #383 5b1b9a32ed fix(ops): reconcile session projections incrementally (#333)
    5 #390 ed3be1eded fix(session): defer Bash publication and support text-only prompt hook
    6 #405 edf367fa80 fix(session): isolate stalled presentation probes by scope family
    7 #392 6b4ea31f2d fix(shell): preserve caller PATH and close empty stdin
    8 #391 688fc1fc25 feat(shell): present captured Bash calls as terminal cards
    9 #396 22ad291e89 feat(shell): summarize intact JSON spills in Native responses
    10 #393 c0fde83093 feat(search): support honest bounded grep scans
    11 #394 da9930285a fix(catalogue): enforce shared publication and settings facts
    12 #395 e1d056ab13 fix(metadata): declare tested hosts and export component icons
    13 #398 f5ca648ba4 feat(ptc): support macOS Intel with verified release artifacts
    14 #388 f95604ea4b docs(provenance): clarify published checksum objects
    15 #397 3daf62f01d docs: clarify upgrades removal and 0.4 migration
    16 #400 b2fb26108c docs: explain owned Bash restrictions and host integration failures
    17 #381 d14bae8cbf feat(bench): slim the report, shrink the fixtures, rewrite the manual
    18 #399 1005be9270 fix(bench): support portable setup and platform comparison guards
    19 #402 66264327ae feat(bench): route and separate model reasoning conditions
    20 #404 0a98673243 feat(bench): configure and compare candidate component switches
    21 #403 dfafddc941 feat(review): add bounded PR snapshot CLI

    Written by the landing run; all remote actions were read-only checks plus the ordered merges above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions