From 8992f667c7d9491baa3141f8e49d1bfe3a906f80 Mon Sep 17 00:00:00 2001
From: webdevcody <webdevcody@gmail.com>
Date: Wed, 7 Jan 2026 23:04:27 -0500
Subject: [PATCH] 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 | 1 -
 apps/ui/scripts/setup-e2e-fixtures.mjs       | 3 ++-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/apps/server/src/services/settings-service.ts b/apps/server/src/services/settings-service.ts
index 15a27b7b..15154655 100644
--- a/apps/server/src/services/settings-service.ts
+++ b/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 (
diff --git a/apps/ui/scripts/setup-e2e-fixtures.mjs b/apps/ui/scripts/setup-e2e-fixtures.mjs
index 55424412..d0d604f4 100644
--- a/apps/ui/scripts/setup-e2e-fixtures.mjs
+++ b/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',