fix: revert accidental TypeProviders.SDK and FSharp.Data upgrades breaking Windows build#332
Conversation
…paket.lock The ca9d297 commit ('fix: downgrade fsharp.core') accidentally also upgraded FSharp.TypeProviders.SDK (32fd6037 → 821de018) and FSharp.Data (1ee7eaf8 → 00034f9a) while only intending to downgrade FSharp.Core in the Server project group. TypeProviders.SDK 821de018 introduced Windows-specific NullReferenceExceptions in type providers: - v2/Swagger.i0027.Tests.fs with SwaggerTypeProvider (PR branch) - v3/Swashbuckle.ReturnControllers.Tests.fs with OpenApiClientTypeProvider (master) Revert TypeProviders.SDK back to 32fd6037 and FSharp.Data back to 1ee7eaf8 — the versions confirmed working in CI run 23306033036 — while preserving the FSharp.Core downgrade that was the intent of ca9d297. Co-authored-by: sergey-tihon <1197905+sergey-tihon@users.noreply.github.com>
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot if it is an issue in sdk, open new issue in sdk project and explain the repro and reference this pr |
Investigated. There are two layers to this: Clarification on my Root cause: The intermittent let ctorDefs = lazy (inp.Methods.Entries |> ... |> Array.map (txILConstructorDef this))
let methDefs = lazy (inp.Methods.Entries |> ... |> Array.map (txILMethodDef this))
// etc.
override __.GetConstructors(bf) = ctorDefs.Force() |> Array.filter ...When the F# compiler invokes a type provider from parallel threads (which it does for multi-file projects), two threads can race on the same I cannot create issues in other repos with the tools available to me. Here is the full issue text you can open at https://github.com/fsprojects/FSharp.TypeProviders.SDK/issues/new: Title: Body:
Hypothesis: The The previous code ( Repro project: fsprojects/SwaggerProvider (see PR #331). Once the SDK issue is filed, the Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot fix build issue |
Apply Fantomas-required blank lines in match expression bodies and incorporate base branch changes (module rename, null-safety checks) to ensure the merged PR result passes the CheckFormat target. Co-authored-by: sergey-tihon <1197905+sergey-tihon@users.noreply.github.com> Agent-Logs-Url: https://github.com/fsprojects/SwaggerProvider/sessions/3bf98de8-e32b-4375-a14a-a98e3b3103ce
Co-authored-by: sergey-tihon <1197905+sergey-tihon@users.noreply.github.com> Agent-Logs-Url: https://github.com/fsprojects/SwaggerProvider/sessions/3bf98de8-e32b-4375-a14a-a98e3b3103ce
Co-authored-by: sergey-tihon <1197905+sergey-tihon@users.noreply.github.com> Agent-Logs-Url: https://github.com/fsprojects/SwaggerProvider/sessions/3bf98de8-e32b-4375-a14a-a98e3b3103ce
Fixed in The fix incorporates those base branch changes alongside the proper blank lines as required by Fantomas 7.0.5. Also added Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
ca9d297("fix: downgrade fsharp.core") inadvertently bumped two GitHub-sourced dependencies inpaket.lockwhile only intending to downgradeFSharp.Corein the Server project group. TypeProviders.SDK821de018introduced Windows-specificNullReferenceExceptions in generated type providers, manifesting as:FS3033inv2/Swagger.i0027.Tests.fs(this branch)FS3033inv3/Swashbuckle.ReturnControllers.Tests.fs(master)Changes
paket.lock— revert two accidentally-upgraded GitHub deps to their last known-good commits (verified passing in CI run23306033036):FSharp.TypeProviders.SDK:821de018→32fd6037FSharp.Data(NameUtils/Pluralizer):00034f9a→1ee7eaf8The intentional
FSharp.Coredowngrade fromca9d297is preserved.📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.