Clojurians Log v2

The syntax to define multi-arity fns is this one, right?

[:function
  [:=> [:cat int? int?] int?]
  [:=> [:cat int?] string?]]

Ok, so I was asking because I'm implementing multi-arity schema support in my malli-instrument library, but I'm unsure how to best deal with instrumenting multi-arity fns with varargs. Due to the way in which varargs are specified, I don't have a way to know which of the arities under :function corresponds to each of the possible lengths of the arg list, or at least not without some analysis, consider the following:

[:function 
  [:=> [:* int?] int?]
  [:=> [:* string?] string?]]

(defn foo
  ([x y z] (+ x y z))
  ([a b c d & _] (str a b c d)))

@setzer22 please check https://github.com/metosin/malli/blob/master/src/malli/core.cljc#L264-L272

and; https://github.com/metosin/malli/blob/master/src/malli/clj_kondo.cljc#L134-L154

nice, thanks! 👍 I was re-inventing the wheel for some of this work in a more inefficient way so this helps a lot

I'm confused, what's this second one doing? 😅

just noticed a small inconsistency when writing the instrumentation code: (m/validate :cat nil) returns false, but when calling (fn [& args]) with zero arguments, args is bound to nil

I can easily work around it on my side using something like (or args '()), but I thought that may be something worth looking into

alright! malli-instrument now supports multi-arity fn schemas 😄 🎉 https://github.com/setzer22/malli-instrument I'll continue working on this and pushing bugfixes