维护者确认:PR #3488 架构改动(Codex 核心路径 / 大 diff)
- Dominant language
- TypeScript
- Stars
- 2.7k
- Forks
- 395
- Avg merge
- 21h 48m
- Merged PRs (30d)
- 776
Description
需要维护者确认后再合并。
## 为什么拦
PR [#3488](https://github.com/makecindy/cindy/pull/3488) 标题是「pass verified context window to thread config」,但当前 head 触达 maker-core Codex 核心路径约 1375 行(≥150 行核心 + ≥800 行总 diff),命中维护者确认门的 **arch** 触发。
这不是一次机械文档编辑:它改的是 Codex 线程窗口如何被核实、快照、在 setModel / turn 启动时重应用。窗口算错会直接影响压缩边界和模型切换行为。
## 请维护者看什么
1. 是否同意在 Codex 会话内存活期内用线程级快照跟踪已应用的 `model_context_window`,并在目录刷新 / 核实值消失时 unload/resume。
2. 并发路径:`send()` 等待窗口重应用时用户再次切模型,是否会用旧窗口开新模型的 turn(当前仍有一条 bot P1 thread 指出此事)。
3. 改动范围是否应拆小;1375 行是否都服务于同一目的。
## 如何放行
直接在 PR 上 **Approve** 即通过维护者确认门。如果需要作者修改,请 **Request Changes**;作者改完后请再 Approve 一次。
不要只在评论或讨论 issue 里说「可以」——auto-review 只认 PR 上的 Approve。
---
关联 PR:#3488(作者 @yuaiccc);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的技术架构调整,维护者确认后 PR 会恢复推进。
Contributor guide
Research direction
Start by reviewing PR #3488 and its changes in the maker-core Codex path. Check the thread-level snapshot, reapplication during setModel/turn startup, the concurrent send/model-switch case, and whether the 1,375-line diff should be split. Done means a maintainer approves the PR or requests changes; this is not a standalone implementation task.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- ai
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100