Fix #8047 - debugging language server and fix completion handling
[roobuilder] / src / Palete / LanguageClientVala.vala
index fe37445..e0156aa 100644 (file)
@@ -35,7 +35,13 @@ namespace Palete {
                }
                void startServer()
                {
-                       this.initProcess("/usr/bin/vala-language-server");
+                       var exe = GLib.Environment.find_program_in_path( "vala-language-server");
+                       if (exe == null) {
+                               GLib.warning("could not find vala-language-server");
+                                
+                               return;
+                       }
+                       this.initProcess(exe);
                }
                
                
@@ -43,12 +49,17 @@ namespace Palete {
                {
                        // extend versions will proably call initialize to start and connect to server.
                        base(project);
-                       this.open_files = new   Gee.ArrayList<JsRender.JsRender>();
+
                        this.change_queue_id = GLib.Timeout.add_seconds(1, () => {
                                if (this.change_queue_file == null) {
                                        return true;
                                }
+                               if (this.getting_diagnostics) {
+                                       return true;
+                               }
                                this.countdown--;
+
+                       
                                if (this.countdown < 0){
                                        this.document_change_force.begin(this.change_queue_file,  this.change_queue_file_source, (o, res) => {
                                                this.document_change_force.end(res);
@@ -69,11 +80,27 @@ namespace Palete {
                        this.log(LanguageClientAction.LAUNCH, process_path);
                        GLib.debug("Launching %s", process_path);
                        this.launcher = new GLib.SubprocessLauncher (SubprocessFlags.STDIN_PIPE | SubprocessFlags.STDOUT_PIPE);
-                       this.launcher.set_environ(GLib.Environ.get());
+                       var env = GLib.Environ.get();
+                       env += "G_MESSAGES_DEBUG=all";
+
+                       this.launcher.set_environ(env);
+                       var logpath = GLib.Environment.get_home_dir() + "/.cache/vala-language-server";
+                       
+                       if (!GLib.FileUtils.test(logpath, GLib.FileTest.IS_DIR)) {
+                               Posix.mkdir(logpath, 0700);
+                       }
+                       // not very reliable..
+                       //this.launcher.set_stderr_file_path( 
+                       //      logpath + "/" + 
+                       //      (new GLib.DateTime.now_local()).format("%Y-%m-%d") + ".log"
+                       //);
+                       //GLib.debug("log lang server to %s", logpath + "/" + 
+                       //      (new GLib.DateTime.now_local()).format("%Y-%m-%d") + ".log");
+
                        try {
 
                                
-                               this.subprocess = launcher.spawnv ({ process_path });
+                               this.subprocess = launcher.spawnv ({ process_path , "2>" , "/tmp/vala-language-server.log" });
                                
                                this.subprocess.wait_async.begin( null, ( obj,res ) => {
                                        try {
@@ -125,9 +152,10 @@ namespace Palete {
                                 
                                this.jsonrpc_client.failed.connect(() => {
                                        this.log(LanguageClientAction.ERROR_RPC, "client failed");
+                                       GLib.debug("language server server has failed");                                        
                                        this.onClose();
                                        
-                                       GLib.debug("language server server has failed");
+
                                });
 
                                this.initialize_server ();
@@ -151,7 +179,11 @@ namespace Palete {
                                    out return_value
                                );
                                GLib.debug ("LS replied with %s", Json.to_string (Json.gvariant_serialize (return_value), true));
+                               this.open_files = new Gee.ArrayList<JsRender.JsRender>((a,b) => {
+                                       return a.path == b.path;
+                               });
                                this.initialized = true;
+                               this.getting_diagnostics = false;
                                return;
                        } catch (GLib.Error e) {
                                GLib.debug ("LS replied with error %s", e.message);
@@ -167,6 +199,7 @@ namespace Palete {
                        if (this.launcher == null) {
                                return;
                        }
+                       this.getting_diagnostics = false;
                        this.in_close = true;
                        GLib.debug("onClose called");
                        
@@ -201,9 +234,7 @@ namespace Palete {
                public async void restartServer()
                {
                        this.startServer();
-                       foreach(var f in this.open_files) {
-                               this.document_open(f);
-                       }
+                        
                }
        
                public bool isReady()
@@ -241,7 +272,12 @@ namespace Palete {
                {
                        switch (method) {
                                case "textDocument/publishDiagnostics":
-                                       this.onDiagnostic(return_value);
+                                       //GLib.debug("got notification %s : %s",  method , Json.to_string (Json.gvariant_serialize (return_value), true));
+                                        
+                                       GLib.Idle.add(() => {
+                                               this.onDiagnostic(return_value);
+                                               return false;
+                                       });
                                        return;
                                default: 
                                        break;
@@ -251,31 +287,32 @@ namespace Palete {
                        
                }
                
+               bool getting_diagnostics = false;
                /***
                
                */
                public void onDiagnostic(Variant? return_value) 
                {
-
+                       //GLib.debug ("LS replied with %s", Json.to_string (Json.gvariant_serialize (return_value), true));                                     
                        var dg = Json.gobject_deserialize (typeof (Lsp.Diagnostics), Json.gvariant_serialize (return_value)) as Lsp.Diagnostics; 
+                       GLib.debug("got diag for %s", dg.filename);
                        this.log(LanguageClientAction.DIAG, dg.filename);
+                       if (this.project.path == dg.filename) {
+                               this.getting_diagnostics = false;
+                               this.log(LanguageClientAction.DIAG_END, "diagnostics done");
+                               return;
+                       
+                       }
+                       this.getting_diagnostics =true;
                        var f = this.project.getByPath(dg.filename);
                        if (f == null) {
                                //GLib.debug("no file %s", dg.uri);
-                               this.project.updateErrorsforFile(null);
+                               //this.project.updateErrorsforFile(null);
                                return;
                        }
-                       foreach(var v in f.errorsByType.values) {
-                               v.remove_all();
-                       }
-                       foreach(var diag in dg.diagnostics) {
-                               var ce = new CompileError.new_from_diagnostic(f, diag);
-                               if (!f.errorsByType.has_key(ce.category)) {
-                                       f.errorsByType.set(ce.category, new  GLib.ListStore(typeof(CompileError)));
-                               }
-                               f.errorsByType.get(ce.category).append(ce);
-                       }
-                       f.project.updateErrorsforFile(f);
+                       //GLib.debug("got Diagnostics for %s", f.path);
+                       f.updateErrors( dg.diagnostics );
+                        
                        
                }
                
@@ -284,9 +321,11 @@ namespace Palete {
                        if (!this.isReady()) {
                                return;
                        }
-                       if (!this.open_files.contains(file)) {
-                               this.open_files.add(file);
+                       if (this.open_files.contains(file)) {
+                               return;
                        }
+                       this.open_files.add(file);
+                       
                        
                        GLib.debug ("LS sent open");                     
                        try {
@@ -400,12 +439,19 @@ namespace Palete {
 
                public override async void document_change_force (JsRender.JsRender file, string contents)  
        {
+                       
+                       
                        if (!this.isReady()) {
                                return;
                        }
-                            
+                       this.countdown = 9; // not really relivant..
+                       this.change_queue_file = null; // this is more important..
+                       
+                   if (!this.open_files.contains(file)) {
+                                this.document_open(file);
+                       }  
                        
-                       GLib.debug ("LS send change");
+                       GLib.debug ("LS send change %s rev %d", file.path, file.version);
                        var ar = new Json.Array();
                        var obj = new Json.Object();
                        obj.set_string_member("text", contents);