Skip to content

Split Workspace.swift (7,437) and the remote-session files #98

Description

@arzafran

Workspace.swift is one class doing session persistence, layout building, theming, remote-connection glue, surface creation, focus/geometry reconciliation, and the entire BonsplitDelegate implementation. WorkspaceRemoteSession.swift (2,642) mixes connection orchestration, daemon install, SSH/SCP process execution, port scanning, and script-string builders. WorkspaceRemoteDaemon.swift (2,300) already contains 5 uncoupled types that just need their own files.

Codex convergence: proposed WorkspaceSidebarState/WorkspaceTelemetryStore extraction so socket telemetry has one mutation target. The remote port-discovery engine's ~10 independently-mutated coordination fields are the one area needing an explicit state model rather than a mechanical move.


From the 2026-07-09 nuclear review (10 module reviewers + Codex cross-model pass, disprove-first). Line refs are against the reductive-audit branch (PR #81).

Metadata

Metadata

Assignees

No one assigned

    Labels

    nuclear-reviewFrom the 2026-07-09 whole-codebase nuclear reviewremote-daemonSSH-remote daemon (programad-remote)tech-debtKnown latent issue / maintenance debt

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions