use packages

[paul-nechifor]

No description provided.

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[greptile-apps]

Greptile Summary

This PR consolidates package dependencies by removing direct aiortc dependency and relying on it transitively through unitree-webrtc-connect-leshy (upgraded to v2.0.7). The aiortc import was moved from module-level to local scope within the raw_video_stream() method. Additionally, dimos-lcm was migrated from a git source to a published PyPI package.

Key Changes:

• Moved aiortc.MediaStreamTrack import to local scope in raw_video_stream() method
• Removed aiortc==1.9.0 as a direct dependency (now provided by unitree-webrtc-connect-leshy)
• Upgraded unitree-webrtc-connect-leshy from >=2.0.4 to >=2.0.7
• Migrated dimos-lcm from git source to PyPI registry
• Updated uv.lock to reflect all dependency changes

Impact:
This refactoring simplifies dependency management by avoiding duplicate aiortc declarations and ensures version consistency through the unitree-webrtc-connect-leshy package. The local import pattern prevents import-time dependencies when the video stream functionality is not immediately needed.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The changes are well-structured dependency management improvements. The aiortc import location change is safe since it's only used within the raw_video_stream() method scope. Package version bumps and migration to PyPI are standard maintenance tasks. The lock file correctly reflects all dependency changes.
• No files require special attention

Important Files Changed

Filename	Overview
dimos/robot/unitree/connection/connection.py	Moved aiortc import from top-level to local scope within raw_video_stream() method to avoid import-time dependency
pyproject.toml	Removed direct aiortc dependency, bumped unitree-webrtc-connect-leshy to >=2.0.7, and migrated dimos-lcm from git to PyPI
uv.lock	Updated lockfile to reflect dependency changes: removed direct aiortc, updated unitree-webrtc-connect-leshy to 2.0.7, and dimos-lcm to PyPI source

Sequence Diagram

sequenceDiagram
    participant App as Application Code
    participant UC as UnitreeWebRTCConnection
    participant UWCL as unitree-webrtc-connect-leshy
    participant AR as aiortc (MediaStreamTrack)
    
    Note over App,AR: Dependency Resolution Changes
    
    App->>UC: raw_video_stream()
    UC->>UC: Import MediaStreamTrack locally
    Note over UC: aiortc import moved from<br/>module-level to function scope
    UC->>AR: from aiortc import MediaStreamTrack
    UC->>UWCL: self.conn.video.add_track_callback(accept_track)
    Note over UWCL: Upgraded to v2.0.7<br/>Now provides aiortc transitively
    UWCL->>AR: Uses aiortc dependency
    
    Note over App,AR: Package Dependency Changes
    
    rect rgb(240, 248, 255)
        Note over App: dimos-lcm: git → PyPI
        Note over App: aiortc: direct dep removed<br/>(now via unitree-webrtc-connect-leshy)
        Note over App: unitree-webrtc-connect-leshy:<br/>2.0.4 → 2.0.7
    end

Loading