Merge pull request #563 from AutoMaker-Org/v0.12.0rc

V0.12.0rc

CLAUDE.md

@@ -172,5 +172,4 @@ Use `resolveModelString()` from `@automaker/model-resolver` to convert model ali
 - `DATA_DIR` - Data storage directory (default: ./data)
 - `ALLOWED_ROOT_DIRECTORY` - Restrict file operations to specific directory
 - `AUTOMAKER_MOCK_AGENT=true` - Enable mock agent mode for CI testing
-- `AUTOMAKER_AUTO_LOGIN=true` - Skip login prompt in development (disabled when NODE_ENV=production)
 - `VITE_HOSTNAME` - Hostname for frontend API URLs (default: localhost)

README.md

@@ -389,7 +389,6 @@ npm run lint
 - `VITE_SKIP_ELECTRON` - Skip Electron in dev mode
 - `OPEN_DEVTOOLS` - Auto-open DevTools in Electron
 - `AUTOMAKER_SKIP_SANDBOX_WARNING` - Skip sandbox warning dialog (useful for dev/CI)
-- `AUTOMAKER_AUTO_LOGIN=true` - Skip login prompt in development (ignored when NODE_ENV=production)
 
 ### Authentication Setup
 

SECURITY_TODO.md

@@ -1,300 +0,0 @@
-# Security Audit Findings - v0.13.0rc Branch
-
-**Date:** $(date)  
-**Audit Type:** Git diff security review against v0.13.0rc branch  
-**Status:** ⚠️ Security vulnerabilities found - requires fixes before release
-
-## Executive Summary
-
-No intentionally malicious code was detected in the changes. However, several **critical security vulnerabilities** were identified that could allow command injection attacks. These must be fixed before release.
-
----
-
-## 🔴 Critical Security Issues
-
-### 1. Command Injection in Merge Handler
-
-**File:** `apps/server/src/routes/worktree/routes/merge.ts`  
-**Lines:** 43, 54, 65-66, 93  
-**Severity:** CRITICAL
-
-**Issue:**
-User-controlled inputs (`branchName`, `mergeTo`, `options?.message`) are directly interpolated into shell commands without validation, allowing command injection attacks.
-
-**Vulnerable Code:**
-
-```typescript
-// Line 43 - branchName not validated
-await execAsync(`git rev-parse --verify ${branchName}`, { cwd: projectPath });
-
-// Line 54 - mergeTo not validated
-await execAsync(`git rev-parse --verify ${mergeTo}`, { cwd: projectPath });
-
-// Lines 65-66 - branchName and message not validated
-const mergeCmd = options?.squash
-  ? `git merge --squash ${branchName}`
-  : `git merge ${branchName} -m "${options?.message || `Merge ${branchName} into ${mergeTo}`}"`;
-
-// Line 93 - message not sanitized
-await execAsync(`git commit -m "${options?.message || `Merge ${branchName} (squash)`}"`, {
-  cwd: projectPath,
-});
-```
-
-**Attack Vector:**
-An attacker could inject shell commands via branch names or commit messages:
-
-- Branch name: `main; rm -rf /`
-- Commit message: `"; malicious_command; "`
-
-**Fix Required:**
-
-1. Validate `branchName` and `mergeTo` using `isValidBranchName()` before use
-2. Sanitize commit messages or use `execGitCommand` with proper escaping
-3. Replace `execAsync` template literals with `execGitCommand` array-based calls
-
-**Note:** `isValidBranchName` is imported but only used AFTER deletion (line 119), not before execAsync calls.
-
----
-
-### 2. Command Injection in Push Handler
-
-**File:** `apps/server/src/routes/worktree/routes/push.ts`  
-**Lines:** 44, 49  
-**Severity:** CRITICAL
-
-**Issue:**
-User-controlled `remote` parameter and `branchName` are directly interpolated into shell commands without validation.
-
-**Vulnerable Code:**
-
-```typescript
-// Line 38 - remote defaults to 'origin' but not validated
-const targetRemote = remote || 'origin';
-
-// Lines 44, 49 - targetRemote and branchName not validated
-await execAsync(`git push -u ${targetRemote} ${branchName} ${forceFlag}`, {
-  cwd: worktreePath,
-});
-await execAsync(`git push --set-upstream ${targetRemote} ${branchName} ${forceFlag}`, {
-  cwd: worktreePath,
-});
-```
-
-**Attack Vector:**
-An attacker could inject commands via the remote name:
-
-- Remote: `origin; malicious_command; #`
-
-**Fix Required:**
-
-1. Validate `targetRemote` parameter (alphanumeric + `-`, `_` only)
-2. Validate `branchName` before use (even though it comes from git output)
-3. Use `execGitCommand` with array arguments instead of template literals
-
----
-
-### 3. Unsafe Environment Variable Export in Shell Script
-
-**File:** `start-automaker.sh`  
-**Lines:** 5068, 5085  
-**Severity:** CRITICAL
-
-**Issue:**
-Unsafe parsing and export of `.env` file contents using `xargs` without proper handling of special characters.
-
-**Vulnerable Code:**
-
-```bash
-export $(grep -v '^#' .env | xargs)
-```
-
-**Attack Vector:**
-If `.env` file contains malicious content with spaces, special characters, or code, it could be executed:
-
-- `.env` entry: `VAR="value; malicious_command"`
-- Could lead to code execution during startup
-
-**Fix Required:**
-Replace with safer parsing method:
-
-```bash
-# Safer approach
-set -a
-source <(grep -v '^#' .env | sed 's/^/export /')
-set +a
-
-# Or even safer - validate each line
-while IFS= read -r line; do
-  [[ "$line" =~ ^[[:space:]]*# ]] && continue
-  [[ -z "$line" ]] && continue
-  if [[ "$line" =~ ^([A-Za-z_][A-Za-z0-9_]*)=(.*)$ ]]; then
-    export "${BASH_REMATCH[1]}"="${BASH_REMATCH[2]}"
-  fi
-done < .env
-```
-
----
-
-## 🟡 Moderate Security Concerns
-
-### 4. Inconsistent Use of Secure Command Execution
-
-**Issue:**
-The codebase has `execGitCommand()` function available (which uses array arguments and is safer), but it's not consistently used. Some places still use `execAsync` with template literals.
-
-**Files Affected:**
-
-- `apps/server/src/routes/worktree/routes/merge.ts`
-- `apps/server/src/routes/worktree/routes/push.ts`
-
-**Recommendation:**
-
-- Audit all `execAsync` calls with template literals
-- Replace with `execGitCommand` where possible
-- Document when `execAsync` is acceptable (only with fully validated inputs)
-
----
-
-### 5. Missing Input Validation
-
-**Issues:**
-
-1. `targetRemote` in `push.ts` defaults to 'origin' but isn't validated
-2. Commit messages in `merge.ts` aren't sanitized before use in shell commands
-3. `worktreePath` validation relies on middleware but should be double-checked
-
-**Recommendation:**
-
-- Add validation functions for remote names
-- Sanitize commit messages (remove shell metacharacters)
-- Add defensive validation even when middleware exists
-
----
-
-## ✅ Positive Security Findings
-
-1. **No Hardcoded Credentials:** No API keys, passwords, or tokens found in the diff
-2. **No Data Exfiltration:** No suspicious network requests or data transmission patterns
-3. **No Backdoors:** No hidden functionality or unauthorized access patterns detected
-4. **Safe Command Execution:** `execGitCommand` function properly uses array arguments in some places
-5. **Environment Variable Handling:** `init-script-service.ts` properly sanitizes environment variables (lines 194-220)
-
----
-
-## 📋 Action Items
-
-### Immediate (Before Release)
-
-- [ ] **Fix command injection in `merge.ts`**
-  - [ ] Validate `branchName` with `isValidBranchName()` before line 43
-  - [ ] Validate `mergeTo` with `isValidBranchName()` before line 54
-  - [ ] Sanitize commit messages or use `execGitCommand` for merge commands
-  - [ ] Replace `execAsync` template literals with `execGitCommand` array calls
-
-- [ ] **Fix command injection in `push.ts`**
-  - [ ] Add validation function for remote names
-  - [ ] Validate `targetRemote` before use
-  - [ ] Validate `branchName` before use (defensive programming)
-  - [ ] Replace `execAsync` template literals with `execGitCommand`
-
-- [ ] **Fix shell script security issue**
-  - [ ] Replace unsafe `export $(grep ... | xargs)` with safer parsing
-  - [ ] Add validation for `.env` file contents
-  - [ ] Test with edge cases (spaces, special chars, quotes)
-
-### Short-term (Next Sprint)
-
-- [ ] **Audit all `execAsync` calls**
-  - [ ] Create inventory of all `execAsync` calls with template literals
-  - [ ] Replace with `execGitCommand` where possible
-  - [ ] Document exceptions and why they're safe
-
-- [ ] **Add input validation utilities**
-  - [ ] Create `isValidRemoteName()` function
-  - [ ] Create `sanitizeCommitMessage()` function
-  - [ ] Add validation for all user-controlled inputs
-
-- [ ] **Security testing**
-  - [ ] Add unit tests for command injection prevention
-  - [ ] Add integration tests with malicious inputs
-  - [ ] Test shell script with malicious `.env` files
-
-### Long-term (Security Hardening)
-
-- [ ] **Code review process**
-  - [ ] Add security checklist for PR reviews
-  - [ ] Require security review for shell command execution changes
-  - [ ] Add automated security scanning
-
-- [ ] **Documentation**
-  - [ ] Document secure coding practices for shell commands
-  - [ ] Create security guidelines for contributors
-  - [ ] Add security section to CONTRIBUTING.md
-
----
-
-## 🔍 Testing Recommendations
-
-### Command Injection Tests
-
-```typescript
-// Test cases for merge.ts
-describe('merge handler security', () => {
-  it('should reject branch names with shell metacharacters', () => {
-    // Test: branchName = "main; rm -rf /"
-    // Expected: Validation error, command not executed
-  });
-
-  it('should sanitize commit messages', () => {
-    // Test: message = '"; malicious_command; "'
-    // Expected: Sanitized or rejected
-  });
-});
-
-// Test cases for push.ts
-describe('push handler security', () => {
-  it('should reject remote names with shell metacharacters', () => {
-    // Test: remote = "origin; malicious_command; #"
-    // Expected: Validation error, command not executed
-  });
-});
-```
-
-### Shell Script Tests
-
-```bash
-# Test with malicious .env content
-echo 'VAR="value; echo PWNED"' > test.env
-# Expected: Should not execute the command
-
-# Test with spaces in values
-echo 'VAR="value with spaces"' > test.env
-# Expected: Should handle correctly
-
-# Test with special characters
-echo 'VAR="value\$with\$dollars"' > test.env
-# Expected: Should handle correctly
-```
-
----
-
-## 📚 References
-
-- [OWASP Command Injection](https://owasp.org/www-community/attacks/Command_Injection)
-- [Node.js Child Process Security](https://nodejs.org/api/child_process.html#child_process_security_concerns)
-- [Shell Script Security Best Practices](https://mywiki.wooledge.org/BashGuide/Practices)
-
----
-
-## Notes
-
-- All findings are based on code diff analysis
-- No runtime testing was performed
-- Assumes attacker has access to API endpoints (authenticated or unauthenticated)
-- Fixes should be tested thoroughly before deployment
-
----
-
-**Last Updated:** $(date)  
-**Next Review:** After fixes are implemented

TODO.md

@@ -2,14 +2,6 @@
 
 - Setting the default model does not seem like it works.
 
-# Performance (completed)
-
-- [x] Graph performance mode for large graphs (compact nodes/edges + visible-only rendering)
-- [x] Render containment on heavy scroll regions (kanban columns, chat history)
-- [x] Reduce blur/shadow effects when lists get large
-- [x] React Query tuning for heavy datasets (less refetch on focus/reconnect)
-- [x] DnD/list rendering optimizations (virtualized kanban + memoized card sections)
-
 # UX
 
 - Consolidate all models to a single place in the settings instead of having AI profiles and all this other stuff

apps/server/src/index.ts

@@ -91,9 +91,6 @@ const PORT = parseInt(process.env.PORT || '3008', 10);
 const HOST = process.env.HOST || '0.0.0.0';
 const HOSTNAME = process.env.HOSTNAME || 'localhost';
 const DATA_DIR = process.env.DATA_DIR || './data';
-logger.info('[SERVER_STARTUP] process.env.DATA_DIR:', process.env.DATA_DIR);
-logger.info('[SERVER_STARTUP] Resolved DATA_DIR:', DATA_DIR);
-logger.info('[SERVER_STARTUP] process.cwd():', process.cwd());
 const ENABLE_REQUEST_LOGGING_DEFAULT = process.env.ENABLE_REQUEST_LOGGING !== 'false'; // Default to true
 
 // Runtime-configurable request logging flag (can be changed via settings)
@@ -113,37 +110,24 @@ export function isRequestLoggingEnabled(): boolean {
   return requestLoggingEnabled;
 }
 
-// Width for log box content (excluding borders)
-const BOX_CONTENT_WIDTH = 67;
-
 // Check for required environment variables
 const hasAnthropicKey = !!process.env.ANTHROPIC_API_KEY;
 
 if (!hasAnthropicKey) {
-  const wHeader = '⚠️  WARNING: No Claude authentication configured'.padEnd(BOX_CONTENT_WIDTH);
-  const w1 = 'The Claude Agent SDK requires authentication to function.'.padEnd(BOX_CONTENT_WIDTH);
-  const w2 = 'Set your Anthropic API key:'.padEnd(BOX_CONTENT_WIDTH);
-  const w3 = '  export ANTHROPIC_API_KEY="sk-ant-..."'.padEnd(BOX_CONTENT_WIDTH);
-  const w4 = 'Or use the setup wizard in Settings to configure authentication.'.padEnd(
-    BOX_CONTENT_WIDTH
-  );
-
   logger.warn(`
-╔═════════════════════════════════════════════════════════════════════╗
-║  ${wHeader}║
-╠═════════════════════════════════════════════════════════════════════╣
-║                                                                     ║
-║  ${w1}║
-║                                                                     ║
-║  ${w2}║
-║  ${w3}║
-║                                                                     ║
-║  ${w4}║
-║                                                                     ║
-╚═════════════════════════════════════════════════════════════════════╝
+╔═══════════════════════════════════════════════════════════════════════╗
+║  ⚠️  WARNING: No Claude authentication configured                      ║
+║                                                                       ║
+║  The Claude Agent SDK requires authentication to function.            ║
+║                                                                       ║
+║  Set your Anthropic API key:                                          ║
+║    export ANTHROPIC_API_KEY="sk-ant-..."                              ║
+║                                                                       ║
+║  Or use the setup wizard in Settings to configure authentication.     ║
+╚═══════════════════════════════════════════════════════════════════════╝
 `);
 } else {
-  logger.info('✓ ANTHROPIC_API_KEY detected');
+  logger.info('✓ ANTHROPIC_API_KEY detected (API key auth)');
 }
 
 // Initialize security
@@ -191,25 +175,14 @@ app.use(
         return;
       }
 
-      // For local development, allow all localhost/loopback origins (any port)
-      try {
-        const url = new URL(origin);
-        const hostname = url.hostname;
-
-        if (
-          hostname === 'localhost' ||
-          hostname === '127.0.0.1' ||
-          hostname === '::1' ||
-          hostname === '0.0.0.0' ||
-          hostname.startsWith('192.168.') ||
-          hostname.startsWith('10.') ||
-          hostname.startsWith('172.')
-        ) {
-          callback(null, origin);
-          return;
-        }
-      } catch (err) {
-        // Ignore URL parsing errors
+      // For local development, allow localhost origins
+      if (
+        origin.startsWith('http://localhost:') ||
+        origin.startsWith('http://127.0.0.1:') ||
+        origin.startsWith('http://[::1]:')
+      ) {
+        callback(null, origin);
+        return;
       }
 
       // Reject other origins by default for security
@@ -249,27 +222,10 @@ notificationService.setEventEmitter(events);
 const eventHistoryService = getEventHistoryService();
 
 // Initialize Event Hook Service for custom event triggers (with history storage)
-eventHookService.initialize(events, settingsService, eventHistoryService, featureLoader);
+eventHookService.initialize(events, settingsService, eventHistoryService);
 
 // Initialize services
 (async () => {
-  // Migrate settings from legacy Electron userData location if needed
-  // This handles users upgrading from versions that stored settings in ~/.config/Automaker (Linux),
-  // ~/Library/Application Support/Automaker (macOS), or %APPDATA%\Automaker (Windows)
-  // to the new shared ./data directory
-  try {
-    const migrationResult = await settingsService.migrateFromLegacyElectronPath();
-    if (migrationResult.migrated) {
-      logger.info(`Settings migrated from legacy location: ${migrationResult.legacyPath}`);
-      logger.info(`Migrated files: ${migrationResult.migratedFiles.join(', ')}`);
-    }
-    if (migrationResult.errors.length > 0) {
-      logger.warn('Migration errors:', migrationResult.errors);
-    }
-  } catch (err) {
-    logger.warn('Failed to check for legacy settings migration:', err);
-  }
-
   // Apply logging settings from saved settings
   try {
     const settings = await settingsService.getGlobalSettings();
@@ -662,74 +618,40 @@ const startServer = (port: number, host: string) => {
         ? 'enabled (password protected)'
         : 'enabled'
       : 'disabled';
-
-    // Build URLs for display
-    const listenAddr = `${host}:${port}`;
-    const httpUrl = `http://${HOSTNAME}:${port}`;
-    const wsEventsUrl = `ws://${HOSTNAME}:${port}/api/events`;
-    const wsTerminalUrl = `ws://${HOSTNAME}:${port}/api/terminal/ws`;
-    const healthUrl = `http://${HOSTNAME}:${port}/api/health`;
-
-    const sHeader = '🚀 Automaker Backend Server'.padEnd(BOX_CONTENT_WIDTH);
-    const s1 = `Listening:    ${listenAddr}`.padEnd(BOX_CONTENT_WIDTH);
-    const s2 = `HTTP API:     ${httpUrl}`.padEnd(BOX_CONTENT_WIDTH);
-    const s3 = `WebSocket:    ${wsEventsUrl}`.padEnd(BOX_CONTENT_WIDTH);
-    const s4 = `Terminal WS:  ${wsTerminalUrl}`.padEnd(BOX_CONTENT_WIDTH);
-    const s5 = `Health:       ${healthUrl}`.padEnd(BOX_CONTENT_WIDTH);
-    const s6 = `Terminal:     ${terminalStatus}`.padEnd(BOX_CONTENT_WIDTH);
-
+    const portStr = port.toString().padEnd(4);
     logger.info(`
-╔═════════════════════════════════════════════════════════════════════╗
-║  ${sHeader}║
-╠═════════════════════════════════════════════════════════════════════╣
-║                                                                     ║
-║  ${s1}║
-║  ${s2}║
-║  ${s3}║
-║  ${s4}║
-║  ${s5}║
-║  ${s6}║
-║                                                                     ║
-╚═════════════════════════════════════════════════════════════════════╝
+╔═══════════════════════════════════════════════════════╗
+║           Automaker Backend Server                    ║
+╠═══════════════════════════════════════════════════════╣
+║  Listening:   ${host}:${port}${' '.repeat(Math.max(0, 34 - host.length - port.toString().length))}║
+║  HTTP API:    http://${HOSTNAME}:${portStr}                 ║
+║  WebSocket:   ws://${HOSTNAME}:${portStr}/api/events        ║
+║  Terminal:    ws://${HOSTNAME}:${portStr}/api/terminal/ws   ║
+║  Health:      http://${HOSTNAME}:${portStr}/api/health      ║
+║  Terminal:    ${terminalStatus.padEnd(37)}║
+╚═══════════════════════════════════════════════════════╝
 `);
   });
 
   server.on('error', (error: NodeJS.ErrnoException) => {
     if (error.code === 'EADDRINUSE') {
-      const portStr = port.toString();
-      const nextPortStr = (port + 1).toString();
-      const killCmd = `lsof -ti:${portStr} | xargs kill -9`;
-      const altCmd = `PORT=${nextPortStr} npm run dev:server`;
-
-      const eHeader = `❌ ERROR: Port ${portStr} is already in use`.padEnd(BOX_CONTENT_WIDTH);
-      const e1 = 'Another process is using this port.'.padEnd(BOX_CONTENT_WIDTH);
-      const e2 = 'To fix this, try one of:'.padEnd(BOX_CONTENT_WIDTH);
-      const e3 = '1. Kill the process using the port:'.padEnd(BOX_CONTENT_WIDTH);
-      const e4 = `   ${killCmd}`.padEnd(BOX_CONTENT_WIDTH);
-      const e5 = '2. Use a different port:'.padEnd(BOX_CONTENT_WIDTH);
-      const e6 = `   ${altCmd}`.padEnd(BOX_CONTENT_WIDTH);
-      const e7 = '3. Use the init.sh script which handles this:'.padEnd(BOX_CONTENT_WIDTH);
-      const e8 = '   ./init.sh'.padEnd(BOX_CONTENT_WIDTH);
-
       logger.error(`
-╔═════════════════════════════════════════════════════════════════════╗
-║  ${eHeader}║
-╠═════════════════════════════════════════════════════════════════════╣
-║                                                                     ║
-║  ${e1}║
-║                                                                     ║
-║  ${e2}║
-║                                                                     ║
-║  ${e3}║
-║  ${e4}║
-║                                                                     ║
-║  ${e5}║
-║  ${e6}║
-║                                                                     ║
-║  ${e7}║
-║  ${e8}║
-║                                                                     ║
-╚═════════════════════════════════════════════════════════════════════╝
+╔═══════════════════════════════════════════════════════╗
+║  ❌ ERROR: Port ${port} is already in use              ║
+╠═══════════════════════════════════════════════════════╣
+║  Another process is using this port.                  ║
+║                                                       ║
+║  To fix this, try one of:                             ║
+║                                                       ║
+║  1. Kill the process using the port:                  ║
+║     lsof -ti:${port} | xargs kill -9                   ║
+║                                                       ║
+║  2. Use a different port:                             ║
+║     PORT=${port + 1} npm run dev:server                ║
+║                                                       ║
+║  3. Use the init.sh script which handles this:        ║
+║     ./init.sh                                         ║
+╚═══════════════════════════════════════════════════════╝
 `);
       process.exit(1);
     } else {

apps/server/src/lib/auth.ts

@@ -23,13 +23,6 @@ const SESSION_COOKIE_NAME = 'automaker_session';
 const SESSION_MAX_AGE_MS = 30 * 24 * 60 * 60 * 1000; // 30 days
 const WS_TOKEN_MAX_AGE_MS = 5 * 60 * 1000; // 5 minutes for WebSocket connection tokens
 
-/**
- * Check if an environment variable is set to 'true'
- */
-function isEnvTrue(envVar: string | undefined): boolean {
-  return envVar === 'true';
-}
-
 // Session store - persisted to file for survival across server restarts
 const validSessions = new Map<string, { createdAt: number; expiresAt: number }>();
 
@@ -137,47 +130,21 @@ function ensureApiKey(): string {
 // API key - always generated/loaded on startup for CSRF protection
 const API_KEY = ensureApiKey();
 
-// Width for log box content (excluding borders)
-const BOX_CONTENT_WIDTH = 67;
-
 // Print API key to console for web mode users (unless suppressed for production logging)
-if (!isEnvTrue(process.env.AUTOMAKER_HIDE_API_KEY)) {
-  const autoLoginEnabled = isEnvTrue(process.env.AUTOMAKER_AUTO_LOGIN);
-  const autoLoginStatus = autoLoginEnabled ? 'enabled (auto-login active)' : 'disabled';
-
-  // Build box lines with exact padding
-  const header = '🔐 API Key for Web Mode Authentication'.padEnd(BOX_CONTENT_WIDTH);
-  const line1 = "When accessing via browser, you'll be prompted to enter this key:".padEnd(
-    BOX_CONTENT_WIDTH
-  );
-  const line2 = API_KEY.padEnd(BOX_CONTENT_WIDTH);
-  const line3 = 'In Electron mode, authentication is handled automatically.'.padEnd(
-    BOX_CONTENT_WIDTH
-  );
-  const line4 = `Auto-login (AUTOMAKER_AUTO_LOGIN): ${autoLoginStatus}`.padEnd(BOX_CONTENT_WIDTH);
-  const tipHeader = '💡 Tips'.padEnd(BOX_CONTENT_WIDTH);
-  const line5 = 'Set AUTOMAKER_API_KEY env var to use a fixed key'.padEnd(BOX_CONTENT_WIDTH);
-  const line6 = 'Set AUTOMAKER_AUTO_LOGIN=true to skip the login prompt'.padEnd(BOX_CONTENT_WIDTH);
-
+if (process.env.AUTOMAKER_HIDE_API_KEY !== 'true') {
   logger.info(`
-╔═════════════════════════════════════════════════════════════════════╗
-║  ${header}║
-╠═════════════════════════════════════════════════════════════════════╣
-║                                                                     ║
-║  ${line1}║
-║                                                                     ║
-║  ${line2}║
-║                                                                     ║
-║  ${line3}║
-║                                                                     ║
-║  ${line4}║
-║                                                                     ║
-╠═════════════════════════════════════════════════════════════════════╣
-║  ${tipHeader}║
-╠═════════════════════════════════════════════════════════════════════╣
-║  ${line5}║
-║  ${line6}║
-╚═════════════════════════════════════════════════════════════════════╝
+╔═══════════════════════════════════════════════════════════════════════╗
+║  🔐 API Key for Web Mode Authentication                               ║
+╠═══════════════════════════════════════════════════════════════════════╣
+║                                                                       ║
+║  When accessing via browser, you'll be prompted to enter this key:    ║
+║                                                                       ║
+║    ${API_KEY}
+║                                                                       ║
+║  In Electron mode, authentication is handled automatically.          ║
+║                                                                       ║
+║  💡 Tip: Set AUTOMAKER_API_KEY env var to use a fixed key for dev    ║
+╚═══════════════════════════════════════════════════════════════════════╝
 `);
 } else {
   logger.info('API key banner hidden (AUTOMAKER_HIDE_API_KEY=true)');
@@ -353,15 +320,6 @@ function checkAuthentication(
     return { authenticated: false, errorType: 'invalid_api_key' };
   }
 
-  // Check for session token in query parameter (web mode - needed for image loads)
-  const queryToken = query.token;
-  if (queryToken) {
-    if (validateSession(queryToken)) {
-      return { authenticated: true };
-    }
-    return { authenticated: false, errorType: 'invalid_session' };
-  }
-
   // Check for session cookie (web mode)
   const sessionToken = cookies[SESSION_COOKIE_NAME];
   if (sessionToken && validateSession(sessionToken)) {
@@ -377,17 +335,10 @@ function checkAuthentication(
  * Accepts either:
  * 1. X-API-Key header (for Electron mode)
  * 2. X-Session-Token header (for web mode with explicit token)
- * 3. apiKey query parameter (fallback for Electron, cases where headers can't be set)
- * 4. token query parameter (fallback for web mode, needed for image loads via CSS/img tags)
- * 5. Session cookie (for web mode)
+ * 3. apiKey query parameter (fallback for cases where headers can't be set)
+ * 4. Session cookie (for web mode)
  */
 export function authMiddleware(req: Request, res: Response, next: NextFunction): void {
-  // Allow disabling auth for local/trusted networks
-  if (isEnvTrue(process.env.AUTOMAKER_DISABLE_AUTH)) {
-    next();
-    return;
-  }
-
   const result = checkAuthentication(
     req.headers as Record<string, string | string[] | undefined>,
     req.query as Record<string, string | undefined>,
@@ -433,10 +384,9 @@ export function isAuthEnabled(): boolean {
  * Get authentication status for health endpoint
  */
 export function getAuthStatus(): { enabled: boolean; method: string } {
-  const disabled = isEnvTrue(process.env.AUTOMAKER_DISABLE_AUTH);
   return {
-    enabled: !disabled,
-    method: disabled ? 'disabled' : 'api_key_or_session',
+    enabled: true,
+    method: 'api_key_or_session',
   };
 }
 
@@ -444,7 +394,6 @@ export function getAuthStatus(): { enabled: boolean; method: string } {
  * Check if a request is authenticated (for status endpoint)
  */
 export function isRequestAuthenticated(req: Request): boolean {
-  if (isEnvTrue(process.env.AUTOMAKER_DISABLE_AUTH)) return true;
   const result = checkAuthentication(
     req.headers as Record<string, string | string[] | undefined>,
     req.query as Record<string, string | undefined>,
@@ -462,6 +411,5 @@ export function checkRawAuthentication(
   query: Record<string, string | undefined>,
   cookies: Record<string, string | undefined>
 ): boolean {
-  if (isEnvTrue(process.env.AUTOMAKER_DISABLE_AUTH)) return true;
   return checkAuthentication(headers, query, cookies).authenticated;
 }

apps/server/src/lib/settings-helpers.ts

@@ -5,17 +5,7 @@
 import type { SettingsService } from '../services/settings-service.js';
 import type { ContextFilesResult, ContextFileInfo } from '@automaker/utils';
 import { createLogger } from '@automaker/utils';
-import type {
-  MCPServerConfig,
-  McpServerConfig,
-  PromptCustomization,
-  ClaudeApiProfile,
-  ClaudeCompatibleProvider,
-  PhaseModelKey,
-  PhaseModelEntry,
-  Credentials,
-} from '@automaker/types';
-import { DEFAULT_PHASE_MODELS } from '@automaker/types';
+import type { MCPServerConfig, McpServerConfig, PromptCustomization } from '@automaker/types';
 import {
   mergeAutoModePrompts,
   mergeAgentPrompts,
@@ -355,376 +345,3 @@ export async function getCustomSubagents(
 
   return Object.keys(merged).length > 0 ? merged : undefined;
 }
-
-/** Result from getActiveClaudeApiProfile */
-export interface ActiveClaudeApiProfileResult {
-  /** The active profile, or undefined if using direct Anthropic API */
-  profile: ClaudeApiProfile | undefined;
-  /** Credentials for resolving 'credentials' apiKeySource */
-  credentials: import('@automaker/types').Credentials | undefined;
-}
-
-/**
- * Get the active Claude API profile and credentials from settings.
- * Checks project settings first for per-project overrides, then falls back to global settings.
- * Returns both the profile and credentials for resolving 'credentials' apiKeySource.
- *
- * @deprecated Use getProviderById and getPhaseModelWithOverrides instead for the new provider system.
- * This function is kept for backward compatibility during migration.
- *
- * @param settingsService - Optional settings service instance
- * @param logPrefix - Prefix for log messages (e.g., '[AgentService]')
- * @param projectPath - Optional project path for per-project override
- * @returns Promise resolving to object with profile and credentials
- */
-export async function getActiveClaudeApiProfile(
-  settingsService?: SettingsService | null,
-  logPrefix = '[SettingsHelper]',
-  projectPath?: string
-): Promise<ActiveClaudeApiProfileResult> {
-  if (!settingsService) {
-    return { profile: undefined, credentials: undefined };
-  }
-
-  try {
-    const globalSettings = await settingsService.getGlobalSettings();
-    const credentials = await settingsService.getCredentials();
-    const profiles = globalSettings.claudeApiProfiles || [];
-
-    // Check for project-level override first
-    let activeProfileId: string | null | undefined;
-    let isProjectOverride = false;
-
-    if (projectPath) {
-      const projectSettings = await settingsService.getProjectSettings(projectPath);
-      // undefined = use global, null = explicit no profile, string = specific profile
-      if (projectSettings.activeClaudeApiProfileId !== undefined) {
-        activeProfileId = projectSettings.activeClaudeApiProfileId;
-        isProjectOverride = true;
-      }
-    }
-
-    // Fall back to global if project doesn't specify
-    if (activeProfileId === undefined && !isProjectOverride) {
-      activeProfileId = globalSettings.activeClaudeApiProfileId;
-    }
-
-    // No active profile selected - use direct Anthropic API
-    if (!activeProfileId) {
-      if (isProjectOverride && activeProfileId === null) {
-        logger.info(`${logPrefix} Project explicitly using Direct Anthropic API`);
-      }
-      return { profile: undefined, credentials };
-    }
-
-    // Find the active profile by ID
-    const activeProfile = profiles.find((p) => p.id === activeProfileId);
-
-    if (activeProfile) {
-      const overrideSuffix = isProjectOverride ? ' (project override)' : '';
-      logger.info(`${logPrefix} Using Claude API profile: ${activeProfile.name}${overrideSuffix}`);
-      return { profile: activeProfile, credentials };
-    } else {
-      logger.warn(
-        `${logPrefix} Active profile ID "${activeProfileId}" not found, falling back to direct Anthropic API`
-      );
-      return { profile: undefined, credentials };
-    }
-  } catch (error) {
-    logger.error(`${logPrefix} Failed to load Claude API profile:`, error);
-    return { profile: undefined, credentials: undefined };
-  }
-}
-
-// ============================================================================
-// New Provider System Helpers
-// ============================================================================
-
-/** Result from getProviderById */
-export interface ProviderByIdResult {
-  /** The provider, or undefined if not found */
-  provider: ClaudeCompatibleProvider | undefined;
-  /** Credentials for resolving 'credentials' apiKeySource */
-  credentials: Credentials | undefined;
-}
-
-/**
- * Get a ClaudeCompatibleProvider by its ID.
- * Returns the provider configuration and credentials for API key resolution.
- *
- * @param providerId - The provider ID to look up
- * @param settingsService - Settings service instance
- * @param logPrefix - Prefix for log messages
- * @returns Promise resolving to object with provider and credentials
- */
-export async function getProviderById(
-  providerId: string,
-  settingsService: SettingsService,
-  logPrefix = '[SettingsHelper]'
-): Promise<ProviderByIdResult> {
-  try {
-    const globalSettings = await settingsService.getGlobalSettings();
-    const credentials = await settingsService.getCredentials();
-    const providers = globalSettings.claudeCompatibleProviders || [];
-
-    const provider = providers.find((p) => p.id === providerId);
-
-    if (provider) {
-      if (provider.enabled === false) {
-        logger.warn(`${logPrefix} Provider "${provider.name}" (${providerId}) is disabled`);
-      } else {
-        logger.debug(`${logPrefix} Found provider: ${provider.name}`);
-      }
-      return { provider, credentials };
-    } else {
-      logger.warn(`${logPrefix} Provider not found: ${providerId}`);
-      return { provider: undefined, credentials };
-    }
-  } catch (error) {
-    logger.error(`${logPrefix} Failed to load provider by ID:`, error);
-    return { provider: undefined, credentials: undefined };
-  }
-}
-
-/** Result from getPhaseModelWithOverrides */
-export interface PhaseModelWithOverridesResult {
-  /** The resolved phase model entry */
-  phaseModel: PhaseModelEntry;
-  /** Whether a project override was applied */
-  isProjectOverride: boolean;
-  /** The provider if providerId is set and found */
-  provider: ClaudeCompatibleProvider | undefined;
-  /** Credentials for API key resolution */
-  credentials: Credentials | undefined;
-}
-
-/**
- * Get the phase model configuration for a specific phase, applying project overrides if available.
- * Also resolves the provider if the phase model has a providerId.
- *
- * @param phase - The phase key (e.g., 'enhancementModel', 'specGenerationModel')
- * @param settingsService - Optional settings service instance (returns defaults if undefined)
- * @param projectPath - Optional project path for checking overrides
- * @param logPrefix - Prefix for log messages
- * @returns Promise resolving to phase model with provider info
- */
-export async function getPhaseModelWithOverrides(
-  phase: PhaseModelKey,
-  settingsService?: SettingsService | null,
-  projectPath?: string,
-  logPrefix = '[SettingsHelper]'
-): Promise<PhaseModelWithOverridesResult> {
-  // Handle undefined settingsService gracefully
-  if (!settingsService) {
-    logger.info(`${logPrefix} SettingsService not available, using default for ${phase}`);
-    return {
-      phaseModel: DEFAULT_PHASE_MODELS[phase] || { model: 'sonnet' },
-      isProjectOverride: false,
-      provider: undefined,
-      credentials: undefined,
-    };
-  }
-
-  try {
-    const globalSettings = await settingsService.getGlobalSettings();
-    const credentials = await settingsService.getCredentials();
-    const globalPhaseModels = globalSettings.phaseModels || {};
-
-    // Start with global phase model
-    let phaseModel = globalPhaseModels[phase];
-    let isProjectOverride = false;
-
-    // Check for project override
-    if (projectPath) {
-      const projectSettings = await settingsService.getProjectSettings(projectPath);
-      const projectOverrides = projectSettings.phaseModelOverrides || {};
-
-      if (projectOverrides[phase]) {
-        phaseModel = projectOverrides[phase];
-        isProjectOverride = true;
-        logger.debug(`${logPrefix} Using project override for ${phase}`);
-      }
-    }
-
-    // If no phase model found, use a default
-    if (!phaseModel) {
-      phaseModel = { model: 'sonnet' };
-      logger.debug(`${logPrefix} No ${phase} configured, using default: sonnet`);
-    }
-
-    // Resolve provider if providerId is set
-    let provider: ClaudeCompatibleProvider | undefined;
-    if (phaseModel.providerId) {
-      const providers = globalSettings.claudeCompatibleProviders || [];
-      provider = providers.find((p) => p.id === phaseModel.providerId);
-
-      if (provider) {
-        if (provider.enabled === false) {
-          logger.warn(
-            `${logPrefix} Provider "${provider.name}" for ${phase} is disabled, falling back to direct API`
-          );
-          provider = undefined;
-        } else {
-          logger.debug(`${logPrefix} Using provider "${provider.name}" for ${phase}`);
-        }
-      } else {
-        logger.warn(
-          `${logPrefix} Provider ${phaseModel.providerId} not found for ${phase}, falling back to direct API`
-        );
-      }
-    }
-
-    return {
-      phaseModel,
-      isProjectOverride,
-      provider,
-      credentials,
-    };
-  } catch (error) {
-    logger.error(`${logPrefix} Failed to get phase model with overrides:`, error);
-    // Return a safe default
-    return {
-      phaseModel: { model: 'sonnet' },
-      isProjectOverride: false,
-      provider: undefined,
-      credentials: undefined,
-    };
-  }
-}
-
-/** Result from getProviderByModelId */
-export interface ProviderByModelIdResult {
-  /** The provider that contains this model, or undefined if not found */
-  provider: ClaudeCompatibleProvider | undefined;
-  /** The model configuration if found */
-  modelConfig: import('@automaker/types').ProviderModel | undefined;
-  /** Credentials for API key resolution */
-  credentials: Credentials | undefined;
-  /** The resolved Claude model ID to use for API calls (from mapsToClaudeModel) */
-  resolvedModel: string | undefined;
-}
-
-/**
- * Find a ClaudeCompatibleProvider by one of its model IDs.
- * Searches through all enabled providers to find one that contains the specified model.
- * This is useful when you have a model string from the UI but need the provider config.
- *
- * Also resolves the `mapsToClaudeModel` field to get the actual Claude model ID to use
- * when calling the API (e.g., "GLM-4.5-Air" -> "claude-haiku-4-5").
- *
- * @param modelId - The model ID to search for (e.g., "GLM-4.7", "MiniMax-M2.1")
- * @param settingsService - Settings service instance
- * @param logPrefix - Prefix for log messages
- * @returns Promise resolving to object with provider, model config, credentials, and resolved model
- */
-export async function getProviderByModelId(
-  modelId: string,
-  settingsService: SettingsService,
-  logPrefix = '[SettingsHelper]'
-): Promise<ProviderByModelIdResult> {
-  try {
-    const globalSettings = await settingsService.getGlobalSettings();
-    const credentials = await settingsService.getCredentials();
-    const providers = globalSettings.claudeCompatibleProviders || [];
-
-    // Search through all enabled providers for this model
-    for (const provider of providers) {
-      // Skip disabled providers
-      if (provider.enabled === false) {
-        continue;
-      }
-
-      // Check if this provider has the model
-      const modelConfig = provider.models?.find(
-        (m) => m.id === modelId || m.id.toLowerCase() === modelId.toLowerCase()
-      );
-
-      if (modelConfig) {
-        logger.info(`${logPrefix} Found model "${modelId}" in provider "${provider.name}"`);
-
-        // Resolve the mapped Claude model if specified
-        let resolvedModel: string | undefined;
-        if (modelConfig.mapsToClaudeModel) {
-          // Import resolveModelString to convert alias to full model ID
-          const { resolveModelString } = await import('@automaker/model-resolver');
-          resolvedModel = resolveModelString(modelConfig.mapsToClaudeModel);
-          logger.info(
-            `${logPrefix} Model "${modelId}" maps to Claude model "${modelConfig.mapsToClaudeModel}" -> "${resolvedModel}"`
-          );
-        }
-
-        return { provider, modelConfig, credentials, resolvedModel };
-      }
-    }
-
-    // Model not found in any provider
-    logger.debug(`${logPrefix} Model "${modelId}" not found in any provider`);
-    return {
-      provider: undefined,
-      modelConfig: undefined,
-      credentials: undefined,
-      resolvedModel: undefined,
-    };
-  } catch (error) {
-    logger.error(`${logPrefix} Failed to find provider by model ID:`, error);
-    return {
-      provider: undefined,
-      modelConfig: undefined,
-      credentials: undefined,
-      resolvedModel: undefined,
-    };
-  }
-}
-
-/**
- * Get all enabled provider models for use in model dropdowns.
- * Returns models from all enabled ClaudeCompatibleProviders.
- *
- * @param settingsService - Settings service instance
- * @param logPrefix - Prefix for log messages
- * @returns Promise resolving to array of provider models with their provider info
- */
-export async function getAllProviderModels(
-  settingsService: SettingsService,
-  logPrefix = '[SettingsHelper]'
-): Promise<
-  Array<{
-    providerId: string;
-    providerName: string;
-    model: import('@automaker/types').ProviderModel;
-  }>
-> {
-  try {
-    const globalSettings = await settingsService.getGlobalSettings();
-    const providers = globalSettings.claudeCompatibleProviders || [];
-
-    const allModels: Array<{
-      providerId: string;
-      providerName: string;
-      model: import('@automaker/types').ProviderModel;
-    }> = [];
-
-    for (const provider of providers) {
-      // Skip disabled providers
-      if (provider.enabled === false) {
-        continue;
-      }
-
-      for (const model of provider.models || []) {
-        allModels.push({
-          providerId: provider.id,
-          providerName: provider.name,
-          model,
-        });
-      }
-    }
-
-    logger.debug(
-      `${logPrefix} Found ${allModels.length} models from ${providers.length} providers`
-    );
-    return allModels;
-  } catch (error) {
-    logger.error(`${logPrefix} Failed to get all provider models:`, error);
-    return [];
-  }
-}

apps/server/src/lib/worktree-metadata.ts

@@ -5,14 +5,18 @@
 
 import * as secureFs from './secure-fs.js';
 import * as path from 'path';
-import type { PRState, WorktreePRInfo } from '@automaker/types';
-
-// Re-export types for backwards compatibility
-export type { PRState, WorktreePRInfo };
 
 /** Maximum length for sanitized branch names in filesystem paths */
 const MAX_SANITIZED_BRANCH_PATH_LENGTH = 200;
 
+export interface WorktreePRInfo {
+  number: number;
+  url: string;
+  title: string;
+  state: string;
+  createdAt: string;
+}
+
 export interface WorktreeMetadata {
   branch: string;
   createdAt: string;

apps/server/src/providers/claude-provider.ts

@@ -10,21 +10,7 @@ import { BaseProvider } from './base-provider.js';
 import { classifyError, getUserFriendlyErrorMessage, createLogger } from '@automaker/utils';
 
 const logger = createLogger('ClaudeProvider');
-import {
-  getThinkingTokenBudget,
-  validateBareModelId,
-  type ClaudeApiProfile,
-  type ClaudeCompatibleProvider,
-  type Credentials,
-} from '@automaker/types';
-
-/**
- * ProviderConfig - Union type for provider configuration
- *
- * Accepts either the legacy ClaudeApiProfile or new ClaudeCompatibleProvider.
- * Both share the same connection settings structure.
- */
-type ProviderConfig = ClaudeApiProfile | ClaudeCompatibleProvider;
+import { getThinkingTokenBudget, validateBareModelId } from '@automaker/types';
 import type {
   ExecuteOptions,
   ProviderMessage,
@@ -35,19 +21,9 @@ import type {
 // Explicit allowlist of environment variables to pass to the SDK.
 // Only these vars are passed - nothing else from process.env leaks through.
 const ALLOWED_ENV_VARS = [
-  // Authentication
   'ANTHROPIC_API_KEY',
-  'ANTHROPIC_AUTH_TOKEN',
-  // Endpoint configuration
   'ANTHROPIC_BASE_URL',
-  'API_TIMEOUT_MS',
-  // Model mappings
-  'ANTHROPIC_DEFAULT_HAIKU_MODEL',
-  'ANTHROPIC_DEFAULT_SONNET_MODEL',
-  'ANTHROPIC_DEFAULT_OPUS_MODEL',
-  // Traffic control
-  'CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC',
-  // System vars (always from process.env)
+  'ANTHROPIC_AUTH_TOKEN',
   'PATH',
   'HOME',
   'SHELL',
@@ -57,132 +33,16 @@ const ALLOWED_ENV_VARS = [
   'LC_ALL',
 ];
 
-// System vars are always passed from process.env regardless of profile
-const SYSTEM_ENV_VARS = ['PATH', 'HOME', 'SHELL', 'TERM', 'USER', 'LANG', 'LC_ALL'];
-
 /**
- * Check if the config is a ClaudeCompatibleProvider (new system)
- * by checking for the 'models' array property
+ * Build environment for the SDK with only explicitly allowed variables
  */
-function isClaudeCompatibleProvider(config: ProviderConfig): config is ClaudeCompatibleProvider {
-  return 'models' in config && Array.isArray(config.models);
-}
-
-/**
- * Build environment for the SDK with only explicitly allowed variables.
- * When a provider/profile is provided, uses its configuration (clean switch - don't inherit from process.env).
- * When no provider is provided, uses direct Anthropic API settings from process.env.
- *
- * Supports both:
- * - ClaudeCompatibleProvider (new system with models[] array)
- * - ClaudeApiProfile (legacy system with modelMappings)
- *
- * @param providerConfig - Optional provider configuration for alternative endpoint
- * @param credentials - Optional credentials object for resolving 'credentials' apiKeySource
- */
-function buildEnv(
-  providerConfig?: ProviderConfig,
-  credentials?: Credentials
-): Record<string, string | undefined> {
+function buildEnv(): Record<string, string | undefined> {
   const env: Record<string, string | undefined> = {};
-
-  if (providerConfig) {
-    // Use provider configuration (clean switch - don't inherit non-system vars from process.env)
-    logger.debug('[buildEnv] Using provider configuration:', {
-      name: providerConfig.name,
-      baseUrl: providerConfig.baseUrl,
-      apiKeySource: providerConfig.apiKeySource ?? 'inline',
-      isNewProvider: isClaudeCompatibleProvider(providerConfig),
-    });
-
-    // Resolve API key based on source strategy
-    let apiKey: string | undefined;
-    const source = providerConfig.apiKeySource ?? 'inline'; // Default to inline for backwards compat
-
-    switch (source) {
-      case 'inline':
-        apiKey = providerConfig.apiKey;
-        break;
-      case 'env':
-        apiKey = process.env.ANTHROPIC_API_KEY;
-        break;
-      case 'credentials':
-        apiKey = credentials?.apiKeys?.anthropic;
-        break;
-    }
-
-    // Warn if no API key found
-    if (!apiKey) {
-      logger.warn(`No API key found for provider "${providerConfig.name}" with source "${source}"`);
-    }
-
-    // Authentication
-    if (providerConfig.useAuthToken) {
-      env['ANTHROPIC_AUTH_TOKEN'] = apiKey;
-    } else {
-      env['ANTHROPIC_API_KEY'] = apiKey;
-    }
-
-    // Endpoint configuration
-    env['ANTHROPIC_BASE_URL'] = providerConfig.baseUrl;
-    logger.debug(`[buildEnv] Set ANTHROPIC_BASE_URL to: ${providerConfig.baseUrl}`);
-
-    if (providerConfig.timeoutMs) {
-      env['API_TIMEOUT_MS'] = String(providerConfig.timeoutMs);
-    }
-
-    // Model mappings - only for legacy ClaudeApiProfile
-    // For ClaudeCompatibleProvider, the model is passed directly (no mapping needed)
-    if (!isClaudeCompatibleProvider(providerConfig) && providerConfig.modelMappings) {
-      if (providerConfig.modelMappings.haiku) {
-        env['ANTHROPIC_DEFAULT_HAIKU_MODEL'] = providerConfig.modelMappings.haiku;
-      }
-      if (providerConfig.modelMappings.sonnet) {
-        env['ANTHROPIC_DEFAULT_SONNET_MODEL'] = providerConfig.modelMappings.sonnet;
-      }
-      if (providerConfig.modelMappings.opus) {
-        env['ANTHROPIC_DEFAULT_OPUS_MODEL'] = providerConfig.modelMappings.opus;
-      }
-    }
-
-    // Traffic control
-    if (providerConfig.disableNonessentialTraffic) {
-      env['CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC'] = '1';
-    }
-  } else {
-    // Use direct Anthropic API - pass through credentials or environment variables
-    // This supports:
-    // 1. API Key mode: ANTHROPIC_API_KEY from credentials (UI settings) or env
-    // 2. Claude Max plan: Uses CLI OAuth auth (SDK handles this automatically)
-    // 3. Custom endpoints via ANTHROPIC_BASE_URL env var (backward compatibility)
-    //
-    // Priority: credentials file (UI settings) -> environment variable
-    // Note: Only auth and endpoint vars are passed. Model mappings and traffic
-    // control are NOT passed (those require a profile for explicit configuration).
-    if (credentials?.apiKeys?.anthropic) {
-      env['ANTHROPIC_API_KEY'] = credentials.apiKeys.anthropic;
-    } else if (process.env.ANTHROPIC_API_KEY) {
-      env['ANTHROPIC_API_KEY'] = process.env.ANTHROPIC_API_KEY;
-    }
-    // If using Claude Max plan via CLI auth, the SDK handles auth automatically
-    // when no API key is provided. We don't set ANTHROPIC_AUTH_TOKEN here
-    // unless it was explicitly set in process.env (rare edge case).
-    if (process.env.ANTHROPIC_AUTH_TOKEN) {
-      env['ANTHROPIC_AUTH_TOKEN'] = process.env.ANTHROPIC_AUTH_TOKEN;
-    }
-    // Pass through ANTHROPIC_BASE_URL if set in environment (backward compatibility)
-    if (process.env.ANTHROPIC_BASE_URL) {
-      env['ANTHROPIC_BASE_URL'] = process.env.ANTHROPIC_BASE_URL;
-    }
-  }
-
-  // Always add system vars from process.env
-  for (const key of SYSTEM_ENV_VARS) {
+  for (const key of ALLOWED_ENV_VARS) {
     if (process.env[key]) {
       env[key] = process.env[key];
     }
   }
-
   return env;
 }
 
@@ -210,15 +70,8 @@ export class ClaudeProvider extends BaseProvider {
       conversationHistory,
       sdkSessionId,
       thinkingLevel,
-      claudeApiProfile,
-      claudeCompatibleProvider,
-      credentials,
     } = options;
 
-    // Determine which provider config to use
-    // claudeCompatibleProvider takes precedence over claudeApiProfile
-    const providerConfig = claudeCompatibleProvider || claudeApiProfile;
-
     // Convert thinking level to token budget
     const maxThinkingTokens = getThinkingTokenBudget(thinkingLevel);
 
@@ -229,9 +82,7 @@ export class ClaudeProvider extends BaseProvider {
       maxTurns,
       cwd,
       // Pass only explicitly allowed environment variables to SDK
-      // When a provider is active, uses provider settings (clean switch)
-      // When no provider, uses direct Anthropic API (from process.env or CLI OAuth)
-      env: buildEnv(providerConfig, credentials),
+      env: buildEnv(),
       // Pass through allowedTools if provided by caller (decided by sdk-options.ts)
       ...(allowedTools && { allowedTools }),
       // AUTONOMOUS MODE: Always bypass permissions for fully autonomous operation
@@ -276,18 +127,6 @@ export class ClaudeProvider extends BaseProvider {
       promptPayload = prompt;
     }
 
-    // Log the environment being passed to the SDK for debugging
-    const envForSdk = sdkOptions.env as Record<string, string | undefined>;
-    logger.debug('[ClaudeProvider] SDK Configuration:', {
-      model: sdkOptions.model,
-      baseUrl: envForSdk?.['ANTHROPIC_BASE_URL'] || '(default Anthropic API)',
-      hasApiKey: !!envForSdk?.['ANTHROPIC_API_KEY'],
-      hasAuthToken: !!envForSdk?.['ANTHROPIC_AUTH_TOKEN'],
-      providerName: providerConfig?.name || '(direct Anthropic)',
-      maxTurns: sdkOptions.maxTurns,
-      maxThinkingTokens: sdkOptions.maxThinkingTokens,
-    });
-
     // Execute via Claude Agent SDK
     try {
       const stream = query({ prompt: promptPayload, options: sdkOptions });

apps/server/src/providers/cursor-config-manager.ts

@@ -44,7 +44,7 @@ export class CursorConfigManager {
 
     // Return default config with all available models
     return {
-      defaultModel: 'cursor-auto',
+      defaultModel: 'auto',
       models: getAllCursorModelIds(),
     };
   }
@@ -77,7 +77,7 @@ export class CursorConfigManager {
    * Get the default model
    */
   getDefaultModel(): CursorModelId {
-    return this.config.defaultModel || 'cursor-auto';
+    return this.config.defaultModel || 'auto';
   }
 
   /**
@@ -93,7 +93,7 @@ export class CursorConfigManager {
    * Get enabled models
    */
   getEnabledModels(): CursorModelId[] {
-    return this.config.models || ['cursor-auto'];
+    return this.config.models || ['auto'];
   }
 
   /**
@@ -174,7 +174,7 @@ export class CursorConfigManager {
    */
   reset(): void {
     this.config = {
-      defaultModel: 'cursor-auto',
+      defaultModel: 'auto',
       models: getAllCursorModelIds(),
     };
     this.saveConfig();

apps/server/src/providers/opencode-provider.ts

@@ -25,6 +25,7 @@ import type {
   InstallationStatus,
   ContentBlock,
 } from '@automaker/types';
+import { stripProviderPrefix } from '@automaker/types';
 import { type SubprocessOptions, getOpenCodeAuthIndicators } from '@automaker/platform';
 import { createLogger } from '@automaker/utils';
 
@@ -327,18 +328,10 @@ export class OpencodeProvider extends CliProvider {
     args.push('--format', 'json');
 
     // Handle model selection
-    // Convert canonical prefix format (opencode-xxx) to CLI slash format (opencode/xxx)
-    // OpenCode CLI expects provider/model format (e.g., 'opencode/big-model')
+    // Strip 'opencode-' prefix if present, OpenCode uses format like 'anthropic/claude-sonnet-4-5'
     if (options.model) {
-      // Strip opencode- prefix if present, then ensure slash format
-      const model = options.model.startsWith('opencode-')
-        ? options.model.slice('opencode-'.length)
-        : options.model;
-
-      // If model has slash, it's already provider/model format; otherwise prepend opencode/
-      const cliModel = model.includes('/') ? model : `opencode/${model}`;
-
-      args.push('--model', cliModel);
+      const model = stripProviderPrefix(options.model);
+      args.push('--model', model);
     }
 
     // Note: OpenCode reads from stdin automatically when input is piped

apps/server/src/providers/simple-query-service.ts

@@ -20,9 +20,6 @@ import type {
   ContentBlock,
   ThinkingLevel,
   ReasoningEffort,
-  ClaudeApiProfile,
-  ClaudeCompatibleProvider,
-  Credentials,
 } from '@automaker/types';
 import { stripProviderPrefix } from '@automaker/types';
 
@@ -57,18 +54,6 @@ export interface SimpleQueryOptions {
   readOnly?: boolean;
   /** Setting sources for CLAUDE.md loading */
   settingSources?: Array<'user' | 'project' | 'local'>;
-  /**
-   * Active Claude API profile for alternative endpoint configuration
-   * @deprecated Use claudeCompatibleProvider instead
-   */
-  claudeApiProfile?: ClaudeApiProfile;
-  /**
-   * Claude-compatible provider for alternative endpoint configuration.
-   * Takes precedence over claudeApiProfile if both are set.
-   */
-  claudeCompatibleProvider?: ClaudeCompatibleProvider;
-  /** Credentials for resolving 'credentials' apiKeySource in Claude API profiles/providers */
-  credentials?: Credentials;
 }
 
 /**
@@ -140,9 +125,6 @@ export async function simpleQuery(options: SimpleQueryOptions): Promise<SimpleQu
     reasoningEffort: options.reasoningEffort,
     readOnly: options.readOnly,
     settingSources: options.settingSources,
-    claudeApiProfile: options.claudeApiProfile, // Legacy: Pass active Claude API profile for alternative endpoint configuration
-    claudeCompatibleProvider: options.claudeCompatibleProvider, // New: Pass Claude-compatible provider (takes precedence)
-    credentials: options.credentials, // Pass credentials for resolving 'credentials' apiKeySource
   };
 
   for await (const msg of provider.executeQuery(providerOptions)) {
@@ -225,9 +207,6 @@ export async function streamingQuery(options: StreamingQueryOptions): Promise<Si
     reasoningEffort: options.reasoningEffort,
     readOnly: options.readOnly,
     settingSources: options.settingSources,
-    claudeApiProfile: options.claudeApiProfile, // Legacy: Pass active Claude API profile for alternative endpoint configuration
-    claudeCompatibleProvider: options.claudeCompatibleProvider, // New: Pass Claude-compatible provider (takes precedence)
-    credentials: options.credentials, // Pass credentials for resolving 'credentials' apiKeySource
   };
 
   for await (const msg of provider.executeQuery(providerOptions)) {

apps/server/src/routes/app-spec/generate-features-from-spec.ts

@@ -14,11 +14,7 @@ import { streamingQuery } from '../../providers/simple-query-service.js';
 import { parseAndCreateFeatures } from './parse-and-create-features.js';
 import { getAppSpecPath } from '@automaker/platform';
 import type { SettingsService } from '../../services/settings-service.js';
-import {
-  getAutoLoadClaudeMdSetting,
-  getPromptCustomization,
-  getPhaseModelWithOverrides,
-} from '../../lib/settings-helpers.js';
+import { getAutoLoadClaudeMdSetting, getPromptCustomization } from '../../lib/settings-helpers.js';
 import { FeatureLoader } from '../../services/feature-loader.js';
 
 const logger = createLogger('SpecRegeneration');
@@ -119,26 +115,13 @@ Generate ${featureCount} NEW features that build on each other logically. Rememb
     '[FeatureGeneration]'
   );
 
-  // Get model from phase settings with provider info
-  const {
-    phaseModel: phaseModelEntry,
-    provider,
-    credentials,
-  } = settingsService
-    ? await getPhaseModelWithOverrides(
-        'featureGenerationModel',
-        settingsService,
-        projectPath,
-        '[FeatureGeneration]'
-      )
-    : {
-        phaseModel: DEFAULT_PHASE_MODELS.featureGenerationModel,
-        provider: undefined,
-        credentials: undefined,
-      };
+  // Get model from phase settings
+  const settings = await settingsService?.getGlobalSettings();
+  const phaseModelEntry =
+    settings?.phaseModels?.featureGenerationModel || DEFAULT_PHASE_MODELS.featureGenerationModel;
   const { model, thinkingLevel } = resolvePhaseModel(phaseModelEntry);
 
-  logger.info('Using model:', model, provider ? `via provider: ${provider.name}` : 'direct API');
+  logger.info('Using model:', model);
 
   // Use streamingQuery with event callbacks
   const result = await streamingQuery({
@@ -151,8 +134,6 @@ Generate ${featureCount} NEW features that build on each other logically. Rememb
     thinkingLevel,
     readOnly: true, // Feature generation only reads code, doesn't write
     settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-    claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-    credentials, // Pass credentials for resolving 'credentials' apiKeySource
     onText: (text) => {
       logger.debug(`Feature text block received (${text.length} chars)`);
       events.emit('spec-regeneration:event', {

apps/server/src/routes/app-spec/generate-spec.ts

@@ -16,11 +16,7 @@ import { streamingQuery } from '../../providers/simple-query-service.js';
 import { generateFeaturesFromSpec } from './generate-features-from-spec.js';
 import { ensureAutomakerDir, getAppSpecPath } from '@automaker/platform';
 import type { SettingsService } from '../../services/settings-service.js';
-import {
-  getAutoLoadClaudeMdSetting,
-  getPromptCustomization,
-  getPhaseModelWithOverrides,
-} from '../../lib/settings-helpers.js';
+import { getAutoLoadClaudeMdSetting, getPromptCustomization } from '../../lib/settings-helpers.js';
 
 const logger = createLogger('SpecRegeneration');
 
@@ -96,26 +92,13 @@ ${prompts.appSpec.structuredSpecInstructions}`;
     '[SpecRegeneration]'
   );
 
-  // Get model from phase settings with provider info
-  const {
-    phaseModel: phaseModelEntry,
-    provider,
-    credentials,
-  } = settingsService
-    ? await getPhaseModelWithOverrides(
-        'specGenerationModel',
-        settingsService,
-        projectPath,
-        '[SpecRegeneration]'
-      )
-    : {
-        phaseModel: DEFAULT_PHASE_MODELS.specGenerationModel,
-        provider: undefined,
-        credentials: undefined,
-      };
+  // Get model from phase settings
+  const settings = await settingsService?.getGlobalSettings();
+  const phaseModelEntry =
+    settings?.phaseModels?.specGenerationModel || DEFAULT_PHASE_MODELS.specGenerationModel;
   const { model, thinkingLevel } = resolvePhaseModel(phaseModelEntry);
 
-  logger.info('Using model:', model, provider ? `via provider: ${provider.name}` : 'direct API');
+  logger.info('Using model:', model);
 
   let responseText = '';
   let structuredOutput: SpecOutput | null = null;
@@ -149,8 +132,6 @@ Your entire response should be valid JSON starting with { and ending with }. No
     thinkingLevel,
     readOnly: true, // Spec generation only reads code, we write the spec ourselves
     settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-    claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-    credentials, // Pass credentials for resolving 'credentials' apiKeySource
     outputFormat: useStructuredOutput
       ? {
           type: 'json_schema',

apps/server/src/routes/app-spec/sync-spec.ts

@@ -15,10 +15,7 @@ import { resolvePhaseModel } from '@automaker/model-resolver';
 import { streamingQuery } from '../../providers/simple-query-service.js';
 import { getAppSpecPath } from '@automaker/platform';
 import type { SettingsService } from '../../services/settings-service.js';
-import {
-  getAutoLoadClaudeMdSetting,
-  getPhaseModelWithOverrides,
-} from '../../lib/settings-helpers.js';
+import { getAutoLoadClaudeMdSetting } from '../../lib/settings-helpers.js';
 import { FeatureLoader } from '../../services/feature-loader.js';
 import {
   extractImplementedFeatures,
@@ -155,27 +152,11 @@ export async function syncSpec(
     '[SpecSync]'
   );
 
-  // Get model from phase settings with provider info
-  const {
-    phaseModel: phaseModelEntry,
-    provider,
-    credentials,
-  } = settingsService
-    ? await getPhaseModelWithOverrides(
-        'specGenerationModel',
-        settingsService,
-        projectPath,
-        '[SpecSync]'
-      )
-    : {
-        phaseModel: DEFAULT_PHASE_MODELS.specGenerationModel,
-        provider: undefined,
-        credentials: undefined,
-      };
+  const settings = await settingsService?.getGlobalSettings();
+  const phaseModelEntry =
+    settings?.phaseModels?.specGenerationModel || DEFAULT_PHASE_MODELS.specGenerationModel;
   const { model, thinkingLevel } = resolvePhaseModel(phaseModelEntry);
 
-  logger.info('Using model:', model, provider ? `via provider: ${provider.name}` : 'direct API');
-
   // Use AI to analyze tech stack
   const techAnalysisPrompt = `Analyze this project and return ONLY a JSON object with the current technology stack.
 
@@ -204,8 +185,6 @@ Return ONLY this JSON format, no other text:
       thinkingLevel,
       readOnly: true,
       settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-      claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-      credentials, // Pass credentials for resolving 'credentials' apiKeySource
       onText: (text) => {
         logger.debug(`Tech analysis text: ${text.substring(0, 100)}`);
       },

apps/server/src/routes/auth/index.ts

@@ -117,27 +117,9 @@ export function createAuthRoutes(): Router {
    *
    * Returns whether the current request is authenticated.
    * Used by the UI to determine if login is needed.
-   *
-   * If AUTOMAKER_AUTO_LOGIN=true is set, automatically creates a session
-   * for unauthenticated requests (useful for development).
    */
-  router.get('/status', async (req, res) => {
-    let authenticated = isRequestAuthenticated(req);
-
-    // Auto-login for development: create session automatically if enabled
-    // Only works in non-production environments as a safeguard
-    if (
-      !authenticated &&
-      process.env.AUTOMAKER_AUTO_LOGIN === 'true' &&
-      process.env.NODE_ENV !== 'production'
-    ) {
-      const sessionToken = await createSession();
-      const cookieOptions = getSessionCookieOptions();
-      const cookieName = getSessionCookieName();
-      res.cookie(cookieName, sessionToken, cookieOptions);
-      authenticated = true;
-    }
-
+  router.get('/status', (req, res) => {
+    const authenticated = isRequestAuthenticated(req);
     res.json({
       success: true,
       authenticated,

apps/server/src/routes/auto-mode/index.ts

@@ -10,8 +10,6 @@ import { validatePathParams } from '../../middleware/validate-paths.js';
 import { createStopFeatureHandler } from './routes/stop-feature.js';
 import { createStatusHandler } from './routes/status.js';
 import { createRunFeatureHandler } from './routes/run-feature.js';
-import { createStartHandler } from './routes/start.js';
-import { createStopHandler } from './routes/stop.js';
 import { createVerifyFeatureHandler } from './routes/verify-feature.js';
 import { createResumeFeatureHandler } from './routes/resume-feature.js';
 import { createContextExistsHandler } from './routes/context-exists.js';
@@ -24,10 +22,6 @@ import { createResumeInterruptedHandler } from './routes/resume-interrupted.js';
 export function createAutoModeRoutes(autoModeService: AutoModeService): Router {
   const router = Router();
 
-  // Auto loop control routes
-  router.post('/start', validatePathParams('projectPath'), createStartHandler(autoModeService));
-  router.post('/stop', validatePathParams('projectPath'), createStopHandler(autoModeService));
-
   router.post('/stop-feature', createStopFeatureHandler(autoModeService));
   router.post('/status', validatePathParams('projectPath?'), createStatusHandler(autoModeService));
   router.post(

apps/server/src/routes/auto-mode/routes/run-feature.ts

@@ -26,24 +26,6 @@ export function createRunFeatureHandler(autoModeService: AutoModeService) {
         return;
       }
 
-      // Check per-worktree capacity before starting
-      const capacity = await autoModeService.checkWorktreeCapacity(projectPath, featureId);
-      if (!capacity.hasCapacity) {
-        const worktreeDesc = capacity.branchName
-          ? `worktree "${capacity.branchName}"`
-          : 'main worktree';
-        res.status(429).json({
-          success: false,
-          error: `Agent limit reached for ${worktreeDesc} (${capacity.currentAgents}/${capacity.maxAgents}). Wait for running tasks to complete or increase the limit.`,
-          details: {
-            currentAgents: capacity.currentAgents,
-            maxAgents: capacity.maxAgents,
-            branchName: capacity.branchName,
-          },
-        });
-        return;
-      }
-
       // Start execution in background
       // executeFeature derives workDir from feature.branchName
       autoModeService

apps/server/src/routes/auto-mode/routes/start.ts

@@ -1,67 +0,0 @@
-/**
- * POST /start endpoint - Start auto mode loop for a project
- */
-
-import type { Request, Response } from 'express';
-import type { AutoModeService } from '../../../services/auto-mode-service.js';
-import { createLogger } from '@automaker/utils';
-import { getErrorMessage, logError } from '../common.js';
-
-const logger = createLogger('AutoMode');
-
-export function createStartHandler(autoModeService: AutoModeService) {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { projectPath, branchName, maxConcurrency } = req.body as {
-        projectPath: string;
-        branchName?: string | null;
-        maxConcurrency?: number;
-      };
-
-      if (!projectPath) {
-        res.status(400).json({
-          success: false,
-          error: 'projectPath is required',
-        });
-        return;
-      }
-
-      // Normalize branchName: undefined becomes null
-      const normalizedBranchName = branchName ?? null;
-      const worktreeDesc = normalizedBranchName
-        ? `worktree ${normalizedBranchName}`
-        : 'main worktree';
-
-      // Check if already running
-      if (autoModeService.isAutoLoopRunningForProject(projectPath, normalizedBranchName)) {
-        res.json({
-          success: true,
-          message: `Auto mode is already running for ${worktreeDesc}`,
-          alreadyRunning: true,
-          branchName: normalizedBranchName,
-        });
-        return;
-      }
-
-      // Start the auto loop for this project/worktree
-      const resolvedMaxConcurrency = await autoModeService.startAutoLoopForProject(
-        projectPath,
-        normalizedBranchName,
-        maxConcurrency
-      );
-
-      logger.info(
-        `Started auto loop for ${worktreeDesc} in project: ${projectPath} with maxConcurrency: ${resolvedMaxConcurrency}`
-      );
-
-      res.json({
-        success: true,
-        message: `Auto mode started with max ${resolvedMaxConcurrency} concurrent features`,
-        branchName: normalizedBranchName,
-      });
-    } catch (error) {
-      logError(error, 'Start auto mode failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}

apps/server/src/routes/auto-mode/routes/status.ts

@@ -1,8 +1,5 @@
 /**
  * POST /status endpoint - Get auto mode status
- *
- * If projectPath is provided, returns per-project status including autoloop state.
- * If no projectPath, returns global status for backward compatibility.
  */
 
 import type { Request, Response } from 'express';
@@ -12,41 +9,10 @@ import { getErrorMessage, logError } from '../common.js';
 export function createStatusHandler(autoModeService: AutoModeService) {
   return async (req: Request, res: Response): Promise<void> => {
     try {
-      const { projectPath, branchName } = req.body as {
-        projectPath?: string;
-        branchName?: string | null;
-      };
-
-      // If projectPath is provided, return per-project/worktree status
-      if (projectPath) {
-        // Normalize branchName: undefined becomes null
-        const normalizedBranchName = branchName ?? null;
-        const projectStatus = autoModeService.getStatusForProject(
-          projectPath,
-          normalizedBranchName
-        );
-        res.json({
-          success: true,
-          isRunning: projectStatus.runningCount > 0,
-          isAutoLoopRunning: projectStatus.isAutoLoopRunning,
-          runningFeatures: projectStatus.runningFeatures,
-          runningCount: projectStatus.runningCount,
-          maxConcurrency: projectStatus.maxConcurrency,
-          projectPath,
-          branchName: normalizedBranchName,
-        });
-        return;
-      }
-
-      // Fall back to global status for backward compatibility
       const status = autoModeService.getStatus();
-      const activeProjects = autoModeService.getActiveAutoLoopProjects();
-      const activeWorktrees = autoModeService.getActiveAutoLoopWorktrees();
       res.json({
         success: true,
         ...status,
-        activeAutoLoopProjects: activeProjects,
-        activeAutoLoopWorktrees: activeWorktrees,
       });
     } catch (error) {
       logError(error, 'Get status failed');

apps/server/src/routes/auto-mode/routes/stop.ts

@@ -1,66 +0,0 @@
-/**
- * POST /stop endpoint - Stop auto mode loop for a project
- */
-
-import type { Request, Response } from 'express';
-import type { AutoModeService } from '../../../services/auto-mode-service.js';
-import { createLogger } from '@automaker/utils';
-import { getErrorMessage, logError } from '../common.js';
-
-const logger = createLogger('AutoMode');
-
-export function createStopHandler(autoModeService: AutoModeService) {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { projectPath, branchName } = req.body as {
-        projectPath: string;
-        branchName?: string | null;
-      };
-
-      if (!projectPath) {
-        res.status(400).json({
-          success: false,
-          error: 'projectPath is required',
-        });
-        return;
-      }
-
-      // Normalize branchName: undefined becomes null
-      const normalizedBranchName = branchName ?? null;
-      const worktreeDesc = normalizedBranchName
-        ? `worktree ${normalizedBranchName}`
-        : 'main worktree';
-
-      // Check if running
-      if (!autoModeService.isAutoLoopRunningForProject(projectPath, normalizedBranchName)) {
-        res.json({
-          success: true,
-          message: `Auto mode is not running for ${worktreeDesc}`,
-          wasRunning: false,
-          branchName: normalizedBranchName,
-        });
-        return;
-      }
-
-      // Stop the auto loop for this project/worktree
-      const runningCount = await autoModeService.stopAutoLoopForProject(
-        projectPath,
-        normalizedBranchName
-      );
-
-      logger.info(
-        `Stopped auto loop for ${worktreeDesc} in project: ${projectPath}, ${runningCount} features still running`
-      );
-
-      res.json({
-        success: true,
-        message: 'Auto mode stopped',
-        runningFeaturesCount: runningCount,
-        branchName: normalizedBranchName,
-      });
-    } catch (error) {
-      logError(error, 'Stop auto mode failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}

apps/server/src/routes/backlog-plan/common.ts

@@ -100,60 +100,11 @@ export function getAbortController(): AbortController | null {
   return currentAbortController;
 }
 
-/**
- * Map SDK/CLI errors to user-friendly messages
- */
-export function mapBacklogPlanError(rawMessage: string): string {
-  // Claude Code spawn failures
-  if (
-    rawMessage.includes('Failed to spawn Claude Code process') ||
-    rawMessage.includes('spawn node ENOENT') ||
-    rawMessage.includes('Claude Code executable not found') ||
-    rawMessage.includes('Claude Code native binary not found')
-  ) {
-    return 'Claude CLI could not be launched. Make sure the Claude CLI is installed and available in PATH, or check that Node.js is correctly installed. Try running "which claude" or "claude --version" in your terminal to verify.';
-  }
-
-  // Claude Code process crash
-  if (rawMessage.includes('Claude Code process exited')) {
-    return 'Claude exited unexpectedly. Try again. If it keeps happening, re-run `claude login` or update your API key in Setup.';
-  }
-
-  // Rate limiting
-  if (rawMessage.toLowerCase().includes('rate limit') || rawMessage.includes('429')) {
-    return 'Rate limited. Please wait a moment and try again.';
-  }
-
-  // Network errors
-  if (
-    rawMessage.toLowerCase().includes('network') ||
-    rawMessage.toLowerCase().includes('econnrefused') ||
-    rawMessage.toLowerCase().includes('timeout')
-  ) {
-    return 'Network error. Check your internet connection and try again.';
-  }
-
-  // Authentication errors
-  if (
-    rawMessage.toLowerCase().includes('not authenticated') ||
-    rawMessage.toLowerCase().includes('unauthorized') ||
-    rawMessage.includes('401')
-  ) {
-    return 'Authentication failed. Please check your API key or run `claude login` to authenticate.';
-  }
-
-  // Return original message for unknown errors
-  return rawMessage;
-}
-
 export function getErrorMessage(error: unknown): string {
-  let rawMessage: string;
   if (error instanceof Error) {
-    rawMessage = error.message;
-  } else {
-    rawMessage = String(error);
+    return error.message;
   }
-  return mapBacklogPlanError(rawMessage);
+  return String(error);
 }
 
 export function logError(error: unknown, context: string): void {

apps/server/src/routes/backlog-plan/generate-plan.ts

@@ -25,11 +25,7 @@ import {
   saveBacklogPlan,
 } from './common.js';
 import type { SettingsService } from '../../services/settings-service.js';
-import {
-  getAutoLoadClaudeMdSetting,
-  getPromptCustomization,
-  getPhaseModelWithOverrides,
-} from '../../lib/settings-helpers.js';
+import { getAutoLoadClaudeMdSetting, getPromptCustomization } from '../../lib/settings-helpers.js';
 
 const featureLoader = new FeatureLoader();
 
@@ -121,39 +117,18 @@ export async function generateBacklogPlan(
       content: 'Generating plan with AI...',
     });
 
-    // Get the model to use from settings or provided override with provider info
+    // Get the model to use from settings or provided override
     let effectiveModel = model;
     let thinkingLevel: ThinkingLevel | undefined;
-    let claudeCompatibleProvider: import('@automaker/types').ClaudeCompatibleProvider | undefined;
-    let credentials: import('@automaker/types').Credentials | undefined;
-
-    if (effectiveModel) {
-      // Use explicit override - just get credentials
-      credentials = await settingsService?.getCredentials();
-    } else if (settingsService) {
-      // Use settings-based model with provider info
-      const phaseResult = await getPhaseModelWithOverrides(
-        'backlogPlanningModel',
-        settingsService,
-        projectPath,
-        '[BacklogPlan]'
-      );
-      const resolved = resolvePhaseModel(phaseResult.phaseModel);
-      effectiveModel = resolved.model;
-      thinkingLevel = resolved.thinkingLevel;
-      claudeCompatibleProvider = phaseResult.provider;
-      credentials = phaseResult.credentials;
-    } else {
-      // Fallback to defaults
-      const resolved = resolvePhaseModel(DEFAULT_PHASE_MODELS.backlogPlanningModel);
+    if (!effectiveModel) {
+      const settings = await settingsService?.getGlobalSettings();
+      const phaseModelEntry =
+        settings?.phaseModels?.backlogPlanningModel || DEFAULT_PHASE_MODELS.backlogPlanningModel;
+      const resolved = resolvePhaseModel(phaseModelEntry);
       effectiveModel = resolved.model;
       thinkingLevel = resolved.thinkingLevel;
     }
-    logger.info(
-      '[BacklogPlan] Using model:',
-      effectiveModel,
-      claudeCompatibleProvider ? `via provider: ${claudeCompatibleProvider.name}` : 'direct API'
-    );
+    logger.info('[BacklogPlan] Using model:', effectiveModel);
 
     const provider = ProviderFactory.getProviderForModel(effectiveModel);
     // Strip provider prefix - providers expect bare model IDs
@@ -198,8 +173,6 @@ ${userPrompt}`;
       settingSources: autoLoadClaudeMd ? ['user', 'project'] : undefined,
       readOnly: true, // Plan generation only generates text, doesn't write files
       thinkingLevel, // Pass thinking level for extended thinking
-      claudeCompatibleProvider, // Pass provider for alternative endpoint configuration
-      credentials, // Pass credentials for resolving 'credentials' apiKeySource
     });
 
     let responseText = '';

apps/server/src/routes/backlog-plan/routes/apply.ts

@@ -85,9 +85,8 @@ export function createApplyHandler() {
         if (!change.feature) continue;
 
         try {
-          // Create the new feature - use the AI-generated ID if provided
+          // Create the new feature
           const newFeature = await featureLoader.create(projectPath, {
-            id: change.feature.id, // Use descriptive ID from AI if provided
             title: change.feature.title,
             description: change.feature.description || '',
             category: change.feature.category || 'Uncategorized',

apps/server/src/routes/backlog-plan/routes/generate.ts

@@ -53,12 +53,13 @@ export function createGenerateHandler(events: EventEmitter, settingsService?: Se
       setRunningState(true, abortController);
 
       // Start generation in background
-      // Note: generateBacklogPlan handles its own error event emission,
-      // so we only log here to avoid duplicate error toasts
       generateBacklogPlan(projectPath, prompt, events, abortController, settingsService, model)
         .catch((error) => {
-          // Just log - error event already emitted by generateBacklogPlan
           logError(error, 'Generate backlog plan failed (background)');
+          events.emit('backlog-plan:event', {
+            type: 'backlog_plan_error',
+            error: getErrorMessage(error),
+          });
         })
         .finally(() => {
           setRunningState(false, null);

apps/server/src/routes/context/routes/describe-file.ts

@@ -12,6 +12,7 @@
 
 import type { Request, Response } from 'express';
 import { createLogger } from '@automaker/utils';
+import { DEFAULT_PHASE_MODELS } from '@automaker/types';
 import { PathNotAllowedError } from '@automaker/platform';
 import { resolvePhaseModel } from '@automaker/model-resolver';
 import { simpleQuery } from '../../../providers/simple-query-service.js';
@@ -21,7 +22,6 @@ import type { SettingsService } from '../../../services/settings-service.js';
 import {
   getAutoLoadClaudeMdSetting,
   getPromptCustomization,
-  getPhaseModelWithOverrides,
 } from '../../../lib/settings-helpers.js';
 
 const logger = createLogger('DescribeFile');
@@ -155,23 +155,15 @@ ${contentToAnalyze}`;
         '[DescribeFile]'
       );
 
-      // Get model from phase settings with provider info
-      const {
-        phaseModel: phaseModelEntry,
-        provider,
-        credentials,
-      } = await getPhaseModelWithOverrides(
-        'fileDescriptionModel',
-        settingsService,
-        cwd,
-        '[DescribeFile]'
-      );
+      // Get model from phase settings
+      const settings = await settingsService?.getGlobalSettings();
+      logger.info(`Raw phaseModels from settings:`, JSON.stringify(settings?.phaseModels, null, 2));
+      const phaseModelEntry =
+        settings?.phaseModels?.fileDescriptionModel || DEFAULT_PHASE_MODELS.fileDescriptionModel;
+      logger.info(`fileDescriptionModel entry:`, JSON.stringify(phaseModelEntry));
       const { model, thinkingLevel } = resolvePhaseModel(phaseModelEntry);
 
-      logger.info(
-        `Resolved model: ${model}, thinkingLevel: ${thinkingLevel}`,
-        provider ? `via provider: ${provider.name}` : 'direct API'
-      );
+      logger.info(`Resolved model: ${model}, thinkingLevel: ${thinkingLevel}`);
 
       // Use simpleQuery - provider abstraction handles routing to correct provider
       const result = await simpleQuery({
@@ -183,8 +175,6 @@ ${contentToAnalyze}`;
         thinkingLevel,
         readOnly: true, // File description only reads, doesn't write
         settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-        claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
       });
 
       const description = result.text;

apps/server/src/routes/context/routes/describe-image.ts

@@ -13,7 +13,7 @@
 
 import type { Request, Response } from 'express';
 import { createLogger, readImageAsBase64 } from '@automaker/utils';
-import { isCursorModel } from '@automaker/types';
+import { DEFAULT_PHASE_MODELS, isCursorModel } from '@automaker/types';
 import { resolvePhaseModel } from '@automaker/model-resolver';
 import { simpleQuery } from '../../../providers/simple-query-service.js';
 import * as secureFs from '../../../lib/secure-fs.js';
@@ -22,7 +22,6 @@ import type { SettingsService } from '../../../services/settings-service.js';
 import {
   getAutoLoadClaudeMdSetting,
   getPromptCustomization,
-  getPhaseModelWithOverrides,
 } from '../../../lib/settings-helpers.js';
 
 const logger = createLogger('DescribeImage');
@@ -274,23 +273,13 @@ export function createDescribeImageHandler(
         '[DescribeImage]'
       );
 
-      // Get model from phase settings with provider info
-      const {
-        phaseModel: phaseModelEntry,
-        provider,
-        credentials,
-      } = await getPhaseModelWithOverrides(
-        'imageDescriptionModel',
-        settingsService,
-        cwd,
-        '[DescribeImage]'
-      );
+      // Get model from phase settings
+      const settings = await settingsService?.getGlobalSettings();
+      const phaseModelEntry =
+        settings?.phaseModels?.imageDescriptionModel || DEFAULT_PHASE_MODELS.imageDescriptionModel;
       const { model, thinkingLevel } = resolvePhaseModel(phaseModelEntry);
 
-      logger.info(
-        `[${requestId}] Using model: ${model}`,
-        provider ? `via provider: ${provider.name}` : 'direct API'
-      );
+      logger.info(`[${requestId}] Using model: ${model}`);
 
       // Get customized prompts from settings
       const prompts = await getPromptCustomization(settingsService, '[DescribeImage]');
@@ -336,8 +325,6 @@ export function createDescribeImageHandler(
         thinkingLevel,
         readOnly: true, // Image description only reads, doesn't write
         settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-        claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
       });
 
       logger.info(`[${requestId}] simpleQuery completed in ${Date.now() - queryStart}ms`);

apps/server/src/routes/enhance-prompt/routes/enhance.ts

@@ -12,7 +12,7 @@ import { resolveModelString } from '@automaker/model-resolver';
 import { CLAUDE_MODEL_MAP, type ThinkingLevel } from '@automaker/types';
 import { simpleQuery } from '../../../providers/simple-query-service.js';
 import type { SettingsService } from '../../../services/settings-service.js';
-import { getPromptCustomization, getProviderByModelId } from '../../../lib/settings-helpers.js';
+import { getPromptCustomization } from '../../../lib/settings-helpers.js';
 import {
   buildUserPrompt,
   isValidEnhancementMode,
@@ -33,8 +33,6 @@ interface EnhanceRequestBody {
   model?: string;
   /** Optional thinking level for Claude models */
   thinkingLevel?: ThinkingLevel;
-  /** Optional project path for per-project Claude API profile */
-  projectPath?: string;
 }
 
 /**
@@ -64,7 +62,7 @@ export function createEnhanceHandler(
 ): (req: Request, res: Response) => Promise<void> {
   return async (req: Request, res: Response): Promise<void> => {
     try {
-      const { originalText, enhancementMode, model, thinkingLevel, projectPath } =
+      const { originalText, enhancementMode, model, thinkingLevel } =
         req.body as EnhanceRequestBody;
 
       // Validate required fields
@@ -123,32 +121,8 @@ export function createEnhanceHandler(
       // Build the user prompt with few-shot examples
       const userPrompt = buildUserPrompt(validMode, trimmedText, true);
 
-      // Check if the model is a provider model (like "GLM-4.5-Air")
-      // If so, get the provider config and resolved Claude model
-      let claudeCompatibleProvider: import('@automaker/types').ClaudeCompatibleProvider | undefined;
-      let providerResolvedModel: string | undefined;
-      let credentials = await settingsService?.getCredentials();
-
-      if (model && settingsService) {
-        const providerResult = await getProviderByModelId(
-          model,
-          settingsService,
-          '[EnhancePrompt]'
-        );
-        if (providerResult.provider) {
-          claudeCompatibleProvider = providerResult.provider;
-          providerResolvedModel = providerResult.resolvedModel;
-          credentials = providerResult.credentials;
-          logger.info(
-            `Using provider "${providerResult.provider.name}" for model "${model}"` +
-              (providerResolvedModel ? ` -> resolved to "${providerResolvedModel}"` : '')
-          );
-        }
-      }
-
-      // Resolve the model - use provider resolved model, passed model, or default to sonnet
-      const resolvedModel =
-        providerResolvedModel || resolveModelString(model, CLAUDE_MODEL_MAP.sonnet);
+      // Resolve the model - use the passed model, default to sonnet for quality
+      const resolvedModel = resolveModelString(model, CLAUDE_MODEL_MAP.sonnet);
 
       logger.debug(`Using model: ${resolvedModel}`);
 
@@ -163,8 +137,6 @@ export function createEnhanceHandler(
         allowedTools: [],
         thinkingLevel,
         readOnly: true, // Prompt enhancement only generates text, doesn't write files
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
-        claudeCompatibleProvider, // Pass provider for alternative endpoint configuration
       });
 
       const enhancedText = result.text;

apps/server/src/routes/features/routes/generate-title.ts

@@ -16,7 +16,6 @@ const logger = createLogger('GenerateTitle');
 
 interface GenerateTitleRequestBody {
   description: string;
-  projectPath?: string;
 }
 
 interface GenerateTitleSuccessResponse {
@@ -34,7 +33,7 @@ export function createGenerateTitleHandler(
 ): (req: Request, res: Response) => Promise<void> {
   return async (req: Request, res: Response): Promise<void> => {
     try {
-      const { description, projectPath } = req.body as GenerateTitleRequestBody;
+      const { description } = req.body as GenerateTitleRequestBody;
 
       if (!description || typeof description !== 'string') {
         const response: GenerateTitleErrorResponse = {
@@ -61,9 +60,6 @@ export function createGenerateTitleHandler(
       const prompts = await getPromptCustomization(settingsService, '[GenerateTitle]');
       const systemPrompt = prompts.titleGeneration.systemPrompt;
 
-      // Get credentials for API calls (uses hardcoded haiku model, no phase setting)
-      const credentials = await settingsService?.getCredentials();
-
       const userPrompt = `Generate a concise title for this feature:\n\n${trimmedDescription}`;
 
       // Use simpleQuery - provider abstraction handles all the streaming/extraction
@@ -73,7 +69,6 @@ export function createGenerateTitleHandler(
         cwd: process.cwd(),
         maxTurns: 1,
         allowedTools: [],
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
       });
 
       const title = result.text;

apps/server/src/routes/fs/routes/image.ts

@@ -1,12 +1,5 @@
 /**
  * GET /image endpoint - Serve image files
- *
- * Requires authentication via auth middleware:
- * - apiKey query parameter (Electron mode)
- * - token query parameter (web mode)
- * - session cookie (web mode)
- * - X-API-Key header (Electron mode)
- * - X-Session-Token header (web mode)
  */
 
 import type { Request, Response } from 'express';

apps/server/src/routes/github/routes/validate-issue.ts

@@ -34,11 +34,7 @@ import {
   ValidationComment,
   ValidationLinkedPR,
 } from './validation-schema.js';
-import {
-  getPromptCustomization,
-  getAutoLoadClaudeMdSetting,
-  getProviderByModelId,
-} from '../../../lib/settings-helpers.js';
+import { getPromptCustomization } from '../../../lib/settings-helpers.js';
 import {
   trySetValidationRunning,
   clearValidationStatus,
@@ -47,6 +43,7 @@ import {
   logger,
 } from './validation-common.js';
 import type { SettingsService } from '../../../services/settings-service.js';
+import { getAutoLoadClaudeMdSetting } from '../../../lib/settings-helpers.js';
 
 /**
  * Request body for issue validation
@@ -167,33 +164,12 @@ ${basePrompt}`;
       }
     }
 
-    // Check if the model is a provider model (like "GLM-4.5-Air")
-    // If so, get the provider config and resolved Claude model
-    let claudeCompatibleProvider: import('@automaker/types').ClaudeCompatibleProvider | undefined;
-    let providerResolvedModel: string | undefined;
-    let credentials = await settingsService?.getCredentials();
-
-    if (settingsService) {
-      const providerResult = await getProviderByModelId(model, settingsService, '[ValidateIssue]');
-      if (providerResult.provider) {
-        claudeCompatibleProvider = providerResult.provider;
-        providerResolvedModel = providerResult.resolvedModel;
-        credentials = providerResult.credentials;
-        logger.info(
-          `Using provider "${providerResult.provider.name}" for model "${model}"` +
-            (providerResolvedModel ? ` -> resolved to "${providerResolvedModel}"` : '')
-        );
-      }
-    }
-
-    // Use provider resolved model if available, otherwise use original model
-    const effectiveModel = providerResolvedModel || (model as string);
-    logger.info(`Using model: ${effectiveModel}`);
+    logger.info(`Using model: ${model}`);
 
     // Use streamingQuery with event callbacks
     const result = await streamingQuery({
       prompt: finalPrompt,
-      model: effectiveModel,
+      model: model as string,
       cwd: projectPath,
       systemPrompt: useStructuredOutput ? issueValidationSystemPrompt : undefined,
       abortController,
@@ -201,8 +177,6 @@ ${basePrompt}`;
       reasoningEffort: effectiveReasoningEffort,
       readOnly: true, // Issue validation only reads code, doesn't write
       settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-      claudeCompatibleProvider, // Pass provider for alternative endpoint configuration
-      credentials, // Pass credentials for resolving 'credentials' apiKeySource
       outputFormat: useStructuredOutput
         ? {
             type: 'json_schema',

apps/server/src/routes/settings/routes/update-global.ts

@@ -45,24 +45,18 @@ export function createUpdateGlobalHandler(settingsService: SettingsService) {
       }
 
       // Minimal debug logging to help diagnose accidental wipes.
-      const projectsLen = Array.isArray((updates as any).projects)
-        ? (updates as any).projects.length
-        : undefined;
-      const trashedLen = Array.isArray((updates as any).trashedProjects)
-        ? (updates as any).trashedProjects.length
-        : undefined;
-      logger.info(
-        `[SERVER_SETTINGS_UPDATE] Request received: projects=${projectsLen ?? 'n/a'}, trashedProjects=${trashedLen ?? 'n/a'}, theme=${
-          (updates as any).theme ?? 'n/a'
-        }, localStorageMigrated=${(updates as any).localStorageMigrated ?? 'n/a'}`
-      );
+      if ('projects' in updates || 'theme' in updates || 'localStorageMigrated' in updates) {
+        const projectsLen = Array.isArray((updates as any).projects)
+          ? (updates as any).projects.length
+          : undefined;
+        logger.info(
+          `Update global settings request: projects=${projectsLen ?? 'n/a'}, theme=${
+            (updates as any).theme ?? 'n/a'
+          }, localStorageMigrated=${(updates as any).localStorageMigrated ?? 'n/a'}`
+        );
+      }
 
-      logger.info('[SERVER_SETTINGS_UPDATE] Calling updateGlobalSettings...');
       const settings = await settingsService.updateGlobalSettings(updates);
-      logger.info(
-        '[SERVER_SETTINGS_UPDATE] Update complete, projects count:',
-        settings.projects?.length ?? 0
-      );
 
       // Apply server log level if it was updated
       if ('serverLogLevel' in updates && updates.serverLogLevel) {

apps/server/src/routes/suggestions/generate-suggestions.ts

@@ -15,11 +15,7 @@ import { FeatureLoader } from '../../services/feature-loader.js';
 import { getAppSpecPath } from '@automaker/platform';
 import * as secureFs from '../../lib/secure-fs.js';
 import type { SettingsService } from '../../services/settings-service.js';
-import {
-  getAutoLoadClaudeMdSetting,
-  getPromptCustomization,
-  getPhaseModelWithOverrides,
-} from '../../lib/settings-helpers.js';
+import { getAutoLoadClaudeMdSetting, getPromptCustomization } from '../../lib/settings-helpers.js';
 
 const logger = createLogger('Suggestions');
 
@@ -171,12 +167,11 @@ ${prompts.suggestions.baseTemplate}`;
     '[Suggestions]'
   );
 
-  // Get model from phase settings with provider info (AI Suggestions = suggestionsModel)
+  // Get model from phase settings (AI Suggestions = suggestionsModel)
   // Use override if provided, otherwise fall back to settings
+  const settings = await settingsService?.getGlobalSettings();
   let model: string;
   let thinkingLevel: ThinkingLevel | undefined;
-  let provider: import('@automaker/types').ClaudeCompatibleProvider | undefined;
-  let credentials: import('@automaker/types').Credentials | undefined;
 
   if (modelOverride) {
     // Use explicit override - resolve the model string
@@ -186,33 +181,16 @@ ${prompts.suggestions.baseTemplate}`;
     });
     model = resolved.model;
     thinkingLevel = resolved.thinkingLevel;
-    // For overrides, just get credentials without a specific provider
-    credentials = await settingsService?.getCredentials();
-  } else if (settingsService) {
-    // Use settings-based model with provider info
-    const phaseResult = await getPhaseModelWithOverrides(
-      'suggestionsModel',
-      settingsService,
-      projectPath,
-      '[Suggestions]'
-    );
-    const resolved = resolvePhaseModel(phaseResult.phaseModel);
-    model = resolved.model;
-    thinkingLevel = resolved.thinkingLevel;
-    provider = phaseResult.provider;
-    credentials = phaseResult.credentials;
   } else {
-    // Fallback to defaults
-    const resolved = resolvePhaseModel(DEFAULT_PHASE_MODELS.suggestionsModel);
+    // Use settings-based model
+    const phaseModelEntry =
+      settings?.phaseModels?.suggestionsModel || DEFAULT_PHASE_MODELS.suggestionsModel;
+    const resolved = resolvePhaseModel(phaseModelEntry);
     model = resolved.model;
     thinkingLevel = resolved.thinkingLevel;
   }
 
-  logger.info(
-    '[Suggestions] Using model:',
-    model,
-    provider ? `via provider: ${provider.name}` : 'direct API'
-  );
+  logger.info('[Suggestions] Using model:', model);
 
   let responseText = '';
 
@@ -245,8 +223,6 @@ Your entire response should be valid JSON starting with { and ending with }. No
     thinkingLevel,
     readOnly: true, // Suggestions only reads code, doesn't write
     settingSources: autoLoadClaudeMd ? ['user', 'project', 'local'] : undefined,
-    claudeCompatibleProvider: provider, // Pass provider for alternative endpoint configuration
-    credentials, // Pass credentials for resolving 'credentials' apiKeySource
     outputFormat: useStructuredOutput
       ? {
           type: 'json_schema',

apps/server/src/routes/worktree/index.ts

@@ -29,13 +29,6 @@ import {
   createGetAvailableEditorsHandler,
   createRefreshEditorsHandler,
 } from './routes/open-in-editor.js';
-import {
-  createOpenInTerminalHandler,
-  createGetAvailableTerminalsHandler,
-  createGetDefaultTerminalHandler,
-  createRefreshTerminalsHandler,
-  createOpenInExternalTerminalHandler,
-} from './routes/open-in-terminal.js';
 import { createInitGitHandler } from './routes/init-git.js';
 import { createMigrateHandler } from './routes/migrate.js';
 import { createStartDevHandler } from './routes/start-dev.js';
@@ -48,8 +41,6 @@ import {
   createDeleteInitScriptHandler,
   createRunInitScriptHandler,
 } from './routes/init-script.js';
-import { createDiscardChangesHandler } from './routes/discard-changes.js';
-import { createListRemotesHandler } from './routes/list-remotes.js';
 import type { SettingsService } from '../../services/settings-service.js';
 
 export function createWorktreeRoutes(
@@ -106,25 +97,9 @@ export function createWorktreeRoutes(
   );
   router.post('/switch-branch', requireValidWorktree, createSwitchBranchHandler());
   router.post('/open-in-editor', validatePathParams('worktreePath'), createOpenInEditorHandler());
-  router.post(
-    '/open-in-terminal',
-    validatePathParams('worktreePath'),
-    createOpenInTerminalHandler()
-  );
   router.get('/default-editor', createGetDefaultEditorHandler());
   router.get('/available-editors', createGetAvailableEditorsHandler());
   router.post('/refresh-editors', createRefreshEditorsHandler());
-
-  // External terminal routes
-  router.get('/available-terminals', createGetAvailableTerminalsHandler());
-  router.get('/default-terminal', createGetDefaultTerminalHandler());
-  router.post('/refresh-terminals', createRefreshTerminalsHandler());
-  router.post(
-    '/open-in-external-terminal',
-    validatePathParams('worktreePath'),
-    createOpenInExternalTerminalHandler()
-  );
-
   router.post('/init-git', validatePathParams('projectPath'), createInitGitHandler());
   router.post('/migrate', createMigrateHandler());
   router.post(
@@ -150,21 +125,5 @@ export function createWorktreeRoutes(
     createRunInitScriptHandler(events)
   );
 
-  // Discard changes route
-  router.post(
-    '/discard-changes',
-    validatePathParams('worktreePath'),
-    requireGitRepoOnly,
-    createDiscardChangesHandler()
-  );
-
-  // List remotes route
-  router.post(
-    '/list-remotes',
-    validatePathParams('worktreePath'),
-    requireValidWorktree,
-    createListRemotesHandler()
-  );
-
   return router;
 }

apps/server/src/routes/worktree/routes/create-pr.ts

@@ -13,7 +13,6 @@ import {
 } from '../common.js';
 import { updateWorktreePRInfo } from '../../../lib/worktree-metadata.js';
 import { createLogger } from '@automaker/utils';
-import { validatePRState } from '@automaker/types';
 
 const logger = createLogger('CreatePR');
 
@@ -269,12 +268,11 @@ export function createCreatePRHandler() {
             prAlreadyExisted = true;
 
             // Store the existing PR info in metadata
-            // GitHub CLI returns uppercase states: OPEN, MERGED, CLOSED
             await updateWorktreePRInfo(effectiveProjectPath, branchName, {
               number: existingPr.number,
               url: existingPr.url,
               title: existingPr.title || title,
-              state: validatePRState(existingPr.state),
+              state: existingPr.state || 'open',
               createdAt: new Date().toISOString(),
             });
             logger.debug(
@@ -321,12 +319,11 @@ export function createCreatePRHandler() {
 
               if (prNumber) {
                 try {
-                  // Note: GitHub doesn't have a 'DRAFT' state - drafts still show as 'OPEN'
                   await updateWorktreePRInfo(effectiveProjectPath, branchName, {
                     number: prNumber,
                     url: prUrl,
                     title,
-                    state: 'OPEN',
+                    state: draft ? 'draft' : 'open',
                     createdAt: new Date().toISOString(),
                   });
                   logger.debug(`Stored PR info for branch ${branchName}: PR #${prNumber}`);
@@ -355,12 +352,11 @@ export function createCreatePRHandler() {
                   prNumber = existingPr.number;
                   prAlreadyExisted = true;
 
-                  // GitHub CLI returns uppercase states: OPEN, MERGED, CLOSED
                   await updateWorktreePRInfo(effectiveProjectPath, branchName, {
                     number: existingPr.number,
                     url: existingPr.url,
                     title: existingPr.title || title,
-                    state: validatePRState(existingPr.state),
+                    state: existingPr.state || 'open',
                     createdAt: new Date().toISOString(),
                   });
                   logger.debug(`Fetched and stored existing PR: #${existingPr.number}`);

apps/server/src/routes/worktree/routes/diffs.ts

@@ -39,10 +39,7 @@ export function createDiffsHandler() {
       }
 
       // Git worktrees are stored in project directory
-      // Sanitize featureId the same way it's sanitized when creating worktrees
-      // (see create.ts: branchName.replace(/[^a-zA-Z0-9_-]/g, '-'))
-      const sanitizedFeatureId = featureId.replace(/[^a-zA-Z0-9_-]/g, '-');
-      const worktreePath = path.join(projectPath, '.worktrees', sanitizedFeatureId);
+      const worktreePath = path.join(projectPath, '.worktrees', featureId);
 
       try {
         // Check if worktree exists

apps/server/src/routes/worktree/routes/discard-changes.ts

@@ -1,112 +0,0 @@
-/**
- * POST /discard-changes endpoint - Discard all uncommitted changes in a worktree
- *
- * This performs a destructive operation that:
- * 1. Resets staged changes (git reset HEAD)
- * 2. Discards modified tracked files (git checkout .)
- * 3. Removes untracked files and directories (git clean -fd)
- *
- * Note: Git repository validation (isGitRepo) is handled by
- * the requireGitRepoOnly middleware in index.ts
- */
-
-import type { Request, Response } from 'express';
-import { exec } from 'child_process';
-import { promisify } from 'util';
-import { getErrorMessage, logError } from '../common.js';
-
-const execAsync = promisify(exec);
-
-export function createDiscardChangesHandler() {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { worktreePath } = req.body as {
-        worktreePath: string;
-      };
-
-      if (!worktreePath) {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath required',
-        });
-        return;
-      }
-
-      // Check for uncommitted changes first
-      const { stdout: status } = await execAsync('git status --porcelain', {
-        cwd: worktreePath,
-      });
-
-      if (!status.trim()) {
-        res.json({
-          success: true,
-          result: {
-            discarded: false,
-            message: 'No changes to discard',
-          },
-        });
-        return;
-      }
-
-      // Count the files that will be affected
-      const lines = status.trim().split('\n').filter(Boolean);
-      const fileCount = lines.length;
-
-      // Get branch name before discarding
-      const { stdout: branchOutput } = await execAsync('git rev-parse --abbrev-ref HEAD', {
-        cwd: worktreePath,
-      });
-      const branchName = branchOutput.trim();
-
-      // Discard all changes:
-      // 1. Reset any staged changes
-      await execAsync('git reset HEAD', { cwd: worktreePath }).catch(() => {
-        // Ignore errors - might fail if there's nothing staged
-      });
-
-      // 2. Discard changes in tracked files
-      await execAsync('git checkout .', { cwd: worktreePath }).catch(() => {
-        // Ignore errors - might fail if there are no tracked changes
-      });
-
-      // 3. Remove untracked files and directories
-      await execAsync('git clean -fd', { cwd: worktreePath }).catch(() => {
-        // Ignore errors - might fail if there are no untracked files
-      });
-
-      // Verify all changes were discarded
-      const { stdout: finalStatus } = await execAsync('git status --porcelain', {
-        cwd: worktreePath,
-      });
-
-      if (finalStatus.trim()) {
-        // Some changes couldn't be discarded (possibly ignored files or permission issues)
-        const remainingCount = finalStatus.trim().split('\n').filter(Boolean).length;
-        res.json({
-          success: true,
-          result: {
-            discarded: true,
-            filesDiscarded: fileCount - remainingCount,
-            filesRemaining: remainingCount,
-            branch: branchName,
-            message: `Discarded ${fileCount - remainingCount} files, ${remainingCount} files could not be removed`,
-          },
-        });
-      } else {
-        res.json({
-          success: true,
-          result: {
-            discarded: true,
-            filesDiscarded: fileCount,
-            filesRemaining: 0,
-            branch: branchName,
-            message: `Discarded ${fileCount} ${fileCount === 1 ? 'file' : 'files'}`,
-          },
-        });
-      }
-    } catch (error) {
-      logError(error, 'Discard changes failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}

apps/server/src/routes/worktree/routes/file-diff.ts

@@ -37,10 +37,7 @@ export function createFileDiffHandler() {
       }
 
       // Git worktrees are stored in project directory
-      // Sanitize featureId the same way it's sanitized when creating worktrees
-      // (see create.ts: branchName.replace(/[^a-zA-Z0-9_-]/g, '-'))
-      const sanitizedFeatureId = featureId.replace(/[^a-zA-Z0-9_-]/g, '-');
-      const worktreePath = path.join(projectPath, '.worktrees', sanitizedFeatureId);
+      const worktreePath = path.join(projectPath, '.worktrees', featureId);
 
       try {
         await secureFs.access(worktreePath);

apps/server/src/routes/worktree/routes/generate-commit-message.ts

@@ -10,14 +10,14 @@ import { exec } from 'child_process';
 import { promisify } from 'util';
 import { existsSync } from 'fs';
 import { join } from 'path';
+import { query } from '@anthropic-ai/claude-agent-sdk';
 import { createLogger } from '@automaker/utils';
-import { isCursorModel, stripProviderPrefix } from '@automaker/types';
+import { DEFAULT_PHASE_MODELS, isCursorModel, stripProviderPrefix } from '@automaker/types';
 import { resolvePhaseModel } from '@automaker/model-resolver';
 import { mergeCommitMessagePrompts } from '@automaker/prompts';
 import { ProviderFactory } from '../../../providers/provider-factory.js';
 import type { SettingsService } from '../../../services/settings-service.js';
 import { getErrorMessage, logError } from '../common.js';
-import { getPhaseModelWithOverrides } from '../../../lib/settings-helpers.js';
 
 const logger = createLogger('GenerateCommitMessage');
 const execAsync = promisify(exec);
@@ -74,6 +74,33 @@ interface GenerateCommitMessageErrorResponse {
   error: string;
 }
 
+async function extractTextFromStream(
+  stream: AsyncIterable<{
+    type: string;
+    subtype?: string;
+    result?: string;
+    message?: {
+      content?: Array<{ type: string; text?: string }>;
+    };
+  }>
+): Promise<string> {
+  let responseText = '';
+
+  for await (const msg of stream) {
+    if (msg.type === 'assistant' && msg.message?.content) {
+      for (const block of msg.message.content) {
+        if (block.type === 'text' && block.text) {
+          responseText += block.text;
+        }
+      }
+    } else if (msg.type === 'result' && msg.subtype === 'success') {
+      responseText = msg.result || responseText;
+    }
+  }
+
+  return responseText;
+}
+
 export function createGenerateCommitMessageHandler(
   settingsService?: SettingsService
 ): (req: Request, res: Response) => Promise<void> {
@@ -157,67 +184,67 @@ export function createGenerateCommitMessageHandler(
 
       const userPrompt = `Generate a commit message for these changes:\n\n\`\`\`diff\n${truncatedDiff}\n\`\`\``;
 
-      // Get model from phase settings with provider info
-      const {
-        phaseModel: phaseModelEntry,
-        provider: claudeCompatibleProvider,
-        credentials,
-      } = await getPhaseModelWithOverrides(
-        'commitMessageModel',
-        settingsService,
-        worktreePath,
-        '[GenerateCommitMessage]'
-      );
+      // Get model from phase settings
+      const settings = await settingsService?.getGlobalSettings();
+      const phaseModelEntry =
+        settings?.phaseModels?.commitMessageModel || DEFAULT_PHASE_MODELS.commitMessageModel;
       const { model } = resolvePhaseModel(phaseModelEntry);
 
-      logger.info(
-        `Using model for commit message: ${model}`,
-        claudeCompatibleProvider ? `via provider: ${claudeCompatibleProvider.name}` : 'direct API'
-      );
+      logger.info(`Using model for commit message: ${model}`);
 
       // Get the effective system prompt (custom or default)
       const systemPrompt = await getSystemPrompt(settingsService);
 
-      // Get provider for the model type
-      const aiProvider = ProviderFactory.getProviderForModel(model);
-      const bareModel = stripProviderPrefix(model);
+      let message: string;
 
-      // For Cursor models, combine prompts since Cursor doesn't support systemPrompt separation
-      const effectivePrompt = isCursorModel(model)
-        ? `${systemPrompt}\n\n${userPrompt}`
-        : userPrompt;
-      const effectiveSystemPrompt = isCursorModel(model) ? undefined : systemPrompt;
+      // Route to appropriate provider based on model type
+      if (isCursorModel(model)) {
+        // Use Cursor provider for Cursor models
+        logger.info(`Using Cursor provider for model: ${model}`);
 
-      logger.info(`Using ${aiProvider.getName()} provider for model: ${model}`);
+        const provider = ProviderFactory.getProviderForModel(model);
+        const bareModel = stripProviderPrefix(model);
 
-      let responseText = '';
-      const stream = aiProvider.executeQuery({
-        prompt: effectivePrompt,
-        model: bareModel,
-        cwd: worktreePath,
-        systemPrompt: effectiveSystemPrompt,
-        maxTurns: 1,
-        allowedTools: [],
-        readOnly: true,
-        claudeCompatibleProvider, // Pass provider for alternative endpoint configuration
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
-      });
+        const cursorPrompt = `${systemPrompt}\n\n${userPrompt}`;
 
-      // Wrap with timeout to prevent indefinite hangs
-      for await (const msg of withTimeout(stream, AI_TIMEOUT_MS)) {
-        if (msg.type === 'assistant' && msg.message?.content) {
-          for (const block of msg.message.content) {
-            if (block.type === 'text' && block.text) {
-              responseText += block.text;
+        let responseText = '';
+        const cursorStream = provider.executeQuery({
+          prompt: cursorPrompt,
+          model: bareModel,
+          cwd: worktreePath,
+          maxTurns: 1,
+          allowedTools: [],
+          readOnly: true,
+        });
+
+        // Wrap with timeout to prevent indefinite hangs
+        for await (const msg of withTimeout(cursorStream, AI_TIMEOUT_MS)) {
+          if (msg.type === 'assistant' && msg.message?.content) {
+            for (const block of msg.message.content) {
+              if (block.type === 'text' && block.text) {
+                responseText += block.text;
+              }
             }
           }
-        } else if (msg.type === 'result' && msg.subtype === 'success' && msg.result) {
-          // Use result if available (some providers return final text here)
-          responseText = msg.result;
         }
-      }
 
-      const message = responseText.trim();
+        message = responseText.trim();
+      } else {
+        // Use Claude SDK for Claude models
+        const stream = query({
+          prompt: userPrompt,
+          options: {
+            model,
+            systemPrompt,
+            maxTurns: 1,
+            allowedTools: [],
+            permissionMode: 'default',
+          },
+        });
+
+        // Wrap with timeout to prevent indefinite hangs
+        message = await extractTextFromStream(withTimeout(stream, AI_TIMEOUT_MS));
+      }
 
       if (!message || message.trim().length === 0) {
         logger.warn('Received empty response from model');

apps/server/src/routes/worktree/routes/info.ts

@@ -28,10 +28,7 @@ export function createInfoHandler() {
       }
 
       // Check if worktree exists (git worktrees are stored in project directory)
-      // Sanitize featureId the same way it's sanitized when creating worktrees
-      // (see create.ts: branchName.replace(/[^a-zA-Z0-9_-]/g, '-'))
-      const sanitizedFeatureId = featureId.replace(/[^a-zA-Z0-9_-]/g, '-');
-      const worktreePath = path.join(projectPath, '.worktrees', sanitizedFeatureId);
+      const worktreePath = path.join(projectPath, '.worktrees', featureId);
       try {
         await secureFs.access(worktreePath);
         const { stdout } = await execAsync('git rev-parse --abbrev-ref HEAD', {

apps/server/src/routes/worktree/routes/list-branches.ts

@@ -110,10 +110,9 @@ export function createListBranchesHandler() {
         }
       }
 
-      // Get ahead/behind count for current branch and check if remote branch exists
+      // Get ahead/behind count for current branch
       let aheadCount = 0;
       let behindCount = 0;
-      let hasRemoteBranch = false;
       try {
         // First check if there's a remote tracking branch
         const { stdout: upstreamOutput } = await execAsync(
@@ -122,7 +121,6 @@ export function createListBranchesHandler() {
         );
 
         if (upstreamOutput.trim()) {
-          hasRemoteBranch = true;
           const { stdout: aheadBehindOutput } = await execAsync(
             `git rev-list --left-right --count ${currentBranch}@{upstream}...HEAD`,
             { cwd: worktreePath }
@@ -132,18 +130,7 @@ export function createListBranchesHandler() {
           behindCount = behind || 0;
         }
       } catch {
-        // No upstream branch set - check if the branch exists on any remote
-        try {
-          // Check if there's a matching branch on origin (most common remote)
-          const { stdout: remoteBranchOutput } = await execAsync(
-            `git ls-remote --heads origin ${currentBranch}`,
-            { cwd: worktreePath, timeout: 5000 }
-          );
-          hasRemoteBranch = remoteBranchOutput.trim().length > 0;
-        } catch {
-          // No remote branch found or origin doesn't exist
-          hasRemoteBranch = false;
-        }
+        // No upstream branch set, that's okay
       }
 
       res.json({
@@ -153,7 +140,6 @@ export function createListBranchesHandler() {
           branches,
           aheadCount,
           behindCount,
-          hasRemoteBranch,
         },
       });
     } catch (error) {

apps/server/src/routes/worktree/routes/list-remotes.ts

@@ -1,127 +0,0 @@
-/**
- * POST /list-remotes endpoint - List all remotes and their branches
- *
- * Note: Git repository validation (isGitRepo, hasCommits) is handled by
- * the requireValidWorktree middleware in index.ts
- */
-
-import type { Request, Response } from 'express';
-import { exec } from 'child_process';
-import { promisify } from 'util';
-import { getErrorMessage, logWorktreeError } from '../common.js';
-
-const execAsync = promisify(exec);
-
-interface RemoteBranch {
-  name: string;
-  fullRef: string;
-}
-
-interface RemoteInfo {
-  name: string;
-  url: string;
-  branches: RemoteBranch[];
-}
-
-export function createListRemotesHandler() {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { worktreePath } = req.body as {
-        worktreePath: string;
-      };
-
-      if (!worktreePath) {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath required',
-        });
-        return;
-      }
-
-      // Get list of remotes
-      const { stdout: remotesOutput } = await execAsync('git remote -v', {
-        cwd: worktreePath,
-      });
-
-      // Parse remotes (each remote appears twice - once for fetch, once for push)
-      const remotesSet = new Map<string, string>();
-      remotesOutput
-        .trim()
-        .split('\n')
-        .filter((line) => line.trim())
-        .forEach((line) => {
-          const match = line.match(/^(\S+)\s+(\S+)\s+\(fetch\)$/);
-          if (match) {
-            remotesSet.set(match[1], match[2]);
-          }
-        });
-
-      // Fetch latest from all remotes (silently, don't fail if offline)
-      try {
-        await execAsync('git fetch --all --quiet', {
-          cwd: worktreePath,
-          timeout: 15000, // 15 second timeout
-        });
-      } catch {
-        // Ignore fetch errors - we'll use cached remote refs
-      }
-
-      // Get all remote branches
-      const { stdout: remoteBranchesOutput } = await execAsync(
-        'git branch -r --format="%(refname:short)"',
-        { cwd: worktreePath }
-      );
-
-      // Group branches by remote
-      const remotesBranches = new Map<string, RemoteBranch[]>();
-      remotesSet.forEach((_, remoteName) => {
-        remotesBranches.set(remoteName, []);
-      });
-
-      remoteBranchesOutput
-        .trim()
-        .split('\n')
-        .filter((line) => line.trim())
-        .forEach((line) => {
-          const cleanLine = line.trim().replace(/^['"]|['"]$/g, '');
-          // Skip HEAD pointers like "origin/HEAD"
-          if (cleanLine.includes('/HEAD')) return;
-
-          // Parse remote name from branch ref (e.g., "origin/main" -> "origin")
-          const slashIndex = cleanLine.indexOf('/');
-          if (slashIndex === -1) return;
-
-          const remoteName = cleanLine.substring(0, slashIndex);
-          const branchName = cleanLine.substring(slashIndex + 1);
-
-          if (remotesBranches.has(remoteName)) {
-            remotesBranches.get(remoteName)!.push({
-              name: branchName,
-              fullRef: cleanLine,
-            });
-          }
-        });
-
-      // Build final result
-      const remotes: RemoteInfo[] = [];
-      remotesSet.forEach((url, name) => {
-        remotes.push({
-          name,
-          url,
-          branches: remotesBranches.get(name) || [],
-        });
-      });
-
-      res.json({
-        success: true,
-        result: {
-          remotes,
-        },
-      });
-    } catch (error) {
-      const worktreePath = req.body?.worktreePath;
-      logWorktreeError(error, 'List remotes failed', worktreePath);
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}

apps/server/src/routes/worktree/routes/list.ts

@@ -14,13 +14,8 @@ import path from 'path';
 import * as secureFs from '../../../lib/secure-fs.js';
 import { isGitRepo } from '@automaker/git-utils';
 import { getErrorMessage, logError, normalizePath, execEnv, isGhCliAvailable } from '../common.js';
-import {
-  readAllWorktreeMetadata,
-  updateWorktreePRInfo,
-  type WorktreePRInfo,
-} from '../../../lib/worktree-metadata.js';
+import { readAllWorktreeMetadata, type WorktreePRInfo } from '../../../lib/worktree-metadata.js';
 import { createLogger } from '@automaker/utils';
-import { validatePRState } from '@automaker/types';
 import {
   checkGitHubRemote,
   type GitHubRemoteStatus,
@@ -173,11 +168,8 @@ async function getGitHubRemoteStatus(projectPath: string): Promise<GitHubRemoteS
 }
 
 /**
- * Fetch all PRs from GitHub and create a map of branch name to PR info.
- * Uses --state all to include merged/closed PRs, allowing detection of
- * state changes (e.g., when a PR is merged on GitHub).
- *
- * This also allows detecting PRs that were created outside the app.
+ * Fetch open PRs from GitHub and create a map of branch name to PR info.
+ * This allows detecting PRs that were created outside the app.
  *
  * Uses cached GitHub remote status to avoid repeated warnings when the
  * project doesn't have a GitHub remote configured.
@@ -200,9 +192,9 @@ async function fetchGitHubPRs(projectPath: string): Promise<Map<string, Worktree
         ? `-R ${remoteStatus.owner}/${remoteStatus.repo}`
         : '';
 
-    // Fetch all PRs from GitHub (including merged/closed to detect state changes)
+    // Fetch open PRs from GitHub
     const { stdout } = await execAsync(
-      `gh pr list ${repoFlag} --state all --json number,title,url,state,headRefName,createdAt --limit 1000`,
+      `gh pr list ${repoFlag} --state open --json number,title,url,state,headRefName,createdAt --limit 1000`,
       { cwd: projectPath, env: execEnv, timeout: 15000 }
     );
 
@@ -220,8 +212,7 @@ async function fetchGitHubPRs(projectPath: string): Promise<Map<string, Worktree
         number: pr.number,
         url: pr.url,
         title: pr.title,
-        // GitHub CLI returns state as uppercase: OPEN, MERGED, CLOSED
-        state: validatePRState(pr.state),
+        state: pr.state,
         createdAt: pr.createdAt,
       });
     }
@@ -360,43 +351,23 @@ export function createListHandler() {
         }
       }
 
-      // Assign PR info to each worktree, preferring fresh GitHub data over cached metadata.
-      // Only fetch GitHub PRs if includeDetails is requested (performance optimization).
-      // Uses --state all to detect merged/closed PRs, limited to 1000 recent PRs.
+      // Add PR info from metadata or GitHub for each worktree
+      // Only fetch GitHub PRs if includeDetails is requested (performance optimization)
       const githubPRs = includeDetails
         ? await fetchGitHubPRs(projectPath)
         : new Map<string, WorktreePRInfo>();
 
       for (const worktree of worktrees) {
-        // Skip PR assignment for the main worktree - it's not meaningful to show
-        // PRs on the main branch tab, and can be confusing if someone created
-        // a PR from main to another branch
-        if (worktree.isMain) {
-          continue;
-        }
-
         const metadata = allMetadata.get(worktree.branch);
-        const githubPR = githubPRs.get(worktree.branch);
-
-        if (githubPR) {
-          // Prefer fresh GitHub data (it has the current state)
-          worktree.pr = githubPR;
-
-          // Sync metadata with GitHub state when:
-          // 1. No metadata exists for this PR (PR created externally)
-          // 2. State has changed (e.g., merged/closed on GitHub)
-          const needsSync = !metadata?.pr || metadata.pr.state !== githubPR.state;
-          if (needsSync) {
-            // Fire and forget - don't block the response
-            updateWorktreePRInfo(projectPath, worktree.branch, githubPR).catch((err) => {
-              logger.warn(
-                `Failed to update PR info for ${worktree.branch}: ${getErrorMessage(err)}`
-              );
-            });
-          }
-        } else if (metadata?.pr && metadata.pr.state === 'OPEN') {
-          // Fall back to stored metadata only if the PR is still OPEN
+        if (metadata?.pr) {
+          // Use stored metadata (more complete info)
           worktree.pr = metadata.pr;
+        } else if (includeDetails) {
+          // Fall back to GitHub PR detection only when includeDetails is requested
+          const githubPR = githubPRs.get(worktree.branch);
+          if (githubPR) {
+            worktree.pr = githubPR;
+          }
         }
       }
 

apps/server/src/routes/worktree/routes/merge.ts

@@ -1,7 +1,5 @@
 /**
- * POST /merge endpoint - Merge feature (merge worktree branch into a target branch)
- *
- * Allows merging a worktree branch into any target branch (defaults to 'main').
+ * POST /merge endpoint - Merge feature (merge worktree branch into main)
  *
  * Note: Git repository validation (isGitRepo, hasCommits) is handled by
  * the requireValidProject middleware in index.ts
@@ -10,21 +8,18 @@
 import type { Request, Response } from 'express';
 import { exec } from 'child_process';
 import { promisify } from 'util';
-import { getErrorMessage, logError, isValidBranchName, execGitCommand } from '../common.js';
-import { createLogger } from '@automaker/utils';
+import { getErrorMessage, logError } from '../common.js';
 
 const execAsync = promisify(exec);
-const logger = createLogger('Worktree');
 
 export function createMergeHandler() {
   return async (req: Request, res: Response): Promise<void> => {
     try {
-      const { projectPath, branchName, worktreePath, targetBranch, options } = req.body as {
+      const { projectPath, branchName, worktreePath, options } = req.body as {
         projectPath: string;
         branchName: string;
         worktreePath: string;
-        targetBranch?: string; // Branch to merge into (defaults to 'main')
-        options?: { squash?: boolean; message?: string; deleteWorktreeAndBranch?: boolean };
+        options?: { squash?: boolean; message?: string };
       };
 
       if (!projectPath || !branchName || !worktreePath) {
@@ -35,10 +30,7 @@ export function createMergeHandler() {
         return;
       }
 
-      // Determine the target branch (default to 'main')
-      const mergeTo = targetBranch || 'main';
-
-      // Validate source branch exists
+      // Validate branch exists
       try {
         await execAsync(`git rev-parse --verify ${branchName}`, { cwd: projectPath });
       } catch {
@@ -49,44 +41,12 @@ export function createMergeHandler() {
         return;
       }
 
-      // Validate target branch exists
-      try {
-        await execAsync(`git rev-parse --verify ${mergeTo}`, { cwd: projectPath });
-      } catch {
-        res.status(400).json({
-          success: false,
-          error: `Target branch "${mergeTo}" does not exist`,
-        });
-        return;
-      }
-
-      // Merge the feature branch into the target branch
+      // Merge the feature branch
       const mergeCmd = options?.squash
         ? `git merge --squash ${branchName}`
-        : `git merge ${branchName} -m "${options?.message || `Merge ${branchName} into ${mergeTo}`}"`;
+        : `git merge ${branchName} -m "${options?.message || `Merge ${branchName}`}"`;
 
-      try {
-        await execAsync(mergeCmd, { cwd: projectPath });
-      } catch (mergeError: unknown) {
-        // Check if this is a merge conflict
-        const err = mergeError as { stdout?: string; stderr?: string; message?: string };
-        const output = `${err.stdout || ''} ${err.stderr || ''} ${err.message || ''}`;
-        const hasConflicts =
-          output.includes('CONFLICT') || output.includes('Automatic merge failed');
-
-        if (hasConflicts) {
-          // Return conflict-specific error message that frontend can detect
-          res.status(409).json({
-            success: false,
-            error: `Merge CONFLICT: Automatic merge of "${branchName}" into "${mergeTo}" failed. Please resolve conflicts manually.`,
-            hasConflicts: true,
-          });
-          return;
-        }
-
-        // Re-throw non-conflict errors to be handled by outer catch
-        throw mergeError;
-      }
+      await execAsync(mergeCmd, { cwd: projectPath });
 
       // If squash merge, need to commit
       if (options?.squash) {
@@ -95,46 +55,17 @@ export function createMergeHandler() {
         });
       }
 
-      // Optionally delete the worktree and branch after merging
-      let worktreeDeleted = false;
-      let branchDeleted = false;
-
-      if (options?.deleteWorktreeAndBranch) {
-        // Remove the worktree
-        try {
-          await execGitCommand(['worktree', 'remove', worktreePath, '--force'], projectPath);
-          worktreeDeleted = true;
-        } catch {
-          // Try with prune if remove fails
-          try {
-            await execGitCommand(['worktree', 'prune'], projectPath);
-            worktreeDeleted = true;
-          } catch {
-            logger.warn(`Failed to remove worktree: ${worktreePath}`);
-          }
-        }
-
-        // Delete the branch (but not main/master)
-        if (branchName !== 'main' && branchName !== 'master') {
-          if (!isValidBranchName(branchName)) {
-            logger.warn(`Invalid branch name detected, skipping deletion: ${branchName}`);
-          } else {
-            try {
-              await execGitCommand(['branch', '-D', branchName], projectPath);
-              branchDeleted = true;
-            } catch {
-              logger.warn(`Failed to delete branch: ${branchName}`);
-            }
-          }
-        }
+      // Clean up worktree and branch
+      try {
+        await execAsync(`git worktree remove "${worktreePath}" --force`, {
+          cwd: projectPath,
+        });
+        await execAsync(`git branch -D ${branchName}`, { cwd: projectPath });
+      } catch {
+        // Cleanup errors are non-fatal
       }
 
-      res.json({
-        success: true,
-        mergedBranch: branchName,
-        targetBranch: mergeTo,
-        deleted: options?.deleteWorktreeAndBranch ? { worktreeDeleted, branchDeleted } : undefined,
-      });
+      res.json({ success: true, mergedBranch: branchName });
     } catch (error) {
       logError(error, 'Merge worktree failed');
       res.status(500).json({ success: false, error: getErrorMessage(error) });

apps/server/src/routes/worktree/routes/open-in-terminal.ts

@@ -1,181 +0,0 @@
-/**
- * Terminal endpoints for opening worktree directories in terminals
- *
- * POST /open-in-terminal - Open in system default terminal (integrated)
- * GET /available-terminals - List all available external terminals
- * GET /default-terminal - Get the default external terminal
- * POST /refresh-terminals - Clear terminal cache and re-detect
- * POST /open-in-external-terminal - Open a directory in an external terminal
- */
-
-import type { Request, Response } from 'express';
-import { isAbsolute } from 'path';
-import {
-  openInTerminal,
-  clearTerminalCache,
-  detectAllTerminals,
-  detectDefaultTerminal,
-  openInExternalTerminal,
-} from '@automaker/platform';
-import { createLogger } from '@automaker/utils';
-import { getErrorMessage, logError } from '../common.js';
-
-const logger = createLogger('open-in-terminal');
-
-/**
- * Handler to open in system default terminal (integrated terminal behavior)
- */
-export function createOpenInTerminalHandler() {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { worktreePath } = req.body as {
-        worktreePath: string;
-      };
-
-      if (!worktreePath || typeof worktreePath !== 'string') {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath required and must be a string',
-        });
-        return;
-      }
-
-      // Security: Validate that worktreePath is an absolute path
-      if (!isAbsolute(worktreePath)) {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath must be an absolute path',
-        });
-        return;
-      }
-
-      // Use the platform utility to open in terminal
-      const result = await openInTerminal(worktreePath);
-      res.json({
-        success: true,
-        result: {
-          message: `Opened terminal in ${worktreePath}`,
-          terminalName: result.terminalName,
-        },
-      });
-    } catch (error) {
-      logError(error, 'Open in terminal failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}
-
-/**
- * Handler to get all available external terminals
- */
-export function createGetAvailableTerminalsHandler() {
-  return async (_req: Request, res: Response): Promise<void> => {
-    try {
-      const terminals = await detectAllTerminals();
-      res.json({
-        success: true,
-        result: {
-          terminals,
-        },
-      });
-    } catch (error) {
-      logError(error, 'Get available terminals failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}
-
-/**
- * Handler to get the default external terminal
- */
-export function createGetDefaultTerminalHandler() {
-  return async (_req: Request, res: Response): Promise<void> => {
-    try {
-      const terminal = await detectDefaultTerminal();
-      res.json({
-        success: true,
-        result: terminal
-          ? {
-              terminalId: terminal.id,
-              terminalName: terminal.name,
-              terminalCommand: terminal.command,
-            }
-          : null,
-      });
-    } catch (error) {
-      logError(error, 'Get default terminal failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}
-
-/**
- * Handler to refresh the terminal cache and re-detect available terminals
- * Useful when the user has installed/uninstalled terminals
- */
-export function createRefreshTerminalsHandler() {
-  return async (_req: Request, res: Response): Promise<void> => {
-    try {
-      // Clear the cache
-      clearTerminalCache();
-
-      // Re-detect terminals (this will repopulate the cache)
-      const terminals = await detectAllTerminals();
-
-      logger.info(`Terminal cache refreshed, found ${terminals.length} terminals`);
-
-      res.json({
-        success: true,
-        result: {
-          terminals,
-          message: `Found ${terminals.length} available external terminals`,
-        },
-      });
-    } catch (error) {
-      logError(error, 'Refresh terminals failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}
-
-/**
- * Handler to open a directory in an external terminal
- */
-export function createOpenInExternalTerminalHandler() {
-  return async (req: Request, res: Response): Promise<void> => {
-    try {
-      const { worktreePath, terminalId } = req.body as {
-        worktreePath: string;
-        terminalId?: string;
-      };
-
-      if (!worktreePath || typeof worktreePath !== 'string') {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath required and must be a string',
-        });
-        return;
-      }
-
-      if (!isAbsolute(worktreePath)) {
-        res.status(400).json({
-          success: false,
-          error: 'worktreePath must be an absolute path',
-        });
-        return;
-      }
-
-      const result = await openInExternalTerminal(worktreePath, terminalId);
-      res.json({
-        success: true,
-        result: {
-          message: `Opened ${worktreePath} in ${result.terminalName}`,
-          terminalName: result.terminalName,
-        },
-      });
-    } catch (error) {
-      logError(error, 'Open in external terminal failed');
-      res.status(500).json({ success: false, error: getErrorMessage(error) });
-    }
-  };
-}

apps/server/src/routes/worktree/routes/push.ts

@@ -15,10 +15,9 @@ const execAsync = promisify(exec);
 export function createPushHandler() {
   return async (req: Request, res: Response): Promise<void> => {
     try {
-      const { worktreePath, force, remote } = req.body as {
+      const { worktreePath, force } = req.body as {
         worktreePath: string;
         force?: boolean;
-        remote?: string;
       };
 
       if (!worktreePath) {
@@ -35,18 +34,15 @@ export function createPushHandler() {
       });
       const branchName = branchOutput.trim();
 
-      // Use specified remote or default to 'origin'
-      const targetRemote = remote || 'origin';
-
       // Push the branch
       const forceFlag = force ? '--force' : '';
       try {
-        await execAsync(`git push -u ${targetRemote} ${branchName} ${forceFlag}`, {
+        await execAsync(`git push -u origin ${branchName} ${forceFlag}`, {
           cwd: worktreePath,
         });
       } catch {
         // Try setting upstream
-        await execAsync(`git push --set-upstream ${targetRemote} ${branchName} ${forceFlag}`, {
+        await execAsync(`git push --set-upstream origin ${branchName} ${forceFlag}`, {
           cwd: worktreePath,
         });
       }
@@ -56,7 +52,7 @@ export function createPushHandler() {
         result: {
           branch: branchName,
           pushed: true,
-          message: `Successfully pushed ${branchName} to ${targetRemote}`,
+          message: `Successfully pushed ${branchName} to origin`,
         },
       });
     } catch (error) {

apps/server/src/routes/worktree/routes/status.ts

@@ -28,10 +28,7 @@ export function createStatusHandler() {
       }
 
       // Git worktrees are stored in project directory
-      // Sanitize featureId the same way it's sanitized when creating worktrees
-      // (see create.ts: branchName.replace(/[^a-zA-Z0-9_-]/g, '-'))
-      const sanitizedFeatureId = featureId.replace(/[^a-zA-Z0-9_-]/g, '-');
-      const worktreePath = path.join(projectPath, '.worktrees', sanitizedFeatureId);
+      const worktreePath = path.join(projectPath, '.worktrees', featureId);
 
       try {
         await secureFs.access(worktreePath);

apps/server/src/services/agent-service.ts

@@ -29,7 +29,6 @@ import {
   getSkillsConfiguration,
   getSubagentsConfiguration,
   getCustomSubagents,
-  getProviderByModelId,
 } from '../lib/settings-helpers.js';
 
 interface Message {
@@ -275,30 +274,6 @@ export class AgentService {
           ? await getCustomSubagents(this.settingsService, effectiveWorkDir)
           : undefined;
 
-      // Get credentials for API calls
-      const credentials = await this.settingsService?.getCredentials();
-
-      // Try to find a provider for the model (if it's a provider model like "GLM-4.7")
-      // This allows users to select provider models in the Agent Runner UI
-      let claudeCompatibleProvider: import('@automaker/types').ClaudeCompatibleProvider | undefined;
-      let providerResolvedModel: string | undefined;
-      const requestedModel = model || session.model;
-      if (requestedModel && this.settingsService) {
-        const providerResult = await getProviderByModelId(
-          requestedModel,
-          this.settingsService,
-          '[AgentService]'
-        );
-        if (providerResult.provider) {
-          claudeCompatibleProvider = providerResult.provider;
-          providerResolvedModel = providerResult.resolvedModel;
-          this.logger.info(
-            `[AgentService] Using provider "${providerResult.provider.name}" for model "${requestedModel}"` +
-              (providerResolvedModel ? ` -> resolved to "${providerResolvedModel}"` : '')
-          );
-        }
-      }
-
       // Load project context files (CLAUDE.md, CODE_QUALITY.md, etc.) and memory files
       // Use the user's message as task context for smart memory selection
       const contextResult = await loadContextFiles({
@@ -324,16 +299,10 @@ export class AgentService {
       // Use thinking level and reasoning effort from request, or fall back to session's stored values
       const effectiveThinkingLevel = thinkingLevel ?? session.thinkingLevel;
       const effectiveReasoningEffort = reasoningEffort ?? session.reasoningEffort;
-
-      // When using a provider model, use the resolved Claude model (from mapsToClaudeModel)
-      // e.g., "GLM-4.5-Air" -> "claude-haiku-4-5"
-      const modelForSdk = providerResolvedModel || model;
-      const sessionModelForSdk = providerResolvedModel ? undefined : session.model;
-
       const sdkOptions = createChatOptions({
         cwd: effectiveWorkDir,
-        model: modelForSdk,
-        sessionModel: sessionModelForSdk,
+        model: model,
+        sessionModel: session.model,
         systemPrompt: combinedSystemPrompt,
         abortController: session.abortController!,
         autoLoadClaudeMd,
@@ -409,8 +378,6 @@ export class AgentService {
         agents: customSubagents, // Pass custom subagents for task delegation
         thinkingLevel: effectiveThinkingLevel, // Pass thinking level for Claude models
         reasoningEffort: effectiveReasoningEffort, // Pass reasoning effort for Codex models
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
-        claudeCompatibleProvider, // Pass provider for alternative endpoint configuration (GLM, MiniMax, etc.)
       };
 
       // Build prompt content with images

apps/server/src/services/claude-usage-service.ts

@@ -468,41 +468,10 @@ export class ClaudeUsageService {
 
   /**
    * Strip ANSI escape codes from text
-   * Handles CSI, OSC, and other common ANSI sequences
    */
   private stripAnsiCodes(text: string): string {
-    // First strip ANSI sequences (colors, etc) and handle CR
     // eslint-disable-next-line no-control-regex
-    let clean = text
-      // CSI sequences: ESC [ ... (letter or @)
-      .replace(/\x1B\[[0-9;?]*[A-Za-z@]/g, '')
-      // OSC sequences: ESC ] ... terminated by BEL, ST, or another ESC
-      .replace(/\x1B\][^\x07\x1B]*(?:\x07|\x1B\\)?/g, '')
-      // Other ESC sequences: ESC (letter)
-      .replace(/\x1B[A-Za-z]/g, '')
-      // Carriage returns: replace with newline to avoid concatenation
-      .replace(/\r\n/g, '\n')
-      .replace(/\r/g, '\n');
-
-    // Handle backspaces (\x08) by applying them
-    // If we encounter a backspace, remove the character before it
-    while (clean.includes('\x08')) {
-      clean = clean.replace(/[^\x08]\x08/, '');
-      clean = clean.replace(/^\x08+/, '');
-    }
-
-    // Explicitly strip known "Synchronized Output" and "Window Title" garbage
-    // even if ESC is missing (seen in some environments)
-    clean = clean
-      .replace(/\[\?2026[hl]/g, '') // CSI ? 2026 h/l
-      .replace(/\]0;[^\x07]*\x07/g, '') // OSC 0; Title BEL
-      .replace(/\]0;.*?(\[\?|$)/g, ''); // OSC 0; Title ... (unterminated or hit next sequence)
-
-    // Strip remaining non-printable control characters (except newline \n)
-    // ASCII 0-8, 11-31, 127
-    clean = clean.replace(/[\x00-\x08\x0B-\x1F\x7F]/g, '');
-
-    return clean;
+    return text.replace(/\x1B\[[0-9;]*[A-Za-z]/g, '');
   }
 
   /**
@@ -581,7 +550,7 @@ export class ClaudeUsageService {
     sectionLabel: string,
     type: string
   ): { percentage: number; resetTime: string; resetText: string } {
-    let percentage: number | null = null;
+    let percentage = 0;
     let resetTime = this.getDefaultResetTime(type);
     let resetText = '';
 
@@ -595,7 +564,7 @@ export class ClaudeUsageService {
     }
 
     if (sectionIndex === -1) {
-      return { percentage: 0, resetTime, resetText };
+      return { percentage, resetTime, resetText };
     }
 
     // Look at the lines following the section header (within a window of 5 lines)
@@ -603,8 +572,7 @@ export class ClaudeUsageService {
 
     for (const line of searchWindow) {
       // Extract percentage - only take the first match (avoid picking up next section's data)
-      // Use null to track "not found" since 0% is a valid percentage (100% left = 0% used)
-      if (percentage === null) {
+      if (percentage === 0) {
         const percentMatch = line.match(/(\d{1,3})\s*%\s*(left|used|remaining)/i);
         if (percentMatch) {
           const value = parseInt(percentMatch[1], 10);
@@ -616,31 +584,18 @@ export class ClaudeUsageService {
 
       // Extract reset time - only take the first match
       if (!resetText && line.toLowerCase().includes('reset')) {
-        // Only extract the part starting from "Resets" (or "Reset") to avoid garbage prefixes
-        const match = line.match(/(Resets?.*)$/i);
-        // If regex fails despite 'includes', likely a complex string issues - verify match before using line
-        // Only fallback to line if it's reasonably short/clean, otherwise skip it to avoid showing garbage
-        if (match) {
-          resetText = match[1];
-        }
+        resetText = line;
       }
     }
 
     // Parse the reset time if we found one
     if (resetText) {
-      // Clean up resetText: remove percentage info if it was matched on the same line
-      // e.g. "46%used Resets5:59pm" -> " Resets5:59pm"
-      resetText = resetText.replace(/(\d{1,3})\s*%\s*(left|used|remaining)/i, '').trim();
-
-      // Ensure space after "Resets" if missing (e.g. "Resets5:59pm" -> "Resets 5:59pm")
-      resetText = resetText.replace(/(resets?)(\d)/i, '$1 $2');
-
       resetTime = this.parseResetTime(resetText, type);
       // Strip timezone like "(Asia/Dubai)" from the display text
       resetText = resetText.replace(/\s*\([A-Za-z_\/]+\)\s*$/, '').trim();
     }
 
-    return { percentage: percentage ?? 0, resetTime, resetText };
+    return { percentage, resetTime, resetText };
   }
 
   /**
@@ -669,7 +624,7 @@ export class ClaudeUsageService {
     }
 
     // Try to parse simple time-only format: "Resets 11am" or "Resets 3pm"
-    const simpleTimeMatch = text.match(/resets\s*(\d{1,2})(?::(\d{2}))?\s*(am|pm)/i);
+    const simpleTimeMatch = text.match(/resets\s+(\d{1,2})(?::(\d{2}))?\s*(am|pm)/i);
     if (simpleTimeMatch) {
       let hours = parseInt(simpleTimeMatch[1], 10);
       const minutes = simpleTimeMatch[2] ? parseInt(simpleTimeMatch[2], 10) : 0;
@@ -694,11 +649,8 @@ export class ClaudeUsageService {
     }
 
     // Try to parse date format: "Resets Dec 22 at 8pm" or "Resets Jan 15, 3:30pm"
-    // The regex explicitly matches only valid 3-letter month abbreviations to avoid
-    // matching words like "Resets" when there's no space separator.
-    // Optional "resets\s*" prefix handles cases with or without space after "Resets"
     const dateMatch = text.match(
-      /(?:resets\s*)?(Jan|Feb|Mar|Apr|May|Jun|Jul|Aug|Sep|Oct|Nov|Dec)\s+(\d{1,2})(?:\s+at\s+|\s*,?\s*)(\d{1,2})(?::(\d{2}))?\s*(am|pm)/i
+      /([A-Za-z]{3,})\s+(\d{1,2})(?:\s+at\s+|\s*,?\s*)(\d{1,2})(?::(\d{2}))?\s*(am|pm)/i
     );
     if (dateMatch) {
       const monthName = dateMatch[1];

apps/server/src/services/event-hook-service.ts

@@ -21,7 +21,6 @@ import { createLogger } from '@automaker/utils';
 import type { EventEmitter } from '../lib/events.js';
 import type { SettingsService } from './settings-service.js';
 import type { EventHistoryService } from './event-history-service.js';
-import type { FeatureLoader } from './feature-loader.js';
 import type {
   EventHook,
   EventHookTrigger,
@@ -58,7 +57,6 @@ interface HookContext {
 interface AutoModeEventPayload {
   type?: string;
   featureId?: string;
-  featureName?: string;
   passes?: boolean;
   message?: string;
   error?: string;
@@ -85,22 +83,19 @@ export class EventHookService {
   private emitter: EventEmitter | null = null;
   private settingsService: SettingsService | null = null;
   private eventHistoryService: EventHistoryService | null = null;
-  private featureLoader: FeatureLoader | null = null;
   private unsubscribe: (() => void) | null = null;
 
   /**
-   * Initialize the service with event emitter, settings service, event history service, and feature loader
+   * Initialize the service with event emitter, settings service, and event history service
    */
   initialize(
     emitter: EventEmitter,
     settingsService: SettingsService,
-    eventHistoryService?: EventHistoryService,
-    featureLoader?: FeatureLoader
+    eventHistoryService?: EventHistoryService
   ): void {
     this.emitter = emitter;
     this.settingsService = settingsService;
     this.eventHistoryService = eventHistoryService || null;
-    this.featureLoader = featureLoader || null;
 
     // Subscribe to events
     this.unsubscribe = emitter.subscribe((type, payload) => {
@@ -125,7 +120,6 @@ export class EventHookService {
     this.emitter = null;
     this.settingsService = null;
     this.eventHistoryService = null;
-    this.featureLoader = null;
   }
 
   /**
@@ -155,23 +149,9 @@ export class EventHookService {
 
     if (!trigger) return;
 
-    // Load feature name if we have featureId but no featureName
-    let featureName: string | undefined = undefined;
-    if (payload.featureId && payload.projectPath && this.featureLoader) {
-      try {
-        const feature = await this.featureLoader.get(payload.projectPath, payload.featureId);
-        if (feature?.title) {
-          featureName = feature.title;
-        }
-      } catch (error) {
-        logger.warn(`Failed to load feature ${payload.featureId} for event hook:`, error);
-      }
-    }
-
     // Build context for variable substitution
     const context: HookContext = {
       featureId: payload.featureId,
-      featureName: payload.featureName,
       projectPath: payload.projectPath,
       projectName: payload.projectPath ? this.extractProjectName(payload.projectPath) : undefined,
       error: payload.error || payload.message,
@@ -333,7 +313,6 @@ export class EventHookService {
         eventType: context.eventType,
         timestamp: context.timestamp,
         featureId: context.featureId,
-        featureName: context.featureName,
         projectPath: context.projectPath,
         projectName: context.projectName,
         error: context.error,

apps/server/src/services/ideation-service.ts

@@ -223,9 +223,6 @@ export class IdeationService {
       // Strip provider prefix - providers need bare model IDs
       const bareModel = stripProviderPrefix(modelId);
 
-      // Get credentials for API calls (uses hardcoded model, no phase setting)
-      const credentials = await this.settingsService?.getCredentials();
-
       const executeOptions: ExecuteOptions = {
         prompt: message,
         model: bareModel,
@@ -235,7 +232,6 @@ export class IdeationService {
         maxTurns: 1, // Single turn for ideation
         abortController: activeSession.abortController!,
         conversationHistory: conversationHistory.length > 0 ? conversationHistory : undefined,
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
       };
 
       const stream = provider.executeQuery(executeOptions);
@@ -682,9 +678,6 @@ export class IdeationService {
       // Strip provider prefix - providers need bare model IDs
       const bareModel = stripProviderPrefix(modelId);
 
-      // Get credentials for API calls (uses hardcoded model, no phase setting)
-      const credentials = await this.settingsService?.getCredentials();
-
       const executeOptions: ExecuteOptions = {
         prompt: prompt.prompt,
         model: bareModel,
@@ -695,7 +688,6 @@ export class IdeationService {
         // Disable all tools - we just want text generation, not codebase analysis
         allowedTools: [],
         abortController: new AbortController(),
-        credentials, // Pass credentials for resolving 'credentials' apiKeySource
       };
 
       const stream = provider.executeQuery(executeOptions);

apps/server/src/services/settings-service.ts

@@ -9,9 +9,6 @@
 
 import { createLogger, atomicWriteJson, DEFAULT_BACKUP_COUNT } from '@automaker/utils';
 import * as secureFs from '../lib/secure-fs.js';
-import os from 'os';
-import path from 'path';
-import fs from 'fs/promises';
 
 import {
   getGlobalSettingsPath,
@@ -31,9 +28,6 @@ import type {
   WorktreeInfo,
   PhaseModelConfig,
   PhaseModelEntry,
-  ClaudeApiProfile,
-  ClaudeCompatibleProvider,
-  ProviderModel,
 } from '../types/settings.js';
 import {
   DEFAULT_GLOBAL_SETTINGS,
@@ -44,12 +38,6 @@ import {
   CREDENTIALS_VERSION,
   PROJECT_SETTINGS_VERSION,
 } from '../types/settings.js';
-import {
-  DEFAULT_MAX_CONCURRENCY,
-  migrateModelId,
-  migrateCursorModelIds,
-  migrateOpencodeModelIds,
-} from '@automaker/types';
 
 const logger = createLogger('SettingsService');
 
@@ -136,14 +124,10 @@ export class SettingsService {
     // Migrate legacy enhancementModel/validationModel to phaseModels
     const migratedPhaseModels = this.migratePhaseModels(settings);
 
-    // Migrate model IDs to canonical format
-    const migratedModelSettings = this.migrateModelSettings(settings);
-
     // Apply any missing defaults (for backwards compatibility)
     let result: GlobalSettings = {
       ...DEFAULT_GLOBAL_SETTINGS,
       ...settings,
-      ...migratedModelSettings,
       keyboardShortcuts: {
         ...DEFAULT_GLOBAL_SETTINGS.keyboardShortcuts,
         ...settings.keyboardShortcuts,
@@ -174,63 +158,6 @@ export class SettingsService {
       needsSave = true;
     }
 
-    // Migration v4 -> v5: Auto-create "Direct Anthropic" profile for existing users
-    // If user has an Anthropic API key in credentials but no profiles, create a
-    // "Direct Anthropic" profile that references the credentials and set it as active.
-    if (storedVersion < 5) {
-      try {
-        const credentials = await this.getCredentials();
-        const hasAnthropicKey = !!credentials.apiKeys?.anthropic;
-        const hasNoProfiles = !result.claudeApiProfiles || result.claudeApiProfiles.length === 0;
-        const hasNoActiveProfile = !result.activeClaudeApiProfileId;
-
-        if (hasAnthropicKey && hasNoProfiles && hasNoActiveProfile) {
-          const directAnthropicProfile = {
-            id: `profile-${Date.now()}-direct-anthropic`,
-            name: 'Direct Anthropic',
-            baseUrl: 'https://api.anthropic.com',
-            apiKeySource: 'credentials' as const,
-            useAuthToken: false,
-          };
-
-          result.claudeApiProfiles = [directAnthropicProfile];
-          result.activeClaudeApiProfileId = directAnthropicProfile.id;
-
-          logger.info(
-            'Migration v4->v5: Created "Direct Anthropic" profile using existing credentials'
-          );
-        }
-      } catch (error) {
-        logger.warn(
-          'Migration v4->v5: Could not check credentials for auto-profile creation:',
-          error
-        );
-      }
-      needsSave = true;
-    }
-
-    // Migration v5 -> v6: Convert claudeApiProfiles to claudeCompatibleProviders
-    // The new system uses a models[] array instead of modelMappings, and removes
-    // the "active profile" concept - models are selected directly in phase model configs.
-    if (storedVersion < 6) {
-      const legacyProfiles = settings.claudeApiProfiles || [];
-      if (
-        legacyProfiles.length > 0 &&
-        (!result.claudeCompatibleProviders || result.claudeCompatibleProviders.length === 0)
-      ) {
-        logger.info(
-          `Migration v5->v6: Converting ${legacyProfiles.length} Claude API profile(s) to compatible providers`
-        );
-        result.claudeCompatibleProviders = this.migrateProfilesToProviders(legacyProfiles);
-      }
-      // Remove the deprecated activeClaudeApiProfileId field
-      if (result.activeClaudeApiProfileId) {
-        logger.info('Migration v5->v6: Removing deprecated activeClaudeApiProfileId');
-        delete result.activeClaudeApiProfileId;
-      }
-      needsSave = true;
-    }
-
     // Update version if any migration occurred
     if (needsSave) {
       result.version = SETTINGS_VERSION;
@@ -296,203 +223,19 @@ export class SettingsService {
    * Convert a phase model value to PhaseModelEntry format
    *
    * Handles migration from string format (v2) to object format (v3).
-   * Also migrates legacy model IDs to canonical prefixed format.
-   * - String values like 'sonnet' become { model: 'claude-sonnet' }
-   * - Object values have their model ID migrated if needed
+   * - String values like 'sonnet' become { model: 'sonnet' }
+   * - Object values are returned as-is (with type assertion)
    *
    * @param value - Phase model value (string or PhaseModelEntry)
-   * @returns PhaseModelEntry object with canonical model ID
+   * @returns PhaseModelEntry object
    */
   private toPhaseModelEntry(value: string | PhaseModelEntry): PhaseModelEntry {
     if (typeof value === 'string') {
-      // v2 format: just a model string - migrate to canonical ID
-      return { model: migrateModelId(value) as PhaseModelEntry['model'] };
+      // v2 format: just a model string
+      return { model: value as PhaseModelEntry['model'] };
     }
-    // v3 format: PhaseModelEntry object - migrate model ID if needed
-    return {
-      ...value,
-      model: migrateModelId(value.model) as PhaseModelEntry['model'],
-    };
-  }
-
-  /**
-   * Migrate ClaudeApiProfiles to ClaudeCompatibleProviders
-   *
-   * Converts the legacy profile format (with modelMappings) to the new
-   * provider format (with models[] array). Each model mapping entry becomes
-   * a ProviderModel with appropriate tier assignment.
-   *
-   * @param profiles - Legacy ClaudeApiProfile array
-   * @returns Array of ClaudeCompatibleProvider
-   */
-  private migrateProfilesToProviders(profiles: ClaudeApiProfile[]): ClaudeCompatibleProvider[] {
-    return profiles.map((profile): ClaudeCompatibleProvider => {
-      // Convert modelMappings to models array
-      const models: ProviderModel[] = [];
-
-      if (profile.modelMappings) {
-        // Haiku mapping
-        if (profile.modelMappings.haiku) {
-          models.push({
-            id: profile.modelMappings.haiku,
-            displayName: this.inferModelDisplayName(profile.modelMappings.haiku, 'haiku'),
-            mapsToClaudeModel: 'haiku',
-          });
-        }
-        // Sonnet mapping
-        if (profile.modelMappings.sonnet) {
-          models.push({
-            id: profile.modelMappings.sonnet,
-            displayName: this.inferModelDisplayName(profile.modelMappings.sonnet, 'sonnet'),
-            mapsToClaudeModel: 'sonnet',
-          });
-        }
-        // Opus mapping
-        if (profile.modelMappings.opus) {
-          models.push({
-            id: profile.modelMappings.opus,
-            displayName: this.inferModelDisplayName(profile.modelMappings.opus, 'opus'),
-            mapsToClaudeModel: 'opus',
-          });
-        }
-      }
-
-      // Infer provider type from base URL or name
-      const providerType = this.inferProviderType(profile);
-
-      return {
-        id: profile.id,
-        name: profile.name,
-        providerType,
-        enabled: true,
-        baseUrl: profile.baseUrl,
-        apiKeySource: profile.apiKeySource ?? 'inline',
-        apiKey: profile.apiKey,
-        useAuthToken: profile.useAuthToken,
-        timeoutMs: profile.timeoutMs,
-        disableNonessentialTraffic: profile.disableNonessentialTraffic,
-        models,
-      };
-    });
-  }
-
-  /**
-   * Infer a display name for a model based on its ID and tier
-   *
-   * @param modelId - The raw model ID
-   * @param tier - The tier hint (haiku/sonnet/opus)
-   * @returns A user-friendly display name
-   */
-  private inferModelDisplayName(modelId: string, tier: 'haiku' | 'sonnet' | 'opus'): string {
-    // Common patterns in model IDs
-    const lowerModelId = modelId.toLowerCase();
-
-    // GLM models
-    if (lowerModelId.includes('glm')) {
-      return modelId.replace(/-/g, ' ').replace(/glm/i, 'GLM');
-    }
-
-    // MiniMax models
-    if (lowerModelId.includes('minimax')) {
-      return modelId.replace(/-/g, ' ').replace(/minimax/i, 'MiniMax');
-    }
-
-    // Claude models via OpenRouter or similar
-    if (lowerModelId.includes('claude')) {
-      return modelId;
-    }
-
-    // Default: use model ID as display name with tier in parentheses
-    return `${modelId} (${tier})`;
-  }
-
-  /**
-   * Infer provider type from profile configuration
-   *
-   * @param profile - The legacy profile
-   * @returns The inferred provider type
-   */
-  private inferProviderType(profile: ClaudeApiProfile): ClaudeCompatibleProvider['providerType'] {
-    const baseUrl = profile.baseUrl.toLowerCase();
-    const name = profile.name.toLowerCase();
-
-    // Check URL patterns
-    if (baseUrl.includes('z.ai') || baseUrl.includes('zhipuai')) {
-      return 'glm';
-    }
-    if (baseUrl.includes('minimax')) {
-      return 'minimax';
-    }
-    if (baseUrl.includes('openrouter')) {
-      return 'openrouter';
-    }
-    if (baseUrl.includes('anthropic.com')) {
-      return 'anthropic';
-    }
-
-    // Check name patterns
-    if (name.includes('glm') || name.includes('zhipu')) {
-      return 'glm';
-    }
-    if (name.includes('minimax')) {
-      return 'minimax';
-    }
-    if (name.includes('openrouter')) {
-      return 'openrouter';
-    }
-    if (name.includes('anthropic') || name.includes('direct')) {
-      return 'anthropic';
-    }
-
-    // Default to custom
-    return 'custom';
-  }
-
-  /**
-   * Migrate model-related settings to canonical format
-   *
-   * Migrates:
-   * - enabledCursorModels: legacy IDs to cursor- prefixed
-   * - enabledOpencodeModels: legacy slash format to dash format
-   * - cursorDefaultModel: legacy ID to cursor- prefixed
-   *
-   * @param settings - Settings to migrate
-   * @returns Settings with migrated model IDs
-   */
-  private migrateModelSettings(settings: Partial<GlobalSettings>): Partial<GlobalSettings> {
-    const migrated: Partial<GlobalSettings> = { ...settings };
-
-    // Migrate Cursor models
-    if (settings.enabledCursorModels) {
-      migrated.enabledCursorModels = migrateCursorModelIds(
-        settings.enabledCursorModels as string[]
-      );
-    }
-
-    // Migrate Cursor default model
-    if (settings.cursorDefaultModel) {
-      const migratedDefault = migrateCursorModelIds([settings.cursorDefaultModel as string]);
-      if (migratedDefault.length > 0) {
-        migrated.cursorDefaultModel = migratedDefault[0];
-      }
-    }
-
-    // Migrate OpenCode models
-    if (settings.enabledOpencodeModels) {
-      migrated.enabledOpencodeModels = migrateOpencodeModelIds(
-        settings.enabledOpencodeModels as string[]
-      );
-    }
-
-    // Migrate OpenCode default model
-    if (settings.opencodeDefaultModel) {
-      const migratedDefault = migrateOpencodeModelIds([settings.opencodeDefaultModel as string]);
-      if (migratedDefault.length > 0) {
-        migrated.opencodeDefaultModel = migratedDefault[0];
-      }
-    }
-
-    return migrated;
+    // v3 format: already a PhaseModelEntry object
+    return value;
   }
 
   /**
@@ -530,39 +273,13 @@ export class SettingsService {
     };
 
     const currentProjectsLen = Array.isArray(current.projects) ? current.projects.length : 0;
-    // Check if this is a legitimate project removal (moved to trash) vs accidental wipe
-    const newTrashedProjectsLen = Array.isArray(sanitizedUpdates.trashedProjects)
-      ? sanitizedUpdates.trashedProjects.length
-      : Array.isArray(current.trashedProjects)
-        ? current.trashedProjects.length
-        : 0;
-
     if (
       Array.isArray(sanitizedUpdates.projects) &&
       sanitizedUpdates.projects.length === 0 &&
       currentProjectsLen > 0
     ) {
-      // Only treat as accidental wipe if trashedProjects is also empty
-      // (If projects are moved to trash, they appear in trashedProjects)
-      if (newTrashedProjectsLen === 0) {
-        logger.warn(
-          '[WIPE_PROTECTION] Attempted to set projects to empty array with no trash! Ignoring update.',
-          {
-            currentProjectsLen,
-            newProjectsLen: 0,
-            newTrashedProjectsLen,
-            currentProjects: current.projects?.map((p) => p.name),
-          }
-        );
-        attemptedProjectWipe = true;
-        delete sanitizedUpdates.projects;
-      } else {
-        logger.info('[LEGITIMATE_REMOVAL] Removing all projects to trash', {
-          currentProjectsLen,
-          newProjectsLen: 0,
-          movedToTrash: newTrashedProjectsLen,
-        });
-      }
+      attemptedProjectWipe = true;
+      delete sanitizedUpdates.projects;
     }
 
     ignoreEmptyArrayOverwrite('trashedProjects');
@@ -570,29 +287,18 @@ export class SettingsService {
     ignoreEmptyArrayOverwrite('recentFolders');
     ignoreEmptyArrayOverwrite('mcpServers');
     ignoreEmptyArrayOverwrite('enabledCursorModels');
-    ignoreEmptyArrayOverwrite('claudeApiProfiles');
-    // Note: claudeCompatibleProviders intentionally NOT guarded - users should be able to delete all providers
 
     // Empty object overwrite guard
-    const ignoreEmptyObjectOverwrite = <K extends keyof GlobalSettings>(key: K): void => {
-      const nextVal = sanitizedUpdates[key] as unknown;
-      const curVal = current[key] as unknown;
-      if (
-        nextVal &&
-        typeof nextVal === 'object' &&
-        !Array.isArray(nextVal) &&
-        Object.keys(nextVal).length === 0 &&
-        curVal &&
-        typeof curVal === 'object' &&
-        !Array.isArray(curVal) &&
-        Object.keys(curVal).length > 0
-      ) {
-        delete sanitizedUpdates[key];
-      }
-    };
-
-    ignoreEmptyObjectOverwrite('lastSelectedSessionByProject');
-    ignoreEmptyObjectOverwrite('autoModeByWorktree');
+    if (
+      sanitizedUpdates.lastSelectedSessionByProject &&
+      typeof sanitizedUpdates.lastSelectedSessionByProject === 'object' &&
+      !Array.isArray(sanitizedUpdates.lastSelectedSessionByProject) &&
+      Object.keys(sanitizedUpdates.lastSelectedSessionByProject).length === 0 &&
+      current.lastSelectedSessionByProject &&
+      Object.keys(current.lastSelectedSessionByProject).length > 0
+    ) {
+      delete sanitizedUpdates.lastSelectedSessionByProject;
+    }
 
     // If a request attempted to wipe projects, also ignore theme changes in that same request.
     if (attemptedProjectWipe) {
@@ -806,27 +512,6 @@ export class SettingsService {
       };
     }
 
-    // Handle activeClaudeApiProfileId special cases:
-    // - "__USE_GLOBAL__" marker means delete the key (use global setting)
-    // - null means explicit "Direct Anthropic API"
-    // - string means specific profile ID
-    if (
-      'activeClaudeApiProfileId' in updates &&
-      updates.activeClaudeApiProfileId === '__USE_GLOBAL__'
-    ) {
-      delete updated.activeClaudeApiProfileId;
-    }
-
-    // Handle phaseModelOverrides special cases:
-    // - "__CLEAR__" marker means delete the key (use global settings for all phases)
-    // - object means partial overrides for specific phases
-    if (
-      'phaseModelOverrides' in updates &&
-      (updates as Record<string, unknown>).phaseModelOverrides === '__CLEAR__'
-    ) {
-      delete updated.phaseModelOverrides;
-    }
-
     await writeSettingsJson(settingsPath, updated);
     logger.info(`Project settings updated for ${projectPath}`);
 
@@ -912,7 +597,7 @@ export class SettingsService {
         theme: (appState.theme as GlobalSettings['theme']) || 'dark',
         sidebarOpen: appState.sidebarOpen !== undefined ? (appState.sidebarOpen as boolean) : true,
         chatHistoryOpen: (appState.chatHistoryOpen as boolean) || false,
-        maxConcurrency: (appState.maxConcurrency as number) || DEFAULT_MAX_CONCURRENCY,
+        maxConcurrency: (appState.maxConcurrency as number) || 3,
         defaultSkipTests:
           appState.defaultSkipTests !== undefined ? (appState.defaultSkipTests as boolean) : true,
         enableDependencyBlocking:
@@ -1081,203 +766,4 @@ export class SettingsService {
   getDataDir(): string {
     return this.dataDir;
   }
-
-  /**
-   * Get the legacy Electron userData directory path
-   *
-   * Returns the platform-specific path where Electron previously stored settings
-   * before the migration to shared data directories.
-   *
-   * @returns Absolute path to legacy userData directory
-   */
-  private getLegacyElectronUserDataPath(): string {
-    const homeDir = os.homedir();
-
-    switch (process.platform) {
-      case 'darwin':
-        // macOS: ~/Library/Application Support/Automaker
-        return path.join(homeDir, 'Library', 'Application Support', 'Automaker');
-      case 'win32':
-        // Windows: %APPDATA%\Automaker
-        return path.join(
-          process.env.APPDATA || path.join(homeDir, 'AppData', 'Roaming'),
-          'Automaker'
-        );
-      default:
-        // Linux and others: ~/.config/Automaker
-        return path.join(process.env.XDG_CONFIG_HOME || path.join(homeDir, '.config'), 'Automaker');
-    }
-  }
-
-  /**
-   * Migrate entire data directory from legacy Electron userData location to new shared data directory
-   *
-   * This handles the migration from when Electron stored data in the platform-specific
-   * userData directory (e.g., ~/.config/Automaker) to the new shared ./data directory.
-   *
-   * Migration only occurs if:
-   * 1. The new location does NOT have settings.json
-   * 2. The legacy location DOES have settings.json
-   *
-   * Migrates all files and directories including:
-   * - settings.json (global settings)
-   * - credentials.json (API keys)
-   * - sessions-metadata.json (chat session metadata)
-   * - agent-sessions/ (conversation histories)
-   * - Any other files in the data directory
-   *
-   * @returns Promise resolving to migration result
-   */
-  async migrateFromLegacyElectronPath(): Promise<{
-    migrated: boolean;
-    migratedFiles: string[];
-    legacyPath: string;
-    errors: string[];
-  }> {
-    const legacyPath = this.getLegacyElectronUserDataPath();
-    const migratedFiles: string[] = [];
-    const errors: string[] = [];
-
-    // Skip if legacy path is the same as current data dir (no migration needed)
-    if (path.resolve(legacyPath) === path.resolve(this.dataDir)) {
-      logger.debug('Legacy path same as current data dir, skipping migration');
-      return { migrated: false, migratedFiles, legacyPath, errors };
-    }
-
-    logger.info(`Checking for legacy data migration from: ${legacyPath}`);
-    logger.info(`Current data directory: ${this.dataDir}`);
-
-    // Check if new settings already exist
-    const newSettingsPath = getGlobalSettingsPath(this.dataDir);
-    let newSettingsExist = false;
-    try {
-      await fs.access(newSettingsPath);
-      newSettingsExist = true;
-    } catch {
-      // New settings don't exist, migration may be needed
-    }
-
-    if (newSettingsExist) {
-      logger.debug('Settings already exist in new location, skipping migration');
-      return { migrated: false, migratedFiles, legacyPath, errors };
-    }
-
-    // Check if legacy directory exists and has settings
-    const legacySettingsPath = path.join(legacyPath, 'settings.json');
-    let legacySettingsExist = false;
-    try {
-      await fs.access(legacySettingsPath);
-      legacySettingsExist = true;
-    } catch {
-      // Legacy settings don't exist
-    }
-
-    if (!legacySettingsExist) {
-      logger.debug('No legacy settings found, skipping migration');
-      return { migrated: false, migratedFiles, legacyPath, errors };
-    }
-
-    // Perform migration of specific application data files only
-    // (not Electron internal caches like Code Cache, GPU Cache, etc.)
-    logger.info('Found legacy data directory, migrating application data to new location...');
-
-    // Ensure new data directory exists
-    try {
-      await ensureDataDir(this.dataDir);
-    } catch (error) {
-      const msg = `Failed to create data directory: ${error}`;
-      logger.error(msg);
-      errors.push(msg);
-      return { migrated: false, migratedFiles, legacyPath, errors };
-    }
-
-    // Only migrate specific application data files/directories
-    const itemsToMigrate = [
-      'settings.json',
-      'credentials.json',
-      'sessions-metadata.json',
-      'agent-sessions',
-      '.api-key',
-      '.sessions',
-    ];
-
-    for (const item of itemsToMigrate) {
-      const srcPath = path.join(legacyPath, item);
-      const destPath = path.join(this.dataDir, item);
-
-      // Check if source exists
-      try {
-        await fs.access(srcPath);
-      } catch {
-        // Source doesn't exist, skip
-        continue;
-      }
-
-      // Check if destination already exists
-      try {
-        await fs.access(destPath);
-        logger.debug(`Skipping ${item} - already exists in destination`);
-        continue;
-      } catch {
-        // Destination doesn't exist, proceed with copy
-      }
-
-      // Copy file or directory
-      try {
-        const stat = await fs.stat(srcPath);
-        if (stat.isDirectory()) {
-          await this.copyDirectory(srcPath, destPath);
-          migratedFiles.push(item + '/');
-          logger.info(`Migrated directory: ${item}/`);
-        } else {
-          const content = await fs.readFile(srcPath);
-          await fs.writeFile(destPath, content);
-          migratedFiles.push(item);
-          logger.info(`Migrated file: ${item}`);
-        }
-      } catch (error) {
-        const msg = `Failed to migrate ${item}: ${error}`;
-        logger.error(msg);
-        errors.push(msg);
-      }
-    }
-
-    if (migratedFiles.length > 0) {
-      logger.info(
-        `Migration complete. Migrated ${migratedFiles.length} item(s): ${migratedFiles.join(', ')}`
-      );
-      logger.info(`Legacy path: ${legacyPath}`);
-      logger.info(`New path: ${this.dataDir}`);
-    }
-
-    return {
-      migrated: migratedFiles.length > 0,
-      migratedFiles,
-      legacyPath,
-      errors,
-    };
-  }
-
-  /**
-   * Recursively copy a directory from source to destination
-   *
-   * @param srcDir - Source directory path
-   * @param destDir - Destination directory path
-   */
-  private async copyDirectory(srcDir: string, destDir: string): Promise<void> {
-    await fs.mkdir(destDir, { recursive: true });
-    const entries = await fs.readdir(srcDir, { withFileTypes: true });
-
-    for (const entry of entries) {
-      const srcPath = path.join(srcDir, entry.name);
-      const destPath = path.join(destDir, entry.name);
-
-      if (entry.isDirectory()) {
-        await this.copyDirectory(srcPath, destPath);
-      } else if (entry.isFile()) {
-        const content = await fs.readFile(srcPath);
-        await fs.writeFile(destPath, content);
-      }
-    }
-  }
 }

apps/server/src/types/settings.ts

@@ -23,16 +23,6 @@ export type {
   PhaseModelConfig,
   PhaseModelKey,
   PhaseModelEntry,
-  // Claude-compatible provider types
-  ApiKeySource,
-  ClaudeCompatibleProviderType,
-  ClaudeModelAlias,
-  ProviderModel,
-  ClaudeCompatibleProvider,
-  ClaudeCompatibleProviderTemplate,
-  // Legacy profile types (deprecated)
-  ClaudeApiProfile,
-  ClaudeApiProfileTemplate,
 } from '@automaker/types';
 
 export {

apps/server/tests/unit/lib/model-resolver.test.ts

@@ -37,18 +37,17 @@ describe('model-resolver.ts', () => {
       const result = resolveModelString('opus');
       expect(result).toBe('claude-opus-4-5-20251101');
       expect(consoleSpy.log).toHaveBeenCalledWith(
-        expect.stringContaining('Migrated legacy ID: "opus" -> "claude-opus"')
+        expect.stringContaining('Resolved Claude model alias: "opus"')
       );
     });
 
-    it('should pass through unknown models unchanged (may be provider models)', () => {
-      // Unknown models now pass through unchanged to support ClaudeCompatibleProvider models
-      // like GLM-4.7, MiniMax-M2.1, o1, etc.
-      const models = ['o1', 'o1-mini', 'o3', 'unknown-model', 'fake-model-123', 'GLM-4.7'];
+    it('should treat unknown models as falling back to default', () => {
+      // Note: Don't include valid Cursor model IDs here (e.g., 'gpt-5.2' is in CURSOR_MODEL_MAP)
+      const models = ['o1', 'o1-mini', 'o3', 'unknown-model', 'fake-model-123'];
       models.forEach((model) => {
         const result = resolveModelString(model);
-        // Should pass through unchanged (could be provider models)
-        expect(result).toBe(model);
+        // Should fall back to default since these aren't supported
+        expect(result).toBe(DEFAULT_MODELS.claude);
       });
     });
 
@@ -74,12 +73,12 @@ describe('model-resolver.ts', () => {
       expect(result).toBe(customDefault);
     });
 
-    it('should pass through unknown model key unchanged (no warning)', () => {
+    it('should return default for unknown model key', () => {
       const result = resolveModelString('unknown-model');
-      // Unknown models pass through unchanged (could be provider models)
-      expect(result).toBe('unknown-model');
-      // No warning - unknown models are valid for providers
-      expect(consoleSpy.warn).not.toHaveBeenCalled();
+      expect(result).toBe(DEFAULT_MODELS.claude);
+      expect(consoleSpy.warn).toHaveBeenCalledWith(
+        expect.stringContaining('Unknown model key "unknown-model"')
+      );
     });
 
     it('should handle empty string', () => {

apps/server/tests/unit/lib/worktree-metadata.test.ts

@@ -121,7 +121,7 @@ describe('worktree-metadata.ts', () => {
           number: 123,
           url: 'https://github.com/owner/repo/pull/123',
           title: 'Test PR',
-          state: 'OPEN',
+          state: 'open',
           createdAt: new Date().toISOString(),
         },
       };
@@ -158,7 +158,7 @@ describe('worktree-metadata.ts', () => {
           number: 456,
           url: 'https://github.com/owner/repo/pull/456',
           title: 'Updated PR',
-          state: 'CLOSED',
+          state: 'closed',
           createdAt: new Date().toISOString(),
         },
       };
@@ -177,7 +177,7 @@ describe('worktree-metadata.ts', () => {
         number: 789,
         url: 'https://github.com/owner/repo/pull/789',
         title: 'New PR',
-        state: 'OPEN',
+        state: 'open',
         createdAt: new Date().toISOString(),
       };
 
@@ -201,7 +201,7 @@ describe('worktree-metadata.ts', () => {
         number: 999,
         url: 'https://github.com/owner/repo/pull/999',
         title: 'Updated PR',
-        state: 'MERGED',
+        state: 'merged',
         createdAt: new Date().toISOString(),
       };
 
@@ -224,7 +224,7 @@ describe('worktree-metadata.ts', () => {
         number: 111,
         url: 'https://github.com/owner/repo/pull/111',
         title: 'PR',
-        state: 'OPEN',
+        state: 'open',
         createdAt: new Date().toISOString(),
       };
 
@@ -259,7 +259,7 @@ describe('worktree-metadata.ts', () => {
         number: 222,
         url: 'https://github.com/owner/repo/pull/222',
         title: 'Has PR',
-        state: 'OPEN',
+        state: 'open',
         createdAt: new Date().toISOString(),
       };
 
@@ -297,7 +297,7 @@ describe('worktree-metadata.ts', () => {
           number: 333,
           url: 'https://github.com/owner/repo/pull/333',
           title: 'PR 3',
-          state: 'OPEN',
+          state: 'open',
           createdAt: new Date().toISOString(),
         },
       };

apps/server/tests/unit/providers/cursor-config-manager.test.ts

@@ -50,8 +50,8 @@ describe('cursor-config-manager.ts', () => {
       manager = new CursorConfigManager(testProjectPath);
 
       const config = manager.getConfig();
-      expect(config.defaultModel).toBe('cursor-auto');
-      expect(config.models).toContain('cursor-auto');
+      expect(config.defaultModel).toBe('auto');
+      expect(config.models).toContain('auto');
     });
 
     it('should use default config if file read fails', () => {
@@ -62,7 +62,7 @@ describe('cursor-config-manager.ts', () => {
 
       manager = new CursorConfigManager(testProjectPath);
 
-      expect(manager.getDefaultModel()).toBe('cursor-auto');
+      expect(manager.getDefaultModel()).toBe('auto');
     });
 
     it('should use default config if JSON parse fails', () => {
@@ -71,7 +71,7 @@ describe('cursor-config-manager.ts', () => {
 
       manager = new CursorConfigManager(testProjectPath);
 
-      expect(manager.getDefaultModel()).toBe('cursor-auto');
+      expect(manager.getDefaultModel()).toBe('auto');
     });
   });
 
@@ -93,7 +93,7 @@ describe('cursor-config-manager.ts', () => {
     });
 
     it('should return default model', () => {
-      expect(manager.getDefaultModel()).toBe('cursor-auto');
+      expect(manager.getDefaultModel()).toBe('auto');
     });
 
     it('should set and persist default model', () => {
@@ -103,13 +103,13 @@ describe('cursor-config-manager.ts', () => {
       expect(fs.writeFileSync).toHaveBeenCalled();
     });
 
-    it('should return cursor-auto if defaultModel is undefined', () => {
+    it('should return auto if defaultModel is undefined', () => {
       vi.mocked(fs.existsSync).mockReturnValue(true);
-      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ models: ['cursor-auto'] }));
+      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ models: ['auto'] }));
 
       manager = new CursorConfigManager(testProjectPath);
 
-      expect(manager.getDefaultModel()).toBe('cursor-auto');
+      expect(manager.getDefaultModel()).toBe('auto');
     });
   });
 
@@ -121,7 +121,7 @@ describe('cursor-config-manager.ts', () => {
     it('should return enabled models', () => {
       const models = manager.getEnabledModels();
       expect(Array.isArray(models)).toBe(true);
-      expect(models).toContain('cursor-auto');
+      expect(models).toContain('auto');
     });
 
     it('should set enabled models', () => {
@@ -131,13 +131,13 @@ describe('cursor-config-manager.ts', () => {
       expect(fs.writeFileSync).toHaveBeenCalled();
     });
 
-    it('should return [cursor-auto] if models is undefined', () => {
+    it('should return [auto] if models is undefined', () => {
       vi.mocked(fs.existsSync).mockReturnValue(true);
-      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ defaultModel: 'cursor-auto' }));
+      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ defaultModel: 'auto' }));
 
       manager = new CursorConfigManager(testProjectPath);
 
-      expect(manager.getEnabledModels()).toEqual(['cursor-auto']);
+      expect(manager.getEnabledModels()).toEqual(['auto']);
     });
   });
 
@@ -146,8 +146,8 @@ describe('cursor-config-manager.ts', () => {
       vi.mocked(fs.existsSync).mockReturnValue(true);
       vi.mocked(fs.readFileSync).mockReturnValue(
         JSON.stringify({
-          defaultModel: 'cursor-auto',
-          models: ['cursor-auto'],
+          defaultModel: 'auto',
+          models: ['auto'],
         })
       );
       manager = new CursorConfigManager(testProjectPath);
@@ -161,14 +161,14 @@ describe('cursor-config-manager.ts', () => {
     });
 
     it('should not add duplicate models', () => {
-      manager.addModel('cursor-auto');
+      manager.addModel('auto');
 
       // Should not save if model already exists
       expect(fs.writeFileSync).not.toHaveBeenCalled();
     });
 
     it('should initialize models array if undefined', () => {
-      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ defaultModel: 'cursor-auto' }));
+      vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify({ defaultModel: 'auto' }));
       manager = new CursorConfigManager(testProjectPath);
 
       manager.addModel('claude-3-5-sonnet');
@@ -293,7 +293,7 @@ describe('cursor-config-manager.ts', () => {
     it('should reset to default values', () => {
       manager.reset();
 
-      expect(manager.getDefaultModel()).toBe('cursor-auto');
+      expect(manager.getDefaultModel()).toBe('auto');
       expect(manager.getMcpServers()).toEqual([]);
       expect(manager.getRules()).toEqual([]);
       expect(fs.writeFileSync).toHaveBeenCalled();

apps/server/tests/unit/services/claude-usage-service.test.ts

@@ -124,59 +124,6 @@ describe('claude-usage-service.ts', () => {
 
       expect(result).toBe('Plain text');
     });
-
-    it('should strip OSC sequences (window title, etc.)', () => {
-      const service = new ClaudeUsageService();
-      // OSC sequence to set window title: ESC ] 0 ; title BEL
-      const input = '\x1B]0;Claude Code\x07Regular text';
-      // @ts-expect-error - accessing private method for testing
-      const result = service.stripAnsiCodes(input);
-
-      expect(result).toBe('Regular text');
-    });
-
-    it('should strip DEC private mode sequences', () => {
-      const service = new ClaudeUsageService();
-      // DEC private mode sequences like ESC[?2026h and ESC[?2026l
-      const input = '\x1B[?2026lClaude Code\x1B[?2026h more text';
-      // @ts-expect-error - accessing private method for testing
-      const result = service.stripAnsiCodes(input);
-
-      expect(result).toBe('Claude Code more text');
-    });
-
-    it('should handle complex terminal output with mixed escape sequences', () => {
-      const service = new ClaudeUsageService();
-      // Simulate the garbled output seen in the bug: "[?2026l ]0;❇ Claude Code [?2026h"
-      // This contains OSC (set title) and DEC private mode sequences
-      const input =
-        '\x1B[?2026l\x1B]0;❇ Claude Code\x07\x1B[?2026hCurrent session 0%used Resets3am';
-      // @ts-expect-error - accessing private method for testing
-      const result = service.stripAnsiCodes(input);
-
-      expect(result).toBe('Current session 0%used Resets3am');
-    });
-
-    it('should strip single character escape sequences', () => {
-      const service = new ClaudeUsageService();
-      // ESC c is the reset terminal command
-      const input = '\x1BcReset text';
-      // @ts-expect-error - accessing private method for testing
-      const result = service.stripAnsiCodes(input);
-
-      expect(result).toBe('Reset text');
-    });
-
-    it('should remove control characters but preserve newlines and tabs', () => {
-      const service = new ClaudeUsageService();
-      // BEL character (\x07) should be stripped, but the word "Bell" is regular text
-      const input = 'Line 1\nLine 2\tTabbed\x07 with bell';
-      // @ts-expect-error - accessing private method for testing
-      const result = service.stripAnsiCodes(input);
-
-      // BEL is stripped, newlines and tabs preserved
-      expect(result).toBe('Line 1\nLine 2\tTabbed with bell');
-    });
   });
 
   describe('parseResetTime', () => {

apps/server/tests/unit/services/ideation-service.test.ts

@@ -63,10 +63,7 @@ describe('IdeationService', () => {
     } as unknown as EventEmitter;
 
     // Create mock settings service
-    mockSettingsService = {
-      getCredentials: vi.fn().mockResolvedValue({}),
-      getGlobalSettings: vi.fn().mockResolvedValue({}),
-    } as unknown as SettingsService;
+    mockSettingsService = {} as SettingsService;
 
     // Create mock feature loader
     mockFeatureLoader = {

apps/server/tests/unit/services/settings-service.test.ts

@@ -647,10 +647,9 @@ describe('settings-service.ts', () => {
       const settings = await settingsService.getGlobalSettings();
 
       // Verify all phase models are now PhaseModelEntry objects
-      // Legacy aliases are migrated to canonical IDs
-      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'claude-sonnet' });
-      expect(settings.phaseModels.fileDescriptionModel).toEqual({ model: 'claude-haiku' });
-      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'claude-opus' });
+      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'sonnet' });
+      expect(settings.phaseModels.fileDescriptionModel).toEqual({ model: 'haiku' });
+      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'opus' });
       expect(settings.version).toBe(SETTINGS_VERSION);
     });
 
@@ -676,17 +675,16 @@ describe('settings-service.ts', () => {
       const settings = await settingsService.getGlobalSettings();
 
       // Verify PhaseModelEntry objects are preserved with thinkingLevel
-      // Legacy aliases are migrated to canonical IDs
       expect(settings.phaseModels.enhancementModel).toEqual({
-        model: 'claude-sonnet',
+        model: 'sonnet',
         thinkingLevel: 'high',
       });
       expect(settings.phaseModels.specGenerationModel).toEqual({
-        model: 'claude-opus',
+        model: 'opus',
         thinkingLevel: 'ultrathink',
       });
       expect(settings.phaseModels.backlogPlanningModel).toEqual({
-        model: 'claude-sonnet',
+        model: 'sonnet',
         thinkingLevel: 'medium',
       });
     });
@@ -712,15 +710,15 @@ describe('settings-service.ts', () => {
 
       const settings = await settingsService.getGlobalSettings();
 
-      // Strings should be converted to objects with canonical IDs
-      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'claude-sonnet' });
-      expect(settings.phaseModels.imageDescriptionModel).toEqual({ model: 'claude-haiku' });
-      // Objects should be preserved with migrated IDs
+      // Strings should be converted to objects
+      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'sonnet' });
+      expect(settings.phaseModels.imageDescriptionModel).toEqual({ model: 'haiku' });
+      // Objects should be preserved
       expect(settings.phaseModels.fileDescriptionModel).toEqual({
-        model: 'claude-haiku',
+        model: 'haiku',
         thinkingLevel: 'low',
       });
-      expect(settings.phaseModels.validationModel).toEqual({ model: 'claude-opus' });
+      expect(settings.phaseModels.validationModel).toEqual({ model: 'opus' });
     });
 
     it('should migrate legacy enhancementModel/validationModel fields', async () => {
@@ -737,11 +735,11 @@ describe('settings-service.ts', () => {
 
       const settings = await settingsService.getGlobalSettings();
 
-      // Legacy fields should be migrated to phaseModels with canonical IDs
-      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'claude-haiku' });
-      expect(settings.phaseModels.validationModel).toEqual({ model: 'claude-opus' });
-      // Other fields should use defaults (canonical IDs)
-      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'claude-opus' });
+      // Legacy fields should be migrated to phaseModels
+      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'haiku' });
+      expect(settings.phaseModels.validationModel).toEqual({ model: 'opus' });
+      // Other fields should use defaults
+      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'opus' });
     });
 
     it('should use default phase models when none are configured', async () => {
@@ -755,10 +753,10 @@ describe('settings-service.ts', () => {
 
       const settings = await settingsService.getGlobalSettings();
 
-      // Should use DEFAULT_PHASE_MODELS (with canonical IDs)
-      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'claude-sonnet' });
-      expect(settings.phaseModels.fileDescriptionModel).toEqual({ model: 'claude-haiku' });
-      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'claude-opus' });
+      // Should use DEFAULT_PHASE_MODELS
+      expect(settings.phaseModels.enhancementModel).toEqual({ model: 'sonnet' });
+      expect(settings.phaseModels.fileDescriptionModel).toEqual({ model: 'haiku' });
+      expect(settings.phaseModels.specGenerationModel).toEqual({ model: 'opus' });
     });
 
     it('should deep merge phaseModels on update', async () => {
@@ -778,13 +776,13 @@ describe('settings-service.ts', () => {
 
       const settings = await settingsService.getGlobalSettings();
 
-      // Both should be preserved (models migrated to canonical format)
+      // Both should be preserved
       expect(settings.phaseModels.enhancementModel).toEqual({
-        model: 'claude-sonnet',
+        model: 'sonnet',
         thinkingLevel: 'high',
       });
       expect(settings.phaseModels.specGenerationModel).toEqual({
-        model: 'claude-opus',
+        model: 'opus',
         thinkingLevel: 'ultrathink',
       });
     });

apps/ui/package.json

@@ -40,7 +40,6 @@
   },
   "dependencies": {
     "@automaker/dependency-resolver": "1.0.0",
-    "@automaker/spec-parser": "1.0.0",
     "@automaker/types": "1.0.0",
     "@codemirror/lang-xml": "6.1.0",
     "@codemirror/language": "^6.12.1",
@@ -80,8 +79,7 @@
     "@radix-ui/react-switch": "1.2.6",
     "@radix-ui/react-tabs": "1.1.13",
     "@radix-ui/react-tooltip": "1.2.8",
-    "@tanstack/react-query": "^5.90.17",
-    "@tanstack/react-query-devtools": "^5.91.2",
+    "@tanstack/react-query": "5.90.12",
     "@tanstack/react-router": "1.141.6",
     "@uiw/react-codemirror": "4.25.4",
     "@xterm/addon-fit": "0.10.0",
@@ -147,7 +145,6 @@
     "productName": "Automaker",
     "artifactName": "${productName}-${version}-${arch}.${ext}",
     "npmRebuild": false,
-    "publish": null,
     "afterPack": "./scripts/rebuild-server-natives.cjs",
     "directories": {
       "output": "release"

apps/ui/src/components/claude-usage-popover.tsx

@@ -1,40 +1,114 @@
-/**
- * Claude Usage Popover
- *
- * Displays Claude API usage statistics using React Query for data fetching.
- */
-
-import { useState, useMemo } from 'react';
+import { useState, useEffect, useMemo, useCallback } from 'react';
 import { Popover, PopoverContent, PopoverTrigger } from '@/components/ui/popover';
 import { Button } from '@/components/ui/button';
 import { RefreshCw, AlertTriangle, CheckCircle, XCircle, Clock, ExternalLink } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
+import { getElectronAPI } from '@/lib/electron';
+import { useAppStore } from '@/store/app-store';
 import { useSetupStore } from '@/store/setup-store';
-import { useClaudeUsage } from '@/hooks/queries';
+
+// Error codes for distinguishing failure modes
+const ERROR_CODES = {
+  API_BRIDGE_UNAVAILABLE: 'API_BRIDGE_UNAVAILABLE',
+  AUTH_ERROR: 'AUTH_ERROR',
+  TRUST_PROMPT: 'TRUST_PROMPT',
+  UNKNOWN: 'UNKNOWN',
+} as const;
+
+type ErrorCode = (typeof ERROR_CODES)[keyof typeof ERROR_CODES];
+
+type UsageError = {
+  code: ErrorCode;
+  message: string;
+};
+
+// Fixed refresh interval (45 seconds)
+const REFRESH_INTERVAL_SECONDS = 45;
 
 export function ClaudeUsagePopover() {
+  const { claudeUsage, claudeUsageLastUpdated, setClaudeUsage } = useAppStore();
   const claudeAuthStatus = useSetupStore((state) => state.claudeAuthStatus);
   const [open, setOpen] = useState(false);
+  const [loading, setLoading] = useState(false);
+  const [error, setError] = useState<UsageError | null>(null);
 
   // Check if CLI is verified/authenticated
   const isCliVerified =
     claudeAuthStatus?.authenticated && claudeAuthStatus?.method === 'cli_authenticated';
 
-  // Use React Query for usage data
-  const {
-    data: claudeUsage,
-    isLoading,
-    isFetching,
-    error,
-    dataUpdatedAt,
-    refetch,
-  } = useClaudeUsage(isCliVerified);
-
-  // Check if data is stale (older than 2 minutes)
+  // Check if data is stale (older than 2 minutes) - recalculates when claudeUsageLastUpdated changes
   const isStale = useMemo(() => {
-    return !dataUpdatedAt || Date.now() - dataUpdatedAt > 2 * 60 * 1000;
-  }, [dataUpdatedAt]);
+    return !claudeUsageLastUpdated || Date.now() - claudeUsageLastUpdated > 2 * 60 * 1000;
+  }, [claudeUsageLastUpdated]);
+
+  const fetchUsage = useCallback(
+    async (isAutoRefresh = false) => {
+      if (!isAutoRefresh) setLoading(true);
+      setError(null);
+      try {
+        const api = getElectronAPI();
+        if (!api.claude) {
+          setError({
+            code: ERROR_CODES.API_BRIDGE_UNAVAILABLE,
+            message: 'Claude API bridge not available',
+          });
+          return;
+        }
+        const data = await api.claude.getUsage();
+        if ('error' in data) {
+          // Detect trust prompt error
+          const isTrustPrompt =
+            data.error === 'Trust prompt pending' ||
+            (data.message && data.message.includes('folder permission'));
+          setError({
+            code: isTrustPrompt ? ERROR_CODES.TRUST_PROMPT : ERROR_CODES.AUTH_ERROR,
+            message: data.message || data.error,
+          });
+          return;
+        }
+        setClaudeUsage(data);
+      } catch (err) {
+        setError({
+          code: ERROR_CODES.UNKNOWN,
+          message: err instanceof Error ? err.message : 'Failed to fetch usage',
+        });
+      } finally {
+        if (!isAutoRefresh) setLoading(false);
+      }
+    },
+    [setClaudeUsage]
+  );
+
+  // Auto-fetch on mount if data is stale (only if CLI is verified)
+  useEffect(() => {
+    if (isStale && isCliVerified) {
+      fetchUsage(true);
+    }
+  }, [isStale, isCliVerified, fetchUsage]);
+
+  useEffect(() => {
+    // Skip if CLI is not verified
+    if (!isCliVerified) return;
+
+    // Initial fetch when opened
+    if (open) {
+      if (!claudeUsage || isStale) {
+        fetchUsage();
+      }
+    }
+
+    // Auto-refresh interval (only when open)
+    let intervalId: NodeJS.Timeout | null = null;
+    if (open) {
+      intervalId = setInterval(() => {
+        fetchUsage(true);
+      }, REFRESH_INTERVAL_SECONDS * 1000);
+    }
+
+    return () => {
+      if (intervalId) clearInterval(intervalId);
+    };
+  }, [open, claudeUsage, isStale, isCliVerified, fetchUsage]);
 
   // Derived status color/icon helper
   const getStatusInfo = (percentage: number) => {
@@ -69,6 +143,7 @@ export function ClaudeUsagePopover() {
     isPrimary?: boolean;
     stale?: boolean;
   }) => {
+    // Check if percentage is valid (not NaN, not undefined, is a finite number)
     const isValidPercentage =
       typeof percentage === 'number' && !isNaN(percentage) && isFinite(percentage);
     const safePercentage = isValidPercentage ? percentage : 0;
@@ -169,10 +244,10 @@ export function ClaudeUsagePopover() {
             <Button
               variant="ghost"
               size="icon"
-              className={cn('h-6 w-6', isFetching && 'opacity-80')}
-              onClick={() => !isFetching && refetch()}
+              className={cn('h-6 w-6', loading && 'opacity-80')}
+              onClick={() => !loading && fetchUsage(false)}
             >
-              <RefreshCw className={cn('w-3.5 h-3.5', isFetching && 'animate-spin')} />
+              <RefreshCw className="w-3.5 h-3.5" />
             </Button>
           )}
         </div>
@@ -183,18 +258,28 @@ export function ClaudeUsagePopover() {
             <div className="flex flex-col items-center justify-center py-6 text-center space-y-3">
               <AlertTriangle className="w-8 h-8 text-yellow-500/80" />
               <div className="space-y-1 flex flex-col items-center">
-                <p className="text-sm font-medium">
-                  {error instanceof Error ? error.message : 'Failed to fetch usage'}
-                </p>
+                <p className="text-sm font-medium">{error.message}</p>
                 <p className="text-xs text-muted-foreground">
-                  Make sure Claude CLI is installed and authenticated via{' '}
-                  <code className="font-mono bg-muted px-1 rounded">claude login</code>
+                  {error.code === ERROR_CODES.API_BRIDGE_UNAVAILABLE ? (
+                    'Ensure the Electron bridge is running or restart the app'
+                  ) : error.code === ERROR_CODES.TRUST_PROMPT ? (
+                    <>
+                      Run <code className="font-mono bg-muted px-1 rounded">claude</code> in your
+                      terminal and approve access to continue
+                    </>
+                  ) : (
+                    <>
+                      Make sure Claude CLI is installed and authenticated via{' '}
+                      <code className="font-mono bg-muted px-1 rounded">claude login</code>
+                    </>
+                  )}
                 </p>
               </div>
             </div>
-          ) : isLoading || !claudeUsage ? (
+          ) : !claudeUsage ? (
+            // Loading state
             <div className="flex flex-col items-center justify-center py-8 space-y-2">
-              <Spinner size="lg" />
+              <RefreshCw className="w-6 h-6 animate-spin text-muted-foreground/50" />
               <p className="text-xs text-muted-foreground">Loading usage data...</p>
             </div>
           ) : (

apps/ui/src/components/codex-usage-popover.tsx

@@ -1,11 +1,11 @@
-import { useState, useMemo } from 'react';
+import { useState, useEffect, useMemo, useCallback } from 'react';
 import { Popover, PopoverContent, PopoverTrigger } from '@/components/ui/popover';
 import { Button } from '@/components/ui/button';
 import { RefreshCw, AlertTriangle, CheckCircle, XCircle, Clock, ExternalLink } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
+import { getElectronAPI } from '@/lib/electron';
+import { useAppStore } from '@/store/app-store';
 import { useSetupStore } from '@/store/setup-store';
-import { useCodexUsage } from '@/hooks/queries';
 
 // Error codes for distinguishing failure modes
 const ERROR_CODES = {
@@ -22,6 +22,9 @@ type UsageError = {
   message: string;
 };
 
+// Fixed refresh interval (45 seconds)
+const REFRESH_INTERVAL_SECONDS = 45;
+
 // Helper to format reset time
 function formatResetTime(unixTimestamp: number): string {
   const date = new Date(unixTimestamp * 1000);
@@ -59,39 +62,95 @@ function getWindowLabel(durationMins: number): { title: string; subtitle: string
 }
 
 export function CodexUsagePopover() {
+  const { codexUsage, codexUsageLastUpdated, setCodexUsage } = useAppStore();
   const codexAuthStatus = useSetupStore((state) => state.codexAuthStatus);
   const [open, setOpen] = useState(false);
+  const [loading, setLoading] = useState(false);
+  const [error, setError] = useState<UsageError | null>(null);
 
   // Check if Codex is authenticated
   const isCodexAuthenticated = codexAuthStatus?.authenticated;
 
-  // Use React Query for data fetching with automatic polling
-  const {
-    data: codexUsage,
-    isLoading,
-    isFetching,
-    error: queryError,
-    dataUpdatedAt,
-    refetch,
-  } = useCodexUsage(isCodexAuthenticated);
-
   // Check if data is stale (older than 2 minutes)
   const isStale = useMemo(() => {
-    return !dataUpdatedAt || Date.now() - dataUpdatedAt > 2 * 60 * 1000;
-  }, [dataUpdatedAt]);
+    return !codexUsageLastUpdated || Date.now() - codexUsageLastUpdated > 2 * 60 * 1000;
+  }, [codexUsageLastUpdated]);
 
-  // Convert query error to UsageError format for backward compatibility
-  const error = useMemo((): UsageError | null => {
-    if (!queryError) return null;
-    const message = queryError instanceof Error ? queryError.message : String(queryError);
-    if (message.includes('not available') || message.includes('does not provide')) {
-      return { code: ERROR_CODES.NOT_AVAILABLE, message };
+  const fetchUsage = useCallback(
+    async (isAutoRefresh = false) => {
+      if (!isAutoRefresh) setLoading(true);
+      setError(null);
+      try {
+        const api = getElectronAPI();
+        if (!api.codex) {
+          setError({
+            code: ERROR_CODES.API_BRIDGE_UNAVAILABLE,
+            message: 'Codex API bridge not available',
+          });
+          return;
+        }
+        const data = await api.codex.getUsage();
+        if ('error' in data) {
+          // Check if it's the "not available" error
+          if (
+            data.message?.includes('not available') ||
+            data.message?.includes('does not provide')
+          ) {
+            setError({
+              code: ERROR_CODES.NOT_AVAILABLE,
+              message: data.message || data.error,
+            });
+          } else {
+            setError({
+              code: ERROR_CODES.AUTH_ERROR,
+              message: data.message || data.error,
+            });
+          }
+          return;
+        }
+        setCodexUsage(data);
+      } catch (err) {
+        setError({
+          code: ERROR_CODES.UNKNOWN,
+          message: err instanceof Error ? err.message : 'Failed to fetch usage',
+        });
+      } finally {
+        if (!isAutoRefresh) setLoading(false);
+      }
+    },
+    [setCodexUsage]
+  );
+
+  // Auto-fetch on mount if data is stale (only if authenticated)
+  useEffect(() => {
+    if (isStale && isCodexAuthenticated) {
+      fetchUsage(true);
     }
-    if (message.includes('bridge') || message.includes('API')) {
-      return { code: ERROR_CODES.API_BRIDGE_UNAVAILABLE, message };
+  }, [isStale, isCodexAuthenticated, fetchUsage]);
+
+  useEffect(() => {
+    // Skip if not authenticated
+    if (!isCodexAuthenticated) return;
+
+    // Initial fetch when opened
+    if (open) {
+      if (!codexUsage || isStale) {
+        fetchUsage();
+      }
     }
-    return { code: ERROR_CODES.AUTH_ERROR, message };
-  }, [queryError]);
+
+    // Auto-refresh interval (only when open)
+    let intervalId: NodeJS.Timeout | null = null;
+    if (open) {
+      intervalId = setInterval(() => {
+        fetchUsage(true);
+      }, REFRESH_INTERVAL_SECONDS * 1000);
+    }
+
+    return () => {
+      if (intervalId) clearInterval(intervalId);
+    };
+  }, [open, codexUsage, isStale, isCodexAuthenticated, fetchUsage]);
 
   // Derived status color/icon helper
   const getStatusInfo = (percentage: number) => {
@@ -229,10 +288,10 @@ export function CodexUsagePopover() {
             <Button
               variant="ghost"
               size="icon"
-              className={cn('h-6 w-6', isFetching && 'opacity-80')}
-              onClick={() => !isFetching && refetch()}
+              className={cn('h-6 w-6', loading && 'opacity-80')}
+              onClick={() => !loading && fetchUsage(false)}
             >
-              <RefreshCw className={cn('w-3.5 h-3.5', isFetching && 'animate-spin')} />
+              <RefreshCw className="w-3.5 h-3.5" />
             </Button>
           )}
         </div>
@@ -274,7 +333,7 @@ export function CodexUsagePopover() {
           ) : !codexUsage ? (
             // Loading state
             <div className="flex flex-col items-center justify-center py-8 space-y-2">
-              <Spinner size="lg" />
+              <RefreshCw className="w-6 h-6 animate-spin text-muted-foreground/50" />
               <p className="text-xs text-muted-foreground">Loading usage data...</p>
             </div>
           ) : codexUsage.rateLimits ? (

apps/ui/src/components/dialogs/board-background-modal.tsx

@@ -1,7 +1,6 @@
 import { useState, useRef, useCallback, useEffect } from 'react';
 import { createLogger } from '@automaker/utils/logger';
-import { ImageIcon, Upload, Trash2 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { ImageIcon, Upload, Loader2, Trash2 } from 'lucide-react';
 
 const logger = createLogger('BoardBackgroundModal');
 import {
@@ -314,7 +313,7 @@ export function BoardBackgroundModal({ open, onOpenChange }: BoardBackgroundModa
                     />
                     {isProcessing && (
                       <div className="absolute inset-0 flex items-center justify-center bg-background/80">
-                        <Spinner size="lg" />
+                        <Loader2 className="w-6 h-6 animate-spin text-brand-500" />
                       </div>
                     )}
                   </div>
@@ -354,7 +353,7 @@ export function BoardBackgroundModal({ open, onOpenChange }: BoardBackgroundModa
                     )}
                   >
                     {isProcessing ? (
-                      <Spinner size="lg" />
+                      <Upload className="h-6 w-6 animate-spin text-muted-foreground" />
                     ) : (
                       <ImageIcon className="h-6 w-6 text-muted-foreground" />
                     )}

apps/ui/src/components/dialogs/new-project-modal.tsx

@@ -14,8 +14,16 @@ import { Input } from '@/components/ui/input';
 import { Label } from '@/components/ui/label';
 import { Tabs, TabsContent, TabsList, TabsTrigger } from '@/components/ui/tabs';
 import { Badge } from '@/components/ui/badge';
-import { FolderPlus, FolderOpen, Rocket, ExternalLink, Check, Link, Folder } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import {
+  FolderPlus,
+  FolderOpen,
+  Rocket,
+  ExternalLink,
+  Check,
+  Loader2,
+  Link,
+  Folder,
+} from 'lucide-react';
 import { starterTemplates, type StarterTemplate } from '@/lib/templates';
 import { getElectronAPI } from '@/lib/electron';
 import { cn } from '@/lib/utils';
@@ -443,7 +451,7 @@ export function NewProjectModal({
           >
             {isCreating ? (
               <>
-                <Spinner size="sm" className="mr-2" />
+                <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                 {activeTab === 'template' ? 'Cloning...' : 'Creating...'}
               </>
             ) : (

apps/ui/src/components/dialogs/workspace-picker-modal.tsx

@@ -1,3 +1,4 @@
+import { useState, useEffect, useCallback } from 'react';
 import {
   Dialog,
   DialogContent,
@@ -7,9 +8,8 @@ import {
   DialogTitle,
 } from '@/components/ui/dialog';
 import { Button } from '@/components/ui/button';
-import { Folder, FolderOpen, AlertCircle } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
-import { useWorkspaceDirectories } from '@/hooks/queries';
+import { Folder, Loader2, FolderOpen, AlertCircle } from 'lucide-react';
+import { getHttpApiClient } from '@/lib/http-api-client';
 
 interface WorkspaceDirectory {
   name: string;
@@ -23,15 +23,41 @@ interface WorkspacePickerModalProps {
 }
 
 export function WorkspacePickerModal({ open, onOpenChange, onSelect }: WorkspacePickerModalProps) {
-  // React Query hook - only fetch when modal is open
-  const { data: directories = [], isLoading, error, refetch } = useWorkspaceDirectories(open);
+  const [isLoading, setIsLoading] = useState(false);
+  const [directories, setDirectories] = useState<WorkspaceDirectory[]>([]);
+  const [error, setError] = useState<string | null>(null);
+
+  const loadDirectories = useCallback(async () => {
+    setIsLoading(true);
+    setError(null);
+
+    try {
+      const client = getHttpApiClient();
+      const result = await client.workspace.getDirectories();
+
+      if (result.success && result.directories) {
+        setDirectories(result.directories);
+      } else {
+        setError(result.error || 'Failed to load directories');
+      }
+    } catch (err) {
+      setError(err instanceof Error ? err.message : 'Failed to load directories');
+    } finally {
+      setIsLoading(false);
+    }
+  }, []);
+
+  // Load directories when modal opens
+  useEffect(() => {
+    if (open) {
+      loadDirectories();
+    }
+  }, [open, loadDirectories]);
 
   const handleSelect = (dir: WorkspaceDirectory) => {
     onSelect(dir.path, dir.name);
   };
 
-  const errorMessage = error instanceof Error ? error.message : null;
-
   return (
     <Dialog open={open} onOpenChange={onOpenChange}>
       <DialogContent className="bg-card border-border max-w-lg max-h-[80vh] flex flex-col">
@@ -48,24 +74,24 @@ export function WorkspacePickerModal({ open, onOpenChange, onSelect }: Workspace
         <div className="flex-1 overflow-y-auto py-4 min-h-[200px]">
           {isLoading && (
             <div className="flex flex-col items-center justify-center h-full gap-3">
-              <Spinner size="xl" />
+              <Loader2 className="w-8 h-8 text-brand-500 animate-spin" />
               <p className="text-sm text-muted-foreground">Loading projects...</p>
             </div>
           )}
 
-          {errorMessage && !isLoading && (
+          {error && !isLoading && (
             <div className="flex flex-col items-center justify-center h-full gap-3 text-center px-4">
               <div className="w-12 h-12 rounded-full bg-destructive/10 flex items-center justify-center">
                 <AlertCircle className="w-6 h-6 text-destructive" />
               </div>
-              <p className="text-sm text-destructive">{errorMessage}</p>
-              <Button variant="secondary" size="sm" onClick={() => refetch()} className="mt-2">
+              <p className="text-sm text-destructive">{error}</p>
+              <Button variant="secondary" size="sm" onClick={loadDirectories} className="mt-2">
                 Try Again
               </Button>
             </div>
           )}
 
-          {!isLoading && !errorMessage && directories.length === 0 && (
+          {!isLoading && !error && directories.length === 0 && (
             <div className="flex flex-col items-center justify-center h-full gap-3 text-center px-4">
               <div className="w-12 h-12 rounded-full bg-muted flex items-center justify-center">
                 <Folder className="w-6 h-6 text-muted-foreground" />
@@ -76,7 +102,7 @@ export function WorkspacePickerModal({ open, onOpenChange, onSelect }: Workspace
             </div>
           )}
 
-          {!isLoading && !errorMessage && directories.length > 0 && (
+          {!isLoading && !error && directories.length > 0 && (
             <div className="space-y-2">
               {directories.map((dir) => (
                 <button

apps/ui/src/components/icons/terminal-icons.tsx

@@ -1,213 +0,0 @@
-import type { ComponentType, ComponentProps } from 'react';
-import { Terminal } from 'lucide-react';
-
-type IconProps = ComponentProps<'svg'>;
-type IconComponent = ComponentType<IconProps>;
-
-/**
- * iTerm2 logo icon
- */
-export function ITerm2Icon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M2.586 0a2.56 2.56 0 00-2.56 2.56v18.88A2.56 2.56 0 002.586 24h18.88a2.56 2.56 0 002.56-2.56V2.56A2.56 2.56 0 0021.466 0H2.586zm8.143 4.027h2.543v15.946h-2.543V4.027zm-3.816 0h2.544v15.946H6.913V4.027zm7.633 0h2.543v15.946h-2.543V4.027z" />
-    </svg>
-  );
-}
-
-/**
- * Warp terminal logo icon
- */
-export function WarpIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M9.2 4.8L7.6 7.6 4.8 5.6l1.6-2.8L9.2 4.8zm5.6 0l1.6 2.8 2.8-2-1.6-2.8-2.8 2zM2.4 12l2.8 1.6L3.6 16 .8 14.4 2.4 12zm19.2 0l1.6 2.4-2.8 1.6-1.6-2.4 2.8-1.6zM7.6 16.4l1.6 2.8-2.8 2-1.6-2.8 2.8-2zm8.8 0l2.8 2-1.6 2.8-2.8-2 1.6-2.8zM12 0L8.4 2 12 4l3.6-2L12 0zm0 20l-3.6 2 3.6 2 3.6-2-3.6-2z" />
-    </svg>
-  );
-}
-
-/**
- * Ghostty terminal logo icon
- */
-export function GhosttyIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M12 2C6.48 2 2 6.48 2 12v8c0 1.1.9 2 2 2h16c1.1 0 2-.9 2-2v-8c0-5.52-4.48-10-10-10zm-3.5 12a1.5 1.5 0 110-3 1.5 1.5 0 010 3zm7 0a1.5 1.5 0 110-3 1.5 1.5 0 010 3zM12 19c-1.5 0-3-.5-4-1.5v-1c2 1 6 1 8 0v1c-1 1-2.5 1.5-4 1.5z" />
-    </svg>
-  );
-}
-
-/**
- * Alacritty terminal logo icon
- */
-export function AlacrittyIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M12 0L1.608 21.6h3.186l1.46-3.032h11.489l1.46 3.032h3.189L12 0zm0 7.29l3.796 7.882H8.204L12 7.29z" />
-    </svg>
-  );
-}
-
-/**
- * WezTerm terminal logo icon
- */
-export function WezTermIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M2 4h20v16H2V4zm2 2v12h16V6H4zm2 2h12v2H6V8zm0 4h8v2H6v-2z" />
-    </svg>
-  );
-}
-
-/**
- * Kitty terminal logo icon
- */
-export function KittyIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M3.5 7.5L1 5V2.5L3.5 5V7.5zM20.5 7.5L23 5V2.5L20.5 5V7.5zM12 4L6 8v8l6 4 6-4V8l-6-4zm0 2l4 2.67v5.33L12 16.67 8 14V8.67L12 6z" />
-    </svg>
-  );
-}
-
-/**
- * Hyper terminal logo icon
- */
-export function HyperIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M11.857 23.995v-7.125H6.486l5.765-10.856-.363-1.072L7.803.001 0 12.191h5.75L0 23.995h11.857zm.286 0h5.753l5.679-11.804h-5.679l5.679-11.804L17.896.388l-5.753 11.803h5.753L12.143 24z" />
-    </svg>
-  );
-}
-
-/**
- * Tabby terminal logo icon
- */
-export function TabbyIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M12 2L4 6v12l8 4 8-4V6l-8-4zm0 2l6 3v10l-6 3-6-3V7l6-3z" />
-    </svg>
-  );
-}
-
-/**
- * Rio terminal logo icon
- */
-export function RioIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M12 2C6.48 2 2 6.48 2 12s4.48 10 10 10 10-4.48 10-10S17.52 2 12 2zm0 18c-4.41 0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8zm-1-13h2v6h-2zm0 8h2v2h-2z" />
-    </svg>
-  );
-}
-
-/**
- * Windows Terminal logo icon
- */
-export function WindowsTerminalIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M8.165 6L0 9.497v5.006L8.165 18l.413-.206v-4.025L3.197 12l5.381-1.769V6.206L8.165 6zm7.67 0l-.413.206v4.025L20.803 12l-5.381 1.769v4.025l.413.206L24 14.503V9.497L15.835 6z" />
-    </svg>
-  );
-}
-
-/**
- * PowerShell logo icon
- */
-export function PowerShellIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M23.181 2.974c.568 0 .923.463.792 1.035l-3.659 15.982c-.13.572-.697 1.035-1.265 1.035H.819c-.568 0-.923-.463-.792-1.035L3.686 4.009c.13-.572.697-1.035 1.265-1.035h18.23zM8.958 16.677c0 .334.276.611.611.611h3.673a.615.615 0 00.611-.611.615.615 0 00-.611-.611h-3.673a.615.615 0 00-.611.611zm5.126-7.016L9.025 14.72c-.241.241-.241.63 0 .872.241.241.63.241.872 0l5.059-5.059c.241-.241.241-.63 0-.872l-5.059-5.059c-.241-.241-.63-.241-.872 0-.241.241-.241.63 0 .872l5.059 5.059c-.334.334-.334.334 0 0z" />
-    </svg>
-  );
-}
-
-/**
- * Command Prompt (cmd) logo icon
- */
-export function CmdIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M2 4h20v16H2V4zm2 2v12h16V6H4zm2.5 1.5l3 3-3 3L5 12l3-3zm5.5 5h6v1.5h-6V12z" />
-    </svg>
-  );
-}
-
-/**
- * Git Bash logo icon
- */
-export function GitBashIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M23.546 10.93L13.067.452c-.604-.603-1.582-.603-2.188 0L8.708 2.627l2.76 2.76c.645-.215 1.379-.07 1.889.441.516.515.658 1.258.438 1.9l2.658 2.66c.645-.223 1.387-.078 1.9.435.721.72.721 1.884 0 2.604-.719.719-1.881.719-2.6 0-.539-.541-.674-1.337-.404-1.996L12.86 8.955v6.525c.176.086.342.203.488.348.713.721.713 1.883 0 2.6-.719.721-1.889.721-2.609 0-.719-.719-.719-1.879 0-2.598.182-.18.387-.316.605-.406V8.835c-.217-.091-.424-.222-.6-.401-.545-.545-.676-1.342-.396-2.009L7.636 3.7.45 10.881c-.6.605-.6 1.584 0 2.189l10.48 10.477c.604.604 1.582.604 2.186 0l10.43-10.43c.605-.603.605-1.582 0-2.187" />
-    </svg>
-  );
-}
-
-/**
- * GNOME Terminal logo icon
- */
-export function GnomeTerminalIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M2 4a2 2 0 00-2 2v12a2 2 0 002 2h20a2 2 0 002-2V6a2 2 0 00-2-2H2zm0 2h20v12H2V6zm2 2v2h2V8H4zm4 0v2h12V8H8zm-4 4v2h2v-2H4zm4 0v2h8v-2H8z" />
-    </svg>
-  );
-}
-
-/**
- * Konsole logo icon
- */
-export function KonsoleIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M3 3h18a2 2 0 012 2v14a2 2 0 01-2 2H3a2 2 0 01-2-2V5a2 2 0 012-2zm0 2v14h18V5H3zm2 2l4 4-4 4V7zm6 6h8v2h-8v-2z" />
-    </svg>
-  );
-}
-
-/**
- * macOS Terminal logo icon
- */
-export function MacOSTerminalIcon(props: IconProps) {
-  return (
-    <svg viewBox="0 0 24 24" fill="currentColor" xmlns="http://www.w3.org/2000/svg" {...props}>
-      <path d="M3 4a2 2 0 00-2 2v12a2 2 0 002 2h18a2 2 0 002-2V6a2 2 0 00-2-2H3zm0 2h18v12H3V6zm2 2l5 4-5 4V8zm7 6h7v2h-7v-2z" />
-    </svg>
-  );
-}
-
-/**
- * Get the appropriate icon component for a terminal ID
- */
-export function getTerminalIcon(terminalId: string): IconComponent {
-  const terminalIcons: Record<string, IconComponent> = {
-    iterm2: ITerm2Icon,
-    warp: WarpIcon,
-    ghostty: GhosttyIcon,
-    alacritty: AlacrittyIcon,
-    wezterm: WezTermIcon,
-    kitty: KittyIcon,
-    hyper: HyperIcon,
-    tabby: TabbyIcon,
-    rio: RioIcon,
-    'windows-terminal': WindowsTerminalIcon,
-    powershell: PowerShellIcon,
-    cmd: CmdIcon,
-    'git-bash': GitBashIcon,
-    'gnome-terminal': GnomeTerminalIcon,
-    konsole: KonsoleIcon,
-    'terminal-macos': MacOSTerminalIcon,
-    // Linux terminals - use generic terminal icon
-    'xfce4-terminal': Terminal,
-    tilix: Terminal,
-    terminator: Terminal,
-    foot: Terminal,
-    xterm: Terminal,
-  };
-
-  return terminalIcons[terminalId] ?? Terminal;
-}

apps/ui/src/components/layout/project-switcher/components/project-context-menu.tsx

@@ -1,4 +1,4 @@
-import { useEffect, useRef, useState, memo, useCallback, useMemo } from 'react';
+import { useEffect, useRef, useState, memo, useCallback } from 'react';
 import type { LucideIcon } from 'lucide-react';
 import { Edit2, Trash2, Palette, ChevronRight, Moon, Sun, Monitor } from 'lucide-react';
 import { toast } from 'sonner';
@@ -6,67 +6,35 @@ import { cn } from '@/lib/utils';
 import { type ThemeMode, useAppStore } from '@/store/app-store';
 import { ConfirmDialog } from '@/components/ui/confirm-dialog';
 import type { Project } from '@/lib/electron';
-import {
-  PROJECT_DARK_THEMES,
-  PROJECT_LIGHT_THEMES,
-  THEME_SUBMENU_CONSTANTS,
-} from '@/components/layout/sidebar/constants';
+import { PROJECT_DARK_THEMES, PROJECT_LIGHT_THEMES } from '@/components/layout/sidebar/constants';
 import { useThemePreview } from '@/components/layout/sidebar/hooks';
 
-/**
- * Constant representing the "use global theme" option.
- * An empty string is used to indicate that no project-specific theme is set.
- */
+// Constant for "use global theme" option
 const USE_GLOBAL_THEME = '' as const;
 
-/**
- * Z-index values for context menu layering.
- * Ensures proper stacking order when menus overlap.
- */
+// Constants for z-index values
 const Z_INDEX = {
-  /** Base z-index for the main context menu */
   CONTEXT_MENU: 100,
-  /** Higher z-index for theme submenu to appear above parent menu */
   THEME_SUBMENU: 101,
 } as const;
 
-/**
- * Represents a selectable theme option in the theme submenu.
- * Uses ThemeMode from app-store for type safety.
- */
+// Theme option type - using ThemeMode for type safety
 interface ThemeOption {
-  /** The theme mode value (e.g., 'dark', 'light', 'dracula') */
   value: ThemeMode;
-  /** Display label for the theme option */
   label: string;
-  /** Lucide icon component to display alongside the label */
   icon: LucideIcon;
-  /** CSS color value for the icon */
   color: string;
 }
 
-/**
- * Props for the ThemeButton component.
- * Defines the interface for rendering individual theme selection buttons.
- */
+// Reusable theme button component to avoid duplication (DRY principle)
 interface ThemeButtonProps {
-  /** The theme option data to display */
   option: ThemeOption;
-  /** Whether this theme is currently selected */
   isSelected: boolean;
-  /** Handler for pointer enter events (used for preview) */
   onPointerEnter: () => void;
-  /** Handler for pointer leave events (used to clear preview) */
   onPointerLeave: (e: React.PointerEvent) => void;
-  /** Handler for click events (used to select theme) */
   onClick: () => void;
 }
 
-/**
- * A reusable button component for individual theme options.
- * Implements hover preview and selection functionality.
- * Memoized to prevent unnecessary re-renders when parent state changes.
- */
 const ThemeButton = memo(function ThemeButton({
   option,
   isSelected,
@@ -95,33 +63,17 @@ const ThemeButton = memo(function ThemeButton({
   );
 });
 
-/**
- * Props for the ThemeColumn component.
- * Defines the interface for rendering a column of related theme options (e.g., dark or light themes).
- */
+// Reusable theme column component
 interface ThemeColumnProps {
-  /** Column header title (e.g., "Dark", "Light") */
   title: string;
-  /** Icon to display in the column header */
   icon: LucideIcon;
-  /** Array of theme options to display in this column */
   themes: ThemeOption[];
-  /** Currently selected theme value, or null if using global theme */
   selectedTheme: ThemeMode | null;
-  /** Handler called when user hovers over a theme option for preview */
   onPreviewEnter: (value: ThemeMode) => void;
-  /** Handler called when user stops hovering over a theme option */
   onPreviewLeave: (e: React.PointerEvent) => void;
-  /** Handler called when user clicks to select a theme */
   onSelect: (value: ThemeMode) => void;
 }
 
-/**
- * A reusable column component for displaying themed options.
- * Renders a group of related themes (e.g., all dark themes or all light themes)
- * with a header and scrollable list of ThemeButton components.
- * Memoized to prevent unnecessary re-renders.
- */
 const ThemeColumn = memo(function ThemeColumn({
   title,
   icon: Icon,
@@ -153,36 +105,13 @@ const ThemeColumn = memo(function ThemeColumn({
   );
 });
 
-/**
- * Props for the ProjectContextMenu component.
- * Defines the interface for the project right-click context menu.
- */
 interface ProjectContextMenuProps {
-  /** The project this context menu is for */
   project: Project;
-  /** Screen coordinates where the context menu should appear */
   position: { x: number; y: number };
-  /** Callback to close the context menu */
   onClose: () => void;
-  /** Callback when user selects "Edit Name & Icon" option */
   onEdit: (project: Project) => void;
 }
 
-/**
- * A context menu component for project-specific actions.
- *
- * Provides options for:
- * - Editing project name and icon
- * - Setting project-specific theme (with live preview on hover)
- * - Removing project from the workspace
- *
- * Features viewport-aware positioning for the theme submenu to prevent
- * overflow, and implements delayed hover handling to improve UX when
- * navigating between the trigger button and submenu.
- *
- * @param props - Component props
- * @returns The rendered context menu or null if not visible
- */
 export function ProjectContextMenu({
   project,
   position,
@@ -201,82 +130,9 @@ export function ProjectContextMenu({
   const [showThemeSubmenu, setShowThemeSubmenu] = useState(false);
   const [removeConfirmed, setRemoveConfirmed] = useState(false);
   const themeSubmenuRef = useRef<HTMLDivElement>(null);
-  const closeTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null);
 
   const { handlePreviewEnter, handlePreviewLeave } = useThemePreview({ setPreviewTheme });
 
-  // Handler to open theme submenu and cancel any pending close
-  const handleThemeMenuEnter = useCallback(() => {
-    // Cancel any pending close timeout
-    if (closeTimeoutRef.current) {
-      clearTimeout(closeTimeoutRef.current);
-      closeTimeoutRef.current = null;
-    }
-    setShowThemeSubmenu(true);
-  }, []);
-
-  // Handler to close theme submenu with a small delay
-  // This prevents the submenu from closing when mouse crosses the gap between trigger and submenu
-  const handleThemeMenuLeave = useCallback(() => {
-    // Add a small delay before closing to allow mouse to reach submenu
-    closeTimeoutRef.current = setTimeout(() => {
-      setShowThemeSubmenu(false);
-      setPreviewTheme(null);
-    }, 100); // 100ms delay is enough to cross the gap
-  }, [setPreviewTheme]);
-
-  /**
-   * Calculates theme submenu position to prevent viewport overflow.
-   *
-   * This memoized calculation determines the optimal vertical position and maximum
-   * height for the theme submenu based on the current viewport dimensions and
-   * the trigger button's position.
-   *
-   * @returns Object containing:
-   *   - top: Vertical offset from default position (negative values shift submenu up)
-   *   - maxHeight: Maximum height constraint to prevent overflow with scrolling
-   */
-  const submenuPosition = useMemo(() => {
-    const { ESTIMATED_SUBMENU_HEIGHT, COLLISION_PADDING, THEME_BUTTON_OFFSET } =
-      THEME_SUBMENU_CONSTANTS;
-
-    const viewportHeight = typeof window !== 'undefined' ? window.innerHeight : 800;
-
-    // Calculate where the submenu's bottom edge would be if positioned normally
-    const submenuBottomY = position.y + THEME_BUTTON_OFFSET + ESTIMATED_SUBMENU_HEIGHT;
-
-    // Check if submenu would overflow bottom of viewport
-    const wouldOverflowBottom = submenuBottomY > viewportHeight - COLLISION_PADDING;
-
-    // If it would overflow, calculate how much to shift it up
-    if (wouldOverflowBottom) {
-      // Calculate the offset needed to align submenu bottom with viewport bottom minus padding
-      const overflowAmount = submenuBottomY - (viewportHeight - COLLISION_PADDING);
-      return {
-        top: -overflowAmount,
-        maxHeight: Math.min(ESTIMATED_SUBMENU_HEIGHT, viewportHeight - COLLISION_PADDING * 2),
-      };
-    }
-
-    // Default: submenu opens at top of parent (aligned with the theme button)
-    return {
-      top: 0,
-      maxHeight: Math.min(
-        ESTIMATED_SUBMENU_HEIGHT,
-        viewportHeight - position.y - THEME_BUTTON_OFFSET - COLLISION_PADDING
-      ),
-    };
-  }, [position.y]);
-
-  // Cleanup timeout on unmount
-  useEffect(() => {
-    return () => {
-      if (closeTimeoutRef.current) {
-        clearTimeout(closeTimeoutRef.current);
-      }
-    };
-  }, []);
-
   useEffect(() => {
     const handleClickOutside = (event: globalThis.MouseEvent) => {
       // Don't close if a confirmation dialog is open (dialog is in a portal)
@@ -386,8 +242,11 @@ export function ProjectContextMenu({
             {/* Theme Submenu Trigger */}
             <div
               className="relative"
-              onMouseEnter={handleThemeMenuEnter}
-              onMouseLeave={handleThemeMenuLeave}
+              onMouseEnter={() => setShowThemeSubmenu(true)}
+              onMouseLeave={() => {
+                setShowThemeSubmenu(false);
+                setPreviewTheme(null);
+              }}
             >
               <button
                 onClick={() => setShowThemeSubmenu(!showThemeSubmenu)}
@@ -414,18 +273,13 @@ export function ProjectContextMenu({
                 <div
                   ref={themeSubmenuRef}
                   className={cn(
-                    'absolute left-full ml-1 min-w-[420px] rounded-lg',
+                    'absolute left-full top-0 ml-1 min-w-[420px] rounded-lg',
                     'bg-popover text-popover-foreground',
                     'border border-border shadow-lg',
                     'animate-in fade-in zoom-in-95 duration-100'
                   )}
-                  style={{
-                    zIndex: Z_INDEX.THEME_SUBMENU,
-                    top: `${submenuPosition.top}px`,
-                  }}
+                  style={{ zIndex: Z_INDEX.THEME_SUBMENU }}
                   data-testid="project-theme-submenu"
-                  onMouseEnter={handleThemeMenuEnter}
-                  onMouseLeave={handleThemeMenuLeave}
                 >
                   <div className="p-2">
                     {/* Use Global Option */}
@@ -452,13 +306,7 @@ export function ProjectContextMenu({
                     <div className="h-px bg-border my-2" />
 
                     {/* Two Column Layout - Using reusable ThemeColumn component */}
-                    {/* Dynamic max height with scroll for viewport overflow handling */}
-                    <div
-                      className="flex gap-2 overflow-y-auto scrollbar-styled"
-                      style={{
-                        maxHeight: `${Math.max(0, submenuPosition.maxHeight - THEME_SUBMENU_CONSTANTS.SUBMENU_HEADER_HEIGHT)}px`,
-                      }}
-                    >
+                    <div className="flex gap-2">
                       <ThemeColumn
                         title="Dark"
                         icon={Moon}

apps/ui/src/components/layout/project-switcher/components/project-switcher-item.tsx

@@ -1,6 +1,6 @@
 import { Folder, LucideIcon } from 'lucide-react';
 import * as LucideIcons from 'lucide-react';
-import { cn, sanitizeForTestId } from '@/lib/utils';
+import { cn } from '@/lib/utils';
 import { getAuthenticatedImageUrl } from '@/lib/api-fetch';
 import type { Project } from '@/lib/electron';
 
@@ -37,15 +37,10 @@ export function ProjectSwitcherItem({
   const IconComponent = getIconComponent();
   const hasCustomIcon = !!project.customIconPath;
 
-  // Combine project.id with sanitized name for uniqueness and readability
-  // Format: project-switcher-{id}-{sanitizedName}
-  const testId = `project-switcher-${project.id}-${sanitizeForTestId(project.name)}`;
-
   return (
     <button
       onClick={onClick}
       onContextMenu={onContextMenu}
-      data-testid={testId}
       className={cn(
         'group w-full aspect-square rounded-xl flex items-center justify-center relative overflow-hidden',
         'transition-all duration-200 ease-out',
@@ -65,6 +60,7 @@ export function ProjectSwitcherItem({
         'hover:scale-105 active:scale-95'
       )}
       title={project.name}
+      data-testid={`project-switcher-${project.id}`}
     >
       {hasCustomIcon ? (
         <img

apps/ui/src/components/layout/project-switcher/project-switcher.tsx

@@ -2,7 +2,7 @@ import { useState, useCallback, useEffect } from 'react';
 import { Plus, Bug, FolderOpen, BookOpen } from 'lucide-react';
 import { useNavigate, useLocation } from '@tanstack/react-router';
 import { cn } from '@/lib/utils';
-import { useAppStore } from '@/store/app-store';
+import { useAppStore, type ThemeMode } from '@/store/app-store';
 import { useOSDetection } from '@/hooks/use-os-detection';
 import { ProjectSwitcherItem } from './components/project-switcher-item';
 import { ProjectContextMenu } from './components/project-context-menu';
@@ -10,7 +10,7 @@ import { EditProjectDialog } from './components/edit-project-dialog';
 import { NotificationBell } from './components/notification-bell';
 import { NewProjectModal } from '@/components/dialogs/new-project-modal';
 import { OnboardingDialog } from '@/components/layout/sidebar/dialogs';
-import { useProjectCreation } from '@/components/layout/sidebar/hooks';
+import { useProjectCreation, useProjectTheme } from '@/components/layout/sidebar/hooks';
 import { SIDEBAR_FEATURE_FLAGS } from '@/components/layout/sidebar/constants';
 import type { Project } from '@/lib/electron';
 import { getElectronAPI } from '@/lib/electron';
@@ -41,6 +41,7 @@ export function ProjectSwitcher() {
     projects,
     currentProject,
     setCurrentProject,
+    trashedProjects,
     upsertAndSetCurrentProject,
     specCreatingForProject,
     setSpecCreatingForProject,
@@ -68,6 +69,9 @@ export function ProjectSwitcher() {
   const appMode = import.meta.env.VITE_APP_MODE || '?';
   const versionSuffix = `${getOSAbbreviation(os)}${appMode}`;
 
+  // Get global theme for project creation
+  const { globalTheme } = useProjectTheme();
+
   // Project creation state and handlers
   const {
     showNewProjectModal,
@@ -80,6 +84,9 @@ export function ProjectSwitcher() {
     handleCreateFromTemplate,
     handleCreateFromCustomUrl,
   } = useProjectCreation({
+    trashedProjects,
+    currentProject,
+    globalTheme,
     upsertAndSetCurrentProject,
   });
 
@@ -154,8 +161,13 @@ export function ProjectSwitcher() {
         }
 
         // Upsert project and set as current (handles both create and update cases)
-        // Theme handling (trashed project recovery or undefined for global) is done by the store
-        upsertAndSetCurrentProject(path, name);
+        // Theme preservation is handled by the store action
+        const trashedProject = trashedProjects.find((p) => p.path === path);
+        const effectiveTheme =
+          (trashedProject?.theme as ThemeMode | undefined) ||
+          (currentProject?.theme as ThemeMode | undefined) ||
+          globalTheme;
+        upsertAndSetCurrentProject(path, name, effectiveTheme);
 
         // Check if app_spec.txt exists
         const specExists = await hasAppSpec(path);
@@ -186,7 +198,7 @@ export function ProjectSwitcher() {
         });
       }
     }
-  }, [upsertAndSetCurrentProject, navigate]);
+  }, [trashedProjects, upsertAndSetCurrentProject, currentProject, globalTheme, navigate]);
 
   // Handler for creating initial spec from the setup dialog
   const handleCreateInitialSpec = useCallback(async () => {

apps/ui/src/components/layout/sidebar.tsx

@@ -4,7 +4,7 @@ import { useNavigate, useLocation } from '@tanstack/react-router';
 
 const logger = createLogger('Sidebar');
 import { cn } from '@/lib/utils';
-import { useAppStore } from '@/store/app-store';
+import { useAppStore, type ThemeMode } from '@/store/app-store';
 import { useNotificationsStore } from '@/store/notifications-store';
 import { useKeyboardShortcuts, useKeyboardShortcutsConfig } from '@/hooks/use-keyboard-shortcuts';
 import { getElectronAPI } from '@/lib/electron';
@@ -34,6 +34,7 @@ import {
   useProjectCreation,
   useSetupDialog,
   useTrashOperations,
+  useProjectTheme,
   useUnviewedValidations,
 } from './sidebar/hooks';
 
@@ -78,6 +79,9 @@ export function Sidebar() {
   // State for trash dialog
   const [showTrashDialog, setShowTrashDialog] = useState(false);
 
+  // Project theme management (must come before useProjectCreation which uses globalTheme)
+  const { globalTheme } = useProjectTheme();
+
   // Project creation state and handlers
   const {
     showNewProjectModal,
@@ -93,6 +97,9 @@ export function Sidebar() {
     handleCreateFromTemplate,
     handleCreateFromCustomUrl,
   } = useProjectCreation({
+    trashedProjects,
+    currentProject,
+    globalTheme,
     upsertAndSetCurrentProject,
   });
 
@@ -191,8 +198,13 @@ export function Sidebar() {
         }
 
         // Upsert project and set as current (handles both create and update cases)
-        // Theme handling (trashed project recovery or undefined for global) is done by the store
-        upsertAndSetCurrentProject(path, name);
+        // Theme preservation is handled by the store action
+        const trashedProject = trashedProjects.find((p) => p.path === path);
+        const effectiveTheme =
+          (trashedProject?.theme as ThemeMode | undefined) ||
+          (currentProject?.theme as ThemeMode | undefined) ||
+          globalTheme;
+        upsertAndSetCurrentProject(path, name, effectiveTheme);
 
         // Check if app_spec.txt exists
         const specExists = await hasAppSpec(path);
@@ -220,7 +232,7 @@ export function Sidebar() {
         });
       }
     }
-  }, [upsertAndSetCurrentProject]);
+  }, [trashedProjects, upsertAndSetCurrentProject, currentProject, globalTheme]);
 
   // Navigation sections and keyboard shortcuts (defined after handlers)
   const { navSections, navigationShortcuts } = useNavigation({

apps/ui/src/components/layout/sidebar/components/project-selector-with-options.tsx

@@ -30,41 +30,17 @@ import {
 import { DndContext, closestCenter } from '@dnd-kit/core';
 import { SortableContext, verticalListSortingStrategy } from '@dnd-kit/sortable';
 import { SortableProjectItem, ThemeMenuItem } from './';
-import { PROJECT_DARK_THEMES, PROJECT_LIGHT_THEMES, THEME_SUBMENU_CONSTANTS } from '../constants';
+import { PROJECT_DARK_THEMES, PROJECT_LIGHT_THEMES } from '../constants';
 import { useProjectPicker, useDragAndDrop, useProjectTheme } from '../hooks';
 import { useKeyboardShortcutsConfig } from '@/hooks/use-keyboard-shortcuts';
 
-/**
- * Props for the ProjectSelectorWithOptions component.
- * Defines the interface for the project selector dropdown with additional options menu.
- */
 interface ProjectSelectorWithOptionsProps {
-  /** Whether the sidebar is currently expanded */
   sidebarOpen: boolean;
-  /** Whether the project picker dropdown is currently open */
   isProjectPickerOpen: boolean;
-  /** Callback to control the project picker dropdown open state */
   setIsProjectPickerOpen: (value: boolean | ((prev: boolean) => boolean)) => void;
-  /** Callback to show the delete project confirmation dialog */
   setShowDeleteProjectDialog: (show: boolean) => void;
 }
 
-/**
- * A project selector component with search, drag-and-drop reordering, and options menu.
- *
- * Features:
- * - Searchable dropdown for quick project switching
- * - Drag-and-drop reordering of projects
- * - Project-specific theme selection with live preview
- * - Project history navigation (previous/next)
- * - Option to move project to trash
- *
- * The component uses viewport-aware positioning via THEME_SUBMENU_CONSTANTS
- * for consistent submenu behavior across the application.
- *
- * @param props - Component props
- * @returns The rendered project selector or null if sidebar is closed or no projects exist
- */
 export function ProjectSelectorWithOptions({
   sidebarOpen,
   isProjectPickerOpen,
@@ -270,7 +246,6 @@ export function ProjectSelectorWithOptions({
               <DropdownMenuSubContent
                 className="w-[420px] bg-popover/95 backdrop-blur-xl"
                 data-testid="project-theme-menu"
-                collisionPadding={THEME_SUBMENU_CONSTANTS.COLLISION_PADDING}
                 onPointerLeave={() => {
                   // Clear preview theme when leaving the dropdown
                   setPreviewTheme(null);
@@ -311,8 +286,7 @@ export function ProjectSelectorWithOptions({
                   </div>
                   <DropdownMenuSeparator />
                   {/* Two Column Layout */}
-                  {/* Max height with scroll to ensure all themes are visible when menu is near screen edge */}
-                  <div className="flex gap-2 p-2 max-h-[60vh] overflow-y-auto scrollbar-styled">
+                  <div className="flex gap-2 p-2">
                     {/* Dark Themes Column */}
                     <div className="flex-1">
                       <div className="flex items-center gap-1.5 px-2 py-1.5 text-xs font-medium text-muted-foreground">

apps/ui/src/components/layout/sidebar/components/sidebar-navigation.tsx

@@ -1,9 +1,9 @@
 import type { NavigateOptions } from '@tanstack/react-router';
+import { Loader2 } from 'lucide-react';
 import { cn } from '@/lib/utils';
 import { formatShortcut } from '@/store/app-store';
 import type { NavSection } from '../types';
 import type { Project } from '@/lib/electron';
-import { Spinner } from '@/components/ui/spinner';
 
 interface SidebarNavigationProps {
   currentProject: Project | null;
@@ -93,10 +93,9 @@ export function SidebarNavigation({
                   >
                     <div className="relative">
                       {item.isLoading ? (
-                        <Spinner
-                          size="md"
+                        <Loader2
                           className={cn(
-                            'shrink-0',
+                            'w-[18px] h-[18px] shrink-0 animate-spin',
                             isActive ? 'text-brand-500' : 'text-muted-foreground'
                           )}
                         />

apps/ui/src/components/layout/sidebar/constants.ts

@@ -1,36 +1,5 @@
 import { darkThemes, lightThemes } from '@/config/theme-options';
 
-/**
- * Shared constants for theme submenu positioning and layout.
- * Used across project-context-menu and project-selector-with-options components
- * to ensure consistent viewport-aware positioning and styling.
- */
-export const THEME_SUBMENU_CONSTANTS = {
-  /**
-   * Estimated total height of the theme submenu content in pixels.
-   * Includes all theme options, headers, padding, and "Use Global" button.
-   */
-  ESTIMATED_SUBMENU_HEIGHT: 620,
-
-  /**
-   * Padding from viewport edges to prevent submenu overflow.
-   * Applied to both top and bottom edges when calculating available space.
-   */
-  COLLISION_PADDING: 32,
-
-  /**
-   * Vertical offset from context menu top to the "Project Theme" button.
-   * Used for calculating submenu position relative to trigger button.
-   */
-  THEME_BUTTON_OFFSET: 50,
-
-  /**
-   * Height reserved for submenu header area (includes "Use Global" button and separator).
-   * Subtracted from maxHeight to get scrollable content area height.
-   */
-  SUBMENU_HEADER_HEIGHT: 80,
-} as const;
-
 export const PROJECT_DARK_THEMES = darkThemes.map((opt) => ({
   value: opt.value,
   label: opt.label,

apps/ui/src/components/layout/sidebar/hooks/use-project-creation.ts

@@ -6,13 +6,22 @@ const logger = createLogger('ProjectCreation');
 import { initializeProject } from '@/lib/project-init';
 import { toast } from 'sonner';
 import type { StarterTemplate } from '@/lib/templates';
-import type { Project } from '@/lib/electron';
+import type { ThemeMode } from '@/store/app-store';
+import type { TrashedProject, Project } from '@/lib/electron';
 
 interface UseProjectCreationProps {
-  upsertAndSetCurrentProject: (path: string, name: string) => Project;
+  trashedProjects: TrashedProject[];
+  currentProject: Project | null;
+  globalTheme: ThemeMode;
+  upsertAndSetCurrentProject: (path: string, name: string, theme: ThemeMode) => Project;
 }
 
-export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCreationProps) {
+export function useProjectCreation({
+  trashedProjects,
+  currentProject,
+  globalTheme,
+  upsertAndSetCurrentProject,
+}: UseProjectCreationProps) {
   // Modal state
   const [showNewProjectModal, setShowNewProjectModal] = useState(false);
   const [isCreatingProject, setIsCreatingProject] = useState(false);
@@ -58,8 +67,14 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
 </project_specification>`
         );
 
-        // Let the store handle theme (trashed project recovery or undefined for global)
-        upsertAndSetCurrentProject(projectPath, projectName);
+        // Determine theme: try trashed project theme, then current project theme, then global
+        const trashedProject = trashedProjects.find((p) => p.path === projectPath);
+        const effectiveTheme =
+          (trashedProject?.theme as ThemeMode | undefined) ||
+          (currentProject?.theme as ThemeMode | undefined) ||
+          globalTheme;
+
+        upsertAndSetCurrentProject(projectPath, projectName, effectiveTheme);
 
         setShowNewProjectModal(false);
 
@@ -77,7 +92,7 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
         throw error;
       }
     },
-    [upsertAndSetCurrentProject]
+    [trashedProjects, currentProject, globalTheme, upsertAndSetCurrentProject]
   );
 
   /**
@@ -154,8 +169,14 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
 </project_specification>`
         );
 
-        // Let the store handle theme (trashed project recovery or undefined for global)
-        upsertAndSetCurrentProject(projectPath, projectName);
+        // Determine theme
+        const trashedProject = trashedProjects.find((p) => p.path === projectPath);
+        const effectiveTheme =
+          (trashedProject?.theme as ThemeMode | undefined) ||
+          (currentProject?.theme as ThemeMode | undefined) ||
+          globalTheme;
+
+        upsertAndSetCurrentProject(projectPath, projectName, effectiveTheme);
         setShowNewProjectModal(false);
         setNewProjectName(projectName);
         setNewProjectPath(projectPath);
@@ -173,7 +194,7 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
         setIsCreatingProject(false);
       }
     },
-    [upsertAndSetCurrentProject]
+    [trashedProjects, currentProject, globalTheme, upsertAndSetCurrentProject]
   );
 
   /**
@@ -223,8 +244,14 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
 </project_specification>`
         );
 
-        // Let the store handle theme (trashed project recovery or undefined for global)
-        upsertAndSetCurrentProject(projectPath, projectName);
+        // Determine theme
+        const trashedProject = trashedProjects.find((p) => p.path === projectPath);
+        const effectiveTheme =
+          (trashedProject?.theme as ThemeMode | undefined) ||
+          (currentProject?.theme as ThemeMode | undefined) ||
+          globalTheme;
+
+        upsertAndSetCurrentProject(projectPath, projectName, effectiveTheme);
         setShowNewProjectModal(false);
         setNewProjectName(projectName);
         setNewProjectPath(projectPath);
@@ -242,7 +269,7 @@ export function useProjectCreation({ upsertAndSetCurrentProject }: UseProjectCre
         setIsCreatingProject(false);
       }
     },
-    [upsertAndSetCurrentProject]
+    [trashedProjects, currentProject, globalTheme, upsertAndSetCurrentProject]
   );
 
   return {

apps/ui/src/components/session-manager.tsx

@@ -1,6 +1,5 @@
-import { useState, useEffect, useCallback, useRef } from 'react';
+import { useState, useEffect } from 'react';
 import { createLogger } from '@automaker/utils/logger';
-import { useQueryClient } from '@tanstack/react-query';
 import { Card, CardContent, CardHeader, CardTitle } from '@/components/ui/card';
 
 const logger = createLogger('SessionManager');
@@ -17,14 +16,12 @@ import {
   Check,
   X,
   ArchiveRestore,
+  Loader2,
 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
 import type { SessionListItem } from '@/types/electron';
 import { useKeyboardShortcutsConfig } from '@/hooks/use-keyboard-shortcuts';
 import { getElectronAPI } from '@/lib/electron';
-import { useSessions } from '@/hooks/queries';
-import { queryKeys } from '@/lib/query-keys';
 import { DeleteSessionDialog } from '@/components/dialogs/delete-session-dialog';
 import { DeleteAllArchivedSessionsDialog } from '@/components/dialogs/delete-all-archived-sessions-dialog';
 
@@ -105,7 +102,7 @@ export function SessionManager({
   onQuickCreateRef,
 }: SessionManagerProps) {
   const shortcuts = useKeyboardShortcutsConfig();
-  const queryClient = useQueryClient();
+  const [sessions, setSessions] = useState<SessionListItem[]>([]);
   const [activeTab, setActiveTab] = useState<'active' | 'archived'>('active');
   const [editingSessionId, setEditingSessionId] = useState<string | null>(null);
   const [editingName, setEditingName] = useState('');
@@ -116,14 +113,8 @@ export function SessionManager({
   const [sessionToDelete, setSessionToDelete] = useState<SessionListItem | null>(null);
   const [isDeleteAllArchivedDialogOpen, setIsDeleteAllArchivedDialogOpen] = useState(false);
 
-  // Use React Query for sessions list - always include archived, filter client-side
-  const { data: sessions = [], refetch: refetchSessions } = useSessions(true);
-
-  // Ref to track if we've done the initial running sessions check
-  const hasCheckedInitialRef = useRef(false);
-
   // Check running state for all sessions
-  const checkRunningSessions = useCallback(async (sessionList: SessionListItem[]) => {
+  const checkRunningSessions = async (sessionList: SessionListItem[]) => {
     const api = getElectronAPI();
     if (!api?.agent) return;
 
@@ -143,25 +134,25 @@ export function SessionManager({
     }
 
     setRunningSessions(runningIds);
-  }, []);
+  };
 
-  // Helper to invalidate sessions cache and refetch
-  const invalidateSessions = useCallback(async () => {
-    await queryClient.invalidateQueries({ queryKey: queryKeys.sessions.all(true) });
-    // Also check running state after invalidation
-    const result = await refetchSessions();
-    if (result.data) {
-      await checkRunningSessions(result.data);
+  // Load sessions
+  const loadSessions = async () => {
+    const api = getElectronAPI();
+    if (!api?.sessions) return;
+
+    // Always load all sessions and filter client-side
+    const result = await api.sessions.list(true);
+    if (result.success && result.sessions) {
+      setSessions(result.sessions);
+      // Check running state for all sessions
+      await checkRunningSessions(result.sessions);
     }
-  }, [queryClient, refetchSessions, checkRunningSessions]);
+  };
 
-  // Check running state on initial load (runs only once when sessions first load)
   useEffect(() => {
-    if (sessions.length > 0 && !hasCheckedInitialRef.current) {
-      hasCheckedInitialRef.current = true;
-      checkRunningSessions(sessions);
-    }
-  }, [sessions, checkRunningSessions]);
+    loadSessions();
+  }, []);
 
   // Periodically check running state for sessions (useful for detecting when agents finish)
   useEffect(() => {
@@ -175,7 +166,7 @@ export function SessionManager({
     }, 3000); // Check every 3 seconds
 
     return () => clearInterval(interval);
-  }, [sessions, runningSessions.size, isCurrentSessionThinking, checkRunningSessions]);
+  }, [sessions, runningSessions.size, isCurrentSessionThinking]);
 
   // Create new session with random name
   const handleCreateSession = async () => {
@@ -189,7 +180,7 @@ export function SessionManager({
     if (result.success && result.session?.id) {
       setNewSessionName('');
       setIsCreating(false);
-      await invalidateSessions();
+      await loadSessions();
       onSelectSession(result.session.id);
     }
   };
@@ -204,7 +195,7 @@ export function SessionManager({
     const result = await api.sessions.create(sessionName, projectPath, projectPath);
 
     if (result.success && result.session?.id) {
-      await invalidateSessions();
+      await loadSessions();
       onSelectSession(result.session.id);
     }
   };
@@ -231,7 +222,7 @@ export function SessionManager({
     if (result.success) {
       setEditingSessionId(null);
       setEditingName('');
-      await invalidateSessions();
+      await loadSessions();
     }
   };
 
@@ -250,7 +241,7 @@ export function SessionManager({
         if (currentSessionId === sessionId) {
           onSelectSession(null);
         }
-        await invalidateSessions();
+        await loadSessions();
       } else {
         logger.error('[SessionManager] Archive failed:', result.error);
       }
@@ -270,7 +261,7 @@ export function SessionManager({
     try {
       const result = await api.sessions.unarchive(sessionId);
       if (result.success) {
-        await invalidateSessions();
+        await loadSessions();
       } else {
         logger.error('[SessionManager] Unarchive failed:', result.error);
       }
@@ -292,7 +283,7 @@ export function SessionManager({
 
     const result = await api.sessions.delete(sessionId);
     if (result.success) {
-      await invalidateSessions();
+      await loadSessions();
       if (currentSessionId === sessionId) {
         // Switch to another session or create a new one
         const activeSessionsList = sessions.filter((s) => !s.isArchived);
@@ -314,7 +305,7 @@ export function SessionManager({
       await api.sessions.delete(session.id);
     }
 
-    await invalidateSessions();
+    await loadSessions();
     setIsDeleteAllArchivedDialogOpen(false);
   };
 
@@ -475,7 +466,7 @@ export function SessionManager({
                       {/* Show loading indicator if this session is running (either current session thinking or any session in runningSessions) */}
                       {(currentSessionId === session.id && isCurrentSessionThinking) ||
                       runningSessions.has(session.id) ? (
-                        <Spinner size="sm" className="shrink-0" />
+                        <Loader2 className="w-4 h-4 text-primary animate-spin shrink-0" />
                       ) : (
                         <MessageSquare className="w-4 h-4 text-muted-foreground shrink-0" />
                       )}

apps/ui/src/components/ui/button.tsx

@@ -1,9 +1,9 @@
 import * as React from 'react';
 import { Slot } from '@radix-ui/react-slot';
 import { cva, type VariantProps } from 'class-variance-authority';
+import { Loader2 } from 'lucide-react';
 
 import { cn } from '@/lib/utils';
-import { Spinner } from '@/components/ui/spinner';
 
 const buttonVariants = cva(
   "inline-flex items-center justify-center gap-2 whitespace-nowrap rounded-md text-sm font-medium transition-all duration-200 cursor-pointer disabled:pointer-events-none disabled:opacity-50 disabled:cursor-not-allowed [&_svg]:pointer-events-none [&_svg:not([class*='size-'])]:size-4 shrink-0 [&_svg]:shrink-0 outline-none focus-visible:border-ring focus-visible:ring-ring/50 focus-visible:ring-[3px] aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 aria-invalid:border-destructive active:scale-[0.98]",
@@ -39,7 +39,7 @@ const buttonVariants = cva(
 
 // Loading spinner component
 function ButtonSpinner({ className }: { className?: string }) {
-  return <Spinner size="sm" className={className} />;
+  return <Loader2 className={cn('size-4 animate-spin', className)} aria-hidden="true" />;
 }
 
 function Button({

apps/ui/src/components/ui/description-image-dropzone.tsx

@@ -3,8 +3,7 @@ import { createLogger } from '@automaker/utils/logger';
 import { cn } from '@/lib/utils';
 
 const logger = createLogger('DescriptionImageDropZone');
-import { ImageIcon, X, FileText } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { ImageIcon, X, Loader2, FileText } from 'lucide-react';
 import { Textarea } from '@/components/ui/textarea';
 import { getElectronAPI } from '@/lib/electron';
 import { getAuthenticatedImageUrl } from '@/lib/api-fetch';
@@ -432,7 +431,7 @@ export function DescriptionImageDropZone({
       {/* Processing indicator */}
       {isProcessing && (
         <div className="flex items-center gap-2 mt-2 text-sm text-muted-foreground">
-          <Spinner size="sm" />
+          <Loader2 className="w-4 h-4 animate-spin" />
           <span>Processing files...</span>
         </div>
       )}

apps/ui/src/components/ui/feature-image-upload.tsx

@@ -3,8 +3,7 @@ import { createLogger } from '@automaker/utils/logger';
 import { cn } from '@/lib/utils';
 
 const logger = createLogger('FeatureImageUpload');
-import { ImageIcon, X } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { ImageIcon, X, Upload } from 'lucide-react';
 import {
   fileToBase64,
   generateImageId,
@@ -197,7 +196,7 @@ export function FeatureImageUpload({
             )}
           >
             {isProcessing ? (
-              <Spinner size="md" />
+              <Upload className="h-5 w-5 animate-spin text-muted-foreground" />
             ) : (
               <ImageIcon className="h-5 w-5 text-muted-foreground" />
             )}

apps/ui/src/components/ui/git-diff-panel.tsx

@@ -1,4 +1,5 @@
-import { useState, useMemo } from 'react';
+import { useState, useEffect, useMemo, useCallback } from 'react';
+import { getElectronAPI } from '@/lib/electron';
 import { cn } from '@/lib/utils';
 import {
   File,
@@ -8,13 +9,12 @@ import {
   FilePen,
   ChevronDown,
   ChevronRight,
+  Loader2,
   RefreshCw,
   GitBranch,
   AlertCircle,
 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { Button } from './button';
-import { useWorktreeDiffs, useGitDiffs } from '@/hooks/queries';
 import type { FileStatus } from '@/types/electron';
 
 interface GitDiffPanelProps {
@@ -350,44 +350,56 @@ export function GitDiffPanel({
   useWorktrees = false,
 }: GitDiffPanelProps) {
   const [isExpanded, setIsExpanded] = useState(!compact);
+  const [isLoading, setIsLoading] = useState(false);
+  const [error, setError] = useState<string | null>(null);
+  const [files, setFiles] = useState<FileStatus[]>([]);
+  const [diffContent, setDiffContent] = useState<string>('');
   const [expandedFiles, setExpandedFiles] = useState<Set<string>>(new Set());
 
-  // Use worktree diffs hook when worktrees are enabled and panel is expanded
-  // Pass undefined for featureId when not using worktrees to disable the query
-  const {
-    data: worktreeDiffsData,
-    isLoading: isLoadingWorktree,
-    error: worktreeError,
-    refetch: refetchWorktree,
-  } = useWorktreeDiffs(
-    useWorktrees && isExpanded ? projectPath : undefined,
-    useWorktrees && isExpanded ? featureId : undefined
-  );
+  const loadDiffs = useCallback(async () => {
+    setIsLoading(true);
+    setError(null);
+    try {
+      const api = getElectronAPI();
 
-  // Use git diffs hook when worktrees are disabled and panel is expanded
-  const {
-    data: gitDiffsData,
-    isLoading: isLoadingGit,
-    error: gitError,
-    refetch: refetchGit,
-  } = useGitDiffs(projectPath, !useWorktrees && isExpanded);
+      // Use worktree API if worktrees are enabled, otherwise use git API for main project
+      if (useWorktrees) {
+        if (!api?.worktree?.getDiffs) {
+          throw new Error('Worktree API not available');
+        }
+        const result = await api.worktree.getDiffs(projectPath, featureId);
+        if (result.success) {
+          setFiles(result.files || []);
+          setDiffContent(result.diff || '');
+        } else {
+          setError(result.error || 'Failed to load diffs');
+        }
+      } else {
+        // Use git API for main project diffs
+        if (!api?.git?.getDiffs) {
+          throw new Error('Git API not available');
+        }
+        const result = await api.git.getDiffs(projectPath);
+        if (result.success) {
+          setFiles(result.files || []);
+          setDiffContent(result.diff || '');
+        } else {
+          setError(result.error || 'Failed to load diffs');
+        }
+      }
+    } catch (err) {
+      setError(err instanceof Error ? err.message : 'Failed to load diffs');
+    } finally {
+      setIsLoading(false);
+    }
+  }, [projectPath, featureId, useWorktrees]);
 
-  // Select the appropriate data based on useWorktrees prop
-  const diffsData = useWorktrees ? worktreeDiffsData : gitDiffsData;
-  const isLoading = useWorktrees ? isLoadingWorktree : isLoadingGit;
-  const queryError = useWorktrees ? worktreeError : gitError;
-
-  // Extract files and diff content from the data
-  const files: FileStatus[] = diffsData?.files ?? [];
-  const diffContent = diffsData?.diff ?? '';
-  const error = queryError
-    ? queryError instanceof Error
-      ? queryError.message
-      : 'Failed to load diffs'
-    : null;
-
-  // Refetch function
-  const loadDiffs = useWorktrees ? refetchWorktree : refetchGit;
+  // Load diffs when expanded
+  useEffect(() => {
+    if (isExpanded) {
+      loadDiffs();
+    }
+  }, [isExpanded, loadDiffs]);
 
   const parsedDiffs = useMemo(() => parseDiff(diffContent), [diffContent]);
 
@@ -472,7 +484,7 @@ export function GitDiffPanel({
         <div className="border-t border-border">
           {isLoading ? (
             <div className="flex items-center justify-center gap-2 py-8 text-muted-foreground">
-              <Spinner size="md" />
+              <Loader2 className="w-5 h-5 animate-spin" />
               <span className="text-sm">Loading changes...</span>
             </div>
           ) : error ? (

apps/ui/src/components/ui/image-drop-zone.tsx

@@ -3,8 +3,7 @@ import { createLogger } from '@automaker/utils/logger';
 import { cn } from '@/lib/utils';
 
 const logger = createLogger('ImageDropZone');
-import { ImageIcon, X } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { ImageIcon, X, Upload } from 'lucide-react';
 import type { ImageAttachment } from '@/store/app-store';
 import {
   fileToBase64,
@@ -205,7 +204,7 @@ export function ImageDropZone({
               )}
             >
               {isProcessing ? (
-                <Spinner size="lg" />
+                <Upload className="h-6 w-6 animate-spin text-muted-foreground" />
               ) : (
                 <ImageIcon className="h-6 w-6 text-muted-foreground" />
               )}

apps/ui/src/components/ui/loading-state.tsx

@@ -1,15 +1,17 @@
-import { Spinner } from '@/components/ui/spinner';
+import { Loader2 } from 'lucide-react';
 
 interface LoadingStateProps {
   /** Optional custom message to display below the spinner */
   message?: string;
+  /** Optional custom size class for the spinner (default: h-8 w-8) */
+  size?: string;
 }
 
-export function LoadingState({ message }: LoadingStateProps) {
+export function LoadingState({ message, size = 'h-8 w-8' }: LoadingStateProps) {
   return (
     <div className="flex-1 flex flex-col items-center justify-center">
-      <Spinner size="xl" />
-      {message && <p className="mt-4 text-sm font-medium text-primary">{message}</p>}
+      <Loader2 className={`${size} animate-spin text-muted-foreground`} />
+      {message && <p className="mt-4 text-sm text-muted-foreground">{message}</p>}
     </div>
   );
 }

apps/ui/src/components/ui/log-viewer.tsx

@@ -22,8 +22,8 @@ import {
   Filter,
   Circle,
   Play,
+  Loader2,
 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
 import {
   parseLogOutput,
@@ -148,7 +148,7 @@ function TodoListRenderer({ todos }: { todos: TodoItem[] }) {
       case 'completed':
         return <CheckCircle2 className="w-4 h-4 text-emerald-400" />;
       case 'in_progress':
-        return <Spinner size="sm" />;
+        return <Loader2 className="w-4 h-4 text-amber-400 animate-spin" />;
       case 'pending':
         return <Circle className="w-4 h-4 text-muted-foreground/70" />;
       default:

apps/ui/src/components/ui/provider-icon.tsx

@@ -523,15 +523,6 @@ function getUnderlyingModelIcon(model?: AgentModel | string): ProviderIconKey {
     }
   }
 
-  // Check for ClaudeCompatibleProvider model patterns (GLM, MiniMax, etc.)
-  // These are model IDs like "GLM-4.5-Air", "GLM-4.7", "MiniMax-M2.1"
-  if (modelStr.includes('glm')) {
-    return 'glm';
-  }
-  if (modelStr.includes('minimax')) {
-    return 'minimax';
-  }
-
   // Check for Cursor-specific models with underlying providers
   if (modelStr.includes('sonnet') || modelStr.includes('opus') || modelStr.includes('claude')) {
     return 'anthropic';
@@ -545,15 +536,7 @@ function getUnderlyingModelIcon(model?: AgentModel | string): ProviderIconKey {
   if (modelStr.includes('grok')) {
     return 'grok';
   }
-  // Cursor models - canonical format includes 'cursor-' prefix
-  // Also support legacy IDs for backward compatibility
-  if (
-    modelStr.includes('cursor') ||
-    modelStr === 'auto' ||
-    modelStr === 'composer-1' ||
-    modelStr === 'cursor-auto' ||
-    modelStr === 'cursor-composer-1'
-  ) {
+  if (modelStr.includes('cursor') || modelStr === 'auto' || modelStr === 'composer-1') {
     return 'cursor';
   }
 

apps/ui/src/components/ui/skeleton.tsx

@@ -1,18 +0,0 @@
-/**
- * Skeleton Components
- *
- * Loading placeholder components for content that's being fetched.
- */
-
-import { cn } from '@/lib/utils';
-
-interface SkeletonPulseProps {
-  className?: string;
-}
-
-/**
- * Pulsing skeleton placeholder for loading states
- */
-export function SkeletonPulse({ className }: SkeletonPulseProps) {
-  return <div className={cn('animate-pulse bg-muted/50 rounded', className)} />;
-}

apps/ui/src/components/ui/spinner.tsx

@@ -1,32 +0,0 @@
-import { Loader2 } from 'lucide-react';
-import { cn } from '@/lib/utils';
-
-type SpinnerSize = 'xs' | 'sm' | 'md' | 'lg' | 'xl';
-
-const sizeClasses: Record<SpinnerSize, string> = {
-  xs: 'h-3 w-3',
-  sm: 'h-4 w-4',
-  md: 'h-5 w-5',
-  lg: 'h-6 w-6',
-  xl: 'h-8 w-8',
-};
-
-interface SpinnerProps {
-  /** Size of the spinner */
-  size?: SpinnerSize;
-  /** Additional class names */
-  className?: string;
-}
-
-/**
- * Themed spinner component using the primary brand color.
- * Use this for all loading indicators throughout the app for consistency.
- */
-export function Spinner({ size = 'md', className }: SpinnerProps) {
-  return (
-    <Loader2
-      className={cn(sizeClasses[size], 'animate-spin text-primary', className)}
-      aria-hidden="true"
-    />
-  );
-}

apps/ui/src/components/ui/task-progress-panel.tsx

@@ -5,8 +5,7 @@ import { createLogger } from '@automaker/utils/logger';
 import { cn } from '@/lib/utils';
 
 const logger = createLogger('TaskProgressPanel');
-import { Check, Circle, ChevronDown, ChevronRight, FileCode } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { Check, Loader2, Circle, ChevronDown, ChevronRight, FileCode } from 'lucide-react';
 import { getElectronAPI } from '@/lib/electron';
 import type { AutoModeEvent } from '@/types/electron';
 import { Badge } from '@/components/ui/badge';
@@ -261,7 +260,7 @@ export function TaskProgressPanel({
                       )}
                     >
                       {isCompleted && <Check className="h-3.5 w-3.5" />}
-                      {isActive && <Spinner size="xs" />}
+                      {isActive && <Loader2 className="h-3.5 w-3.5 animate-spin" />}
                       {isPending && <Circle className="h-2 w-2 fill-current opacity-50" />}
                     </div>
 

apps/ui/src/components/ui/xml-syntax-editor.tsx

@@ -1,6 +1,9 @@
 import CodeMirror from '@uiw/react-codemirror';
 import { xml } from '@codemirror/lang-xml';
 import { EditorView } from '@codemirror/view';
+import { Extension } from '@codemirror/state';
+import { HighlightStyle, syntaxHighlighting } from '@codemirror/language';
+import { tags as t } from '@lezer/highlight';
 import { cn } from '@/lib/utils';
 
 interface XmlSyntaxEditorProps {
@@ -11,19 +14,52 @@ interface XmlSyntaxEditorProps {
   'data-testid'?: string;
 }
 
-// Simple editor theme - inherits text color from parent
+// Syntax highlighting that uses CSS variables from the app's theme system
+// This automatically adapts to any theme (dark, light, dracula, nord, etc.)
+const syntaxColors = HighlightStyle.define([
+  // XML tags - use primary color
+  { tag: t.tagName, color: 'var(--primary)' },
+  { tag: t.angleBracket, color: 'var(--muted-foreground)' },
+
+  // Attributes
+  { tag: t.attributeName, color: 'var(--chart-2, oklch(0.6 0.118 184.704))' },
+  { tag: t.attributeValue, color: 'var(--chart-1, oklch(0.646 0.222 41.116))' },
+
+  // Strings and content
+  { tag: t.string, color: 'var(--chart-1, oklch(0.646 0.222 41.116))' },
+  { tag: t.content, color: 'var(--foreground)' },
+
+  // Comments
+  { tag: t.comment, color: 'var(--muted-foreground)', fontStyle: 'italic' },
+
+  // Special
+  { tag: t.processingInstruction, color: 'var(--muted-foreground)' },
+  { tag: t.documentMeta, color: 'var(--muted-foreground)' },
+]);
+
+// Editor theme using CSS variables
 const editorTheme = EditorView.theme({
   '&': {
     height: '100%',
     fontSize: '0.875rem',
+    fontFamily: 'ui-monospace, monospace',
     backgroundColor: 'transparent',
+    color: 'var(--foreground)',
   },
   '.cm-scroller': {
     overflow: 'auto',
+    fontFamily: 'ui-monospace, monospace',
   },
   '.cm-content': {
     padding: '1rem',
     minHeight: '100%',
+    caretColor: 'var(--primary)',
+  },
+  '.cm-cursor, .cm-dropCursor': {
+    borderLeftColor: 'var(--primary)',
+  },
+  '&.cm-focused .cm-selectionBackground, .cm-selectionBackground, .cm-content ::selection': {
+    backgroundColor: 'oklch(0.55 0.25 265 / 0.3)',
   },
   '.cm-activeLine': {
     backgroundColor: 'transparent',
@@ -37,8 +73,15 @@ const editorTheme = EditorView.theme({
   '.cm-gutters': {
     display: 'none',
   },
+  '.cm-placeholder': {
+    color: 'var(--muted-foreground)',
+    fontStyle: 'italic',
+  },
 });
 
+// Combine all extensions
+const extensions: Extension[] = [xml(), syntaxHighlighting(syntaxColors), editorTheme];
+
 export function XmlSyntaxEditor({
   value,
   onChange,
@@ -51,16 +94,16 @@ export function XmlSyntaxEditor({
       <CodeMirror
         value={value}
         onChange={onChange}
-        extensions={[xml(), editorTheme]}
+        extensions={extensions}
         theme="none"
         placeholder={placeholder}
-        className="h-full [&_.cm-editor]:h-full [&_.cm-content]:text-foreground"
+        className="h-full [&_.cm-editor]:h-full"
         basicSetup={{
           lineNumbers: false,
           foldGutter: false,
           highlightActiveLine: false,
-          highlightSelectionMatches: false,
-          autocompletion: false,
+          highlightSelectionMatches: true,
+          autocompletion: true,
           bracketMatching: true,
           indentOnInput: true,
         }}

apps/ui/src/components/ui/xterm-log-viewer.tsx

@@ -1,6 +1,6 @@
 import { useEffect, useRef, useCallback, useState, forwardRef, useImperativeHandle } from 'react';
 import { useAppStore } from '@/store/app-store';
-import { getTerminalTheme, getTerminalFontFamily } from '@/config/terminal-themes';
+import { getTerminalTheme, DEFAULT_TERMINAL_FONT } from '@/config/terminal-themes';
 
 // Types for dynamically imported xterm modules
 type XTerminal = InstanceType<typeof import('@xterm/xterm').Terminal>;
@@ -20,7 +20,7 @@ export interface XtermLogViewerRef {
 export interface XtermLogViewerProps {
   /** Initial content to display */
   initialContent?: string;
-  /** Font size in pixels (uses terminal settings if not provided) */
+  /** Font size in pixels (default: 13) */
   fontSize?: number;
   /** Whether to auto-scroll to bottom when new content is added (default: true) */
   autoScroll?: boolean;
@@ -42,7 +42,7 @@ export const XtermLogViewer = forwardRef<XtermLogViewerRef, XtermLogViewerProps>
   (
     {
       initialContent,
-      fontSize,
+      fontSize = 13,
       autoScroll = true,
       className,
       minHeight = 300,
@@ -58,14 +58,9 @@ export const XtermLogViewer = forwardRef<XtermLogViewerRef, XtermLogViewerProps>
     const autoScrollRef = useRef(autoScroll);
     const pendingContentRef = useRef<string[]>([]);
 
-    // Get theme and font settings from store
+    // Get theme from store
     const getEffectiveTheme = useAppStore((state) => state.getEffectiveTheme);
     const effectiveTheme = getEffectiveTheme();
-    const terminalFontFamily = useAppStore((state) => state.terminalState.fontFamily);
-    const terminalFontSize = useAppStore((state) => state.terminalState.defaultFontSize);
-
-    // Use prop if provided, otherwise use store value, fallback to 13
-    const effectiveFontSize = fontSize ?? terminalFontSize ?? 13;
 
     // Track system dark mode for "system" theme
     const [systemIsDark, setSystemIsDark] = useState(() => {
@@ -107,17 +102,12 @@ export const XtermLogViewer = forwardRef<XtermLogViewerRef, XtermLogViewerProps>
 
         const terminalTheme = getTerminalTheme(resolvedTheme);
 
-        // Get font settings from store at initialization time
-        const terminalState = useAppStore.getState().terminalState;
-        const fontFamily = getTerminalFontFamily(terminalState.fontFamily);
-        const initFontSize = fontSize ?? terminalState.defaultFontSize ?? 13;
-
         const terminal = new Terminal({
           cursorBlink: false,
           cursorStyle: 'underline',
           cursorInactiveStyle: 'none',
-          fontSize: initFontSize,
-          fontFamily,
+          fontSize,
+          fontFamily: DEFAULT_TERMINAL_FONT,
           lineHeight: 1.2,
           theme: terminalTheme,
           disableStdin: true, // Read-only mode
@@ -191,18 +181,10 @@ export const XtermLogViewer = forwardRef<XtermLogViewerRef, XtermLogViewerProps>
     // Update font size when it changes
     useEffect(() => {
       if (xtermRef.current && isReady) {
-        xtermRef.current.options.fontSize = effectiveFontSize;
+        xtermRef.current.options.fontSize = fontSize;
         fitAddonRef.current?.fit();
       }
-    }, [effectiveFontSize, isReady]);
-
-    // Update font family when it changes
-    useEffect(() => {
-      if (xtermRef.current && isReady) {
-        xtermRef.current.options.fontFamily = getTerminalFontFamily(terminalFontFamily);
-        fitAddonRef.current?.fit();
-      }
-    }, [terminalFontFamily, isReady]);
+    }, [fontSize, isReady]);
 
     // Handle resize
     useEffect(() => {

apps/ui/src/components/usage-popover.tsx

@@ -1,13 +1,13 @@
-import { useState, useEffect, useMemo } from 'react';
+import { useState, useEffect, useMemo, useCallback } from 'react';
 import { Popover, PopoverContent, PopoverTrigger } from '@/components/ui/popover';
 import { Button } from '@/components/ui/button';
 import { Tabs, TabsContent, TabsList, TabsTrigger } from '@/components/ui/tabs';
 import { RefreshCw, AlertTriangle, CheckCircle, XCircle, Clock, ExternalLink } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
+import { getElectronAPI } from '@/lib/electron';
+import { useAppStore } from '@/store/app-store';
 import { useSetupStore } from '@/store/setup-store';
 import { AnthropicIcon, OpenAIIcon } from '@/components/ui/provider-icon';
-import { useClaudeUsage, useCodexUsage } from '@/hooks/queries';
 
 // Error codes for distinguishing failure modes
 const ERROR_CODES = {
@@ -60,63 +60,22 @@ function getCodexWindowLabel(durationMins: number): { title: string; subtitle: s
 }
 
 export function UsagePopover() {
+  const { claudeUsage, claudeUsageLastUpdated, setClaudeUsage } = useAppStore();
+  const { codexUsage, codexUsageLastUpdated, setCodexUsage } = useAppStore();
   const claudeAuthStatus = useSetupStore((state) => state.claudeAuthStatus);
   const codexAuthStatus = useSetupStore((state) => state.codexAuthStatus);
 
   const [open, setOpen] = useState(false);
   const [activeTab, setActiveTab] = useState<'claude' | 'codex'>('claude');
+  const [claudeLoading, setClaudeLoading] = useState(false);
+  const [codexLoading, setCodexLoading] = useState(false);
+  const [claudeError, setClaudeError] = useState<UsageError | null>(null);
+  const [codexError, setCodexError] = useState<UsageError | null>(null);
 
   // Check authentication status
   const isClaudeAuthenticated = !!claudeAuthStatus?.authenticated;
   const isCodexAuthenticated = codexAuthStatus?.authenticated;
 
-  // Use React Query hooks for usage data
-  // Only enable polling when popover is open AND the tab is active
-  const {
-    data: claudeUsage,
-    isLoading: claudeLoading,
-    error: claudeQueryError,
-    dataUpdatedAt: claudeUsageLastUpdated,
-    refetch: refetchClaude,
-  } = useClaudeUsage(open && activeTab === 'claude' && isClaudeAuthenticated);
-
-  const {
-    data: codexUsage,
-    isLoading: codexLoading,
-    error: codexQueryError,
-    dataUpdatedAt: codexUsageLastUpdated,
-    refetch: refetchCodex,
-  } = useCodexUsage(open && activeTab === 'codex' && isCodexAuthenticated);
-
-  // Parse errors into structured format
-  const claudeError = useMemo((): UsageError | null => {
-    if (!claudeQueryError) return null;
-    const message =
-      claudeQueryError instanceof Error ? claudeQueryError.message : String(claudeQueryError);
-    // Detect trust prompt error
-    const isTrustPrompt = message.includes('Trust prompt') || message.includes('folder permission');
-    if (isTrustPrompt) {
-      return { code: ERROR_CODES.TRUST_PROMPT, message };
-    }
-    if (message.includes('API bridge')) {
-      return { code: ERROR_CODES.API_BRIDGE_UNAVAILABLE, message };
-    }
-    return { code: ERROR_CODES.AUTH_ERROR, message };
-  }, [claudeQueryError]);
-
-  const codexError = useMemo((): UsageError | null => {
-    if (!codexQueryError) return null;
-    const message =
-      codexQueryError instanceof Error ? codexQueryError.message : String(codexQueryError);
-    if (message.includes('not available') || message.includes('does not provide')) {
-      return { code: ERROR_CODES.NOT_AVAILABLE, message };
-    }
-    if (message.includes('API bridge')) {
-      return { code: ERROR_CODES.API_BRIDGE_UNAVAILABLE, message };
-    }
-    return { code: ERROR_CODES.AUTH_ERROR, message };
-  }, [codexQueryError]);
-
   // Determine which tab to show by default
   useEffect(() => {
     if (isClaudeAuthenticated) {
@@ -135,9 +94,137 @@ export function UsagePopover() {
     return !codexUsageLastUpdated || Date.now() - codexUsageLastUpdated > 2 * 60 * 1000;
   }, [codexUsageLastUpdated]);
 
-  // Refetch functions for manual refresh
-  const fetchClaudeUsage = () => refetchClaude();
-  const fetchCodexUsage = () => refetchCodex();
+  const fetchClaudeUsage = useCallback(
+    async (isAutoRefresh = false) => {
+      if (!isAutoRefresh) setClaudeLoading(true);
+      setClaudeError(null);
+      try {
+        const api = getElectronAPI();
+        if (!api.claude) {
+          setClaudeError({
+            code: ERROR_CODES.API_BRIDGE_UNAVAILABLE,
+            message: 'Claude API bridge not available',
+          });
+          return;
+        }
+        const data = await api.claude.getUsage();
+        if ('error' in data) {
+          // Detect trust prompt error
+          const isTrustPrompt =
+            data.error === 'Trust prompt pending' ||
+            (data.message && data.message.includes('folder permission'));
+          setClaudeError({
+            code: isTrustPrompt ? ERROR_CODES.TRUST_PROMPT : ERROR_CODES.AUTH_ERROR,
+            message: data.message || data.error,
+          });
+          return;
+        }
+        setClaudeUsage(data);
+      } catch (err) {
+        setClaudeError({
+          code: ERROR_CODES.UNKNOWN,
+          message: err instanceof Error ? err.message : 'Failed to fetch usage',
+        });
+      } finally {
+        if (!isAutoRefresh) setClaudeLoading(false);
+      }
+    },
+    [setClaudeUsage]
+  );
+
+  const fetchCodexUsage = useCallback(
+    async (isAutoRefresh = false) => {
+      if (!isAutoRefresh) setCodexLoading(true);
+      setCodexError(null);
+      try {
+        const api = getElectronAPI();
+        if (!api.codex) {
+          setCodexError({
+            code: ERROR_CODES.API_BRIDGE_UNAVAILABLE,
+            message: 'Codex API bridge not available',
+          });
+          return;
+        }
+        const data = await api.codex.getUsage();
+        if ('error' in data) {
+          if (
+            data.message?.includes('not available') ||
+            data.message?.includes('does not provide')
+          ) {
+            setCodexError({
+              code: ERROR_CODES.NOT_AVAILABLE,
+              message: data.message || data.error,
+            });
+          } else {
+            setCodexError({
+              code: ERROR_CODES.AUTH_ERROR,
+              message: data.message || data.error,
+            });
+          }
+          return;
+        }
+        setCodexUsage(data);
+      } catch (err) {
+        setCodexError({
+          code: ERROR_CODES.UNKNOWN,
+          message: err instanceof Error ? err.message : 'Failed to fetch usage',
+        });
+      } finally {
+        if (!isAutoRefresh) setCodexLoading(false);
+      }
+    },
+    [setCodexUsage]
+  );
+
+  // Auto-fetch on mount if data is stale
+  useEffect(() => {
+    if (isClaudeStale && isClaudeAuthenticated) {
+      fetchClaudeUsage(true);
+    }
+  }, [isClaudeStale, isClaudeAuthenticated, fetchClaudeUsage]);
+
+  useEffect(() => {
+    if (isCodexStale && isCodexAuthenticated) {
+      fetchCodexUsage(true);
+    }
+  }, [isCodexStale, isCodexAuthenticated, fetchCodexUsage]);
+
+  // Auto-refresh when popover is open
+  useEffect(() => {
+    if (!open) return;
+
+    // Fetch based on active tab
+    if (activeTab === 'claude' && isClaudeAuthenticated) {
+      if (!claudeUsage || isClaudeStale) {
+        fetchClaudeUsage();
+      }
+      const intervalId = setInterval(() => {
+        fetchClaudeUsage(true);
+      }, REFRESH_INTERVAL_SECONDS * 1000);
+      return () => clearInterval(intervalId);
+    }
+
+    if (activeTab === 'codex' && isCodexAuthenticated) {
+      if (!codexUsage || isCodexStale) {
+        fetchCodexUsage();
+      }
+      const intervalId = setInterval(() => {
+        fetchCodexUsage(true);
+      }, REFRESH_INTERVAL_SECONDS * 1000);
+      return () => clearInterval(intervalId);
+    }
+  }, [
+    open,
+    activeTab,
+    claudeUsage,
+    isClaudeStale,
+    isClaudeAuthenticated,
+    codexUsage,
+    isCodexStale,
+    isCodexAuthenticated,
+    fetchClaudeUsage,
+    fetchCodexUsage,
+  ]);
 
   // Derived status color/icon helper
   const getStatusInfo = (percentage: number) => {
@@ -329,7 +416,7 @@ export function UsagePopover() {
                   variant="ghost"
                   size="icon"
                   className={cn('h-6 w-6', claudeLoading && 'opacity-80')}
-                  onClick={() => !claudeLoading && fetchClaudeUsage()}
+                  onClick={() => !claudeLoading && fetchClaudeUsage(false)}
                 >
                   <RefreshCw className="w-3.5 h-3.5" />
                 </Button>
@@ -362,7 +449,7 @@ export function UsagePopover() {
                 </div>
               ) : !claudeUsage ? (
                 <div className="flex flex-col items-center justify-center py-8 space-y-2">
-                  <Spinner size="lg" />
+                  <RefreshCw className="w-6 h-6 animate-spin text-muted-foreground/50" />
                   <p className="text-xs text-muted-foreground">Loading usage data...</p>
                 </div>
               ) : (
@@ -436,7 +523,7 @@ export function UsagePopover() {
                   variant="ghost"
                   size="icon"
                   className={cn('h-6 w-6', codexLoading && 'opacity-80')}
-                  onClick={() => !codexLoading && fetchCodexUsage()}
+                  onClick={() => !codexLoading && fetchCodexUsage(false)}
                 >
                   <RefreshCw className="w-3.5 h-3.5" />
                 </Button>
@@ -481,7 +568,7 @@ export function UsagePopover() {
                 </div>
               ) : !codexUsage ? (
                 <div className="flex flex-col items-center justify-center py-8 space-y-2">
-                  <Spinner size="lg" />
+                  <RefreshCw className="w-6 h-6 animate-spin text-muted-foreground/50" />
                   <p className="text-xs text-muted-foreground">Loading usage data...</p>
                 </div>
               ) : codexUsage.rateLimits ? (

apps/ui/src/components/views/agent-tools-view.tsx

@@ -11,12 +11,12 @@ import {
   Terminal,
   CheckCircle,
   XCircle,
+  Loader2,
   Play,
   File,
   Pencil,
   Wrench,
 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn } from '@/lib/utils';
 import { getElectronAPI } from '@/lib/electron';
 
@@ -236,7 +236,7 @@ export function AgentToolsView() {
               >
                 {isReadingFile ? (
                   <>
-                    <Spinner size="sm" className="mr-2" />
+                    <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                     Reading...
                   </>
                 ) : (
@@ -315,7 +315,7 @@ export function AgentToolsView() {
               >
                 {isWritingFile ? (
                   <>
-                    <Spinner size="sm" className="mr-2" />
+                    <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                     Writing...
                   </>
                 ) : (
@@ -383,7 +383,7 @@ export function AgentToolsView() {
               >
                 {isRunningCommand ? (
                   <>
-                    <Spinner size="sm" className="mr-2" />
+                    <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                     Running...
                   </>
                 ) : (

apps/ui/src/components/views/agent-view.tsx

@@ -42,7 +42,7 @@ export function AgentView() {
     return () => window.removeEventListener('resize', updateVisibility);
   }, []);
 
-  const [modelSelection, setModelSelection] = useState<PhaseModelEntry>({ model: 'claude-sonnet' });
+  const [modelSelection, setModelSelection] = useState<PhaseModelEntry>({ model: 'sonnet' });
 
   // Input ref for auto-focus
   const inputRef = useRef<HTMLTextAreaElement>(null);

apps/ui/src/components/views/agent-view/components/thinking-indicator.tsx

@@ -1,5 +1,4 @@
 import { Bot } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 
 export function ThinkingIndicator() {
   return (
@@ -9,7 +8,20 @@ export function ThinkingIndicator() {
       </div>
       <div className="bg-card border border-border rounded-2xl px-4 py-3 shadow-sm">
         <div className="flex items-center gap-3">
-          <Spinner size="sm" />
+          <div className="flex items-center gap-1">
+            <span
+              className="w-2 h-2 rounded-full bg-primary animate-pulse"
+              style={{ animationDelay: '0ms' }}
+            />
+            <span
+              className="w-2 h-2 rounded-full bg-primary animate-pulse"
+              style={{ animationDelay: '150ms' }}
+            />
+            <span
+              className="w-2 h-2 rounded-full bg-primary animate-pulse"
+              style={{ animationDelay: '300ms' }}
+            />
+          </div>
           <span className="text-sm text-muted-foreground">Thinking...</span>
         </div>
       </div>

apps/ui/src/components/views/analysis-view.tsx

@@ -1,9 +1,7 @@
 import { useCallback, useState } from 'react';
 import { createLogger } from '@automaker/utils/logger';
-import { useQueryClient } from '@tanstack/react-query';
 import { useAppStore, FileTreeNode, ProjectAnalysis } from '@/store/app-store';
 import { getElectronAPI } from '@/lib/electron';
-import { queryKeys } from '@/lib/query-keys';
 import { Card, CardContent, CardDescription, CardHeader, CardTitle } from '@/components/ui/card';
 import { Button } from '@/components/ui/button';
 import {
@@ -16,12 +14,12 @@ import {
   RefreshCw,
   BarChart3,
   FileCode,
+  Loader2,
   FileText,
   CheckCircle,
   AlertCircle,
   ListChecks,
 } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
 import { cn, generateUUID } from '@/lib/utils';
 
 const logger = createLogger('AnalysisView');
@@ -74,7 +72,6 @@ export function AnalysisView() {
   const [isGeneratingFeatureList, setIsGeneratingFeatureList] = useState(false);
   const [featureListGenerated, setFeatureListGenerated] = useState(false);
   const [featureListError, setFeatureListError] = useState<string | null>(null);
-  const queryClient = useQueryClient();
 
   // Recursively scan directory
   const scanDirectory = useCallback(
@@ -650,11 +647,6 @@ ${Object.entries(projectAnalysis.filesByExtension)
         } as any);
       }
 
-      // Invalidate React Query cache to sync UI
-      queryClient.invalidateQueries({
-        queryKey: queryKeys.features.all(currentProject.path),
-      });
-
       setFeatureListGenerated(true);
     } catch (error) {
       logger.error('Failed to generate feature list:', error);
@@ -664,7 +656,7 @@ ${Object.entries(projectAnalysis.filesByExtension)
     } finally {
       setIsGeneratingFeatureList(false);
     }
-  }, [currentProject, projectAnalysis, queryClient]);
+  }, [currentProject, projectAnalysis]);
 
   // Toggle folder expansion
   const toggleFolder = (path: string) => {
@@ -750,7 +742,7 @@ ${Object.entries(projectAnalysis.filesByExtension)
         <Button onClick={runAnalysis} disabled={isAnalyzing} data-testid="analyze-project-button">
           {isAnalyzing ? (
             <>
-              <Spinner size="sm" className="mr-2" />
+              <Loader2 className="w-4 h-4 mr-2 animate-spin" />
               Analyzing...
             </>
           ) : (
@@ -779,7 +771,7 @@ ${Object.entries(projectAnalysis.filesByExtension)
           </div>
         ) : isAnalyzing ? (
           <div className="flex flex-col items-center justify-center h-full">
-            <Spinner size="xl" className="mb-4" />
+            <Loader2 className="w-12 h-12 animate-spin text-primary mb-4" />
             <p className="text-muted-foreground">Scanning project files...</p>
           </div>
         ) : projectAnalysis ? (
@@ -858,7 +850,7 @@ ${Object.entries(projectAnalysis.filesByExtension)
                   >
                     {isGeneratingSpec ? (
                       <>
-                        <Spinner size="sm" className="mr-2" />
+                        <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                         Generating...
                       </>
                     ) : (
@@ -911,7 +903,7 @@ ${Object.entries(projectAnalysis.filesByExtension)
                   >
                     {isGeneratingFeatureList ? (
                       <>
-                        <Spinner size="sm" className="mr-2" />
+                        <Loader2 className="w-4 h-4 mr-2 animate-spin" />
                         Generating...
                       </>
                     ) : (

apps/ui/src/components/views/board-view/board-header.tsx

@@ -182,13 +182,6 @@ export function BoardHeader({
             >
               Auto Mode
             </Label>
-            <span
-              className="text-[10px] font-medium text-muted-foreground bg-muted/60 px-1.5 py-0.5 rounded"
-              data-testid="auto-mode-max-concurrency"
-              title="Max concurrent agents"
-            >
-              {maxConcurrency}
-            </span>
             <Switch
               id="auto-mode-toggle"
               checked={isAutoModeRunning}

apps/ui/src/components/views/board-view/board-search-bar.tsx

@@ -1,7 +1,6 @@
 import { useRef, useEffect } from 'react';
 import { Input } from '@/components/ui/input';
-import { Search, X } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import { Search, X, Loader2 } from 'lucide-react';
 
 interface BoardSearchBarProps {
   searchQuery: string;
@@ -76,7 +75,7 @@ export function BoardSearchBar({
           title="Creating App Specification"
           data-testid="spec-creation-badge"
         >
-          <Spinner size="xs" className="shrink-0" />
+          <Loader2 className="w-3 h-3 animate-spin text-brand-500 shrink-0" />
           <span className="text-xs font-medium text-brand-500 whitespace-nowrap">
             Creating spec
           </span>

apps/ui/src/components/views/board-view/components/kanban-card/agent-info-panel.tsx

@@ -1,4 +1,5 @@
-import { memo, useEffect, useState, useMemo } from 'react';
+// @ts-nocheck
+import { useEffect, useState, useMemo } from 'react';
 import { Feature, ThinkingLevel, ParsedTask } from '@/store/app-store';
 import type { ReasoningEffort } from '@automaker/types';
 import { getProviderFromModel } from '@/lib/utils';
@@ -10,12 +11,19 @@ import {
 } from '@/lib/agent-context-parser';
 import { cn } from '@/lib/utils';
 import type { AutoModeEvent } from '@/types/electron';
-import { Brain, ListTodo, Sparkles, Expand, CheckCircle2, Circle, Wrench } from 'lucide-react';
-import { Spinner } from '@/components/ui/spinner';
+import {
+  Brain,
+  ListTodo,
+  Sparkles,
+  Expand,
+  CheckCircle2,
+  Circle,
+  Loader2,
+  Wrench,
+} from 'lucide-react';
 import { getElectronAPI } from '@/lib/electron';
 import { SummaryDialog } from './summary-dialog';
 import { getProviderIconForModel } from '@/components/ui/provider-icon';
-import { useFeature, useAgentOutput } from '@/hooks/queries';
 
 /**
  * Formats thinking level for compact display
@@ -50,62 +58,30 @@ function formatReasoningEffort(effort: ReasoningEffort | undefined): string {
 
 interface AgentInfoPanelProps {
   feature: Feature;
-  projectPath: string;
   contextContent?: string;
   summary?: string;
   isCurrentAutoTask?: boolean;
 }
 
-export const AgentInfoPanel = memo(function AgentInfoPanel({
+export function AgentInfoPanel({
   feature,
-  projectPath,
   contextContent,
   summary,
   isCurrentAutoTask,
 }: AgentInfoPanelProps) {
+  const [agentInfo, setAgentInfo] = useState<AgentTaskInfo | null>(null);
   const [isSummaryDialogOpen, setIsSummaryDialogOpen] = useState(false);
   const [isTodosExpanded, setIsTodosExpanded] = useState(false);
   // Track real-time task status updates from WebSocket events
   const [taskStatusMap, setTaskStatusMap] = useState<
     Map<string, 'pending' | 'in_progress' | 'completed'>
   >(new Map());
-
-  // Determine if we should poll for updates
-  const shouldPoll = isCurrentAutoTask || feature.status === 'in_progress';
-  const shouldFetchData = feature.status !== 'backlog';
-
-  // Fetch fresh feature data for planSpec (store data can be stale for task progress)
-  const { data: freshFeature } = useFeature(projectPath, feature.id, {
-    enabled: shouldFetchData && !contextContent,
-    pollingInterval: shouldPoll ? 3000 : false,
-  });
-
-  // Fetch agent output for parsing
-  const { data: agentOutputContent } = useAgentOutput(projectPath, feature.id, {
-    enabled: shouldFetchData && !contextContent,
-    pollingInterval: shouldPoll ? 3000 : false,
-  });
-
-  // Parse agent output into agentInfo
-  const agentInfo = useMemo(() => {
-    if (contextContent) {
-      return parseAgentContext(contextContent);
-    }
-    if (agentOutputContent) {
-      return parseAgentContext(agentOutputContent);
-    }
-    return null;
-  }, [contextContent, agentOutputContent]);
-
-  // Fresh planSpec data from API (more accurate than store data for task progress)
-  const freshPlanSpec = useMemo(() => {
-    if (!freshFeature?.planSpec) return null;
-    return {
-      tasks: freshFeature.planSpec.tasks,
-      tasksCompleted: freshFeature.planSpec.tasksCompleted || 0,
-      currentTaskId: freshFeature.planSpec.currentTaskId,
-    };
-  }, [freshFeature?.planSpec]);
+  // Fresh planSpec data fetched from API (store data is stale for task progress)
+  const [freshPlanSpec, setFreshPlanSpec] = useState<{
+    tasks?: ParsedTask[];
+    tasksCompleted?: number;
+    currentTaskId?: string;
+  } | null>(null);
 
   // Derive effective todos from planSpec.tasks when available, fallback to agentInfo.todos
   // Uses freshPlanSpec (from API) for accurate progress, with taskStatusMap for real-time updates
@@ -157,6 +133,73 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
     taskStatusMap,
   ]);
 
+  useEffect(() => {
+    const loadContext = async () => {
+      if (contextContent) {
+        const info = parseAgentContext(contextContent);
+        setAgentInfo(info);
+        return;
+      }
+
+      if (feature.status === 'backlog') {
+        setAgentInfo(null);
+        setFreshPlanSpec(null);
+        return;
+      }
+
+      try {
+        const api = getElectronAPI();
+        const currentProject = (window as any).__currentProject;
+        if (!currentProject?.path) return;
+
+        if (api.features) {
+          // Fetch fresh feature data to get up-to-date planSpec (store data is stale)
+          try {
+            const featureResult = await api.features.get(currentProject.path, feature.id);
+            const freshFeature: any = (featureResult as any).feature;
+            if (featureResult.success && freshFeature?.planSpec) {
+              setFreshPlanSpec({
+                tasks: freshFeature.planSpec.tasks,
+                tasksCompleted: freshFeature.planSpec.tasksCompleted || 0,
+                currentTaskId: freshFeature.planSpec.currentTaskId,
+              });
+            }
+          } catch {
+            // Ignore errors fetching fresh planSpec
+          }
+
+          const result = await api.features.getAgentOutput(currentProject.path, feature.id);
+
+          if (result.success && result.content) {
+            const info = parseAgentContext(result.content);
+            setAgentInfo(info);
+          }
+        } else {
+          const contextPath = `${currentProject.path}/.automaker/features/${feature.id}/agent-output.md`;
+          const result = await api.readFile(contextPath);
+
+          if (result.success && result.content) {
+            const info = parseAgentContext(result.content);
+            setAgentInfo(info);
+          }
+        }
+      } catch {
+        console.debug('[KanbanCard] No context file for feature:', feature.id);
+      }
+    };
+
+    loadContext();
+
+    // Poll for updates when feature is in_progress (not just isCurrentAutoTask)
+    // This ensures planSpec progress stays in sync
+    if (isCurrentAutoTask || feature.status === 'in_progress') {
+      const interval = setInterval(loadContext, 3000);
+      return () => {
+        clearInterval(interval);
+      };
+    }
+  }, [feature.id, feature.status, contextContent, isCurrentAutoTask]);
+
   // Listen to WebSocket events for real-time task status updates
   // This ensures the Kanban card shows the same progress as the Agent Output modal
   // Listen for ANY in-progress feature with planSpec tasks, not just isCurrentAutoTask
@@ -295,7 +338,7 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
                       {todo.status === 'completed' ? (
                         <CheckCircle2 className="w-2.5 h-2.5 text-[var(--status-success)] shrink-0" />
                       ) : todo.status === 'in_progress' ? (
-                        <Spinner size="xs" className="w-2.5 h-2.5 shrink-0" />
+                        <Loader2 className="w-2.5 h-2.5 text-[var(--status-warning)] animate-spin shrink-0" />
                       ) : (
                         <Circle className="w-2.5 h-2.5 text-muted-foreground/50 shrink-0" />
                       )}
@@ -405,4 +448,4 @@ export const AgentInfoPanel = memo(function AgentInfoPanel({
       onOpenChange={setIsSummaryDialogOpen}
     />
   );
-});
+}

apps/ui/src/components/views/board-view/components/kanban-card/card-actions.tsx

@@ -1,5 +1,4 @@
 // @ts-nocheck
-import { memo } from 'react';
 import { Feature } from '@/store/app-store';
 import { Button } from '@/components/ui/button';
 import {
@@ -33,7 +32,7 @@ interface CardActionsProps {
   onApprovePlan?: () => void;
 }
 
-export const CardActions = memo(function CardActions({
+export function CardActions({
   feature,
   isCurrentAutoTask,
   hasContext,
@@ -345,4 +344,4 @@ export const CardActions = memo(function CardActions({
       )}
     </div>
   );
-});
+}

apps/ui/src/components/views/board-view/components/kanban-card/card-badges.tsx

@@ -1,11 +1,10 @@
 // @ts-nocheck
-import { memo, useEffect, useMemo, useState } from 'react';
+import { useEffect, useMemo, useState } from 'react';
 import { Feature, useAppStore } from '@/store/app-store';
 import { cn } from '@/lib/utils';
 import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from '@/components/ui/tooltip';
 import { AlertCircle, Lock, Hand, Sparkles } from 'lucide-react';
 import { getBlockingDependencies } from '@automaker/dependency-resolver';
-import { useShallow } from 'zustand/react/shallow';
 
 /** Uniform badge style for all card badges */
 const uniformBadgeClass =
@@ -19,7 +18,7 @@ interface CardBadgesProps {
  * CardBadges - Shows error badges below the card header
  * Note: Blocked/Lock badges are now shown in PriorityBadges for visual consistency
  */
-export const CardBadges = memo(function CardBadges({ feature }: CardBadgesProps) {
+export function CardBadges({ feature }: CardBadgesProps) {
   if (!feature.error) {
     return null;
   }
@@ -47,19 +46,14 @@ export const CardBadges = memo(function CardBadges({ feature }: CardBadgesProps)
       </TooltipProvider>
     </div>
   );
-});
+}
 
 interface PriorityBadgesProps {
   feature: Feature;
 }
 
-export const PriorityBadges = memo(function PriorityBadges({ feature }: PriorityBadgesProps) {
-  const { enableDependencyBlocking, features } = useAppStore(
-    useShallow((state) => ({
-      enableDependencyBlocking: state.enableDependencyBlocking,
-      features: state.features,
-    }))
-  );
+export function PriorityBadges({ feature }: PriorityBadgesProps) {
+  const { enableDependencyBlocking, features } = useAppStore();
   const [currentTime, setCurrentTime] = useState(() => Date.now());
 
   // Calculate blocking dependencies (if feature is in backlog and has incomplete dependencies)
@@ -229,4 +223,4 @@ export const PriorityBadges = memo(function PriorityBadges({ feature }: Priority
       )}
     </div>
   );
-});
+}