Odin Dahlström

Actually, I think I may have found a more "correct" way to work around it (how I'm guessing it's done in the Eclipse IDE). What we probably should do is...

I guess another workaround would be to always call `CoreASTProvider#getAST` after disposing the old AST when we get the document change request. If we rely on using `JobHelpers.waitForJobs(DocumentLifeCycleHandler.DOCUMENT_LIFE_CYCLE_JOBS, monitor);`, I...

I didn't thoroughly test performance, but I don't remember there being any noticeable impact. As far as I understand, using `IWorkspace#run()` instead of `JobHelpers.waitForJobs` should be fairly equivalent, the main...

Yeah that's a mistake on my part, `DocumentMonitor#initialVersion` should be an `Integer` since it comes from a map and could be `null` (hence the NPE when unboxing it). But it...

Any updates on this? I noticed [you can now](https://github.com/microsoft/vscode-java-dependency/pull/299) create new classes/packages from the context menu, which is great! But it's really annoying to have to go back to the...

Well yes, because you're using Purpur, not Spigot. My point was that Commodore seems to support only Paper (and derivatives like Purpur), not Spigot, as of Minecraft 1.19. Which, again,...

> But it is a bit strange that the document version is `null` when a `DocumentMonitor` is constructed for a semantic tokens request. That would either suggest a client error...

I have now had some time to investigate this in more detail again. First of all, I don't think my previous suggestion to use `IWorkspace#run` is a good idea anymore,...

> I feel like this issue with the shared AST provider might affect other parts of the code Indeed, pretty much anything that uses `CoreASTProvider` could be affected by this...

The second error (from the Terralith JAR) is caused by AppleDouble metadata, see douira/fractureiser-web-detector#6. I don't know how many JARs contain AppleDouble metadata, but it's probably not an insignificant amount.