fix(ui): eliminate activity tab lag during execution streaming

Root cause: the virtualized activity log list used an unstable React key
that included logs.length. Every new tool call log appended increased
logs.length by 1, which changed the key of EVERY visible activity item,
forcing React to unmount and remount all DOM nodes on every single log
entry. During execution this happens 10+ times per second, freezing
the entire app on WebKitGTK.

Fixes:
- Key: use virtualRow.key (stable per index) instead of logs.length
- Remove measureElement ref overhead (fixed estimateSize is sufficient)
- Tune virtualizer: estimateSize 60, overscan 3 (was 88/8)
- Remove scrollTop=0 useEffect that fought the virtualizer
- Add will-change:transform hint for compositor efficiency
- execution-drawer: change smooth scroll to auto (eliminates continuous
  animation thrashing while logs stream in)

Author: kaizen403

ISSUES.md

@@ -5,6 +5,67 @@
 
 ---
 
+## [2026-05-26] — Fix sidecar startup crash (missing openid-client + otpauth + ALLOW_SHARED_OPENCODE)
+
+**Status:** Done
+**Agent:** Sisyphus (OpenCode)
+
+### What was done
+- Sidecar was crashing at startup with `ERR_MODULE_NOT_FOUND: Cannot find package 'openid-client'`. Root cause: `apps/api/src/services/sso.ts` imports `openid-client` and `apps/api/src/routes/totp.ts` imports `otpauth` — both declared in `apps/api/package.json` but not installed in `node_modules`.
+- Ran `pnpm install` at workspace root to pull the missing packages (`+17` packages installed).
+- Sidecar was also immediately exiting after recovery due to multi-user guard (`OPENLINEAR_ALLOW_SHARED_OPENCODE` not set, 4 users in DB). Added `OPENLINEAR_ALLOW_SHARED_OPENCODE=1` to `.env` for the dev machine.
+- Verified sidecar now starts cleanly: binds port 3001 + 1455, runs recovery, and stays up.
+
+### Files changed
+- `.env` — added `OPENLINEAR_ALLOW_SHARED_OPENCODE=1`
+- `pnpm-lock.yaml` — updated by `pnpm install` (openid-client, otpauth, and transitive deps)
+
+### Issues encountered
+- None after the fix.
+
+### Next steps / blockers
+- None. `pnpm dev-live` / `pnpm start` should now boot the sidecar without crashing.
+
+---
+
+## [2026-05-26] — Model provider onboarding step + execution model-not-configured guard
+
+**Status:** Done
+**Agent:** Sisyphus (OpenCode)
+
+### What was done
+- Added a 4th onboarding step "AI Model" with Back / Finish setup / Skip buttons. Team creation now advances to the new step instead of completing directly. Both Finish and Skip complete onboarding; Skip shows a toast directing the user to Settings.
+- Built `ModelProviderStep` component that shows live provider status fetched from the sidecar, lists connected providers, and links to Settings → AI Providers. Handles sidecar-unavailable gracefully (shows retry + lets user skip).
+- Added `pickAutoDetectedModel()` pure helper and `ensureModelConfigured(userId)` service in the sidecar. Auto-detection priority: anthropic → openai → google → openrouter → first connected. When a provider is authenticated but no model is selected, the function picks and persists the provider's default model automatically.
+- Updated `GET /api/opencode/setup-status` to run auto-detection and return `hasProvider` + `hasModel`. `ready` is now `hasProvider && hasModel` (previously only `hasProvider`), so it correctly reports false when a provider is connected but no model is selected.
+- Added pre-flight `ensureModelConfigured` check to `POST /api/tasks/:id/execute` and `POST /api/batches`. Returns `400 { code: 'MODEL_NOT_CONFIGURED', details: { hasProvider } }` with a human-readable message instead of silently starting a doomed agent.
+- Added `isModelNotConfiguredApiError()` helper in `lib/api/model-errors.ts` to detect the new error code on the frontend.
+- Updated single-execute path (`use-kanban-board.ts`) and batch-execute path (`use-batch-execution.ts`) to catch `MODEL_NOT_CONFIGURED` — toast the specific message and reopen the provider setup dialog automatically.
+- Provider setup dialog now distinguishes "no provider" vs "provider connected but no model selected" in toasts.
+
+### Files changed
+- `apps/sidecar/src/services/opencode-model.ts` — new file: `pickAutoDetectedModel`, `ensureModelConfigured`
+- `apps/sidecar/src/routes/opencode.ts` — `setup-status` uses auto-detect, exposes `hasProvider`/`hasModel`, stricter `ready`
+- `apps/sidecar/src/routes/execution.ts` — pre-flight model check on single execute
+- `apps/sidecar/src/routes/batches.ts` — pre-flight model check on batch create
+- `apps/desktop-ui/lib/api/opencode.ts` — `SetupStatus` extended with `hasProvider`/`hasModel`
+- `apps/desktop-ui/lib/api/model-errors.ts` — new file: `isModelNotConfiguredApiError`, `readModelNotConfiguredDetails`
+- `apps/desktop-ui/components/board/use-kanban-board.ts` — MODEL_NOT_CONFIGURED handling in single execute + retry path
+- `apps/desktop-ui/hooks/use-batch-execution.ts` — MODEL_NOT_CONFIGURED handling in batch execute; improved provider/model-specific toasts
+- `apps/desktop-ui/components/onboarding/steps/onboarding-utils.ts` — added "AI Model" to STEP_LABELS
+- `apps/desktop-ui/components/onboarding/steps/model-provider-step.tsx` — new file: onboarding step 3 UI
+- `apps/desktop-ui/components/onboarding/onboarding-wizard.tsx` — wired step 3, `handleFinishOnboarding`, `handleSkipOnboarding`, step clamp bumped to 3
+
+### Issues encountered
+- Pre-existing unrelated typecheck failures in `apps/api/` (`totp.ts`, `invitations.ts`, `sso.ts`, `tasks.ts`) for half-built features missing Prisma models and npm packages. Left untouched per protocol.
+
+### Next steps / blockers
+- Manually verify: complete onboarding on a fresh workspace → should land on AI Model step → Finish or Skip both enter the app.
+- Manually verify: try executing a task/batch with no model configured → should get clear toast + provider dialog opens.
+- Consider adding a "model configured" status indicator somewhere on the board (e.g., a subtle warning badge on the Execute buttons when `!status.ready`).
+
+---
+
 ## [2026-05-24] — Add Electron desktop wrapper for Linux (fixing WebKitGTK scroll/font issues)
 
 **Status:** Done

apps/desktop-ui/components/execution-drawer.tsx

@@ -81,7 +81,7 @@ export function ExecutionDrawer({ task, logs, progress, open, onClose }: Executi
 
   useEffect(() => {
     if (logsEndRef.current && open) {
-      logsEndRef.current.scrollIntoView({ behavior: 'smooth' })
+      logsEndRef.current.scrollIntoView({ behavior: 'auto' })
     }
   }, [logs, open])
 

apps/desktop-ui/components/task-detail-view.tsx

@@ -95,8 +95,8 @@ export function TaskDetailView({ task, progress, open, onClose, onDelete, deleti
   const logVirtualizer = useVirtualizer({
     count: reversedLogs.length,
     getScrollElement: () => logsContainerRef.current,
-    estimateSize: () => 88,
-    overscan: 8,
+    estimateSize: () => 60,
+    overscan: 3,
   })
   const [editingTitle, setEditingTitle] = useState(false)
   const [editingDescription, setEditingDescription] = useState(false)
@@ -107,12 +107,6 @@ export function TaskDetailView({ task, progress, open, onClose, onDelete, deleti
   const titleSavedRef = useRef(false)
   const descriptionSavedRef = useRef(false)
 
-  useEffect(() => {
-    if (logsContainerRef.current && open && logs.length > 0) {
-      logsContainerRef.current.scrollTop = 0
-    }
-  }, [logs, open])
-
   useEffect(() => {
     if (!detailIsExecuting) setCancelling(false)
   }, [detailIsExecuting])
@@ -427,11 +421,10 @@ export function TaskDetailView({ task, progress, open, onClose, onDelete, deleti
 
                           return (
                             <div
-                              key={`${log.timestamp}-${logs.length - virtualRow.index}`}
+                              key={virtualRow.key}
                               data-index={virtualRow.index}
-                              ref={logVirtualizer.measureElement}
                               className="absolute left-0 top-0 flex w-full gap-3"
-                              style={{ transform: `translateY(${virtualRow.start}px)` }}
+                              style={{ transform: `translateY(${virtualRow.start}px)`, willChange: 'transform' }}
                             >
                               <div className="flex flex-col items-center flex-shrink-0 w-6">
                                 <div className={cn(