Skip to content

devlog: wp14 carry record and wp15 disposition - #3221

Merged
lidge-jun merged 2 commits into
devfrom
codex/devlog-wp14-wp15
Sep 1, 2026
Merged

devlog: wp14 carry record and wp15 disposition#3221
lidge-jun merged 2 commits into
devfrom
codex/devlog-wp14-wp15

Conversation

@lidge-jun

Copy link
Copy Markdown
Owner

Summary

Verification

  • bun test tests/repo-hygiene.test.ts -> 12 pass. Docs-only.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

@lidge-jun
lidge-jun requested a review from Ingwannu as a code owner September 1, 2026 20:56
@lidge-jun
lidge-jun merged commit 85f7ef9 into dev Sep 1, 2026
@lidge-jun
lidge-jun deleted the codex/devlog-wp14-wp15 branch September 1, 2026 20:57
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Deterministic PR hygiene checks passed.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6f0e0407ae

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

rebase and security pass; not this batch.

## Executed
PR #2496 closed 2026-09-02 with the rationale above; #2495 commented with reopen conditions

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Move the completed backlog unit to _fin

This records execution of wp15, the final item listed in 000_scope.md, while the preceding fourteen items already have terminal outcomes, so the overall backlog unit is now complete but remains under devlog/_plan. That directory represents open work; move the whole 260902_nonbug_adoption_backlog directory to devlog/_fin as part of this closeout so completed work is not presented as active.

AGENTS.md reference: AGENTS.md:L83-L86

Useful? React with 👍 / 👎.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-01T21:00:29.097753Z 6f0e040 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@lidge-jun

Copy link
Copy Markdown
Owner Author

리뷰 · 우선순위 34 / 80

이 PR은 런타임 코드가 아니라 devlog/_plan/260902_nonbug_adoption_backlog/ 아래 작업 패키지 기록입니다. 제목에는 wp14 캐리와 wp15 disposition이 같이 들어가 있지만, 실제 합본 diff는 wp15 쪽 두 파일뿐입니다. 150_wp15_plaintext_v2_disposition.md## Executed 절 세 줄을 더하고, 151_wp15_audit_r1_synthesis.md를 새로 넣습니다. 추가 9줄, 삭제 0줄. src/·테스트·타입·설정 파일은 건드리지 않습니다. types.ts/config.ts 분리 캠페인과도 충돌하지 않습니다.

현재 dev 헤드 d23eab43a는 이미 이 PR을 조상으로 갖고 있습니다. 합본 커밋은 85f7ef92a이고, 그 위에 #3224(codex-spark reserved functions 그룹 유지)까지 올라간 상태입니다. 그래서 이 리뷰는 “합칠까 말까”가 아니라 “이미 올라간 기록 문서가 현재 트리 진실과 맞는지”를 확인하는 사후 리뷰입니다. 문서-only라서 빌드·타입체크·테스트 경로가 읽지 않는다는 PR 본문 설명과도 맞습니다. bun test tests/repo-hygiene.test.ts 12 pass라는 검증도 문서 PR 규모에 어울립니다.

wp15의 내용은 이슈 #2495와 PR #2496에 대한 처분입니다. 조사 기록에 따르면 #2496 헤드 e4b88af4f는 +2729/18파일짜리 프로토콜 재작성으로, 문서화되지 않은 ChatGPT/Codex 동작(Fernet을 우회하려는 예약 네임스페이스·툴 이름 바꾸기, encrypted_function_args: [] 수신 경로)에 기대고 있습니다. exact-head CI는 포크 승인 때문에 한 번도 안 돌았고, 마지막 실행 스위트는 PR 전용 assertion 두 개에서 빨갛습니다. 예전 헤드의 CHANGES_REQUESTED도 재리뷰되지 않았습니다. 그래서 “작은 슬라이스로 이슈를 닫을 수 없다 / 정직한 합본 경로는 메인테이너 리베이스+보안 패스 8–12시간 / 이번 배치에서는 안 한다”는 disposition이 나왔고, 실행 절로 PR #2496을 닫고 #2495는 재오픈 조건만 남긴 채 열어 둔다고 적습니다.

실제로 GitHub 상태도 그 서술과 같습니다. #2496은 CLOSED이고, #2495는 아직 OPEN입니다. 재오픈 조건으로 적힌 “메인테이너 소유 리베이스, exact-head 그린 런, plaintext 보존에 대한 보안 패스”는 이후 기여자·봇이 같은 대형 재작성을 다시 밀어 넣을 때 기준점이 됩니다. 문서가 없으면 “왜 닫았는지”가 이슈 코멘트에만 흩어지고, 비-버그 채택 백로그 표(000_scope.md의 wp15 행)와도 어긋나기 쉽습니다. 그런 점에서 이 기록은 코드 출시를 막지도 열지도 않지만, 릴리즈 트레인 의사결정의 흔적을 dev에 고정합니다.

제목의 wp14는 같은 폴더의 140_wp14_upstream_ws_carry.md / 141_wp14_audit_r1_synthesis.md를 가리킵니다. wp14 자체(#2816 캐리 of #2817, 옵트인 upstream Responses WebSocket)는 이미 #3216으로 dev에 들어가 있고, 스냅샷 invariant에도 responsesUpstreamWebsocket으로 잡혀 있습니다. 그런데 이 PR의 합본 파일 목록에는 140/141이 없습니다. 제목만 보면 wp14 캐리 기록까지 이 PR이 넣은 것처럼 보이지만, 페이로드는 wp15 실행 기록뿐입니다. 나중에 로그를 훑는 사람이 제목만 보고 “#3221이 wp14 문서를 추가했다”고 오해할 수 있습니다. 점수를 34로 둔 이유는 문서 가치가 분명하지만 출시를 막거나 푸는 코드가 아니고, 제목·본문·실제 diff의 범위가 살짝 어긋나며, 이미 합쳐진 뒤라 추가 조치가 거의 없기 때문입니다.

devlog/_plan/260902_nonbug_adoption_backlog/150_wp15_plaintext_v2_disposition.md 15-17행 - Executed 절이 disposition과 일치한다. #2496 닫힘·#2495 열림·재오픈 조건 세 가지가 한 문단으로 묶여 있어 읽기 좋다.
devlog/_plan/260902_nonbug_adoption_backlog/151_wp15_audit_r1_synthesis.md 전체 - 감사 요약이 disposition과 같은 결론(PR 닫기, 이슈 유지)을 Adopted로 박아 둔다. 중복이지만 “조사 → 결정 → 실행” 체인으로는 자연스럽다.
경로/제목 - PR 제목의 “wp14 carry record”는 이 합본 diff에 파일이 없다. wp14 문서는 트리에 이미 있으나 #3221 페이로드는 아니다. 히스토리 오해를 줄이려면 제목을 wp15 실행 기록 중심으로 읽거나, 후속 노트에 “wp14는 선행 기록 / 이 커밋은 wp15 Executed”라고 한 줄 보강하면 된다.
전체 - src 변경 없음. 보안·시크릿·unsafe default 경로 없음. 중복 기능 PR 아님. 닫을 leftover 원본 PR도 이 문서 PR 자체에는 없다(#2496은 이미 처분 완료).

메인테이너의 판단이 필요한 지점

  • 이미 합쳐진 문서 PR에 대해 제목/범위 불일치(wp14 언급 vs wp15-only diff)를 후속 한 줄 정정할지, 그대로 둘지
  • Feature: opt-in plaintext V2 collaboration rewrite for native-to-routed sub-agents #2495 재오픈 조건을 이슈 본문/라벨에도 고정할지, 지금처럼 코멘트+devlog만으로 둘지
  • wp14/wp15가 nonbug adoption backlog의 끝단인지, 같은 폴더에 남은 open 항목이 더 있는지 인덱스(000_scope.md)와 한 번 더 맞출지
  • 이 기록 PR에 라벨을 붙일 필요는 없어 보인다(지시: 라벨 교체 금지)

너의 추천
추가 코드 작업은 하지 말자. 이미 85f7ef92adev에 있고 #2496 닫힘/#2495 열림도 문서와 일치한다. 후속이 필요하면 #2495에 “재오픈 조건은 150_wp15 disposition Executed 절과 동일” 한 줄만 이슈에 고정해 두면 충분하다. 제목의 wp14 오해는 치명적이지 않으니 굳이 amend/새 PR로 고치지 말고, 다음 backlog 인덱스 정리 때 “#3221 = wp15 execution record”로 표기하면 된다. 라벨은 바꾸지 말 것.

이 댓글은 grok-bot이 작성했습니다

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