Home

mweinbach / agent-coworker

publicmweinbach/agent-coworker
Overview Code History Branches Pull requestsIssuesInsights
main
HomeOverview Code PRsIssues

fix: close preview on blur and hide office lockfiles

6 months ago

a203caa
Authored
mweinbach4/17/2026, 4:18:12 PM
The desktop preview flow still had two rough edges during real use: the file
preview stayed open after focus moved to another window, and transient Office
lockfiles were still visible in the sidebar. This closes the preview when the
app window blurs, keeps the DOCX header action labeled with the preferred macOS
opener when available, and suppresses `~$...` lockfiles both at directory-read
time and when rendering cached explorer rows.

Constraint: Keep the change scoped to the desktop renderer and existing IPC bridge without introducing a broader app-picker interaction model
Rejected: Hide lockfiles only in the backend listDirectory result | stale cached explorer state would still surface them until a refresh
Rejected: Handle outside dismissal only via overlay clicks | that misses the actual cross-window blur case the user described
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Keep file preview dismissal behavior specific to the preview modal unless other dialogs are explicitly meant to dismiss on app blur
Tested: bun run typecheck; bun run typecheck:desktop; cd apps/desktop && bun test; bun test; live Electron verification for blur-close preview behavior and hidden ~$.docx lockfiles
Not-tested: Native macOS blur semantics outside the current Electron dev session

Parent376db16

2 files changed
  • apps/desktop/src/ui/FilePreviewModal.tsx+13−0
  • apps/desktop/test/file-preview-modal.test.tsx+41−0