feat: Enhance ascleandict to handle non-picklable fields

[MrKevinWeiss]

This pull request refactors the ascleandict utility in hlpr.py to improve its handling of dataclass-to-dictionary conversion, especially for non-serializable and non-picklable fields. The update introduces a new json_serializable option to ensure all returned dictionaries can be safely serialized to JSON, and includes new tests to verify this behavior.

Improvements to dataclass conversion:

• Refactored the ascleandict method to avoid using dataclasses.asdict, preventing errors with non-picklable fields such as thread locks, by implementing a custom recursive converter.
• Added the json_serializable parameter to ascleandict, which converts non-JSON-serializable types to strings, ensuring compatibility with JSON serialization.
• Updated imports in hlpr.py to support the new implementation, including adding json, fields, and is_dataclass.

Testing enhancements:

• Added a test (test_ascleandict_non_picklable_field) to verify that ascleandict correctly handles dataclasses containing non-picklable fields and that the json_serializable option enables JSON serialization without errors.
• Imported json in tests/test_lob_hlpr.py to support new test cases for JSON serialization.

[Copilot]

Pull request overview

Refactors LobHlpr.ascleandict to avoid dataclasses.asdict()/deepcopy issues with non-picklable fields and adds an option intended to ensure JSON-safe output, along with tests for the new behavior.

Changes:

• Replaced dataclasses.asdict() usage with a custom recursive converter to avoid deepcopy/pickling failures.
• Added json_serializable option intended to coerce non-JSON-serializable values to strings.
• Added a regression test covering dataclasses containing a thread lock and JSON serialization behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
src/lob_hlpr/hlpr.py	Reimplements ascleandict conversion/cleanup logic and introduces json_serializable.
tests/test_lob_hlpr.py	Adds a new test for non-picklable fields and JSON serialization compatibility.

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.