Skip to content

[WIP] [WIP] Address feedback from review on porting PRs from YOUTUBE-EXTENSION#119

Merged
groupthinking merged 7 commits intomainfrom
copilot/sub-pr-113-please-work
Mar 20, 2026
Merged

[WIP] [WIP] Address feedback from review on porting PRs from YOUTUBE-EXTENSION#119
groupthinking merged 7 commits intomainfrom
copilot/sub-pr-113-please-work

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 6 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>
Co-authored-by: Copilot <175728472+Copilot@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.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
v0-uvai Skipped Skipped 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
@groupthinking groupthinking self-requested a review as a code owner March 20, 2026 21:36
Copilot AI review requested due to automatic review settings March 20, 2026 21:36
@groupthinking groupthinking merged commit 9cd4687 into main Mar 20, 2026
11 of 14 checks passed
@groupthinking groupthinking deleted the copilot/sub-pr-113-please-work branch March 20, 2026 21:37
Copilot stopped work on behalf of groupthinking due to an error March 20, 2026 21:37
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 encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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