chore(workspace): fix bugs, remove dead code, improve performance (#193)

* Comprehensive codebase cleanup: fix bugs, remove dead code, fix performance issues

- Fix watcher tests: add serial_test to prevent race conditions in cortex-skills
- Fix unsafe arithmetic: use saturating_sub/floor_char_boundary in TUI rendering
  to prevent panics on small terminal sizes
- Remove dead code in cortex-tui: delete unused functions (_save_provider_api_key,
  _remove_mcp_server, _get_chat_area, _handle_backend_event, _handle_app_event,
  _handle_subagent_event, _handle_subagent_completed, _handle_subagent_failed,
  _get_remote_server_config, _render_tool_call, _render_subagent,
  _render_chat_with_welcome, _generate_welcome_lines, _generate_message_lines,
  _render_messages_only, calculate_messages_height, render_message,
  render_text_content, render_streaming_content, render_motd_compact,
  render_welcome_text, cmd_mcp_auth, cmd_mcp_logs, is_cjk_char,
  allocate_remaining, remaining_height), remove dead variables
- Remove dead code in cortex-engine: move NoOpDelegate, ChannelDelegate,
  CompositeDelegate, TruncateTransformer, StripAnsiTransformer, NonEmptyFilter,
  TurnBuilder into #[cfg(test)] modules; delete unused ProfileSessionState
- Performance: replace unnecessary Vec clones in config_loader and template
  iteration; replace format!("{}", x) with x.to_string() in 5 locations
- Remove 3 unnecessary clippy lint suppressions from cortex-engine lib.rs

* style: fix formatting issues — remove extra blank lines, fix import ordering

* style: fix remaining formatting issue in login.rs

Author: echobt

.gitignore

@@ -34,3 +34,6 @@ Thumbs.db
 # Backup files
 *.bak
 *.backup
+.cargo-home/
+.sccache/
+.rustup/

Cargo.lock

@@ -382,33 +382,6 @@ fn is_word_break(grapheme: &str) -> bool {
     false
 }
 
-/// Check if a character is a potential word break point (for CJK text).
-///
-/// CJK characters can break anywhere, so we treat them as word boundaries.
-#[allow(dead_code)]
-fn is_cjk_char(c: char) -> bool {
-    matches!(c as u32,
-        // CJK Unified Ideographs
-        0x4E00..=0x9FFF |
-        // CJK Extension A
-        0x3400..=0x4DBF |
-        // CJK Extension B-F (surrogate pairs in UTF-16)
-        0x20000..=0x2A6DF |
-        0x2A700..=0x2B73F |
-        0x2B740..=0x2B81F |
-        0x2B820..=0x2CEAF |
-        0x2CEB0..=0x2EBEF |
-        // CJK Compatibility Ideographs
-        0xF900..=0xFAFF |
-        // Hiragana
-        0x3040..=0x309F |
-        // Katakana
-        0x30A0..=0x30FF |
-        // Hangul Syllables
-        0xAC00..=0xD7AF
-    )
-}
-
 /// Iterator that yields wrapped lines.
 pub struct WrapIterator<'a> {
     remaining: &'a str,

cortex-tui-framework/crates/cortex-tui-text/src/wrap.rs

@@ -15,7 +15,7 @@ pub fn timestamp_now() -> u64 {
 /// Get current timestamp as formatted string for filenames.
 pub fn chrono_timestamp() -> String {
     let now = timestamp_now();
-    format!("{}", now)
+    now.to_string()
 }
 
 /// Estimate available disk space in bytes (platform-specific).