Skip to content

Fix aspect ratio limitation for long videos#2

Merged
BSoDium merged 2 commits intomainfrom
fix/aspect-ratio-limitation
Feb 23, 2026
Merged

Fix aspect ratio limitation for long videos#2
BSoDium merged 2 commits intomainfrom
fix/aspect-ratio-limitation

Conversation

@BSoDium
Copy link
Owner

@BSoDium BSoDium commented Feb 23, 2026

This pull request introduces several enhancements to the development workflow, improves video handling in Telegram sync, and refactors authentication code for clarity and maintainability. The most significant changes include adding a devcontainer for a streamlined development environment, improving how video metadata is extracted and sent to Telegram, and refactoring authentication token handling.

Development Environment Improvements:

  • Added a .devcontainer/devcontainer.json file to enable development in a standardized containerized environment, including Python and debugging extensions, and a post-create command to install dependencies.
  • Refactored the Dockerfile to use multi-stage builds for separate production and development images, and updated compose.yml to build using the production target. [1] [2]

Video Metadata Extraction and Telegram Sync:

  • Added a new get_video_metadata function in tools.py to extract width, height, and duration from MP4 video bytes using ffprobe, and integrated it into telegram_sync.py to send videos to Telegram with accurate metadata (width, height, duration). [1] [2] [3]

Authentication and Code Quality Improvements:

  • Refactored google_auth_wrapper.py to import authentication constants and utility functions directly, improving readability and maintainability. Enhanced error handling for homegraph responses and improved logging by using a censor function for access tokens. [1] [2] [3] [4] [5]

VSCode Workspace Enhancements:

  • Updated google-nest-telegram-sync.code-workspace to add a debug configuration for main.py and included the new devuser in spellchecker settings.

Copilot AI review requested due to automatic review settings February 23, 2026 17:08
@BSoDium BSoDium changed the title Fix/aspect ratio limitation Fix aspect ratio limitation for long videos Feb 23, 2026
@BSoDium BSoDium merged commit 8ffa198 into main Feb 23, 2026
3 checks passed
@BSoDium BSoDium deleted the fix/aspect-ratio-limitation branch February 23, 2026 17:11
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