makecindy / makecindy/cindy

维护者确认:PR #3488 架构改动(Codex 核心路径 / 大 diff)

Open
#3,536 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.