lang/org: fix external org-capture workflow

+ Fix frame-spawning when calling bin/org-capture
+ Integrate counsel-org-capture into bin/org-capture workflow
+ Ensure frame is closed if counsel-org-capture is cancelled
+ Ensure org-capture buffer closes the associated frame
+ Add transient frame property for org-capture frames

If you are using a tiling window manager, you'll need to add a rule for
a window named "org-capture" in order to make a floating window.

bin/org-capture

@@ -33,7 +33,7 @@ shift $((OPTIND-1))
 
 if [[ $daemon ]]; then
     emacsclient -a "" \
-        -c -F '((name . "org-capture") (width . 70) (height . 25))' \
+        -c -F '((name . "org-capture") (width . 70) (height . 25) (transient . t))' \
         -e "(+org-capture/open-frame \"$str\" ${key:-nil})"
 else
     # Non-daemon servers flicker a lot if frames are created from terminal, so

modules/lang/org/autoload/org-capture.el

@@ -34,6 +34,7 @@ Uses the capture template specified by KEY. Otherwise, prompts you for one."
   `((name . "org-capture")
     (width . 70)
     (height . 25)
+    (transient . t)
     (window-system . ,(cond (IS-MAC 'ns)
                             (IS-LINUX 'x)
                             (t 'w32)))
@@ -50,15 +51,19 @@ Uses the capture template specified by KEY. Otherwise, prompts you for one."
 (defun +org-capture-frame-p (&rest _)
   "Return t if the current frame is an org-capture frame opened by
 `+org-capture/open-frame'."
-  (equal "org-capture" (frame-parameter nil 'name)))
+  (and (equal "org-capture" (frame-parameter nil 'name))
+       (frame-parameter nil 'transient)))
 
 ;;;###autoload
 (defun +org-capture/open-frame (&optional string key)
   "Opens the org-capture window in a floating frame that cleans itself up once
 you're done. This can be called from an external shell script."
   (interactive)
-  (require 'org)
+  (when (and string (string-empty-p string))
-  (let (after-make-frame-functions before-make-frame-hook)
+    (setq string nil))
+  (when (and key (string-empty-p key))
+    (setq key nil))
+  (require 'org-capture)
   (let ((frame (if (+org-capture-frame-p)
                    (selected-frame)
                  (make-frame +org-capture-window-params))))
@@ -66,15 +71,26 @@ you're done. This can be called from an external shell script."
       (condition-case ex
           (cl-letf (((symbol-function #'pop-to-buffer)
                      (symbol-function #'switch-to-buffer)))
-              (if (and (stringp string)
+            (switch-to-buffer (doom-fallback-buffer))
-                       (not (string-empty-p string)))
+            (let ((org-capture-initial string)
-                  (org-capture-string string key)
+                  (org-capture-mode-hook org-capture-mode-hook)
-                (org-capture nil key))
+                  org-capture-entry)
-              (when (featurep 'solaire-mode)
+              (when (and key (not (string-empty-p key)))
-                (solaire-mode +1)))
+                (setq org-capture-entry (org-capture-select-template key)))
+              (if (or org-capture-entry
+                      (not (fboundp 'counsel-org-capture)))
+                  (org-capture)
+                (unwind-protect
+                    (counsel-org-capture)
+                  (if-let* ((buf (cl-loop for buf in (buffer-list)
+                                          if (buffer-local-value 'org-capture-mode buf)
+                                          return buf)))
+                      (with-current-buffer buf
+                        (add-hook 'kill-buffer-hook #'+org-capture|cleanup-frame nil t))
+                    (delete-frame frame))))))
         ('error
          (message "org-capture: %s" (error-message-string ex))
-           (delete-frame frame)))))))
+         (delete-frame frame))))))
 
 ;;;###autoload
 (defun +org-capture-available-keys ()