clojure-emacs / cljs-tooling

[DEPRECATED] Tooling support for ClojureScript
60 stars 10 forks source link

code complete don't support plain js object #23

Open shaqtsui opened 6 years ago

shaqtsui commented 6 years ago

When type: (js/THR

Expected complete list: (js/THREE ...

I think a enhancement required here

arichiardi commented 6 years ago

This is something that has been done in lumo and can be easily ported over: https://github.com/anmonteiro/lumo/blob/master/src/cljs/snapshot/lumo/repl.cljs#L1297-L1320

arichiardi commented 6 years ago

Not easily :smile: it looks like the implementation is Clojure for vanilla JVM so the above code would only work for self-hosted REPLs

cichli commented 5 years ago

For the JVM side we can implement this once CLJS-1047 is done. Feel free to vote on that ticket if you're registered on JIRA.

I'm not sure what makes most sense for the self-host case but what Lumo does with completion-candidates-for-node-modules and completion-candidates-for-closure-js looks sane. Maybe we should port that over.

rksm commented 5 years ago

FYI this has been implemented here https://github.com/rksm/clj-suitable but uses dynamic runtime inspection and not static code analysis.