Revert "Merge pull request #5321 from Tyriar/5319"

This reverts commit a260f7d, reversing
changes made to 2042bb8.

Author: Tyriar

package.json

@@ -95,7 +95,7 @@
     "jsdom": "^18.0.1",
     "mocha": "^10.1.0",
     "mustache": "^4.2.0",
-    "node-pty": "^1.1.0-beta31",
+    "node-pty": "1.1.0-beta19",
     "nyc": "^15.1.0",
     "source-map-loader": "^3.0.0",
     "source-map-support": "^0.5.20",

src/common/buffer/Buffer.ts

@@ -320,37 +320,7 @@ export class Buffer implements IBuffer {
     if (toRemove.length > 0) {
       const newLayoutResult = reflowLargerCreateNewLayout(this.lines, toRemove);
       reflowLargerApplyNewLayout(this.lines, newLayoutResult.layout);
-
-      // For conpty, it has its own copy of the buffer _without scrollback_ internally. Its behavior
-      // when reflowing larger is to insert empty lines at the bottom of the buffer as when lines
-      // unwrap conpty's view cannot pull scrollback down, so it adds empty lines at the end.
-      let removedInViewport = 0;
-      const isWindowsMode = this._optionsService.rawOptions.windowsMode || this._optionsService.rawOptions.windowsPty.backend !== undefined || this._optionsService.rawOptions.windowsPty.buildNumber !== undefined;
-      if (isWindowsMode) {
-        for (let i = (toRemove.length / 2) - 1; i >= 0; i--) {
-          if (toRemove[i * 2 + 0] > this.ybase + removedInViewport) {
-            removedInViewport += toRemove[i * 2 + 1];
-          }
-        }
-      }
-
       this._reflowLargerAdjustViewport(newCols, newRows, newLayoutResult.countRemoved);
-
-      // Apply empty lines for any removed in viewport for conpty.
-      if (isWindowsMode) {
-        if (removedInViewport > 0) {
-          for (let i = 0; i < removedInViewport; i++) {
-            // Just add the new missing rows on Windows as conpty reprints the screen with it's
-            // view of the world. Once a line enters scrollback for conpty it remains there
-            this.lines.push(new BufferLine(newCols, this.getNullCell(DEFAULT_ATTR_DATA)));
-          }
-          if (this.ybase === this.ydisp) {
-            this.ydisp += removedInViewport;
-          }
-          this.ybase += removedInViewport;
-          this.y -= removedInViewport;
-        }
-      }
     }
   }
 
@@ -382,7 +352,7 @@ export class Buffer implements IBuffer {
     const nullCell = this.getNullCell(DEFAULT_ATTR_DATA);
     // Gather all BufferLines that need to be inserted into the Buffer here so that they can be
     // batched up and only committed once
-    const toInsert: { start: number, newLines: IBufferLine[] }[] = [];
+    const toInsert = [];
     let countToInsert = 0;
     // Go backwards as many lines may be trimmed and this will avoid considering them
     for (let y = this.lines.length - 1; y >= 0; y--) {
@@ -497,20 +467,6 @@ export class Buffer implements IBuffer {
       this.savedY = Math.min(this.savedY + linesToAdd, this.ybase + newRows - 1);
     }
 
-    // For conpty, it has its own copy of the buffer _without scrollback_ internally. Its behavior
-    // when reflowing smaller is to reflow all lines inside the viewport, and removing empty or
-    // whitespace only lines from the bottom, until non-whitespace is hit in order to prevent
-    // content from being pushed into the scrollback.
-    let addedInViewport = 0;
-    const isWindowsMode = this._optionsService.rawOptions.windowsMode || this._optionsService.rawOptions.windowsPty.backend !== undefined || this._optionsService.rawOptions.windowsPty.buildNumber !== undefined;
-    if (isWindowsMode) {
-      for (let i = toInsert.length - 1; i >= 0; i--) {
-        if (toInsert[i].start > this.ybase + addedInViewport) {
-          addedInViewport += toInsert[i].newLines.length;
-        }
-      }
-    }
-
     // Rearrange lines in the buffer if there are any insertions, this is done at the end rather
     // than earlier so that it's a single O(n) pass through the buffer, instead of O(n^2) from many
     // costly calls to CircularList.splice.
@@ -564,35 +520,6 @@ export class Buffer implements IBuffer {
         this.lines.onTrimEmitter.fire(amountToTrim);
       }
     }
-
-    // Apply empty lines to remove calculated earlier for conpty.
-    if (isWindowsMode) {
-      if (addedInViewport > 0) {
-        let emptyLinesAtBottom = 0;
-        for (let i = this.lines.length - 1; i >= this.ybase + this.y; i--) {
-          const line = this.lines.get(i) as BufferLine;
-          if (line.isWrapped || line.getTrimmedLength() > 0) {
-            break;
-          }
-          emptyLinesAtBottom++;
-        }
-        const emptyLinesToRemove = Math.min(addedInViewport, emptyLinesAtBottom);
-        if (emptyLinesToRemove > 0) {
-          for (let i = 0; i < emptyLinesToRemove; i++) {
-            this.lines.pop();
-          }
-          if (this.ybase === this.ydisp) {
-            this.ydisp -= emptyLinesToRemove;
-          }
-          this.ybase -= emptyLinesToRemove;
-          this.y += emptyLinesToRemove;
-          this.lines.onDeleteEmitter.fire({
-            index: this.lines.length - emptyLinesToRemove,
-            amount: emptyLinesToRemove
-          });
-        }
-      }
-    }
   }
 
   /**

yarn.lock

@@ -3302,10 +3302,10 @@ node-preload@^0.2.1:
   dependencies:
     process-on-spawn "^1.0.0"
 
-node-pty@^1.1.0-beta31:
-  version "1.1.0-beta9"
-  resolved "https://registry.yarnpkg.com/node-pty/-/node-pty-1.1.0-beta9.tgz#ed643cb3b398d031b4e31c216e8f3b0042435f1d"
-  integrity sha512-/Ue38pvXJdgRZ3+me1FgfglLd301GhJN0NStiotdt61tm43N5htUyR/IXOUzOKuNaFmCwIhy6nwb77Ky41LMbw==
+[email protected]beta19:
+  version "1.1.0-beta19"
+  resolved "https://registry.yarnpkg.com/node-pty/-/node-pty-1.1.0-beta19.tgz#a74dc04429903c5ac49ee81a15a24590da67d4f3"
+  integrity sha512-/p4Zu56EYDdXjjaLWzrIlFyrBnND11LQGP0/L6GEVGURfCNkAlHc3Twg/2I4NPxghimHXgvDlwp7Z2GtvDIh8A==
   dependencies:
     node-addon-api "^7.1.0"