I think you need to make it a sequence. A string seems like a good choice?

The function does what it needs to do, just wondered if theres a more idiomatic way for it

My comment was about ”[I] have to convert t the keyword to a string”. Which I think there is no (sane) way around.

Thats what I thought, had been searching to see if Clojure had indeed some sane way to identify a colon inside a keyword. Convert to string is the easiest then.

As for the removal, I would have done it with a regex replace, without checking for a : first. Think that reveals intention better.

Thats a good idea indeed, I only need the part after the middle colon so replacing everything infront including the colon with regex is the same but more clear

thanks

To me, more idiomatic would be to perhaps extract your anon fn into two fns: (defn keyword-and-contains-colon? [x] …) and (defn fixup-keyword [kw] …) These two are easy to test at the REPL or with unit/propert-based tests. Then I’d finally compose them:

(defn split-keywords [input]
  (clojure.walk/prewalk
    #(cond-> (keyword-and-contains-colon %) (fixup-keyword))))

It was probably clear, but anyway, I had something like this in mind:

(walk/prewalk (fn fix-any-keywords [x]
                  (if (keyword? x)
                    (string/replace-first (str x) #"^:[^:]+:" ":")
                    x))
                input)

Yes, indeed 🙂