revert: undo #782 squash until independent APPROVE - #808
Conversation
The leftover-map coordinates stack (v2.24–v2.28) was squash-merged to main without an independent APPROVE. Restore main to fc13aca (v2.23.0) so the leftover-map PRs stay review-gated. Do not self-approve.
📝 WalkthroughWalkthroughLeftover map 좌표의 계산, 저장, API 노출, 그래픽 표시, 테스트, 마이그레이션 및 관련 문서가 제거되었습니다. Leftover pair는 공유율, 재구성값, 잔여값 및 거리를 유지합니다. 프로젝트 버전은 2.23.0으로 변경되었습니다. ChangesLeftover 맵 데이터 계약
Frontend 표시 계약
문서 및 릴리스 메타데이터
Estimated code review effort: 3 (Moderate) | ~25 minutes Merge Risk: 🟡 Moderate · up to This revert removes coordinate fields from persisted reports and API responses; without a documented compatibility path for existing databases and clients, rollout or rollback may mishandle historical reports. A Storybook fixture also has inconsistent share values. Merge should wait for compatibility and fixture corrections, or explicit owner acceptance. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 93.10% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 29 functions across 7 files. (8 skipped: 8 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
Independent APPROVE is still required on exact head |
|
Cycle write retry 2026-08-30T07:18Z: #808 still has no independent APPROVE (Devin/CodeRabbit comments are not APPROVE). Leave open. Do not squash-merge this revert without independent APPROVE. Do not self-approve. Leftover stack continues on #809 @ |
|
Cycle write retry 2026-08-30T20:12+09:00. Revert of unauthorized #782 squash still needs independent collaborator APPROVE before merge. Do not self-approve. |
|
Write retry this cycle (2026-08-30 20:26 KST). Head |
|
Cycle 2026-08-30T11:26Z write retry: revert of unauthorized leftover-coordinates squash (#808 @ 1af3e53 → main cb187ca) stays open. mergeable_state blocked. Independent APPROVE still required; Copilot/Devin COMMENT is not independent APPROVE. Do not merge. Do not self-approve. Protected main remains cb187ca after the unauthorized #782 squash. Issues #79 and #87 stay open. NVIDIA_NIM_API_KEY only; synthetic fixtures only. |
|
Cycle 2026-08-30T11:42Z write retry: revert of unauthorized leftover-coordinates squash (#808 @ 1af3e53 → main cb187ca) stays open. Independent APPROVE still required; Copilot/Devin COMMENT is not independent APPROVE. Do not merge. Do not self-approve. Leftover-map stack now includes #815 (incomplete item coverage / v2.41.0) on #814. Issues #79 and #87 stay open. NVIDIA_NIM_API_KEY only; synthetic fixtures only. |
Pull request was converted to draft
Summary
Reverts the 2026-08-30 squash-merge of leftover-map coordinates (v2.24–v2.28, #782 @
9bdd3e4d) because that merge lacked the independent APPROVE required by live governance and its Strix evidence had failed closed. The revert preserves the #782 head branch so the leftover-map stack can be reviewed and relanded normally rather than treating predecessor evidence as current.Do not self-approve, bypass, force-push, or squash-merge this revert without the required current-head approval and terminal gates.
Exact live identity
main@3f61c8242b9c02dec307a7396e83e28f7cdd9f3d009d32e61d796335807a2ed3f6b4da730e94428adocs/index.mdthrough docs: add repository Pages landing source #905; that disjoint delta was incorporated by an ordinary non-destructive two-parent merge commit, preserving the revert tree plus the current Pages landing sourceVerification / governance
The convergence changed the exact head, so all predecessor checks and reviews are historical. Fresh successor-head workflows/check-runs have started and remain non-passing until terminal. Keep Draft until the unchanged current head satisfies repository tests, security/supply-chain requirements, review-thread resolution, and the live ruleset's independent approval.
Evidence boundary
No real records or credentials. Synthetic fixtures only.