Syntax error on valid bb.edn

[favetelinguis]

Just did an install from melpa to Doom emacs. I have the following bb.edn file.

{:tasks
 {db-stop {:doc "Stop the devbox distrobox"
           :task (shell "distrobox stop --yes devbox")}
  db-rm {:doc "Remove the devbox distrobox"
         :task (shell "distrobox rm --force devbox")}
  db-create {:doc "Create the devbox distrobox"
             :task (shell "distrobox create devbox -i devbox")}
  db-enter {:doc "Enter the created distrobox"
            :task (shell "distrobox enter devbox")}
  remake {:doc "Will kill and build and create a new devbox ready to enter"
          :task (do (run 'pm-build)
                    (run 'db-stop)
                    (run 'db-rm)
                    (run 'db-create))}
  pm-build {:doc "Build docker image"
            :task (shell "podman build -f CoreBox -t devbox .")}
  pm-clean {:doc "Cleanup unused images"
            :task (shell "podman image prune --all")}}}

I am getting the following error when running babashka-tasks.

parseclj--reduce-coll: parseclj: Syntax error: "At position 528, unmatched :rparen"

What am I doing wrong here?

[favetelinguis]

Did open the following clojure-emacs/parseedn#18

[licht1stein]

As a workaround you can put all of this into bb/pm-builld.clj and require it in bb.edn

[formsandlines]

I am getting a similar unmatched :rparen syntax error with my config and found that a symbol in my task (e.g. ’foo in (run ’foo)) caused the issue. It seems like this has been confirmed by the maintainer in your issue on parseedn, so I guess we have to wait until they can fix it.

[teodorlu]

I also hit this problem today, having quote characters in bb.edn.

An alternative approach to parseedn.el is to shell out to Babashka.

(defun babashka-tasks ()
  (s-lines (s-trim (shell-command-to-string (concat "bb -e " (shell-quote-argument "(run! println (keys (:tasks (read-string (slurp \"bb.edn\")))))"))))))

(defun babashka-run-task ()
  (interactive)
  (let* ((default-directory (project-root (project-current t)))
         (task (completing-read "> " (babashka-tasks))))
    (async-shell-command (concat "bb " (shell-quote-argument task)))
    (shell-command (concat "bb " (shell-quote-argument task)))))

@licht1stein any interest in making this the default approach?

• Babashka must be installed in order to shell out to Babashka, but we do that anyway.
• I see that I'm using s.el, which isn't currently required by babashka.el. I think it's possible to do the same without s.el if s.el is a deal breaker.

Credit to @borkdude for suggesting this approach.

[licht1stein]

Makes sense, we know babashka will be installed anyway.