language-servers/tinymist
1
0
Fork 0
mirror of https://github.com/Myriad-Dreamin/tinymist.git synced 2025-07-24 13:13:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

feat: clean up logs (#1473)

Browse source
This commit is contained in:
Myriad-Dreamin 2025-03-10 11:41:41 +08:00 • committed by GitHub
parent 2639dbd49e
commit be1bf802c0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
5 changed files with 14 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

16
crates/sync-lsp/src/lib.rs
View file

@ -297,7 +297,7 @@ impl LspClient {
let mut req_queue = self.req_queue.lock(); let mut req_queue = self.req_queue.lock();
let method = request.method.clone(); let method = request.method.clone();
let req_id = request.id.clone(); let req_id = request.id.clone();
self.start_request(&req_id, &method, received_at); self.start_request(&req_id, &method);
req_queue.incoming.register(req_id, (method, received_at)); req_queue.incoming.register(req_id, (method, received_at));
} }
@ -378,8 +378,8 @@ impl LspClient {
} }
impl LspClient { impl LspClient {
fn start_request(&self, req_id: &RequestId, method: &str, received_at: Instant) { fn start_request(&self, req_id: &RequestId, method: &str) {
log::info!("handling {method} - ({req_id}) at {received_at:0.2?}"); log::info!("handling {method} - ({req_id})");
} }
fn stop_request(&self, req_id: &RequestId, method: &str, received_at: Instant) { fn stop_request(&self, req_id: &RequestId, method: &str, received_at: Instant) {
@ -387,16 +387,16 @@ impl LspClient {
log::info!("handled {method} - ({req_id}) in {duration:0.2?}"); log::info!("handled {method} - ({req_id}) in {duration:0.2?}");
} }
fn start_notification(&self, method: &str, received_at: Instant) { fn start_notification(&self, method: &str) {
log::info!("notifying {method} at {received_at:0.2?}"); log::info!("notifying {method}");
} }
fn stop_notification(&self, method: &str, received_at: Instant, result: LspResult<()>) { fn stop_notification(&self, method: &str, received_at: Instant, result: LspResult<()>) {
let request_duration = received_at.elapsed(); let request_duration = received_at.elapsed();
if let Err(err) = result { if let Err(err) = result {
log::error!("notifying {method} failed in {request_duration:0.2?}: {err:?}"); log::error!("notify {method} failed in {request_duration:0.2?}: {err:?}");
} else { } else {
log::info!("notifying {method} succeeded in {request_duration:0.2?}"); log::info!("notify {method} succeeded in {request_duration:0.2?}");
} }
} }
} }
@ -884,7 +884,7 @@ where
/// Handles an incoming notification. /// Handles an incoming notification.
fn on_notification(&mut self, received_at: Instant, not: Notification) -> anyhow::Result<()> { fn on_notification(&mut self, received_at: Instant, not: Notification) -> anyhow::Result<()> {
self.client.start_notification(&not.method, received_at); self.client.start_notification(&not.method);
let handle = |s, Notification { method, params }: Notification| { let handle = |s, Notification { method, params }: Notification| {
let Some(handler) = self.notifications.get(method.as_str()) else { let Some(handler) = self.notifications.get(method.as_str()) else {
log::warn!("unhandled notification: {method}"); log::warn!("unhandled notification: {method}");

2
crates/tinymist-query/src/analysis.rs
View file

@ -103,7 +103,7 @@ impl LspWorldExt for tinymist_project::LspWorld {
fn uri_for_id(&self, fid: FileId) -> Result<Url, FileError> { fn uri_for_id(&self, fid: FileId) -> Result<Url, FileError> {
let res = path_res_to_url(self.path_for_id(fid)?); let res = path_res_to_url(self.path_for_id(fid)?);
log::info!("uri_for_id: {fid:?} -> {res:?}"); crate::log_debug_ct!("uri_for_id: {fid:?} -> {res:?}");
res.map_err(|err| FileError::Other(Some(eco_format!("convert to url: {err:?}")))) res.map_err(|err| FileError::Other(Some(eco_format!("convert to url: {err:?}"))))
} }

6
crates/tinymist/src/input.rs
View file

@ -15,7 +15,7 @@ use crate::*;
impl ServerState { impl ServerState {
/// Updates a set of source files. /// Updates a set of source files.
fn update_sources(&mut self, files: FileChangeSet) -> Result<()> { fn update_sources(&mut self, files: FileChangeSet) -> Result<()> {
log::info!("update source: {files:?}"); log::trace!("update source: {files:?}");
let intr = Interrupt::Memory(MemoryEvent::Update(files.clone())); let intr = Interrupt::Memory(MemoryEvent::Update(files.clone()));
self.project.interrupt(intr); self.project.interrupt(intr);
@ -25,7 +25,7 @@ impl ServerState {
/// Creates a new source file. /// Creates a new source file.
pub fn create_source(&mut self, path: ImmutPath, content: String) -> Result<()> { pub fn create_source(&mut self, path: ImmutPath, content: String) -> Result<()> {
log::info!("create source: {path:?}"); log::trace!("create source: {path:?}");
self.memory_changes self.memory_changes
.insert(path.clone(), Source::detached(content.clone())); .insert(path.clone(), Source::detached(content.clone()));
@ -40,7 +40,7 @@ impl ServerState {
/// Removes a source file. /// Removes a source file.
pub fn remove_source(&mut self, path: ImmutPath) -> Result<()> { pub fn remove_source(&mut self, path: ImmutPath) -> Result<()> {
self.memory_changes.remove(&path); self.memory_changes.remove(&path);
log::info!("remove source: {path:?}"); log::trace!("remove source: {path:?}");
// todo: is it safe to believe that the path is normalized? // todo: is it safe to believe that the path is normalized?
let files = FileChangeSet::new_removes(vec![path]); let files = FileChangeSet::new_removes(vec![path]);

2
crates/tinymist/src/lsp.rs
View file

@ -80,7 +80,7 @@ impl ServerState {
/// LSP Document Synchronization /// LSP Document Synchronization
impl ServerState { impl ServerState {
pub(crate) fn did_open(&mut self, params: DidOpenTextDocumentParams) -> LspResult<()> { pub(crate) fn did_open(&mut self, params: DidOpenTextDocumentParams) -> LspResult<()> {
log::info!("did open {:?}", params.text_document.uri); log::info!("did open {}", params.text_document.uri);
let path: ImmutPath = as_path_(params.text_document.uri).as_path().into(); let path: ImmutPath = as_path_(params.text_document.uri).as_path().into();
let text = params.text_document.text; let text = params.text_document.text;

2
crates/tinymist/src/main.rs
View file

@ -116,7 +116,7 @@ pub fn lsp_main(args: LspArgs) -> Result<()> {
.map(|e| e.splitn(2, ":").map(|e| e.trim()).collect::<Vec<_>>()) .map(|e| e.splitn(2, ":").map(|e| e.trim()).collect::<Vec<_>>())
.collect::<Vec<_>>(); .collect::<Vec<_>>();
log::info!("tinymist version information: {pairs:?}"); log::info!("tinymist version information: {pairs:?}");
log::info!("starting Language server: {args:#?}"); log::info!("starting language server: {args:?}");
let is_replay = !args.mirror.replay.is_empty(); let is_replay = !args.mirror.replay.is_empty();
with_stdio_transport(args.mirror.clone(), |conn| { with_stdio_transport(args.mirror.clone(), |conn| {