Merge pull request #19 from Spacecraft-Software/ui-reduced-motion

feat(vault-config): ui.reduced_motion key (reserved groundwork)

Author: UnbreakableMJ

CHANGELOG.md

@@ -10,6 +10,16 @@ range may break in any release.
 
 ### Added
 
+- **`ui.reduced_motion` config key (reserved).** Completes the PRD §7.1 config
+  registry with the accessibility preference to suppress animated TUI elements.
+  The TUI has no animations yet (it's fully event-driven — no spinner, no
+  lock-countdown, no blink), so the key is **inert groundwork**: `vault-config`
+  records/validates it and `vault-tui` populates an `App.reduced_motion` flag
+  from it, ready for a future spinner / lock-countdown to honor without
+  re-plumbing. It's a TUI-rendering preference, read by the TUI directly (not
+  relayed to the agent). Tested in `vault-config` (round-trip, rejects
+  non-booleans, never emitted as an agent flag).
+
 - **TUI About overlay.** `vault-tui` gains a read-only About screen (open with
   `?` or `:about`, dismiss with `Esc`/`q`) showing the version, maintainer,
   copyright/license, and canonical URL — rendered from the same

README.md

@@ -148,11 +148,14 @@ vault config unset clipboard.clear_secs
 Recognised keys: `clipboard.clear_secs` (auto-clear window, `0` disables),
 `clipboard.backend` (`auto`/`arboard`/`osc52`; see below),
 `agent.idle_lock_secs` (idle-lock timeout, `0` disables),
-`agent.session_keyring` (resume across restarts; see above), and
-`sync.interval_secs` (background `/sync` interval while unlocked, `0` disables).
-When the CLI auto-starts the agent, these populate its launch flags (changes
-apply on the next agent spawn). Wipe the on-disk item cache (and drop a running
-agent's keys) with `vault purge`.
+`agent.session_keyring` (resume across restarts; see above),
+`sync.interval_secs` (background `/sync` interval while unlocked, `0` disables),
+and `ui.reduced_motion` (suppress animated TUI elements — **reserved**: the TUI
+has no animations yet, so this records the preference for when a spinner /
+lock-countdown lands). When the CLI auto-starts the agent, the agent-side keys
+populate its launch flags (changes apply on the next agent spawn);
+`ui.reduced_motion` is read by `vault-tui` directly. Wipe the on-disk item cache
+(and drop a running agent's keys) with `vault purge`.
 
 With `sync.interval_secs` set, the agent re-pulls `/sync` on that cadence while
 unlocked — keeping the in-memory vault and offline cache fresh without a manual

crates/vault-config/src/lib.rs

@@ -28,6 +28,7 @@ pub const KNOWN_KEYS: &[&str] = &[
     "agent.idle_lock_secs",
     "agent.session_keyring",
     "sync.interval_secs",
+    "ui.reduced_motion",
 ];
 
 /// Accepted values for `clipboard.backend`.
@@ -45,6 +46,8 @@ pub struct Config {
     pub agent: AgentCfg,
     /// Background-sync settings.
     pub sync: SyncCfg,
+    /// TUI rendering preferences.
+    pub ui: UiCfg,
     /// Registered account profile (written by `vault register`). Skipped from
     /// the file until something is set, so an unregistered config carries no
     /// empty `[account]` table.
@@ -86,6 +89,17 @@ pub struct SyncCfg {
     pub interval_secs: Option<u64>,
 }
 
+/// `[ui]` table — TUI-only rendering preferences (the agent never renders, so
+/// these are read by `vault-tui` directly, not relayed via `agent_args`).
+#[derive(Clone, Debug, Default, PartialEq, Eq, Deserialize, Serialize)]
+#[serde(default, deny_unknown_fields)]
+pub struct UiCfg {
+    /// Suppress animated TUI elements (spinner / lock-countdown) for
+    /// accessibility. Reserved: the TUI has no animations yet, so this records
+    /// the preference for when they land.
+    pub reduced_motion: Option<bool>,
+}
+
 /// `[account]` table — the registered account, written by `vault register`
 /// and read by `login`/`unlock` to default `server`/`email`/`device_id`.
 #[derive(Clone, Debug, Default, PartialEq, Eq, Deserialize, Serialize)]
@@ -142,6 +156,12 @@ impl Config {
         self.sync.interval_secs
     }
 
+    /// Effective `ui.reduced_motion`, if set.
+    #[must_use]
+    pub const fn reduced_motion(&self) -> Option<bool> {
+        self.ui.reduced_motion
+    }
+
     /// The registered account profile.
     #[must_use]
     pub const fn account(&self) -> &AccountCfg {
@@ -171,6 +191,7 @@ impl Config {
             "agent.idle_lock_secs" => Ok(self.agent.idle_lock_secs.map(|v| v.to_string())),
             "agent.session_keyring" => Ok(self.agent.session_keyring.map(|v| v.to_string())),
             "sync.interval_secs" => Ok(self.sync.interval_secs.map(|v| v.to_string())),
+            "ui.reduced_motion" => Ok(self.ui.reduced_motion.map(|v| v.to_string())),
             other => Err(other.to_owned()),
         }
     }
@@ -203,6 +224,10 @@ impl Config {
                 self.sync.interval_secs = Some(parse_u64(key, raw)?);
                 Ok(())
             }
+            "ui.reduced_motion" => {
+                self.ui.reduced_motion = Some(parse_bool(key, raw)?);
+                Ok(())
+            }
             other => Err(unknown_key(other)),
         }
     }
@@ -234,6 +259,10 @@ impl Config {
                 self.sync.interval_secs = None;
                 Ok(())
             }
+            "ui.reduced_motion" => {
+                self.ui.reduced_motion = None;
+                Ok(())
+            }
             other => Err(unknown_key(other)),
         }
     }
@@ -524,6 +553,27 @@ mod tests {
         assert_eq!(c.clipboard_backend(), None);
     }
 
+    #[test]
+    fn reduced_motion_round_trips_and_is_tui_only() {
+        let mut c = Config::default();
+        assert_eq!(c.reduced_motion(), None);
+        c.set("ui.reduced_motion", "true").expect("set true");
+        assert_eq!(c.reduced_motion(), Some(true));
+        // It's a TUI-rendering preference — never relayed to the agent.
+        assert!(
+            !agent_args(&c).contains(&OsString::from("--reduced-motion")),
+            "ui.reduced_motion must not become an agent flag"
+        );
+        // Survives a toml round-trip.
+        let text = toml::to_string_pretty(&c).expect("serialise");
+        let back: Config = toml::from_str(&text).expect("parse");
+        assert_eq!(back.reduced_motion(), Some(true));
+        // Non-boolean rejected; unset clears.
+        assert!(c.set("ui.reduced_motion", "sometimes").is_err());
+        c.unset("ui.reduced_motion").expect("unset");
+        assert_eq!(c.reduced_motion(), None);
+    }
+
     #[test]
     fn sync_interval_round_trips() {
         let mut c = Config::default();

crates/vault-tui/src/app.rs

@@ -747,6 +747,10 @@ pub struct App {
     pub toast: Option<String>,
     /// Interactive unlock state, `Some` while [`Screen::Unlock`] is shown.
     pub unlock: Option<UnlockState>,
+    /// Suppress animated UI (`ui.reduced_motion`). Reserved: the TUI has no
+    /// animations yet, so nothing reads this — it's populated from config so a
+    /// future spinner / lock-countdown can honor it without re-plumbing.
+    pub reduced_motion: bool,
     /// Set when the user asks to quit.
     pub should_quit: bool,
 }
@@ -775,6 +779,7 @@ impl App {
             osc52_clear_at: None,
             toast: None,
             unlock: None,
+            reduced_motion: false,
             should_quit: false,
         }
     }
@@ -808,6 +813,7 @@ impl App {
             osc52_clear_at: None,
             toast: None,
             unlock: None,
+            reduced_motion: false,
             should_quit: false,
         }
     }

crates/vault-tui/src/main.rs

@@ -178,7 +178,7 @@ async fn run(socket: &Path) -> anyhow::Result<()> {
 
 /// Query the agent and build the initial (or refreshed) [`App`].
 async fn load_app(socket: &Path) -> App {
-    match client::request(socket, &Request::Status).await {
+    let mut app = match client::request(socket, &Request::Status).await {
         Err(e) => App::message("No agent", e.to_string(), None),
         Ok(Response::Status(s)) if s.unlocked => {
             match client::request(socket, &Request::List).await {
@@ -193,7 +193,14 @@ async fn load_app(socket: &Path) -> App {
         Ok(Response::Status(s)) => locked_screen(socket, s).await,
         Ok(Response::Error(err)) => App::message("Error", err.to_string(), None),
         Ok(other) => App::message("Error", format!("unexpected response: {other:?}"), None),
-    }
+    };
+    // Reserved accessibility preference (`ui.reduced_motion`): record it on the
+    // App so a future animated element can honor it. No visible effect yet.
+    app.reduced_motion = vault_config::load()
+        .ok()
+        .and_then(|c| c.reduced_motion())
+        .unwrap_or(false);
+    app
 }
 
 /// Build the locked screen: an interactive unlock prompt when an account is