docs(inspector): reconcile plan with PR TableProApp#1463 collapseBehavior pivot and conflict resolution

Author: J2TeamNNL

.docs/inspector/README.md

@@ -6,23 +6,35 @@ Scope: `fix(inspector)`
 
 Toggling inspector pane (right panel) trong cửa sổ nhỏ gây nội dung tràn ra ngoài frame. Ba pane có tổng `minimumThickness` (~954pt) vượt `window.minSize.width` (720pt).
 
-## Fix
-
-`MainSplitViewController` recompute `window.minSize.width` động mỗi khi pane visibility thay đổi. Nếu cửa sổ đang nhỏ hơn min mới, cửa sổ tự expand.
+> **Reconcile 2026-06-09 (verified vs PR thật).** PR đã PIVOT. Commit `58cd1102`
+> ("refactor(inspector): grow the window via NSSplitViewItem.collapseBehavior instead of
+> custom min-size code") **bỏ hẳn** toàn bộ cơ chế recompute: xóa `recomputeWindowMinSize()`,
+> xóa override `splitViewDidResizeSubviews`, không còn `PaneMinimum`/`resolvedContentMinSize`/
+> `originalContentMinSize`, và **không còn** test `MainSplitViewControllerWindowMinimumSizeTests`.
+> Cách cuối: set `inspectorSplitItem.collapseBehavior = .preferResizingSplitViewWithFixedSiblings`
+> để AppKit tự nới window khi inspector bật. Floor 720×480 vẫn nằm tĩnh ở `TabWindowController`.
+> Toàn bộ kế hoạch "GỘP vào recomputeWindowMinSize" + ADR-006/007/010 bên dưới là **moot**
+> (xem `decisions.md` ADR-011).
+>
+> Conflict đã resolve (merge `upstream/main` vào `fix/inspector`, commit local `69e3555e`):
+> hết conflict, net diff vs main = chỉ `MainSplitViewController.swift` (collapseBehavior) + 1
+> dòng `CHANGELOG.md`. 3 fix kèm theo (mcp/connections/editor) bị bản trên main (#1587) thay hết.
+> **Push đang chờ**: bị GitHub chặn vì commit mang email `longnn@senprints.com` (account bật
+> block push lộ email). Build/lint **chưa verify cục bộ** (máy không có Xcode/swiftlint) → CI là cổng.
+
+## Fix (cách cuối — collapseBehavior)
+
+`inspectorSplitItem.collapseBehavior = .preferResizingSplitViewWithFixedSiblings`: khi inspector
+bật, AppKit nới window thay vì ép detail pane co lại. Bỏ toàn bộ code recompute `window.minSize`
+thủ công. Floor cố định 720×480 vẫn do `TabWindowController` set.
 
 ## Status
 
-**OPEN-fork — PR [#1463](https://github.com/TableProApp/TablePro/pull/1463)**, branch `fix/inspector`. Chưa merge.
-
-> Verified 2026-05-30: "DONE — 2026-05-25" trước đây là **superseded**. PR vẫn open.
+**OPEN — PR [#1463](https://github.com/TableProApp/TablePro/pull/1463)**, branch `fix/inspector`.
+Conflict đã resolve cục bộ; chờ push (email-privacy block) + CI build.
 
-**Blocker (CONFLICTING logic thật)**: main đã có `recomputeWindowMinSize()`
-(`MainSplitViewController.swift:516`, call site `:185,:217,:478,:484`); PR thêm
-`recomputeWindowMinimumSize()` (tên mới) + `PaneMinimum`/`resolvedContentMinSize`/
-`originalContentMinSize`/`setCollapsed`. **CRITICAL fix**: GỘP logic PR vào method cũ
-`:516`, không tạo method thứ hai; không thêm override `splitViewDidResizeSubviews()`
-(đã có `:183`). Commit atomic. Plus drop noise (`.gitignore` + `CLAUDE.md:171`).
-Chi tiết: `context.md` mục "Open fork PR #1463", `tasks.md` agent checklist.
+> Verified 2026-05-30: "DONE — 2026-05-25" trước đây là **superseded**.
+> Reconcile 2026-06-09: kế hoạch merge "GỘP vào recomputeWindowMinSize" cũng **superseded** — PR pivot sang collapseBehavior.
 
 ## Files trong folder này
 
@@ -40,5 +52,5 @@ Chi tiết: `context.md` mục "Open fork PR #1463", `tasks.md` agent checklist.
 
 | File | Vai trò |
 |---|---|
-| `TablePro/Core/Services/Infrastructure/MainSplitViewController.swift` | `recomputeWindowMinSize()` (def `:516`), `showInspector()` (`:474`), `hideInspector()` (`:481`), `splitViewDidResizeSubviews` (`:183`), `viewWillAppear` (`:194`); call sites `:185,:217,:478,:484` (verified 2026-05-30) |
-| `TablePro/Core/Services/Infrastructure/TabWindowController.swift` | `window.minSize` baseline (720×480) |
+| `TablePro/Core/Services/Infrastructure/MainSplitViewController.swift` | `inspectorSplitItem.collapseBehavior` (cách cuối), `setCollapsed(_:for:)` helper, `showInspector()`, `hideInspector()`. `recomputeWindowMinSize()` + `splitViewDidResizeSubviews` đã **bị xóa** trong commit `58cd1102`. |
+| `TablePro/Core/Services/Infrastructure/TabWindowController.swift` | `window.minSize` baseline 720×480 (`:75`) — floor tĩnh duy nhất sau khi bỏ recompute |

.docs/inspector/brief.md

@@ -1,5 +1,11 @@
 # Inspector pane overflow — brief
 
+> **Reconcile 2026-06-09.** Cách fix cuối của PR đã đổi sang native `NSSplitViewItem.collapseBehavior`
+> (commit `58cd1102`), **bỏ hẳn** mọi thứ về `recomputeWindowMinSize`/`PaneMinimum`/
+> `resolvedContentMinSize`/`originalContentMinSize` và geometry test. Mục "Open fork PR #1463",
+> "Kết quả", và bảng pane-sum bên dưới mô tả cách CŨ, nay **superseded**. Floor 720×480 vẫn tĩnh ở
+> `TabWindowController:75`. Chi tiết: `README.md`, `decisions.md` ADR-011, `tasks.md`.
+
 ## Vấn đề
 
 Khi user toggle inspector pane (panel bên phải) trong một cửa sổ hẹp (~720pt), nội dung và toolbar bị tràn ra ngoài viền window. AppKit cố gắng fit 954pt content vào 720pt frame.

.docs/inspector/changelog.md

@@ -1,5 +1,18 @@
 # inspector — changelog
 
+## 2026-06-09 — PR pivot sang collapseBehavior + conflict resolved
+
+| Date | Event |
+|---|---|
+| ~2026-06 | PR pivot: commit `58cd1102` ("refactor(inspector): grow the window via NSSplitViewItem.collapseBehavior instead of custom min-size code") **xóa hẳn** `recomputeWindowMinSize()`, override `splitViewDidResizeSubviews`, `PaneMinimum`/`resolvedContentMinSize`/`originalContentMinSize`, và test `MainSplitViewControllerWindowMinimumSizeTests`. Thay bằng `inspectorSplitItem.collapseBehavior = .preferResizingSplitViewWithFixedSiblings` + helper `setCollapsed(_:for:)`. |
+| 2026-06-09 | Resolve conflict: merge `upstream/main` vào `fix/inspector` (commit local `69e3555e`). Hết conflict. 3 conflict file: `SQLContextAnalyzer.swift` + `PasswordSourceResolver.swift` lấy bản main (bị #1581/#1587/#1601 thay), `CHANGELOG.md` giữ wording main + chèn lại dòng inspector. Net diff vs main = `MainSplitViewController.swift` (collapseBehavior) + 1 dòng CHANGELOG. |
+| 2026-06-09 | Cập nhật PR description cho khớp collapseBehavior (bỏ mô tả recomputeWindowMinimumSize + test cũ). |
+| 2026-06-09 | Push **chưa xong**: GitHub chặn vì commit mang email `longnn@senprints.com` (account bật block push lộ email). Build/lint chưa verify cục bộ (không có Xcode/swiftlint) → CI là cổng. |
+
+> Supersede note: toàn bộ timeline 2026-05-25/28/30 bên dưới dựa trên cách `recomputeWindowMinSize`
+> (recompute `window.minSize` thủ công). PR đã pivot sang collapseBehavior nên các fix đó **moot**.
+> Giữ lại để tham chiếu lịch sử.
+
 ## 2026-05-30 — Reconcile vs main (verified)
 
 | Date | Event |

.docs/inspector/context.md

@@ -1,5 +1,13 @@
 # Inspector pane overflow — case study chi tiết
 
+> **Reconcile 2026-06-09.** Toàn bộ phần "Fix" (snippet `recomputeWindowMinSize`, 4 call site,
+> bảng scenario minSize) và mục "Open fork PR #1463 — code review & conflict resolution" bên dưới
+> mô tả cách CŨ. PR đã pivot sang `NSSplitViewItem.collapseBehavior = .preferResizingSplitViewWithFixedSiblings`
+> (commit `58cd1102`) và **xóa hết** code recompute + `PaneMinimum`/`resolvedContentMinSize`/
+> `originalContentMinSize` + geometry test. Bug Codex P2 "base-min-size chỉ tăng" cũng **moot** (không
+> còn code đọc `window.minSize` mutate). Symptom + root cause (TabWindowController set minSize tĩnh,
+> các pane minimumThickness) vẫn đúng. Chi tiết cách cuối: `decisions.md` ADR-011.
+
 ## Symptom
 
 Toggle inspector pane khi window width ~720pt → content area bị squeeze/tràn, toolbar controls bị che.

.docs/inspector/decisions.md

@@ -65,6 +65,10 @@
 
 # Open fork PR #1463 — decisions (2026-05-30)
 
+> **Superseded 2026-06-09 bởi ADR-011.** ADR-005..010 thiết kế quanh việc gộp logic vào
+> `recomputeWindowMinSize`. PR đã pivot sang `collapseBehavior` (commit `58cd1102`) và xóa
+> hết code recompute, nên ADR-006/008/010 moot, ADR-007 bị đảo. Giữ để tham chiếu lịch sử.
+
 ## ADR-005: Drop noise `.gitignore` / `CLAUDE.md`
 **Context**: upstream/main đã có entries; PR add lại = conflict giả.
 **Decision**: reset 2 file về upstream khi rebase.
@@ -109,3 +113,28 @@ không bị overwrite bởi giá trị widen). Body `recomputeWindowMinSize()` s
 **Verify khi fix**: confirm `originalContentMinSize` của PR capture đúng base gốc; test cycle
 show → hide phải relax về 720pt (fail nếu base đọc từ minSize mutate).
 **Trade-off**: không — đây là điều kiện đúng đắn của tính năng, không phải optional.
+
+---
+
+## ADR-011: Dùng `NSSplitViewItem.collapseBehavior` native, bỏ toàn bộ recompute thủ công (supersede ADR-001/002/006/007/008/010)
+**Date**: 2026-06-09
+**Context**: ADR-001..010 thiết kế quanh việc tính `window.minSize` thủ công
+(`recomputeWindowMinSize` + `PaneMinimum`/`resolvedContentMinSize`/`originalContentMinSize` +
+4 call site + override `splitViewDidResizeSubviews`). Cách này phức tạp, có bug "floor chỉ
+tăng" (ADR-010), và cần geometry test riêng. Commit `58cd1102` của PR chọn hướng khác:
+AppKit đã có sẵn `NSSplitViewItem.collapseBehavior` để xử lý đúng tình huống này.
+**Decision**: set `inspectorSplitItem.collapseBehavior = .preferResizingSplitViewWithFixedSiblings`.
+Khi inspector mở rộng, AppKit nới chính cửa sổ (resize split view với fixed siblings) thay vì ép
+detail pane co. **Xóa hẳn** `recomputeWindowMinSize()`, override `splitViewDidResizeSubviews`,
+`PaneMinimum`, `resolvedContentMinSize`, `originalContentMinSize`. Floor 720×480 vẫn do
+`TabWindowController:75` set tĩnh. Helper `setCollapsed(_:for:)` chỉ còn nhiệm vụ animate collapse
+khi `window.isVisible` (dùng chung cho sidebar + inspector).
+**Lý do**: native hơn (đúng nguyên tắc "Native only" của CLAUDE.md); ít code hơn; không còn bug
+floor-chỉ-tăng vì không còn đọc `window.minSize` mutate; bỏ được geometry test dễ vỡ.
+**Hệ quả với các ADR cũ**:
+- ADR-001 (dynamic minSize), ADR-002 (4 call site), ADR-006 (gộp method), ADR-008 (commit atomic gộp), ADR-010 (baseline immutable): **moot** — không còn recompute.
+- ADR-007 (giữ geometry test): **đảo** — test bị xóa cùng code; không có unit test mới (resize/animation không deterministic dưới test harness).
+- ADR-009 (`setCollapsed` pre-hook): **đổi vai** — `setCollapsed` vẫn còn nhưng chỉ animate-when-visible, không còn pre-widen/pre-shrink.
+- ADR-003 (dùng `minSize` không `contentMinSize`): vẫn đúng — floor tĩnh dùng `window.minSize`.
+**Trade-off**: dựa vào hành vi AppKit thay vì geometry tự tính → khó unit-test; bù lại đơn giản và
+đúng native. Cần manual UI verify + CI build (chưa verify cục bộ: máy không có Xcode).

.docs/inspector/flow.md

@@ -1,5 +1,18 @@
 # Inspector pane overflow — flow
 
+> **Reconcile 2026-06-09.** Các diagram bên dưới mô tả luồng `recomputeWindowMinSize` (cách CŨ) và
+> "merged sizing flow" (kế hoạch GỘP). PR đã pivot sang `collapseBehavior` (commit `58cd1102`), bỏ
+> hết recompute → các diagram đó **superseded**. Luồng thực tế hiện tại:
+>
+> ```mermaid
+> flowchart TD
+>     A[Setup: inspectorSplitItem.collapseBehavior<br/>= .preferResizingSplitViewWithFixedSiblings] --> B[User toggle inspector]
+>     B --> C[showInspector / hideInspector]
+>     C --> D[setCollapsed _:for_:<br/>animate isCollapsed CHỈ khi window.isVisible]
+>     D --> E[AppKit tự nới/thu window<br/>để fit, không ép detail pane]
+>     E --> F[Floor 720×480 tĩnh từ TabWindowController:75]
+> ```
+
 ## Luồng toggle inspector
 
 ```mermaid