refactor: clean up settings service and improve E2E fixture descriptions

- Removed the redundant call to ignore empty array overwrite for 'enabledCodexModels' in the SettingsService.
- Reformatted the description of the 'Heavy Task' profile in the E2E fixture setup script for better readability.

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

@@ -302,7 +302,6 @@ export class SettingsService {
     ignoreEmptyArrayOverwrite('aiProfiles');
     ignoreEmptyArrayOverwrite('mcpServers');
     ignoreEmptyArrayOverwrite('enabledCursorModels');
-    ignoreEmptyArrayOverwrite('enabledCodexModels');
 
     // Empty object overwrite guard
     if (

apps/ui/scripts/setup-e2e-fixtures.mjs

@@ -98,7 +98,8 @@ const E2E_SETTINGS = {
     {
       id: 'profile-heavy-task',
       name: 'Heavy Task',
-      description: 'Claude Opus with Ultrathink for complex architecture, migrations, or deep debugging.',
+      description:
+        'Claude Opus with Ultrathink for complex architecture, migrations, or deep debugging.',
       model: 'opus',
       thinkingLevel: 'ultrathink',
       provider: 'claude',