remove terminal reconnect hack (we will fix it properly somehow if still an issue)

Author: williamstein

src/packages/frontend/frame-editors/terminal-editor/connected-terminal.ts

@@ -46,7 +46,7 @@ declare const $: any;
 const SCROLLBACK = 5000;
 const MAX_HISTORY_LENGTH = 100 * SCROLLBACK;
 
-const MAX_DELAY = 15000;
+// const MAX_DELAY = 15000;
 
 const ENABLE_WEBGL = false;
 
@@ -77,9 +77,9 @@ export class Terminal<T extends CodeEditorState = CodeEditorState> {
   private pauseKeyCount: number = 0;
   private keyhandler_initialized: boolean = false;
   // last time user typed something
-  private lastSend = 0;
+  // private lastSend = 0;
   // last time we received data back from project
-  private lastReceive = 0;
+  // private lastReceive = 0;
   /* We initially have to ignore when rendering the initial history.
     To TEST this, do this in a terminal, then reconnect:
          printf "\E[c\n" ; sleep 1 ; echo
@@ -187,7 +187,7 @@ export class Terminal<T extends CodeEditorState = CodeEditorState> {
     this.init_settings();
     this.init_touch();
     this.set_connection_status("disconnected");
-    this.reconnectIfNotResponding();
+    // this.reconnectIfNotResponding();
 
     // The docs https://xtermjs.org/docs/api/terminal/classes/terminal/#resize say
     // "It’s best practice to debounce calls to resize, this will help ensure that
@@ -320,10 +320,10 @@ export class Terminal<T extends CodeEditorState = CodeEditorState> {
         ephemeral: EPHEMERAL,
       });
       this.conn = conn as any;
-      conn.on("close", this.connect);
+      conn.once("close", this.connect);
       conn.on("kick", this.close_request);
       conn.on("data", this.handleDataFromProject);
-      conn.on("init", (data) => {
+      conn.once("init", (data) => {
         this.terminal.clear();
         this.render(data);
       });
@@ -371,19 +371,19 @@ export class Terminal<T extends CodeEditorState = CodeEditorState> {
       return;
     }
     this.conn.write(data);
-    this.lastSend = Date.now();
+    //this.lastSend = Date.now();
   };
 
   // this should never ever be necessary.  It's a just-in-case things
   // were myseriously totally broken measure...
-  private reconnectIfNotResponding = async () => {
-    while (this.state != "closed") {
-      if (this.lastSend - this.lastReceive >= MAX_DELAY) {
-        await this.connect();
-      }
-      await delay(MAX_DELAY / 2);
-    }
-  };
+  //   private reconnectIfNotResponding = async () => {
+  //     while (this.state != "closed") {
+  //       if (this.lastSend - this.lastReceive >= MAX_DELAY) {
+  //         await this.connect();
+  //       }
+  //       await delay(MAX_DELAY / 2);
+  //     }
+  //   };
 
   private handleDataFromProject = (data: any): void => {
     this.assert_not_closed();
@@ -399,7 +399,7 @@ export class Terminal<T extends CodeEditorState = CodeEditorState> {
   };
 
   private activity = () => {
-    this.lastReceive = Date.now();
+    //this.lastReceive = Date.now();
     this.project_actions.flag_file_activity(this.path);
   };