refactor: redirect all output to stderr for LSP compatibility

Move progress/status messages to stderr to keep stdout clean for
LSP protocol communication. This eliminates the need to set quiet
mode in the LSP server startup.

Author: omarabid

codeinput/src/lsp/server.rs

@@ -17,10 +17,7 @@ use url::Url;
 
 use crate::core::cache::sync_cache;
 use crate::core::types::{CodeownersCache, Owner, OwnerType, Tag};
-use crate::utils::{
-    app_config::AppConfig,
-    error::{Error, Result},
-};
+use crate::utils::error::{Error, Result};
 
 /// LSP Server state
 pub struct LspServer {
@@ -468,9 +465,6 @@ fn is_codeowners_file(uri: &Url) -> bool {
 
 /// Run the LSP server
 pub async fn run_lsp_server() -> Result<()> {
-    // Set quiet mode for LSP (suppresses progress output)
-    AppConfig::set("quiet", "true")?;
-
     let stdin = tokio::io::stdin();
     let stdout = tokio::io::stdout();
 

codeinput/src/utils/output.rs

@@ -1,6 +1,8 @@
 //! Output utilities for CLI progress and status messages
+//!
+//! All output goes to stderr to keep stdout clean for data/LSP protocol.
 
-use std::io::Write;
+use std::io::{stderr, Write};
 
 use super::app_config::AppConfig;
 
@@ -9,17 +11,18 @@ fn is_quiet() -> bool {
     AppConfig::fetch().map(|c| c.quiet).unwrap_or(false)
 }
 
-/// Print a message if not in quiet mode
+/// Print a message to stderr if not in quiet mode
 pub fn print(msg: &str) {
     if !is_quiet() {
-        print!("{}", msg);
-        std::io::stdout().flush().ok();
+        let mut stderr = stderr();
+        write!(stderr, "{}", msg).ok();
+        stderr.flush().ok();
     }
 }
 
-/// Print a line if not in quiet mode
+/// Print a line to stderr if not in quiet mode
 pub fn println(msg: &str) {
     if !is_quiet() {
-        println!("{}", msg);
+        writeln!(stderr(), "{}", msg).ok();
     }
 }