revert: undo #782 squash until independent APPROVE by seonghobae · Pull Request #808 · ContextualWisdomLab/LineageWeave

seonghobae · 2026-08-30T04:10:24Z

Summary / prerequisite boundary

Revert the unauthorized #782 squash landing until its valid presentation/read-model deltas can be reviewed and relanded through normal governance. This prerequisite does not move psychometric/statistical truth into LineageWeave. Any surviving descendant must consume released canonical-owner outputs through an explicit contract/ACL and remain a LineageWeave-owned report/read-model/UI projection; duplicated model computation/source truth is not accepted.

Do not mechanically restack the old leftover-map graph directly onto a main that still contains the landing this PR is designed to revert. After this prerequisite lands normally, reconstruct retained descendants in dependency order and remove any canonical-owner duplication instead of merely changing ancestry.

Protected-main convergence

Protected main advanced through #931 to 83eba56149eb802cd63642c507c324c9976ec78e. This branch does not own #931's workflow/test-contract files, so two-parent convergence ef38ddc9cf4f81c6a30050fa2409d294779fbd93 adopts the exact protected blobs while preserving the revert delta. No force push, destructive rebase, or whole-tree replacement was used.

Exact live boundary

protected base: main@83eba56149eb802cd63642c507c324c9976ec78e
exact head: ef38ddc9cf4f81c6a30050fa2409d294779fbd93
PR: open / Draft / mechanically mergeable
repository-local Tests 33889445238, PROV-O 33889445211, Ontology Pages 33889445281: completed skipped under Draft admission
central required Security 33889445237, CodeQL 33889445313, SAST 33889445197: queued
exact-head GREEN is not claimed; qualifying independent current-head APPROVE: none claimed

Keep Draft. Do not self-approve, bypass the product/governance prerequisite, force-push, squash around the governance finding, or close instead of preserving valid successor deltas.

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.

coderabbitai · 2026-08-30T04:11:11Z

📝 Walkthrough

Walkthrough

Leftover map 좌표의 계산, 저장, API 노출, 그래픽 표시, 테스트, 마이그레이션 및 관련 문서가 제거되었습니다. Leftover pair는 공유율, 재구성값, 잔여값 및 거리를 유지합니다. 프로젝트 버전은 2.23.0으로 변경되었습니다.

Changes

Leftover 맵 데이터 계약

Layer / File(s)	Summary
계산 및 pair 계약 축소 `lineageweave/leftover_pairs.py`, `tests/test_leftover_pairs.py`	`LeftoverPair`와 후보 행에서 네 좌표 필드를 제거했습니다. 좌표 지속성 및 좌표 유효성 테스트를 제거했습니다. 거리와 공유율 계산은 유지했습니다.
보고서 저장 및 조회 정리 `backend/app/report_ingestion.py`, `scripts/seed_demo_data.py`, `migrations/0245`, `tests/test_schema.py`, `backend/tests/test_api.py`	보고서 INSERT, SELECT, API payload, 시드 데이터에서 네 좌표 열을 제거했습니다. 좌표 마이그레이션 적용과 관련 스키마 검증을 제거했습니다.

Frontend 표시 계약

Layer / File(s)	Summary
Leftover pair 목록 단순화 `frontend/src/api.ts`, `frontend/src/App.tsx`, `frontend/src/components/LeftoverPairList.tsx`, `frontend/src/components/LeftoverPairList.stories.tsx`, `frontend/src/components/LeftoverPairList.test.tsx`, `frontend/src/App.test.tsx`	좌표 필드와 `leftoverMapAxes` 전달을 제거했습니다. `LeftoverMapPlot` 렌더링, 좌표 배지, 좌표 기반 선택 동작을 제거했습니다. 설명된 leftover share를 다음 동작 문구에서 우선 사용합니다.
LeftoverMapPlot 기능 삭제 `frontend/src/components/LeftoverMapPlot.tsx`, `frontend/src/components/LeftoverMapPlot.css`, `frontend/src/components/LeftoverMapPlot.stories.tsx`, `frontend/src/leftoverMapCoordinates.ts`, `frontend/src/leftoverMapPlotLayout.ts`, `frontend/src/leftoverMapPlotAxisShare.ts`, 관련 테스트	SVG 플롯, 좌표 포맷터, 레이아웃 계산, 축 공유 포맷터, Storybook 스토리와 테스트를 삭제했습니다.
번역 검증 정리 `frontend/src/i18n.ts`, `frontend/src/i18n.test.ts`	좌표, 축, 눈금, 거리, 그래픽 표시 및 좌표 기반 열기 동작의 번역 키와 테스트를 제거했습니다.

문서 및 릴리스 메타데이터

Layer / File(s)	Summary
ADR 및 프로젝트 문서 정리 `AGENTS.md`, `ARCHITECTURE.md`, `CLAUDE.md`, `docs/adr/*`, `docs/product-technical-gap-baseline.md`, `docs/storybook-inventory.md`	ADR 0267~0271과 leftover-map 좌표 및 그래픽 표시 설명을 제거했습니다. 관련 Storybook 항목과 운영 상태 기록을 제거했습니다.
변경 로그 및 버전 갱신 `CHANGELOG.md`, `CHANGELOG.d/leftover-map`, `frontend/package.json`, `pyproject.toml`	Leftover-map 좌표 및 그래픽 표시 변경 로그를 제거했습니다. Frontend와 Python 프로젝트 버전을 2.23.0으로 변경했습니다.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Merge Risk: 🟡 Moderate · up to a10c2

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)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	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 …
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 PR `#782의` squash merge를 되돌리고 독립 승인을 요구하는 주요 변경 사항을 명확하게 요약합니다.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

Create stacked PR
Commit on current branch

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch revert-pr782-unauthorized

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

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

devin-ai-integration

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

seonghobae · 2026-08-30T04:22:33Z

Independent APPROVE is still required on exact head a10c2101 before this revert of the unauthorized #782 squash can land. Devin COMMENT is not independent APPROVE. Do not self-approve. Do not squash-merge without independent APPROVE + exact-head required success. Leftover-map stack #802–#806 stays on feat/leftover-map-coordinates-v2240; this revert only restores protected main.

seonghobae · 2026-08-30T07:18:22Z

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 @ b334b00e plus leftover expected E (v2.36.0) stacked after it. Token X-OAuth-Scopes remains empty; git push and issue comments succeeded this cycle.

seonghobae · 2026-08-30T11:16:19Z

Write retry this cycle (2026-08-30 20:12 KST): revert of unauthorized #782 squash still needs independent APPROVE. Do not self-approve. Do not merge #808 without independent APPROVE. Token collaborator is seonghobae only.

seonghobae · 2026-08-30T11:25:25Z

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.

seonghobae · 2026-08-30T11:28:31Z

Write retry this cycle (2026-08-30 20:26 KST). Head 1af3e53e still lacks independent APPROVE. Do not self-approve. Do not squash-merge this revert without independent APPROVE. Copilot/Devin COMMENT is not independent APPROVE. Only collaborator is seonghobae; no independent reviewer can be requested from this token.

seonghobae · 2026-08-30T11:31:58Z

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.

seonghobae · 2026-08-30T11:42:24Z

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.

…main Adopt protected main #923's docs-only Tests workflow delta while preserving the governance-motivated #782 revert unchanged.

opencode-agent · 2026-09-04T13:38:08Z

Scheduled review-feedback autofix for this PR head.

Head SHA: 04fc5cb79facbca11e9934502b083de57c72bab7

revert: undo #782 squash until independent APPROVE

a10c210

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.

devin-ai-integration Bot reviewed Aug 30, 2026

View reviewed changes

This comment was marked as resolved.

Sign in to view

test(reports): keep leftover story shares coherent

1af3e53

seonghobae enabled auto-merge (squash) August 30, 2026 04:51

seonghobae marked this pull request as draft August 30, 2026 14:16

auto-merge was automatically disabled August 30, 2026 14:16
Pull request was converted to draft

merge(main): converge revert onto current protected head

009d32e

chore(stack): converge unauthorized-coordinate revert with protected …

04fc5cb

…main Adopt protected main #923's docs-only Tests workflow delta while preserving the governance-motivated #782 revert unchanged.

seonghobae added 4 commits September 4, 2026 22:59

chore(stack): adopt protected tests trigger contract

591289f

merge(main): converge revert with protected test-contract head

6269448

chore(main): non-force converge protected #930

938e528

chore(main): non-force converge protected #931

ef38ddc

seonghobae added maintenance priority: medium type: maintenance labels Sep 7, 2026 — with ChatGPT Codex Connector

Navigation Menu

Uh oh!

revert: undo #782 squash until independent APPROVE - #808

revert: undo #782 squash until independent APPROVE#808
seonghobae wants to merge 8 commits into
mainfrom
revert-pr782-unauthorized

seonghobae commented Aug 30, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Aug 30, 2026 •

edited

Loading

Walkthrough

Changes

Uh oh!

devin-ai-integration Bot left a comment

Uh oh!

This comment was marked as resolved.

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

opencode-agent Bot commented Sep 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

seonghobae commented Aug 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary / prerequisite boundary

Protected-main convergence

Exact live boundary

Uh oh!

coderabbitai Bot commented Aug 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

✅ Devin Review: No Issues Found

Uh oh!

This comment was marked as resolved.

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

seonghobae commented Aug 30, 2026

Uh oh!

opencode-agent Bot commented Sep 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

seonghobae commented Aug 30, 2026 •

edited

Loading

coderabbitai Bot commented Aug 30, 2026 •

edited

Loading