fix(review): address round-2 code review findings on PR #42

- factory.ts: don't use apiKey (openai-compatible endpoint key) when
  falling back to Anthropic for VLM verification; use env var only
- worker.ts: remove redundant SearchProviderKey cast now that the
  type is correct on JobData
- HomePage.tsx: add openrouter and openai-compatible to DISPLAY_NAMES,
  remove dead ollama entry

Author: Talha Jubair Siam

src/providers/factory.ts

@@ -229,7 +229,7 @@ export function createVerificationModel(
       if (!baseUrl) {
         // BYOK path: base URL was passed via job keys, not env var.
         // Fall back to Anthropic for stock verification (needs VLM).
-        const antKey = apiKey ?? process.env["ANTHROPIC_API_KEY"];
+        const antKey = process.env["ANTHROPIC_API_KEY"];
         if (!antKey) {
           throw new Error(
             "Stock verification requires a VLM. Set OPENREELS_LLM_BASE_URL or ANTHROPIC_API_KEY, or use --no-stock-verify.",

src/worker.ts

@@ -129,9 +129,7 @@ const worker = new Worker<JobData>(
       keys,
       llmModel: providers.llmModel,
       llmBaseUrl: providers.llmBaseUrl,
-      searchProvider: providers.searchProvider as
-        | import("./schema/providers.js").SearchProviderKey
-        | undefined,
+      searchProvider: providers.searchProvider,
     });
 
     // Build callbacks that emit BullMQ progress events and update meta.json

web/src/pages/HomePage.tsx

@@ -61,7 +61,8 @@ const DISPLAY_NAMES: Record<string, string> = {
   anthropic: "Anthropic (Claude)",
   openai: "OpenAI (GPT)",
   gemini: "Google Gemini",
-  ollama: "Ollama (Local)",
+  openrouter: "OpenRouter",
+  "openai-compatible": "Custom (OpenAI-compatible)",
   // TTS
   elevenlabs: "ElevenLabs",
   inworld: "Inworld",