Skip to content

[WIP] [WIP] Address changes based on feedback for porting PRs from YOUTUBE-EXTENSION#116

Merged
groupthinking merged 6 commits intomainfrom
copilot/sub-pr-113-another-one
Mar 20, 2026
Merged

[WIP] [WIP] Address changes based on feedback for porting PRs from YOUTUBE-EXTENSION#116
groupthinking merged 6 commits intomainfrom
copilot/sub-pr-113-another-one

Conversation

Copy link
Contributor

Copilot AI commented Mar 20, 2026

Thanks for the feedback on #113. I've created this new PR, which merges into #113, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #113
Triggering review: #113 (comment)

@copilot apply changes based on this feedback


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

groupthinking and others added 5 commits March 20, 2026 19:32
Ported from YOUTUBE-EXTENSION PRs #703, #706, #707, #708 which were
opened by Copilot/Claude agents right before the repo was archived.

## PR #706 — Fix duplicate skeleton repo generation
- Added _extract_video_id() and _build_title() helpers to code_generator.py
- deployment_manager.py: replaced timestamp % 10000 with UUID4 suffix
- Root cause fix for the 11 identical uvai-generated-project-* repos

## PR #703 — Skill Builder (deployment learning system)
- New: src/youtube_extension/services/skill_builder.py (368 lines)
- Records deployment outcomes and derives lessons via EMA weighting
- Surfaces top-N lessons for injection into future LLM prompts

## PR #708 — Cloud-native services (Vertex AI + Firestore + Cloud Tasks)
- New: src/youtube_extension/services/cloud/ (5 service modules)
- Firestore shared state replacing in-memory dicts
- Cloud Tasks async video processing queue with retry/backoff
- Vertex AI Agent Builder integration with Google Embedded 2
- New: cloud_api_endpoints.py (/api/v3/* routes)
- New: Dockerfile.cloudrun, infrastructure/cloudrun/setup.sh
- New: docs/cloud-native-architecture.md

## PR #707 — Gemini Vision multimodal ingestion
- Updated gemini_service.py with frame extraction + batch analysis
- Updated enhanced_video_processor.py with visual context pipeline
- Updated videopack/schema.py with VisualContext + VisualElement models
- New: docs/GEMINI_VISION_INTEGRATION.md

Closes YOUTUBE-EXTENSION PRs: #702, #703, #704, #705, #706, #707, #708, #709, #710, #711
Related: EventRelay #111 (secrets still needed)
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@vercel
Copy link
Contributor

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-uvai Error Error Mar 20, 2026 9:37pm

Base automatically changed from port/youtube-extension-salvage to main March 20, 2026 21:35
@groupthinking groupthinking marked this pull request as ready for review March 20, 2026 21:36
Copilot AI review requested due to automatic review settings March 20, 2026 21:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@groupthinking groupthinking merged commit 810197e into main Mar 20, 2026
10 of 15 checks passed
@groupthinking groupthinking deleted the copilot/sub-pr-113-another-one branch March 20, 2026 21:36
Copilot AI requested a review from groupthinking March 20, 2026 21:37
Copilot stopped work on behalf of groupthinking due to an error March 20, 2026 21:37
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.

3 participants