ionide-vscode-fsharp icon indicating copy to clipboard operation
ionide-vscode-fsharp copied to clipboard

v7.21.0 is causing intellisense issues in VS code

Open fwaris opened this issue 1 year ago • 12 comments

Just upgraded to v7.21.0.

Seeing red squiggles in files that should compile fine. Hovering over a squiggle gives 'index out of bounds' error message.

Fixed by going back to v7.20.3

Running Window 11 with the following sdks installed

C:\Users\Faisa>dotnet --list-sdks
6.0.416 [C:\Program Files\dotnet\sdk]
8.0.301 [C:\Program Files\dotnet\sdk]
8.0.400 [C:\Program Files\dotnet\sdk]

fwaris avatar Aug 19 '24 14:08 fwaris

A stacktrace would be helpful, to understand if it happens in FSAC or FCS.

vzarytovskii avatar Aug 20 '24 07:08 vzarytovskii

This is how it looks: telegram-cloud-photo-size-2-5343543798967429993-y

And also this: telegram-cloud-photo-size-2-5343543798967429995-y

I have no stacktrace at this point.

vzarytovskii avatar Aug 20 '24 07:08 vzarytovskii

For the record. There no logs anywhere to be found. Even after enabling

"FSharp.verboseLogging": true,
"FSharp.trace.server": "verbose",

It appear to me when developing using Oxpecker library, and when I modify computation expression. It's hard to reproduce for now. so does computatoin expression is the culprit is speculative.

kant2002 avatar Aug 20 '24 08:08 kant2002

Likely suspect is https://github.com/dotnet/fsharp/pull/17096/files which uses HashMultiMap as an underlying structure, which is not threadsafe.

vzarytovskii avatar Aug 20 '24 08:08 vzarytovskii

With the latest version -v7.21.1 - there are some issues still remaining - prob. related to the same fix.

When I edit a file, sometimes I get red squiggles. The error is (when hovering over the squiggles):

internal error: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct. F# Compiler[193](https://docs.microsoft.com/en-us/dotnet/fsharp/language-reference/compiler-messages/fs0193)

fwaris avatar Sep 17 '24 17:09 fwaris

With the latest version -v7.21.1 - there are some issues still remaining - prob. related to the same fix.

When I edit a file, sometimes I get red squiggles.

The error is (when hovering over the squiggles):


internal error: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct. F# Compiler[193](https://docs.microsoft.com/en-us/dotnet/fsharp/language-reference/compiler-messages/fs0193)

This should've been fixed in FCS already. Are there any examples of code, or stack trace?

vzarytovskii avatar Sep 17 '24 20:09 vzarytovskii

This should've been fixed in FCS already

@vzarytovskii was this released in a nuget package yet?

TheAngryByrd avatar Sep 17 '24 20:09 TheAngryByrd

We're using the 8.0.400 build - did the fix come out after?

baronfel avatar Sep 17 '24 20:09 baronfel

We're using the 8.0.400 build - did the fix come out after?

Yes, not in 8 branch though, we didn't backport it there

vzarytovskii avatar Sep 17 '24 20:09 vzarytovskii

Ok, so FSAC updating to https://www.nuget.org/packages/FSharp.Compiler.Service/43.9.100-preview.24422.2 or similar should unblock this.

baronfel avatar Sep 17 '24 20:09 baronfel

I suppose now that RC1 is out we should probably look at doing this anyway.

baronfel avatar Sep 17 '24 20:09 baronfel

Yeah, there are whole bunch of fixes there. Should be fine despite it being a "preview".

vzarytovskii avatar Sep 17 '24 21:09 vzarytovskii