style: fix CI formatting issues in login_screen.rs (#4)

* style: fix formatting issues in login_screen.rs

- Fix trailing whitespace on empty lines
- Reformat tracing::trace! macro call to proper multi-line format

Fixes CI Format check failures.

* fix(test): prevent flaky test_caching test with serial execution

The test_caching test uses a static cache shared across all tests. When
tests run in parallel, other tests can modify the cache between
clear_cache() and the cache_size() assertions, causing spurious failures.

Fix by adding #[serial] attribute from serial_test crate to ensure this
test runs in isolation from other tests that use the cache.

Author: echobt

Cargo.lock

@@ -132,3 +132,4 @@ tempfile = { workspace = true }
 tokio-test = { workspace = true }
 wiremock = { workspace = true }
 insta = { workspace = true }
+serial_test = { workspace = true }

src/cortex-engine/Cargo.toml

@@ -247,6 +247,7 @@ pub fn git_root(path: &Path) -> Option<PathBuf> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
     use tempfile::TempDir;
 
     fn setup_test_dir() -> TempDir {
@@ -344,8 +345,9 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_caching() {
-        // Clear cache first since tests run in parallel and share static cache
+        // Clear cache first - must use #[serial] since tests share static cache
         clear_cache();
         assert_eq!(cache_size(), 0);
 

src/cortex-engine/src/config/config_discovery.rs

@@ -223,7 +223,7 @@ impl LoginScreen {
                         return Ok(result);
                     }
                 }
-                
+
                 // Handle async messages from token polling
                 msg = async {
                     if let Some(ref mut rx) = self.async_rx {
@@ -237,7 +237,7 @@ impl LoginScreen {
                         self.handle_async_message(msg);
                     }
                 }
-                
+
                 // Periodic render tick to keep UI responsive
                 _ = render_interval.tick() => {
                     // Just continue to re-render
@@ -713,7 +713,7 @@ async fn poll_for_token_async(
     tx: mpsc::Sender<AsyncMessage>,
 ) {
     tracing::debug!("Token polling started");
-    
+
     let client = match cortex_engine::create_default_client() {
         Ok(c) => c,
         Err(e) => {
@@ -736,7 +736,11 @@ async fn poll_for_token_async(
             return;
         }
 
-        tracing::trace!("Polling for token (attempt {}/{})", attempt + 1, max_attempts);
+        tracing::trace!(
+            "Polling for token (attempt {}/{})",
+            attempt + 1,
+            max_attempts
+        );
 
         let response = match client
             .post(format!("{}/auth/device/token", API_BASE_URL))
@@ -756,7 +760,7 @@ async fn poll_for_token_async(
 
         if status.is_success() {
             tracing::debug!("Token response received (success)");
-            
+
             #[derive(serde::Deserialize)]
             struct TokenResponse {
                 access_token: String,