Make null handling explicit in toQueryParams and fix misleading test comment

[Copilot]

• Analyze the reviewer's feedback about box Option<int>.None being null and inconsistency with test expectations
• Add explicit null guard (if isNull obj then []) at the start of toQueryParams to make null handling unambiguous
• Update the misleading test comment that said "None for reference option types" (all option Nones compile to null, including value-type options like int option)
• Apply Fantomas formatting (dotnet fantomas) to fix formatting issues
• Validate all 177 tests still pass

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

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:

• petstore.swagger.io
  • Triggering command: /usr/share/dotnet/dotnet /usr/share/dotnet/dotnet /usr/share/dotnet/sdk/10.0.102/FSharp/fsc.dll @/tmp/MSBuildTempUixkFM/tmpcc3bc09d085244edb13985cdeb565af3.rsp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[sergey-tihon]

@copilot (Some files need formatting, run dotnet fake build -t Format to format them)

[Copilot]

@copilot (Some files need formatting, run dotnet fake build -t Format to format them)

Applied Fantomas formatting in e80051f — the match block inside the else branch needed additional indentation.

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:

• petstore.swagger.io
  • Triggering command: /usr/share/dotnet/dotnet /usr/share/dotnet/dotnet /usr/share/dotnet/sdk/10.0.102/FSharp/fsc.dll @/tmp/MSBuildTempggrWpZ/tmp62dc5558b5d8490dbeff0380695d3b60.rsp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

Pull request overview

This PR clarifies null/Option.None behavior for query parameter serialization by making null handling explicit in RuntimeHelpers.toQueryParams, and fixes a misleading test comment about how F# options are represented at the CLR level.

Changes:

• Add an explicit null guard at the start of toQueryParams to return an empty query-param list.
• Update the test comment to reflect that None for any F# option<_> is represented as null at runtime.
• Apply Fantomas formatting to the touched areas.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/SwaggerProvider.Runtime/RuntimeHelpers.fs	Makes toQueryParams return [] for null inputs (avoids NullReferenceException on obj.ToString()).
tests/SwaggerProvider.Tests/RuntimeHelpersTests.fs	Corrects test commentary about Option.None CLR representation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

array<Option<Guid>> is currently serialized via toStrArray, which will call ToString() on Some g (producing e.g. Some ...) and will throw NullReferenceException for None elements (since None is represented as null). This should use the same option-array handling as the other option cases (e.g., toStrArrayOpt) so None values are skipped and Some values serialize as the underlying Guid.

Suggested change

	| :? array<Option<Guid>> as xs -> xs |> toStrArray name
	| :? array<Option<Guid>> as xs -> xs |> toStrArrayOpt name