clojure-emacs / refactor-nrepl

nREPL middleware to support refactorings in an editor agnostic way
Eclipse Public License 1.0
256 stars 69 forks source link

resolve-missing: don't suggest already imported classes #402

Closed vemv closed 11 months ago

vemv commented 11 months ago

resolve-missing should not suggest e.g. Object since it's already imported by default.

This is best accomplished by inspecting (ns-interns my-ns).

Note that refactor-nrepl.ns.resolve-missing/resolve-missing does not currently accept a ns param (should be optional)

Probably, it's better to add an :already-interned true k-v than to remove the item altogether. That way, we can offer a better UX in clj-refactor.