quota.
- !pathname.startsWith('/api/') ||
- shouldSkipRateLimit(pathname)
+ request.method === 'OPTIONS'
) {
return NextResponse.next();
}
diff --git a/apps/web/tests/e2e/production.spec.ts b/apps/web/tests/e2e/production.spec.ts
new file mode 100644
index 000000000..b4dc11edb
--- /dev/null
+++ b/apps/web/tests/e2e/production.spec.ts
@@ -0,0 +1,32 @@
+import { test, expect } from '@playwright/test';
+
+const BASE_URL = process.env.BASE_URL || 'http://localhost:3000';
+
+test.describe('EventRelay Production E2E', () => {
+ test('homepage connectivity', async ({ page }) => {
+ await page.goto(BASE_URL);
+ await expect(page).toHaveTitle(/UVAI/i);
+ });
+
+ test('dashboard navigation', async ({ page }) => {
+ await page.goto(`${BASE_URL}/dashboard`);
+ await expect(page.locator('h1').first()).toBeVisible({ timeout: 15000 });
+ });
+
+ test('v1 health check proxy', async ({ page }) => {
+ const response = await page.request.get(`${BASE_URL}/api`);
+ expect(response.ok()).toBeTruthy();
+ const data = await response.json();
+ expect(data.status).toBe('operational');
+ });
+
+ test('features page routing', async ({ page }) => {
+ const response = await page.goto(`${BASE_URL}/features`);
+ if (response?.status() === 429) {
+ console.warn('Rate limited on features page, skipping content check');
+ return;
+ }
+ expect(response?.ok()).toBeTruthy();
+ await expect(page.locator('body')).toContainText(/workflow|template/i);
+ });
+});
diff --git a/config/agent_network.json b/config/agent_network.json
index 4e8041ce9..83aa07bca 100644
--- a/config/agent_network.json
+++ b/config/agent_network.json
@@ -50,18 +50,6 @@
"get_capabilities",
"get_context"
]
- },
- "gtm-skills": {
- "endpoint": "internal://eventrelay_skills",
- "tools": [
- "content-generation",
- "seo-optimizer",
- "social-scheduler",
- "lead-scorer",
- "email-campaign",
- "analytics-dashboard",
- "ab-testing"
- ]
}
},
"agents": [
@@ -69,338 +57,113 @@
"id": "video-ingest",
"name": "Video Ingest Agent",
"role": "Extract and analyze video content",
- "tools": [
- "process_video_markdown",
- "get_transcript",
- "analyze_video"
- ],
- "capabilities": [
- "youtube_processing",
- "transcript_extraction",
- "content_analysis"
- ]
+ "tools": ["process_video_markdown", "get_transcript", "analyze_video"],
+ "capabilities": ["youtube_processing", "transcript_extraction", "content_analysis"]
},
{
"id": "architect",
"name": "Architecture Agent",
"role": "Determine tech stack and project structure",
- "tools": [
- "determine_architecture",
- "get_context",
- "get_capabilities"
- ],
- "capabilities": [
- "architecture_design",
- "tech_selection",
- "knowledge_integration"
- ]
+ "tools": ["determine_architecture", "get_context", "get_capabilities"],
+ "capabilities": ["architecture_design", "tech_selection", "knowledge_integration"]
},
{
"id": "code-gen",
"name": "Code Generation Agent",
"role": "Generate application code from specifications",
- "tools": [
- "generate_fullstack",
- "generate_files",
- "get_error_patterns"
- ],
- "capabilities": [
- "code_generation",
- "fullstack_apps",
- "template_application"
- ]
+ "tools": ["generate_fullstack", "generate_files", "get_error_patterns"],
+ "capabilities": ["code_generation", "fullstack_apps", "template_application"]
},
{
"id": "build-validator",
"name": "Build Validator Agent",
"role": "Test builds and fix errors",
- "tools": [
- "validate_build",
- "get_error_patterns",
- "learn_from_error",
- "suggest_fix"
- ],
- "capabilities": [
- "build_testing",
- "error_resolution",
- "skill_learning"
- ]
+ "tools": ["validate_build", "get_error_patterns", "learn_from_error", "suggest_fix"],
+ "capabilities": ["build_testing", "error_resolution", "skill_learning"]
},
{
"id": "deployer",
"name": "Deployment Agent",
"role": "Deploy to GitHub and Vercel",
- "tools": [
- "deploy_to_github_and_vercel",
- "create_repo",
- "push_code",
- "deploy_project",
- "get_deployment_status"
- ],
- "capabilities": [
- "github_deployment",
- "vercel_deployment",
- "ci_cd"
- ]
+ "tools": ["deploy_to_github_and_vercel", "create_repo", "push_code", "deploy_project", "get_deployment_status"],
+ "capabilities": ["github_deployment", "vercel_deployment", "ci_cd"]
},
{
"id": "knowledge-capture",
"name": "Knowledge Capture Agent",
"role": "Learn from each pipeline run",
- "tools": [
- "capture_technology",
- "get_capabilities",
- "learn_from_error"
- ],
- "capabilities": [
- "continuous_learning",
- "pattern_recognition",
- "capability_generation"
- ]
+ "tools": ["capture_technology", "get_capabilities", "learn_from_error"],
+ "capabilities": ["continuous_learning", "pattern_recognition", "capability_generation"]
},
{
"id": "vercel-background",
"name": "Vercel Background Execution Agent",
"role": "Migrate custom fireAndForget to official waitUntil for post-response work per confirmed outcomes",
- "tools": [
- "validate_build",
- "suggest_fix",
- "get_error_patterns"
- ],
- "capabilities": [
- "vercel_functions",
- "background_execution",
- "stream_close_independence"
- ]
+ "tools": ["validate_build", "suggest_fix", "get_error_patterns"],
+ "capabilities": ["vercel_functions", "background_execution", "stream_close_independence"]
},
{
"id": "rate-limit-middleware",
"name": "Rate Limit Middleware Activator",
"role": "Wire proxy.ts as active middleware, remove in-process memory cache, enforce dev-only fallback",
- "tools": [
- "validate_build",
- "suggest_fix"
- ],
- "capabilities": [
- "middleware",
- "rate_limiting",
- "redis_vs_memory"
- ]
+ "tools": ["validate_build", "suggest_fix"],
+ "capabilities": ["middleware", "rate_limiting", "redis_vs_memory"]
},
{
"id": "vercel-foundation",
"name": "Vercel Functions Foundation Agent",
"role": "Add @vercel/functions package and establish consistent pattern + vercel.json notes",
- "tools": [
- "validate_build",
- "suggest_fix"
- ],
- "capabilities": [
- "dependency_management",
- "vercel_config",
- "package_foundation"
- ]
+ "tools": ["validate_build", "suggest_fix"],
+ "capabilities": ["dependency_management", "vercel_config", "package_foundation"]
},
{
"id": "verification-gate",
"name": "Verification Gate Agent",
"role": "Implement and execute comparison methods (greps, curl bursts, stream timing, runbook smokes, build/lint) against defined outcomes",
- "tools": [
- "validate_build",
- "get_error_patterns",
- "learn_from_error"
- ],
- "capabilities": [
- "verification",
- "e2e_testing",
- "observability"
- ]
+ "tools": ["validate_build", "get_error_patterns", "learn_from_error"],
+ "capabilities": ["verification", "e2e_testing", "observability"]
},
{
"id": "performance-observability",
"name": "Performance & Observability Agent",
"role": "Analyze cold starts, duration, cost impact of waitUntil + rate limits; recommend vercel.json + logging",
- "tools": [
- "get_capabilities",
- "suggest_fix"
- ],
- "capabilities": [
- "performance_analysis",
- "cost_optimization",
- "dashboard_metrics"
- ]
+ "tools": ["get_capabilities", "suggest_fix"],
+ "capabilities": ["performance_analysis", "cost_optimization", "dashboard_metrics"]
},
{
"id": "security-auditor",
"name": "Security Auditor (Rate Limit Focus)",
"role": "Audit rate limiting for bypasses, header integrity, prod bypass behavior",
- "tools": [
- "suggest_fix"
- ],
- "capabilities": [
- "security_audit",
- "rate_limit_security",
- "header_validation"
- ]
+ "tools": ["suggest_fix"],
+ "capabilities": ["security_audit", "rate_limit_security", "header_validation"]
},
{
"id": "quality-agent",
"name": "Quality Agent",
"role": "Code quality, style, and test coverage for the changes; run full verification matrix",
- "tools": [
- "validate_build",
- "get_error_patterns",
- "learn_from_error",
- "suggest_fix"
- ],
- "capabilities": [
- "code_quality",
- "test_enhancement",
- "lint_build"
- ]
+ "tools": ["validate_build", "get_error_patterns", "learn_from_error", "suggest_fix"],
+ "capabilities": ["code_quality", "test_enhancement", "lint_build"]
},
{
"id": "strategy-agent",
"name": "Strategy Agent",
"role": "Ensure changes align with overall architecture, anti-framework rules, and REAL_MODE_ONLY",
- "tools": [
- "get_capabilities",
- "get_context"
- ],
- "capabilities": [
- "strategic_review",
- "architecture_alignment"
- ]
+ "tools": ["get_capabilities", "get_context"],
+ "capabilities": ["strategic_review", "architecture_alignment"]
},
{
"id": "precision-extractor",
"name": "Precision Extractor (for side-effect sites)",
"role": "Identify and precisely migrate every remaining fire-and-forget / .catch site across all routes",
- "tools": [
- "suggest_fix"
- ],
- "capabilities": [
- "pattern_extraction",
- "minimal_migration"
- ]
+ "tools": ["suggest_fix"],
+ "capabilities": ["pattern_extraction", "minimal_migration"]
},
{
"id": "launch-plan",
"name": "Launch Plan Agent",
"role": "Update runbooks, docs, and deployment checklist; coordinate final smoke + prod verification",
- "tools": [
- "suggest_fix",
- "capture_technology"
- ],
- "capabilities": [
- "documentation",
- "runbook_update",
- "deployment_readiness"
- ]
- },
- {
- "id": "content-generation",
- "name": "Content Generation Skill",
- "role": "Generate blog/social posts from video transcripts",
- "tools": [
- "generate_fullstack"
- ],
- "capabilities": [
- "content_generation",
- "blog_posts",
- "social_posts"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "youtube.video.published"
- ]
- },
- {
- "id": "seo-optimizer",
- "name": "SEO Optimizer Skill",
- "role": "Optimize video titles, descriptions, tags for search discoverability",
- "tools": [
- "analyze_video"
- ],
- "capabilities": [
- "seo_optimization",
- "metadata_enhancement"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "youtube.video.uploaded"
- ]
- },
- {
- "id": "social-scheduler",
- "name": "Social Scheduler Skill",
- "role": "Schedule cross-platform social media posts",
- "tools": [],
- "capabilities": [
- "social_media",
- "scheduling",
- "cross_platform"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "ai.content.generated"
- ]
- },
- {
- "id": "lead-scorer",
- "name": "Lead Scorer Skill",
- "role": "Score leads based on engagement signals",
- "tools": [],
- "capabilities": [
- "lead_scoring",
- "engagement_analysis"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "youtube.analytics.updated"
- ]
- },
- {
- "id": "email-campaign",
- "name": "Email Campaign Skill",
- "role": "Generate and send email sequences based on lead scoring",
- "tools": [],
- "capabilities": [
- "email_generation",
- "campaign_management"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "crm.lead.scored"
- ]
- },
- {
- "id": "analytics-dashboard",
- "name": "Analytics Dashboard Skill",
- "role": "Aggregate metrics into dashboard data",
- "tools": [],
- "capabilities": [
- "metrics_aggregation",
- "dashboard_generation"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "system.cron.daily"
- ]
- },
- {
- "id": "ab-testing",
- "name": "A/B Testing Skill",
- "role": "Run A/B tests on thumbnails and titles",
- "tools": [],
- "capabilities": [
- "ab_testing",
- "variant_management"
- ],
- "skill_source": "uvai-skills",
- "trigger_events": [
- "youtube.video.uploaded"
- ]
+ "tools": ["suggest_fix", "capture_technology"],
+ "capabilities": ["documentation", "runbook_update", "deployment_readiness"]
}
]
}
\ No newline at end of file
diff --git a/docs/maintenance/stale-pr-report-2026-07-14.md b/docs/maintenance/stale-pr-report-2026-07-14.md
deleted file mode 100644
index 15fa03f5b..000000000
--- a/docs/maintenance/stale-pr-report-2026-07-14.md
+++ /dev/null
@@ -1,46 +0,0 @@
-# Stale PR Analysis and Remediation Report
-
-Date: 2026-07-14
-
-## Summary
-A comprehensive audit of the 70 open Pull Requests was performed to identify stale or redundant updates. Nine (9) PRs were identified as stale or superseded by newer, more comprehensive PRs.
-
-## Identified Stale PRs
-
-| Stale PR | Title | Superseded By | Reason |
-| :--- | :--- | :--- | :--- |
-| **#729** | Implement Redis consumer for orchestrator service | **#756** | #756 carries identical work plus a critical fix for transient Redis errors. |
-| **#695** | fix(main): resolve all committed merge-conflict markers... | **#737** | #737 adopted this resolution and added a CI integrity gate. |
-| **#735** | fix(main): resolve committed merge-conflict markers... | **#737** | #737 is the most up-to-date rebase including the CI gate. |
-| **#736** | fix(main): resolve committed merge-conflict markers... | **#737** | Duplicate of #735/#737. |
-| **#716** | ⚡ Bolt: Optimize batch query execution... | **#749** | #749 is the refined optimization replacing earlier attempts. |
-| **#747** | ⚡ Bolt: Consistently use asyncio.gather... | **#749** | #749 is the final version of this performance bolt. |
-| **#707** | 🔒 Fix insecure SSL certificate verification... | **#721** | Cluster of SSL fixes; #721 covers multiple modules. |
-| **#714** | 🔒 Fix insecure SSL configuration... | **#721** | Superseded by the broader #721 fix. |
-| **#722** | 🔒 Fix insecure SSL verification... | **#721** | Superseded by the broader #721 fix. |
-| **#746** | 🎨 Palette: Add screen reader label... | **#750** | #750 provides more comprehensive ARIA labels for the same component. |
-
-## Actions Taken (Executed)
-On 2026-07-14 the GitHub write surface (GitHub MCP) was available, so all 9 stale
-PRs were closed with a supersession comment linking the replacement PR. Each close
-is reversible — reopen the PR if the superseding PR fails to land.
-
-| Closed PR | Superseded by |
-| :--- | :--- |
-| #729 | #756 |
-| #695, #735, #736 | #737 |
-| #716, #747 | #749 |
-| #707, #714, #722 | #721 |
-| #746 | #750 |
-
-Backlog reduced from 72 → 63 open PRs.
-
-## Outstanding — requires human triage (not auto-actionable)
-- **Conflicting Tailwind cluster**: multiple mutually-exclusive PRs (revert to v3 vs.
- migrate to v4): #613, #616, #623, #629 (v3) vs. #630, #632 (v4). Only one strategy
- can land; the rest should be closed once the owner picks a direction.
-- **Conflict-marker fix cluster**: #737 vs. #700 (draft) still overlap after the closes
- above — pick one to merge.
-- **~40 draft bot PRs** (google-labs-jules[bot]) are DEFERRED per the runbook scope gate
- until marked ready for review.
-- **Merges to protected `main`** require human sign-off (no PR is labeled `automerge`).
diff --git a/infrastructure/docker/docker-compose.full.yml b/infrastructure/docker/docker-compose.full.yml
index 7dd509fc7..3c8a6f1c8 100644
--- a/infrastructure/docker/docker-compose.full.yml
+++ b/infrastructure/docker/docker-compose.full.yml
@@ -79,20 +79,19 @@ services:
context: .
dockerfile: Dockerfile
image: youtube-extension-orchestrator:dev
- command: python -m youtube_extension.orchestrator.main
+ command: python -m youtube_extension.backend.services.phase3_integration_test
restart: unless-stopped
environment:
- APP_ENV=${APP_ENV:-production}
- DATABASE_URL=${DATABASE_URL}
- REDIS_URL=redis://redis:6379/1
- - MESSAGE_QUEUE_URL=redis://redis:6379/1
- - ORCHESTRATOR_QUEUE_NAME=orchestrator_tasks
+ - RABBITMQ_URL=amqp://guest:guest@rabbitmq:5672/
- OPENAI_API_KEY=${OPENAI_API_KEY}
- ANTHROPIC_API_KEY=${ANTHROPIC_API_KEY}
- GOOGLE_AI_API_KEY=${GOOGLE_AI_API_KEY}
depends_on:
- backend
- - redis
+ - rabbitmq
networks:
- uvai-network
diff --git a/package-lock.json b/package-lock.json
index ccc9762bd..57caaeaf9 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -11,28 +11,28 @@
"apps/*"
],
"dependencies": {
- "@ai-sdk/gateway": "^4.0.19",
+ "@ai-sdk/gateway": "^4.0.12",
"@dataconnect/generated": "file:src/dataconnect-generated",
"@google-cloud/text-to-speech": "^6.4.0",
- "@google/genai": "^2.11.0",
+ "@google/genai": "^2.10.0",
"@opentelemetry/core": "^2.9.0",
- "@types/node": "^26.1.1",
- "ai": "^7.0.26",
- "chrome-devtools-mcp": "^1.6.0",
+ "@types/node": "^26.1.0",
+ "ai": "^7.0.15",
+ "chrome-devtools-mcp": "^1.5.0",
"dotenv": "^17.4.2",
- "openai": "^6.46.0",
+ "openai": "^6.45.0",
"react": "^19",
"react-dom": "^19",
- "tsx": "^4.23.1"
+ "tsx": "^4.23.0"
},
"devDependencies": {
"@modelcontextprotocol/sdk": "^1.26.0",
"brace-expansion": "^5.0.7",
- "eslint": "^9.39.5",
+ "eslint": "^9.39.0",
"next": "^16.2.10",
- "turbo": "^2.10.5",
+ "turbo": "^2.10.3",
"typescript": "^6.0.3",
- "vitest": "^4.1.10"
+ "vitest": "^4.1.9"
},
"engines": {
"node": ">=20.6.0",
@@ -43,51 +43,51 @@
"name": "building-production-ai-infrastructure-platform",
"version": "0.1.0",
"dependencies": {
- "@ai-sdk/gateway": "^4.0.19",
+ "@ai-sdk/gateway": "^4.0.12",
"@dataconnect/generated": "file:src/dataconnect-generated",
- "@google/genai": "^2.11.0",
+ "@google/genai": "^2.10.0",
"@google/generative-ai": "^0.24.1",
"@opentelemetry/api": "^1.9.0",
"@opentelemetry/core": "^2.9.0",
- "@opentelemetry/exporter-trace-otlp-http": "^0.219.0",
"@opentelemetry/instrumentation": "^0.220.0",
"@opentelemetry/resources": "^2.9.0",
"@opentelemetry/sdk-trace-base": "^2.9.0",
- "@opentelemetry/semantic-conventions": "^1.43.0",
- "@sentry/nextjs": "^10.65.0",
+ "@opentelemetry/semantic-conventions": "^1.41.0",
+ "@sentry/nextjs": "^10.63.0",
"@stripe/stripe-js": "^9.9.0",
- "@supabase/supabase-js": "^2.110.5",
+ "@supabase/supabase-js": "^2.110.0",
"@upstash/redis": "^1.38.0",
"@upstash/search": "^0.1.7",
"@vercel/analytics": "^2.0.1",
"@vercel/functions": "^3.7.5",
"@vercel/speed-insights": "^2.0.0",
- "ai": "^7.0.26",
+ "ai": "^7.0.15",
"class-variance-authority": "^0.7.0",
"clsx": "^2.1.1",
- "lucide-react": "^1.24.0",
+ "lucide-react": "^1.23.0",
"next": "^16.2.10",
"next-auth": "^4.24.14",
- "openai": "^6.46.0",
+ "openai": "^6.45.0",
"react": "^19",
"react-dom": "^19",
"server-only": "^0.0.1",
- "stripe": "^22.3.1",
+ "stripe": "^22.3.0",
"tailwind-merge": "^3.6.0",
"use-sync-external-store": "^1.6.0",
"zod": "^4.4.3",
"zustand": "^5.0.14"
},
"devDependencies": {
+ "@playwright/test": "^1.61.1",
"@tailwindcss/postcss": "^4.3.2",
"@types/node": "^26",
"@types/react": "^19",
"@types/react-dom": "^19",
"autoprefixer": "^10.5.2",
- "eslint": "^9.39.5",
+ "eslint": "^9.39.0",
"eslint-config-next": "^16.2.10",
"playwright": "^1.61.1",
- "postcss": "^8.5.19",
+ "postcss": "^8.5.16",
"tailwindcss": "^4.3.1",
"typescript": "^6.0.3",
"vite": "^8.1.3",
@@ -864,6 +864,96 @@
"dev": true,
"license": "MIT"
},
+ "apps/web/node_modules/vitest": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/vitest/-/vitest-4.1.9.tgz",
+ "integrity": "sha512-nE3/LEyc0z87uHYLZebqCUOaJr2hdtuPp7BQ4BosVFnfltxgAvMG08NyrSGlPpOUWvR27c5flSmYFTNr78L9GQ==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@vitest/expect": "4.1.9",
+ "@vitest/mocker": "4.1.9",
+ "@vitest/pretty-format": "4.1.9",
+ "@vitest/runner": "4.1.9",
+ "@vitest/snapshot": "4.1.9",
+ "@vitest/spy": "4.1.9",
+ "@vitest/utils": "4.1.9",
+ "es-module-lexer": "^2.0.0",
+ "expect-type": "^1.3.0",
+ "magic-string": "^0.30.21",
+ "obug": "^2.1.1",
+ "pathe": "^2.0.3",
+ "picomatch": "^4.0.3",
+ "std-env": "^4.0.0-rc.1",
+ "tinybench": "^2.9.0",
+ "tinyexec": "^1.0.2",
+ "tinyglobby": "^0.2.15",
+ "tinyrainbow": "^3.1.0",
+ "vite": "^6.0.0 || ^7.0.0 || ^8.0.0",
+ "why-is-node-running": "^2.3.0"
+ },
+ "bin": {
+ "vitest": "vitest.mjs"
+ },
+ "engines": {
+ "node": "^20.0.0 || ^22.0.0 || >=24.0.0"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ },
+ "peerDependencies": {
+ "@edge-runtime/vm": "*",
+ "@opentelemetry/api": "^1.9.0",
+ "@types/node": "^20.0.0 || ^22.0.0 || >=24.0.0",
+ "@vitest/browser-playwright": "4.1.9",
+ "@vitest/browser-preview": "4.1.9",
+ "@vitest/browser-webdriverio": "4.1.9",
+ "@vitest/coverage-istanbul": "4.1.9",
+ "@vitest/coverage-v8": "4.1.9",
+ "@vitest/ui": "4.1.9",
+ "happy-dom": "*",
+ "jsdom": "*",
+ "vite": "^6.0.0 || ^7.0.0 || ^8.0.0"
+ },
+ "peerDependenciesMeta": {
+ "@edge-runtime/vm": {
+ "optional": true
+ },
+ "@opentelemetry/api": {
+ "optional": true
+ },
+ "@types/node": {
+ "optional": true
+ },
+ "@vitest/browser-playwright": {
+ "optional": true
+ },
+ "@vitest/browser-preview": {
+ "optional": true
+ },
+ "@vitest/browser-webdriverio": {
+ "optional": true
+ },
+ "@vitest/coverage-istanbul": {
+ "optional": true
+ },
+ "@vitest/coverage-v8": {
+ "optional": true
+ },
+ "@vitest/ui": {
+ "optional": true
+ },
+ "happy-dom": {
+ "optional": true
+ },
+ "jsdom": {
+ "optional": true
+ },
+ "vite": {
+ "optional": false
+ }
+ }
+ },
"apps/web/node_modules/zod": {
"version": "4.4.3",
"resolved": "https://registry.npmjs.org/zod/-/zod-4.4.3.tgz",
@@ -2946,184 +3036,6 @@
"@opentelemetry/api": ">=1.0.0 <1.10.0"
}
},
- "node_modules/@opentelemetry/exporter-trace-otlp-http": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/exporter-trace-otlp-http/-/exporter-trace-otlp-http-0.219.0.tgz",
- "integrity": "sha512-9t6SvBXXBEjOBcIzgozvBbd3jWrv3Gt3ngGhl1fhdZ/zRc7oZDVOFEqbi2zlBpW9BXhgDMKv422J0DL/3iQWfw==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/otlp-exporter-base": "0.219.0",
- "@opentelemetry/otlp-transformer": "0.219.0",
- "@opentelemetry/resources": "2.8.0",
- "@opentelemetry/sdk-trace-base": "2.8.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": "^1.3.0"
- }
- },
- "node_modules/@opentelemetry/exporter-trace-otlp-http/node_modules/@opentelemetry/core": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/core/-/core-2.8.0.tgz",
- "integrity": "sha512-hd1Lfh8p545nNz+jq1Ejfz+Mn1hyLuxYn1YzTfFNrxr8urEWMNQLPf1Th8kjOH+HxwawCrtgBp8JpBUR4ZSgww==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.0.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/exporter-trace-otlp-http/node_modules/@opentelemetry/resources": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/resources/-/resources-2.8.0.tgz",
- "integrity": "sha512-qmXQ27ilDbUK/vGMqwL8D4/rhn76C+sherM4wTbjlfknR8Nvfc/hCxjRJPhkzZzUsPiNg16SA31NxMabwttRjg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/exporter-trace-otlp-http/node_modules/@opentelemetry/sdk-trace-base": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/sdk-trace-base/-/sdk-trace-base-2.8.0.tgz",
- "integrity": "sha512-mhU4jp+vW0mGbFRd+GeXHvmfA4aDqWjBjLC3pE5XMpLs0IE2ryYb019Ts2AQrOq67gaTF25D91+fgvEHDZEnuQ==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/resources": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/otlp-exporter-base": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/otlp-exporter-base/-/otlp-exporter-base-0.219.0.tgz",
- "integrity": "sha512-zvIxQX/AZUVKDU+hCuYx+7UkiP7GRdnk1ZbFQRYzHvYp47cAWR4j3IhoPhV9KaeXEv2xdGq3IA6PnpzDmLcmSA==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/otlp-transformer": "0.219.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": "^1.3.0"
- }
- },
- "node_modules/@opentelemetry/otlp-exporter-base/node_modules/@opentelemetry/core": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/core/-/core-2.8.0.tgz",
- "integrity": "sha512-hd1Lfh8p545nNz+jq1Ejfz+Mn1hyLuxYn1YzTfFNrxr8urEWMNQLPf1Th8kjOH+HxwawCrtgBp8JpBUR4ZSgww==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.0.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/otlp-transformer": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/otlp-transformer/-/otlp-transformer-0.219.0.tgz",
- "integrity": "sha512-aaYKAyXhw9VchKZVGOopD3Gw/kPsyrX2c6IQ0AW32mTjqmZOh5Y6Gf5OYqTNqVktAeBjmFinhyFaCwW6GYK9YQ==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/api-logs": "0.219.0",
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/resources": "2.8.0",
- "@opentelemetry/sdk-logs": "0.219.0",
- "@opentelemetry/sdk-metrics": "2.8.0",
- "@opentelemetry/sdk-trace-base": "2.8.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": "^1.3.0"
- }
- },
- "node_modules/@opentelemetry/otlp-transformer/node_modules/@opentelemetry/api-logs": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/api-logs/-/api-logs-0.219.0.tgz",
- "integrity": "sha512-FFx7YnaYJlIjqWW/AG/yAZ0L/NEY724PipXXXQLdtZPbLwBGbUMTGL1i/esI56TWfTUXxhLfpgrnWJCG8aUJyg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/api": "^1.3.0"
- },
- "engines": {
- "node": ">=8.0.0"
- }
- },
- "node_modules/@opentelemetry/otlp-transformer/node_modules/@opentelemetry/core": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/core/-/core-2.8.0.tgz",
- "integrity": "sha512-hd1Lfh8p545nNz+jq1Ejfz+Mn1hyLuxYn1YzTfFNrxr8urEWMNQLPf1Th8kjOH+HxwawCrtgBp8JpBUR4ZSgww==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.0.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/otlp-transformer/node_modules/@opentelemetry/resources": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/resources/-/resources-2.8.0.tgz",
- "integrity": "sha512-qmXQ27ilDbUK/vGMqwL8D4/rhn76C+sherM4wTbjlfknR8Nvfc/hCxjRJPhkzZzUsPiNg16SA31NxMabwttRjg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/otlp-transformer/node_modules/@opentelemetry/sdk-trace-base": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/sdk-trace-base/-/sdk-trace-base-2.8.0.tgz",
- "integrity": "sha512-mhU4jp+vW0mGbFRd+GeXHvmfA4aDqWjBjLC3pE5XMpLs0IE2ryYb019Ts2AQrOq67gaTF25D91+fgvEHDZEnuQ==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/resources": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
"node_modules/@opentelemetry/resources": {
"version": "2.9.0",
"resolved": "https://registry.npmjs.org/@opentelemetry/resources/-/resources-2.9.0.tgz",
@@ -3140,114 +3052,6 @@
"@opentelemetry/api": ">=1.3.0 <1.10.0"
}
},
- "node_modules/@opentelemetry/sdk-logs": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/sdk-logs/-/sdk-logs-0.219.0.tgz",
- "integrity": "sha512-s6lTKRakaPClvKoWHRChxnXjDMkM/TQ30ff78jN6EBGf7MI7VzANE5PU3f4z9qDUudWjvZjOLHG0rBnBKYvoXA==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/api-logs": "0.219.0",
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/resources": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.4.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/sdk-logs/node_modules/@opentelemetry/api-logs": {
- "version": "0.219.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/api-logs/-/api-logs-0.219.0.tgz",
- "integrity": "sha512-FFx7YnaYJlIjqWW/AG/yAZ0L/NEY724PipXXXQLdtZPbLwBGbUMTGL1i/esI56TWfTUXxhLfpgrnWJCG8aUJyg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/api": "^1.3.0"
- },
- "engines": {
- "node": ">=8.0.0"
- }
- },
- "node_modules/@opentelemetry/sdk-logs/node_modules/@opentelemetry/core": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/core/-/core-2.8.0.tgz",
- "integrity": "sha512-hd1Lfh8p545nNz+jq1Ejfz+Mn1hyLuxYn1YzTfFNrxr8urEWMNQLPf1Th8kjOH+HxwawCrtgBp8JpBUR4ZSgww==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.0.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/sdk-logs/node_modules/@opentelemetry/resources": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/resources/-/resources-2.8.0.tgz",
- "integrity": "sha512-qmXQ27ilDbUK/vGMqwL8D4/rhn76C+sherM4wTbjlfknR8Nvfc/hCxjRJPhkzZzUsPiNg16SA31NxMabwttRjg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/sdk-metrics": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/sdk-metrics/-/sdk-metrics-2.8.0.tgz",
- "integrity": "sha512-UDBGaj6W0Rgy5rTTaoxs8gVGF/aGkAKyjurJv7se6wjRxJu7FoquTLT/vt54DZfo4crbprYfhX/SOK9+BPw1qg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/resources": "2.8.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.9.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/sdk-metrics/node_modules/@opentelemetry/core": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/core/-/core-2.8.0.tgz",
- "integrity": "sha512-hd1Lfh8p545nNz+jq1Ejfz+Mn1hyLuxYn1YzTfFNrxr8urEWMNQLPf1Th8kjOH+HxwawCrtgBp8JpBUR4ZSgww==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.0.0 <1.10.0"
- }
- },
- "node_modules/@opentelemetry/sdk-metrics/node_modules/@opentelemetry/resources": {
- "version": "2.8.0",
- "resolved": "https://registry.npmjs.org/@opentelemetry/resources/-/resources-2.8.0.tgz",
- "integrity": "sha512-qmXQ27ilDbUK/vGMqwL8D4/rhn76C+sherM4wTbjlfknR8Nvfc/hCxjRJPhkzZzUsPiNg16SA31NxMabwttRjg==",
- "license": "Apache-2.0",
- "dependencies": {
- "@opentelemetry/core": "2.8.0",
- "@opentelemetry/semantic-conventions": "^1.29.0"
- },
- "engines": {
- "node": "^18.19.0 || >=20.6.0"
- },
- "peerDependencies": {
- "@opentelemetry/api": ">=1.3.0 <1.10.0"
- }
- },
"node_modules/@opentelemetry/sdk-trace": {
"version": "2.9.0",
"resolved": "https://registry.npmjs.org/@opentelemetry/sdk-trace/-/sdk-trace-2.9.0.tgz",
@@ -3321,6 +3125,22 @@
"node": ">=14"
}
},
+ "node_modules/@playwright/test": {
+ "version": "1.61.1",
+ "resolved": "https://registry.npmjs.org/@playwright/test/-/test-1.61.1.tgz",
+ "integrity": "sha512-8nKv6+0RJSL9FE4jYOEGXnPeM/Hg12qZpmqzZjRh3qM0Y7c3z1mrOTfFLids72RDQYVh9WpLEfR5WdpNX4fkig==",
+ "dev": true,
+ "license": "Apache-2.0",
+ "dependencies": {
+ "playwright": "1.61.1"
+ },
+ "bin": {
+ "playwright": "cli.js"
+ },
+ "engines": {
+ "node": ">=18"
+ }
+ },
"node_modules/@protobufjs/aspromise": {
"version": "1.1.2",
"resolved": "https://registry.npmjs.org/@protobufjs/aspromise/-/aspromise-1.1.2.tgz",
@@ -5292,6 +5112,129 @@
}
}
},
+ "node_modules/@vitest/expect": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/expect/-/expect-4.1.9.tgz",
+ "integrity": "sha512-vl/rYsUKcBr3SnQn166+XR5ZQcgMx3DQhFWdfli/cWpLnLUmbxZvyrJZotLFUryib+LtArYMSTJ5RbQ57ZqrlA==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@standard-schema/spec": "^1.1.0",
+ "@types/chai": "^5.2.2",
+ "@vitest/spy": "4.1.9",
+ "@vitest/utils": "4.1.9",
+ "chai": "^6.2.2",
+ "tinyrainbow": "^3.1.0"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
+ "node_modules/@vitest/mocker": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/mocker/-/mocker-4.1.9.tgz",
+ "integrity": "sha512-EVkXzBjrPGM+cK8/ANWgBrkUCfJfb38/EfTSO8h7pWvKkyPkpWxvR7BkD2MyItMF62C97zAEoqdpUixwR/e+Rw==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@vitest/spy": "4.1.9",
+ "estree-walker": "^3.0.3",
+ "magic-string": "^0.30.21"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ },
+ "peerDependencies": {
+ "msw": "^2.4.9",
+ "vite": "^6.0.0 || ^7.0.0 || ^8.0.0"
+ },
+ "peerDependenciesMeta": {
+ "msw": {
+ "optional": true
+ },
+ "vite": {
+ "optional": true
+ }
+ }
+ },
+ "node_modules/@vitest/mocker/node_modules/estree-walker": {
+ "version": "3.0.3",
+ "resolved": "https://registry.npmjs.org/estree-walker/-/estree-walker-3.0.3.tgz",
+ "integrity": "sha512-7RUKfXgSMMkzt6ZuXmqapOurLGPPfgj6l9uRZ7lRGolvk0y2yocc35LdcxKC5PQZdn2DMqioAQ2NoWcrTKmm6g==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@types/estree": "^1.0.0"
+ }
+ },
+ "node_modules/@vitest/pretty-format": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/pretty-format/-/pretty-format-4.1.9.tgz",
+ "integrity": "sha512-s0iufns3iIFitdgm+YR7g1whCAaGtXz459VS9/PqyKDEEFgYIhsHOQmXgIgDuYCt7DeQmiZT0Qe2OA2p4ZPu5A==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "tinyrainbow": "^3.1.0"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
+ "node_modules/@vitest/runner": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/runner/-/runner-4.1.9.tgz",
+ "integrity": "sha512-KXLMDtc7oe70+3mJfGrPUWPesswH+3sTxAMAMl8DG7I8IUQT4XW718dY5ID3vPUcmlu27CcKfY4P3h3I29SLJg==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@vitest/utils": "4.1.9",
+ "pathe": "^2.0.3"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
+ "node_modules/@vitest/snapshot": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/snapshot/-/snapshot-4.1.9.tgz",
+ "integrity": "sha512-Jc7RKGNBo8Z28WYIm0Niej4xdSPByRf6mU58VpHQkd6Zh05rlnA+twjbK5HyeIGHxrzsc3mJgS43uM0CZKzaIA==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@vitest/pretty-format": "4.1.9",
+ "@vitest/utils": "4.1.9",
+ "magic-string": "^0.30.21",
+ "pathe": "^2.0.3"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
+ "node_modules/@vitest/spy": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/spy/-/spy-4.1.9.tgz",
+ "integrity": "sha512-fHpsS6mIi+PiEW+vcRVOMkX1oSaPKne3VOclSFICPcGOmfKgXPU5iAah+wcNcj2xPrCCmfq99IDGf+EojhhvhA==",
+ "dev": true,
+ "license": "MIT",
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
+ "node_modules/@vitest/utils": {
+ "version": "4.1.9",
+ "resolved": "https://registry.npmjs.org/@vitest/utils/-/utils-4.1.9.tgz",
+ "integrity": "sha512-A51o8ymO5PpqlWNnBP9ZHPXDIpuMtTLlGSjN7la4US+LJzoUMyhwjA5QXlm39JexgwHKW4Xjs8Z2d3dLCXOeuA==",
+ "dev": true,
+ "license": "MIT",
+ "dependencies": {
+ "@vitest/pretty-format": "4.1.9",
+ "convert-source-map": "^2.0.0",
+ "tinyrainbow": "^3.1.0"
+ },
+ "funding": {
+ "url": "https://opencollective.com/vitest"
+ }
+ },
"node_modules/@workflow/serde": {
"version": "4.1.0",
"resolved": "https://registry.npmjs.org/@workflow/serde/-/serde-4.1.0.tgz",
diff --git a/package.json b/package.json
index 21a6980a0..bcbad1b4b 100644
--- a/package.json
+++ b/package.json
@@ -22,11 +22,11 @@
"devDependencies": {
"@modelcontextprotocol/sdk": "^1.26.0",
"brace-expansion": "^5.0.7",
- "eslint": "^9.39.5",
+ "eslint": "^9.39.0",
"next": "^16.2.10",
- "turbo": "^2.10.5",
+ "turbo": "^2.10.3",
"typescript": "^6.0.3",
- "vitest": "^4.1.10"
+ "vitest": "^4.1.9"
},
"overrides": {
"react": "^19",
@@ -40,9 +40,8 @@
"@temporalio/client": {
"uuid": "11.1.1"
},
- "vite": "^8.1.4",
- "vitest": "^4.1.10",
- "picomatch": "^4.0.5",
+ "vite": ">=6.4.3",
+ "picomatch": ">=2.3.2",
"protobufjs": "^7.6.2",
"minimatch@3.1.5": {
"brace-expansion": "^1.1.12"
@@ -52,18 +51,18 @@
}
},
"dependencies": {
- "@ai-sdk/gateway": "^4.0.19",
+ "@ai-sdk/gateway": "^4.0.12",
"@dataconnect/generated": "file:src/dataconnect-generated",
"@google-cloud/text-to-speech": "^6.4.0",
- "@google/genai": "^2.11.0",
+ "@google/genai": "^2.10.0",
"@opentelemetry/core": "^2.9.0",
- "@types/node": "^26.1.1",
- "ai": "^7.0.26",
- "chrome-devtools-mcp": "^1.6.0",
+ "@types/node": "^26.1.0",
+ "ai": "^7.0.15",
+ "chrome-devtools-mcp": "^1.5.0",
"dotenv": "^17.4.2",
- "openai": "^6.46.0",
+ "openai": "^6.45.0",
"react": "^19",
"react-dom": "^19",
- "tsx": "^4.23.1"
+ "tsx": "^4.23.0"
}
}
diff --git a/pyproject.toml b/pyproject.toml
index 1935a9f07..91c828d91 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -69,7 +69,6 @@ dependencies = [
"opencv-python>=4.8.0",
"orjson>=3.9.0",
"aiohttp>=3.8.0",
- "redis>=5.0.0",
]
[project.optional-dependencies]
@@ -107,7 +106,7 @@ youtube = [
"youtube-transcript-api>=1.0.0",
"google-api-python-client>=2.100.0",
"google-cloud-speech>=2.26.0",
- "yt-dlp>=2025.05.22", # Minimum version with PO Token plugin support and current bot-detection mitigations
+ "yt-dlp>=2023.7.6",
"asyncio-throttle>=1.0.0",
"websockets>=12.0",
"gitpython>=3.1.0",
diff --git a/requirements.txt b/requirements.txt
index 5cb8dfea7..770c1f27b 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -70,9 +70,12 @@ opencv-python-headless>=5.0.0.93
asyncio-throttle>=1.0.0
websockets>=12.0
gitpython>=3.1.0
-redis>=5.0.0
# Observability (optional - can be removed for minimal builds)
# ddtrace>=2.1.0
# opentelemetry-distro>=0.40b0
# opentelemetry-exporter-otlp>=1.20.0
+
+# Security Scanning
+bandit>=1.7.5
+safety>=2.3.5
diff --git a/scripts/analysis/repo_health_check.py b/scripts/analysis/repo_health_check.py
index 4ee6ba98e..1d88ed9ba 100644
--- a/scripts/analysis/repo_health_check.py
+++ b/scripts/analysis/repo_health_check.py
@@ -98,8 +98,11 @@ def check_cross_reference():
try:
conn = get_pg_connection()
cur = conn.cursor()
- cur.execute("SELECT metadata->>'filename' as filename FROM vector_items WHERE metadata->>'filename' IS NOT NULL")
- pg_files: Set[str] = {row[0] for row in cur}
+ cur.execute("SELECT metadata->>'filename' as filename FROM vector_items")
+ pg_files: Set[str] = set()
+ for row in cur.fetchall():
+ if row[0]:
+ pg_files.add(row[0])
conn.close()
print(f"Found {len(pg_files)} files in Vector DB (Cloud SQL).")
diff --git a/scripts/archive/Testing Video Agent.md b/scripts/archive/Testing Video Agent.md
index 13f07ed93..7fec945b8 100644
--- a/scripts/archive/Testing Video Agent.md
+++ b/scripts/archive/Testing Video Agent.md
@@ -1024,7 +1024,7 @@ Replace the URL with any YouTube video you want to analyze. The script will:
**Example with a real video:**
```bash
-python src/agents/gemini_video_master_agent.py "https://www.youtube.com/watch?v=auJzb1D-fag"
+python src/agents/gemini_video_master_agent.py "https://www.youtube.com/watch?v=dQw4w9WgXcQ"
```
Would you like me to run a test with a specific video URL, or did you want to add a default/interactive mode to the script so it prompts for a URL instead of requiring a CLI argument?
@@ -1042,7 +1042,7 @@ here are some examples above, but default should be any business or technical vi
Lets try each of the videos below, watch the output and work error if any
SIDE NOTE - CONTEXT
-https://www.youtube.com/watch?v=auJzb1D-fag = this should be banned from surving as example, its a music video meme please remember this ( auJzb1D-fag )
+https://www.youtube.com/watch?v=dQw4w9WgXcQ = this should be banned from surving as example, its a music video meme please remember this ( dQw4w9WgXcQ )
*User accepted the command `cd /Users/garvey/Dev/projects/EventRelay && python src/agents/gemini_video_master_agent.py "https://www.youtube.com/watch?v=QKHL_gmlteQ" 2>&1 | head -100`*
@@ -1247,7 +1247,7 @@ here are some examples above, but default should be any business or technical vi
Lets try each of the videos below, watch the output and work error if any
SIDE NOTE - CONTEXT
-https://www.youtube.com/watch?v=auJzb1D-fag = this should be banned from surving as example, its a music video meme please remember this ( auJzb1D-fag )
+https://www.youtube.com/watch?v=dQw4w9WgXcQ = this should be banned from surving as example, its a music video meme please remember this ( dQw4w9WgXcQ )
### User Input
@@ -1271,4 +1271,4 @@ here are some examples above, but default should be any business or technical vi
Lets try each of the videos below, watch the output and work error if any
SIDE NOTE - CONTEXT
-https://www.youtube.com/watch?v=auJzb1D-fag = this should be banned from surving as example, its a music video meme please remember this ( auJzb1D-fag )
\ No newline at end of file
+https://www.youtube.com/watch?v=dQw4w9WgXcQ = this should be banned from surving as example, its a music video meme please remember this ( dQw4w9WgXcQ )
\ No newline at end of file
diff --git a/scripts/archive/gap_fixing_workflow.py b/scripts/archive/gap_fixing_workflow.py
index bd01f835a..836696a79 100644
--- a/scripts/archive/gap_fixing_workflow.py
+++ b/scripts/archive/gap_fixing_workflow.py
@@ -342,7 +342,7 @@ def _validate_mcp_tool_count(self) -> Dict[str, Any]:
def _validate_service_files(self) -> Dict[str, Any]:
"""Validate that service classes are implemented"""
# FIX: Point to the actual services location in src/youtube_extension
- services_dir = self.project_root / "src/youtube_extension/services"
+ services_dir = self.project_root / "src/youtube_extension/backend/services"
if not services_dir.exists():
# Try alternative path just in case
@@ -365,7 +365,7 @@ def _validate_service_files(self) -> Dict[str, Any]:
def _validate_service_imports(self) -> Dict[str, Any]:
"""Validate service imports"""
- services_dir = self.project_root / "src/youtube_extension/services"
+ services_dir = self.project_root / "src/youtube_extension/backend/services"
if not services_dir.exists():
services_dir = self.project_root / "backend/services"
diff --git a/scripts/archive/generate_screen_payload.json b/scripts/archive/generate_screen_payload.json
index 1bc1a692a..4a8261cca 100644
--- a/scripts/archive/generate_screen_payload.json
+++ b/scripts/archive/generate_screen_payload.json
@@ -8,7 +8,7 @@
"projectId": "11701385696503779403",
"modelId": "GEMINI_3_PRO",
"deviceType": "DESKTOP",
- "prompt": "Create a modern, professional 'Video Intelligence Dashboard' for developers called EventRelay. \n\n**Visual Style:**\n- 'Brutalist Utility' aesthetic: High contrast, data-dense, monospaced fonts for code/data.\n- Dark mode default.\n- Clean, sharp edges (not overly rounded).\n\n**Layout (Split-Screen):**\n- **Header:** Minimal bar with 'EventRelay' logo, System Status indicator (Green pulse), and User Profile.\n- **Left Sidebar (40% width):** \n - Prominent YouTube URL Input field at the top (Pre-filled with: 'https://www.youtube.com/watch?v=auJzb1D-fag').\n - Video Player area (16:9 aspect ratio placeholder).\n - Scrollable Metadata section below video:\n - **Title:** Rick Astley - Never Gonna Give You Up (Official Video) (4K Remaster)\n - **Channel:** Rick Astley\n - **Views:** 1,736,443,825\n - **Duration:** 3m 34s\n- **Right Content Area (60% width):**\n - Tabbed Interface: 'Analysis', 'Transcript', 'Events', 'Build'.\n - **Analysis Tab:** Markdown-styled text area displaying the actual system output:\n ```markdown\n ## 🎯 Content Summary\n unsupported_music_content\n \n ## 📊 System Note\n The Multi-Agent Video Processor correctly identified this input as a music video, which falls outside the educational content scope. No further analysis was performed.\n ```\n - **Transcript Tab:** (Empty/Disabled for this video type)\n - **Events Tab:** Empty state 'No events detected for music content'.\n - **Build Tab:** A code editor view showing generated code artifacts (if any).\n\n**Functionality Notes for Code Generation:**\n- The frontend must be React + Tailwind CSS.\n- It needs to call `POST http://localhost:8000/process_video` to populate the Analysis tab.\n- It needs to call `POST http://localhost:8000/api/v1/transcript-action` when action buttons are clicked.\n\n**Proof of Functionality:**\n- The dashboard successfully handles edge cases like 'unsupported_music_content' gracefully."
+ "prompt": "Create a modern, professional 'Video Intelligence Dashboard' for developers called EventRelay. \n\n**Visual Style:**\n- 'Brutalist Utility' aesthetic: High contrast, data-dense, monospaced fonts for code/data.\n- Dark mode default.\n- Clean, sharp edges (not overly rounded).\n\n**Layout (Split-Screen):**\n- **Header:** Minimal bar with 'EventRelay' logo, System Status indicator (Green pulse), and User Profile.\n- **Left Sidebar (40% width):** \n - Prominent YouTube URL Input field at the top (Pre-filled with: 'https://www.youtube.com/watch?v=dQw4w9WgXcQ').\n - Video Player area (16:9 aspect ratio placeholder).\n - Scrollable Metadata section below video:\n - **Title:** Rick Astley - Never Gonna Give You Up (Official Video) (4K Remaster)\n - **Channel:** Rick Astley\n - **Views:** 1,736,443,825\n - **Duration:** 3m 34s\n- **Right Content Area (60% width):**\n - Tabbed Interface: 'Analysis', 'Transcript', 'Events', 'Build'.\n - **Analysis Tab:** Markdown-styled text area displaying the actual system output:\n ```markdown\n ## 🎯 Content Summary\n unsupported_music_content\n \n ## 📊 System Note\n The Multi-Agent Video Processor correctly identified this input as a music video, which falls outside the educational content scope. No further analysis was performed.\n ```\n - **Transcript Tab:** (Empty/Disabled for this video type)\n - **Events Tab:** Empty state 'No events detected for music content'.\n - **Build Tab:** A code editor view showing generated code artifacts (if any).\n\n**Functionality Notes for Code Generation:**\n- The frontend must be React + Tailwind CSS.\n- It needs to call `POST http://localhost:8000/process_video` to populate the Analysis tab.\n- It needs to call `POST http://localhost:8000/api/v1/transcript-action` when action buttons are clicked.\n\n**Proof of Functionality:**\n- The dashboard successfully handles edge cases like 'unsupported_music_content' gracefully."
}
}
}
\ No newline at end of file
diff --git a/scripts/archive/software-on-demand/samples/video-to-learning-app/public/data/examples.json b/scripts/archive/software-on-demand/samples/video-to-learning-app/public/data/examples.json
index 3061c5e1a..8c429ea1e 100644
--- a/scripts/archive/software-on-demand/samples/video-to-learning-app/public/data/examples.json
+++ b/scripts/archive/software-on-demand/samples/video-to-learning-app/public/data/examples.json
@@ -1 +1 @@
-[{"title":"How chords work","url":"https://www.youtube.com/watch?v=JfD0nHrJDC0","spec":"Build me an interactive web app to help a learner understand triads and their inversions.\n\nSPECIFICATIONS:\n1. The app must feature an interactive keyboard. When a key is pressed, the corresponding note should sound.\n2. The app must allow the user to select a root note (C, D, E, F, G, A, or B). This root note should be displayed clearly. The root note should default to C.\n3. The app must allow the user to select a triad type (Major, Minor, Diminished, Augmented). The triad type should be displayed clearly. The triad type should default to Major.\n4. The app must allow the user to select an inversion (Root position, First Inversion, Second Inversion). The inversion should be displayed clearly. The inversion should default to Root Position.\n5. Based on the selected root, triad type, and inversion, the app must highlight the corresponding notes on the keyboard. The notes should also sound.\n6. The app must display the formula for generating the triad in terms of intervals (e.g., for C Major, it should display 'Root: C, Third: Major Third, Fifth: Perfect Fifth'). For inversions, it should explicitly state which note is the lowest (e.g., 'Lowest note is Third').\n7. The app must be responsive and work well on both desktop and mobile devices.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Triads and Inversions\n \n\n\n\n \n
Triads and Inversions
\n\n
\n
\n \n \n
\n
\n \n \n
\n
\n \n \n
\n
\n\n
\n
Selection: C Major, Root Position
\n
Formula: Root: C, Third: Major Third (E), Fifth: Perfect Fifth (G)
\n
Notes: C4, E4, G4
\n
Lowest Note:
\n
\n\n
\n
Koch Snowflake Fractal Explorer
\n\n \n
Fractals are patterns that repeat themselves at different scales (self-similarity).
\n
The Koch snowflake starts with an equilateral triangle. Each iteration adds a smaller triangle to the middle third of every side.
\n
Use mouse wheel to zoom, click and drag to pan.
\n
\n Iterations: 0 |\n Formula for sides: 3 * 4n |\n Current Sides: 3\n
\n
\n \n \n \n \n \n
\n\n \n \n
\n\n \n\n\n"},{"title":"Logic behind Chinese characters","url":"https://youtu.be/U0EySK4T2aY?si=HV_ZHWS8KdJZmZJP","spec":"Chinese characters are logograms, symbols that represent a word or phrase. Most modern-day characters are constructed of two components: a radical (which hints at the meaning of the character) and a sound component (which hints at its pronunciation). Radicals can appear in different positions: to the left, to the right, above, below, or even surrounding the rest of the character.\n\nBuild me an interactive web app to help a learner understand the structure of Chinese characters.\n\nSPECIFICATIONS:\n1. The app must feature a central display area for a Chinese character.\n2. The app must feature a palette containing a set of radicals (at least 5).\n3. The app must feature a palette containing a set of sound components (at least 5).\n4. The app must allow the user to drag radicals and/or sound components onto the central display area. When a radical and sound component are combined in this way, the resulting character should be displayed.\n5. The app must feature a short explanation of the radical component and the sound component of each character that is displayed. The short explanation should appear whenever a new character is displayed.\n6. When a new character is displayed, the app should also state the pronunciation of the character (using Pinyin).\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Chinese Character Builder\n \n\n\n\n Chinese Character Builder
\n Drag a radical and a sound component to the central area to form a character.
\n\n \n\n
\n
\n
Radicals (Meaning Hint)
\n \n \n\n
\n
Sound Components (Pronunciation Hint)
\n \n \n
\n\n
\n
\n
\n\n
\n
Drop components above...
\n \n
\n
\n\n
\n
Magical Mitosis
\n
Interphase
\n\n
\n \n
\n\n
The cell grows, copies its DNA (chromosomes), and prepares for division.
\n
\n
Identical Daughter Cells!
\n
\n \n \n 2023\n \n
\n \n
Broadway in 2023
\n
Loading historical context...
\n
\n
Pantry
\n
\n
Base
\n
\n
Rice
\n
Pasta
\n
Potatoes
\n
Polenta
\n
\n
\n
\n
Protein
\n
\n
Ground Beef
\n
Chicken
\n
Tuna
\n
Tofu
\n
\n
\n
\n
Vegetables
\n
\n
Green Beans
\n
Onions
\n
Mushrooms
\n
Tomatoes
\n
Broccoli
\n
\n
\n
\n
Sauce
\n
\n
Tomato Sauce
\n
Cream Soup
\n
Béchamel
\n
Enchilada Sauce
\n
\n
\n
\n
Cheese
\n
\n
Cheddar
\n
Mozzarella
\n
Parmesan
\n
Goat Cheese
\n
\n
\n
\n
Topping
\n
\n
Potato Chips
\n
Fried Onions
\n
Bread Crumbs
\n
More Cheese
\n
\n
\n
\n
Build your casserole!
\n
\n
Drag ingredients to the dish in order!
\n
\n
\n
Casserole Baked!
\n
Delicious! You've successfully built your casserole.
\n \n
![\"Baked](\"data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iMjAwIiBoZWlnaHQ9IjEzMCIgdmlld0JveD0iMCAwIDIwMCAxMzAiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+CiAgPGRlZnM+CiAgICA8cmFkaWFsR3JhZGllbnQgaWQ9ImJnR3JhZGllbnQiIGN4PSI1MCUiIGN5PSI1MCUiIHI9IjcwJSIgZng9IjUwJSIgZnk9IjUwJSI+CiAgICAgIDxzdG9wIG9mZnNldD0iMCUiIHN0b3AtY29sb3I9IiNmZmU1YjMiLz4KICAgICAgPHN0b3Agb2Zmc2V0PSIxMDAlIiBzdG9wLWNvbG9yPSIjZGVhMjU3Ii8+CiAgICA8L3JhZGlhbEdyYWRpZW50PgogIDwvZGVmcz4KICAgIDwhLS0gRGlzaCAtLT4KICAgIDxlbGxpcHNlIGN4PSIxMDAiIGN5PSIxMDUiIHJ4PSI5NSIgcnk9IjI1IiBmaWxsPSIjYTlhNWE0IiBzdHJva2U9IiM2OTY5NjkiIHN0cm9rZS13aWR0aD0iMiIvPgogICAgPHBhdGggZD0iTSA1IDgwIEEgOTUgMjUgMCAwIDEgMTk1IDgwIFYgMTA1IEEgOTUgMjUgMCAwIDEgNSAxMDUgWiIgZmlsbD0iI2E5YTVhNCIgc3Ryb2tlPSIjNjk2OTY5IiBzdHJva2Utd2lkdGg9IjIiLz4KICAgIDxlbGxpcHNlIGN4PSIxMDAiIGN5PSI4MCIgcng9Ijk1IiByeT0iMjUiIGZpbGw9IiNhOWE1YTQiIHN0cm9rZT0iIzY5Njk2OSIgc3Ryb2tlLXdpZHRoPSIyIi8+CiAgCiAgICA8IS0tIEZvb2QgLS0+CiAgICA8ZWxsaXBzZSBjeD0iMTAwIiBjeT0iODAiIHJ4PSI5MCIgcnk9IjIwIiBmaWxsPSJ1cmwoI2JnR3JhZGllbnQpIiAvPgogICAgCiAgICA8IS0tIEJ1YmJsZXMgLS0+CiAgICA8Y2lyY2xlIGN4PSI3MCIgY3k9IjcwIiByPSI1IiBmaWxsPSIjZmZmIiBvcGFjaXR5PSIwLjciLz4KICAgIDxjaXJjbGUgY3g9IjEyMCIgY3k9Ijc1IiByPSI3IiBmaWxsPSIjZmZmIiBvcGFjaXR5PSIwLjYiLz4KICAgIDxjaXJjbGUgY3g9IjE1MCIgY3k9Ijg1IiByPSI0IiBmaWxsPSIjZmZmIiBvcGFjaXR5PSIwLjgiLz4KICAgIDxjaXJjbGUgY3g9IjkwIiBjeT0iOTAiIHI9IjYiIGZpbGw9IiNmZmYiIG9wYWNpdHk9IjAuNiIvPgo8L3N2Zz4K\")
\n
\n
\n Select a cocktail to begin...\n \n
\n\n \n\n\n"},{"title":"Calligraphy & handlettering","url":"https://www.youtube.com/watch?v=sBoVGqiSzr4","spec":"Build an interactive web app to teach the fundamentals of modern calligraphy. The app should emphasize the contrast between thick downstrokes and thin upstrokes, and allow the user to practice creating this contrast.\n\nSPECIFICATIONS:\n\n1. The app should display an interactive canvas or drawing area where the user can draw lines.\n2. The app must have a control (e.g., a slider or button) that allows the user to adjust the \"pressure\" of the brush or pen. This pressure will determine the thickness of the line being drawn.\n3. The drawing area should include an optional grid overlay, mimicking the guidelines used in traditional calligraphy as seen in the video.\n4. The app should provide an exercise where a user is shown a simple waveform (like a sine wave) and asked to trace it with the interactive brush, creating thick downstrokes and thin upstrokes based on the direction of movement.\n5. The app must also allow the user to enter a word or phrase, and then trace the letters with appropriate thick and thin strokes, i.e., downstrokes will be thicker, upstrokes will be thinner\n6. The app should include a reset button to clear the canvas and start over.\n7. Provide visual feedback to the user when they successfully complete a stroke with the correct pressure (e.g., a color change or a small animation).\n8. The app should feature an option to display X-height lines to help users maintain consistent letter sizing.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Calligraphy Practice Pad\n \n\n\n\n Modern Calligraphy Practice
\n Practice thick downstrokes and thin upstrokes.
\n\n \n\n \n \n \n
\n\n \n\n\n"},{"title":"Making friends","url":"https://www.youtube.com/watch?v=I9hJ_Rux9y0","spec":"Build an interactive web app to help users explore the principles of making and maintaining friendships, based on the Kurzgesagt video about friendship. \n\nSPECIFICATIONS:\n1. The app should present the user with a series of scenarios, each representing a different stage or situation in the process of making and maintaining friendships. Scenarios should be drawn from the video's examples (e.g., attending a class, joining a club, moving to a new city, having a breakup, scrolling through contacts on your phone).\n\n2. For each scenario, the app should pose a question to the user, prompting them to choose from a set of actions or behaviors. Actions should be simple, concrete, and reflect either positive or negative friendship-building behaviors discussed in the video.\n\n3. Each action chosen by the user should result in a visual or numerical representation of the impact on their friendship \"score\" or \"network\". This could be a simple bar graph or a network diagram showing connections growing or shrinking. Positive actions increase the score, negative actions decrease it.\n\n4. The app should track the user's overall friendship score throughout the scenarios. The user should receive feedback at the end of each scenario, and at the end of the game, based on their final score. This feedback can provide encouragement and tips for improving their friendship skills.\n\n5. The app should incorporate at least three scenarios, and ideally closer to 5-7. The scenarios should be varied, and should cover both initiating new friendships and maintaining existing ones.\n\n6. Keep the interface simple and visually appealing. Use colors and imagery that evoke a sense of warmth, connection, and fun, but avoid being overly sentimental or saccharine.\n\n7. The actions presented should be easily understandable and should clearly relate back to the principles discussed in the video (e.g., prioritizing time, showing genuine interest, being open and receptive, inviting people to do fun things, sharing experiences).\n\n8. The app's logic and scoring mechanism should be transparent, so users understand how their actions impact their friendship network. Avoid arbitrary scoring, and instead focus on rewarding behaviors that are known to build strong relationships.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Friendship Explorer\n \n\n\n\n \n
Making friends
\n
Navigate scenarios based on Kurzgesagt's friendship principles!
\n\n
\n
Scenario Title
\n
Scenario description goes here.
\n
Question goes here?
\n
\n \n
\n
\n
\n\n
\n
Your Friendship Potential
\n
\n
Score: 50\n
\n\n
\n
Journey Complete!
\n
Your final score is: X
\n
Final feedback message goes here.
\n
\n
\n
\n
Baseball Hitting Tilt Trainer
\n
\n
\n
\n
\n
\n
\n
Shoulder Tilt: 0 degrees\n
\n Ideal shoulder tilt at launch position is slightly downward (approximately -9 degrees).\n
\n
\n
\n
\n
\n
Barrel Tilt: 0 degrees\n
\n Ideal barrel tilt at launch position is slightly inward towards the plate (a small positive value, e.g., 5-15 degrees).\n
\n
\n
\n
\n \n \n
\n
Learn to tie your shoelaces!
\n (Bunny Ears Method)
\n\n \n
\n
Press the first button to start!
\n\n \n \n
\n\n \n\n \n \n\n\n \n\n\n"},{"title":"","url":"https://www.youtube.com/watch?v=auJzb1D-fag","spec":"The video features the song 'Never Gonna Give You Up,' and is associated with the internet phenomenon known as 'Rickrolling.'\n\nBuild me an interactive web app to teach users about avoiding unwanted experiences, and to help them identify potential 'Rickrolls' in the future.\n\nSPECIFICATIONS:\n\n1. The app should present the user with a series of fake links.\n\n2. The app must somehow indicate that clicking one of the fake links will lead to an unwanted experience. For example, the app could show a button that says 'Click me to find out more' that, if clicked, navigates the user to a screen that says 'You've been Rickrolled'.\n\n3. The app must challenge the user to distinguish between safe and unsafe links. If the user correctly identifies an unsafe link, they should be rewarded with positive feedback. If the user incorrectly identifies a safe link as unsafe, they should be given constructive criticism.\n\n4. The app should track the user's progress. The app should record the number of unsafe links that the user correctly identifies, and the number of safe links that the user incorrectly identifies as unsafe.\n\n5. The app should provide a summary of the user's performance at the end of the game. The app should show the number of unsafe links that the user correctly identified, and the number of safe links that the user incorrectly identified as unsafe.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Rickroll Prevention Training\n \n\n\n\n \n
Rickroll Prevention Training
\n\n \n
\n
First, let's see what happens when you click a suspicious link unprepared. This button simulates that experience.
\n
\n
\n
💥 You've been Rickrolled! 💥
\n
(Well, simulated anyway. The goal is to NOT see messages like this unexpectedly!)
\n
\n
\n\n
\n\n \n
\n
Identify the Suspicious Links!
\n
Examine the link description and hint below. Decide if it looks like a potential 'Rickroll' (unsafe) or a genuine link (safe). Good luck!
\n\n
\n \n
\n\n
\n \n
\n\n
\n Score:\n Correctly Spotted Unsafe: 0 |\n Mistakenly Flagged Safe: 0\n
\n
\n\n \n
\n
Training Complete!
\n
Here's how you did:
\n
\n Correctly Identified Unsafe Links: 0
\n Safe Links Incorrectly Marked as Unsafe: 0\n
\n
\n
\n
\n\n
\n
Triads and Inversions
\n\n
\n
\n \n \n
\n
\n \n \n
\n
\n \n \n
\n
\n\n
\n
Selection: C Major, Root Position
\n
Formula: Root: C, Third: Major Third (E), Fifth: Perfect Fifth (G)
\n
Notes: C4, E4, G4
\n
Lowest Note:
\n
\n\n
\n
Koch Snowflake Fractal Explorer
\n\n \n
Fractals are patterns that repeat themselves at different scales (self-similarity).
\n
The Koch snowflake starts with an equilateral triangle. Each iteration adds a smaller triangle to the middle third of every side.
\n
Use mouse wheel to zoom, click and drag to pan.
\n
\n Iterations: 0 |\n Formula for sides: 3 * 4n |\n Current Sides: 3\n
\n
\n \n \n \n \n \n
\n\n \n \n
\n\n \n\n\n"},{"title":"Logic behind Chinese characters","url":"https://youtu.be/U0EySK4T2aY?si=HV_ZHWS8KdJZmZJP","spec":"Chinese characters are logograms, symbols that represent a word or phrase. Most modern-day characters are constructed of two components: a radical (which hints at the meaning of the character) and a sound component (which hints at its pronunciation). Radicals can appear in different positions: to the left, to the right, above, below, or even surrounding the rest of the character.\n\nBuild me an interactive web app to help a learner understand the structure of Chinese characters.\n\nSPECIFICATIONS:\n1. The app must feature a central display area for a Chinese character.\n2. The app must feature a palette containing a set of radicals (at least 5).\n3. The app must feature a palette containing a set of sound components (at least 5).\n4. The app must allow the user to drag radicals and/or sound components onto the central display area. When a radical and sound component are combined in this way, the resulting character should be displayed.\n5. The app must feature a short explanation of the radical component and the sound component of each character that is displayed. The short explanation should appear whenever a new character is displayed.\n6. When a new character is displayed, the app should also state the pronunciation of the character (using Pinyin).\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Chinese Character Builder\n \n\n\n\n Chinese Character Builder
\n Drag a radical and a sound component to the central area to form a character.
\n\n \n\n
\n
\n
Radicals (Meaning Hint)
\n \n \n\n
\n
Sound Components (Pronunciation Hint)
\n \n \n
\n\n
\n
\n
\n\n
\n
Drop components above...
\n \n
\n
\n\n
\n
Magical Mitosis
\n
Interphase
\n\n
\n \n
\n\n
The cell grows, copies its DNA (chromosomes), and prepares for division.
\n
\n
Identical Daughter Cells!
\n
\n \n \n 2023\n \n
\n \n
Broadway in 2023
\n
Loading historical context...
\n
\n
Pantry
\n
\n
Base
\n
\n
Rice
\n
Pasta
\n
Potatoes
\n
Polenta
\n
\n
\n
\n
Protein
\n
\n
Ground Beef
\n
Chicken
\n
Tuna
\n
Tofu
\n
\n
\n
\n
Vegetables
\n
\n
Green Beans
\n
Onions
\n
Mushrooms
\n
Tomatoes
\n
Broccoli
\n
\n
\n
\n
Sauce
\n
\n
Tomato Sauce
\n
Cream Soup
\n
Béchamel
\n
Enchilada Sauce
\n
\n
\n
\n
Cheese
\n
\n
Cheddar
\n
Mozzarella
\n
Parmesan
\n
Goat Cheese
\n
\n
\n
\n
Topping
\n
\n
Potato Chips
\n
Fried Onions
\n
Bread Crumbs
\n
More Cheese
\n
\n
\n
\n
Build your casserole!
\n
\n
Drag ingredients to the dish in order!
\n
\n
\n
Casserole Baked!
\n
Delicious! You've successfully built your casserole.
\n \n
\n
\n
\n Select a cocktail to begin...\n \n
\n\n \n\n\n"},{"title":"Calligraphy & handlettering","url":"https://www.youtube.com/watch?v=sBoVGqiSzr4","spec":"Build an interactive web app to teach the fundamentals of modern calligraphy. The app should emphasize the contrast between thick downstrokes and thin upstrokes, and allow the user to practice creating this contrast.\n\nSPECIFICATIONS:\n\n1. The app should display an interactive canvas or drawing area where the user can draw lines.\n2. The app must have a control (e.g., a slider or button) that allows the user to adjust the \"pressure\" of the brush or pen. This pressure will determine the thickness of the line being drawn.\n3. The drawing area should include an optional grid overlay, mimicking the guidelines used in traditional calligraphy as seen in the video.\n4. The app should provide an exercise where a user is shown a simple waveform (like a sine wave) and asked to trace it with the interactive brush, creating thick downstrokes and thin upstrokes based on the direction of movement.\n5. The app must also allow the user to enter a word or phrase, and then trace the letters with appropriate thick and thin strokes, i.e., downstrokes will be thicker, upstrokes will be thinner\n6. The app should include a reset button to clear the canvas and start over.\n7. Provide visual feedback to the user when they successfully complete a stroke with the correct pressure (e.g., a color change or a small animation).\n8. The app should feature an option to display X-height lines to help users maintain consistent letter sizing.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Calligraphy Practice Pad\n \n\n\n\n Modern Calligraphy Practice
\n Practice thick downstrokes and thin upstrokes.
\n\n \n\n \n \n \n
\n\n \n\n\n"},{"title":"Making friends","url":"https://www.youtube.com/watch?v=I9hJ_Rux9y0","spec":"Build an interactive web app to help users explore the principles of making and maintaining friendships, based on the Kurzgesagt video about friendship. \n\nSPECIFICATIONS:\n1. The app should present the user with a series of scenarios, each representing a different stage or situation in the process of making and maintaining friendships. Scenarios should be drawn from the video's examples (e.g., attending a class, joining a club, moving to a new city, having a breakup, scrolling through contacts on your phone).\n\n2. For each scenario, the app should pose a question to the user, prompting them to choose from a set of actions or behaviors. Actions should be simple, concrete, and reflect either positive or negative friendship-building behaviors discussed in the video.\n\n3. Each action chosen by the user should result in a visual or numerical representation of the impact on their friendship \"score\" or \"network\". This could be a simple bar graph or a network diagram showing connections growing or shrinking. Positive actions increase the score, negative actions decrease it.\n\n4. The app should track the user's overall friendship score throughout the scenarios. The user should receive feedback at the end of each scenario, and at the end of the game, based on their final score. This feedback can provide encouragement and tips for improving their friendship skills.\n\n5. The app should incorporate at least three scenarios, and ideally closer to 5-7. The scenarios should be varied, and should cover both initiating new friendships and maintaining existing ones.\n\n6. Keep the interface simple and visually appealing. Use colors and imagery that evoke a sense of warmth, connection, and fun, but avoid being overly sentimental or saccharine.\n\n7. The actions presented should be easily understandable and should clearly relate back to the principles discussed in the video (e.g., prioritizing time, showing genuine interest, being open and receptive, inviting people to do fun things, sharing experiences).\n\n8. The app's logic and scoring mechanism should be transparent, so users understand how their actions impact their friendship network. Avoid arbitrary scoring, and instead focus on rewarding behaviors that are known to build strong relationships.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Friendship Explorer\n \n\n\n\n \n
Making friends
\n
Navigate scenarios based on Kurzgesagt's friendship principles!
\n\n
\n
Scenario Title
\n
Scenario description goes here.
\n
Question goes here?
\n
\n \n
\n
\n
\n\n
\n
Your Friendship Potential
\n
\n
Score: 50\n
\n\n
\n
Journey Complete!
\n
Your final score is: X
\n
Final feedback message goes here.
\n
\n
\n
\n
Baseball Hitting Tilt Trainer
\n
\n
\n
\n
\n
\n
\n
Shoulder Tilt: 0 degrees\n
\n Ideal shoulder tilt at launch position is slightly downward (approximately -9 degrees).\n
\n
\n
\n
\n
\n
Barrel Tilt: 0 degrees\n
\n Ideal barrel tilt at launch position is slightly inward towards the plate (a small positive value, e.g., 5-15 degrees).\n
\n
\n
\n
\n \n \n
\n
Learn to tie your shoelaces!
\n (Bunny Ears Method)
\n\n \n
\n
Press the first button to start!
\n\n \n \n
\n\n \n\n \n \n\n\n \n\n\n"},{"title":"","url":"https://www.youtube.com/watch?v=dQw4w9WgXcQ","spec":"The video features the song 'Never Gonna Give You Up,' and is associated with the internet phenomenon known as 'Rickrolling.'\n\nBuild me an interactive web app to teach users about avoiding unwanted experiences, and to help them identify potential 'Rickrolls' in the future.\n\nSPECIFICATIONS:\n\n1. The app should present the user with a series of fake links.\n\n2. The app must somehow indicate that clicking one of the fake links will lead to an unwanted experience. For example, the app could show a button that says 'Click me to find out more' that, if clicked, navigates the user to a screen that says 'You've been Rickrolled'.\n\n3. The app must challenge the user to distinguish between safe and unsafe links. If the user correctly identifies an unsafe link, they should be rewarded with positive feedback. If the user incorrectly identifies a safe link as unsafe, they should be given constructive criticism.\n\n4. The app should track the user's progress. The app should record the number of unsafe links that the user correctly identifies, and the number of safe links that the user incorrectly identifies as unsafe.\n\n5. The app should provide a summary of the user's performance at the end of the game. The app should show the number of unsafe links that the user correctly identified, and the number of safe links that the user incorrectly identified as unsafe.\n\nProvide the code as a single, self-contained HTML document. All styles and scripts must be inline. In the result, encase the code between \"```\" and \"```\" for easy parsing.","code":"\n\n\n \n \n Rickroll Prevention Training\n \n\n\n\n \n
Rickroll Prevention Training
\n\n \n
\n
First, let's see what happens when you click a suspicious link unprepared. This button simulates that experience.
\n
\n
\n
💥 You've been Rickrolled! 💥
\n
(Well, simulated anyway. The goal is to NOT see messages like this unexpectedly!)
\n
\n
\n\n
\n\n \n
\n
Identify the Suspicious Links!
\n
Examine the link description and hint below. Decide if it looks like a potential 'Rickroll' (unsafe) or a genuine link (safe). Good luck!
\n\n
\n \n
\n\n
\n \n
\n\n
\n Score:\n Correctly Spotted Unsafe: 0 |\n Mistakenly Flagged Safe: 0\n
\n
\n\n \n
\n
Training Complete!
\n
Here's how you did:
\n
\n Correctly Identified Unsafe Links: 0
\n Safe Links Incorrectly Marked as Unsafe: 0\n
\n
\n
\n
\n\n
✅ Generation Complete!
const data = await response.json();
- // Display results securely using DOM APIs
- resultContent.textContent = ''; // Clear previous contents safely
-
- const sanitizeUrl = (url) => {
- if (!url) return '#';
- const strUrl = String(url).trim();
- // Block dangerous protocols
- if (/^(javascript|vbscript|data):/i.test(strUrl)) {
- return '#';
- }
- return strUrl;
- };
-
- const appendResultItem = (label, value, isLink = false) => {
- if (value === undefined || value === null) return;
-
- const div = document.createElement('div');
- div.className = 'result-item';
-
- const strong = document.createElement('strong');
- strong.textContent = label + ': ';
- div.appendChild(strong);
-
- if (isLink) {
- const a = document.createElement('a');
- a.href = sanitizeUrl(value);
- a.target = '_blank';
- a.className = 'link';
- a.textContent = String(value);
- div.appendChild(a);
- } else {
- div.appendChild(document.createTextNode(String(value)));
- }
-
- resultContent.appendChild(div);
- };
-
- appendResultItem('Project Name', data.project_name);
- appendResultItem('Live URL', data.live_url, true);
- appendResultItem('GitHub Repo', data.github_repo, true);
- appendResultItem('Build Status', data.build_status);
- appendResultItem('Processing Time', data.processing_time);
-
- if (data.code_generation) {
- appendResultItem('Framework', data.code_generation.framework || 'N/A');
- appendResultItem('Files Created', data.code_generation.files_created?.length || 0);
- }
+ // Display results
+ resultContent.innerHTML = `
+
+ Project Name: ${data.project_name}
+
+
+
+
+ Build Status: ${data.build_status}
+
+
+ Processing Time: ${data.processing_time}
+
+ ${data.code_generation ? `
+
+ Framework: ${data.code_generation.framework || 'N/A'}
+
+
+ Files Created: ${data.code_generation.files_created?.length || 0}
+
+ ` : ''}
+ `;
result.style.display = 'block';
diff --git a/src/youtube_extension/backend/video_processor_factory.py b/src/youtube_extension/backend/video_processor_factory.py
index 37ce32e01..64dedc349 100644
--- a/src/youtube_extension/backend/video_processor_factory.py
+++ b/src/youtube_extension/backend/video_processor_factory.py
@@ -6,25 +6,15 @@
Factory module to provide working video processors with proper fallbacks.
"""
-from __future__ import annotations
-
import logging
import os
-from typing import TYPE_CHECKING
-
-if TYPE_CHECKING:
- from youtube_extension.backend.deepmcp.deepmcp_processor import (
- DeepMCPAgentProcessor,
- )
-
- from .enhanced_video_processor import EnhancedVideoProcessor
- from .real_video_processor import RealVideoProcessor
+from typing import Union
from youtube_extension.utils.proxy import get_proxy_url
logger = logging.getLogger(__name__)
-def get_video_processor(processor_type: str = "auto") -> EnhancedVideoProcessor | RealVideoProcessor | DeepMCPAgentProcessor:
+def get_video_processor(processor_type: str = "auto") -> Union['EnhancedVideoProcessor', 'RealVideoProcessor', 'DeepMCPAgentProcessor']:
"""
Get appropriate video processor based on configuration
@@ -80,7 +70,7 @@ def get_video_processor(processor_type: str = "auto") -> EnhancedVideoProcessor
return EnhancedVideoProcessor()
except Exception as e2:
logger.error(f"All processors failed: {e2}")
- raise ValueError(f"No working video processor available: {e2}") from e2
+ raise ValueError(f"No working video processor available: {e2}")
if processor_type == "hybrid":
"""Hybrid processor using FastVLM + Gemini pipeline for video understanding.
@@ -155,7 +145,7 @@ async def process_video(self, video_url: str) -> dict[str, Any]:
return EnhancedVideoProcessor()
except Exception as e2:
logger.error(f"Hybrid fallback failed: {e2}")
- raise ValueError(f"No working video processor available: {e2}") from e2
+ raise ValueError(f"No working video processor available: {e2}")
# Final fallback
try:
@@ -164,7 +154,7 @@ async def process_video(self, video_url: str) -> dict[str, Any]:
return EnhancedVideoProcessor()
except Exception as e:
logger.error(f"Final fallback failed: {e}")
- raise ValueError(f"No working video processor available: {e}") from e
+ raise ValueError(f"No working video processor available: {e}")
# Compatibility wrapper for gradual migration
diff --git a/src/youtube_extension/backend/video_processor_interface.py b/src/youtube_extension/backend/video_processor_interface.py
index 98b1618f5..d473f3b7f 100644
--- a/src/youtube_extension/backend/video_processor_interface.py
+++ b/src/youtube_extension/backend/video_processor_interface.py
@@ -11,5 +11,5 @@
class VideoProcessor(Protocol):
- async def process_video(self, video_url: str) -> dict[str, Any]:
- ...
+ async def process_video(self, video_url: str) -> dict[str, Any]:
+ ...
diff --git a/src/youtube_extension/orchestrator/main.py b/src/youtube_extension/orchestrator/main.py
index 551b108ac..804577bd3 100644
--- a/src/youtube_extension/orchestrator/main.py
+++ b/src/youtube_extension/orchestrator/main.py
@@ -1,15 +1,7 @@
-from __future__ import annotations
-
import asyncio
import logging
import os
import signal
-from urllib.parse import urlparse
-
-try:
- import redis.asyncio as redis
-except ImportError:
- redis = None
# Configure logging
logging.basicConfig(
@@ -18,66 +10,14 @@
)
logger = logging.getLogger("orchestrator")
-
-def redact_url(url: str) -> str:
- """Redact credentials from URL for safe logging."""
- try:
- parsed = urlparse(url)
- if parsed.password or parsed.username:
- redacted = parsed._replace(netloc=f"{parsed.username or ''}:***@{parsed.hostname}:{parsed.port or ''}")
- return redacted.geturl()
- return url.split('@')[-1] if '@' in url else url
- except Exception:
- return "redis://***"
-
-
-async def process(msg: dict) -> None:
- """Handle a single consumed message.
-
- No real task handler is wired up yet. Per the REAL_MODE_ONLY policy we must
- not fake success with a mock delay: raising here leaves the message
- unacknowledged (retained in the stream's pending list) rather than silently
- dropping real work behind a stub that immediately gets xack'ed.
- """
- logger.info(f"Received message (no handler implemented yet): {msg}")
- raise NotImplementedError(
- "Orchestrator task handler is not implemented; message left unacknowledged"
- )
-
-
-async def ensure_consumer_group(
- redis_client: redis.Redis, stream_name: str, consumer_group: str
-) -> None:
- """Ensure the Redis Streams consumer group exists.
-
- Only the "already exists" (BUSYGROUP) case is treated as success. Any other
- error — most importantly a transient ConnectionError while Redis is still
- starting up — is re-raised so the caller can retry. Swallowing those errors
- would leave the group uncreated while the consumer keeps looping, producing a
- permanent NOGROUP failure that never recovers and never consumes any tasks.
- """
- try:
- await redis_client.xgroup_create(
- stream_name, consumer_group, id='0', mkstream=True
- )
- logger.info(
- f"Created consumer group '{consumer_group}' for stream '{stream_name}'"
- )
- except Exception as e:
- if "BUSYGROUP" in str(e):
- logger.debug(f"Consumer group '{consumer_group}' already exists")
- else:
- raise
-
-
-async def main() -> None:
+async def main():
"""
Main Orchestrator Loop.
In a full production environment, this service would consume messages from
RabbitMQ or Redis to trigger video processing tasks asynchronously.
- Current Status: Implemented Redis Streams consumer with acknowledged delivery.
+ Current Status: Placeholder for future async worker implementation.
"""
logger.info("🚀 Orchestrator Service Starting...")
@@ -85,92 +25,30 @@ async def main() -> None:
loop = asyncio.get_running_loop()
stop_event = asyncio.Event()
- def signal_handler() -> None:
+ def signal_handler():
logger.info("🛑 Shutdown signal received")
stop_event.set()
for sig in (signal.SIGTERM, signal.SIGINT):
loop.add_signal_handler(sig, signal_handler)
- # Accept REDIS_URL as fallback for deployed environments
- redis_url = os.getenv("MESSAGE_QUEUE_URL") or os.getenv("REDIS_URL", "redis://localhost:6379")
- stream_name = os.getenv("ORCHESTRATOR_QUEUE_NAME", "orchestrator_tasks")
- consumer_group = os.getenv("ORCHESTRATOR_CONSUMER_GROUP", "orchestrator_workers")
- consumer_name = os.getenv("HOSTNAME", "orchestrator_1")
- redis_client = None
-
- if redis is not None:
- try:
- # Bounded timeouts so a hung/half-open connection surfaces as an
- # exception (which the loop handles) instead of blocking xreadgroup /
- # xack / xgroup_create indefinitely. socket_timeout must exceed the
- # 1s xreadgroup block below.
- redis_client = redis.from_url(
- redis_url,
- socket_connect_timeout=5,
- socket_timeout=10,
- )
- # Redact credentials from URL for safe logging
- safe_url = redact_url(redis_url)
- logger.info(f"✅ Orchestrator initialized, connecting to Redis at {safe_url} (Stream: {stream_name})")
- except Exception as e:
- logger.error(f"Failed to initialize Redis client: {e}")
- redis_client = None
-
- if redis_client is None:
- logger.info("✅ Orchestrator initialized and waiting for tasks (Mode: Standby)")
-
- # Whether the consumer group has been confirmed to exist. Created lazily inside
- # the loop so a transient failure at startup is retried instead of stranding the
- # consumer, and reset on any loop error so a lost connection or a missing group
- # (NOGROUP) triggers re-creation on the next iteration.
- group_ready = False
+ logger.info("✅ Orchestrator initialized and waiting for tasks (Mode: Standby)")
# Main loop
while not stop_event.is_set():
try:
- if redis_client:
- if not group_ready:
- await ensure_consumer_group(redis_client, stream_name, consumer_group)
- group_ready = True
+ # TODO: Implement RabbitMQ/Redis consumer here
+ # msg = await queue.get()
+ # process(msg)
- # Use Redis Streams with consumer groups for acknowledged delivery
- # Read with 1 second block timeout so we can check stop_event frequently
- results = await redis_client.xreadgroup(
- consumer_group,
- consumer_name,
- {stream_name: '>'},
- count=1,
- block=1000 # 1 second in milliseconds
- )
-
- if results:
- for _stream, messages in results:
- for message_id, data in messages:
- try:
- # Process the message
- await process(data)
- # Acknowledge successful processing
- await redis_client.xack(stream_name, consumer_group, message_id)
- logger.debug(f"Acknowledged message {message_id}")
- except Exception as proc_error:
- logger.error(f"Failed to process message {message_id}: {proc_error}")
- # Message remains unacknowledged and can be reclaimed
- else:
- # Heartbeat for standby mode
- await asyncio.sleep(60)
- logger.debug("❤️ Orchestrator heartbeat")
+ # Heartbeat
+ await asyncio.sleep(60)
+ logger.debug("❤️ Orchestrator heartbeat")
except Exception as e:
- # Force the group to be re-ensured next iteration: the failure may be a
- # dropped connection or a missing group (NOGROUP) that needs re-creating.
- group_ready = False
logger.error(f"Error in orchestrator loop: {e}")
await asyncio.sleep(5)
- if redis_client:
- await redis_client.aclose()
-
logger.info("👋 Orchestrator shutting down")
if __name__ == "__main__":
diff --git a/src/youtube_extension/services/agents/adapters/agent_orchestrator.py b/src/youtube_extension/services/agents/adapters/agent_orchestrator.py
index 66c0ba3c0..52d1d3f03 100644
--- a/src/youtube_extension/services/agents/adapters/agent_orchestrator.py
+++ b/src/youtube_extension/services/agents/adapters/agent_orchestrator.py
@@ -10,7 +10,6 @@
import asyncio
import logging
import uuid
-from collections import deque
from dataclasses import dataclass, field
from datetime import datetime
from typing import Any, Optional
@@ -61,10 +60,7 @@ def __init__(self):
self.logger = logging.getLogger("agent_orchestrator")
self._agents: dict[str, BaseAgent] = {}
self._agent_types: dict[str, type[BaseAgent]] = {}
- # Bounded: the module-level `orchestrator` singleton lives for the whole
- # process and every dispatch appends here, so an unbounded list would
- # grow without limit. maxlen evicts the oldest entries automatically.
- self._a2a_log: deque[A2AContextMessage] = deque(maxlen=1000)
+ self._a2a_log: list[A2AContextMessage] = []
self._task_mappings: dict[str, list[str]] = {
"video_analysis": [
"video_master",
@@ -304,132 +300,6 @@ def add_task_mapping(self, task_type: str, agent_names: list[str]):
self._task_mappings[task_type] = agent_names
self.logger.info(f"Added task mapping: {task_type} -> {agent_names}")
- # --- Single-agent dispatch ---
-
- async def execute_single(
- self,
- agent_type: str,
- context: dict[str, Any],
- config: Optional[dict[str, Any]] = None,
- ) -> dict[str, Any]:
- """
- Execute a single agent by type with the given context.
-
- Used by the agent dispatch system to run one agent against one event.
- The agent is resolved via registered types or the global registry.
-
- Args:
- agent_type: The agent type/name to execute.
- context: Context data (e.g. the extracted event) passed to the agent.
- config: Optional agent-specific configuration.
-
- Returns:
- dict with the agent's output, or an error dict if execution fails.
- """
- agent = await self.get_agent(agent_type, config)
- if not agent:
- self.logger.warning(
- "Agent type %s not found for execute_single", agent_type
- )
- # Record the failed dispatch so the session/audit trail is complete
- # (matches the success, agent-failure, and exception paths below).
- self._a2a_log.append(
- A2AContextMessage(
- sender="orchestrator",
- recipient=agent_type,
- content={
- "type": "agent_dispatch",
- "agent_type": agent_type,
- "context": context,
- "status": "error",
- "error": "agent_not_found",
- },
- )
- )
- return {"error": f"Agent type '{agent_type}' not found"}
-
- try:
- request = AgentRequest(task=agent_type, params=context)
- result = await agent.run(request)
-
- # Log execution in A2A log for session tracking
- self._a2a_log.append(
- A2AContextMessage(
- sender="orchestrator",
- recipient=agent_type,
- content={
- "type": "agent_dispatch",
- "agent_type": agent_type,
- "context": context,
- "status": result.status,
- },
- )
- )
-
- if result.status == "ok":
- return result.output
- else:
- error_msg = (
- "; ".join(result.logs) or "Agent execution failed"
- )
- return {"error": error_msg, "output": result.output}
- except Exception as e:
- self.logger.error("execute_single failed for %s: %s", agent_type, e)
- self._a2a_log.append(
- A2AContextMessage(
- sender="orchestrator",
- recipient=agent_type,
- content={
- "type": "agent_dispatch",
- "agent_type": agent_type,
- "context": context,
- "status": "error",
- "error": str(e),
- },
- )
- )
- return {"error": str(e)}
-
- def get_session_logs(
- self,
- agent_type: str | None = None,
- limit: int = 50,
- ) -> list[dict[str, Any]]:
- """Return agent dispatch session logs, optionally filtered by agent type.
-
- Session logs track which agents were dispatched, what context they received,
- and their execution status. This enables the recursive feedback loop where
- agent findings can be reviewed and re-dispatched.
-
- Args:
- agent_type: Filter to a specific agent type, or None for all.
- limit: Maximum entries to return.
-
- Returns:
- List of session log entries.
- """
- dispatch_msgs = [
- m for m in self._a2a_log
- if m.content.get("type") == "agent_dispatch"
- ]
- if agent_type:
- dispatch_msgs = [
- m for m in dispatch_msgs
- if m.content.get("agent_type") == agent_type
- ]
- return [
- {
- "sender": m.sender,
- "recipient": m.recipient,
- "agent_type": m.content.get("agent_type"),
- "context": m.content.get("context"),
- "status": m.content.get("status"),
- "timestamp": m.timestamp,
- "conversation_id": m.conversation_id,
- }
- for m in dispatch_msgs[-limit:]
- ]
-
# --- A2A messaging ---
async def send_a2a_message(
@@ -464,9 +334,7 @@ def get_a2a_log(
limit: int = 50,
) -> list[dict[str, Any]]:
"""Return recent A2A messages, optionally filtered by conversation."""
- # Materialize to a list so `[-limit:]` slicing works (deque is not
- # sliceable).
- msgs = list(self._a2a_log)
+ msgs = self._a2a_log
if conversation_id:
msgs = [m for m in msgs if m.conversation_id == conversation_id]
return [
diff --git a/src/youtube_extension/services/ai/vercel_gateway_provider.py b/src/youtube_extension/services/ai/vercel_gateway_provider.py
index dec61e80f..193c53b6e 100644
--- a/src/youtube_extension/services/ai/vercel_gateway_provider.py
+++ b/src/youtube_extension/services/ai/vercel_gateway_provider.py
@@ -120,16 +120,9 @@ def extract_events(
'"title": short string, "description": string or null, '
'"timestamp": string or null}.'
)
- # Increase context window to 120k chars (~30k tokens) to avoid silent data loss
- limit = 120_000
- truncated_transcript = transcript[:limit]
- if len(transcript) > limit:
- logger.info("Transcript truncated from %d to %d chars for Vercel AI Gateway extraction", len(transcript), limit)
- truncated_transcript += "\n[... transcript truncated ...]"
-
user = (
f"Extract up to {max_events} key events from this transcript text:\n\n"
- + truncated_transcript
+ + transcript[:8000]
)
content = chat(
[{"role": "system", "content": system}, {"role": "user", "content": user}],
diff --git a/src/youtube_extension/services/workflows/transcript_action_workflow.py b/src/youtube_extension/services/workflows/transcript_action_workflow.py
index a64a63b6b..480cde3ff 100644
--- a/src/youtube_extension/services/workflows/transcript_action_workflow.py
+++ b/src/youtube_extension/services/workflows/transcript_action_workflow.py
@@ -765,25 +765,6 @@ async def _fallback_transcript_with_gemini(
video_metadata=video_metadata,
)
- # Retry once for transient network errors (timeouts, 503/504, deadline
- # exceeded) before falling through to the yt-dlp download fallback.
- # This avoids triggering the expensive and bot-blocked download path
- # for what are often short-lived Gemini API hiccups.
- _transient_markers = ("timeout", "503", "504", "deadline", "unavailable")
- if not (primary_result.success and primary_result.response) and primary_result.error:
- if any(m in str(primary_result.error).lower() for m in _transient_markers):
- logger.info(
- "Transient error in Gemini process_youtube (%s); retrying before download fallback",
- primary_result.error,
- )
- await asyncio.sleep(2.0)
- primary_result = await gemini_service.process_youtube(
- video_url,
- transcription_prompt,
- response_mime_type="application/json",
- video_metadata=video_metadata,
- )
-
await self._record_metric(
"transcript_fallback_latency_seconds",
(primary_result.latency or 0.0),
@@ -1003,56 +984,24 @@ async def _download_video_file(self, video_url: str) -> tuple[Path | None, Path
def _download() -> tuple[Path | None, Path | None]:
temp_dir = Path(tempfile.mkdtemp(prefix="gemini_video_"))
- try:
- output_template = str(temp_dir / "%(id)s.%(ext)s")
- ydl_opts = {
- "skip_download": False,
- # Use mweb then web_embedded clients — both are significantly
- # less scrutinised than the default web client on GCP/Cloud Run
- # IPs. web_embedded does not require a PO Token for public videos.
- "extractor_args": {
- "youtube": {
- "player_client": ["mweb", "web_embedded"],
- }
- },
- # Prefer a pre-muxed stream (no ffmpeg required) before falling
- # back to separate video+audio tracks that need merging.
- "format": "best[ext=mp4]/bestvideo[ext=mp4]+bestaudio[ext=m4a]/best",
- "merge_output_format": "mp4",
- "outtmpl": output_template,
- "noplaylist": True,
- "quiet": True,
- # With noplaylist=True, ignoreerrors makes extract_info return
- # None on a failed download instead of raising; the None guard
- # below turns that into an explicit FileNotFoundError. (Format
- # fallback via the "/" chain is handled by yt-dlp's format
- # selection regardless. `abort_on_error` is a CLI-only compat
- # option ignored by the API.)
- "ignoreerrors": True,
- }
- proxy_url = get_proxy_url()
- if proxy_url:
- ydl_opts["proxy"] = proxy_url
-
- with yt_dlp.YoutubeDL(ydl_opts) as ydl: # type: ignore[attr-defined]
- info = ydl.extract_info(video_url, download=True)
- # With ignoreerrors=True, extract_info returns None on a failed
- # download instead of raising — guard so callers get the
- # intended FileNotFoundError rather than a cryptic TypeError
- # from prepare_filename(None).
- if info is None:
- raise FileNotFoundError("Video download failed")
- filename = Path(ydl.prepare_filename(info))
- if not filename.exists():
- raise FileNotFoundError("Video download failed")
- return filename, temp_dir
- except Exception:
- # Clean up the temp dir on any failure. The caller only receives
- # temp_dir (and later removes it) on the success path, so an
- # exception here would otherwise leak an orphaned directory —
- # a frequent event now that ignoreerrors makes the None-return
- # the primary download-failure mode.
- shutil.rmtree(temp_dir, ignore_errors=True)
- raise
+ output_template = str(temp_dir / "%(id)s.%(ext)s")
+ ydl_opts = {
+ "skip_download": False,
+ "format": "bestvideo[ext=mp4]+bestaudio[ext=m4a]/mp4",
+ "merge_output_format": "mp4",
+ "outtmpl": output_template,
+ "noplaylist": True,
+ "quiet": True,
+ }
+ proxy_url = get_proxy_url()
+ if proxy_url:
+ ydl_opts["proxy"] = proxy_url
+
+ with yt_dlp.YoutubeDL(ydl_opts) as ydl: # type: ignore[attr-defined]
+ info = ydl.extract_info(video_url, download=True)
+ filename = Path(ydl.prepare_filename(info))
+ if not filename.exists():
+ raise FileNotFoundError("Video download failed")
+ return filename, temp_dir
return await asyncio.to_thread(_download)
diff --git a/src/youtube_extension/utils/video_utils.py b/src/youtube_extension/utils/video_utils.py
index 66ad89e25..705c11128 100644
--- a/src/youtube_extension/utils/video_utils.py
+++ b/src/youtube_extension/utils/video_utils.py
@@ -57,13 +57,9 @@ def extract_video_id(url: str) -> str:
r'^([0-9A-Za-z_-]{11})$'
]
- # Try each pattern. IGNORECASE so an uppercase scheme/host (e.g.
- # HTTPS://YOUTUBE.COM/WATCH?V=...) — which the API-boundary validator now
- # accepts (models._YOUTUBE_URL_REGEX is case-insensitive) — is parsed here
- # too, keeping acceptance and extraction consistent. Capture groups still
- # preserve the case-sensitive 11-char video id verbatim.
+ # Try each pattern
for pattern in patterns:
- match = re.search(pattern, url, re.IGNORECASE)
+ match = re.search(pattern, url)
if match:
video_id = match.group(1)
# Validate it's exactly 11 characters (YouTube standard)
diff --git a/tests/check_data_service.py b/tests/check_data_service.py
index dd694774c..c667648c7 100644
--- a/tests/check_data_service.py
+++ b/tests/check_data_service.py
@@ -7,7 +7,7 @@ def check_data():
# Initialize DataService directly. It uses default directories unless specified.
# The container usually passes enhanced_analysis_dir="youtube_processed_videos/enhanced_analysis"
service = DataService()
- video_id = "auJzb1D-fag"
+ video_id = "dQw4w9WgXcQ"
print(f"Checking data for {video_id}...")
detail = service.get_video_detail(video_id)
diff --git a/tests/integration/test_skill_di.py b/tests/integration/test_skill_di.py
deleted file mode 100644
index 6bdf26a4f..000000000
--- a/tests/integration/test_skill_di.py
+++ /dev/null
@@ -1,53 +0,0 @@
-from unittest.mock import MagicMock
-
-import pytest
-
-from agents.mcp_ecosystem_coordinator import SkillRegistry
-from skills.base import SkillResult
-from youtube_extension.backend.containers.service_container import (
- get_service_container,
-)
-
-
-@pytest.mark.asyncio
-async def test_skill_di_wiring():
- """Verify skills are instantiated with their dependencies from the ServiceContainer."""
- container = get_service_container()
-
- # Inject mock services into the container for testing.
- mock_gemini = MagicMock()
- mock_db = MagicMock()
- container._singletons["gemini_service"] = mock_gemini
- container._singletons["database_service"] = mock_db
-
- registry = SkillRegistry()
-
- # ContentGenerationSkill requires gemini_service and database_service.
- instance = registry._load_skill_instance("content-generation")
- assert instance.gemini is mock_gemini
- assert instance.db is mock_db
-
- # LeadScorerSkill requires only database_service.
- instance = registry._load_skill_instance("lead-scorer")
- assert instance.db is mock_db
- assert not hasattr(instance, "gemini") or instance.gemini is None
-
-
-@pytest.mark.asyncio
-async def test_invoke_skill_with_di():
- """Verify that invoking a skill routes the payload to the DI-injected instance."""
- registry = SkillRegistry()
-
- instance = registry._load_skill_instance("seo-optimizer")
-
- # Replace execute with an async callable (a coroutine function), so that
- # `await instance.execute(payload)` inside invoke_skill works correctly.
- async def fake_execute(payload):
- return SkillResult(status="success", output={"test": "ok"})
-
- instance.execute = fake_execute
-
- result = await registry.invoke_skill("seo-optimizer", {"video_id": "test_video"})
-
- assert result["status"] == "success"
- assert result["output"]["test"] == "ok"
diff --git a/tests/load/basic-load-test.js b/tests/load/basic-load-test.js
new file mode 100644
index 000000000..1c343697d
--- /dev/null
+++ b/tests/load/basic-load-test.js
@@ -0,0 +1,51 @@
+import http from 'k6/http';
+import { check, sleep, group } from 'k6';
+
+export const options = {
+ stages: [
+ { duration: '30s', target: 10 }, // Ramp up to 10 users
+ { duration: '1m', target: 50 }, // Ramp up to 50 users
+ { duration: '2m', target: 50 }, // Stay at 50 users
+ { duration: '30s', target: 0 }, // Ramp down
+ ],
+ thresholds: {
+ http_req_duration: ['p(95)<500', 'p(99)<1000'],
+ http_req_failed: ['rate<0.01'],
+ },
+};
+
+export default function () {
+ const baseUrl = __ENV.BASE_URL || 'http://localhost:8000';
+ const params = {
+ headers: {
+ 'Content-Type': 'application/json',
+ },
+ };
+
+ group('Warmup and Status', function () {
+ const healthRes = http.get(`${baseUrl}/health`);
+ check(healthRes, {
+ 'health status is 200': (r) => r.status === 200,
+ });
+
+ const capRes = http.get(`${baseUrl}/api/v1/capabilities`);
+ check(capRes, {
+ 'capabilities status is 200': (r) => r.status === 200,
+ });
+ });
+
+ group('Core Pipeline', function () {
+ const payload = JSON.stringify({
+ video_url: 'https://www.youtube.com/watch?v=auJzb1D-fag',
+ language: 'en'
+ });
+
+ // Using transcript-action for realistic workflow load
+ const actionRes = http.post(`${baseUrl}/api/v1/transcript-action`, payload, params);
+ check(actionRes, {
+ 'transcript-action accepted or processing': (r) => r.status === 200 || r.status === 202,
+ });
+ });
+
+ sleep(Math.random() * 3 + 2);
+}
diff --git a/tests/load/locustfile.py b/tests/load/locustfile.py
new file mode 100644
index 000000000..d090ceda6
--- /dev/null
+++ b/tests/load/locustfile.py
@@ -0,0 +1,33 @@
+from locust import HttpUser, task, between, constant
+import json
+
+class EventRelayUser(HttpUser):
+ # More realistic wait time for a "thinking" or "observing" user
+ wait_time = between(2, 5)
+
+ @task(10)
+ def health_warmup(self):
+ """GET /health - warmup"""
+ self.client.get("/health")
+
+ @task(5)
+ def status_check(self):
+ """GET /api/v1/capabilities - status check"""
+ self.client.get("/api/v1/capabilities")
+
+ @task(2)
+ def api_health(self):
+ """GET /api/v1/health - detailed health"""
+ self.client.get("/api/v1/health")
+
+ # High-impact endpoints (simulated)
+ @task(1)
+ def process_video_simulation(self):
+ """POST /api/v1/process-video - video processing"""
+ payload = {
+ "video_url": "https://www.youtube.com/watch?v=auJzb1D-fag",
+ "options": {"force_refresh": False}
+ }
+ headers = {"Content-Type": "application/json"}
+ # Note: This is an expensive operation; in real load tests we might mock the processing delay
+ self.client.post("/api/v1/process-video", json=payload, headers=headers)
diff --git a/tests/notebooklm_test_ingest.py b/tests/notebooklm_test_ingest.py
index 95b8c18cf..cb28927e6 100644
--- a/tests/notebooklm_test_ingest.py
+++ b/tests/notebooklm_test_ingest.py
@@ -4,7 +4,7 @@
from src.utils.notebooklm_ingest import upload_to_notebooklm
# Define test file path
-TEST_FILE = "youtube_processed_videos/enhanced_analysis/General/auJzb1D-fag_20260211_192116_enhanced.md"
+TEST_FILE = "youtube_processed_videos/enhanced_analysis/General/dQw4w9WgXcQ_20260211_192116_enhanced.md"
if __name__ == "__main__":
if not os.path.exists(TEST_FILE):
diff --git a/tests/populate_video_data.py b/tests/populate_video_data.py
index 9ce9a3921..142b580b1 100644
--- a/tests/populate_video_data.py
+++ b/tests/populate_video_data.py
@@ -7,7 +7,7 @@ async def populate_video():
print("Populating video data...")
url = "http://localhost:8000/api/v1/transcript-action"
- payload = {"video_url": "https://www.youtube.com/watch?v=auJzb1D-fag"}
+ payload = {"video_url": "https://www.youtube.com/watch?v=dQw4w9WgXcQ"}
print(f"Sending request to {url}...")
try:
diff --git a/tests/test_gemini_video_master_agent.py b/tests/test_gemini_video_master_agent.py
index bd7a51216..8b697a321 100644
--- a/tests/test_gemini_video_master_agent.py
+++ b/tests/test_gemini_video_master_agent.py
@@ -79,8 +79,8 @@ def test_default_input_does_not_surface_banned_video_id():
prompt = master.build_default_video_prompt()
assert master.extract_video_id(master.DEFAULT_VIDEO_URL) not in master.BANNED_VIDEO_IDS
- assert "auJzb1D-fag" not in master.DEFAULT_VIDEO_URL
- assert "auJzb1D-fag" not in prompt
+ assert "dQw4w9WgXcQ" not in master.DEFAULT_VIDEO_URL
+ assert "dQw4w9WgXcQ" not in prompt
def test_normalize_youtube_url_removes_timestamp_params():
diff --git a/tests/test_skills_integration.py b/tests/test_skills_integration.py
deleted file mode 100644
index 646c22934..000000000
--- a/tests/test_skills_integration.py
+++ /dev/null
@@ -1,359 +0,0 @@
-"""Integration tests for GTM skill discovery and invocation.
-
-Tests verify:
-- SkillRegistry discovers all 7 GTM skills from skills-lock.json
-- Skills can be invoked and return expected results
-- Trigger-based skill matching works correctly
-- Env var pass-through works without relying on inheritance
-"""
-
-from __future__ import annotations
-
-import json
-import os
-import sys
-import types
-from pathlib import Path
-from unittest.mock import patch
-
-import pytest
-
-# Ensure src is on path for imports
-_SRC = Path(__file__).resolve().parents[1] / "src"
-if str(_SRC) not in sys.path:
- sys.path.insert(0, str(_SRC))
-
-_REPO_ROOT = Path(__file__).resolve().parents[1]
-
-# Avoid importing the full agents package (which pulls heavy deps like aiohttp).
-# Instead, import the coordinator module directly.
-_agents_pkg = sys.modules.get("agents")
-if _agents_pkg is None:
- _agents_pkg = types.ModuleType("agents")
- _agents_pkg.__path__ = [str(_SRC / "agents")] # type: ignore[attr-defined]
- _agents_pkg.__package__ = "agents"
- sys.modules["agents"] = _agents_pkg
-
-# Stub youtube_extension.processors to avoid pulling in heavy ML deps
-for _mod_name in [
- "youtube_extension",
- "youtube_extension.processors",
- "youtube_extension.processors.enhanced_extractor",
-]:
- if _mod_name not in sys.modules:
- _stub = types.ModuleType(_mod_name)
- _stub.__path__ = [] # type: ignore[attr-defined]
- _stub.__package__ = _mod_name
- # Provide stub classes so the coordinator imports fine
- if _mod_name == "youtube_extension.processors.enhanced_extractor":
- _stub.EnhancedVideoExtractor = type("EnhancedVideoExtractor", (), {}) # type: ignore[attr-defined]
- _stub.VideoContent = type("VideoContent", (), {}) # type: ignore[attr-defined]
- sys.modules[_mod_name] = _stub
-
-# Now we can safely import just the coordinator module
-from agents.mcp_ecosystem_coordinator import SkillRegistry # noqa: E402
-
-
-# ---------------------------------------------------------------------------
-# Fixtures
-# ---------------------------------------------------------------------------
-
-LOCK_FILE = str(_REPO_ROOT / "skills-lock.json")
-
-
-@pytest.fixture
-def registry() -> SkillRegistry:
- """Create a SkillRegistry pointed at the repo's skills-lock.json."""
- return SkillRegistry(lock_file_path=LOCK_FILE)
-
-
-# ---------------------------------------------------------------------------
-# Discovery tests
-# ---------------------------------------------------------------------------
-
-
-class TestSkillDiscovery:
- """Verify that SkillRegistry can discover all 7 GTM skills."""
-
- def test_list_skills_returns_seven(self, registry: SkillRegistry) -> None:
- skills = registry.list_skills()
- assert len(skills) == 7
-
- def test_all_expected_skill_ids_present(self, registry: SkillRegistry) -> None:
- skills = registry.list_skills()
- skill_ids = {s["id"] for s in skills}
- expected = {
- "content-generation",
- "seo-optimizer",
- "social-scheduler",
- "lead-scorer",
- "email-campaign",
- "analytics-dashboard",
- "ab-testing",
- }
- assert skill_ids == expected
-
- def test_each_skill_has_required_metadata(self, registry: SkillRegistry) -> None:
- skills = registry.list_skills()
- for skill in skills:
- assert "id" in skill
- assert "name" in skill
- assert "version" in skill
- assert "triggers" in skill
- assert "entry_point" in skill
- assert isinstance(skill["triggers"], list)
- assert len(skill["triggers"]) >= 1
-
- def test_get_skill_by_id(self, registry: SkillRegistry) -> None:
- skill = registry.get_skill("content-generation")
- assert skill is not None
- assert skill["id"] == "content-generation"
- assert skill["name"] == "Content Generation"
- assert skill["class_name"] == "ContentGenerationSkill"
- assert skill["version"] == "1.0.0"
- assert "youtube.video.published" in skill["triggers"]
-
- def test_get_nonexistent_skill_returns_none(self, registry: SkillRegistry) -> None:
- assert registry.get_skill("nonexistent-skill") is None
-
-
-# ---------------------------------------------------------------------------
-# Trigger matching tests
-# ---------------------------------------------------------------------------
-
-
-class TestSkillTriggerMatching:
- """Verify trigger-based skill discovery."""
-
- def test_video_published_triggers_content_generation(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("youtube.video.published")
- skill_ids = {s["id"] for s in skills}
- assert "content-generation" in skill_ids
-
- def test_video_uploaded_triggers_seo_and_ab(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("youtube.video.uploaded")
- skill_ids = {s["id"] for s in skills}
- assert "seo-optimizer" in skill_ids
- assert "ab-testing" in skill_ids
-
- def test_content_generated_triggers_social_scheduler(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("ai.content.generated")
- skill_ids = {s["id"] for s in skills}
- assert "social-scheduler" in skill_ids
-
- def test_analytics_updated_triggers_lead_scorer(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("youtube.analytics.updated")
- skill_ids = {s["id"] for s in skills}
- assert "lead-scorer" in skill_ids
-
- def test_lead_scored_triggers_email_campaign(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("crm.lead.scored")
- skill_ids = {s["id"] for s in skills}
- assert "email-campaign" in skill_ids
-
- def test_daily_cron_triggers_analytics_dashboard(
- self, registry: SkillRegistry
- ) -> None:
- skills = registry.get_skills_for_trigger("system.cron.daily")
- skill_ids = {s["id"] for s in skills}
- assert "analytics-dashboard" in skill_ids
-
- def test_unknown_trigger_returns_empty(self, registry: SkillRegistry) -> None:
- skills = registry.get_skills_for_trigger("unknown.event.type")
- assert skills == []
-
-
-# ---------------------------------------------------------------------------
-# Invocation tests
-# ---------------------------------------------------------------------------
-
-
-class TestSkillInvocation:
- """Verify that skills can be invoked with payloads."""
-
- @pytest.mark.asyncio
- async def test_invoke_content_generation_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "content-generation",
- {"transcript": "Hello world test transcript", "video_id": "auJzb1D-fag"},
- )
- assert result["status"] == "success"
- assert result["output"]["video_id"] == "auJzb1D-fag"
- assert result["output"]["generated"] is True
-
- @pytest.mark.asyncio
- async def test_invoke_content_generation_missing_transcript(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "content-generation",
- {"video_id": "auJzb1D-fag"},
- )
- assert result["status"] == "error"
- assert "transcript" in (result.get("error") or "")
-
- @pytest.mark.asyncio
- async def test_invoke_seo_optimizer_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "seo-optimizer",
- {"video_id": "auJzb1D-fag", "title": "Test Video", "tags": ["ai"]},
- )
- assert result["status"] == "success"
- assert result["output"]["optimized"] is True
-
- @pytest.mark.asyncio
- async def test_invoke_social_scheduler_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "social-scheduler",
- {"content": "Check out this video!", "platforms": ["twitter"]},
- )
- assert result["status"] == "success"
- assert result["output"]["scheduled"] is True
-
- @pytest.mark.asyncio
- async def test_invoke_lead_scorer_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "lead-scorer",
- {"lead_id": "lead_001", "signals": {"views": 100, "comments": 5}},
- )
- assert result["status"] == "success"
- assert result["output"]["lead_id"] == "lead_001"
-
- @pytest.mark.asyncio
- async def test_invoke_email_campaign_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "email-campaign",
- {"lead_id": "lead_001", "campaign_type": "nurture"},
- )
- assert result["status"] == "success"
- assert result["output"]["campaign_type"] == "nurture"
-
- @pytest.mark.asyncio
- async def test_invoke_analytics_dashboard_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "analytics-dashboard",
- {"date_range": "2024-01-01/2024-01-31"},
- )
- assert result["status"] == "success"
- assert result["output"]["generated"] is True
-
- @pytest.mark.asyncio
- async def test_invoke_ab_testing_success(
- self, registry: SkillRegistry
- ) -> None:
- result = await registry.invoke_skill(
- "ab-testing",
- {
- "video_id": "auJzb1D-fag",
- "test_type": "thumbnail",
- "variants": [{"url": "thumb1.jpg"}, {"url": "thumb2.jpg"}],
- },
- )
- assert result["status"] == "success"
- assert result["output"]["variant_count"] == 2
-
- @pytest.mark.asyncio
- async def test_invoke_nonexistent_skill(self, registry: SkillRegistry) -> None:
- result = await registry.invoke_skill("nonexistent", {"foo": "bar"})
- assert result["status"] == "error"
-
-
-# ---------------------------------------------------------------------------
-# MCP env pass-through tests
-# ---------------------------------------------------------------------------
-
-
-class TestEnvPassthrough:
- """Verify explicit env var pass-through for skill subprocesses."""
-
- def test_gemini_skill_gets_api_key(self, registry: SkillRegistry) -> None:
- with patch.dict(os.environ, {"GEMINI_API_KEY": "test-key-123"}):
- env = registry.get_env_for_skill("content-generation")
- assert env["GEMINI_API_KEY"] == "test-key-123"
-
- def test_database_skill_gets_database_url(
- self, registry: SkillRegistry
- ) -> None:
- with patch.dict(os.environ, {"DATABASE_URL": "sqlite:///test.db"}):
- env = registry.get_env_for_skill("lead-scorer")
- assert env["DATABASE_URL"] == "sqlite:///test.db"
-
- def test_multi_dep_skill_gets_both_vars(self, registry: SkillRegistry) -> None:
- with patch.dict(
- os.environ,
- {"GEMINI_API_KEY": "gkey", "DATABASE_URL": "sqlite:///test.db"},
- ):
- env = registry.get_env_for_skill("ab-testing")
- assert env["GEMINI_API_KEY"] == "gkey"
- assert env["DATABASE_URL"] == "sqlite:///test.db"
-
- def test_missing_env_var_not_included(self, registry: SkillRegistry) -> None:
- with patch.dict(os.environ, {}, clear=True):
- # Remove the vars if they exist
- os.environ.pop("GEMINI_API_KEY", None)
- os.environ.pop("DATABASE_URL", None)
- env = registry.get_env_for_skill("content-generation")
- assert "GEMINI_API_KEY" not in env
-
- def test_nonexistent_skill_env_empty(self, registry: SkillRegistry) -> None:
- env = registry.get_env_for_skill("nonexistent")
- assert env == {}
-
-
-# ---------------------------------------------------------------------------
-# Lock file validation
-# ---------------------------------------------------------------------------
-
-
-class TestSkillsLockFile:
- """Verify skills-lock.json structure and validity."""
-
- def test_lock_file_is_valid_json(self) -> None:
- with open(LOCK_FILE) as f:
- data = json.load(f)
- assert "skills" in data
- assert isinstance(data["skills"], dict)
-
- def test_lock_file_contains_gtm_skills(self) -> None:
- with open(LOCK_FILE) as f:
- data = json.load(f)
- gtm_skills = {
- k: v
- for k, v in data["skills"].items()
- if v.get("source") == "uvai-skills"
- }
- assert len(gtm_skills) == 7
-
- def test_each_gtm_skill_has_required_fields(self) -> None:
- with open(LOCK_FILE) as f:
- data = json.load(f)
- for skill_id, meta in data["skills"].items():
- if meta.get("source") != "uvai-skills":
- continue
- assert "skillPath" in meta, f"{skill_id} missing skillPath"
- assert "className" in meta, f"{skill_id} missing className"
- assert "version" in meta, f"{skill_id} missing version"
- assert "triggers" in meta, f"{skill_id} missing triggers"
- assert "dependencies" in meta, f"{skill_id} missing dependencies"
diff --git a/tests/testing/test_full_pipeline.py b/tests/testing/test_full_pipeline.py
index 70c9e3ed5..b75370e4a 100644
--- a/tests/testing/test_full_pipeline.py
+++ b/tests/testing/test_full_pipeline.py
@@ -27,7 +27,7 @@ async def test_full_pipeline():
print()
# Short tech video for testing
- test_video = "https://www.youtube.com/watch?v=auJzb1D-fag"
+ test_video = "https://www.youtube.com/watch?v=dQw4w9WgXcQ"
print(f"Test Video: {test_video}")
print()
diff --git a/tests/testing/test_real_video_processing.py b/tests/testing/test_real_video_processing.py
index 5c74b78f5..e3b575f8a 100644
--- a/tests/testing/test_real_video_processing.py
+++ b/tests/testing/test_real_video_processing.py
@@ -21,7 +21,7 @@ async def test_real_video_processing():
print()
# Test video (short tech explainer)
- test_video = "https://www.youtube.com/watch?v=auJzb1D-fag" # Classic test video
+ test_video = "https://www.youtube.com/watch?v=dQw4w9WgXcQ" # Classic test video
print(f"Test Video: {test_video}")
print()
diff --git a/tests/testing/test_video_utils.py b/tests/testing/test_video_utils.py
index 3891c397e..06febc2b3 100644
--- a/tests/testing/test_video_utils.py
+++ b/tests/testing/test_video_utils.py
@@ -54,19 +54,6 @@ def test_v_format_url(self):
url = "https://www.youtube.com/v/auJzb1D-fag"
assert extract_video_id(url) == "auJzb1D-fag"
- def test_uppercase_scheme_and_host(self):
- """Uppercase scheme/host extracts, and the 11-char id keeps its case."""
- url = "HTTPS://YOUTUBE.COM/WATCH?V=auJzb1D-fag"
- assert extract_video_id(url) == "auJzb1D-fag"
-
- def test_mobile_and_music_hosts(self):
- """m./music. subdomains resolve to the same underlying video id."""
- assert extract_video_id("https://m.youtube.com/watch?v=auJzb1D-fag") == "auJzb1D-fag"
- assert (
- extract_video_id("https://music.youtube.com/watch?v=auJzb1D-fag")
- == "auJzb1D-fag"
- )
-
def test_invalid_url_raises_error(self):
"""Test that invalid URL raises ValueError"""
with pytest.raises(ValueError, match="Could not extract valid YouTube video ID"):
diff --git a/tests/unit/test_agent_orchestrator.py b/tests/unit/test_agent_orchestrator.py
index 23b0013c9..6bf09d7be 100644
--- a/tests/unit/test_agent_orchestrator.py
+++ b/tests/unit/test_agent_orchestrator.py
@@ -140,7 +140,7 @@ def test_agent_types_starts_empty(self):
def test_a2a_log_starts_empty(self):
orch = AgentOrchestrator()
- assert list(orch._a2a_log) == []
+ assert orch._a2a_log == []
def test_default_task_mappings_present(self):
orch = AgentOrchestrator()
@@ -602,166 +602,6 @@ async def test_log_entry_content_matches(self):
assert entry["content"] == {"hello": "world"}
-# ===========================================================================
-# execute_single
-# ===========================================================================
-
-
-class TestExecuteSingle:
- async def test_returns_output_on_success(self):
- orch = AgentOrchestrator()
- agent = _make_ok_agent("analyzer", output={"findings": ["issue_1"]})
- orch._agents["analyzer"] = agent
-
- result = await orch.execute_single(agent_type="analyzer", context={"event": "test"})
- assert result == {"findings": ["issue_1"]}
-
- async def test_returns_error_for_unknown_agent(self):
- orch = AgentOrchestrator()
- result = await orch.execute_single(agent_type="nonexistent", context={})
- assert "error" in result
- assert "not found" in result["error"]
-
- async def test_unknown_agent_dispatch_is_logged(self):
- orch = AgentOrchestrator()
- await orch.execute_single(agent_type="nonexistent", context={"job": "x"})
- # Failed "agent not found" dispatches must appear in the session audit trail
- logs = orch.get_session_logs()
- assert len(logs) == 1
- assert logs[0]["agent_type"] == "nonexistent"
- assert logs[0]["status"] == "error"
-
- async def test_returns_error_on_agent_failure(self):
- orch = AgentOrchestrator()
- agent = _make_error_agent("bad_analyzer")
- orch._agents["bad_analyzer"] = agent
-
- result = await orch.execute_single(agent_type="bad_analyzer", context={"event": "x"})
- assert "error" in result
-
- async def test_returns_error_on_exception(self):
- orch = AgentOrchestrator()
- agent = _make_raising_agent("crash_analyzer")
- orch._agents["crash_analyzer"] = agent
-
- result = await orch.execute_single(agent_type="crash_analyzer", context={})
- assert "error" in result
- assert "agent exploded" in result["error"]
- # Exception dispatches are also logged for session tracking
- assert len(orch._a2a_log) == 1
- assert orch._a2a_log[0].content["status"] == "error"
-
- async def test_logs_dispatch_to_a2a_log(self):
- orch = AgentOrchestrator()
- agent = _make_ok_agent("tracked", output={"ok": True})
- orch._agents["tracked"] = agent
-
- await orch.execute_single(agent_type="tracked", context={"job": "123"})
- assert len(orch._a2a_log) == 1
- msg = orch._a2a_log[0]
- assert msg.content["type"] == "agent_dispatch"
- assert msg.content["agent_type"] == "tracked"
- assert msg.content["status"] == "ok"
-
- async def test_uses_registered_type(self):
- orch = AgentOrchestrator()
- orch.register_agent_type("simple", _SimpleAgent)
-
- result = await orch.execute_single(agent_type="simple", context={"data": "value"})
- assert result == {"task": "simple"}
-
- async def test_passes_config_to_agent(self):
- orch = AgentOrchestrator()
-
- class ConfigAgent(BaseAgent):
- name = "cfgagent"
-
- def __init__(self, config=None):
- self._cfg = config
-
- async def run(self, req):
- return AgentResult(status="ok", output={"cfg": self._cfg})
-
- orch.register_agent_type("cfgagent", ConfigAgent)
- result = await orch.execute_single(
- agent_type="cfgagent",
- context={},
- config={"key": "val"},
- )
- assert result == {"cfg": {"key": "val"}}
-
-
-# ===========================================================================
-# get_session_logs
-# ===========================================================================
-
-
-class TestGetSessionLogs:
- async def test_empty_when_no_dispatches(self):
- orch = AgentOrchestrator()
- assert orch.get_session_logs() == []
-
- async def test_returns_dispatch_logs_after_execute_single(self):
- orch = AgentOrchestrator()
- agent = _make_ok_agent("analyzer", output={"result": "done"})
- orch._agents["analyzer"] = agent
-
- await orch.execute_single(agent_type="analyzer", context={"event_id": "evt1"})
- logs = orch.get_session_logs()
- assert len(logs) == 1
- assert logs[0]["agent_type"] == "analyzer"
- assert logs[0]["status"] == "ok"
- assert logs[0]["context"] == {"event_id": "evt1"}
-
- async def test_filters_by_agent_type(self):
- orch = AgentOrchestrator()
- agent_a = _make_ok_agent("type_a", output={"a": 1})
- agent_b = _make_ok_agent("type_b", output={"b": 2})
- orch._agents["type_a"] = agent_a
- orch._agents["type_b"] = agent_b
-
- await orch.execute_single(agent_type="type_a", context={})
- await orch.execute_single(agent_type="type_b", context={})
-
- logs_a = orch.get_session_logs(agent_type="type_a")
- assert len(logs_a) == 1
- assert logs_a[0]["agent_type"] == "type_a"
-
- async def test_respects_limit(self):
- orch = AgentOrchestrator()
- agent = _make_ok_agent("repeater", output={})
- orch._agents["repeater"] = agent
-
- for _ in range(5):
- await orch.execute_single(agent_type="repeater", context={})
-
- logs = orch.get_session_logs(limit=3)
- assert len(logs) == 3
-
- async def test_does_not_include_non_dispatch_a2a_messages(self):
- orch = AgentOrchestrator()
- # Add a regular A2A message
- await orch.send_a2a_message("alice", "bob", {"hello": True})
-
- logs = orch.get_session_logs()
- assert len(logs) == 0
-
- async def test_log_entry_has_expected_keys(self):
- orch = AgentOrchestrator()
- agent = _make_ok_agent("checker", output={"x": 1})
- orch._agents["checker"] = agent
-
- await orch.execute_single(agent_type="checker", context={"evt": "e1"})
- entry = orch.get_session_logs()[0]
- assert "sender" in entry
- assert "recipient" in entry
- assert "agent_type" in entry
- assert "context" in entry
- assert "status" in entry
- assert "timestamp" in entry
- assert "conversation_id" in entry
-
-
# ===========================================================================
# Global orchestrator instance
# ===========================================================================
diff --git a/tests/unit/test_api_models.py b/tests/unit/test_api_models.py
index 016afd6d3..872671069 100644
--- a/tests/unit/test_api_models.py
+++ b/tests/unit/test_api_models.py
@@ -17,7 +17,6 @@
AgentExecution,
AgentStatus,
ApiResponse,
- ChatRequest,
EventExtractRequest,
ExtractedEvent,
FeedbackRequest,
@@ -214,12 +213,6 @@ def test_invalid_url_rejected(self):
with pytest.raises(ValidationError):
VideoToSoftwareRequest(url="https://not-youtube.com/video/123")
- def test_non_string_url_is_validation_error_not_500(self):
- # pre=True validator sees the raw payload; a non-str must surface as a
- # ValidationError (422), never a bare TypeError (500).
- with pytest.raises(ValidationError, match="Invalid YouTube URL"):
- VideoToSoftwareRequest(url=123)
-
def test_features_default_to_empty_list(self):
req = VideoToSoftwareRequest(url=self._VALID_URL)
assert req.features == []
@@ -295,39 +288,6 @@ def test_with_video_options(self):
req = TranscriptActionRequest(video_url=self._VALID_URL, video_options=opts)
assert req.video_options.end_seconds == 120.0
- # --- shared _YOUTUBE_URL_REGEX behaviour (mobile/music hosts + SSRF guard) ---
-
- @pytest.mark.parametrize(
- "url",
- [
- "https://m.youtube.com/watch?v=auJzb1D-fag",
- "https://music.youtube.com/watch?v=auJzb1D-fag",
- "HTTPS://M.YOUTUBE.COM/watch?v=auJzb1D-fag",
- "https://youtu.be/auJzb1D-fag",
- "https://www.youtube.com/shorts/auJzb1D-fag",
- ],
- )
- def test_mobile_music_and_case_insensitive_hosts_accepted(self, url):
- assert TranscriptActionRequest(video_url=url).video_url == url
- assert ChatRequest(message="hi", video_url=url).video_url == url
-
- @pytest.mark.parametrize(
- "url",
- [
- "http://169.254.169.254/aaaaaaaaaaa", # SSRF: cloud metadata
- "--config-locations=/etc/passwd", # CWE-88: arg injection
- "https://vimeo.com/123456789", # non-YouTube host
- "https://not-youtube.com/watch?v=12345678901",
- "https://m.youtu.be/auJzb1D-fag", # fabricated youtu.be subdomain
- "https://music.youtu.be/auJzb1D-fag", # fabricated youtu.be subdomain
- ],
- )
- def test_non_youtube_and_injection_urls_rejected(self, url):
- with pytest.raises(ValidationError, match="Invalid YouTube URL"):
- TranscriptActionRequest(video_url=url)
- with pytest.raises(ValidationError, match="Invalid YouTube URL"):
- ChatRequest(message="hi", video_url=url)
-
# ===========================================================================
# FeedbackRequest
diff --git a/tests/unit/test_base_repository.py b/tests/unit/test_base_repository.py
index 90e16d7af..b84c3fce4 100644
--- a/tests/unit/test_base_repository.py
+++ b/tests/unit/test_base_repository.py
@@ -533,27 +533,26 @@ async def test_bulk_update_returns_count(self, repo, session):
]
result = await repo.bulk_update(updates)
assert result == 2
- assert session.execute.await_count == 1
+ assert session.execute.await_count == 2
async def test_bulk_update_skips_entry_without_id(self, repo, session):
- # We no longer manually pop 'id' or skip. SQLAlchemy's bulk update handles it.
- # But we verify it executes once with the full list.
session.execute.return_value = MagicMock()
updates = [
{"name": "no-id-entry"},
{"id": "id-1", "name": "has-id"},
]
result = await repo.bulk_update(updates)
+ # returns len(updates), but only executes for entries with id
assert result == 2
assert session.execute.await_count == 1
async def test_bulk_update_mutates_dicts(self, repo, session):
- """bulk_update used to pop 'id' from each dict. New SQLAlchemy bulk update does not."""
+ """bulk_update pops 'id' from each dict — verify that happens."""
session.execute.return_value = MagicMock()
data = [{"id": "x", "field": "v"}]
await repo.bulk_update(data)
- # 'id' should NOT be popped
- assert "id" in data[0]
+ # 'id' should have been popped
+ assert "id" not in data[0]
# ===========================================================================
diff --git a/tests/unit/test_bigquery_export.py b/tests/unit/test_bigquery_export.py
deleted file mode 100644
index c96d8037f..000000000
--- a/tests/unit/test_bigquery_export.py
+++ /dev/null
@@ -1,12 +0,0 @@
-"""Tests for BigQuery export functionality."""
-
-from unittest.mock import patch
-
-from uvai.ml.bigquery_export import _get_bq_client
-
-
-def test_get_bq_client_import_error() -> None:
- """Test that _get_bq_client returns None when google.cloud.bigquery is missing."""
- with patch.dict("sys.modules", {"google.cloud.bigquery": None}):
- client = _get_bq_client()
- assert client is None
diff --git a/tests/unit/test_cache_service.py b/tests/unit/test_cache_service.py
index 5aae4fac8..8d5cde9f6 100644
--- a/tests/unit/test_cache_service.py
+++ b/tests/unit/test_cache_service.py
@@ -65,7 +65,7 @@ def test_different_urls_different_keys(self, cache):
k2 = cache._get_cache_key("https://www.youtube.com/watch?v=bbbbbbbbbbb")
assert k1 != k2
- def test_matches_md5_prefix(self, cache):
+ def test_matches_sha256_prefix(self, cache):
expected = hashlib.sha256(_VIDEO_URL.encode()).hexdigest()[:12]
assert cache._get_cache_key(_VIDEO_URL) == expected
diff --git a/tests/unit/test_cloud_routes.py b/tests/unit/test_cloud_routes.py
index daa292c72..ecdb27400 100644
--- a/tests/unit/test_cloud_routes.py
+++ b/tests/unit/test_cloud_routes.py
@@ -1300,7 +1300,11 @@ def test_generate_dashboard_url_success(self):
assert "looker.example.com" in data["embed_url"]
def test_generate_dashboard_url_service_error(self):
- """Service error propagates as HTTP 500."""
+ """Service error propagates as a sanitized HTTP 500.
+
+ Regression guard for information disclosure: the response must carry
+ the generic message and must NOT leak the raw exception text.
+ """
mock_service = MagicMock()
mock_service.get_tenant_dashboard_url = MagicMock(
side_effect=Exception("Looker unavailable")
@@ -1316,7 +1320,8 @@ def test_generate_dashboard_url_service_error(self):
}
)
assert response.status_code == 500
- assert "Failed to generate" in response.json()["detail"]
+ assert response.json()["detail"] == "Internal server error"
+ assert "Looker unavailable" not in response.text
def test_generate_dashboard_url_missing_fields(self):
"""Missing required fields return 422."""
diff --git a/tests/unit/test_database_optimizer.py b/tests/unit/test_database_optimizer.py
index 2744cbedc..ad880e12d 100644
--- a/tests/unit/test_database_optimizer.py
+++ b/tests/unit/test_database_optimizer.py
@@ -1092,7 +1092,6 @@ async def test_health_check_error_response_has_error_key(self):
class TestConvenienceFunctions:
"""Tests for module-level convenience functions"""
- @pytest.mark.asyncio
async def test_execute_optimized_query_delegates(self, tmp_path) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.query_optimizer.execute_query
@@ -1103,7 +1102,6 @@ async def test_execute_optimized_query_delegates(self, tmp_path) -> None:
finally:
_mod.query_optimizer.execute_query = orig
- @pytest.mark.asyncio
async def test_execute_batch_delegates(self) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.query_optimizer.execute_batch_queries
@@ -1114,7 +1112,6 @@ async def test_execute_batch_delegates(self) -> None:
finally:
_mod.query_optimizer.execute_batch_queries = orig
- @pytest.mark.asyncio
async def test_get_database_performance_report_delegates(self) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.query_optimizer.get_performance_report
@@ -1125,7 +1122,6 @@ async def test_get_database_performance_report_delegates(self) -> None:
finally:
_mod.query_optimizer.get_performance_report = orig
- @pytest.mark.asyncio
async def test_get_database_health_status_delegates(self) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.health_monitor.run_health_check
@@ -1136,7 +1132,6 @@ async def test_get_database_health_status_delegates(self) -> None:
finally:
_mod.health_monitor.run_health_check = orig
- @pytest.mark.asyncio
async def test_initialize_database_optimization_calls_initialize(self, tmp_path) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig_init = _mod.connection_pool.initialize
@@ -1156,7 +1151,6 @@ async def test_initialize_database_optimization_calls_initialize(self, tmp_path)
_mod.connection_pool.get_connection = orig_get
_mod.connection_pool.release_connection = orig_rel
- @pytest.mark.asyncio
async def test_shutdown_database_optimization_calls_close(self) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.connection_pool.close
@@ -1178,7 +1172,6 @@ def _make_pool(self) -> MagicMock:
pool.release_connection = AsyncMock()
return pool
- @pytest.mark.asyncio
async def test_batch_executes_all_queries(self, tmp_path) -> None:
import sqlite3
pool = DatabaseConnectionPool(f"sqlite:///{tmp_path}/test.db")
@@ -1191,7 +1184,6 @@ async def test_batch_executes_all_queries(self, tmp_path) -> None:
results = await optimizer.execute_batch_queries(queries)
assert len(results) == 2
- @pytest.mark.asyncio
async def test_batch_exception_propagated(self) -> None:
pool = self._make_pool()
pool.get_connection.side_effect = RuntimeError("No DB")
@@ -1203,19 +1195,16 @@ async def test_batch_exception_propagated(self) -> None:
class TestConnectionPoolInitialize:
"""DatabaseConnectionPool.initialize with different URL types"""
- @pytest.mark.asyncio
async def test_sqlite_file_creates_dir(self, tmp_path) -> None:
db_path = tmp_path / "subdir" / "test.db"
pool = DatabaseConnectionPool(f"sqlite:///{db_path}")
await pool.initialize()
# No error should occur
- @pytest.mark.asyncio
async def test_sqlite_memory_initializes(self) -> None:
pool = DatabaseConnectionPool("sqlite:///:memory:")
await pool.initialize()
- @pytest.mark.asyncio
async def test_haspg_false_uses_sqlite_path(self, tmp_path) -> None:
from youtube_extension.backend.services import database_optimizer as _mod
orig = _mod.HAS_POSTGRESQL
diff --git a/tests/unit/test_dependabot_automation_workflow.py b/tests/unit/test_dependabot_automation_workflow.py
index 8fe0264db..397ff83a2 100644
--- a/tests/unit/test_dependabot_automation_workflow.py
+++ b/tests/unit/test_dependabot_automation_workflow.py
@@ -9,7 +9,6 @@
Path(__file__).resolve().parents[2]
/ ".github/workflows/dependabot-auto-merge.yml"
)
-DEPENDABOT_CONFIG_PATH = Path(__file__).resolve().parents[2] / ".github/dependabot.yml"
def _load_workflow() -> dict:
@@ -17,11 +16,6 @@ def _load_workflow() -> dict:
return yaml.safe_load(WORKFLOW_PATH.read_text())
-def _load_dependabot_config() -> dict:
- assert DEPENDABOT_CONFIG_PATH.exists(), "Dependabot config should exist"
- return yaml.safe_load(DEPENDABOT_CONFIG_PATH.read_text())
-
-
def test_dependabot_workflow_uses_safe_triggers_and_permissions() -> None:
workflow = _load_workflow()
@@ -54,16 +48,10 @@ def test_dependabot_workflow_approves_and_merges_without_checkout() -> None:
approve_steps = approve_job["steps"]
merge_steps = merge_job["steps"]
- assert all(
- not step.get("uses", "").startswith("actions/checkout@")
- for step in approve_steps
- )
- assert all(
- not step.get("uses", "").startswith("actions/checkout@")
- for step in merge_steps
- )
+ assert all(step.get("uses") != "actions/checkout@v4" for step in approve_steps)
+ assert all(step.get("uses") != "actions/checkout@v4" for step in merge_steps)
assert any(
- step.get("uses", "").startswith("dependabot/fetch-metadata@")
+ step.get("uses") == "dependabot/fetch-metadata@v2"
and step.get("id") == "metadata"
for step in approve_steps
)
@@ -84,24 +72,3 @@ def test_dependabot_workflow_approves_and_merges_without_checkout() -> None:
and "semver-major" in step.get("with", {}).get("script", "")
for step in merge_steps
)
-
-
-def test_dependabot_ignores_eslint_v10() -> None:
- config = _load_dependabot_config()
- npm_updates = [
- update
- for update in config["updates"]
- if update["package-ecosystem"] == "npm"
- and update["directory"] in {"/", "/apps/web"}
- ]
-
- assert len(npm_updates) >= 2, (
- f"Expected npm update entries for both '/' and '/apps/web', got {len(npm_updates)}: "
- f"{[u['directory'] for u in npm_updates]}"
- )
-
- for update in npm_updates:
- assert {
- "dependency-name": "eslint",
- "versions": [">=10"],
- } in update.get("ignore", [])
diff --git a/tests/unit/test_intelligent_cache.py b/tests/unit/test_intelligent_cache.py
index 829a5e83f..9a38a112e 100644
--- a/tests/unit/test_intelligent_cache.py
+++ b/tests/unit/test_intelligent_cache.py
@@ -714,7 +714,7 @@ async def test_cache_key_kwargs_sorted(self):
async def test_cache_key_returns_hex_string(self):
from youtube_extension.backend.services.intelligent_cache import cache_key
k = cache_key("test")
- assert len(k) == 64 # sha256 hex digest (migrated from md5's 32)
+ assert len(k) == 64
int(k, 16) # should not raise
diff --git a/tests/unit/test_metrics_service.py b/tests/unit/test_metrics_service.py
index 1da7b4c1f..cdd165746 100644
--- a/tests/unit/test_metrics_service.py
+++ b/tests/unit/test_metrics_service.py
@@ -3,7 +3,9 @@
from __future__ import annotations
import sys
+import time
import types
+from collections import deque
from datetime import datetime, timedelta
from pathlib import Path
from unittest.mock import AsyncMock, MagicMock, patch
@@ -216,19 +218,6 @@ def test_custom_collection_interval(self, tmp_path, monkeypatch):
svc = MetricsService(config={"collection_interval": 30})
assert svc.collection_interval == 30
-
-class TestMetricsServicePersistMetrics:
- @pytest.mark.asyncio
- async def test_persist_metrics_writes_metrics_file(self, tmp_path, monkeypatch):
- monkeypatch.chdir(tmp_path)
- svc = MetricsService()
- await svc.record_metric("audit.active_sample", 1.0)
-
- await svc.persist_metrics()
-
- assert svc.metrics_file.exists()
- assert "audit.active_sample" in svc.metrics_file.read_text()
-
def test_custom_retention_period(self, tmp_path, monkeypatch):
monkeypatch.chdir(tmp_path)
svc = MetricsService(config={"retention_period": 7200})
diff --git a/tests/unit/test_ml_serve.py b/tests/unit/test_ml_serve.py
deleted file mode 100644
index 017b0a1bb..000000000
--- a/tests/unit/test_ml_serve.py
+++ /dev/null
@@ -1,206 +0,0 @@
-import pytest
-from unittest.mock import MagicMock, AsyncMock, patch
-import json
-import sys
-import os
-from pathlib import Path
-
-# Add src to sys.path
-sys.path.insert(0, str(Path(__file__).resolve().parents[2] / "src"))
-
-# We mock 'ray' because ray serve might not be importable or has side effects
-def mock_deployment(*args, **kwargs):
- def decorator(cls):
- # Ray's serve.deployment binds to the class, giving it a 'bind' method
- cls.bind = MagicMock()
- return cls
- return decorator
-
-mock_ray = MagicMock()
-mock_serve = MagicMock()
-mock_serve.deployment = mock_deployment
-mock_ray.serve = mock_serve
-sys.modules['ray'] = mock_ray
-sys.modules['ray.serve'] = mock_serve
-
-from uvai.ml.serve import UVAIMLRouter
-from starlette.requests import Request
-
-class MockRequest:
- def __init__(self, method, path, json_data=None, json_exception=None):
- self.method = method
- class URL:
- def __init__(self, p):
- self.path = p
- def rstrip(self, s):
- return self.path.rstrip(s) if self.path.endswith(s) else self.path
- self.url = URL(path)
- if json_exception:
- self.json = AsyncMock(side_effect=json_exception)
- elif json_data is not None:
- self.json = AsyncMock(return_value=json_data)
- else:
- self.json = AsyncMock(return_value={})
-
-@pytest.fixture
-def mock_scorer():
- scorer = MagicMock()
- scorer.get_model_info.remote = AsyncMock(return_value={"name": "scorer", "version": "1.0"})
- scorer.predict.remote = AsyncMock(return_value={"quality_score": 0.9, "recommended_source": "speech_v2"})
- scorer.record_outcome.remote = AsyncMock(return_value={"recorded": True})
- scorer.get_serialized_state.remote = AsyncMock(return_value={"training_samples": 10})
- return scorer
-
-@pytest.fixture
-def mock_ranker():
- ranker = MagicMock()
- ranker.get_model_info.remote = AsyncMock(return_value={"name": "ranker", "version": "1.0"})
- ranker.rank.remote = AsyncMock(return_value={"ranked_actions": []})
- ranker.record_feedback.remote = AsyncMock(return_value={"recorded": True})
- ranker.get_serialized_state.remote = AsyncMock(return_value={"training_samples": 20})
- return ranker
-
-@pytest.fixture
-def router(mock_scorer, mock_ranker):
- return UVAIMLRouter(scorer=mock_scorer, ranker=mock_ranker)
-
-
-@pytest.mark.asyncio
-async def test_health_endpoint(router):
- request = MockRequest("GET", "/health")
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["status"] == "healthy"
- assert "uptime_seconds" in body
-
-@pytest.mark.asyncio
-async def test_health_endpoint_empty_path(router):
- request = MockRequest("GET", "/")
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["status"] == "healthy"
-
-@pytest.mark.asyncio
-async def test_models_metadata(router):
- request = MockRequest("GET", "/models")
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert "transcript_quality_scorer" in body["models"]
- assert "action_priority_ranker" in body["models"]
- assert body["models"]["transcript_quality_scorer"]["name"] == "scorer"
-
-@pytest.mark.asyncio
-async def test_invalid_json_body(router):
- request = MockRequest("POST", "/score-transcript", json_exception=ValueError("Invalid JSON"))
- response = await router(request)
- assert response.status_code == 400
- body = json.loads(response.body)
- assert body["error"] == "Invalid JSON body"
-
-@pytest.mark.asyncio
-async def test_score_transcript_missing_metadata(router):
- request = MockRequest("POST", "/score-transcript", json_data={})
- response = await router(request)
- assert response.status_code == 400
- body = json.loads(response.body)
- assert body["error"] == "Missing 'metadata' dict"
-
-@pytest.mark.asyncio
-async def test_score_transcript_success(router, mock_scorer):
- request = MockRequest("POST", "/score-transcript", json_data={"metadata": {"foo": "bar"}})
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["quality_score"] == 0.9
- mock_scorer.predict.remote.assert_called_once_with({"foo": "bar"})
-
-@pytest.mark.asyncio
-async def test_score_transcript_outcome(router, mock_scorer):
- json_data = {"metadata": {"foo": "bar"}, "success": True}
- request = MockRequest("POST", "/score-transcript/outcome", json_data=json_data)
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["recorded"] is True
- mock_scorer.record_outcome.remote.assert_called_once_with(json_data)
-
-@pytest.mark.asyncio
-async def test_rank_actions_missing_actions(router):
- request = MockRequest("POST", "/rank-actions", json_data={})
- response = await router(request)
- assert response.status_code == 400
- body = json.loads(response.body)
- assert body["error"] == "Missing 'actions' list"
-
-@pytest.mark.asyncio
-async def test_rank_actions_success(router, mock_ranker):
- request = MockRequest("POST", "/rank-actions", json_data={"actions": ["action1", "action2"], "video_context": {"cat": "edu"}})
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["ranked_actions"] == []
- mock_ranker.rank.remote.assert_called_once_with(["action1", "action2"], {"cat": "edu"})
-
-@pytest.mark.asyncio
-async def test_rank_actions_feedback(router, mock_ranker):
- json_data = {"action_text": "do it", "clicked": True}
- request = MockRequest("POST", "/rank-actions/feedback", json_data=json_data)
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["recorded"] is True
- mock_ranker.record_feedback.remote.assert_called_once_with(json_data)
-
-@pytest.mark.asyncio
-@patch("uvai.ml.serve.save_checkpoint")
-async def test_checkpoint_post_success(mock_save, router, mock_scorer, mock_ranker):
- with patch("uvai.ml.bigquery_export.export_model_checkpoint", create=True) as mock_export:
- request = MockRequest("POST", "/checkpoint")
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert body["saved"] is True
- assert body["scorer_samples"] == 10
- assert body["ranker_samples"] == 20
-
- mock_save.assert_called_once_with({"training_samples": 10}, {"training_samples": 20})
- mock_export.assert_called_once_with({"training_samples": 10}, {"training_samples": 20})
-
-@pytest.mark.asyncio
-@patch("uvai.ml.serve.save_checkpoint", side_effect=Exception("Save failed"))
-async def test_checkpoint_post_failure(mock_save, router):
- request = MockRequest("POST", "/checkpoint")
- response = await router(request)
- assert response.status_code == 500
- body = json.loads(response.body)
- assert body["error"] == "Save failed"
-
-@pytest.mark.asyncio
-@patch("uvai.ml.serve.load_checkpoint", return_value={"scorer_state": {}, "ranker_state": {}})
-async def test_checkpoint_get_success(mock_load, router):
- request = MockRequest("GET", "/checkpoint")
- response = await router(request)
- assert response.status_code == 200
- body = json.loads(response.body)
- assert "scorer_state" in body
-
-@pytest.mark.asyncio
-@patch("uvai.ml.serve.load_checkpoint", return_value=None)
-async def test_checkpoint_get_not_found(mock_load, router):
- request = MockRequest("GET", "/checkpoint")
- response = await router(request)
- assert response.status_code == 404
- body = json.loads(response.body)
- assert body["error"] == "No checkpoint found"
-
-@pytest.mark.asyncio
-async def test_unknown_path(router):
- request = MockRequest("GET", "/unknown-path")
- response = await router(request)
- assert response.status_code == 404
- body = json.loads(response.body)
- assert body["error"] == "Unknown path: /unknown-path"
- assert "/health" in body["available_endpoints"]
diff --git a/tests/unit/test_nightly_audit_agent.py b/tests/unit/test_nightly_audit_agent.py
deleted file mode 100644
index 5ef3c61a0..000000000
--- a/tests/unit/test_nightly_audit_agent.py
+++ /dev/null
@@ -1,103 +0,0 @@
-from __future__ import annotations
-
-import importlib.util
-from datetime import datetime, timedelta, timezone
-from pathlib import Path
-
-import pytest
-
-
-def _load_audit_module():
- repo_root = Path(__file__).resolve().parents[2]
- module_path = repo_root / "scripts" / "nightly_audit_agent.py"
- spec = importlib.util.spec_from_file_location("nightly_audit_agent", module_path)
- module = importlib.util.module_from_spec(spec)
- assert spec.loader is not None
- spec.loader.exec_module(module)
- return module
-
-
-@pytest.mark.asyncio
-async def test_scan_logs_uses_extended_72_hour_timeframe(tmp_path, monkeypatch):
- module = _load_audit_module()
- monkeypatch.chdir(tmp_path)
- log_dir = tmp_path / "logs"
- log_dir.mkdir()
- timestamp = (datetime.now(timezone.utc) - timedelta(hours=48)).isoformat()
- (log_dir / "structured_logs.jsonl").write_text(
- f'{{"timestamp": "{timestamp}", "status_code": 503, "message": "stale outage"}}\n'
- )
-
- agent = module.AuditAgent(dry_run=True)
- agent.health_service = None
- await agent._scan_logs()
-
- assert agent.lookback_hours == 72
- assert any("stale outage" in issue["description"] for issue in agent.issues)
-
-
-class _RecordingMetricsService:
- def __init__(self):
- self.started = False
- self.stopped = False
- self.persisted = False
- self.samples = 0
-
- async def start_collection(self):
- self.started = True
-
- async def stop_collection(self):
- self.stopped = True
-
- async def get_system_metrics(self):
- self.samples += 1
- return {"timestamp": datetime.now(timezone.utc).isoformat()}
-
- async def persist_metrics(self):
- self.persisted = True
-
-
-@pytest.mark.asyncio
-async def test_run_audit_collects_active_measurements_before_analysis(tmp_path, monkeypatch):
- module = _load_audit_module()
- monkeypatch.chdir(tmp_path)
-
- metrics_service = _RecordingMetricsService()
- agent = module.AuditAgent(
- dry_run=True,
- active_measurement=True,
- measurement_samples=2,
- measurement_interval=0,
- )
- agent.health_service = None
- agent.metrics_service = metrics_service
-
- await agent.run_audit()
-
- assert metrics_service.started is True
- assert metrics_service.samples == 2
- assert metrics_service.persisted is True
- assert metrics_service.stopped is True
-
-
-@pytest.mark.asyncio
-async def test_active_measurement_uses_fallback_when_metrics_service_unavailable(
- tmp_path, monkeypatch
-):
- module = _load_audit_module()
- monkeypatch.chdir(tmp_path)
-
- agent = module.AuditAgent(
- dry_run=True,
- active_measurement=True,
- measurement_samples=2,
- measurement_interval=0,
- )
- agent.metrics_service = None
-
- await agent._collect_active_measurements()
-
- metrics_file = tmp_path / "logs" / "active_measurements.jsonl"
- lines = metrics_file.read_text().strip().splitlines()
- assert len(lines) == 2
- assert any("ACTIVE MEASUREMENT" in line for line in agent.report)
diff --git a/tests/unit/test_orchestrator_consumer.py b/tests/unit/test_orchestrator_consumer.py
deleted file mode 100644
index 2cf2575e9..000000000
--- a/tests/unit/test_orchestrator_consumer.py
+++ /dev/null
@@ -1,82 +0,0 @@
-"""Unit tests for youtube_extension/orchestrator/main.py.
-
-Covers the hardened Redis Streams consumer-group bootstrap (the paths this PR is
-meant to harden) plus the credential-redaction and stub-handler contracts. The
-Redis client is mocked, so these run without a live Redis or the redis-py package.
-"""
-
-from __future__ import annotations
-
-from unittest.mock import AsyncMock
-
-import pytest
-
-from youtube_extension.orchestrator.main import (
- ensure_consumer_group,
- process,
- redact_url,
-)
-
-# ---------------------------------------------------------------------------
-# ensure_consumer_group — the core of the hardening fix
-# ---------------------------------------------------------------------------
-
-@pytest.mark.asyncio
-async def test_ensure_consumer_group_creates_when_absent() -> None:
- client = AsyncMock()
- await ensure_consumer_group(client, "stream", "group")
- client.xgroup_create.assert_awaited_once_with(
- "stream", "group", id="0", mkstream=True
- )
-
-
-@pytest.mark.asyncio
-async def test_ensure_consumer_group_tolerates_busygroup() -> None:
- client = AsyncMock()
- client.xgroup_create.side_effect = Exception(
- "BUSYGROUP Consumer Group name already exists"
- )
- # Must NOT raise: an existing group is the expected idempotent case.
- await ensure_consumer_group(client, "stream", "group")
-
-
-@pytest.mark.asyncio
-async def test_ensure_consumer_group_reraises_transient_errors() -> None:
- client = AsyncMock()
- client.xgroup_create.side_effect = Exception(
- "Error 111 connecting to localhost:6379. Connection refused."
- )
- # A transient ConnectionError must propagate so the caller retries instead of
- # silently proceeding without a group (which would stall on NOGROUP forever).
- with pytest.raises(Exception, match="Connection refused"):
- await ensure_consumer_group(client, "stream", "group")
-
-
-# ---------------------------------------------------------------------------
-# redact_url — credentials must never reach logs
-# ---------------------------------------------------------------------------
-
-def test_redact_url_strips_credentials() -> None:
- redacted = redact_url("redis://admin:supersecret@redis.internal:6379/1")
- assert "supersecret" not in redacted
- assert "redis.internal" in redacted
-
-
-def test_redact_url_passthrough_without_credentials() -> None:
- assert redact_url("redis://localhost:6379") == "redis://localhost:6379"
-
-
-def test_redact_url_never_raises_on_garbage() -> None:
- # Malformed input must degrade to a safe placeholder, never throw.
- assert redact_url("::not a url::") is not None
-
-
-# ---------------------------------------------------------------------------
-# process — REAL_MODE_ONLY: no silent fake success
-# ---------------------------------------------------------------------------
-
-@pytest.mark.asyncio
-async def test_process_fails_loudly_until_implemented() -> None:
- # The stub must raise so the consumer never xack's unprocessed work.
- with pytest.raises(NotImplementedError):
- await process({"field": "value"})
diff --git a/tests/unit/test_processors_strategies.py b/tests/unit/test_processors_strategies.py
index 978b0c5b0..86889fc75 100644
--- a/tests/unit/test_processors_strategies.py
+++ b/tests/unit/test_processors_strategies.py
@@ -270,7 +270,7 @@ async def test_cache_hit_increments_counter(self):
_cache[cache_key] = {"cached": True}
try:
result = await opt.process_video(_VALID_URL)
- assert result == {"cached": True}
+ assert result.get("cached") is True
assert opt.processing_stats["cache_hits"] == 1
finally:
del _cache[cache_key]
diff --git a/tests/unit/test_service_container.py b/tests/unit/test_service_container.py
index b91365413..6f1068026 100644
--- a/tests/unit/test_service_container.py
+++ b/tests/unit/test_service_container.py
@@ -36,7 +36,7 @@ def _bare_container() -> ServiceContainer:
class TestLoadConfiguration:
def test_defaults_set_without_env(self, monkeypatch):
for key in [
- "CACHE_DIR", "ENHANCED_ANALYSIS_DIR", "FEEDBACK_DIR", "KNOWLEDGE_DIR",
+ "CACHE_DIR", "ENHANCED_ANALYSIS_DIR", "FEEDBACK_DIR",
"RATE_LIMIT_RPS", "MAX_RECENT_REQUESTS", "VIDEO_PROCESSOR_TYPE",
"USE_LANGEXTRACT_FALLBACK", "LIVEKIT_URL", "MOZILLA_AI_URL",
"GEMINI_API_KEY", "GOOGLE_API_KEY", "YOUTUBE_API_KEY",
@@ -49,7 +49,6 @@ def test_defaults_set_without_env(self, monkeypatch):
assert sc._config["cache_dir"] == "/tmp/uvai_cache/markdown_analysis"
assert sc._config["enhanced_analysis_dir"] == "/tmp/uvai_cache/enhanced_analysis"
assert sc._config["feedback_dir"] == "/tmp/uvai_cache/feedback"
- assert sc._config["knowledge_dir"] == "/tmp/uvai_cache/knowledge"
assert sc._config["rate_limit_rps"] == 5
assert sc._config["max_recent_requests"] == 1000
assert sc._config["video_processor_type"] == "auto"
diff --git a/tests/unit/test_skills_event_validator.py b/tests/unit/test_skills_event_validator.py
deleted file mode 100644
index 78a82d5dd..000000000
--- a/tests/unit/test_skills_event_validator.py
+++ /dev/null
@@ -1,473 +0,0 @@
-"""Unit tests for agents.skills_event_validator.
-
-Tests cover:
-- SkillsEventValidator.validate() happy and error paths
-- SkillsEventValidator.validate_file() with temp JSON files
-- SkillsEventValidator.get_disconnected_triggers() return shape
-- validate_skills_lock() convenience wrapper
-- Backward-compat: skills without subscribed_triggers pass unchanged
-"""
-
-from __future__ import annotations
-
-import importlib
-import json
-import sys
-import types
-from pathlib import Path
-from typing import Any
-
-import pytest
-
-# -------------------------------------------------------------------------
-# Ensure src is on the import path and stub heavy agents/__init__.py to
-# avoid transitive aiohttp / websockets dependencies at collection time.
-# -------------------------------------------------------------------------
-_SRC = Path(__file__).resolve().parents[2] / "src"
-if str(_SRC) not in sys.path:
- sys.path.insert(0, str(_SRC))
-
-# Stub the agents package so its __init__.py (which imports aiohttp etc.)
-# is never executed. The same technique is used in test_skill_builder.py
-# for the youtube_extension.services package.
-if "agents" not in sys.modules:
- _agents_stub = types.ModuleType("agents")
- _agents_stub.__path__ = [str(_SRC / "agents")]
- _agents_stub.__package__ = "agents"
- sys.modules["agents"] = _agents_stub
-
-# Force fresh import of the specific submodule we want to test.
-sys.modules.pop("agents.skills_event_validator", None)
-
-from agents.skills_event_validator import ( # noqa: E402
- SkillEventContractError,
- SkillsEventValidator,
- validate_skills_lock,
-)
-
-# -------------------------------------------------------------------------
-# Fixtures / helpers
-# -------------------------------------------------------------------------
-
-_KNOWN_EVENTS = [
- "pipeline.event",
- "com.eventrelay.transcript.received",
- "com.eventrelay.transcript.completed",
-]
-
-
-def _lock(
- emitted: list[str] | None = None,
- skills: dict[str, Any] | None = None,
-) -> dict[str, Any]:
- """Build a minimal skills-lock dict for testing."""
- return {
- "version": 2,
- "emitted_events": emitted if emitted is not None else list(_KNOWN_EVENTS),
- "skills": skills or {},
- }
-
-
-def _skill(triggers: list[str]) -> dict[str, Any]:
- """Return a minimal skill entry with the given subscribed_triggers."""
- return {
- "source": "test/repo",
- "sourceType": "github",
- "skillPath": "skills/test/SKILL.md",
- "computedHash": "abc123",
- "subscribed_triggers": triggers,
- }
-
-
-# =========================================================================
-# SkillEventContractError
-# =========================================================================
-
-
-class TestSkillEventContractError:
- def test_is_exception_subclass(self):
- assert issubclass(SkillEventContractError, Exception)
-
- def test_message_preserved(self):
- err = SkillEventContractError("broken contract")
- assert "broken contract" in str(err)
-
-
-# =========================================================================
-# SkillsEventValidator.get_disconnected_triggers
-# =========================================================================
-
-
-class TestGetDisconnectedTriggers:
- def setup_method(self):
- self.validator = SkillsEventValidator()
-
- def test_empty_skills_returns_empty_list(self):
- assert self.validator.get_disconnected_triggers(_lock()) == []
-
- def test_skill_with_no_triggers_returns_empty(self):
- lock = _lock(skills={"my-skill": _skill([])})
- assert self.validator.get_disconnected_triggers(lock) == []
-
- def test_skill_without_triggers_key_returns_empty(self):
- lock = _lock(
- skills={
- "my-skill": {
- "source": "a/b",
- "sourceType": "github",
- "skillPath": "x.md",
- "computedHash": "ff",
- }
- }
- )
- assert self.validator.get_disconnected_triggers(lock) == []
-
- def test_trigger_in_emitted_events_no_disconnect(self):
- lock = _lock(skills={"my-skill": _skill(["pipeline.event"])})
- assert self.validator.get_disconnected_triggers(lock) == []
-
- def test_trigger_not_in_emitted_events_returns_disconnect(self):
- lock = _lock(skills={"my-skill": _skill(["content_generated"])})
- result = self.validator.get_disconnected_triggers(lock)
- assert len(result) == 1
- assert result[0]["skill"] == "my-skill"
- assert result[0]["trigger"] == "content_generated"
-
- def test_multiple_disconnected_triggers_all_returned(self):
- lock = _lock(
- skills={
- "skill-a": _skill(["content_generated", "lead_scored"]),
- }
- )
- result = self.validator.get_disconnected_triggers(lock)
- triggers = {r["trigger"] for r in result}
- assert "content_generated" in triggers
- assert "lead_scored" in triggers
- assert all(r["skill"] == "skill-a" for r in result)
-
- def test_multiple_skills_each_violation_reported(self):
- lock = _lock(
- skills={
- "skill-a": _skill(["unknown.event.a"]),
- "skill-b": _skill(["unknown.event.b"]),
- }
- )
- result = self.validator.get_disconnected_triggers(lock)
- assert len(result) == 2
- skill_names = {r["skill"] for r in result}
- assert "skill-a" in skill_names
- assert "skill-b" in skill_names
-
- def test_partial_match_only_unknown_triggers_reported(self):
- lock = _lock(
- skills={
- "skill-mixed": _skill(["pipeline.event", "unknown.event"]),
- }
- )
- result = self.validator.get_disconnected_triggers(lock)
- assert len(result) == 1
- assert result[0]["trigger"] == "unknown.event"
-
- def test_missing_emitted_events_key_treats_as_empty_set(self):
- lock = {"version": 2, "skills": {"skill-x": _skill(["pipeline.event"])}}
- result = self.validator.get_disconnected_triggers(lock)
- assert len(result) == 1
- assert result[0]["trigger"] == "pipeline.event"
-
- def test_empty_emitted_events_list_triggers_all_fail(self):
- lock = _lock(emitted=[], skills={"s": _skill(["any.event"])})
- result = self.validator.get_disconnected_triggers(lock)
- assert len(result) == 1
-
- def test_result_items_have_skill_and_trigger_keys(self):
- lock = _lock(skills={"s": _skill(["gone.event"])})
- result = self.validator.get_disconnected_triggers(lock)
- assert "skill" in result[0]
- assert "trigger" in result[0]
-
- def test_non_dict_skill_entry_is_skipped(self):
- lock = _lock(skills={"broken": "not-a-dict"})
- assert self.validator.get_disconnected_triggers(lock) == []
-
-
-# =========================================================================
-# SkillsEventValidator.validate
-# =========================================================================
-
-
-class TestValidate:
- def setup_method(self):
- self.validator = SkillsEventValidator()
-
- def test_valid_lock_does_not_raise(self):
- lock = _lock(skills={"s": _skill(["pipeline.event"])})
- self.validator.validate(lock) # must not raise
-
- def test_no_skills_does_not_raise(self):
- self.validator.validate(_lock()) # must not raise
-
- def test_invalid_trigger_raises_skill_event_contract_error(self):
- lock = _lock(skills={"s": _skill(["content_generated"])})
- with pytest.raises(SkillEventContractError):
- self.validator.validate(lock)
-
- def test_error_message_contains_skill_name(self):
- lock = _lock(skills={"my-bad-skill": _skill(["ghost.event"])})
- with pytest.raises(SkillEventContractError, match="my-bad-skill"):
- self.validator.validate(lock)
-
- def test_error_message_contains_trigger_name(self):
- lock = _lock(skills={"s": _skill(["ghost.event"])})
- with pytest.raises(SkillEventContractError, match="ghost.event"):
- self.validator.validate(lock)
-
- def test_error_message_contains_violation_count(self):
- lock = _lock(
- skills={
- "a": _skill(["event.x"]),
- "b": _skill(["event.y"]),
- }
- )
- with pytest.raises(SkillEventContractError, match="2 event contract violation"):
- self.validator.validate(lock)
-
- def test_mixed_valid_invalid_triggers_raises(self):
- # skill-a is fine, skill-b is broken
- lock = _lock(
- skills={
- "skill-a": _skill(["pipeline.event"]),
- "skill-b": _skill(["lead_scored"]),
- }
- )
- with pytest.raises(SkillEventContractError):
- self.validator.validate(lock)
-
- def test_all_skills_with_empty_triggers_passes(self):
- lock = _lock(
- skills={
- "a": _skill([]),
- "b": _skill([]),
- "c": _skill([]),
- }
- )
- self.validator.validate(lock) # must not raise
-
- def test_backward_compat_v1_format_without_triggers(self):
- """v1 skills-lock without subscribed_triggers or emitted_events should pass."""
- lock_v1 = {
- "version": 1,
- "skills": {
- "firebase-basics": {
- "source": "firebase/agent-skills",
- "sourceType": "github",
- "skillPath": "skills/firebase-basics/SKILL.md",
- "computedHash": "88fb9ee785fa7aaa74b2c662e53b2aca0b9ee4b67c84587ee017460f54b97471",
- }
- },
- }
- self.validator.validate(lock_v1) # must not raise (backward compat)
-
-
-# =========================================================================
-# SkillsEventValidator.validate_file
-# =========================================================================
-
-
-class TestValidateFile:
- def setup_method(self):
- self.validator = SkillsEventValidator()
-
- def test_valid_file_does_not_raise(self, tmp_path):
- lock_file = tmp_path / "skills-lock.json"
- lock_file.write_text(json.dumps(_lock(skills={"s": _skill(["pipeline.event"])})))
- self.validator.validate_file(lock_file) # must not raise
-
- def test_invalid_file_raises_skill_event_contract_error(self, tmp_path):
- lock_file = tmp_path / "skills-lock.json"
- lock_file.write_text(json.dumps(_lock(skills={"s": _skill(["ghost.event"])})))
- with pytest.raises(SkillEventContractError):
- self.validator.validate_file(lock_file)
-
- def test_missing_file_raises_file_not_found(self, tmp_path):
- with pytest.raises(FileNotFoundError):
- self.validator.validate_file(tmp_path / "nonexistent.json")
-
- def test_invalid_json_raises_json_decode_error(self, tmp_path):
- bad = tmp_path / "bad.json"
- bad.write_text("{not valid json")
- with pytest.raises(json.JSONDecodeError):
- self.validator.validate_file(bad)
-
- def test_accepts_string_path(self, tmp_path):
- lock_file = tmp_path / "skills-lock.json"
- lock_file.write_text(json.dumps(_lock()))
- self.validator.validate_file(str(lock_file)) # must not raise
-
-
-# =========================================================================
-# validate_skills_lock convenience wrapper
-# =========================================================================
-
-
-class TestValidateSkillsLock:
- def test_valid_custom_path_does_not_raise(self, tmp_path):
- lock_file = tmp_path / "skills-lock.json"
- lock_file.write_text(json.dumps(_lock()))
- validate_skills_lock(lock_file) # must not raise
-
- def test_custom_path_with_disconnect_raises(self, tmp_path):
- lock_file = tmp_path / "skills-lock.json"
- lock_file.write_text(json.dumps(_lock(skills={"x": _skill(["bad.event"])})))
- with pytest.raises(SkillEventContractError):
- validate_skills_lock(lock_file)
-
-
-# =========================================================================
-# Integration: validate the real skills-lock.json in the repo
-# =========================================================================
-
-# Resolve the repo-root skills-lock.json relative to this test file.
-_REPO_ROOT = Path(__file__).resolve().parents[2]
-_SKILLS_LOCK_PATH = _REPO_ROOT / "skills-lock.json"
-
-
-class TestRealSkillsLock:
- """Validates that the committed skills-lock.json satisfies the event contract.
-
- This acts as a living regression guard: if a developer adds a skill with
- a ``subscribed_triggers`` entry that has no corresponding ``emitted_events``
- entry, this test will fail loudly instead of silently.
- """
-
- def test_skills_lock_file_exists(self):
- assert _SKILLS_LOCK_PATH.exists(), (
- f"skills-lock.json not found at {_SKILLS_LOCK_PATH}; "
- "did the file get moved or deleted?"
- )
-
- def test_skills_lock_is_valid_json(self):
- content = _SKILLS_LOCK_PATH.read_text(encoding="utf-8")
- lock = json.loads(content)
- assert isinstance(lock, dict)
-
- def test_skills_lock_has_emitted_events_key(self):
- lock = json.loads(_SKILLS_LOCK_PATH.read_text(encoding="utf-8"))
- assert "emitted_events" in lock, (
- "skills-lock.json is missing the 'emitted_events' manifest. "
- "Add an emitted_events list so trigger→emitter contracts can be validated."
- )
-
- def test_emitted_events_is_non_empty_list(self):
- lock = json.loads(_SKILLS_LOCK_PATH.read_text(encoding="utf-8"))
- assert isinstance(lock.get("emitted_events"), list)
- assert len(lock["emitted_events"]) > 0
-
- def test_no_event_contract_violations(self):
- """Main contract gate: all subscribed_triggers must have registered emitters."""
- validate_skills_lock(_SKILLS_LOCK_PATH) # raises SkillEventContractError on violation
-
- def test_all_skill_entries_have_subscribed_triggers_key(self):
- lock = json.loads(_SKILLS_LOCK_PATH.read_text(encoding="utf-8"))
- missing = [
- name
- for name, data in lock.get("skills", {}).items()
- if isinstance(data, dict) and "subscribed_triggers" not in data
- ]
- assert not missing, (
- f"The following skills are missing 'subscribed_triggers' (add [] if none): "
- f"{missing}"
- )
-
-
-# =========================================================================
-# Integration: emit → skill trigger flow simulation
-# =========================================================================
-
-
-class TestEmitToSkillTriggerFlow:
- """Verifies the full emit → skill trigger resolution path.
-
- Uses a simple in-process event bus to ensure that when an event is
- emitted that matches a skill's subscribed_triggers, the skill is
- correctly identified as triggered.
- """
-
- def _build_trigger_index(self, skills_lock: dict) -> dict[str, list[str]]:
- """Build {event_type: [skill_name, ...]} index from skills-lock."""
- index: dict[str, list[str]] = {}
- for skill_name, skill_data in skills_lock.get("skills", {}).items():
- if not isinstance(skill_data, dict):
- continue
- for trigger in skill_data.get("subscribed_triggers", []):
- index.setdefault(trigger, []).append(skill_name)
- return index
-
- def test_emitting_known_event_triggers_subscribed_skills(self):
- lock = _lock(
- emitted=["pipeline.event", "com.eventrelay.transcript.completed"],
- skills={
- "skill-pipeline": _skill(["pipeline.event"]),
- "skill-transcript": _skill(["com.eventrelay.transcript.completed"]),
- "skill-nothing": _skill([]),
- },
- )
- index = self._build_trigger_index(lock)
-
- # Simulate: orchestrator emits pipeline.event
- triggered = index.get("pipeline.event", [])
- assert "skill-pipeline" in triggered
- assert "skill-transcript" not in triggered
- assert "skill-nothing" not in triggered
-
- def test_emitting_event_not_subscribed_by_anyone_triggers_nothing(self):
- lock = _lock(
- emitted=["pipeline.event"],
- skills={
- "skill-a": _skill(["pipeline.event"]),
- },
- )
- index = self._build_trigger_index(lock)
- triggered = index.get("com.eventrelay.transcript.completed", [])
- assert triggered == []
-
- def test_multiple_skills_can_subscribe_to_same_event(self):
- lock = _lock(
- emitted=["pipeline.event"],
- skills={
- "skill-a": _skill(["pipeline.event"]),
- "skill-b": _skill(["pipeline.event"]),
- },
- )
- index = self._build_trigger_index(lock)
- triggered = index.get("pipeline.event", [])
- assert "skill-a" in triggered
- assert "skill-b" in triggered
-
- def test_disconnect_means_event_never_reaches_skill(self):
- """Demonstrates the bug: skill subscribes to event that is never emitted."""
- emitted = {"pipeline.event"} # orchestrator only emits this
- skills_subscriptions = {"ghost-skill": {"content_generated"}}
-
- # Check which skills would be triggered by each emitted event
- triggered_skills: set[str] = set()
- for event in emitted:
- for skill, triggers in skills_subscriptions.items():
- if event in triggers:
- triggered_skills.add(skill)
-
- # ghost-skill subscribed to content_generated which is never emitted
- assert "ghost-skill" not in triggered_skills, (
- "ghost-skill should never fire because 'content_generated' is not "
- "in the emitted_events set. This is the quiet failure the validator prevents."
- )
-
- def test_validator_would_catch_the_disconnect(self):
- """The validator raises loudly before the system can silently fail."""
- lock = _lock(
- emitted=["pipeline.event"],
- skills={"ghost-skill": _skill(["content_generated"])},
- )
- with pytest.raises(SkillEventContractError) as exc_info:
- SkillsEventValidator().validate(lock)
- assert "ghost-skill" in str(exc_info.value)
- assert "content_generated" in str(exc_info.value)
diff --git a/tests/unit/test_unified_ai_sdk.py b/tests/unit/test_unified_ai_sdk.py
index 3d6155de2..2c396fbe8 100644
--- a/tests/unit/test_unified_ai_sdk.py
+++ b/tests/unit/test_unified_ai_sdk.py
@@ -1,22 +1,18 @@
from __future__ import annotations
from types import SimpleNamespace
-from unittest.mock import AsyncMock, MagicMock
-
-import pytest
+from unittest.mock import MagicMock
import unified_ai_sdk.unified_ai_sdk as sdk_mod
from unified_ai_sdk import AIRequest, ModelProvider, TaskType, UnifiedAISDK
class TestUnifiedAISDK:
- @pytest.mark.asyncio
async def test_unified_request_uses_openai_client(self):
sdk = UnifiedAISDK({"retry_attempts": 1})
choice = SimpleNamespace(message=SimpleNamespace(content="openai response"))
usage = SimpleNamespace(total_tokens=123)
-
- sdk._openai_client = AsyncMock()
+ sdk._openai_client = MagicMock()
sdk._openai_client.chat.completions.create.return_value = SimpleNamespace(
choices=[choice],
usage=usage,
@@ -36,10 +32,9 @@ async def test_unified_request_uses_openai_client(self):
assert result.provider == "openai"
assert result.tokens_used == 123
- @pytest.mark.asyncio
async def test_unified_request_uses_anthropic_client(self):
sdk = UnifiedAISDK({"retry_attempts": 1})
- sdk._anthropic_client = AsyncMock()
+ sdk._anthropic_client = MagicMock()
sdk._anthropic_client.messages.create.return_value = SimpleNamespace(
content=[
SimpleNamespace(type="thinking", text="hidden"),
@@ -63,7 +58,6 @@ async def test_unified_request_uses_anthropic_client(self):
assert result.provider == "claude"
assert result.tokens_used == 33
- @pytest.mark.asyncio
async def test_unified_request_uses_gemini_client(self, monkeypatch):
sdk = UnifiedAISDK({"retry_attempts": 1})
config_factory = MagicMock(side_effect=lambda **kwargs: kwargs)
@@ -74,13 +68,8 @@ async def test_unified_request_uses_gemini_client(self, monkeypatch):
SimpleNamespace(GenerateContentConfig=config_factory),
raising=False,
)
-
sdk._gemini_client = MagicMock()
- sdk._gemini_client.aio = MagicMock()
- sdk._gemini_client.aio.models = MagicMock()
- sdk._gemini_client.aio.models.generate_content = AsyncMock()
-
- sdk._gemini_client.aio.models.generate_content.return_value = SimpleNamespace(
+ sdk._gemini_client.models.generate_content.return_value = SimpleNamespace(
text=None,
candidates=[
SimpleNamespace(
@@ -113,39 +102,11 @@ async def test_unified_request_uses_gemini_client(self, monkeypatch):
"max_output_tokens": 4000,
}
- @pytest.mark.asyncio
- async def test_unified_request_uses_grok_client(self):
- sdk = UnifiedAISDK({"retry_attempts": 1})
- choice = SimpleNamespace(message=SimpleNamespace(content="grok response"))
- usage = SimpleNamespace(total_tokens=456)
-
- sdk._grok_client = AsyncMock()
- sdk._grok_client.chat.completions.create.return_value = SimpleNamespace(
- choices=[choice],
- usage=usage,
- )
-
- result = await sdk.unified_request(
- AIRequest(
- prompt="What is the latest trend?",
- model="grok-3",
- provider=ModelProvider.GROK,
- task_type=TaskType.TREND_ANALYSIS,
- )
- )
-
- assert result.success is True
- assert result.content == "grok response"
- assert result.provider == "grok"
- assert result.tokens_used == 456
-
- @pytest.mark.asyncio
async def test_unified_request_retries_before_succeeding(self):
sdk = UnifiedAISDK({"retry_attempts": 2, "retry_base_delay": 0})
choice = SimpleNamespace(message=SimpleNamespace(content="retried response"))
usage = SimpleNamespace(total_tokens=77)
-
- sdk._openai_client = AsyncMock()
+ sdk._openai_client = MagicMock()
sdk._openai_client.chat.completions.create.side_effect = [
RuntimeError("temporary failure"),
SimpleNamespace(choices=[choice], usage=usage),
@@ -164,10 +125,9 @@ async def test_unified_request_retries_before_succeeding(self):
assert result.content == "retried response"
assert result.metadata["attempts"] == 2
- @pytest.mark.asyncio
async def test_unified_request_returns_error_after_retries(self):
sdk = UnifiedAISDK({"retry_attempts": 2, "retry_base_delay": 0})
- sdk._openai_client = AsyncMock()
+ sdk._openai_client = MagicMock()
sdk._openai_client.chat.completions.create.side_effect = RuntimeError(
"provider unavailable"
)
@@ -186,142 +146,67 @@ async def test_unified_request_returns_error_after_retries(self):
assert "provider unavailable" in (result.error or "")
assert result.metadata["attempts"] == 2
- @pytest.mark.asyncio
- async def test_unified_request_does_not_retry_on_auth_error(self):
- sdk = UnifiedAISDK({"retry_attempts": 3, "retry_base_delay": 0})
- sdk._openai_client = AsyncMock()
- sdk._openai_client.chat.completions.create.side_effect = RuntimeError(
- "Authentication failed"
- )
+ async def test_unified_request_reports_model_when_gemini_client_missing(self):
+ sdk = UnifiedAISDK({"retry_attempts": 1})
+ # Force the precondition explicitly: in CI ambient GEMINI_API_KEY /
+ # GOOGLE_API_KEY plus a globally-patched `_genai` (leaked from other
+ # tests) can make _init_clients build a client, so don't rely on it
+ # defaulting to None.
+ sdk._gemini_client = None
result = await sdk.unified_request(
AIRequest(
- prompt="Fail fast",
- model="gpt-4o",
- provider=ModelProvider.OPENAI,
+ prompt="Fail cleanly",
+ model="gemini-2.5-flash",
+ provider=ModelProvider.GEMINI,
task_type=TaskType.GENERIC,
)
)
assert result.success is False
- assert result.metadata["attempts"] == 1
+ assert "gemini-2.5-flash" in (result.error or "")
- def test_should_retry_rejects_gemini_400_with_incidental_500(self):
- sdk = UnifiedAISDK({"retry_attempts": 3, "retry_base_delay": 0})
-
- assert (
- sdk._should_retry(
- RuntimeError(
- "400 INVALID_ARGUMENT: input token count 500 exceeds model limit"
- )
- )
- is False
- )
-
- def test_should_retry_ignores_non_status_colon_numbers(self):
- sdk = UnifiedAISDK({"retry_attempts": 3, "retry_base_delay": 0})
-
- assert (
- sdk._should_retry(
- RuntimeError("invalid_request: expected 3 items: 503 found")
- )
- is False
- )
-
- @pytest.mark.parametrize(
- "message",
- [
- "500 INTERNAL: upstream unavailable",
- "Response: 500 Internal Server Error",
- "429 RESOURCE_EXHAUSTED: quota exceeded",
- ],
- )
- def test_should_retry_accepts_retryable_status_formats(self, message):
- sdk = UnifiedAISDK({"retry_attempts": 3, "retry_base_delay": 0})
-
- assert sdk._should_retry(RuntimeError(message)) is True
-
- @pytest.mark.asyncio
- async def test_structured_output_support(self):
+ async def test_unified_request_unsupported_provider_returns_failure(self):
+ # ModelProvider.GROK has no client in this SDK and MCPBridge routes it;
+ # the contract is a structured failure, not an unhandled ValueError.
sdk = UnifiedAISDK({"retry_attempts": 1})
- sdk._openai_client = AsyncMock()
- sdk._openai_client.chat.completions.create.return_value = SimpleNamespace(
- choices=[SimpleNamespace(message=SimpleNamespace(content='{"key": "value"}'))],
- usage=SimpleNamespace(total_tokens=10),
- )
- await sdk.unified_request(
+ result = await sdk.unified_request(
AIRequest(
- prompt="Give me JSON",
- model="gpt-4o",
- provider=ModelProvider.OPENAI,
+ prompt="Analyze trend",
+ model="grok-beta",
+ provider=ModelProvider.GROK,
task_type=TaskType.GENERIC,
- structured_output=True,
)
)
- _, kwargs = sdk._openai_client.chat.completions.create.call_args
- assert kwargs["response_format"] == {"type": "json_object"}
-
- @pytest.mark.asyncio
- async def test_empty_content_triggers_retry(self):
- sdk = UnifiedAISDK({"retry_attempts": 2, "retry_base_delay": 0})
- sdk._openai_client = AsyncMock()
+ assert result.success is False
+ assert result.content == ""
+ assert "grok" in (result.error or "").lower()
+ assert result.metadata["attempts"] == 0
- # First call returns empty content, second call succeeds
- sdk._openai_client.chat.completions.create.side_effect = [
- SimpleNamespace(choices=[SimpleNamespace(message=SimpleNamespace(content=""))], usage=SimpleNamespace(total_tokens=0)),
- SimpleNamespace(choices=[SimpleNamespace(message=SimpleNamespace(content="success"))], usage=SimpleNamespace(total_tokens=10)),
- ]
+ async def test_anthropic_call_omits_temperature_with_thinking(self):
+ # Anthropic rejects a non-1 temperature when extended thinking is on;
+ # the call must not forward request.temperature.
+ sdk = UnifiedAISDK({"retry_attempts": 1})
+ sdk._anthropic_client = MagicMock()
+ sdk._anthropic_client.messages.create.return_value = SimpleNamespace(
+ content=[SimpleNamespace(type="text", text="ok")],
+ usage=SimpleNamespace(input_tokens=1, output_tokens=1),
+ )
result = await sdk.unified_request(
AIRequest(
- prompt="Don't be empty",
- model="gpt-4o",
- provider=ModelProvider.OPENAI,
- task_type=TaskType.GENERIC,
+ prompt="Summarize this",
+ model="claude-opus-4-8",
+ provider=ModelProvider.CLAUDE,
+ task_type=TaskType.SUMMARIZATION,
)
)
assert result.success is True
- assert result.content == "success"
- assert result.metadata["attempts"] == 2
-
-
-class TestModelProviderContract:
- """Regression guard for the #773 enum drift.
-
- #773 trimmed ``ModelProvider`` to ``{OPENAI, ANTHROPIC, GEMINI}`` while
- ``UnifiedAISDK._rate_limit_provider`` still referenced ``ModelProvider.CLAUDE``
- and ``.GROK`` by name, so every Claude/Grok request raised
- ``AttributeError`` at runtime and the CI ``test`` job went red. These tests
- assert the enum stays consistent with its consumers so the same class of
- regression fails fast in unit tests rather than in production.
- """
-
- def test_enum_has_all_members_referenced_by_consumers(self):
- # Members referenced directly by UnifiedAISDK._rate_limit_provider and
- # the provider test suite. Removing any of these reintroduces #773.
- for name, value in (
- ("OPENAI", "openai"),
- ("CLAUDE", "claude"),
- ("ANTHROPIC", "anthropic"),
- ("GEMINI", "gemini"),
- ("GROK", "grok"),
- ):
- member = getattr(ModelProvider, name)
- assert member.value == value
-
- def test_rate_limit_provider_resolves_every_dispatchable_provider(self):
- # Every provider the SDK can dispatch (see UnifiedAISDK._handlers /
- # _normalize_provider) must map to a ModelProvider without raising.
- sdk = UnifiedAISDK({"retry_attempts": 1})
- for provider_name in ("openai", "claude", "grok", "gemini"):
- provider = sdk._rate_limit_provider(provider_name)
- assert isinstance(provider, ModelProvider)
-
- def test_normalize_provider_maps_anthropic_alias_to_claude(self):
- # ModelProvider.ANTHROPIC and .CLAUDE both resolve to the "claude" path.
- sdk = UnifiedAISDK({"retry_attempts": 1})
- assert sdk._normalize_provider(ModelProvider.ANTHROPIC) == "claude"
- assert sdk._normalize_provider(ModelProvider.CLAUDE) == "claude"
+ _, kwargs = sdk._anthropic_client.messages.create.call_args
+ assert "temperature" not in kwargs
+ assert kwargs["thinking"] == {"type": "adaptive"}
+ # Explicit timeout is forwarded so a slow upstream can't pin the worker.
+ assert kwargs["timeout"] == 60.0
diff --git a/tests/verify_chat_api.py b/tests/verify_chat_api.py
index 0bdada5ba..8a16b80b5 100644
--- a/tests/verify_chat_api.py
+++ b/tests/verify_chat_api.py
@@ -33,7 +33,7 @@ async def test_chat_endpoint():
# Test case 2: Chat with video context (Rick Roll)
payload_with_video = {
"query": "What is this video about?",
- "video_url": "https://www.youtube.com/watch?v=auJzb1D-fag",
+ "video_url": "https://www.youtube.com/watch?v=dQw4w9WgXcQ",
"session_id": "test_session_2",
"context": "universal-ask",
}
diff --git a/tests/verify_router_integration.py b/tests/verify_router_integration.py
index 709df63ff..d27ca8d9d 100644
--- a/tests/verify_router_integration.py
+++ b/tests/verify_router_integration.py
@@ -49,7 +49,7 @@ async def run_tests():
# It calls get_enhanced_video_processor() which might fail if dependencies are missing
# We expect 503 or 500 if dependencies fail, or 200 if working.
# But we mostly care that the route exists.
- response = await client.post("/process_video", json={"youtube_url": "https://www.youtube.com/watch?v=auJzb1D-fag", "include_transcript": False, "include_ai_analysis": False})
+ response = await client.post("/process_video", json={"youtube_url": "https://www.youtube.com/watch?v=dQw4w9WgXcQ", "include_transcript": False, "include_ai_analysis": False})
print(f"Status: {response.status_code}")
# print(f"Response: {response.json()}") # Might be error details
diff --git a/tests/video_packs/validation_test/README.md b/tests/video_packs/validation_test/README.md
index 89a0195e7..384ad1000 100644
--- a/tests/video_packs/validation_test/README.md
+++ b/tests/video_packs/validation_test/README.md
@@ -3,7 +3,7 @@
## Video Information
- **Channel**: Code Academy Pro
- **Duration**: PT2H15M30S
-- **Video ID**: auJzb1D-fag
+- **Video ID**: dQw4w9WgXcQ
## Project Analysis
- **Type**: Tutorial
diff --git a/tests/video_packs/validation_test/pack.json b/tests/video_packs/validation_test/pack.json
index 31be921d2..48fe4362d 100644
--- a/tests/video_packs/validation_test/pack.json
+++ b/tests/video_packs/validation_test/pack.json
@@ -7,7 +7,7 @@
"schema_version": "1.0.0"
},
"video_metadata": {
- "video_id": "auJzb1D-fag",
+ "video_id": "dQw4w9WgXcQ",
"title": "React Tutorial - Complete Guide for Beginners",
"channel": "Code Academy Pro",
"duration": "PT2H15M30S",