BetterThanTomorrow / calva

Clojure & ClojureScript Interactive Programming for VS Code
https://marketplace.visualstudio.com/items?itemName=betterthantomorrow.calva
Other
1.61k stars 213 forks source link

Feature Request: autocomplete (and validate) known map keys #2241

Open ikitommi opened 1 year ago

ikitommi commented 1 year ago

Autocomplete (and validate) known map keys

Related to the Slack discussion on #calva and in #clj-kondo: Clojure is a dynamically typed language, but we could leverage the runtime schema/spec data in IDE to provide better type linting and auto-complete of known map keys. This would radically improve developer experience, help people come from ecosystem like TypeScript and lessen the map fatigue.

The below examples are based on malli, but the same approach could work with schema and spec too.

Use Case

  1. describe a data structure as malli schema
  2. attach the schema to one of the following: i. function argument or return value ii. var value iii. edn file (e.g. config file)
  3. instead of just getting a runtime errors, provide static analysis to provide both i. errors without evaluating the code ii. autocomplete on known map keys

Current Status

  1. describe a data structure as malli schema
  2. attach the schema to one of the following:
    1. function argument or return value
    2. var value
    3. edn file (e.g. config file)
  3. instead of just getting a runtime errors, provide static analysis to provide both
    1. errors without evaluating the code
    2. autocomplete on known map keys
      • πŸ‘ this issue

Suggestion

Add IDE-support for autocompleting map keys in case we know the data form via malli schemas for the data. Integrate this to when creating literal data maps (2ii & 2iii) and when accessing schematized function arguments within the function body.

This might require work on all of malli, clj-kondo, lsp-clojure and calva.

Happy to help from malli side.

Screenshots

Malli + clj-kondo βœ…

malli+clj-kondo

Malli Vars 🚧 πŸ‘

malli

Imaginary Autocomplete of known keys πŸ‘

clj-kondo-lsp-malli-autocomplete

JSON + JSON Schema + VSCode βœ…

json

Typescript + VSCode βœ…

typescript

Bonus

As Malli supports lite-syntax, the syntax is not far from typescript (in the simple cases):

const Page = (props: { message: string }) => 
  <div>{props.message}</div>;
(mx/defn page [props :- {:message :string}]
  [:div (:message props)])
bpringe commented 1 year ago

Thanks for the extensive info. @ericdallo, any thoughts here about clojure-lsp's side of supporting this?

PEZ commented 1 year ago

Very cool! Iiuc, we wouldn't need to do anything on the Calva side of things if clojure-lsp is made to support this. Is that how you understand it too, @bpringe?

bpringe commented 1 year ago

Yes, that seems to be the case to me as well.

ericdallo commented 1 year ago

Thanks for the detailed issue description, I do think clojure-lsp with clj-kondo's help should make that work. @ikitommi we have this issue in clojure-lsp similar to this one, but we are blocked because we need clj-kondo analysis (not only findings, but the analysis feature) to be able to provider autocomplete in clojure-lsp.