fix(codex): follow active conversations after rollback - #139
Open
Yoonwoo-Ha wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
최근 Codex CLI는 TUI별 프로세스가 아니라 공유
app-server가 rollout 파일을 열어 둡니다. 이 상태에서 사용자가Esc로 이전 시점으로 돌아가 새 분기를 시작하면:forked_from_id를 가진 새 대화에 기록되어같은 작업 폴더에 Codex 창이 여러 개 있으면 cwd나 최신 생성 시각만으로는 어느 분기인지 안전하게 구분할 수도 없습니다.
변경 내용
session_meta.forked_from_id계보가 현재 pane의 기존 대화로 이어지는 후보만 남깁니다.시간창이나 cwd만으로 연결하지 않으며, 모호하거나 파일/DB 형식이 예상과 다르면 기존 연결을 유지하고 실패를 닫힌 상태로 처리합니다.
안전 범위
이 경로는 대화 표시용 추론으로만 유지됩니다. 직접적인 provider identity receipt가 아니므로
observed권한으로 승격하지 않고, 승인이나 session-addressed 입력 권한도 부여하지 않습니다. Codex 설정·터미널 제목·tmux pane 태그는 변경하지 않습니다.자원 사용도 다음처럼 제한했습니다.
확인
npm test: 720 tests 통과npm run typecheck,npm run check:core,npm run lint,npm run check:identity,npm run build통과npm run verify는 변경하지 않은 upstreampackage-lock.json의 현재 npm advisory 4건(hono,js-yaml,multer,sharp) 때문에 audit 단계에서 중단됩니다. 이 PR은 의존성 파일을 변경하지 않습니다.제한 사항
새 분기의 첫 답변이 화면과 transcript를 안전하게 대조할 만큼 충분한 길이가 된 뒤 연결됩니다. 너무 짧거나 여러 후보가 같은 문장을 포함하면 오연결하지 않고 다음 명확한 답변까지 기존 대화를 유지합니다.