style: Format code and add CLAUDE.md memory files

- Reorder imports for consistency (db::vault before models)
- Format long lines for better readability
- Add #[allow(dead_code)] for unused SyncStatus variants
- Update test formatting with consistent line breaks
- Add CLAUDE.md memory tracking files for modules

Co-Authored-By: Claude (glm-4.7) <noreply@anthropic.com>

Author: alphaqiu

.github/CLAUDE.md

@@ -0,0 +1,11 @@
+<claude-mem-context>
+# Recent Activity
+
+<!-- This section is auto-generated by claude-mem. Edit content outside the tags. -->
+
+### Jan 29, 2026
+
+| ID | Time | T | Title | Read |
+|----|------|---|-------|------|
+| #244 | 12:16 AM | 🔴 | Fixed multiple GitHub Actions test failures in keyring-cli | ~381 |
+</claude-mem-context>

.github/workflows/CLAUDE.md

@@ -0,0 +1,12 @@
+<claude-mem-context>
+# Recent Activity
+
+<!-- This section is auto-generated by claude-mem. Edit content outside the tags. -->
+
+### Jan 29, 2026
+
+| ID | Time | T | Title | Read |
+|----|------|---|-------|------|
+| #251 | 12:19 AM | 🔴 | Fixed Test Coverage workflow missing dependencies | ~264 |
+| #250 | " | 🔴 | Fixed Windows MSRV check shell conflict in security workflow | ~284 |
+</claude-mem-context>

src/CLAUDE.md

@@ -7,6 +7,9 @@
 
 | ID | Time | T | Title | Read |
 |----|------|---|-------|------|
+| #15 | 10:32 PM | 🟣 | Delivered complete keyboard shortcuts system for TUI with configurable bindings, CLI management, and responsive statusline widget | ~1123 |
 | #13 | 10:31 PM | 🟣 | Completed keyboard shortcuts system implementation with full test coverage | ~886 |
+| #8 | 10:28 PM | 🟣 | Implemented CLI keybindings management commands | ~703 |
+| #4 | 10:27 PM | 🟣 | Implemented CLI keybindings management commands | ~724 |
 | #1 | 10:24 PM | 🟣 | Implemented configurable keyboard shortcuts system for TUI | ~613 |
 </claude-mem-context>

src/cli/CLAUDE.md

@@ -0,0 +1,12 @@
+<claude-mem-context>
+# Recent Activity
+
+<!-- This section is auto-generated by claude-mem. Edit content outside the tags. -->
+
+### Jan 29, 2026
+
+| ID | Time | T | Title | Read |
+|----|------|---|-------|------|
+| #240 | 12:13 AM | 🔴 | Fixed sysinfo compatibility issue in test environment | ~220 |
+| #228 | 12:09 AM | 🔴 | Fixed unused import warning in Rust test module | ~154 |
+</claude-mem-context>

src/cli/commands/CLAUDE.md

@@ -7,7 +7,10 @@
 
 | ID | Time | T | Title | Read |
 |----|------|---|-------|------|
+| #15 | 10:32 PM | 🟣 | Delivered complete keyboard shortcuts system for TUI with configurable bindings, CLI management, and responsive statusline widget | ~1123 |
 | #13 | 10:31 PM | 🟣 | Completed keyboard shortcuts system implementation with full test coverage | ~886 |
 | #8 | 10:28 PM | 🟣 | Implemented CLI keybindings management commands | ~703 |
+| #6 | " | 🔴 | Fixed CLI keybindings tests to use direct flag parsing instead of full command | ~497 |
 | #5 | 10:27 PM | 🟣 | Fixed type conversion in keybindings validation conflict detection | ~361 |
+| #4 | " | 🟣 | Implemented CLI keybindings management commands | ~724 |
 </claude-mem-context>

src/cli/commands/config.rs

@@ -223,7 +223,10 @@ async fn execute_reset(force: bool) -> Result<()> {
     }
 
     if !custom_keys.is_empty() {
-        println!("   ✓ Cleared {} custom configuration value(s)", custom_keys.len());
+        println!(
+            "   ✓ Cleared {} custom configuration value(s)",
+            custom_keys.len()
+        );
     } else {
         println!("   No custom configuration to clear");
     }

src/cli/commands/keybindings.rs

@@ -6,7 +6,7 @@ use crate::error::{KeyringError, Result};
 use crate::tui::keybindings::KeyBindingManager;
 use clap::Parser;
 use std::fs;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 use std::process::Command;
 
 #[derive(Parser, Debug)]
@@ -35,10 +35,9 @@ pub async fn manage_keybindings(args: KeybindingsArgs) -> Result<()> {
     // Ensure config directory exists
     if let Some(parent) = config_path.parent() {
         if !parent.exists() {
-            fs::create_dir_all(parent).map_err(|e| KeyringError::IoError(format!(
-                "Failed to create config directory: {}",
-                e
-            )))?;
+            fs::create_dir_all(parent).map_err(|e| {
+                KeyringError::IoError(format!("Failed to create config directory: {}", e))
+            })?;
         }
     }
 
@@ -65,12 +64,15 @@ fn get_config_path() -> PathBuf {
         config_dir.join("open-keyring").join("keybindings.yaml")
     } else {
         let home = std::env::var("HOME").unwrap_or_else(|_| ".".to_string());
-        PathBuf::from(home).join(".config").join("open-keyring").join("keybindings.yaml")
+        PathBuf::from(home)
+            .join(".config")
+            .join("open-keyring")
+            .join("keybindings.yaml")
     }
 }
 
 /// List all keyboard shortcuts
-fn list_keybindings(config_path: &PathBuf) -> Result<()> {
+fn list_keybindings(config_path: &Path) -> Result<()> {
     let manager = KeyBindingManager::new();
     let bindings = manager.all_bindings();
 
@@ -95,7 +97,7 @@ fn list_keybindings(config_path: &PathBuf) -> Result<()> {
 }
 
 /// Validate keybindings configuration
-fn validate_keybindings(config_path: &PathBuf) -> Result<()> {
+fn validate_keybindings(config_path: &Path) -> Result<()> {
     println!("🔍 Validating keybindings configuration...");
     println!("   File: {}", config_path.display());
     println!();
@@ -106,10 +108,8 @@ fn validate_keybindings(config_path: &PathBuf) -> Result<()> {
     }
 
     // Try to parse the file
-    let content = fs::read_to_string(config_path).map_err(|e| KeyringError::IoError(format!(
-        "Failed to read config file: {}",
-        e
-    )))?;
+    let content = fs::read_to_string(config_path)
+        .map_err(|e| KeyringError::IoError(format!("Failed to read config file: {}", e)))?;
 
     match serde_yaml::from_str::<serde_yaml::Value>(&content) {
         Ok(value) => {
@@ -124,11 +124,16 @@ fn validate_keybindings(config_path: &PathBuf) -> Result<()> {
                     if let Some(shortcut_str) = shortcut_val.as_str() {
                         if let Some(existing_action) = seen.get(shortcut_str) {
                             let action_str = action_key.as_str().unwrap_or("?");
-                            println!("⚠️  Conflict: '{}' is used by both '{}' and '{}'",
-                                shortcut_str, existing_action, action_str);
+                            println!(
+                                "⚠️  Conflict: '{}' is used by both '{}' and '{}'",
+                                shortcut_str, existing_action, action_str
+                            );
                             has_conflicts = true;
                         } else {
-                            seen.insert(shortcut_str.to_string(), action_key.as_str().unwrap_or("?").to_string());
+                            seen.insert(
+                                shortcut_str.to_string(),
+                                action_key.as_str().unwrap_or("?").to_string(),
+                            );
                         }
                     }
                 }
@@ -140,16 +145,14 @@ fn validate_keybindings(config_path: &PathBuf) -> Result<()> {
 
             Ok(())
         }
-        Err(e) => {
-            Err(KeyringError::InvalidInput {
-                context: format!("Invalid YAML: {}", e),
-            })
-        }
+        Err(e) => Err(KeyringError::InvalidInput {
+            context: format!("Invalid YAML: {}", e),
+        }),
     }
 }
 
 /// Reset keybindings to defaults
-fn reset_keybindings(config_path: &PathBuf) -> Result<()> {
+fn reset_keybindings(config_path: &Path) -> Result<()> {
     println!("🔄 Resetting keybindings to defaults...");
 
     // Write default configuration
@@ -163,7 +166,7 @@ fn reset_keybindings(config_path: &PathBuf) -> Result<()> {
 }
 
 /// Edit keybindings configuration
-fn edit_keybindings(config_path: &PathBuf) -> Result<()> {
+fn edit_keybindings(config_path: &Path) -> Result<()> {
     // Ensure default config exists
     if !config_path.exists() {
         fs::write(config_path, crate::tui::keybindings::DEFAULT_KEYBINDINGS)
@@ -266,7 +269,7 @@ mod tests {
     fn test_keybindings_args_list() {
         use clap::Parser;
 
-        let args = KeybindingsArgs::parse_from(&["ok", "keybindings", "--list"]);
+        let args = KeybindingsArgs::parse_from(&["ok", "--list"]);
         assert!(args.list);
         assert!(!args.validate);
         assert!(!args.reset);
@@ -277,7 +280,7 @@ mod tests {
     fn test_keybindings_args_validate() {
         use clap::Parser;
 
-        let args = KeybindingsArgs::parse_from(&["ok", "keybindings", "--validate"]);
+        let args = KeybindingsArgs::parse_from(&["ok", "--validate"]);
         assert!(args.validate);
         assert!(!args.list);
     }
@@ -286,7 +289,7 @@ mod tests {
     fn test_keybindings_args_reset() {
         use clap::Parser;
 
-        let args = KeybindingsArgs::parse_from(&["ok", "keybindings", "--reset"]);
+        let args = KeybindingsArgs::parse_from(&["ok", "--reset"]);
         assert!(args.reset);
         assert!(!args.list);
     }
@@ -295,7 +298,7 @@ mod tests {
     fn test_keybindings_args_edit() {
         use clap::Parser;
 
-        let args = KeybindingsArgs::parse_from(&["ok", "keybindings", "--edit"]);
+        let args = KeybindingsArgs::parse_from(&["ok", "--edit"]);
         assert!(args.edit);
         assert!(!args.list);
     }

src/cli/commands/search.rs

@@ -41,9 +41,7 @@ pub async fn search_records(args: SearchArgs) -> Result<()> {
 
     // Apply tags filter (records must have ALL specified tags)
     if !args.tags.is_empty() {
-        records.retain(|r| {
-            args.tags.iter().all(|tag| r.tags.contains(tag))
-        });
+        records.retain(|r| args.tags.iter().all(|tag| r.tags.contains(tag)));
     }
 
     // Apply limit

src/cli/commands/sync.rs

@@ -84,7 +84,11 @@ async fn perform_dry_run(vault: &Vault, sync_dir: &Path) -> Result<()> {
     Ok(())
 }
 
-async fn perform_sync(vault: &mut Vault, sync_dir: &Path, conflict_resolution: ConflictResolution) -> Result<()> {
+async fn perform_sync(
+    vault: &mut Vault,
+    sync_dir: &Path,
+    conflict_resolution: ConflictResolution,
+) -> Result<()> {
     let sync_service = SyncService::new();
 
     println!("🔄 Starting sync...");
@@ -98,14 +102,12 @@ async fn perform_sync(vault: &mut Vault, sync_dir: &Path, conflict_resolution: C
     }
 
     // Import records from sync directory
-    let stats = sync_service.import_from_directory(
-        vault,
-        sync_dir,
-        conflict_resolution,
-    )?;
-
-    println!("   Imported: {}, Updated: {}, Resolved: {}",
-             stats.imported, stats.updated, stats.conflicts);
+    let stats = sync_service.import_from_directory(vault, sync_dir, conflict_resolution)?;
+
+    println!(
+        "   Imported: {}, Updated: {}, Resolved: {}",
+        stats.imported, stats.updated, stats.conflicts
+    );
     println!("✅ Sync completed");
 
     Ok(())

src/cli/commands/update.rs

@@ -42,8 +42,8 @@ pub async fn update_record(args: UpdateArgs) -> Result<()> {
     println!("🔄 Updating record: {}", args.name);
 
     // Parse existing encrypted data as JSON
-    let mut payload: serde_json::Value = serde_json::from_slice(&record.encrypted_data)
-        .map_err(|e| Error::InvalidInput {
+    let mut payload: serde_json::Value =
+        serde_json::from_slice(&record.encrypted_data).map_err(|e| Error::InvalidInput {
             context: format!("Failed to parse record data: {}", e),
         })?;