ir should surface as a error value on the mutation response

I think I’ve figured out what was causing my issue. I have a mutation that’s properly surfacing the error (from the plugin) when I call it regularly. However, if I specify keys to return from the mutation it’s resulting in the error not surfacing.

(pco/defmutation mutate [params]
  {::plugins/params-spec (ds/spec ::spec {::id keyword?})}
  {::params params})

(p.a.eql/process registry `[(mutate {::id "fail")])
;; Returns with a ::pcr/mutation-error

(p.a.eql/process registry `[{(mutate {::id "fail"}) [::params]}])
;; Returns {`mutate {}}

hello @U2U78HT5G, I found the issue on the part that filters the output, that was swallowing the error, fixed on master

on top of that, a few things I like to suggest after looking at your code:

- No need to provide plugin-id when using defplugin - p.a.eql alias should be used for async EQL, for sync it should be p.eql

Awesome, I’ll pull down latest! Also thanks for the suggestions! That’ll clean up the code!

oh, since you were so fast

I acutally pushed a bug, but just fixed

c50d2e75dd084190c58c0e5a599c8e3f1a88d9ef should be fine

Perfect! Will use that SHA 🙂

Just validated that the fix worked as expected. Thanks for the quick fix!