Skip to content

feat(realtime): port set() method with timeout updates and cleanup#25

Merged
AdirAmsalem merged 1 commit intomainfrom
feat/realtime-set-port-timeout
Feb 9, 2026
Merged

feat(realtime): port set() method with timeout updates and cleanup#25
AdirAmsalem merged 1 commit intomainfrom
feat/realtime-set-port-timeout

Conversation

@AdirAmsalem
Copy link
Contributor

@AdirAmsalem AdirAmsalem commented Feb 9, 2026

Summary

Add the set() method to the realtime client.

Changes

  • Add set() method: Implement set() method in realtime client with proper async/await handling
  • Pydantic validation: Convert SetInput from dataclass to Pydantic BaseModel for robust input validation
  • Cleanup: Remove unused SetAvatarImageMessage import from messages module
  • Image handling: Improve URL parsing with proper data URI and HTTP(S) URL support
  • Timeout configuration: Add timeout parameters to webrtc_manager for set() operations
  • Tests: Add comprehensive unit tests covering set() method with various input scenarios

Testing

  • Unit tests added for set() method with prompt, image, and enhance parameters
  • Tested with data URIs, HTTP URLs, and binary image inputs
  • Verified timeout handling and error cases
  • All existing tests pass

- Port set() method to realtime client with proper timeout handling
- Convert SetInput from dataclass to Pydantic BaseModel for validation
- Remove unused SetAvatarImageMessage import
- Update image URL parsing with proper data URI handling
- Add comprehensive unit tests for set() method
- Update webrtc_manager with timeout configuration
@AdirAmsalem AdirAmsalem merged commit 2bfd372 into main Feb 9, 2026
8 checks passed
@AdirAmsalem AdirAmsalem deleted the feat/realtime-set-port-timeout branch February 9, 2026 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant