[repo-assist] fix: skip empty Cookie header when no cookie parameters are present

[github-actions]

🤖 This PR was created by Repo Assist, an automated AI assistant.

Summary

When an API operation has no cookie parameters (the common case), OperationCompiler was always prepending ("Cookie", "") to the headers list. fillHeaders only filters null values, not empty strings, so an empty Cookie: header was sent with every HTTP request.

Root cause

The cookie header construction unconditionally built:

("Cookie", cookieHeader) :: (%headers)

even when cookieHeader was "" (no cookies declared on the operation).

Fix

Only prepend the Cookie header when the constructed cookie string is non-empty:

if String.IsNullOrEmpty cookieHeader then
    %headers
else
    ("Cookie", cookieHeader) :: (%headers)

This means:

• Operations with cookie parameters: behaviour unchanged
• Operations without cookie parameters: no spurious Cookie: header sent

Impact

• Eliminates the unnecessary Cookie: header from every non-cookie API call
• Reduces request header overhead (minor, but correct)
• No breaking change: callers using cookie parameters work identically

Test Status

Total: 465, Errors: 0, Failed: 0, Skipped: 1, Time: 1.146s

All 465 unit tests pass. Format check passes (dotnet fantomas --check).

Generated by 🌈 Repo Assist, see workflow run. Learn more.

Generated by 🌈 Repo Assist, see workflow run. Learn more.

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@298f992955146a6731d380a9de808e17861708e5