chore: remove unecessary dispose logic, instantiate services inline

[Justintime50]

Description

After much research (online articles, consulting AI, reviewing other large client libraries that don't use it), I determined the dispose logic is not necessary. The C# HTTP client under the hood handles its own dispose and our service objects are not large enough to warrant any real memory issues. A lot of this was noise and most likely did not have a meaningful improvement on memory management.

Separately, the client service instantiation was moved inline to reduce complexity as it was also determined that instantiating them in the constructor vs inline had no real performance implications.

Finally, I fixed the added but yet released RetrieveBatch function to use the parameter set instead of a dictionary. We are moving to have this library only support parameter sets moving forward (eg: ClaimService) which is the newest service added does not have any functions that accept a dictionary, so adjusted the parameter definition.

Finally finally... fixes a nasty compiler error with the Dictionaries file I had to track down.

Testing

Tests pass and re-recorded the new retrieve batch cassette verifying that the adjustment to service instantiation works.

Pull Request Type

Please select the option(s) that are relevant to this PR.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Improvement (fixing a typo, updating readme, renaming a variable name, etc)

[stephenjwatkins]

Looks good as far as i can tell!