strawberry-django icon indicating copy to clipboard operation
strawberry-django copied to clipboard

LSP: warn on `fields="__all__"` and add code action to expose all fields

Open patrick91 opened this issue 1 year ago • 0 comments

We don't have an LSP for Strawberry Django, but in case we add one I think this feature would be neat to have.

We allow to use fields="__all__", which might lead to security issues, so the LSP could warn when using that, and also provide a code action that does what __all__ does, but with code

From this:

@strawberry.django.type(models.Flyer, fields="__all__")
class Flyer:
    ...

To this:

@strawberry.django.type(models.Flyer)
class Flyer:
    id: auto
    title: auto

We could also add code action to expose additional fields :)

patrick91 avatar Jun 14 '24 16:06 patrick91