Scope the selection to the current view

A selection made in one workspace survived switching workspace or tab, so
the selection bar and its batch actions (copy, delete) kept acting on items
that were no longer visible. Switching workspace, switching tab, or closing
search now clears the selection.

AGENTS.md

@@ -44,19 +44,28 @@ clicking an item copies it back to the clipboard.
   newline, Esc forwards to the panel (clear selection, else hide). Height is
   measured from the text and pushed into a `height` binding the panel applies
   as a frame — a representable cannot resize itself through
-  `intrinsicContentSize` once SwiftUI offers a larger size.
+  `intrinsicContentSize` once SwiftUI offers a larger size. The vertical
+  scroller is disabled until the text is taller than the max height, so it
+  never flashes during growth. The composer re-asserts first responder on
+  updates only while `canStealFocus` is true; the panel turns it off whenever
+  a modal field (search, edit, new workspace) or the sync overlay owns the
+  keyboard — otherwise every render stole focus back mid-edit.
 - `Sources/Prompt/PanelView.swift` — the panel UI. Header (workspace switcher,
-  search, add photos), item list, composer. There is no in-header close button:
-  the window's own close button hides the panel. Composer `# Name` creates/
+  search, add photos), a segmented Workspace/All tab (All lists every prompt
+  labeled with its workspace), item list, in-panel edit modal, composer.
+  There is no in-header close button: the window's own close button hides the
+  panel. Composer `# Name` creates/
   switches sections. Click = select (Shift+click for a range); the selection
   bar offers Copy (single item), Copy as List, Copy as Lines and Delete;
   right-click = context menu acting on the whole selection (Copy, Copy as List,
   Copy as Lines, Done, Edit, Open, Merge, Move To, Delete). Esc clears the
-  selection before it hides the panel. Drop target for images/files.
+  selection before it hides the panel. Selection is scoped to the current view:
+  switching workspace, switching tab, or closing search clears it. Drop target for images/files.
 - `Sources/Prompt/PanelController.swift` — owns the floating `NSPanel`
   (`.nonactivatingPanel`, `.floating`, right-edge docked, frame autosaved as
-  `PromptPanel`), the Cmd+V image-paste monitor, and `EditWindowController`
-  (separate edit window with a `TextEditor`).
+  `PromptPanel`), and the Cmd+V paste monitor. Editing a note is an in-panel
+  modal in `PanelView`, not a window — a separate edit window used to float
+  loose from the panel and its theme.
 - `Sources/Prompt/HotkeyManager.swift` — double-Shift detection via a
   CGEvent tap on `flagsChanged` (left Shift ×2 = capture, right Shift ×2 =
   toggle panel), plus Carbon `RegisterEventHotKey` fallbacks (Cmd+Shift+Space
@@ -174,6 +183,14 @@ brew upgrade --cask prompt
 
 ## Gotchas
 
+- **`@Published` publishes on every assignment, even when the value is
+  unchanged.** The 5-second sync poll used to assign `lastError = nil`
+  unconditionally; `PanelView` observes `SyncService`, so the entire panel
+  (rows, overlays, composer) rebuilt every 5 seconds, forever. Over five days
+  of uptime that was ~185 CPU-minutes and ~190 MB of footprint, and every
+  click started to lag. Poll state now goes through `setError`, which no-ops
+  when equal, and the header's sync dot is its own view so sync publishes
+  cannot invalidate the list.
 - **Never assign `NSApp.mainMenu`.** SwiftUI installs a menu whose Edit items
   are how AppKit routes Cmd+C/V/X/A/Z; replacing it with a hand-built menu
   silently breaks pasting, copying and select-all in the composer (this shipped
@@ -210,6 +227,16 @@ brew upgrade --cask prompt
   firing. The tap is re-enabled on `tapDisabledByTimeout`.
 - `Store` is `@MainActor`; anything reaching it from monitors/callbacks must
   go through `Task { @MainActor in }`.
+- **The composer steals focus via `updateNSView`, so gate it.** It used to
+  re-assert first responder on every render, which yanked focus out of the
+  edit dialog (and search/sync fields) on every keystroke. The gate is
+  `canStealFocus`; any new modal text field must be included in it.
+- **`doCommandBy` has no fall-through once the delegate implements it.**
+  Returning false for `insertNewline` on Shift+Return killed the event instead
+  of inserting a newline, so Shift+Return never actually worked. The composer
+  now inserts it explicitly, and reads the Shift state from the `keyDown`
+  currently being handled because `NSApp.currentEvent` is not reliably
+  populated when the command fires.
 - `captureSelection()` blocks up to 0.5s polling the pasteboard — keep it off
   the main thread (the hotkey handler already dispatches to a global queue).
 - The double-Shift tap needs Accessibility; without it the event tap silently

README.md

@@ -19,6 +19,10 @@ to your work. Select what you need and send it back to your clipboard.
   the Cmd+K switcher (which opens a name modal); Cmd+K switches, and the
   up/down arrows in the switcher reorder them. (The code and the saved JSON
   still call these `sections` so existing data keeps working.)
+- **An "All" tab** — a global view of every prompt across every workspace,
+  each labeled with the workspace it lives in.
+- **Edit in place** — right-click → Edit opens an in-panel modal on the same
+  theme, not a separate window.
 - **Right-click for actions** — Copy, Copy as List, Copy as Lines, Mark as Done,
   Edit, Open, Merge Notes, Move To, Delete — each acting on the whole selection.
 - **Search** — Cmd+F searches across all workspaces.

Sources/Prompt/ComposerView.swift

@@ -16,6 +16,10 @@ struct ComposerView: NSViewRepresentable {
     @Binding var isFocused: Bool
     @Binding var height: CGFloat
     var placeholder: String
+    /// When false, the composer never makes itself first responder. The panel
+    /// turns this off while a modal field (search, edit, new workspace) or the
+    /// sync overlay is up — otherwise every render stole focus back here.
+    var canStealFocus: Bool = true
     var onSubmit: () -> Void
     var onEscape: (() -> Void)? = nil
 
@@ -34,7 +38,10 @@ struct ComposerView: NSViewRepresentable {
         let scrollView = NSScrollView()
         scrollView.drawsBackground = false
         scrollView.borderType = .noBorder
-        scrollView.hasVerticalScroller = true
+        // The scroller starts disabled and is only switched on once the text
+        // is taller than the max height. With it always on, every growth step
+        // briefly overflows and the scroller flashes for one frame.
+        scrollView.hasVerticalScroller = false
         scrollView.autohidesScrollers = true
         scrollView.horizontalScrollElasticity = .none
 
@@ -67,8 +74,10 @@ struct ComposerView: NSViewRepresentable {
             textView.string = text
             updateHeight(textView)
         }
+        if textView.placeholderText != placeholder {
             textView.placeholderText = placeholder
-        if isFocused, scrollView.window?.firstResponder != textView {
+        }
+        if isFocused, canStealFocus, scrollView.window?.firstResponder != textView {
             scrollView.window?.makeFirstResponder(textView)
         }
     }
@@ -79,7 +88,9 @@ struct ComposerView: NSViewRepresentable {
         layoutManager.ensureLayout(for: textContainer)
         let content = layoutManager.usedRect(for: textContainer).height + 2 * textView.textContainerInset.height
         let line = Self.oneLineHeight
-        let clamped = max(line, min(content, line * maxLines + 1))
+        let maxHeight = line * maxLines + 1
+        textView.enclosingScrollView?.hasVerticalScroller = content > maxHeight
+        let clamped = max(line, min(content, maxHeight))
         if abs(height - clamped) > 0.5 {
             // Defer so the binding is not written from inside a layout pass.
             DispatchQueue.main.async { self.height = clamped }
@@ -101,7 +112,7 @@ struct ComposerView: NSViewRepresentable {
             ) { [weak self, weak textView] note in
                 guard let self, let textView,
                       (note.object as? NSWindow) === textView.window,
-                      self.parent.isFocused else { return }
+                      self.parent.isFocused, self.parent.canStealFocus else { return }
                 textView.window?.makeFirstResponder(textView)
             }
         }
@@ -129,8 +140,14 @@ struct ComposerView: NSViewRepresentable {
                 return true
             }
             guard commandSelector == #selector(NSResponder.insertNewline(_:)) else { return false }
-            if NSApp.currentEvent?.modifierFlags.contains(.shift) == true {
-                return false
+            // NSApp.currentEvent is not reliably set when the command fires,
+            // so the Shift state comes from the keyDown currently being handled.
+            // And returning false here does NOT fall through to the default
+            // newline insertion once a delegate implements doCommandBy — the
+            // event just dies — so insert the newline explicitly.
+            if (textView as? ComposerTextView)?.keyDownModifiers.contains(.shift) == true {
+                textView.insertNewline(nil)
+                return true
             }
             parent.onSubmit()
             return true
@@ -143,6 +160,16 @@ private final class ComposerTextView: NSTextView {
         didSet { needsDisplay = true }
     }
 
+    /// Modifier flags of the keyDown currently being handled. `doCommandBy`
+    /// runs inside that call tree and cannot rely on `NSApp.currentEvent`.
+    var keyDownModifiers: NSEvent.ModifierFlags = []
+
+    override func keyDown(with event: NSEvent) {
+        keyDownModifiers = event.modifierFlags
+        super.keyDown(with: event)
+        keyDownModifiers = []
+    }
+
     override func draw(_ dirtyRect: NSRect) {
         super.draw(dirtyRect)
         guard string.isEmpty, !placeholderText.isEmpty else { return }

Sources/Prompt/PanelController.swift

@@ -16,7 +16,6 @@ final class PanelController: NSObject {
 
     private var panel: KeyablePanel?
     private var pasteMonitor: Any?
-    let editController = EditWindowController()
 
     var isVisible: Bool { panel?.isVisible ?? false }
 
@@ -102,56 +101,4 @@ final class PanelController: NSObject {
     }
 }
 
-@MainActor
-final class EditWindowController: NSObject {
-    private var window: NSWindow?
 
-    func edit(itemID: UUID, text: String) {
-        let view = EditView(
-            text: text,
-            onSave: { [weak self] newText in
-                Store.shared.updateText(itemID, newText: newText)
-                self?.window?.close()
-            },
-            onCancel: { [weak self] in self?.window?.close() }
-        )
-        let window = NSWindow(
-            contentRect: NSRect(x: 0, y: 0, width: 480, height: 320),
-            styleMask: [.titled, .closable, .resizable],
-            backing: .buffered,
-            defer: false
-        )
-        window.title = "Edit Note"
-        window.contentView = NSHostingView(rootView: view)
-        window.isReleasedWhenClosed = false
-        window.center()
-        window.makeKeyAndOrderFront(nil)
-        NSApp.activate(ignoringOtherApps: true)
-        self.window = window
-    }
-}
-
-private struct EditView: View {
-    @State var text: String
-    var onSave: (String) -> Void
-    var onCancel: () -> Void
-
-    var body: some View {
-        VStack(spacing: 12) {
-            TextEditor(text: $text)
-                .font(.body)
-                .padding(4)
-                .background(Color.primary.opacity(0.05))
-                .cornerRadius(6)
-            HStack {
-                Spacer()
-                Button("Cancel", action: onCancel)
-                    .keyboardShortcut(.cancelAction)
-                Button("Save") { onSave(text) }
-                    .keyboardShortcut(.defaultAction)
-                    .keyboardShortcut("s", modifiers: .command)
-            }
-        }
-        .padding(16)
-    }
-}

Sources/Prompt/PanelView.swift

@@ -5,7 +5,6 @@ import UniformTypeIdentifiers
 struct PanelView: View {
     @ObservedObject var store: Store
     @ObservedObject private var sync = SyncService.shared
-
     @State private var composer = ""
     @State private var composerFocused = false
     @State private var composerHeight: CGFloat = ComposerView.oneLineHeight
@@ -18,9 +17,16 @@ struct PanelView: View {
     @State private var joinCode = ""
     @State private var selection = Set<UUID>()
     @State private var lastSelectedID: UUID?
+    @State private var editingItem: PromptItem?
+    @State private var editText = ""
+    @State private var editFocused = false
+    @State private var editHeight: CGFloat = ComposerView.oneLineHeight
     @State private var copiedID: UUID?
     @State private var accessibilityGranted = AXIsProcessTrusted()
 
+    private enum Tab { case workspace, all }
+    @State private var tab: Tab = .workspace
+
     private enum Field: Hashable { case search, newWorkspace }
     @FocusState private var focusedField: Field?
 
@@ -30,6 +36,7 @@ struct PanelView: View {
             if !accessibilityGranted {
                 permissionBanner
             }
+            tabPicker
             if showSearch {
                 searchField
             }
@@ -47,12 +54,18 @@ struct PanelView: View {
         .overlay(sectionSwitcher)
         .overlay(syncOverlay)
         .overlay(newWorkspaceModal)
+        .overlay(editModal)
         .onDrop(of: [UTType.image.identifier, UTType.fileURL.identifier], isTargeted: nil, perform: handleDrop)
         .onReceive(NotificationCenter.default.publisher(for: .promptPanelDidShow)) { _ in
             composerFocused = true
             accessibilityGranted = AXIsProcessTrusted()
         }
         .onAppear { composerFocused = true }
+        // Selection is scoped to the current view: switching workspace, tab,
+        // or leaving search clears it. Otherwise a selection made in one
+        // workspace kept acting (copy, delete) on items you can no longer see.
+        .onChange(of: store.activeSectionID) { clearSelection() }
+        .onChange(of: tab) { clearSelection() }
     }
 
     // MARK: - Header
@@ -77,8 +90,7 @@ struct PanelView: View {
             Spacer()
 
             Button { withAnimation(.easeInOut(duration: 0.15)) { showSync.toggle() } } label: {
-                Image(systemName: "arrow.triangle.2.circlepath")
-                    .foregroundStyle(sync.code == nil ? Color.secondary : Color.accentColor)
+                SyncDot()
             }
             .buttonStyle(.plain).help("Sync")
             Button { toggleSearch() } label: { Image(systemName: "magnifyingglass") }
@@ -90,6 +102,20 @@ struct PanelView: View {
         .padding(.vertical, 10)
     }
 
+    // MARK: - Tabs
+
+    /// The global view: every prompt across every workspace, each labeled.
+    private var tabPicker: some View {
+        Picker("View", selection: $tab) {
+            Text(store.activeSection.name).tag(Tab.workspace)
+            Text("All").tag(Tab.all)
+        }
+        .pickerStyle(.segmented)
+        .labelsHidden()
+        .padding(.horizontal, 12)
+        .padding(.vertical, 6)
+    }
+
     private var permissionBanner: some View {
         HStack(spacing: 8) {
             Image(systemName: "exclamationmark.triangle.fill")
@@ -122,6 +148,8 @@ struct PanelView: View {
                 .onExitCommand {
                     search = ""
                     showSearch = false
+                    focusedField = nil
+                    clearSelection()
                     composerFocused = true
                 }
             if !search.isEmpty {
@@ -135,21 +163,27 @@ struct PanelView: View {
 
     // MARK: - Content
 
+    /// Every item across every workspace, each labeled with its workspace.
+    private var allLabeledItems: [(section: String, item: PromptItem)] {
+        store.sections.flatMap { section in
+            section.items.map { (section.name, $0) }
+        }
+    }
+
     private var displayedItems: [(section: String, item: PromptItem)] {
-        if search.isEmpty {
-            return store.activeSection.items.map { ("", $0) }
-        }
+        if !search.isEmpty {
             let query = search.lowercased()
-        return store.sections.flatMap { section in
-            section.items.compactMap { item in
-                switch item.content {
-                case .text(let text) where text.lowercased().contains(query):
-                    return (section.name, item)
-                default:
-                    return nil
+            return allLabeledItems.filter { entry in
+                if case .text(let text) = entry.item.content {
+                    return text.lowercased().contains(query)
+                }
+                return false
             }
         }
+        if tab == .all {
+            return allLabeledItems
         }
+        return store.activeSection.items.map { ("", $0) }
     }
 
     private var content: some View {
@@ -296,7 +330,10 @@ struct PanelView: View {
             }
             if case .text(let text) = item.content, !multiple {
                 Button("Edit…") {
-                    PanelController.shared.editController.edit(itemID: item.id, text: text)
+                    editText = text
+                    editHeight = ComposerView.oneLineHeight
+                    editingItem = item
+                    editFocused = true
                 }
             }
             if case .image(let filename) = item.content, !multiple {
@@ -376,6 +413,7 @@ struct PanelView: View {
                     .contentShape(Rectangle())
                     .onTapGesture {
                         store.activeSectionID = section.id
+                        tab = .workspace
                         showSectionSwitcher = false
                     }
                 }
@@ -512,6 +550,70 @@ struct PanelView: View {
         }
     }
 
+    // MARK: - Edit
+
+    /// The editor is a ComposerView too, so it behaves exactly like the main
+    /// input: Return saves, Shift+Return inserts a newline, Esc cancels, and
+    /// pasting goes through the same working path.
+    @ViewBuilder
+    private var editModal: some View {
+        if let item = editingItem {
+            ZStack {
+                Color.black.opacity(0.25)
+                    .onTapGesture { dismissEditor() }
+                VStack(alignment: .leading, spacing: 12) {
+                    Text("Edit")
+                        .font(.headline)
+                    ComposerView(
+                        text: $editText,
+                        isFocused: $editFocused,
+                        height: $editHeight,
+                        placeholder: "",
+                        onSubmit: { saveEdit(item) },
+                        onEscape: { dismissEditor() }
+                    )
+                    .frame(height: editHeight)
+                    .padding(8)
+                    .background(Theme.rowSwiftUI)
+                    .cornerRadius(8)
+                    HStack {
+                        Spacer()
+                        Button("Cancel") { dismissEditor() }
+                            .keyboardShortcut(.cancelAction)
+                        Button("Save") { saveEdit(item) }
+                            .keyboardShortcut("s", modifiers: .command)
+                            .disabled(editText.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty)
+                    }
+                }
+                .padding(16)
+                .frame(maxWidth: 320)
+                .background(Theme.backgroundSwiftUI)
+                .cornerRadius(12)
+                .overlay(
+                    RoundedRectangle(cornerRadius: 12)
+                        .stroke(Color.primary.opacity(0.12), lineWidth: 1)
+                )
+                .shadow(radius: 18)
+            }
+            .transition(.opacity)
+        }
+    }
+
+    private func saveEdit(_ item: PromptItem) {
+        let text = editText.trimmingCharacters(in: .whitespacesAndNewlines)
+        if !text.isEmpty {
+            store.updateText(item.id, newText: text)
+        }
+        dismissEditor()
+    }
+
+    private func dismissEditor() {
+        editingItem = nil
+        editText = ""
+        editFocused = false
+        composerFocused = true
+    }
+
     // MARK: - New workspace
 
     @ViewBuilder
@@ -556,6 +658,7 @@ struct PanelView: View {
     private func dismissNewWorkspace() {
         showNewWorkspace = false
         newWorkspaceName = ""
+        focusedField = nil
         composerFocused = true
     }
 
@@ -629,6 +732,9 @@ struct PanelView: View {
                 isFocused: $composerFocused,
                 height: $composerHeight,
                 placeholder: "Add a note or prompt (\(store.activeSection.name))",
+                // A modal field (search, edit, new workspace) or the sync
+                // overlay owns the keyboard; don't steal focus back mid-edit.
+                canStealFocus: focusedField == nil && !showSync && editingItem == nil,
                 onSubmit: submit,
                 onEscape: {
                     if selection.isEmpty {
@@ -751,6 +857,8 @@ struct PanelView: View {
             focusedField = .search
         } else {
             search = ""
+            focusedField = nil
+            clearSelection()
             composerFocused = true
         }
     }
@@ -795,3 +903,15 @@ struct PanelView: View {
         return (try? AttributedString(markdown: string, options: options)) ?? AttributedString(string)
     }
 }
+
+/// Sync status lives in its own view so a `SyncService` publish cannot
+/// invalidate the panel's list. Watching it from `PanelView`'s body meant every
+/// sync event — including the 5-second poll — re-rendered every row.
+private struct SyncDot: View {
+    @ObservedObject private var sync = SyncService.shared
+
+    var body: some View {
+        Image(systemName: "arrow.triangle.2.circlepath")
+            .foregroundStyle(sync.code == nil ? Color.secondary : Color.accentColor)
+    }
+}

Sources/Prompt/SyncService.swift

@@ -193,13 +193,20 @@ final class SyncService: ObservableObject {
 
     private func poll() {
         guard !isSyncing, let code, let versionURL = endpoint("/api/sync/\(code)/version") else { return }
+        // A cached /version would keep reporting the version the client already
+        // has, so polls must always hit the network. The responses are a few
+        // bytes; letting CFNetwork write them to the disk cache every 5 s only
+        // adds churn.
+        var versionRequest = URLRequest(url: versionURL)
+        versionRequest.cachePolicy = .reloadIgnoringLocalCacheData
+        versionRequest.timeoutInterval = 15
         Task {
             do {
                 // Cheap poll first: the full payload can be megabytes of
                 // base64 images, so only download it when the version moved.
-                let (versionData, versionResponse) = try await URLSession.shared.data(from: versionURL)
+                let (versionData, versionResponse) = try await URLSession.shared.data(for: versionRequest)
                 if (versionResponse as? HTTPURLResponse)?.statusCode == 404 {
-                    lastError = "Sync group no longer exists on the server"
+                    setError("Sync group no longer exists on the server")
                     return
                 }
                 guard let versionHTTP = versionResponse as? HTTPURLResponse, versionHTTP.statusCode == 200 else {
@@ -208,12 +215,15 @@ final class SyncService: ObservableObject {
                 }
                 let remoteVersion = try JSONDecoder().decode([String: Int64].self, from: versionData)["version"] ?? 0
                 guard remoteVersion > lastVersion else {
-                    lastError = nil
+                    setError(nil)
                     return
                 }
                 syncLog("poll remote v\(remoteVersion) local v\(lastVersion) — fetching payload")
                 guard let payloadURL = endpoint("/api/sync/\(code)") else { return }
-                let (data, response) = try await URLSession.shared.data(from: payloadURL)
+                var payloadRequest = URLRequest(url: payloadURL)
+                payloadRequest.cachePolicy = .reloadIgnoringLocalCacheData
+                payloadRequest.timeoutInterval = 30
+                let (data, response) = try await URLSession.shared.data(for: payloadRequest)
                 guard let http = response as? HTTPURLResponse, http.statusCode == 200 else { return }
                 let remote = try JSONDecoder().decode(SyncGetResponse.self, from: data)
                 if let payload = remote.data {
@@ -221,15 +231,25 @@ final class SyncService: ObservableObject {
                 } else {
                     lastVersion = remote.version
                 }
-                lastError = nil
+                setError(nil)
             } catch {
                 syncLog("poll failed: \(error.localizedDescription)")
                 // Polling errors are transient; stay quiet unless nothing ever worked.
-                if lastSyncedAt == nil { lastError = "Cannot reach sync server" }
+                if lastSyncedAt == nil { setError("Cannot reach sync server") }
             }
         }
     }
 
+    /// `@Published` fires on every assignment, even when the value is
+    /// unchanged. The 5-second poll would otherwise republish `lastError`
+    /// forever, and since `PanelView` observes this object, the whole panel
+    /// (list, rows, overlays) re-rendered every 5 seconds — the app grew to
+    /// ~190 MB and ~90 ms of layout per poll over five days of uptime.
+    private func setError(_ message: String?) {
+        guard lastError != message else { return }
+        lastError = message
+    }
+
     private func restartPolling() {
         pollTimer?.invalidate()
         pollTimer = nil