mailmate 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5e25504c9cfa00a8d2c0c94e387b593c9151a66981150c17e1fbd328d531707d
-  data.tar.gz: 4f0ac4ca053f30eed50305e2f333e9969871bbf02565115c2a72dacb0072c023
+  metadata.gz: b986acdac48e1c98c6cc80ae0cc1686af00161a5de3cc17b0946ea09c43b17e1
+  data.tar.gz: 6867d4736f7c4118ea775c60b108a36dc76a5234312b20a36cb5c60070233712
 SHA512:
-  metadata.gz: 8953a409f2855ca1b04a1f4c574c73c4a724e29398d5a5951423b61fd0313f06a21242581219d14f1de3c28436d777f09ee1289f5cb69ca02deeb70bedb8a5ef
-  data.tar.gz: b4dc5f74fa58a58c592ce125c2e42dce062ebbeb841284d74c878e12c6202856f2624f2da4adba6bc24a2919d150a1b8f5b54c302fd71507d1c9085d6d5e4c35
+  metadata.gz: 45d6fc1c66f4549fdf14a899bdba83d1b7d0510b5a20e9469e75f44d92e8b5285bd4cba548a2524d9561831cd34ee64cf3fe3239829afe8b4dc19aa27e4cdbda
+  data.tar.gz: 7b88d5bb8668614368c969d0e6ba10f14a444594b4fe55fdf6dc14bf536e35b29c8845818bff44b65f880c4c5a724d551b245c23c4f3ee56d78247043da1af9c

data/README.md

@@ -60,6 +60,10 @@ mmopen 'message://%3Cabc%40example.com%3E'
 
 # Print the mid: URL instead of opening it (useful in pipelines)
 mmopen 183715 --print
+
+# Spawn the viewer in the background — MailMate stays where it is and
+# your keyboard focus is not stolen. Useful when scripting / cross-app.
+mmopen 183715 --background    # also: -g
 ```
 
 ### `mm-mailboxes` — list accounts and mailboxes
@@ -113,6 +117,11 @@ mm-modify 183715 archive --dry-run
 
 # Verify the new flags after acting
 mm-modify 183715 read --verify
+
+# As of 1.2.0, `--verify` works in `--dry-run` mode too — pair them as a
+# post-hoc state probe (run the action first, then re-run with --dry-run
+# --verify to confirm the change took effect).
+mm-modify 183715 read --dry-run --verify
 ```
 
 **Tip — batch your actions.** Doing all related changes in **one** `mm-modify` invocation is still worthwhile: one open/close cycle instead of two, and the chain runs deterministically without you having to think about ordering. Splitting is safe — `path_for` falls back to a filesystem glob if MailMate's `#source` index is briefly stale after a fast-move — just slower than batching.
@@ -148,11 +157,15 @@ The `mm` prefix is for tab completion: typing `mm<tab>` in a shell lists every c
 
 A few rough edges to be aware of:
 
-1. **Non-move `mm-modify` actions still take over the UI.** Same-account `move` actions use a fast path — a direct `.eml` rename on disk — so they're silent and don't activate MailMate. Everything else (`read`, `flag`, `tag`, `archive`, `junk`, `delete`, etc.) still drives MailMate's UI: each invocation opens a message-viewer window via the `mid:` URL, runs AppleScript key-binding selectors against it, then closes the window. Two consequences:
-   - **Focus is stolen.** When the `mid:` URL fires, macOS brings MailMate forward and the spawned message-viewer window takes keyboard focus. Anything you were typing into another app goes to MailMate instead.
-   - **The close at the end can close the wrong window.** `mm-modify` ends by sending the standard "close window" keystroke. If focus has drifted (or another app's window has come forward in the meantime), that keystroke lands on **your** window — your editor, your browser tab — not MailMate's viewer.
+1. **Non-move `mm-modify` actions briefly spawn a MailMate viewer window.** Same-account `move` actions use a fast path — a direct `.eml` rename on disk — so they're entirely silent. Everything else (`read`, `flag`, `tag`, `archive`, `junk`, `delete`, etc.) drives MailMate's URL handler: each invocation opens a message-viewer window via the `mid:` URL in the background, runs AppleScript key-binding selectors against it, then closes the window.
+
+   **As of 1.2.0 this runs in the background.** MailMate is not brought to the foreground, your keyboard focus stays in whatever app you were using, and you can keep working — even during bulk loops. The viewer windows still appear briefly in MailMate's own window list before closing, but they don't take over your screen. The previous behavior (full-screen takeover, close-keystroke landing on the wrong window) is gone.
 
-   For one-off changes this is just annoying; for a loop of hundreds of messages it makes the machine unusable while it runs. Batch multiple actions into one `mm-modify` invocation when you can — they share a single open/close cycle (or skip it entirely for pure-move). The `--keep-window` flag avoids the close-keystroke entirely if you don't mind cleaning up viewers manually.
+   Two residual caveats:
+   - **Don't bring MailMate to the foreground during a run.** If you click into MailMate or Cmd-Tab to it while `mm-modify` is mid-flight, MailMate's responder chain shifts and subsequent perform-selectors may misbehave. Wait for the run to finish.
+   - **MailMate must be running.** See #3.
+
+   The `--keep-window` flag leaves the spawned viewers open if you want to inspect them. Batch multiple actions into one `mm-modify` invocation when you can — they share a single open/close cycle (or skip it entirely for pure-move).
 
 2. **`eml-id` is machine-local; prefer `Message-ID:`.** The integer eml-id (also shown as MailMate's "Msg ID" column) is just the filename of the `.eml` on disk and differs on every install — copy/pasting an eml-id from your desktop to your laptop will refer to a different message (or none at all). For anything you want to keep, store the RFC `Message-ID:` header (which `mmmessage` prints) and pass that to the CLIs. The `mid:%3C<message-id>%3E` URL scheme works portably for the same reason.
 
@@ -162,6 +175,8 @@ A few rough edges to be aware of:
 
 ## Status
 
+1.2.0 — `mm-modify` no longer brings MailMate to the foreground and is roughly 8× faster on single-action invocations. Internally: the open call uses `open -g -a MailMate <url>` to keep MailMate in the background, and the fixed `--settle` sleeps are replaced by active waits (polling for the spawned viewer window to appear). `mmopen` gains a `--background` / `-g` flag for ad-hoc use. `mm-modify --verify` now works in `--dry-run` mode as a post-hoc state probe. `--settle` is preserved for backward compat; it now caps the active-wait timeout rather than fixing sleep duration.
+
 1.1.0 — `reverse_markdown` (and its transitive `nokogiri` dep) is now opt-in rather than auto-installed. Run `gem install reverse_markdown` if you want `mmmessage --markdown`; everything else is unchanged.
 
 1.0.0 — initial public release; API stable from this point. Breaking changes bump the major version going forward.

data/lib/mailmate/applescript_driver.rb

@@ -32,13 +32,20 @@ module Mailmate
     end
 
     # Open a URL in MailMate (used for `mid:` URLs to select a message).
+    # Uses `open -g -a MailMate` so the URL is dispatched to MailMate's URL
+    # handler without bringing MailMate to the foreground — keeping the
+    # user's keyboard focus where it was. The viewer window is still spawned
+    # for `mid:` URLs. The AppleScript-native alternative
+    # (`tell application "MailMate" to open location`) was tried but is
+    # synchronous and can hang on `mid:` URLs while MailMate processes the
+    # spawn — `open -g` is asynchronous and avoids the wait.
     def open_url(url)
       Mailmate::PlatformError.check_darwin!(component: "AppleScriptDriver") unless dry_run
       if dry_run
-        @output.puts "DRY: open -a MailMate #{url.inspect}"
+        @output.puts "DRY: open -g -a MailMate #{url.inspect}"
         return
       end
-      success = system("open", "-a", "MailMate", url)
+      success = system("/usr/bin/open", "-g", "-a", "MailMate", url)
       raise Error, "open command failed for #{url}" unless success
     end
 

data/lib/mailmate/cli/modify.rb

@@ -98,9 +98,9 @@ module Mailmate
               mute                          Toggle mute state
               delete                        Delete (move to trash)
           BANNER
-          o.on("--verify", "After running, print the message's current flags") { opts[:verify] = true }
+          o.on("--verify", "Print the message's current flags. Works with --dry-run as a post-hoc 'check state' probe (run the action first, then re-run with --dry-run --verify).") { opts[:verify] = true }
           o.on("--dry-run", "Print the actions; don't run") { opts[:dry_run] = true }
-          o.on("--settle SECONDS", Float, "Sleep between operations (default 3.5)") { |s| opts[:settle] = s }
+          o.on("--settle SECONDS", Float, "Timeout for waiting for MailMate's viewer window to spawn after open_url (default 3.5)") { |s| opts[:settle] = s }
           o.on("--keep-window", "Don't close the spawned message-viewer window") { opts[:keep_window] = true }
         end
         parser.parse!(argv)
@@ -258,9 +258,15 @@ module Mailmate
 
         windows_before = driver.window_ids
         driver.open_url(mid_url)
-        sleep(opts[:settle]) unless opts[:dry_run]
-        new_windows = driver.window_ids - windows_before
-
+        # Active wait: poll for the new viewer window instead of sleeping a
+        # fixed `settle`. Windows typically spawn in 200-500ms; the previous
+        # fixed 3.5s sleep was wildly conservative. `--settle` now controls
+        # the timeout for this wait rather than the sleep duration.
+        new_windows = opts[:dry_run] ? [] : wait_for_new_window(driver, windows_before, timeout: opts[:settle])
+
+        # Send each action's selector without sleeping between them.
+        # AppleEvents queue per-app and process in order, so a subsequent
+        # `close window id N` will execute only after `perform` is committed.
         actions.each do |name, selector, args|
           case selector
           when :ensure_flagged, :ensure_not_flagged
@@ -271,16 +277,19 @@ module Mailmate
               $stdout.puts "#{name}: already #{want ? "flagged" : "not flagged"} — no-op"
             else
               driver.perform("toggleFlag:")
-              sleep(opts[:settle]) unless opts[:dry_run]
             end
           else
             driver.perform(selector, *args)
-            sleep(opts[:settle]) unless opts[:dry_run]
           end
         end
 
-        if opts[:verify] && !opts[:dry_run]
-          sleep(opts[:settle])
+        # --verify is now permitted in --dry-run mode so callers can use
+        # `mm-modify <id> <any-action> --dry-run --verify` as a post-hoc
+        # "what's the current flag state?" probe after a separate action run.
+        # In non-dry-run mode, a short fixed wait gives MailMate time to
+        # flush #flags before we read it back; in dry-run no wait is needed.
+        if opts[:verify]
+          sleep(1) unless opts[:dry_run]
           $stdout.puts "Flags now: #{current_flags(eml_id).inspect}"
         end
 
@@ -289,6 +298,20 @@ module Mailmate
         end
       end
 
+      # Poll for a new MailMate window appearing in `driver.window_ids` that
+      # isn't in `windows_before`. Returns the set of new window IDs, or an
+      # empty array if `timeout` elapses without a new window appearing.
+      def wait_for_new_window(driver, windows_before, timeout:, poll: 0.05)
+        deadline = Time.now + timeout
+        loop do
+          diff = driver.window_ids - windows_before
+          return diff unless diff.empty?
+          break if Time.now >= deadline
+          sleep(poll)
+        end
+        []
+      end
+
       def current_flags(eml_id)
         # AppleScript actions write the index asynchronously — bust just the
         # #flags cache to pick up the latest values without throwing away

data/lib/mailmate/cli/open.rb

@@ -44,26 +44,30 @@ module Mailmate
           return 0
         end
 
-        system("/usr/bin/open", url)
+        cmd = ["/usr/bin/open"]
+        cmd << "-g" if opts[:background]
+        cmd << url
+        system(*cmd)
         $?.exitstatus
       end
 
       def parse_options(argv)
-        opts = { print_only: false }
+        opts = { print_only: false, background: false }
         OptionParser.new do |o|
-          o.banner = "Usage: mmopen <id> [--print]"
+          o.banner = "Usage: mmopen <id> [--print] [--background|-g]"
           o.separator ""
           o.separator "Open a MailMate message in MailMate's UI. <id> can be a local"
           o.separator "eml-id, an RFC Message-ID (with or without angle brackets), or"
           o.separator "a message://… or mid:… URL."
           o.on("--print", "Print the mid: URL instead of opening it (for piping)") { opts[:print_only] = true }
+          o.on("-g", "--background", "Open without bringing MailMate to the foreground") { opts[:background] = true }
         end.parse!(argv)
         opts
       end
 
       def usage_error(msg)
         warn "mmopen: #{msg}"
-        warn "Usage: mmopen <id> [--print]"
+        warn "Usage: mmopen <id> [--print] [--background|-g]"
         2
       end
     end

data/lib/mailmate/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Mailmate
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mailmate
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Brian Murphy-Dye