vinhnx
diff --git a/Collapse file
‎.vtcode/tool-policy.json‎
Copy file name to clipboard
+5-6Lines changed: 5 additions & 6 deletions b/Collapse file
‎.vtcode/tool-policy.json‎
Copy file name to clipboard
+5-6Lines changed: 5 additions & 6 deletions
diff --git a/Collapse file
‎docs/project/TODO.md‎
Copy file name to clipboard
+1-13Lines changed: 1 addition & 13 deletions
Display the source diff
Display the rich diff b/Collapse file
‎docs/project/TODO.md‎
Copy file name to clipboard
+1-13Lines changed: 1 addition & 13 deletions
Display the source diff
Display the rich diff
diff --git a/Collapse file
‎tests/cli_harness_failures.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎tests/cli_harness_failures.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎vtcode-config/src/constants.rs‎
Copy file name to clipboard
+8Lines changed: 8 additions & 0 deletions b/Collapse file
‎vtcode-config/src/constants.rs‎
Copy file name to clipboard
+8Lines changed: 8 additions & 0 deletions
diff --git a/Collapse file
‎vtcode-core/src/llm/providers/openai.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎vtcode-core/src/llm/providers/openai.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎vtcode-core/src/tools/file_ops.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎vtcode-core/src/tools/file_ops.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎vtcode-core/src/tools/mcp.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎vtcode-core/src/tools/mcp.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎vtcode-core/src/tools/parallel_executor.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎vtcode-core/src/tools/parallel_executor.rs‎
Copy file name to clipboard
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎vtcode-core/src/ui/markdown.rs‎
Copy file name to clipboard
+31-15Lines changed: 31 additions & 15 deletions b/Collapse file
‎vtcode-core/src/ui/markdown.rs‎
Copy file name to clipboard
+31-15Lines changed: 31 additions & 15 deletions
diff --git a/Collapse file
‎vtcode-core/src/ui/tui/log.rs‎
Copy file name to clipboard
+36-3Lines changed: 36 additions & 3 deletions b/Collapse file
‎vtcode-core/src/ui/tui/log.rs‎
Copy file name to clipboard
+36-3Lines changed: 36 additions & 3 deletions
@@ -31,7 +31,7 @@
     "analyze_agent": "allow",
     "apply_patch": "prompt",
     "close_pty_session": "allow",
-    "create_file": "allow",
+    "create_file": "prompt",
     "create_pty_session": "allow",
     "debug_agent": "allow",
     "delete_file": "prompt",
@@ -50,15 +50,14 @@
     "web_fetch": "prompt",
     "write_file": "allow",
     "search_replace": "prompt",
-    "skill": "allow",
+    "skill": "prompt",
     "code_intelligence": "prompt",
+    "mcp_fetch": "prompt",
+    "mcp_fetch_fetch": "prompt",
     "mcp_get_current_time": "prompt",
     "mcp_time_get_current_time": "prompt",
     "mcp_convert_time": "prompt",
-    "mcp_time_convert_time": "prompt",
-    "mcp_fetch": "prompt",
-    "mcp_fetch_fetch": "prompt",
-    "run_pty": "allow"
+    "mcp_time_convert_time": "prompt"
   },
   "constraints": {},
   "mcp": {
 
@@ -1,20 +1,12 @@
 Add LSP (Language Server Protocol) tool for code intelligence features like go-to-definition, find references, and hover documentation
 
--
-
-fix skill allowed-tools not being applied to tools invoked by the skill
-
---
-
-fix a potential crash when syntax highlighting isn't initialized correctly
-
 --
 
 add support for custom session IDs when forking sessions with --session-id combined with --resume or --continue and --fork-session
 
 --
 
-implement input history cycling
+implement input history cycling with up/down arrow keys in the terminal input box
 
 --
 
@@ -24,10 +16,6 @@ fix slow input history cycling and race condition that could overwrite text afte
 
 Reduced terminal flickering and scrolling performance issues for large and long transcripts. improve cpu and memory usage
 
----
-
-add loading indicator when resuming conversations for better feedback
-
 --
 
 add loading indicator when using skills
 
@@ -45,4 +45,3 @@ fn config_override_failure_is_reported() {
             .and(predicate::str::contains(missing_config.to_string_lossy())),
     );
 }
-
@@ -820,6 +820,14 @@ pub mod ui {
     pub const STATUS_LINE_REFRESH_INTERVAL_MS: u64 = 1000;
     pub const STATUS_LINE_COMMAND_TIMEOUT_MS: u64 = 200;
 
+    // TUI tick rate constants for smooth scrolling
+    /// Tick rate (Hz) when user is actively interacting with the TUI
+    pub const TUI_ACTIVE_TICK_RATE_HZ: f64 = 16.0;
+    /// Tick rate (Hz) when TUI is idle to save CPU
+    pub const TUI_IDLE_TICK_RATE_HZ: f64 = 4.0;
+    /// Duration (ms) to remain in active mode after last input
+    pub const TUI_ACTIVE_TIMEOUT_MS: u64 = 500;
+
     // Theme and color constants
     pub const THEME_MIN_CONTRAST_RATIO: f64 = 4.5;
     pub const THEME_FOREGROUND_LIGHTEN_RATIO: f64 = 0.25;
 
@@ -3668,4 +3668,3 @@ mod streaming_tests {
         }
     }
 }
-
@@ -2602,4 +2602,3 @@ mod paging_tests {
         );
     }
 }
-
@@ -87,4 +87,3 @@ impl Tool for McpProxyTool {
         ToolPolicy::Prompt
     }
 }
-
@@ -351,4 +351,3 @@ mod tests {
         assert_ne!(read_group.unwrap().group_id, write_group.unwrap().group_id);
     }
 }
-
@@ -149,15 +149,26 @@ fn heading_level_from_u8(level: u8) -> HeadingLevel {
     }
 }
 
-static SYNTAX_SET: Lazy<SyntaxSet> = Lazy::new(SyntaxSet::load_defaults_newlines);
+static SYNTAX_SET: Lazy<SyntaxSet> = Lazy::new(|| {
+    // Attempt to load default syntaxes, but provide empty fallback if it fails
+    SyntaxSet::load_defaults_newlines()
+});
 static THEME_CACHE: Lazy<parking_lot::RwLock<HashMap<String, Theme>>> = Lazy::new(|| {
-    let defaults = ThemeSet::load_defaults();
-    let mut entries: Vec<(String, Theme)> = defaults.themes.into_iter().collect();
-    if entries.len() > MAX_THEME_CACHE_SIZE {
-        entries.truncate(MAX_THEME_CACHE_SIZE);
+    match ThemeSet::load_defaults() {
+        defaults if !defaults.themes.is_empty() => {
+            let mut entries: Vec<(String, Theme)> = defaults.themes.into_iter().collect();
+            if entries.len() > MAX_THEME_CACHE_SIZE {
+                entries.truncate(MAX_THEME_CACHE_SIZE);
+            }
+            let themes: HashMap<_, _> = entries.into_iter().collect();
+            parking_lot::RwLock::new(themes)
+        }
+        _ => {
+            // If theme loading fails, create empty cache and log warning
+            warn!("Failed to load default syntax highlighting themes; syntax highlighting will be disabled");
+            parking_lot::RwLock::new(HashMap::new())
+        }
     }
-    let themes: HashMap<_, _> = entries.into_iter().collect();
-    parking_lot::RwLock::new(themes)
 });
 
 /// A styled text segment.
@@ -1196,15 +1207,20 @@ fn load_theme(theme_name: &str, cache: bool) -> Theme {
         theme
     } else {
         warn!(
-            "theme" = theme_name,
-            "Falling back to default syntax highlighting theme"
+            theme = theme_name,
+            "Unknown syntax highlighting theme, falling back to first available theme"
         );
-        defaults
-            .themes
-            .into_iter()
-            .next()
-            .map(|(_, theme)| theme)
-            .unwrap_or_default()
+        if defaults.themes.is_empty() {
+            warn!("No syntax highlighting themes available at all");
+            Theme::default()
+        } else {
+            defaults
+                .themes
+                .into_iter()
+                .next()
+                .map(|(_, theme)| theme)
+                .unwrap_or_default()
+        }
     }
 }
 
 
@@ -10,6 +10,7 @@ use ratatui::{
 };
 use tokio::sync::mpsc::UnboundedSender;
 use tracing::{Event, Level, Subscriber, field::Visit};
+use tracing::warn;
 use tracing_subscriber::layer::{Context, Layer};
 use tracing_subscriber::registry::LookupSpan;
 use tui_syntax_highlight::{
@@ -53,8 +54,23 @@ impl LogForwarder {
 static LOG_FORWARDER: Lazy<Arc<LogForwarder>> = Lazy::new(|| Arc::new(LogForwarder::default()));
 
 const DEFAULT_THEME_NAME: &str = "base16-ocean.dark";
-static LOG_SYNTAX_SET: Lazy<SyntaxSet> = Lazy::new(SyntaxSet::load_defaults_newlines);
-static LOG_THEME_SET: Lazy<ThemeSet> = Lazy::new(ThemeSet::load_defaults);
+static LOG_SYNTAX_SET: Lazy<SyntaxSet> = Lazy::new(|| {
+    // Attempt to load default syntaxes, but provide empty fallback if it fails
+    let syntax_set = SyntaxSet::load_defaults_newlines();
+    if syntax_set.syntaxes().is_empty() {
+        warn!("Failed to load syntax set for log highlighting; log syntax highlighting disabled");
+    }
+    syntax_set
+});
+static LOG_THEME_SET: Lazy<ThemeSet> = Lazy::new(|| {
+    match ThemeSet::load_defaults() {
+        theme_set if !theme_set.themes.is_empty() => theme_set,
+        _ => {
+            warn!("Failed to load theme set for log highlighting; using empty theme set");
+            ThemeSet::new()
+        }
+    }
+});
 static LOG_THEME_NAME: Lazy<RwLock<Option<String>>> = Lazy::new(|| RwLock::new(None));
 static LOG_HIGHLIGHTER_CACHE: Lazy<RwLock<Option<(String, Highlighter)>>> =
     Lazy::new(|| RwLock::new(None));
@@ -186,6 +202,13 @@ pub fn set_log_theme_name(theme: Option<String>) {
 
 fn resolve_theme(theme_name: Option<String>) -> Theme {
     let name = theme_name.unwrap_or_else(|| DEFAULT_THEME_NAME.to_string());
+    
+    // If themes are empty, return default theme early
+    if LOG_THEME_SET.themes.is_empty() {
+        warn!("No log highlighting themes available");
+        return Theme::default();
+    }
+    
     if let Some(theme) = LOG_THEME_SET.themes.get(&name) {
         return theme.clone();
     }
@@ -197,7 +220,10 @@ fn resolve_theme(theme_name: Option<String>) -> Theme {
         .values()
         .next()
         .cloned()
-        .unwrap_or_default()
+        .unwrap_or_else(|| {
+            warn!("No themes available in LOG_THEME_SET despite non-empty check");
+            Theme::default()
+        })
 }
 
 fn highlighter_for_current_theme() -> Highlighter {
@@ -287,6 +313,13 @@ fn select_syntax(message: &str) -> &'static SyntaxReference {
 }
 
 pub fn highlight_log_entry(entry: &LogEntry) -> Text<'static> {
+    // If syntax set is empty, skip highlighting entirely
+    if LOG_SYNTAX_SET.syntaxes().is_empty() {
+        let mut text = Text::raw(entry.formatted.as_ref().to_string());
+        prepend_metadata(&mut text, entry);
+        return text;
+    }
+    
     let syntax = select_syntax(entry.message.as_ref());
     let mut highlighted = highlighter_for_current_theme()
         .highlight_lines(entry.message.lines(), syntax, &LOG_SYNTAX_SET)
Original file line number	Diff line number	Diff line change
`@@ -45,4 +45,3 @@ fn config_override_failure_is_reported() {`
`45`	`45`	`.and(predicate::str::contains(missing_config.to_string_lossy())),`
`46`	`46`	`);`
`47`	`47`	`}`
`48`		`-`
Original file line number	Diff line number	Diff line change
`@@ -3668,4 +3668,3 @@ mod streaming_tests {`
`3668`	`3668`	`}`
`3669`	`3669`	`}`
`3670`	`3670`	`}`
`3671`		`-`
Original file line number	Diff line number	Diff line change
`@@ -2602,4 +2602,3 @@ mod paging_tests {`
`2602`	`2602`	`);`
`2603`	`2603`	`}`
`2604`	`2604`	`}`
`2605`		`-`
Original file line number	Diff line number	Diff line change
`@@ -87,4 +87,3 @@ impl Tool for McpProxyTool {`
`87`	`87`	`ToolPolicy::Prompt`
`88`	`88`	`}`
`89`	`89`	`}`
`90`		`-`
Original file line number	Diff line number	Diff line change
`@@ -351,4 +351,3 @@ mod tests {`
`351`	`351`	`assert_ne!(read_group.unwrap().group_id, write_group.unwrap().group_id);`
`352`	`352`	`}`
`353`	`353`	`}`
`354`		`-`