cleanup

Author: sxlijin

engine/language_server/src/playground/error_page.html

@@ -3,7 +3,6 @@ use std::{collections::HashMap, time::Instant};
 use lsp_types::{
     notification::DidChangeTextDocument, DidChangeTextDocumentParams, PublishDiagnosticsParams,
 };
-
 use playground_server::{FrontendMessage, PreSendToWasmMessage};
 
 use crate::{
@@ -66,8 +65,6 @@ impl SyncNotificationHandler for DidChangeTextDocumentHandler {
             .internal_error()?;
 
         // Broadcast update to playground clients
-        // #[cfg(feature = "playground-server")]
-        // if let Some(state) = &session.playground_state {
         {
             let project = project.lock();
             let files_map: std::collections::HashMap<String, String> = project
@@ -96,14 +93,6 @@ impl SyncNotificationHandler for DidChangeTextDocumentHandler {
                 ))
                 .unwrap();
         }
-        // let root_path = project.root_path().to_string_lossy().to_string();
-        // let state = state.clone();
-        // if let Some(runtime) = &session.playground_runtime {
-        //     runtime.spawn(async move {
-        //         let _ = broadcast_project_update(&state, &root_path, files_map).await;
-        //     });
-        // }
-        // }
 
         tracing::info!("publishing diagnostics");
 

engine/language_server/src/playground/mod.rs

@@ -55,18 +55,11 @@ impl SyncRequestHandler for CodeActionHandler {
             .find(|f| f.span.file_path == document_key.path().to_string_lossy())
             .map(|f| f.name);
 
-        // Get the actual playground port from session (determined by server after availability check)
-        // Fall back to configured port if actual port not set yet
-        #[cfg(feature = "playground-server")]
-        let port = session
-            .get_session_playground_port()
-            .unwrap_or_else(|| session.baml_settings.playground_port.unwrap_or(3030));
-
         let action = CodeActionOrCommand::CodeAction(CodeAction {
-            title: format!("Open Playground Localhost:{port}"),
+            title: "Open Playground".to_string(),
             kind: Some(CodeActionKind::EMPTY),
             command: Some(Command {
-                title: format!("Open Playground Localhost:{port}"),
+                title: "Open Playground".to_string(),
                 command: "openPlayground".to_string(),
                 arguments: function_name.map(|name| vec![Value::String(name)]),
             }),

engine/language_server/src/server/api/notifications/did_change.rs

@@ -4,8 +4,6 @@ use lsp_types::{
     self, request as req, GotoDefinitionParams, GotoDefinitionResponse, Location, Position, Range,
     Url,
 };
-// #[cfg(feature = "playground-server")]
-// use crate::playground::broadcast_function_change;
 use playground_server::{FrontendMessage, PreSendToWasmMessage};
 
 use crate::{
@@ -115,38 +113,19 @@ impl SyncRequestHandler for GotoDefinition {
                     .into_iter()
                     .find(|f| f.span.file_path == document_key.path().to_string_lossy())
                 {
-                    session
-                        .playground_tx
-                        .send(PreSendToWasmMessage::FrontendMessage(
-                            FrontendMessage::select_function {
-                                root_path: guard.root_path().to_string_lossy().to_string(),
-                                function_name: function.name.clone(),
-                            },
-                        ))
-                        .unwrap();
+                    if let Err(e) =
+                        session
+                            .playground_tx
+                            .send(PreSendToWasmMessage::FrontendMessage(
+                                FrontendMessage::select_function {
+                                    root_path: guard.root_path().to_string_lossy().to_string(),
+                                    function_name: function.name.clone(),
+                                },
+                            ))
+                    {
+                        tracing::warn!("Error forwarding function change to playground: {}", e);
+                    }
                 }
-                // #[cfg(feature = "playground-server")]
-                // if let Some(state) = &session.playground_state {
-                //     // Get the first function from the current file if available
-                //     if let Some(function) = guard
-                //         .list_functions()
-                //         .unwrap_or_default()
-                //         .into_iter()
-                //         .find(|f| f.span.file_path == document_key.path().to_string_lossy())
-                //     {
-                //         tracing::info!("Broadcasting function change for: {}", function.name);
-                //         let root_path = guard.root_path().to_string_lossy().to_string();
-                //         let state = state.clone();
-                //         let function_name = function.name.clone();
-                //         if let Some(runtime) = &session.playground_runtime {
-                //             runtime.spawn(async move {
-                //                 let _ =
-                //                     broadcast_function_change(&state, &root_path, function_name)
-                //                         .await;
-                //             });
-                //         }
-                //     }
-                // }
 
                 Ok(Some(goto_definition_response))
             }