disables using change_id() as it was introducing complications.

moved view population outside of buffer adding, but it is still broken.

main.py

@@ -94,8 +94,10 @@ class CodempBrowseServerCommand(sublime_plugin.WindowCommand):
 
 
 class CodempReplaceTextCommand(sublime_plugin.TextCommand):
-    def run(self, edit, start, end, content, change_id):
+    def run(self, edit, start, end, content, change_id = None):
         # we modify the region to account for any change that happened in the mean time
+        region = sublime.Region(start, end)
+        if change_id:
             region = self.view.transform_region_from(sublime.Region(start, end), change_id)
         self.view.replace(edit, region, content)
 
@@ -155,7 +157,7 @@ class CodempClientViewEventListener(sublime_plugin.ViewEventListener):
             return
 
         vws.send_cursor(vbuff.id, start, end)
-        logger.debug(f"selection modified! {vws.id}, {vbuff.id} - {start}, {end}")
+        # logger.debug(f"selection modified! {vws.id}, {vbuff.id} - {start}, {end}")
 
     def on_activated(self):
         logger.debug(f"'{self.view}' view activated!")

plugin/commands/workspace.py

@@ -7,7 +7,7 @@ from ..core.workspace import workspaces
 from ..core.buffers import buffers
 
 from ..text_listener import TEXT_LISTENER
-from ..utils import safe_listener_attach, safe_listener_detach
+from ..utils import safe_listener_attach, safe_listener_detach, populate_view
 from ..input_handlers import SimpleListInput, SimpleTextInput
 
 logger = logging.getLogger(__name__)
@@ -67,11 +67,11 @@ class CodempJoinBufferCommand(sublime_plugin.WindowCommand):
 
         # now we can defer the attaching process
         logger.debug(f"attempting to attach to {buffer_id}...")
-        promise = vws.handle.attach_buffer(buffer_id)
+        ctl_promise = vws.handle.attach_buffer(buffer_id)
 
         def _():
             try:
-                buff_ctl = promise.wait()
+                buff_ctl = ctl_promise.wait()
                 logger.debug("attach successfull!")
             except Exception as e:
                 logging.error(f"error when attaching to buffer '{id}':\n\n {e}")
@@ -79,8 +79,11 @@ class CodempJoinBufferCommand(sublime_plugin.WindowCommand):
                 return
 
             safe_listener_detach(TEXT_LISTENER)
+            content_promise = buff_ctl.content()
             vbuff = buffers.add(buff_ctl, vws)
 
+            content = content_promise.wait()
+            populate_view(vbuff.view, content)
             if self.window.active_view() == vbuff.view:
                 # if view is already active focusing it won't trigger `on_activate`.
                 safe_listener_attach(TEXT_LISTENER, vbuff.view.buffer())
@@ -115,8 +118,8 @@ class CodempLeaveBufferCommand(sublime_plugin.WindowCommand):
             buffers.lookupId(buffer_id)
             vws = workspaces.lookupId(workspace_id)
         except KeyError:
-            sublime.error_message(f"You are not attached to the buffer '{id}'")
+            sublime.error_message(f"You are not attached to the buffer '{buffer_id}'")
-            logging.warning(f"You are not attached to the buffer '{id}'")
+            logging.warning(f"You are not attached to the buffer '{buffer_id}'")
             return
 
         if not vws.handle.get_buffer(buffer_id):

plugin/core/buffers.py

@@ -7,10 +7,12 @@ if TYPE_CHECKING:
 import sublime
 import os
 import logging
+import threading
 
 from codemp import TextChange
 from .. import globals as g
 from ..utils import populate_view
+from ..utils import get_contents
 from ..utils import safe_listener_attach
 from ..utils import safe_listener_detach
 from ..utils import bidict
@@ -18,9 +20,15 @@ from ..utils import bidict
 logger = logging.getLogger(__name__)
 
 def bind_callback(v: sublime.View):
+    # we need this lock to prevent multiple instance of try_recv() to spin up
+    # which would cause out of order insertion of changes.
+    multi_tryrecv_lock = threading.Lock()
+
     def _callback(bufctl: codemp.BufferController):
         def _():
-            change_id = v.change_id()
+            try:
+                # change_id = v.change_id()
+                change_id = None
                 while buffup := bufctl.try_recv().wait():
                     logger.debug("received remote buffer change!")
                     if buffup is None:
@@ -35,26 +43,30 @@ def bind_callback(v: sublime.View):
                     # to avoid echoing back the change just received.
                     if v == sublime.active_window().active_view():
                         v.settings()[g.CODEMP_IGNORE_NEXT_TEXT_CHANGE] = True
-
-
                     # we need to go through a sublime text command, since the method,
                     # view.replace needs an edit token, that is obtained only when calling
                     # a textcommand associated with a view.
-                try:
+
                     change = buffup.change
                     v.run_command(
                         "codemp_replace_text",
                         {
-                            "start": change.start,
+                            "start": change.start_idx,
-                            "end": change.end,
+                            "end": change.end_idx,
                             "content": change.content,
                             "change_id": change_id,
                         },  # pyright: ignore
                     )
-                except Exception as e:
-                    raise e
 
                     bufctl.ack(buffup.version)
+            except Exception as e:
+                raise e
+            finally:
+                logger.debug("releasing lock")
+                multi_tryrecv_lock.release()
+
+        if multi_tryrecv_lock.acquire(blocking=False):
+            logger.debug("acquiring lock")
             sublime.set_timeout(_)
     return _callback
 
@@ -78,11 +90,11 @@ class BufferManager():
         # sequential indexing, assuming the changes are applied in the order they are received.
         for change in changes:
             region = sublime.Region(change.a.pt, change.b.pt)
-            logger.debug(
+            # logger.debug(
-                "sending txt change: Reg({} {}) -> '{}'".format(
+            #     "sending txt change: Reg({} {}) -> '{}'".format(
-                    region.begin(), region.end(), change.str
+            #         region.begin(), region.end(), change.str
-                )
+            #     )
-            )
+            # )
 
             # we must block and wait the send request to make sure the change went through ok
             self.handle.send(TextChange(start=region.begin(), end=region.end(), content=change.str))
@@ -91,9 +103,14 @@ class BufferManager():
         promise = self.handle.content()
         def _():
             content = promise.wait()
+            current_contents = get_contents(self.view)
+            if content == current_contents:
+                return
+
             safe_listener_detach(text_listener)
             populate_view(self.view, content)
             safe_listener_attach(text_listener, self.view.buffer())
+            sublime.status_message("Syncd contents.")
         sublime.set_timeout_async(_)
 
 class BufferRegistry():
@@ -120,7 +137,6 @@ class BufferRegistry():
         bid = bhandle.path()
         # tmpfile = os.path.join(wsm.rootdir, bid)
         # open(tmpfile, "a").close()
-        content = bhandle.content()
     
         win = sublime.active_window()
         view = win.open_file(bid)
@@ -129,7 +145,6 @@ class BufferRegistry():
         view.settings().set(g.CODEMP_VIEW_TAG, True)
         view.settings().set(g.CODEMP_BUFFER_ID, bid)
         view.set_status(g.SUBLIME_STATUS_ID, "[Codemp]")
-        populate_view(view, content.wait())
 
         tmpfile = "DISABLE"
         bfm = BufferManager(bhandle, view, tmpfile)

plugin/text_listener.py

@@ -1,3 +1,4 @@
+import sublime
 import sublime_plugin
 import logging
 

plugin/utils.py

@@ -88,7 +88,7 @@ def populate_view(view, content):
             "start": 0,
             "end": view.size(),
             "content": content,
-            "change_id": view.change_id(),
+            "change_id": None,
         },
     )