Fix clearTimeout leaks, transcript_segments shape mismatch, and ENABLE_VERTEX_AI boolean parsing

[Copilot]

Four bugs from PR #33 review: dangling timers on fetch abort, wrong property path for segment count, and Vertex AI importing when ENABLE_VERTEX_AI=0.

Changes

• video/route.ts & transcribe/route.ts — clearTimeout in finally
clearTimeout was placed after await fetch(...), so it never ran on abort or network error, leaving timers pending. Moved into finally blocks on both the 15s backend timeout and the 8s YouTube transcript timeout.

  // Before
  const response = await fetch(..., { signal: controller.signal });
  clearTimeout(timeout);
  
  // After
  try {
    response = await fetch(..., { signal: controller.signal });
  } finally {
    clearTimeout(timeout);
  }

• video/route.ts — transcript_segments shape
  Was reading result.transcript?.length, but the backend returns transcript as { text, source, segments }. Fixed to result.transcript?.segments?.length.

• gemini_service.py — ENABLE_VERTEX_AI boolean gate
  Raw os.getenv("ENABLE_VERTEX_AI") is truthy for any non-empty string including "0", defeating the intent. Now parsed strictly:

  os.getenv("ENABLE_VERTEX_AI", "").lower() in {"1", "true", "yes"}

• video/route.ts — duplicate let transcript declaration removed (would cause a compile error).

---

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

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
v0-uvai	Error		Feb 27, 2026 11:10pm

[railway-app]

🚅 Deployed to the EventRelay-pr-35 environment in EventRelay

Service	Status	Web	Updated (UTC)
guides	✅ Success (View Logs)	Web	Feb 27, 2026 at 11:02 pm
web	✅ Success (View Logs)	Web	Feb 27, 2026 at 11:02 pm
EventRelay	✅ Success (View Logs)	Web	Feb 27, 2026 at 11:01 pm
eslint-config	✅ Success (View Logs)	Web	Feb 27, 2026 at 11:01 pm
supabase	✅ Success (View Logs)		Feb 27, 2026 at 11:01 pm

[vercel]

Additional Suggestion:

Multiple TypeScript compilation errors in embedding.ts: missing module import, unknown type errors, and implicit any parameters

[Copilot]

Pull request overview

Fixes request-timeout cleanup in the Next.js API routes to prevent dangling timers when fetch() aborts/errors, and aligns the /api/video response’s transcript_segments count with the backend’s transcript shape.

Changes:

• Move clearTimeout(...) into finally blocks so timers are always cleared (even on abort/network error) in both /api/video (15s) and /api/transcribe (8s) backend calls.
• Update /api/video to compute transcript_segments from result.transcript.segments.length (while still tolerating the legacy “array transcript” shape).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
apps/web/src/app/api/video/route.ts	Ensures backend timeout is always cleared; fixes transcript_segments counting against { text, source, segments }.
apps/web/src/app/api/transcribe/route.ts	Ensures YouTube-backend timeout is always cleared to avoid dangling timers on abort/errors.