docs(cursor): record the stack landing SHAs and close residual R5

[lidge-jun]

Summary

The three Cursor unified-identity PRs are on dev. This records where they landed so the devlog unit is not the only place without the SHAs.

PR	merged head	squash commit
#3222 umbrella seed + labels	419e89625	7aa64bb0b
#3225 cursor-variant Fast toggle	61d6d38d9	83838e7fa
#3233 fastMode -fast listing	f26169712	8d2dd6639

It also closes residual R5. That one was worth writing down: while landing the stack, tests/agent-task-recovery.test.ts was red, and it would have been easy to attribute it to this work. It reproduced on a clean origin/dev checkout, so it was dev's own — and dev's #3242 (revert of #3239/#3240) then fixed it. The file is 19/19 on the landed dev, so the follow-up fix PR this unit was going to open is unnecessary.

One practical note recorded for the next stacked chain: each child was re-stacked by cherry-picking its unique commits onto the landed parent, not by rebasing. A parent squash absorbs the child's content under a different commit id, so a plain rebase conflicts against work already in the base.

Verification

Docs-only. Ancestry proven for all three:

git fetch origin dev
git merge-base --is-ancestor 7aa64bb0b FETCH_HEAD   -> ANCESTOR OK
git merge-base --is-ancestor 83838e7fa FETCH_HEAD   -> ANCESTOR OK
git merge-base --is-ancestor 8d2dd6639 FETCH_HEAD   -> ANCESTOR OK

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.

Summary by CodeRabbit

• Documentation
  • Added a record of a previously observed test failure and its resolution.
  • Documented the landing of related changes, including merge order and restacking details.

[github-actions]

✅ Deterministic PR hygiene checks passed.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 6779d75a-733d-4fc4-816d-34a05c76efba

📥 Commits

Reviewing files that changed from the base of the PR and between 8d2dd66 and 2f02f05.

📒 Files selected for processing (1)

• devlog/_plan/260902_cursor_unified_identity/040_residuals.md

---

📝 Walkthrough

Walkthrough

The residuals document adds an R5 entry for a pre-existing test failure and records its closure by revert. It also adds the 2026-09-02 landing record for PRs #3222, #3225, and #3233.

Changes

Landing documentation

Layer / File(s)	Summary
Residual and landing record devlog/_plan/260902_cursor_unified_identity/040_residuals.md	Adds the R5 residual with the failure cause and revert closure. Adds merged heads, squash commits, review status, and cherry-pick restacking details for the landing record.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: ingwannu

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/cursor-landing-record

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[lidge-jun]

리뷰 · 우선순위 73 / 80

이 PR은 런타임 코드를 건드리지 않습니다. Cursor unified-identity 스택 (#3222 umbrella seed + labels, #3225 cursor-variant Fast toggle, #3233 fastMode -fast listing)이 이미 dev에 들어간 뒤, 그 착지 위치를 SHA로 남기고 residual R5를 닫는 문서 전용 기록입니다. 파일은 하나뿐입니다: devlog/_plan/260902_cursor_unified_identity/040_residuals.md (+54 / −0).

현재 로컬 dev HEAD는 7d25f99660e95519a114a6d90b2d7a30c5870595 (docs(cursor): dashboard section and read-only surface note (#3249)) 입니다. 이 PR 자체는 이미 21416a7af 로 머지되어 있고, 그 위에 #3244 클로즈아웃 검증, #3247 읽기 전용 Private Inference 상태 라우트, #3248 GUI Cursor 통합 탭, #3249 대시보드 문서가 이어서 올라간 상태입니다. 즉 트리거 시점의 “PR opened” 와 달리, 리뷰 시점에는 이미 landed 입니다.

PR이 하는 일을 초등학생도 이해할 수 있게 말하면 이렇습니다. 세 장의 코드 PR이 먼저 dev에 들어갔습니다. 그런데 계획 문서(residuals)에는 “어디에 착지했는지” SHA가 비어 있으면, 나중에 누가 봐도 “진짜로 들어갔는지 / 어떤 커밋인지”를 다시 찾아다녀야 합니다. 그래서 이 PR은 표로 merged head와 squash commit을 적고, git merge-base --is-ancestor 로 조상 관계를 증명했다고 적습니다. 또한 랜딩 중에 tests/agent-task-recovery.test.ts 가 빨갛게 보였는데, 그걸 Cursor identity 작업 탓으로 오해하지 말라고 R5로 적어 둡니다. 그 빨강은 깨끗한 origin/dev (b54508c8c, #3241) 에서도 재현됐고, 이후 #3242가 #3239/#3240 합성 네이티브 체인을 되돌리면서 파일은 19 pass / 0 fail로 회복됐다는 이야기도 같이 닫습니다.

실무적으로 아주 중요한 한 줄이 있습니다. 자식 PR을 부모 위에 다시 올릴 때 rebase가 아니라 cherry-pick 했다는 기록입니다. 부모를 squash merge하면 자식이 이미 들고 있던 내용이 다른 커밋 id로 base에 흡수되므로, 그냥 rebase하면 “이미 들어간 작업”과 충돌합니다. #3225에서 그 함정을 실제로 밟았다고 적어 둔 것은, 다음 stacked chain을 다루는 사람(그리고 봇)에게 바로 도움이 됩니다.

현재 dev 방향과 맞춰 보면, 스냅샷이 가리키는 Cursor Private Inference + Fast identity train (#3222/#3225/#3230/#3231/#3233/#3218 + #3243/#3244/#3247/#3248/#3249) 한가운데에 정확히 들어맞는 문서입니다. types.ts/config.ts 분할 캠페인과도 무관하고, 중복 close-don't-rebase 대상도 아닙니다. 코드 위험이 없고, 감사 추적(audit trail) 가치가 큽니다. 이미 머지된 뒤라 “머지할지” 판단은 끝났고, 이 리뷰는 내용이 HEAD와 맞는지 / 후속 #3244와 모순이 없는지를 확인하는 사후 검증에 가깝습니다.

로컬에서 조상 검증을 다시 돌렸습니다. 7aa64bb0b (#3222), 83838e7fa (#3225), 8d2dd6639 (#3233), 21416a7af (#3243) 모두 현재 HEAD의 ancestor입니다. 체크아웃의 040_residuals.md 에는 이 PR이 넣은 R5·Landing record 뒤에 #3244가 붙인 Closeout verification 블록까지 이어져 있어, 문서 흐름이 자연스럽게 닫혀 있습니다.

라인 49-82 - R5 진단(502가 encrypted-child 400 게이트를 우회한 합성 체인 탓)과 #3242 클로즈 서술이 현재 dev invariant(encryptedV2SubagentRecovery)와 맞습니다. 리뷰 시점에 테스트를 다시 돌리지는 않았지만, 원인→되돌림→불필요해진 follow-up PR 서술이 논리적으로 일관됩니다.
라인 89-93 - 랜딩 표의 squash SHA 세 개가 현재 HEAD ancestor로 확인됐습니다. 표 자체는 짧지 않은 full SHA를 쓰고 있어 추적에 충분합니다.
라인 95-96 - --admin 이 review requirement만 풀었고 red check는 우회하지 않았다는 문장. 감사 관점에서 중요하니 남겨 둔 것이 좋습니다. 이 리뷰가 그 주장을 CI API로 재검증하지는 않았습니다(문서 PR·이미 머지).
라인 98-101 - cherry-pick restack 규칙 기록. 다음 스택에 재사용할 운영 메모로 가치가 큽니다. 코드 변경이 아니라서 types/config 분할과 충돌하지 않습니다.
경로 040_residuals.md / R1–R4 - 이 PR이 손대지 않은 기존 residual. R4의 다른 pre-existing red 목록은 여전히 열려 있는 “기록”이지 이 PR 범위가 아닙니다. 혼동 없음.
경로 전체 - 런타임/테스트/설정 파일 변경 없음. 문서 전용 범위가 지켜졌습니다.

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

• 이미 21416a7af 로 머지됐고 docs(cursor): closeout verification on the landed dev #3244 클로즈아웃까지 올라간 상태라, 추가 코드 액션은 사실상 없습니다. 남은 건 remote branch 삭제 거부(ruleset) 정도인데 문서에도 expected로 적혀 있습니다.
• R5가 가리킨 agent-task-recovery 빨강이 이후 다른 PR에서 다시 보이는지 (회귀 감시)만 가끔 보면 됩니다. 이 문서 PR이 고칠 일은 아닙니다.
• stacked PR 운영 규칙(부모 squash 후 자식은 rebase 말고 cherry-pick)을 AGENTS/기여 가이드 한곳에 승격할지 여부는 별도 판단. 이 residuals에만 있으면 다음 기여자가 못 볼 수 있습니다.

너의 추천
추가 수정·리베이스 없이 그대로 두기. 이미 dev에 올바르게 착지했고 SHA 조상 검증도 통과했습니다. 후속 문서(#3244)와 Private Inference/GUI 문서(#3247–#3249)가 이어서 닫힌 흐름과도 맞습니다. 라벨 변경·재오픈·후속 fix PR은 필요 없습니다. 다음 stacked Cursor/identity 작업이 있으면 이 파일의 cherry-pick 메모를 먼저 읽고 진행하면 됩니다.

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