fix: updated Client.watch_snapshot to use proper dataclass as argument type

[petruki]

This pull request refactors the snapshot watching functionality to use a strongly typed callback class (WatchSnapshotCallback) instead of a generic dictionary for handling success and error callbacks. This change improves type safety, code clarity, and maintainability across the codebase. The update affects the main client API, internal watcher logic, tests, and documentation.

Refactoring to typed callback interface:

• Introduced the WatchSnapshotCallback dataclass in switcher_client/lib/snapshot_watcher.py to replace the dictionary-based callback contract, providing explicit success and reject handlers.
• Updated Client.watch_snapshot and all related internal methods to accept and use WatchSnapshotCallback instead of a dictionary, ensuring type safety and clearer callback invocation. [1] [2] [3] [4]

API and usage updates:

• Modified imports and exports in switcher_client/__init__.py, switcher_client/client.py, and all relevant files to expose and use WatchSnapshotCallback. [1] [2]
• Updated usage examples in README.md and test/playground scripts to demonstrate the new callback interface. [1] [2] [3]

Test suite migration:

• Refactored all tests in tests/test_client_watch_snapshot.py to use WatchSnapshotCallback for snapshot watching, ensuring coverage of the new interface and error handling. [1] [2] [3] [4]

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud