SqlServiceBrokerOptionsBuilder: WithConnectionString before AddSqlServiceBrokerOptions throws NullReferenceException; compression cannot be disabled fluently

[brenpike]

Summary

Two usability defects in SqlServiceBrokerOptionsBuilder (src/Chatter.MessageBrokers.SqlServiceBroker/src/Chatter.MessageBrokers.SqlServiceBroker/Configuration/SqlServiceBrokerOptionsBuilder.cs). Found while rewriting the module README (branch docs/readme-overhaul); the README tells readers to call an AddSqlServiceBrokerOptions overload first.

1. With* methods throw NullReferenceException before an AddSqlServiceBrokerOptions overload

_sqlServiceBrokerOptions is only assigned inside the AddSqlServiceBrokerOptions overloads (:19, :25, :38). WithConnectionString (:53-57) and the other With* setters dereference it directly, so calling them first throws NullReferenceException. Meanwhile the Build() error message (:144-149) says "Use an overload of AddSqlServiceBrokerOptions or WithConnectionString to configure SqlServiceBrokerOptions", which suggests WithConnectionString alone is enough.

Fix options: initialise a default options instance so WithConnectionString works on its own, or throw a clear InvalidOperationException and correct the Build() message.

2. Compression cannot be turned off through the fluent API

WithMessageBodyCompression() (:117-121) sets CompressMessageBody = true, which is already the default. There is no fluent way to disable compression; you have to pass compressMessageBody: false to an AddSqlServiceBrokerOptions overload. Consider WithMessageBodyCompression(bool enabled = true) or a WithoutMessageBodyCompression() counterpart.

Impact

Low: developer-experience traps, no runtime data impact.