Yannick Daveluy

Results 16 comments of Yannick Daveluy

I confirm that it is with the test unreleased version.

I think that the OS and configuration doesn't matter. Here is a test case to reproduce the issue: ` static void testIncompleEscapeSequence() { System.out.print("normal text\u001b"); System.out.flush(); try { Thread.sleep(2000l); }...

The same issue with dolphincoder ``` ollama run dolphincoder pulling manifest pulling 338f3218c250... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████████▏ 9.1 GB pulling 4ec42cd966c9... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████████▏ 12 KB pulling 62fbfd9ed093... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████████▏ 182 B pulling...

Another alternative would be to import types with a specific prefix (e.g. `_langium_`) and to prohibit rule names in the grammar from starting with this prefix ? e.g: ``` import...

Hey @msujew, I can try to provide a PR for this issue. I think that doing a single import like ``` import * as langium from 'langium'; ``` and adding...

Yes, in your refactoring, 'student' and 'teacher' are on different AstNodes, and it works properly. In my case, I think the issue comes from `DefaultCompletionProvider.findFeaturesAt`, which does not return the...

Thanks for the workaround. I continued investigating this issue and I noticed that we enter in this branch for the missing feature: https://github.com/eclipse-langium/langium/blob/b2a6fe9684434117eb6a344fc2721e3c26d989bf/packages/langium/src/lsp/completion/follow-element-computation.ts#L131 It means that the previous feature is...

Hi @msujew , I have some doubts about the usability of this feature. It will generate an AST that will be good for scoping and validation but annoying for code...

@spoenemann Yes you're right, a more sophisticated version of DeppPartial works well; it makes properties optional except if it starts with `$` or is a boolean or an array: ```ts...

@spoenemann Yes I think it could be usefull to add it to Langium. For the new name, no idea :-) @msujew According to me it is still very error prone...