Stop Debug builds writing StandardOutput.txt/StandardError.txt into the project directory

docs/release-notes/.FSharp.Compiler.Service/11.0.100.md

@@ -1,6 +1,8 @@
 ### Fixed
 
 * Fix internal error FS0192 "Iterate2D" when a `[<ReflectedDefinition(true)>]` parameter auto-quotes an argument that captures a not-yet-generalized use of an inferred generically-recursive function. The auto-quoted (`Expr.WithValue`) copy now keeps a fresh link to the recursive-value use so it receives the same inferred type arguments as the executable expression at the letrec point. ([Issue #20379](https://github.com/dotnet/fsharp/issues/20379))
+* Debug builds of FCS no longer write `StandardOutput.txt`/`StandardError.txt` into the project (or script) directory when probing `dotnet --version` for the SDK. With a Debug VSIX in Visual Studio those files landed inside SDK-style project folders and kept re-triggering project re-evaluation -> design-time build -> NuGet restore -> F# re-check.
+* Debug builds of FCS no longer write `StandardOutput.txt`/`StandardError.txt` into the project (or script) directory when probing `dotnet --version` for the SDK. With a Debug VSIX in Visual Studio those files landed inside SDK-style project folders and kept re-triggering project re-evaluation -> design-time build -> NuGet restore -> F# re-check. ([PR #20458](https://github.com/dotnet/fsharp/pull/20458))
 * Fix `NativePtr.stackalloc` nested in a larger expression (e.g. a call argument or the right of an assignment) producing an assembly that throws `InvalidProgramException` at load. ([Issue #8083](https://github.com/dotnet/fsharp/issues/8083), [PR #20302](https://github.com/dotnet/fsharp/pull/20302))
 * Fix internal error "Unexpected generalized type variables when compiling an active pattern" when an active pattern is used in a `let` binding whose right-hand side is a generic value, e.g. `let (T) = id`. Such a binding is now checked like the equivalent `match` and is not generalized. ([Issue #16856](https://github.com/dotnet/fsharp/issues/16856), [PR #20383](https://github.com/dotnet/fsharp/pull/20383))
 * Fix Release-only (`--optimize+`) `System.InvalidProgramException` from `Seq.collect` / `yield!` over a value-type (struct) collection implementing `seq<'T>` (e.g. `ImmutableArray<_>`) when materialised with `List.ofSeq` / `Seq.toList` / `Seq.toArray` or a list/array comprehension. The collector lowering now boxes a struct sub-collection to `seq<'T>` before calling `AddMany`/`AddManyAndClose` (matching the coercion the type checker already inserts for `yield!`), and uses `unit` as the try/finally result type instead of the body type (removing a spurious `ldnull` store). ([Issue #20203](https://github.com/dotnet/fsharp/issues/20203))

src/Compiler/Driver/FxResolver.fs

@@ -96,12 +96,7 @@ type internal FxResolver
                     raise (TimeoutException(sprintf "Timeout executing command '%s' '%s'" psi.FileName psi.Arguments))
                 else
                     p.WaitForExit()
-#if DEBUG
-            if workingDir.IsSome then
-                FileSystem.OpenFileForWriteShim(Path.Combine(workingDir.Value, "StandardOutput.txt")).WriteAllLines(outputList)
 
-                FileSystem.OpenFileForWriteShim(Path.Combine(workingDir.Value, "StandardError.txt")).WriteAllLines(errorsList)
-#endif
             p.ExitCode, outputList.ToArray(), errorsList.ToArray()
         else
             -1, Array.empty, Array.empty