[inspect] Add support for printing compact qualified keywords

[alexander-yakushev]

This is an idea to address clojure-emacs/cider#3841. I think most of the time when people use long qualified keywords, it happens when using shortened syntax ::foo or ::alias/foo. The proposed feature tries to reconstruct the shortened syntax by providing a config paramter :pov-ns (Point-of-View namespace) from where the inspection is initiated. Example, inspecting this

Yields this:

---

• You've added tests to cover your change(s)
• You've updated the changelog (if adding/changing user-visible functionality)

[alexander-yakushev]

This stuff is O(N) from the number of namespace aliases in the POV-ns, so it banks on the assumption that people don't alias hundreds of namespaces. I think it is good enough for now, can improve later.

[bbatsov]

Yeah, I think that's fine. Perhaps you can leave this in the code as a comment for future reference.

[bbatsov]

Maybe mention in the docstring that pov stands for "point-of-view", so people don't wonder about this?

[bbatsov]

Nice improvement!

[yuhan0]

Isn't this conceptually similar to the notion of 'tidy' output from the macroexpand middleware? https://github.com/clojure-emacs/cider-nrepl/blob/master/src/cider/nrepl/middleware/macroexpand.clj

I haven't looked into it closely, but this might be a good opportunity to extract some common logic or at least compare the impls for semantic alignment - eg whether keywords and symbols should be handled differently and how

[alexander-yakushev]

Good point, I should take a look if something can be reused between the two.