Skip to content

feat: move desktop settings into dedicated window#252

Merged
ksamaschke merged 2 commits intodevfrom
codex/issue-227-settings-window-20260315063633
Mar 15, 2026
Merged

feat: move desktop settings into dedicated window#252
ksamaschke merged 2 commits intodevfrom
codex/issue-227-settings-window-20260315063633

Conversation

@ksamaschke
Copy link
Copy Markdown
Contributor

Summary

  • move theme controls and desktop preferences out of the main shell into a dedicated Settings window
  • add a typed desktop bridge path for opening/focusing the reusable Settings BrowserWindow
  • extract shared appearance/window-role helpers and remove obsolete main-window Settings ownership paths

Changes

  • removed the main-window Settings tab and inline appearance popover from InstallerDashboard
  • added windowRole-based renderer composition plus a dedicated Settings appearance component
  • wired openSettingsWindow() through bridge, preload, control-plane client, and Electron main window management
  • added focused Settings-ownership tests plus adjacent desktop bridge/client coverage

Test Plan

  • npm run build:quick
  • node --test dist/tests/installer/desktop-settings-window-red-phase.test.js
  • node --test dist/tests/installer/control-plane-client.test.js dist/tests/installer/desktop-bridge-red-phase.test.js dist/tests/installer/desktop-preview-startup.test.js dist/tests/installer/desktop-shell-ux.test.js
  • node --test dist/tests/installer/desktop-dashboard-build.test.js

Breaking Changes

  • None

@ksamaschke
Copy link
Copy Markdown
Contributor Author

ICA-REVIEW
ICA-REVIEW-RECEIPT
Reviewer-Stage: 3 (temp checkout)
Reviewer-Agent: reviewer (subagent)
PR: #252
Base: dev
Head-SHA: d848a15
Date-UTC: 2026-03-15T06:02:13Z

Findings: 0
NO FINDINGS

Checks/Tests:

  • npm test (PASS)
  • gh pr checks 252 (PASS)

Notes:

  • Fresh temp checkout review completed against the PR head SHA.
  • Dedicated Settings ownership is isolated to the Settings window; the main window now launches it instead of owning theme/preferences UI.

Result: PASS

ICA-SECURITY-REVIEW
ICA-SECURITY-REVIEW-RECEIPT
Security-Reviewer-Stage: post-pr (temp checkout)
Security-Reviewer-Agent: security reviewer (subagent)
Security-Reviewer-Execution: dedicated-security-subagent
Security-Reviewer-Executor: github:ksamaschke
Security-Reviewer-Run-ID: sec-20260315T060213Z-d848a1550690
PR: #252
Base: dev
Head-SHA: d848a15
Date-UTC: 2026-03-15T06:02:13Z

Findings: 0
NO FINDINGS

Result: PASS

@ksamaschke ksamaschke merged commit 640ba30 into dev Mar 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant