better error handling and read api key from rc only

Author: mohammedahmed18

codeflash/api/cfapi.py

@@ -101,7 +101,7 @@ def get_user_id() -> Optional[str]:
                 console.print(f"[bold red]{msg}[/bold red]")
                 if console.quiet:  # lsp
                     logger.debug(msg)
-                    return None
+                    return f"Error: {msg}"
                 sys.exit(1)
             return userid
 

codeflash/code_utils/env_utils.py

@@ -36,7 +36,7 @@ def check_formatter_installed(formatter_cmds: list[str], exit_on_failure: bool =
 def get_codeflash_api_key() -> str:
     if console.quiet:  # lsp
         # prefer shell config over env var in lsp mode
-        api_key = read_api_key_from_shell_config() or os.environ.get("CODEFLASH_API_KEY")
+        api_key = read_api_key_from_shell_config()
     else:
         api_key = os.environ.get("CODEFLASH_API_KEY") or read_api_key_from_shell_config()
 

codeflash/lsp/beta.py

@@ -130,6 +130,10 @@ def _initialize_optimizer_if_valid(server: CodeflashLanguageServer) -> dict[str,
     if user_id is None:
         return {"status": "error", "message": "api key not found or invalid"}
 
+    if user_id.startswith("Error: "):
+        error_msg = user_id[7:]
+        return {"status": "error", "message": error_msg}
+
     from codeflash.optimization.optimizer import Optimizer
 
     server.optimizer = Optimizer(server.args)