CLI should handle errors from Action monad sensibly

[srid]

A malformed zquery in a zettel can trigger this:

This is bad. We should trigger a non-exit return code.

---

In Rib, make runShake take a exitOnException flag, and enable it unless -w/-s is on (because in watch or serve mode we don't want to exit, but only log the error).

[srid]

Same situation with the upcoming neuron new:

[srid]

An user encountered this with neuron-mode which happens when the (default) zettelkasten directory does not exist:

Command "neuron --zettelkasten-dir /home/suppi/zettelkasten query --uri 'zquery://search'" exited with code 1: neuron: user error (Zettelkasten directory /home/suppi/zettelkasten/ does not exist.)

neuron--read-query-result: Wrong type argument: char-or-string-p, nil

[srid]

After this is resolved, the query CLI JSON output will get a new errors field, that contains all errors in the zettekasten (a map of zettel id to errors lits).

[0ihsan]

I'm getting a '0' exit code after executing neuron query --uri "z:zettels" | jq even if I get an error from neuron query command.

neuron version: 0.5.10.3