feat: provider selection + searchable model picker in workflows

[alari76]

Summary

• Add Claude Code / OpenCode provider toggle to workflow Add and Edit modals (only shown when OpenCode is available)
• Replace hardcoded 3-button model picker with adaptive WorkflowModelPicker: inline buttons for ≤5 models, searchable dropdown with recent selections for >5 models
• Add provider field to frontend ReviewRepoConfig type to match server schema

Test plan

• Open Add Workflow modal → verify model picker shows 3 Claude buttons (inline mode) when OpenCode is unavailable
• With OpenCode running, verify provider toggle appears and switching to OpenCode loads models dynamically
• With >5 OpenCode models, verify searchable dropdown appears with search field and recent selections
• Edit an existing workflow → verify provider and model are pre-populated correctly
• Save workflow with OpenCode provider → verify provider field persists in config

🤖 Generated with Claude Code

[alari76]

CI Fix — Lint Error

Root cause: setLoadingOcModels in src/components/workflows/ProviderModelSection.tsx:39 was declared via useState but never called, triggering an @typescript-eslint/no-unused-vars error — the only error among 743 warnings, causing CI to fail.

Fix: Wired up setLoadingOcModels(true) before the fetchOpenCodeModels call and setLoadingOcModels(false) in a .finally() block. This resolves the lint error and also makes the existing isLoadingModels derived state (line 67) functional for the loading indicator.

Commit: 56299fe

[alari76]

CI Fix — react-hooks/set-state-in-effect lint error

Root cause: In ProviderModelSection.tsx, setLoadingOcModels(true) was called synchronously inside a useEffect body (line 44). The react-hooks/set-state-in-effect rule flags this as an error because synchronous setState in an effect body triggers cascading renders.

Fix: Initialize loadingOcModels to true in the useState call instead (useState(true)), since the fetch fires immediately on mount and the loading state should start as true anyway. This removes the synchronous setState from the effect body entirely.

Commit: 207d8b3

[alari76]

CI Fix Analysis (Workflow Run #929)

Root cause: In src/components/workflows/ProviderModelSection.tsx, setLoadingOcModels(true) was called synchronously in the useEffect body (line 44). The react-hooks linter flags this as an error because synchronous setState inside an effect can trigger cascading renders.

Fix: Initialize loadingOcModels state as true (since the fetch fires immediately on mount) and remove the synchronous setLoadingOcModels(true) call from the effect body. This preserves the same loading behavior — the component starts in a loading state and transitions to loaded once the fetch completes.

This was the only error among 726 warnings. The fix is in commit 207d8b3.