fix: widen token columns and add overflow-safe formatting

Input (8) and Output (9) columns were too narrow for large token counts.
A value like 106529574 (9 digits) got its leading digit clipped in the
8-wide Input column, displaying as 06529574.

- Unify all token columns (Cached, Input, Output, Reasoning) to width 12
  via TOKEN_COL_WIDTH constant, in both daily and session views
- Add format_number_fit() that cascades through progressively compact
  formats (user pref → human-readable → fewer decimals → plain digits)
  so significant digits are never silently clipped
- Separator rows now use the constant for consistency
- Fix clippy::collapsible_match in tui.rs, codex_cli.rs, copilot.rs
- 10 new tests for format_number_fit

Author: mike1858

src/analyzers/codex_cli.rs

@@ -392,45 +392,43 @@ pub(crate) fn parse_codex_cli_jsonl_file(
                                 session_name: effective_name,
                             });
                         }
-                        "assistant" => {
-                            // Token usage is now emitted immediately when processing token_count
-                            // events. We still track assistant messages without additional stats
-                            // to avoid double-counting when Codex emits separate reasoning/tool
-                            // outputs.
-                            if !saw_token_usage {
-                                let model_state = session_model.clone().unwrap_or_else(|| {
-                                    let fallback = SessionModel::inferred(
-                                        DEFAULT_FALLBACK_MODEL.to_string(),
-                                    );
-                                    warn_once(format!(
-                                        "WARNING: session {file_path_str} missing model metadata; using fallback model {} for cost estimation.",
-                                        fallback.name
-                                    ));
-                                    session_model = Some(fallback.clone());
-                                    fallback
-                                });
-
-                                entries.push(ConversationMessage {
-                                    application: Application::CodexCli,
-                                    model: Some(model_state.name.clone()),
-                                    global_hash: hash_text(&format!(
-                                        "{}_{}_assistant_{}",
-                                        file_path_str,
-                                        wrapper.timestamp.to_rfc3339(),
-                                        entries.len()
-                                    )),
-                                    local_hash: None,
-                                    conversation_hash: hash_text(&file_path_str),
-                                    date: wrapper.timestamp,
-                                    project_hash: "".to_string(),
-                                    stats: Stats::default(),
-                                    role: MessageRole::Assistant,
-                                    uuid: None,
-                                    session_name: session_name
-                                        .clone()
-                                        .or_else(|| fallback_session_name.clone()),
-                                });
-                            }
+                        // Token usage is now emitted immediately when processing token_count
+                        // events. We still track assistant messages without additional stats
+                        // to avoid double-counting when Codex emits separate reasoning/tool
+                        // outputs.
+                        "assistant" if !saw_token_usage => {
+                            let model_state = session_model.clone().unwrap_or_else(|| {
+                                let fallback = SessionModel::inferred(
+                                    DEFAULT_FALLBACK_MODEL.to_string(),
+                                );
+                                warn_once(format!(
+                                    "WARNING: session {file_path_str} missing model metadata; using fallback model {} for cost estimation.",
+                                    fallback.name
+                                ));
+                                session_model = Some(fallback.clone());
+                                fallback
+                            });
+
+                            entries.push(ConversationMessage {
+                                application: Application::CodexCli,
+                                model: Some(model_state.name.clone()),
+                                global_hash: hash_text(&format!(
+                                    "{}_{}_assistant_{}",
+                                    file_path_str,
+                                    wrapper.timestamp.to_rfc3339(),
+                                    entries.len()
+                                )),
+                                local_hash: None,
+                                conversation_hash: hash_text(&file_path_str),
+                                date: wrapper.timestamp,
+                                project_hash: "".to_string(),
+                                stats: Stats::default(),
+                                role: MessageRole::Assistant,
+                                uuid: None,
+                                session_name: session_name
+                                    .clone()
+                                    .or_else(|| fallback_session_name.clone()),
+                            });
                         }
                         _ => {}
                     }

src/analyzers/copilot.rs

@@ -149,15 +149,14 @@ fn count_tokens(text: &str) -> u64 {
 // Recursively extract all text content from a nested JSON structure
 fn extract_text_from_value(value: &simd_json::OwnedValue, accumulated_text: &mut String) {
     match value {
-        simd_json::OwnedValue::String(s) => {
-            // Only accumulate if it's a "text" field value, not metadata like URIs
+        // Only accumulate if it's a "text" field value, not metadata like URIs
+        simd_json::OwnedValue::String(s)
             if !s.starts_with("vscode-")
                 && !s.starts_with("file://")
-                && !s.starts_with("ssh-remote")
-            {
-                accumulated_text.push_str(s);
-                accumulated_text.push(' ');
-            }
+                && !s.starts_with("ssh-remote") =>
+        {
+            accumulated_text.push_str(s);
+            accumulated_text.push(' ');
         }
         simd_json::OwnedValue::Object(obj) => {
             // Look for "text" fields specifically