feat(claude-code): add model selection for the claude-code engine

[r-sayar]

Summary

The claude-code engine currently passes no --model flag to the claude CLI, meaning every session uses whatever the CLI defaults to. This PR lets users pick a specific Claude model from the Settings panel.

• Settings → Anthropic card: a "Model" dropdown appears once you're connected (OAuth or API key). Selecting a model persists immediately.
• Default option omits the --model flag entirely, preserving the CLI's current default (claude-sonnet-4-6).
• Available options: Haiku 4.5, Sonnet 4.5, Sonnet 4.6, Opus 4.5, Opus 4.7.

Changes

File	What changed
authStore.ts	Added claudeCodeModel field to credentials blob + loadClaudeCodeModel / saveClaudeCodeModel
apiKeyIpc.ts	Registered settings:claude-code:model:get and settings:claude-code:model:set IPC handlers
preload/shell.ts	Exposed getModel / setModel on the claudeCode API bridge
globals.d.ts	Added TypeScript types for the two new IPC methods
runEngine.ts	Loads saved model for the claude-code engine path and populates SpawnContext.model
adapter.ts	Passes --model <id> to the claude CLI when a model is set; omits it when null
ConnectionsPane.tsx	Added <select> under the Anthropic card, visible when connected; persists on change

Test plan

• With no model selected, sessions use the CLI default (claude-sonnet-4-6)
• Selecting Haiku / Opus in Settings → Connections → Anthropic card takes effect on the next session
• Model preference survives an app restart
• Switching back to "Default" removes the --model flag

🤖 Generated with Claude Code

---

Summary by cubic

Adds model selection to the claude-code engine so you can choose a Claude model in Settings; the engine passes --model to the claude CLI when set and falls back to the CLI default otherwise. Also fixes yarn.lock to resolve install failures from stale git SHAs.

• New Features

  • Adds a "Model" dropdown in Settings → Connections → Anthropic when connected; selection saves immediately and persists across restarts.
  • Engine loads the saved model and calls claude with --model <id>; "Default" omits the flag (CLI default is claude-sonnet-4-6).
  • Options include Haiku 4.5, Sonnet 4.5/4.6, and Opus 4.5/4.7.

• Dependencies

  • Replaces stale commit pins in yarn.lock for @electron/node-gyp and libsignal with branch heads to allow clean yarn install.

^{Written for commit 1e1c2bf. Summary will update on new commits. Review in cubic}

[cubic-dev-ai]

1 issue found across 8 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="app/src/main/settings/apiKeyIpc.ts">

<violation number="1" location="app/src/main/settings/apiKeyIpc.ts:582">
P2: `handleClaudeCodeModelSet` accepts renderer-provided `model` without runtime validation (no `assertString` or allowlist check), unlike every other IPC handler in this file that validates user input before persisting. This risks invalid persisted state and downstream CLI invocation with malformed `--model` values.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Fix all with cubic | Re-trigger cubic}

[cubic-dev-ai]

P2: handleClaudeCodeModelSet accepts renderer-provided model without runtime validation (no assertString or allowlist check), unlike every other IPC handler in this file that validates user input before persisting. This risks invalid persisted state and downstream CLI invocation with malformed --model values.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At app/src/main/settings/apiKeyIpc.ts, line 582:

<comment>`handleClaudeCodeModelSet` accepts renderer-provided `model` without runtime validation (no `assertString` or allowlist check), unlike every other IPC handler in this file that validates user input before persisting. This risks invalid persisted state and downstream CLI invocation with malformed `--model` values.</comment>

<file context>
@@ -570,6 +574,19 @@ async function handleClaudeCodeLogout(): Promise<{ opened: boolean; error?: stri
+  return { model };
+}
+
+async function handleClaudeCodeModelSet(
+  _e: Electron.IpcMainInvokeEvent,
+  model: string | null,
</file context>

Suggested change

	async function handleClaudeCodeModelSet(
	_e: Electron.IpcMainInvokeEvent,
	model: string | null,
	): Promise<void> {
	mainLogger.info('apiKeyIpc.claudeCode.model.set', { model });
	async function handleClaudeCodeModelSet(
	_e: Electron.IpcMainInvokeEvent,
	model: string | null,
	): Promise<void> {
	const validated = model === null ? null : assertString(model, 'model', 160);
	mainLogger.info('apiKeyIpc.claudeCode.model.set', { model: validated });
	await saveClaudeCodeModel(validated);
	}

[Cheggin]

thanks! will take a look at this

didnt add this yet bc I didn't get requests for it but clearly I should haha