chore(notes): collect loose root docs into notes/; add reconcile edge-cases note

Move all loose root-level .md files (plans, reports, gemini reviews, incident
notes) into a single notes/ directory, and update the doc-reference breadcrumbs in
code comments/test labels to the notes/ path.

Add notes/queue-interrupt-reconcile-edge-cases.md: documents why the
queue/interrupt/turn-sealed reconcile path keeps surfacing edge cases (a catalog of
the four review-pass bugs, the no-loss/no-duplicate invariants, the recommended
membership-based reconcile refactor, and interleaving-test guidance).

1 files changed, 38 insertions, 0 deletions

diff --git a/notes/report.md b/notes/report.md
new file mode 100644
index 0000000..3c6d6f7
--- /dev/null
+++ b/notes/report.md
@@ -0,0 +1,38 @@
+# Sidebar Layout Persistence Review
+
+## Verdict
+SHIP
+
+## Block-level findings
+None. The implementation is robust and follows the established patterns for localStorage usage in this codebase.
+
+## Ship-with-followup findings
+None.
+
+## Nits
+None.
+
+## What was checked
+- **A. sidebar-storage.ts correctness**: 
+    - `loadSidebarPanels` correctly handles all failure modes (missing key, malformed JSON, non-array types, and mixed-type arrays) without throwing. It returns a defensive shallow copy of the default layout.
+    - `saveSidebarPanels` is best-effort and safely swallows any storage errors (e.g., QuotaExceededError or SecurityError).
+    - The localStorage key `dispatch-sidebar-panels` is correctly namespaced.
+- **B. SidebarPanel.svelte integration**:
+    - Initialization correctly seeds the `panels` state using `loadSidebarPanels()`.
+    - The `$effect` correctly captures all changes to the `panels` array (addition, removal, and selection changes) due to the reactive read in the map function.
+    - Session-ephemeral `id` fields are correctly regenerated and not persisted, avoiding potential collisions across sessions.
+    - The 'minimum 1 panel' invariant is preserved by the existing UI logic (`{#if idx > 0}`) and reinforced by the storage fallback.
+- **C. Test coverage**:
+    - The unit tests in `sidebar-storage.test.ts` are comprehensive, covering initial load, valid round-trips, corruption, malformed data, filtering, and storage exceptions.
+    - Verified mutation isolation (fresh array on each load).
+- **D. Regression risks**:
+    - Add/remove and dropdown handlers are preserved and correctly trigger persistence via state reassignment.
+    - No infinite loops or race conditions identified; the `$effect` is one-way (state -> localStorage).
+- **E. Stylistic / consistency**:
+    - Matches the `dispatch-` prefix pattern seen in `config.ts`.
+    - Documentation and comments are thorough and clear.
+
+## What was NOT checked
+- Persistence of the `sidebarOpen` toggle (explicitly out of scope).
+- Drag-to-reorder support (not implemented in the UI).
+- Backend synchronization (feature designed for per-device localStorage).