JohnnyMorganz
JohnnyMorganz
Thanks for the info! Will try to take a look this weekend
> Also can you add some tests to Autocomplete.test.cpp for this? Did you end up adding the DataModel types for the new solver? I don't see a test case for...
@checkraisefold Types smoketest is segfaulting, is this because of the flags? Ideally we don't crash even if the flags are not enabled
It actually seems related to the recent luau update to 0.651. Going to undo that sync for now
So sorry for the delay here. I think https://github.com/JohnnyMorganz/StyLua/pull/913 should solve this issue
I've just released v1.7.0 for the extension which should have those fixes. Let me know if have any issues with it
Sounds reasonable to support to me! We can definitely be more clever in our auto require logic, the current implementation is quite rudimentary. > I'm not sure Luau LSP will...
Do you mean whitespace between the fixed variable (Plugin) and the block of requires (Script etc.?) Also do you reckon this should follow the existing config `completion.imports.separateGroupsWithLine`? Or whitespace between...
Seems the same issue as https://github.com/JohnnyMorganz/luau-lsp/issues/583#issuecomment-2226895942 Can you double check the URI used to open the document is the same URI used to send the request?
Ah I think the problem is the incorrect call to `Uri::file` for a non-file-based URI, rather than the added managed files lookup (getTextDocument already handles lookup in managed files)