Skip to content

feat(demohouse/multimedia): multimedia agent backend#233

Merged
cuericlee merged 3 commits into
volcengine:mainfrom
zakahan:feat/mutimedia
Dec 16, 2025
Merged

feat(demohouse/multimedia): multimedia agent backend#233
cuericlee merged 3 commits into
volcengine:mainfrom
zakahan:feat/mutimedia

Conversation

@zakahan

@zakahan zakahan commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@zakahan zakahan changed the title feat: demohouse of mutimedia backend feat(demohouse): demohouse of mutimedia backend Dec 15, 2025
@zakahan zakahan force-pushed the feat/mutimedia branch 4 times, most recently from ceb4911 to f7cd342 Compare December 15, 2025 06:17
@zakahan zakahan changed the title feat(demohouse): demohouse of mutimedia backend feat(demohouse/multimedia): mutimedia agent backend Dec 15, 2025
cuericlee
cuericlee previously approved these changes Dec 15, 2025

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request introduces a comprehensive multimedia agent backend system for the demohouse project. The implementation includes multiple specialized agents for e-commerce video generation, along with supporting services for URL shortening and media processing.

Key Changes:

  • Multi-agent system for video generation workflow (market, director, evaluate, and release agents)
  • Short link service for URL management
  • Image and video generation capabilities with evaluation pipeline

Reviewed changes

Copilot reviewed 101 out of 101 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
demohouse/mutimedia/backend/pyproject.toml Defines project dependencies including veadk-python, google-adk, and multimedia processing libraries
demohouse/mutimedia/backend/app/short_link/* Implements short URL service with Redis and in-memory storage modes
demohouse/mutimedia/backend/app/release-agent/* Agent for video composition and merging with VOD support
demohouse/mutimedia/backend/app/multimedia-agent/* Root agent coordinating all sub-agents for video generation
demohouse/mutimedia/backend/app/market-agent/* Handles product analysis and marketing script generation
demohouse/mutimedia/backend/app/evaluate-agent/* Evaluates generated images and videos for quality
demohouse/mutimedia/backend/app/director-agent/* Manages storyboard creation, image and video generation
Comments suppressed due to low confidence (4)

demohouse/mutimedia/backend/pyproject.toml:1

  • The project name contains 'veadk' but the directory path uses 'mutimedia' (should be 'multimedia'). This inconsistency suggests a typo in the directory structure.
    demohouse/mutimedia/backend/app/release-agent/src/release_agent/tools/video_combine_vod.py:1
  • Missing await keyword for async function call. This will assign a coroutine object instead of the resolved URL value.
    demohouse/mutimedia/backend/app/release-agent/src/release_agent/tools/video_combine.py:1
  • Error message states "10GB" but the actual limit check at line 131 is 512MB (max_file_size = 512 * 1024 * 1024). The error message should match the actual limit.
    demohouse/mutimedia/backend/app/market-agent/src/market_agent/hook/format_hook.py:1
  • This creates a list of boolean True values, but line 93-96 attempts to zip it with resources and check boolean values. Should be: image_indices = [i for i, (_, is_image, _) in enumerate(is_images_results) if is_image] to get the actual indices.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread demohouse/mutimedia/backend/app/director-agent/src/requirements.txt Outdated
@zakahan zakahan changed the title feat(demohouse/multimedia): mutimedia agent backend feat(demohouse/multimedia): multimedia agent backend Dec 15, 2025
@cuericlee cuericlee merged commit fce32e8 into volcengine:main Dec 16, 2025
3 checks passed
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.

5 participants