refactor: split App god-struct into domain sub-states

Moved 43 domain-owned fields off the App god-struct (down from 104 to
66 pub lines in src/app.rs) into six focused sub-structs. Each sub-state
owns its fields and the helper methods that operate purely on them.
App keeps thin shims where external call-sites used to dispatch
through its public surface.

New sub-states in src/app/:
- tunnel_state.rs    (TunnelState)    list, form, active, form_baseline,
                                      pending_delete, summaries_cache
- status_state.rs    (StatusCenter)   status, toast, toast_queue and
                                      the set_/tick_ helpers
- snippet_state.rs   (SnippetState)   store, form, pending, output,
                                      param_form, pending_terminal,
                                      form_baseline, pending_delete
- provider_state.rs  (ProviderState)  config, form, syncing, sync_done,
                                      sync_had_errors, pending_delete,
                                      sync_history, form_baseline
- form_state.rs      (FormState)      host, host_baseline, bulk_tag_editor,
                                      bulk_tag_undo, pending_discard_confirm
- host_state.rs      (HostState)      ssh_config, list, patterns,
                                      display_list, render_cache,
                                      undo_stack, multi_select, sort_mode,
                                      group_by, view_mode, group_filter,
                                      group_tab_index, group_tab_order,
                                      group_host_counts

Access patterns:
  app.tunnels.*, app.status_center.*, app.snippets.*, app.providers.*,
  app.forms.*, app.hosts_state.*

Pure move. No behaviour changes. No new features. Visual goldens
(apart from version string and What's New overlay embedding the
CHANGELOG) and Cargo.toml semver patch bump only.

Bumps version to 2.43.1.

Closes #36

Author: erickochen

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 2.43.1 - 2026-04-18
+
+- feat: Tighter foundations for faster, safer releases.
+- change: App state is split into six focused domains (hosts, tunnels, snippets, providers, forms, status) so changes land in one place and are easier to reason about.
+- change: New behaviour tests cover status routing, toast expiry and provider ordering, catching regressions earlier.
+- change: Fewer surprise file reads at startup. Provider state loads explicitly in one place, so tests and tools run predictably.
+
 ## 2.43.0 - 2026-04-18
 
 - feat: Confident edits on every host, shared lines included.

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "purple-ssh"
-version = "2.43.0"
+version = "2.43.1"
 edition = "2024"
 description = "Open-source terminal SSH manager and SSH config editor. Search hundreds of hosts, sync from 16 clouds, transfer files, manage Docker and Podman over SSH, sign short-lived Vault SSH certs and expose an MCP server for AI agents. Rust TUI, MIT licensed."
 license = "MIT"

Cargo.toml

@@ -407,7 +407,7 @@ A: Press m in the host list to open the theme picker with live preview. 11 built
 
 ## Status
 
-- Current version: 2.43.0 (April 2026)
+- Current version: 2.43.1 (April 2026)
 - Release cadence: approximately bi-weekly
 - Test suite: 6500+ tests (unit, integration, property-based, HTTP mocking and OpenSSH ground-truth cross-validation)
 - CI: fmt, clippy, build, test on macOS and Linux, cargo-deny, MSRV 1.86 check, rustdoc warnings, site sync, TUI smoke test, design system, message centralization, keybinding invariants and visual regression

llms.txt

@@ -31,18 +31,30 @@ if grep -rn 'super::footer_action\|super::footer_key_span' src/ui/ \
 fi
 
 # 3. No old notification API outside method definitions and delegations.
+#
+# Exclusions:
+#  - `src/app/status_state.rs` is the defining module, so its own inline
+#    `#[cfg(test)] mod tests` may call its deprecated `set_*` methods to
+#    verify their behaviour. The deprecation is already signalled by the
+#    `#[deprecated]` attribute on the definition site.
+#  - `src/app.rs` dispatches through `status_center.set_*` shims; the
+#    regex is anchored to that file so the exception cannot leak.
 if grep -rn 'set_status\|set_background_status\|set_sticky_status\|set_info_status' \
     src/ --include='*.rs' \
     | grep -v 'tests\.rs' | grep -v 'test_' | grep -v '#\[deprecated' \
     | grep -v 'pub fn ' | grep -v 'pub use ' \
-    | grep -v 'self\.set_' | grep -v '// ' | grep -v '/// ' \
+    | grep -v 'self\.set_' | grep -Ev '^src/app\.rs:[0-9]+:.*status_center\.set_' \
+    | grep -v '^src/app/status_state\.rs:' \
+    | grep -v '// ' | grep -v '/// ' \
     | grep -q .; then
     echo "ERROR: Old notification API used. Use app.notify/notify_error/etc."
     grep -rn 'set_status\|set_background_status\|set_sticky_status\|set_info_status' \
         src/ --include='*.rs' \
         | grep -v 'tests\.rs' | grep -v 'test_' | grep -v '#\[deprecated' \
         | grep -v 'pub fn ' | grep -v 'pub use ' \
-        | grep -v 'self\.set_' | grep -v '// ' | grep -v '/// '
+        | grep -v 'self\.set_' | grep -Ev '^src/app\.rs:[0-9]+:.*status_center\.set_' \
+        | grep -v '^src/app/status_state\.rs:' \
+        | grep -v '// ' | grep -v '/// '
     exit 1
 fi
 

scripts/check-design-system.sh

@@ -41,7 +41,7 @@
   "url": "https://getpurple.sh",
   "downloadUrl": "https://getpurple.sh",
   "installUrl": "https://github.com/erickochen/purple/releases",
-  "softwareVersion": "2.43.0",
+  "softwareVersion": "2.43.1",
   "datePublished": "2024-10-01",
   "dateModified": "2026-04-18",
   "softwareRequirements": "macOS or Linux",
@@ -778,7 +778,7 @@ <h2 class="tagline">Open-source terminal SSH manager and SSH config editor for m
           <button class="copy-btn copy-inline" id="copy-btn" onclick="copy(this)" style="display:none">copy</button>
         </div>
         <div class="install-output" id="install-output" style="display:none">
-          <div>Downloading purple v2.43.0 for darwin-arm64...</div>
+          <div>Downloading purple v2.43.1 for darwin-arm64...</div>
           <div>Installing to /usr/local/bin/purple... <span class="success">done.</span></div>
         </div>
         <div class="alt-installs" id="alt-installs" style="display:none">
@@ -900,7 +900,7 @@ <h2 class="tagline">Open-source terminal SSH manager and SSH config editor for m
         <summary>How do I troubleshoot connection problems?</summary>
         <div class="answer">Run with <code>--verbose</code> to enable debug logging, then <code>purple logs --tail</code> in another terminal. Logs are written to <code>~/.purple/purple.log</code> with fault domain prefixes: <code>[external]</code> for remote/tool errors, <code>[config]</code> for local config issues. Set <code>PURPLE_LOG=trace</code> for maximum detail.</div>
       </details>
-      <div class="man-foot"><span>purple v2.43.0</span><span>2026-04-18</span><span>PURPLE(1)</span></div>
+      <div class="man-foot"><span>purple v2.43.1</span><span>2026-04-18</span><span>PURPLE(1)</span></div>
     </div>
   </div>
 

site/page.html

@@ -177,7 +177,7 @@ const LANDING_PAGE = `<!DOCTYPE html>
   "url": "https://getpurple.sh",
   "downloadUrl": "https://getpurple.sh",
   "installUrl": "https://github.com/erickochen/purple/releases",
-  "softwareVersion": "2.43.0",
+  "softwareVersion": "2.43.1",
   "datePublished": "2024-10-01",
   "dateModified": "2026-04-18",
   "softwareRequirements": "macOS or Linux",
@@ -914,7 +914,7 @@ footer .sep { margin: 0 0.3em; }
           <button class="copy-btn copy-inline" id="copy-btn" onclick="copy(this)" style="display:none">copy</button>
         </div>
         <div class="install-output" id="install-output" style="display:none">
-          <div>Downloading purple v2.43.0 for darwin-arm64...</div>
+          <div>Downloading purple v2.43.1 for darwin-arm64...</div>
           <div>Installing to /usr/local/bin/purple... <span class="success">done.</span></div>
         </div>
         <div class="alt-installs" id="alt-installs" style="display:none">
@@ -1036,7 +1036,7 @@ footer .sep { margin: 0 0.3em; }
         <summary>How do I troubleshoot connection problems?</summary>
         <div class="answer">Run with <code>--verbose</code> to enable debug logging, then <code>purple logs --tail</code> in another terminal. Logs are written to <code>~/.purple/purple.log</code> with fault domain prefixes: <code>[external]</code> for remote/tool errors, <code>[config]</code> for local config issues. Set <code>PURPLE_LOG=trace</code> for maximum detail.</div>
       </details>
-      <div class="man-foot"><span>purple v2.43.0</span><span>2026-04-18</span><span>PURPLE(1)</span></div>
+      <div class="man-foot"><span>purple v2.43.1</span><span>2026-04-18</span><span>PURPLE(1)</span></div>
     </div>
   </div>
 
@@ -1500,7 +1500,7 @@ A: Press m in the host list to open the theme picker with live preview. 11 built
 
 ## Status
 
-- Current version: 2.43.0 (April 2026)
+- Current version: 2.43.1 (April 2026)
 - Release cadence: approximately bi-weekly
 - Test suite: 6500+ tests (unit, integration, property-based, HTTP mocking and OpenSSH ground-truth cross-validation)
 - CI: fmt, clippy, build, test on macOS and Linux, cargo-deny, MSRV 1.86 check, rustdoc warnings, site sync, TUI smoke test, design system, message centralization, keybinding invariants and visual regression

site/worker.ts

@@ -163,7 +163,7 @@ impl AnimationState {
         // Detail panel toggle
         if app.detail_toggle_pending {
             app.detail_toggle_pending = false;
-            let opening = app.view_mode == crate::app::ViewMode::Detailed;
+            let opening = app.hosts_state.view_mode == crate::app::ViewMode::Detailed;
             let start_progress =
                 self.detail_anim_progress()
                     .unwrap_or(if opening { 0.0 } else { 1.0 });
@@ -262,7 +262,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         anim.detect_transitions(&mut app);
         assert!(anim.is_animating(&app));
     }
@@ -466,7 +466,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         anim.detect_transitions(&mut app);
         assert!(!app.detail_toggle_pending);
         assert!(anim.detail_anim.is_some());
@@ -477,7 +477,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Compact;
+        app.hosts_state.view_mode = crate::app::ViewMode::Compact;
         anim.detect_transitions(&mut app);
         assert!(anim.detail_anim.is_some());
     }
@@ -487,7 +487,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         anim.detect_transitions(&mut app);
         let p = anim.detail_anim_progress();
         assert!(p.is_some());
@@ -505,7 +505,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         anim.detect_transitions(&mut app);
         anim.detail_anim.as_mut().unwrap().start =
             Instant::now() - std::time::Duration::from_millis(300);
@@ -518,12 +518,12 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         anim.detect_transitions(&mut app);
         let _ = anim.detail_anim_progress();
 
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Compact;
+        app.hosts_state.view_mode = crate::app::ViewMode::Compact;
         anim.detect_transitions(&mut app);
         assert!(anim.detail_anim.is_some());
         assert!(!anim.detail_anim.as_ref().unwrap().opening);
@@ -534,7 +534,7 @@ mod tests {
         let mut app = make_app();
         let mut anim = AnimationState::new();
         app.detail_toggle_pending = true;
-        app.view_mode = crate::app::ViewMode::Detailed;
+        app.hosts_state.view_mode = crate::app::ViewMode::Detailed;
         app.screen = Screen::Help {
             return_screen: Box::new(Screen::HostList),
         };