Skip to content

feat(orchestration): 补齐专项角色开关与同难度兜底 - #326

Merged
TimWongUp merged 10 commits into
mainfrom
codex/specialist-roles
Nov 7, 2026
Merged

TimWongUp merged 10 commits into
mainfrom
codex/specialist-roles

Conversation

@TimWongUp

@TimWongUp TimWongUp commented Nov 7, 2026 •

Copy link
Copy Markdown
Owner

高级编排此前只有通用探索、通用执行和界面实现,无法分别为写作、检索等任务绑定模型。本次补齐九组固定角色:通用探索、通用执行、代码实现、界面实现、写作、网页知识检索、社交调研、数据分析、文档制作;每组独立开关,简单/困难分别绑定 CLI 共用模型池中的模型与 effort。

  • App、Python 服务与生成规则共用角色目录。同难度专项优先,由 Manager 明确选择同难度通用工位兜底,不静默换模型或跨难度。
  • 关闭角色保留绑定,从目录、工具 schema 和生成规则移除;显式 Provider、缺失配置和继承旧 seat 的续接同样校验开关。编排关闭后可显式给出新职责 role 脱离继承工位。无可见工位时隐藏 seat,保持 schema 合法。
  • 旧配置默认保留原三组,新六组默认关闭;界面组仅在自身和通用执行都开启时沿用通用执行同档。生图保持独立设置。
  • 自定义规则迁移保留完整派发指引,角色/生图开关与编辑器同步,首尾生成目录清理与前端一致;手写正文保留。
  • 已集成 feat: 统一十分钟等待并补齐宿主超时 #327,解决规则与 ADR 索引冲突,同时保留十分钟等待、seq 和专项角色规则。

验证:集成候选 f8cb2bd 上 Python 3.12 全量 546 通过、5 跳过;Rust 295 通过、2 忽略;Node 160 通过。Ruff format/check、Pyright、cargo fmt、普通及预览 clippy 通过,该候选全平台 Python/Tray CI 成功。最终 c282567 只修正文尾部换行、两份跨端 fixture 前置字段和架构说明;已补规则 8 项、跨端 Python 85 项和 Rust fixture 定向检查并通过,静态检查通过。

原生预览已验证九组角色及自定义编辑器:打开/关闭写作时目录同步增删,用户正文保留,测试草稿丢弃,未保存正式数据或调用真实 Provider CLI。完整独立评审与定向整改结论见 PR 评论。

剩余边界:默认状态保留合法但无人引用的 not:{} 工位定义,Draft202012Validator/Ajv/pi-schema 通过,未实测各云端 API 子集;未来删组/回滚时未知 seat 的历史读取兼容风险未在本次扩展。Windows 首轮既有 50ms 结果保留测试曾时序失败,同 head 重跑通过。未发布。

Copy link
Copy Markdown
Owner Author

评审完成(R2,可合并)

  • 最终 head: c282567;base: eb49c24(feat: 统一十分钟等待并补齐宿主超时 #327 已合入)。范围 git diff base...head;原完整评审基于 abed52a / 407c7b8,整改及集成增量由原工位定向复核。
  • 等级及分工:R2,角色启停、续接、跨语言配置迁移与公开工具 schema 有独立风险维度。correctness-reviewer(Antigravity / gemini-4-argon-medium)完整评审;architecture-reviewer(Claude Code / claude-opus-5-5)负责跨语言目录、schema、所有权和集成。
  • Findings:缺失编排配置时仍放行旧工位、空 enum、不一致的自定义旧规则迁移/首尾目录清理与编辑器角色和生图同步,均 CONFIRMED 并修复。编排宿主级关闭的续接边界补入 ADR0065,错误给出新非空职责 role 脱离继承 seat 的办法;不静默改职责。跨端 fixtures 补合法新角色及未知/重复组,非法样例 preset 前置字段补齐;架构说明对齐显式 Provider 也拒绝损坏普通工位配置。原正确性及架构工位最终定向复核均通过,无未解决阻塞。
  • 集成:保留 feat: 统一十分钟等待并补齐宿主超时 #327 的全部已核实宿主超时和等待改动;rules.rs 同时保留专项开关/兜底和十分钟/seq,ADR0065/0066 都保留;Manager Skill 的300000ms外层等待写法保留。
  • 本地验证:f8cb2bdd 上 Python3.12全量546通过/5跳过,Rust295通过/2忽略,Node160通过;Ruff/Pyright/fmt/普通与预览clippy通过。最终c2825671仅4文件小修,补跑Rust规则8项、跨端Python85项/Rust fixture、clippy/静态检查通过,其余全量证据仍适用。预览实测九组角色、自定义写作开关增删目录并保留用户正文,草稿已丢弃,无正式配置写入或真实Provider调用。
  • 剩余低风险:默认空工位时保留合法但无人引用的 not:{} 定义,Draft202012Validator/Ajv/pi-schema接受,云端模型API子集未实测;TaskRecord.seat当前数据合法,未来删组或回滚时未知seat的历史兼容未扩展;生成规则中的seq/十分钟文字未加镜像断言。首次Windows既有50ms保留测试时序失败,同head重跑成功。
  • CI(最终head):Windows Python成功、Windows/macOS Tray成功;macOS/Linux Python仍在运行,本地对应平台检查通过。按用户偏好不以本机可测平台CI阻挡合并,合并后继续核对main。

@TimWongUp
TimWongUp merged commit 82d074f into main Nov 7, 2026
6 checks passed
@TimWongUp
TimWongUp deleted the codex/specialist-roles branch November 7, 2026 17:08

Copy link
Copy Markdown
Owner Author

合并回执:已按最终评审 head c282567 squash 合入 main,提交 82d074f。该候选最终 Python(macOS/Linux/Windows)和 Tray(macOS/Windows)CI 已全部成功。主 checkout 已同步,两个已合并 PR 的一次性 worktree 与本地分支已清理,远端分支由仓库合并设置自动删除。合并后的 main push 工作流正在运行。

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