View Refactor

View Refactor

Overview

Refactor macOS views toward small, explicit, stable scene and view types. Default to native SwiftUI for layout, selection, commands, and settings. Reach for AppKit only at the narrow edges where desktop behavior truly requires it.

Core Guidelines

1) Model scenes explicitly

• Break the app into meaningful scene roots: main window, settings, utility windows, inspectors, or menu bar extras.
• Do not let one giant root view silently own every desktop surface.

2) Keep a predictable file shape

• Follow this ordering unless the file already has a stronger local convention:
• Environment
• private/public let
• @State / other stored properties
• computed var (non-view)
• init
• body
• computed view builders / other view helpers
• helper / async functions

2b) Split files by responsibility

• For non-trivial apps, do not keep the full app, all views, models, stores, networking clients, process clients, and helpers in one Swift file.
• Accept a single Swift file only for tiny throwaway examples or snippets: roughly under 50 lines, one screen, no persistence, no networking/process client, and no reusable models.
• Use App/<AppName>App.swift for the @main app and AppDelegate only.
• Keep Views/ContentView.swift focused on root layout and composition; move feature UI into files such as Views/SidebarView.swift, Views/DetailView.swift, and Views/ComposerView.swift.
• Move value types and selection enums into Models/*.swift, stores into Stores/*.swift, app-server/network/process clients into Services/*.swift, and small formatters/resolvers/extensions into Support/*.swift.
• Keep files small and named after the primary type they contain.

3) Prefer dedicated subview types over many computed some View fragments

• Extract meaningful desktop sections like sidebar rows, detail panels, inspectors, or toolbar content into focused subviews.
• Keep computed some View helpers small and rare.
• Pass explicit data, bindings, and actions into subviews instead of handing down the whole scene model.

4) Keep selection and layout stable

• Prefer one stable split or window layout with local conditionals inside it.
• Avoid top-level branch swapping between radically different roots when selection changes.
• Let the layout be constant; let state drive the content inside it.

5) Extract commands, toolbars, and actions out of body

• Do not bury non-trivial button logic inline.
• Do not mix command routing, menu state, and layout in the same block if they can be named clearly.
• Keep body readable as UI, not as a desktop view controller.

6) Use scene and app storage intentionally

• Use @SceneStorage for per-window ephemeral state when it truly helps restore the scene.
• Use @AppStorage for durable preferences, not transient UI toggles that only matter in one window.
• Keep scene-owned state close to the scene root.

7) Keep AppKit escape hatches narrow

• If a representable or NSWindow bridge exists, isolate it behind a small wrapper or helper.
• Do not let AppKit references spread through unrelated SwiftUI views.
• If the bridge starts owning the feature, re-evaluate the architecture.