fix(browser): place imports before module const in TimeBasedDebouncer

RENDER_DEBOUNCE_THRESHOLD_MS was declared above the import, which breaks ES module conventions and can confuse tooling. Move the import to the top of the module after the license header. Co-authored-by: Cursor <cursoragent@cursor.com>
fix(browser): clear idle blink timer when handle is 0
2026-06-01 15:28:12 +00:00 · 2026-06-01 15:28:12 +00:00 · 2026-06-01 15:28:12 +00:00 · 2026-06-01 15:28:11 +00:00 · 2026-06-01 15:27:58 +00:00 · 2026-06-01 15:27:58 +00:00
62 changed files with 352 additions and 852 deletions
@@ -39,7 +39,7 @@ The preferred way to report bugs or request features is to use

 It's pretty common for maintainers of large open source projects to suffer from burnout, especially when needing to triage a large number of incoming issues instead of actually building things. Here are some of the steps we take to try mitigate this:

- Support questions live in [GH discussions](https://github.com/xtermjs/xterm.js/discussions), issues may be transferred there without further comment and core maintainers may or may not participate in discussions.
+- Support questions live in [GH discussions](https://github.com/xtermjs/xterm.js/discussions), issues may be transfered there without further comment and core maintainers may or may not participate in discussions.
 - Issues are strictly for well defined features or bugs that are _actionable_.
 - Sometimes features are out of scope. A common example of this is a niche feature that the pricipal implementation ([VS Code](https://code.visualstudio.com/)) won't leverage and therefore would be difficult to maintain and likely suffer from bitrot. The reporter may not agree with this, but you could always create an addon if that works or maintain your own fork if it comes to that.
 - If a feature does not have a clear way forward or needs more discussion it may be closed ro moved to a discussion.
@@ -15,7 +15,7 @@ test.beforeAll(async ({ browser }) => {
 });
 test.afterAll(async () => await ctx.page.close());

-test.describe('AttachAddon', () => {
+test.describe('Search Tests', () => {

  test.beforeEach(async () => {
    await ctx.proxy.reset();
@@ -82,7 +82,7 @@ declare module '@xterm/addon-clipboard' {
  /**
   * The clipboard provider interface that enables xterm.js to access the browser clipboard.
   *
-   * Note that this clipboard provider does not implement own cut buffers as xterm does,
+   * Note this this clipboard provider does not implement own cut buffers as xterm does,
   * but redirect the selection parameter always to navigator.clipboard.
   */
  export class BrowserClipboardProvider implements IClipboardProvider{
@@ -90,7 +90,7 @@ terminal.loadAddon(imageAddon);
  therefore palette animations cannot be used.

 - **SIXEL Raster Attributes Handling**  
-  If raster attributes were found in the SIXEL data (level 2), the image will always be truncated to the given height/width extent. We deviate here from the specification on purpose, as it allows several processing optimizations. For level 1 SIXEL data without any raster attributes the image can freely grow in width and height up to the last data byte, which has a much higher processing penalty. In general encoding libraries should not create level 1 data anymore and should not produce pixel information beyond the announced height/width extent. Both is discouraged by the >30 years old specification.
+  If raster attributes were found in the SIXEL data (level 2), the image will always be truncated to the given height/width extend. We deviate here from the specification on purpose, as it allows several processing optimizations. For level 1 SIXEL data without any raster attributes the image can freely grow in width and height up to the last data byte, which has a much higher processing penalty. In general encoding libraries should not create level 1 data anymore and should not produce pixel information beyond the announced height/width extend. Both is discouraged by the >30 years old specification.

  Currently the SIXEL implementation of the addon does not take custom pixel sizes into account, a SIXEL pixel will map 1:1 to a screen pixel.

@@ -149,9 +149,9 @@ The addon provides the following API endpoints to retrieve raw image data as can

 ### Memory Usage

-The addon does most image processing in JavaScript and therefore can occupy a rather big amount of memory. To get an idea where the memory gets eaten, let's look at the data flow and processing steps:
- Incoming image data chunk at `term.write` (terminal)  
-  `term.write` might stack up incoming chunks. To circumvent this, you will need proper flow control (see xterm.js docs). Note that with image output it is more likely to run into this issue, as it can create lots of MBs in very short time.
+The addon does most image processing in Javascript and therefore can occupy a rather big amount of memory. To get an idea where the memory gets eaten, lets look at the data flow and processing steps:
+- Incomming image data chunk at `term.write` (terminal)  
+  `term.write` might stock up incoming chunks. To circumvent this, you will need proper flow control (see xterm.js docs). Note that with image output it is more likely to run into this issue, as it can create lots of MBs in very short time.
 - Sequence Parser (terminal)  
  The parser operates on a buffer containing up to 2^17 codepoints (~0.5 MB).
 - Sequence Handler - Chunk Decoding (addon)  
@@ -177,18 +177,18 @@ const totalActive = storageBytes + decodingBytes;
 ```

 Note that browsers have offloading tricks for rarely touched memory segments, esp. `storageBytes` might not directly translate into real memory usage. Usage peaks will happen during active decoding of multiple big images due to the need of 2 full pixel buffers at the same time, which cannot be offloaded. Thus you may want to keep an eye on `pixelLimit` under limited memory conditions.  
-Further note that the formulas above do not respect the JavaScript object's overhead. Compared to the raw buffer needs the bookkeeping by these objects is rather small (<<5%).
+Further note that the formulas above do not respect the Javascript object's overhead. Compared to the raw buffer needs the book keeping by these objects is rather small (<<5%).

 _Why should I care about memory usage at all?_  
-Well you don't have to, and it probably will just work fine with the addon defaults. But for bigger integrations, where much more data is held in the JavaScript context (like multiple terminals on one page), it is likely to hit the engine's memory limit sooner or later under decoding and/or storage pressure.
+Well you don't have to, and it probably will just work fine with the addon defaults. But for bigger integrations, where much more data is held in the Javascript context (like multiple terminals on one page), it is likely to hit the engine's memory limit sooner or later under decoding and/or storage pressure.

 _How can I adjust the memory usage?_  
 - `pixelLimit`  
-  A constructor setting, thus you would have to anticipate, whether (multiple) terminals in your page gonna do lots of concurrent decoding. Since this is normally not the case and the memory usage only peaks temporarily, a rather high value should work even with multiple terminals in one page.
+  A constructor setting, thus you would have to anticipate, whether (multiple) terminals in your page gonna do lots of concurrent decoding. Since this is normally not the case and the memory usage is only temporarily peaking, a rather high value should work even with multiple terminals in one page.
 - `storageLimit`  
-  A constructor and a runtime setting. In conjunction with `storageUsage` you can do runtime checks and adjust the limit to your needs. If you have to lower the limit below the current usage, images will be removed in FIFO order and may turn into a placeholder in the terminal's scrollback (if `showPlaceholder` is set). When adjusting keep in mind to leave enough room for peak memory use during decoding.
+  A constructor and a runtime setting. In conjunction with `storageUsage` you can do runtime checks and adjust the limit to your needs. If you have to lower the limit below the current usage, images will be removed in FIFO order and may turn into a placeholder in the terminal's scrollback (if `showPlaceholder` is set). When adjusting keep in mind to leave enough room for memory peaking for decoding.
 - `sixelSizeLimit | iipSizeLimit | kittySizeLimit`  
-  Constructor settings. This has only a small impact on peak memory during decoding. It is meant to avoid processing of overly big or broken image sequences at an earlier phase, thus may stop the invoked decoders from entering memory intensive tasks for potentially invalid data.
+  Constructor settings. This has only a small impact on peaking memory during decoding. It is meant to avoid processing of overly big or broken image sequences at an earlier phase, thus may stop the invoked decoders from entering memory intensive tasks for potentially invalid data.


 ### Terminal Interaction
@@ -21,7 +21,7 @@ const CACHE_SIZE = 100000;

 /**
 * Enable ligature support for the provided Terminal instance. To function
- * properly, this must be called after `open()` is called on the terminal. If
+ * properly, this must be called after `open()` is called on the therminal. If
 * the font currently in use supports ligatures, the terminal will automatically
 * start to render them.
 * @param term Terminal instance from xterm.js
@@ -1,81 +0,0 @@
-/**
- * Copyright (c) 2026 The xterm.js authors. All rights reserved.
- * @license MIT
- */
-
-import { assert } from 'chai';
-import { DecorationManager } from './DecorationManager';
-import { SearchEngine } from './SearchEngine';
-import { SearchLineCache } from './SearchLineCache';
-import { Terminal } from 'browser/public/Terminal';
-import type { ISearchDecorationOptions } from '@xterm/addon-search';
-import type { IDecorationOptions } from '@xterm/xterm';
-import { DisposableStore } from 'common/Lifecycle';
-
-function writeP(terminal: Terminal, data: string): Promise<void> {
-  return new Promise(r => terminal.write(data, r));
-}
-
-describe('DecorationManager', () => {
-  let store: DisposableStore;
-  let terminal: Terminal;
-  let decorationManager: DecorationManager;
-
-  beforeEach(() => {
-    store = new DisposableStore();
-    terminal = store.add(new Terminal({ cols: 10, rows: 5 }));
-    decorationManager = store.add(new DecorationManager(terminal));
-  });
-
-  it('should split highlight decorations for a wrapped match', async () => {
-    await writeP(terminal, '0123456789abcde');
-    const searchEngine = new SearchEngine(terminal, store.add(new SearchLineCache(terminal)));
-    const match = searchEngine.find('9abc', 0, 0);
-    assert.ok(match);
-
-    const decorationOptions: IDecorationOptions[] = [];
-    const registerDecoration = terminal.registerDecoration.bind(terminal);
-    terminal.registerDecoration = (options: IDecorationOptions) => {
-      decorationOptions.push(options);
-      return registerDecoration(options);
-    };
-
-    const options: ISearchDecorationOptions = {
-      matchOverviewRuler: '#ff0000',
-      activeMatchColorOverviewRuler: '#00ff00'
-    };
-    decorationManager.createHighlightDecorations([match], options);
-
-    assert.strictEqual(decorationOptions.length, 2);
-    assert.strictEqual(decorationOptions[0].x, 9);
-    assert.strictEqual(decorationOptions[0].width, 1);
-    assert.strictEqual(decorationOptions[1].x, 0);
-    assert.strictEqual(decorationOptions[1].width, 3);
-
-    const withOverviewRuler = decorationOptions.filter(o => o.overviewRulerOptions !== undefined);
-    assert.strictEqual(withOverviewRuler.length, 2);
-  });
-
-  it('should only add one overview ruler marker per buffer line', async () => {
-    await writeP(terminal, 'abcdefghij');
-    const decorationOptions: IDecorationOptions[] = [];
-    const registerDecoration = terminal.registerDecoration.bind(terminal);
-    terminal.registerDecoration = (options: IDecorationOptions) => {
-      decorationOptions.push(options);
-      return registerDecoration(options);
-    };
-
-    const options: ISearchDecorationOptions = {
-      matchOverviewRuler: '#ff0000',
-      activeMatchColorOverviewRuler: '#00ff00'
-    };
-    decorationManager.createHighlightDecorations([
-      { term: 'a', col: 0, row: 0, size: 1 },
-      { term: 'f', col: 5, row: 0, size: 1 }
-    ], options);
-
-    const withOverviewRuler = decorationOptions.filter(o => o.overviewRulerOptions !== undefined);
-    assert.strictEqual(withOverviewRuler.length, 1);
-    assert.strictEqual(withOverviewRuler[0].x, 0);
-  });
-});
@@ -60,7 +60,7 @@ export class SearchEngine {
      this._terminal.clearSelection();
      return undefined;
    }
-    if (startCol >= this._terminal.cols) {
+    if (startCol > this._terminal.cols) {
      throw new Error(`Invalid col: ${startCol} to search in terminal of ${this._terminal.cols} cols`);
    }

@@ -417,7 +417,7 @@ test.describe('Search Tests', () => {
    test.describe('#2444 wrapped line content not being found', () => {
      let fixture: string;
      test.beforeAll(async () => {
-        fixture = (await new Promise<Buffer>((res, rej) => readFile(resolve(__dirname, '../fixtures/issue-2444'), (err, data) => err ? rej(err) : res(data)))).toString();
+        fixture = (await new Promise<Buffer>(r => readFile(resolve(__dirname, '../fixtures/issue-2444'), (err, data) => r(data)))).toString();
        if (process.platform !== 'win32') {
          fixture = fixture.replace(/\n/g, '\n\r');
        }
@@ -501,7 +501,7 @@ test.describe('Search Tests', () => {
        window.search.onDidChangeResults(e => window.calls.push(e));
      `);
      // Move cursor forward 1 time to create a null character, as opposed to regular whitespace
-      await ctx.proxy.write('\x1b[CHi Hi');
+      await ctx.proxy.write('\\x1b[CHi Hi');
      strictEqual(await ctx.page.evaluate(`window.search.findPrevious('h', { decorations: { activeMatchColorOverviewRuler: '#ff0000' } })`), true);
      deepStrictEqual(await ctx.page.evaluate('window.calls'), [
        { resultCount: 2, resultIndex: 1 }
@@ -173,7 +173,7 @@ class StringSerializeHandler extends BaseSerializeHandler {
  private _thisRowLastSecondChar: IBufferCell = this._buffer.getNullCell();
  private _nextRowFirstChar: IBufferCell = this._buffer.getNullCell();
  protected _rowEnd(row: number, isLastRow: boolean): void {
-    // if there is colorful empty cell at line end, we must pad it back, or the color block
+    // if there is colorful empty cell at line end, whe must pad it back, or the the color block
    // will missing
    if (this._nullCellCount > 0 && !equalBg(this._cursorStyle, this._backgroundCell)) {
      // use clear right to set background.
@@ -221,7 +221,7 @@ class StringSerializeHandler extends BaseSerializeHandler {
            // you can't use control sequence to move cursor to (x === row)
            (thisRowLastChar.getChars() || thisRowLastChar.getWidth() === 0) &&
            // change background of the first wrapped cell also affects BCE
-            // so we mark it as invalid to simplify the process to determine line separator
+            // so we mark it as invalid to simply the process to determine line separator
            equalBg(thisRowLastChar, nextRowFirstChar)
          ) {
            isValid = true;
@@ -233,7 +233,7 @@ class StringSerializeHandler extends BaseSerializeHandler {
            isNextRowFirstCharDoubleWidth &&
            (thisRowLastSecondChar.getChars() || thisRowLastSecondChar.getWidth() === 0) &&
            // change background of the first wrapped cell also affects BCE
-            // so we mark it as invalid to simplify the process to determine line separator
+            // so we mark it as invalid to simply the process to determine line separator
            equalBg(thisRowLastChar, nextRowFirstChar) &&
            equalBg(thisRowLastSecondChar, nextRowFirstChar)
          ) {
@@ -192,7 +192,7 @@ test.describe('SerializeAddon', () => {
    const cols = 10;
    const line = '+'.repeat(cols);
    const lines: string[] = [
-      sgr(FG_P16_GREEN) + line,  // Workaround: If we clear all flags at the end, serialize will use \x1b[0m to clear instead of the specific disable sequence
+      sgr(FG_P16_GREEN) + line,  // Workaround: If we clear all flags a the end, serialize will use \x1b[0m to clear instead of the sepcific disable sequence
      sgr(INVERSE) + line,
      sgr(BOLD) + line,
      sgr(UNDERLINED) + line,
@@ -2,7 +2,7 @@
 * Copyright (c) 2023 The xterm.js authors. All rights reserved.
 * @license MIT
 *
- * UnicodeVersionProvider for V15 with grapheme cluster handling.
+ * UnicodeVersionProvider for V15 with grapeme cluster handleing.
 */

 import type { Terminal, ITerminalAddon, IUnicodeHandling } from '@xterm/xterm';
@@ -57,8 +57,7 @@ function isUrl(urlString: string): boolean {

 export class LinkComputer {
  public static computeLink(y: number, regex: RegExp, terminal: Terminal, activate: (event: MouseEvent, uri: string) => void): ILink[] {
-    const flags = regex.flags.includes('g') ? regex.flags : `${regex.flags}g`;
-    const rex = new RegExp(regex.source, flags);
+    const rex = new RegExp(regex.source, (regex.flags || '') + 'g');

    const [lines, startLineIndex] = LinkComputer._getWindowedLineStrings(y - 1, terminal);
    const line = lines.join('');
@@ -7,13 +7,13 @@ import type { Terminal, ITerminalAddon, IDisposable } from '@xterm/xterm';
 import type { WebLinksAddon as IWebLinksApi } from '@xterm/addon-web-links';
 import { ILinkProviderOptions, WebLinkProvider } from './WebLinkProvider';

-// consider everything starting with http:// or https://
+// consider everthing starting with http:// or https://
 // up to first whitespace, `"` or `'` as url
 // NOTE: The repeated end clause is needed to not match a dangling `:`
 // resembling the old (...)*([^:"\'\\s]) final path clause
 // additionally exclude early + final:
 // - unsafe from rfc3986: !*'()
-// - unsafe chars from rfc1738: {}|\^~[]` (minus [] for ipv6 addresses, also allow ~)
+// - unsafe chars from rfc1738: {}|\^~[]` (minus [] as we need them for ipv6 adresses, also allow ~)
 // also exclude as finals:
 // - final interpunction like ,.!?
 // - any sort of brackets <>()[]{} (not spec conform, but often used to enclose urls)
@@ -23,7 +23,7 @@ See the full [API](https://github.com/xtermjs/xterm.js/blob/master/addons/addon-

 ### Handling Context Loss

-The browser may drop WebGL contexts for various reasons like OOM or after the system has been suspended. There is an API exposed that fires the `webglcontextlost` event on the canvas so embedders can handle it however they wish. An easy, but suboptimal way, to handle this is by disposing of WebglAddon when the event fires:
+The browser may drop WebGL contexts for various reasons like OOM or after the system has been suspended. There is an API exposed that fires the `webglcontextlost` event fired on the canvas so embedders can handle it however they wish. An easy, but suboptimal way, to handle this is by disposing of WebglAddon when the event fires:

 ```ts
 const terminal = new Terminal();
@@ -220,7 +220,7 @@ export class GlyphRenderer extends Disposable {
  public updateCell(x: number, y: number, code: number, bg: number, fg: number, ext: number, chars: string, width: number, lastBg: number): void {
    // Since this function is called for every cell (`rows*cols`), it must be very optimized. It
    // should not instantiate any variables unless a new glyph is drawn to the cache where the
-    // slight slowdown is acceptable for the developer ergonomics provided as it's a one-off for
+    // slight slowdown is acceptable for the developer ergonomics provided as it's a once of for
    // each glyph.
    this._updateCell(this._vertices.attributes, x, y, code, bg, fg, ext, chars, width, lastBg);
  }
@@ -69,7 +69,7 @@ export class TextureAtlas implements ITextureAtlas {
  private _overflowSizePage: AtlasPage | undefined;

  private _tmpCanvas: HTMLCanvasElement;
-  // A temporary context that glyphs are drawn to before being transferred to the atlas.
+  // A temporary context that glyphs are drawn to before being transfered to the atlas.
  private _tmpCtx: CanvasRenderingContext2D;

  private _workBoundingBox: IBoundingBox = { top: 0, left: 0, bottom: 0, right: 0 };
@@ -152,7 +152,7 @@ export class TextureAtlas implements ITextureAtlas {
  }

  private _createNewPage(): AtlasPage {
-    // Try merge the set of the 4 most used pages of the largest size. This is deferred to a
+    // Try merge the set of the 4 most used pages of the largest size. This is is deferred to a
    // microtask to ensure it does not interrupt textures that will be rendered in the current
    // animation frame which would result in blank rendered areas. This is actually not that
    // expensive relative to drawing the glyphs, so there is no need to wait for an idle callback.
@@ -191,7 +191,7 @@ export class TextureAtlas implements ITextureAtlas {
        return newPage;
      }

-      const sortedMergingPagesIndexes = mergingPages.map(e => e.glyphs[0].texturePage).sort((a, b) => a - b);
+      const sortedMergingPagesIndexes = mergingPages.map(e => e.glyphs[0].texturePage).sort((a, b) => a > b ? 1 : -1);
      const mergedPageIndex = this.pages.length - mergingPages.length;

      // Merge into the new page
@@ -1125,7 +1125,7 @@ function clearColor(imageData: ImageData, bg: IColor, fg: IColor, enableThreshol
  // were covered (fg=#8ae234, bg=#c4a000).
  const threshold = Math.floor((Math.abs(r - fgR) + Math.abs(g - fgG) + Math.abs(b - fgB)) / 12);

-  // Set alpha channel of relevant pixels to 0
+  // Set alpha channel of relevent pixels to 0
  let isEmpty = true;
  for (let offset = 0; offset < imageData.data.length; offset += 4) {
    // Check exact match
@@ -40,7 +40,7 @@ export class WebglAddon extends Disposable implements ITerminalAddon, IWebglApi
      };
      const gl = document.createElement('canvas').getContext('webgl2', contextAttributes) as IWebGL2RenderingContext;
      if (!gl) {
-        throw new Error('WebGL2 is only supported on Safari 16 and above');
+        throw new Error('Webgl2 is only supported on Safari 16 and above');
      }
    }
    super();
@@ -664,10 +664,10 @@ export class WebglRenderer extends Disposable implements IRenderer {
    this.dimensions.device.canvas.height = this._terminal.rows * this.dimensions.device.cell.height;
    this.dimensions.device.canvas.width = this._terminal.cols * this.dimensions.device.cell.width;

-    // The size of the canvas on the page. It's important that this rounds to nearest integer
+    // The the size of the canvas on the page. It's important that this rounds to nearest integer
    // and not ceils as browsers often have floating point precision issues where
    // `window.devicePixelRatio` ends up being something like `1.100000023841858` for example, when
-    // it's actually 1.1. Ceiling may cause blurriness as the backing canvas image is 1 pixel too
+    // it's actually 1.1. Ceiling may causes blurriness as the backing canvas image is 1 pixel too
    // large for the canvas element size.
    this.dimensions.css.canvas.height = Math.round(this.dimensions.device.canvas.height / this._devicePixelRatio);
    this.dimensions.css.canvas.width = Math.round(this.dimensions.device.canvas.width / this._devicePixelRatio);
@@ -23,12 +23,12 @@ declare module '@xterm/addon-webgl' {
    public readonly onChangeTextureAtlas: IEvent<HTMLCanvasElement>;

    /**
-     * An event that is fired when a new page is added to the texture atlas.
+     * An event that is fired when the a new page is added to the texture atlas.
     */
    public readonly onAddTextureAtlasCanvas: IEvent<HTMLCanvasElement>;

    /**
-     * An event that is fired when a page is removed from the texture atlas.
+     * An event that is fired when the a page is removed from the texture atlas.
     */
    public readonly onRemoveTextureAtlasCanvas: IEvent<HTMLCanvasElement>;

@@ -19,17 +19,14 @@ for (const addon of addons) {
 // Demo job - This requires the others to be built so it's not included when building all
 // jobs.push(createJob('demo-client', [`--demo-client`]));

-const codes = await Promise.all(jobs.map(job => {
+await Promise.all(jobs.map((job, i) => {
  return new Promise(r => {
    job.cp.on('exit', code => {
      log(`Finished \x1b[32m${job.name}\x1b[0m${code ? ' \x1b[31mwith errors\x1b[0m' : ''}`);
-      r(code ?? 1);
+      r(code);
    });
  });
 }));
-if (codes.some(code => code !== 0)) {
-  process.exit(1);
-}

 /**
 * @param {string} message
@@ -5,7 +5,7 @@ import { spawnSync } from 'node:child_process';
 import { basename, dirname, resolve, sep } from 'node:path';
 import { fileURLToPath } from 'node:url';

-const repoRoot = resolve(dirname(fileURLToPath(import.meta.url)), '..');
+const repoRoot = resolve(dirname(fileURLToPath(import.meta.url)), '../..');
 const nodeModulesPath = resolve(repoRoot, 'node_modules');
 const npmExecutable = process.platform === 'win32' ? 'npm.cmd' : 'npm';

@@ -13,7 +13,7 @@ const npmExecutable = process.platform === 'win32' ? 'npm.cmd' : 'npm';

 /** @param {string} message */
 function log(message) {
-  console.info(`[setup-full] ${message}`);
+  console.info(`[setup-fast] ${message}`);
 }

 /** @param {string[]} args */
@@ -12,7 +12,7 @@ let argv = process.argv.slice(2);
 let suiteFilter = undefined;
 while (argv.some(e => e.startsWith('--suite='))) {
  const i = argv.findIndex(e => e.startsWith('--suite='));
-  const match = argv[i].match(/--suite=(?<suitename>.+)/);
+  const match = argv[i].match(/--suite=(?<suitename>.+)/)
  suiteFilter = match?.groups?.suitename ?? undefined;
  argv.splice(i, 1);
 }
@@ -42,18 +42,12 @@ if (suiteFilter) {
  configs = configs.filter(e => e.name === suiteFilter);
 }

-if (suiteFilter && configs.length === 0) {
-  console.error(`Unknown suite: ${suiteFilter}`);
-  process.exit(1);
-}
-
 function npmBinScript(script) {
  return path.resolve(__dirname, `../node_modules/.bin/` + (process.platform === 'win32' ?
    `${script}.cmd` : script));
 }

 async function run() {
-  let exitCode = 0;
  for (const config of configs) {
    const command = npmBinScript('playwright');
    const args = ['test', '-c', config.path, ...argv];
@@ -68,13 +62,10 @@ async function run() {

    if (run.error) {
      console.error(run.error);
-      process.exit(run.status ?? 1);
+      process.exit(run.status ?? -1);
    }

-    if (run.status) {
-      exitCode = run.status;
-    }
+    process.exit(run.status);
  }
-  process.exit(exitCode);
 }
 run();
@@ -177,7 +177,7 @@ const ENC = {
 }

 function printMenu() {
-  console.log('\x1b[2J\x1b[HTest mouse reports [Ctrl-C to exit]');
+  console.log('\x1b[2J\x1b\[HTest mouse reports [Ctrl-C to exit]');
  console.log();
  console.log('  Selected protocol [Ctrl-A to switch]');
  const protocols = Object.keys(PROTOCOLS);
@@ -21,7 +21,7 @@ if (process.argv.length > 2) {
  flagArgs = args.filter(e => e.startsWith('--'));
  // ability to inject particular test files via
  // npm run test [testFileA testFileB ...]
-  const files = args.filter(e => !e.startsWith('--'));
+  files = args.filter(e => !e.startsWith('--'));
  if (files.length) {
    testFiles = files;
  }
@@ -39,6 +39,8 @@ if (checkCoverage) {
    ...testFiles,
    ...flagArgs
  ];
+  console.info('executable', executable);
+  console.info('args', args);
  const run = cp.spawnSync(
    executable,
    args,
@@ -49,7 +51,7 @@ if (checkCoverage) {
      stdio: 'inherit'
    }
  );
-  process.exit(run.status ?? -1);
+  process.exit(run.status);
 }

 const run = cp.spawnSync(
@@ -4700,18 +4700,22 @@
      }
    },
    "node_modules/glob": {
-      "version": "13.0.6",
-      "resolved": "https://registry.npmjs.org/glob/-/glob-13.0.6.tgz",
-      "integrity": "sha512-Wjlyrolmm8uDpm/ogGyXZXb1Z+Ca2B8NbJwqBVg0axK9GbBeoS7yGV6vjXnYdGm6X53iehEuxxbyiKp8QmN4Vw==",
+      "version": "10.5.0",
+      "resolved": "https://registry.npmjs.org/glob/-/glob-10.5.0.tgz",
+      "integrity": "sha512-DfXN8DfhJ7NH3Oe7cFmu3NCu1wKbkReJ8TorzSAFbSKrlNaQSKfIzqYqVY8zlbs2NLBbWpRiU52GX2PbaBVNkg==",
+      "deprecated": "Old versions of glob are not supported, and contain widely publicized security vulnerabilities, which have been fixed in the current version. Please update. Support for old versions may be purchased (at exorbitant rates) by contacting i@izs.me",
      "dev": true,
-      "license": "BlueOak-1.0.0",
+      "license": "ISC",
      "dependencies": {
-        "minimatch": "^10.2.2",
-        "minipass": "^7.1.3",
-        "path-scurry": "^2.0.2"
+        "foreground-child": "^3.1.0",
+        "jackspeak": "^3.1.2",
+        "minimatch": "^9.0.4",
+        "minipass": "^7.1.2",
+        "package-json-from-dist": "^1.0.0",
+        "path-scurry": "^1.11.1"
      },
-      "engines": {
-        "node": "18 || 20 || >=22"
+      "bin": {
+        "glob": "dist/esm/bin.mjs"
      },
      "funding": {
        "url": "https://github.com/sponsors/isaacs"
@@ -4737,6 +4741,39 @@
      "dev": true,
      "license": "BSD-2-Clause"
    },
+    "node_modules/glob/node_modules/balanced-match": {
+      "version": "1.0.2",
+      "resolved": "https://registry.npmjs.org/balanced-match/-/balanced-match-1.0.2.tgz",
+      "integrity": "sha512-3oSeUO0TMV67hN1AmbXsK4yaqU7tjiHlbxRDZOpH0KW9+CeX4bRAaX0Anxt0tx2MrpRpWwQaPwIlISEJhYU5Pw==",
+      "dev": true,
+      "license": "MIT"
+    },
+    "node_modules/glob/node_modules/brace-expansion": {
+      "version": "2.1.0",
+      "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-2.1.0.tgz",
+      "integrity": "sha512-TN1kCZAgdgweJhWWpgKYrQaMNHcDULHkWwQIspdtjV4Y5aurRdZpjAqn6yX3FPqTA9ngHCc4hJxMAMgGfve85w==",
+      "dev": true,
+      "license": "MIT",
+      "dependencies": {
+        "balanced-match": "^1.0.0"
+      }
+    },
+    "node_modules/glob/node_modules/minimatch": {
+      "version": "9.0.9",
+      "resolved": "https://registry.npmjs.org/minimatch/-/minimatch-9.0.9.tgz",
+      "integrity": "sha512-OBwBN9AL4dqmETlpS2zasx+vTeWclWzkblfZk7KTA5j3jeOONz/tRCnZomUyvNg83wL5Zv9Ss6HMJXAgL8R2Yg==",
+      "dev": true,
+      "license": "ISC",
+      "dependencies": {
+        "brace-expansion": "^2.0.2"
+      },
+      "engines": {
+        "node": ">=16 || 14 >=14.17"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
    "node_modules/gopd": {
      "version": "1.2.0",
      "resolved": "https://registry.npmjs.org/gopd/-/gopd-1.2.0.tgz",
@@ -5758,44 +5795,15 @@
      "license": "MIT"
    },
    "node_modules/mocha/node_modules/brace-expansion": {
-      "version": "2.1.1",
-      "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-2.1.1.tgz",
-      "integrity": "sha512-WR1cURNjuvBLMZBMbqM0UoE+WAfdUcEV1ccD8PVBVOI+Z3ND4+SZbN8RsfT2bMuG1qwz5RFvPukSZm5fF2D5eA==",
+      "version": "2.1.0",
+      "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-2.1.0.tgz",
+      "integrity": "sha512-TN1kCZAgdgweJhWWpgKYrQaMNHcDULHkWwQIspdtjV4Y5aurRdZpjAqn6yX3FPqTA9ngHCc4hJxMAMgGfve85w==",
      "dev": true,
      "license": "MIT",
      "dependencies": {
        "balanced-match": "^1.0.0"
      }
    },
-    "node_modules/mocha/node_modules/glob": {
-      "version": "10.5.0",
-      "resolved": "https://registry.npmjs.org/glob/-/glob-10.5.0.tgz",
-      "integrity": "sha512-DfXN8DfhJ7NH3Oe7cFmu3NCu1wKbkReJ8TorzSAFbSKrlNaQSKfIzqYqVY8zlbs2NLBbWpRiU52GX2PbaBVNkg==",
-      "deprecated": "Old versions of glob are not supported, and contain widely publicized security vulnerabilities, which have been fixed in the current version. Please update. Support for old versions may be purchased (at exorbitant rates) by contacting i@izs.me",
-      "dev": true,
-      "license": "ISC",
-      "dependencies": {
-        "foreground-child": "^3.1.0",
-        "jackspeak": "^3.1.2",
-        "minimatch": "^9.0.4",
-        "minipass": "^7.1.2",
-        "package-json-from-dist": "^1.0.0",
-        "path-scurry": "^1.11.1"
-      },
-      "bin": {
-        "glob": "dist/esm/bin.mjs"
-      },
-      "funding": {
-        "url": "https://github.com/sponsors/isaacs"
-      }
-    },
-    "node_modules/mocha/node_modules/lru-cache": {
-      "version": "10.4.3",
-      "resolved": "https://registry.npmjs.org/lru-cache/-/lru-cache-10.4.3.tgz",
-      "integrity": "sha512-JNAzZcXrCt42VGLuYz0zfAzDfAvJWW6AfYlDBQyDV5DClI2m5sAmK+OIO7s59XfsRsWHp02jAJrRadPRGTt6SQ==",
-      "dev": true,
-      "license": "ISC"
-    },
    "node_modules/mocha/node_modules/minimatch": {
      "version": "9.0.9",
      "resolved": "https://registry.npmjs.org/minimatch/-/minimatch-9.0.9.tgz",
@@ -5812,23 +5820,6 @@
        "url": "https://github.com/sponsors/isaacs"
      }
    },
-    "node_modules/mocha/node_modules/path-scurry": {
-      "version": "1.11.1",
-      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-1.11.1.tgz",
-      "integrity": "sha512-Xa4Nw17FS9ApQFJ9umLiJS4orGjm7ZzwUrwamcGQuHSzDyth9boKDaycYdDcZDuqYATXw4HFXgaqWTctW/v1HA==",
-      "dev": true,
-      "license": "BlueOak-1.0.0",
-      "dependencies": {
-        "lru-cache": "^10.2.0",
-        "minipass": "^5.0.0 || ^6.0.2 || ^7.0.0"
-      },
-      "engines": {
-        "node": ">=16 || 14 >=14.18"
-      },
-      "funding": {
-        "url": "https://github.com/sponsors/isaacs"
-      }
-    },
    "node_modules/ms": {
      "version": "2.1.3",
      "resolved": "https://registry.npmjs.org/ms/-/ms-2.1.3.tgz",
@@ -5996,6 +5987,24 @@
        "node": ">=8"
      }
    },
+    "node_modules/nyc/node_modules/glob": {
+      "version": "13.0.6",
+      "resolved": "https://registry.npmjs.org/glob/-/glob-13.0.6.tgz",
+      "integrity": "sha512-Wjlyrolmm8uDpm/ogGyXZXb1Z+Ca2B8NbJwqBVg0axK9GbBeoS7yGV6vjXnYdGm6X53iehEuxxbyiKp8QmN4Vw==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "minimatch": "^10.2.2",
+        "minipass": "^7.1.3",
+        "path-scurry": "^2.0.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
    "node_modules/nyc/node_modules/locate-path": {
      "version": "5.0.0",
      "resolved": "https://registry.npmjs.org/locate-path/-/locate-path-5.0.0.tgz",
@@ -6038,6 +6047,23 @@
        "node": ">=8"
      }
    },
+    "node_modules/nyc/node_modules/path-scurry": {
+      "version": "2.0.2",
+      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-2.0.2.tgz",
+      "integrity": "sha512-3O/iVVsJAPsOnpwWIeD+d6z/7PmqApyQePUtCndjatj/9I5LylHvt5qluFaBT3I5h3r1ejfR056c+FCv+NnNXg==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "lru-cache": "^11.0.0",
+        "minipass": "^7.1.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
    "node_modules/nyc/node_modules/signal-exit": {
      "version": "3.0.7",
      "resolved": "https://registry.npmjs.org/signal-exit/-/signal-exit-3.0.7.tgz",
@@ -6435,22 +6461,29 @@
      "license": "MIT"
    },
    "node_modules/path-scurry": {
-      "version": "2.0.2",
-      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-2.0.2.tgz",
-      "integrity": "sha512-3O/iVVsJAPsOnpwWIeD+d6z/7PmqApyQePUtCndjatj/9I5LylHvt5qluFaBT3I5h3r1ejfR056c+FCv+NnNXg==",
+      "version": "1.11.1",
+      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-1.11.1.tgz",
+      "integrity": "sha512-Xa4Nw17FS9ApQFJ9umLiJS4orGjm7ZzwUrwamcGQuHSzDyth9boKDaycYdDcZDuqYATXw4HFXgaqWTctW/v1HA==",
      "dev": true,
      "license": "BlueOak-1.0.0",
      "dependencies": {
-        "lru-cache": "^11.0.0",
-        "minipass": "^7.1.2"
+        "lru-cache": "^10.2.0",
+        "minipass": "^5.0.0 || ^6.0.2 || ^7.0.0"
      },
      "engines": {
-        "node": "18 || 20 || >=22"
+        "node": ">=16 || 14 >=14.18"
      },
      "funding": {
        "url": "https://github.com/sponsors/isaacs"
      }
    },
+    "node_modules/path-scurry/node_modules/lru-cache": {
+      "version": "10.4.3",
+      "resolved": "https://registry.npmjs.org/lru-cache/-/lru-cache-10.4.3.tgz",
+      "integrity": "sha512-JNAzZcXrCt42VGLuYz0zfAzDfAvJWW6AfYlDBQyDV5DClI2m5sAmK+OIO7s59XfsRsWHp02jAJrRadPRGTt6SQ==",
+      "dev": true,
+      "license": "ISC"
+    },
    "node_modules/path-to-regexp": {
      "version": "8.4.2",
      "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-8.4.2.tgz",
@@ -6656,6 +6689,16 @@
        "url": "https://github.com/sponsors/ljharb"
      }
    },
+    "node_modules/randombytes": {
+      "version": "2.1.0",
+      "resolved": "https://registry.npmjs.org/randombytes/-/randombytes-2.1.0.tgz",
+      "integrity": "sha512-vYl3iOX+4CKUWuxGi9Ukhie6fsqXqS9FE2Zaic4tNFD2N2QQaXOMFbuKK4QmDHC0JO6B1Zp41J0LpT0oR68amQ==",
+      "dev": true,
+      "license": "MIT",
+      "dependencies": {
+        "safe-buffer": "^5.1.0"
+      }
+    },
    "node_modules/range-parser": {
      "version": "1.2.1",
      "resolved": "https://registry.npmjs.org/range-parser/-/range-parser-1.2.1.tgz",
@@ -6827,6 +6870,41 @@
        "url": "https://github.com/sponsors/isaacs"
      }
    },
+    "node_modules/rimraf/node_modules/glob": {
+      "version": "13.0.6",
+      "resolved": "https://registry.npmjs.org/glob/-/glob-13.0.6.tgz",
+      "integrity": "sha512-Wjlyrolmm8uDpm/ogGyXZXb1Z+Ca2B8NbJwqBVg0axK9GbBeoS7yGV6vjXnYdGm6X53iehEuxxbyiKp8QmN4Vw==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "minimatch": "^10.2.2",
+        "minipass": "^7.1.3",
+        "path-scurry": "^2.0.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
+    "node_modules/rimraf/node_modules/path-scurry": {
+      "version": "2.0.2",
+      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-2.0.2.tgz",
+      "integrity": "sha512-3O/iVVsJAPsOnpwWIeD+d6z/7PmqApyQePUtCndjatj/9I5LylHvt5qluFaBT3I5h3r1ejfR056c+FCv+NnNXg==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "lru-cache": "^11.0.0",
+        "minipass": "^7.1.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
    "node_modules/router": {
      "version": "2.2.0",
      "resolved": "https://registry.npmjs.org/router/-/router-2.2.0.tgz",
@@ -6854,6 +6932,27 @@
        "tslib": "^2.1.0"
      }
    },
+    "node_modules/safe-buffer": {
+      "version": "5.2.1",
+      "resolved": "https://registry.npmjs.org/safe-buffer/-/safe-buffer-5.2.1.tgz",
+      "integrity": "sha512-rp3So07KcdmmKbGvgaNxQSJr7bGVSVk5S9Eq1F+ppbRo70+YeaDxkw5Dd8NPN+GD6bjnYm2VuPuCXmpuYvmCXQ==",
+      "dev": true,
+      "funding": [
+        {
+          "type": "github",
+          "url": "https://github.com/sponsors/feross"
+        },
+        {
+          "type": "patreon",
+          "url": "https://www.patreon.com/feross"
+        },
+        {
+          "type": "consulting",
+          "url": "https://feross.org/support"
+        }
+      ],
+      "license": "MIT"
+    },
    "node_modules/safer-buffer": {
      "version": "2.1.2",
      "resolved": "https://registry.npmjs.org/safer-buffer/-/safer-buffer-2.1.2.tgz",
@@ -6979,13 +7078,13 @@
      }
    },
    "node_modules/serialize-javascript": {
-      "version": "7.0.5",
-      "resolved": "https://registry.npmjs.org/serialize-javascript/-/serialize-javascript-7.0.5.tgz",
-      "integrity": "sha512-F4LcB0UqUl1zErq+1nYEEzSHJnIwb3AF2XWB94b+afhrekOUijwooAYqFyRbjYkm2PAKBabx6oYv/xDxNi8IBw==",
+      "version": "6.0.2",
+      "resolved": "https://registry.npmjs.org/serialize-javascript/-/serialize-javascript-6.0.2.tgz",
+      "integrity": "sha512-Saa1xPByTTq2gdeFZYLLo+RFE35NHZkAbqZeWNd3BpzppeVisAqpDjcp8dyf6uIvEqJRd46jemmyA4iFIeVk8g==",
      "dev": true,
      "license": "BSD-3-Clause",
-      "engines": {
-        "node": ">=20.0.0"
+      "dependencies": {
+        "randombytes": "^2.1.0"
      }
    },
    "node_modules/serve-static": {
@@ -7586,6 +7685,41 @@
        "node": "20 || >=22"
      }
    },
+    "node_modules/test-exclude/node_modules/glob": {
+      "version": "13.0.6",
+      "resolved": "https://registry.npmjs.org/glob/-/glob-13.0.6.tgz",
+      "integrity": "sha512-Wjlyrolmm8uDpm/ogGyXZXb1Z+Ca2B8NbJwqBVg0axK9GbBeoS7yGV6vjXnYdGm6X53iehEuxxbyiKp8QmN4Vw==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "minimatch": "^10.2.2",
+        "minipass": "^7.1.3",
+        "path-scurry": "^2.0.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
+    "node_modules/test-exclude/node_modules/path-scurry": {
+      "version": "2.0.2",
+      "resolved": "https://registry.npmjs.org/path-scurry/-/path-scurry-2.0.2.tgz",
+      "integrity": "sha512-3O/iVVsJAPsOnpwWIeD+d6z/7PmqApyQePUtCndjatj/9I5LylHvt5qluFaBT3I5h3r1ejfR056c+FCv+NnNXg==",
+      "dev": true,
+      "license": "BlueOak-1.0.0",
+      "dependencies": {
+        "lru-cache": "^11.0.0",
+        "minipass": "^7.1.2"
+      },
+      "engines": {
+        "node": "18 || 20 || >=22"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/isaacs"
+      }
+    },
    "node_modules/tiny-emitter": {
      "version": "2.1.0",
      "resolved": "https://registry.npmjs.org/tiny-emitter/-/tiny-emitter-2.1.0.tgz",
@@ -61,9 +61,9 @@
    "test-integration-firefox": "node ./bin/test_integration.js --workers=75% \"--project=FirefoxStable\"",
    "test-integration-webkit": "node ./bin/test_integration.js --workers=75% \"--project=WebKit\"",
    "test-integration-debug": "node ./bin/test_integration.js  --workers=1 --headed --timeout=30000",
-    "benchmark": "NODE_PATH=./out xterm-benchmark -r 5 -c test/benchmark/benchmark.json",
-    "benchmark-baseline": "NODE_PATH=./out xterm-benchmark -r 5 -c test/benchmark/benchmark.json --baseline out-test/benchmark/*benchmark.js",
-    "benchmark-eval": "NODE_PATH=./out xterm-benchmark -r 5 -c test/benchmark/benchmark.json --eval out-test/benchmark/*benchmark.js",
+    "benchmark": "xterm-benchmark -r 5 -c test/benchmark/benchmark.json",
+    "benchmark-baseline": "xterm-benchmark -r 5 -c test/benchmark/benchmark.json --baseline out-test/benchmark/*benchmark.js",
+    "benchmark-eval": "xterm-benchmark -r 5 -c test/benchmark/benchmark.json --eval out-test/benchmark/*benchmark.js",
    "clean": "rm -rf lib out addons/*/lib addons/*/out",
    "vtfeatures": "node bin/extract_vtfeatures.js src/*/*.ts src/*/*/*.ts src/*.ts addons/**/src/*.ts",
    "prepackage": "npm run build",
@@ -115,8 +115,5 @@
    "webpack-cli": "^7.0.2",
    "ws": "^8.20.0",
    "xterm-benchmark": "^0.3.2"
-  },
-  "overrides": {
-    "serialize-javascript": "^7.0.5"
  }
 }
@@ -102,7 +102,7 @@ export class CoreBrowserTerminal extends CoreTerminal implements ITerminal {
  private _keyDownHandled: boolean = false;

  /**
-   * Records whether a keydown event has occurred since the last keyup event, i.e. whether a key
+   * Records whether a keydown event has occured since the last keyup event, i.e. whether a key
   * is currently "pressed".
   */
  private _keyDownSeen: boolean = false;
@@ -31,7 +31,9 @@ export class RenderDebouncer implements IRenderDebouncerWithCallback {

  public addRefreshCallback(callback: FrameRequestCallback): number {
    this._refreshCallbacks.push(callback);
-    this._animationFrame ??= this._coreBrowserService.window.requestAnimationFrame(() => this._innerRefresh());
+    if (this._animationFrame === undefined) {
+      this._animationFrame = this._coreBrowserService.window.requestAnimationFrame(() => this._innerRefresh());
+    }
    return this._animationFrame;
  }

@@ -31,7 +31,7 @@ export function getCoordsRelativeToElement(window: Pick<Window, 'getComputedStyl
 * select that cell and the right half will select the next cell.
 */
 export function getCoords(window: Pick<Window, 'getComputedStyle'>, event: Pick<MouseEvent, 'clientX' | 'clientY'>, element: HTMLElement, colCount: number, rowCount: number, hasValidCharSize: boolean, cssCellWidth: number, cssCellHeight: number, isSelection?: boolean): [number, number] | undefined {
-  // Coordinates cannot be measured if there is no valid character size.
+  // Coordinates cannot be measured if there are no valid
  if (!hasValidCharSize) {
    return undefined;
  }
@@ -41,7 +41,7 @@ export function getCoords(window: Pick<Window, 'getComputedStyle'>, event: Pick<
  coords[1] = Math.ceil(coords[1] / cssCellHeight);

  // Ensure coordinates are within the terminal viewport. Note that selections
-  // need an additional point of precision to cover the end point (as characters
+  // need an addition point of precision to cover the end point (as characters
  // cover half of one char and half of the next).
  coords[0] = Math.min(Math.max(coords[0], 1), colCount + (isSelection ? 1 : 0));
  coords[1] = Math.min(Math.max(coords[1], 1), rowCount);
@@ -176,7 +176,7 @@ describe('MouseService _triggerMouseEvent', () => {
      for (let i = 0; i < bufferService.cols; ++i) {
        assert.equal(trigger({ col: i, row: 0, x: 0, y: 0, button: CoreMouseButton.LEFT, action: CoreMouseAction.DOWN }), true);
        if (i > 222) {
-          // suppress mouse reports if we are out of addressable range (max. 222)
+          // supress mouse reports if we are out of addressible range (max. 222)
          assert.deepEqual(toBytes(reports.pop()), []);
        } else {
          assert.deepEqual(toBytes(reports.pop()), [0x1b, 0x5b, 0x4d, 0x20, i + 33, 0x21]);
@@ -132,7 +132,7 @@ export class ThemeService extends Disposable implements IThemeService {
        colors.ansi[i + 16] = parseColor(theme.extendedAnsi[i], DEFAULT_ANSI_COLORS[i + 16]);
      }
    }
-    // Clear the cache
+    // Clear our the cache
    this._contrastCache.clear();
    this._halfContrastCache.clear();
    this._updateRestoreColors();
@@ -373,7 +373,7 @@ export function toPaddedHex(c: number): string {
 /**
 * Gets the contrast ratio between two relative luminance values.
 * @param l1 The first relative luminance.
- * @param l2 The second relative luminance.
+ * @param l2 The first relative luminance.
 * @see https://www.w3.org/TR/WCAG20/#contrast-ratiodef
 */
 export function contrastRatio(l1: number, l2: number): number {
@@ -16,10 +16,8 @@ import { MockCoreService, MockBufferService, MockOptionsService, MockLogService,
 import { IBufferService, ICoreService, type IOscLinkService } from './services/Services';
 import { DEFAULT_OPTIONS } from './services/OptionsService';
 import { BufferService } from './services/BufferService';
-import { CharsetService } from './services/CharsetService';
 import { CoreService } from './services/CoreService';
 import { OscLinkService } from './services/OscLinkService';
-import { CHARSETS } from './data/Charsets';


 function getCursor(bufferService: IBufferService): number[] {
@@ -641,38 +639,19 @@ describe('InputHandler', () => {
  });
  describe('print', () => {
    it('should not cause an infinite loop (regression test)', () => {
+      const inputHandler = new TestInputHandler(
+        new MockBufferService(80, 30),
+        new MockCharsetService(),
+        new MockCoreService(),
+        new MockLogService(),
+        new MockOptionsService(),
+        new MockOscLinkService(),
+        new MockMouseStateService(),
+        new MockUnicodeService()
+      );
      const container = new Uint32Array(10);
      container[0] = 0x200B;
-      const lineCountBefore = bufferService.buffer.lines.length;
      inputHandler.print(container, 0, 1);
-      assert.strictEqual(bufferService.buffer.y, 0);
-      assert.strictEqual(bufferService.buffer.lines.length, lineCountBefore);
-    });
-    it('should join combining characters in a single print', async () => {
-      await inputHandler.parseP('e\u0301');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), 'e\u0301');
-      assert.strictEqual(bufferService.buffer.x, 1);
-    });
-    it('should join combining characters split across parse calls', async () => {
-      await inputHandler.parseP('e');
-      await inputHandler.parseP('\u0301');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), 'e\u0301');
-      assert.strictEqual(bufferService.buffer.x, 1);
-    });
-    it('should repeat preceding grapheme cluster via REP', async () => {
-      await inputHandler.parseP('e\u0301\x1b[2b');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), 'e\u0301e\u0301e\u0301');
-      assert.strictEqual(bufferService.buffer.x, 3);
-    });
-    it('should not repeat when REP has no preceding join state', async () => {
-      await inputHandler.parseP('\x1b[2b');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), '');
-      assert.strictEqual(bufferService.buffer.x, 0);
-    });
-    it('should not repeat after an intervening escape sequence', async () => {
-      await inputHandler.parseP('a\x1b[0m\x1b[2b');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), 'a');
-      assert.strictEqual(bufferService.buffer.x, 1);
    });
    it('should clear cells to the right on early wrap-around', async () => {
      bufferService.resize(5, 5);
@@ -689,49 +668,6 @@ describe('InputHandler', () => {
    });
  });

-  describe('ISO-2022 character sets', () => {
-    let charsetService: CharsetService;
-
-    beforeEach(() => {
-      charsetService = new CharsetService();
-      inputHandler = new TestInputHandler(
-        bufferService,
-        charsetService,
-        coreService,
-        new MockLogService(),
-        optionsService,
-        oscLinkService,
-        new MockMouseStateService(),
-        new MockUnicodeService()
-      );
-    });
-
-    it('should map G0 line drawing via ESC ( 0', async () => {
-      await inputHandler.parseP('\x1b(0q\x1b(Bq');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), '\u2500q');
-    });
-
-    it('should map G1 line drawing after ESC ) 0 and SO', async () => {
-      await inputHandler.parseP('\x1b)0\x0eq\x0f\x1b(Bq');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), '\u2500q');
-    });
-
-    it('should restore charset and glevel on ESC 7 / ESC 8', async () => {
-      await inputHandler.parseP('\x1b)0\x0e');
-      assert.strictEqual(charsetService.glevel, 1);
-      assert.strictEqual(charsetService.charset, CHARSETS['0']);
-      await inputHandler.parseP('\x1b7');
-      await inputHandler.parseP('\x0f\x1b(B');
-      assert.strictEqual(charsetService.glevel, 0);
-      assert.ok(charsetService.charset === undefined);
-      await inputHandler.parseP('\x1b8');
-      assert.strictEqual(charsetService.glevel, 1);
-      assert.strictEqual(charsetService.charset, CHARSETS['0']);
-      await inputHandler.parseP('q');
-      assert.strictEqual(bufferService.buffer.translateBufferLineToString(0, true), '\u2500');
-    });
-  });
-
  describe('alt screen', () => {
    let bufferService: IBufferService;
    let handler: TestInputHandler;
@@ -1246,19 +1182,6 @@ describe('InputHandler', () => {
      await inputHandler.parseP('\x1b[100;100H');
      assert.deepEqual(getCursor(bufferService), [9, 9]);
    });
-    it('cursor position (CUP) with DECOM and scroll margins', async () => {
-      await inputHandler.parseP('\x1b[?6h\x1b[2;3r\x1b[1;1H');
-      assert.deepEqual(getCursor(bufferService), [0, 1]);
-      await inputHandler.parseP('X');
-      assert.strictEqual(getLines(bufferService, 3)[1], 'X');
-      await inputHandler.parseP('\x1b[2;1H');
-      assert.deepEqual(getCursor(bufferService), [0, 2]);
-      await inputHandler.parseP('\x1b[10;10H');
-      assert.deepEqual(getCursor(bufferService), [9, 2]);
-      await inputHandler.parseP('\x1b[?6l');
-      await inputHandler.parseP('\x1b[2;1H');
-      assert.deepEqual(getCursor(bufferService), [0, 1]);
-    });
    it('horizontal position absolute (HPA)', async () => {
      await inputHandler.parseP('\x1b[`');
      assert.deepEqual(getCursor(bufferService), [0, 0]);
@@ -1318,7 +1241,7 @@ describe('InputHandler', () => {
      await inputHandler.parseP('\x1b[e');
      assert.deepEqual(getCursor(bufferService), [8, 5]);
    });
-    describe('should clamp cursor into addressable range', () => {
+    describe('should clamp cursor into addressible range', () => {
      it('CUF', async () => {
        bufferService.buffer.x = 10000;
        bufferService.buffer.y = 10000;
@@ -44,7 +44,7 @@ export function getSafariVersion(): number {
  return parseInt(majorVersion[1], 10);
 }

-// Find the user's platform. We use this to interpret the meta key
+// Find the users platform. We use this to interpret the meta key
 // and ISO third level shifts.
 // http://stackoverflow.com/q/19877924/577598
 export const isMac = ['Macintosh', 'MacIntel', 'MacPPC', 'Mac68K'].includes(platform);
@@ -8,7 +8,7 @@ import type { ILogService } from './services/Services';
 interface ITaskQueue {
  /**
   * Adds a task to the queue which will run in a future idle callback.
-   * To avoid perceivable stalls on the main thread, tasks with heavy workload
+   * To avoid perceivable stalls on the mainthread, tasks with heavy workload
   * should split their work into smaller pieces and return `true` to get
   * called again until the work is done (on falsy return value).
   */
@@ -181,7 +181,7 @@ export interface ICoreMouseEvent {
  x: number;
  y: number;
  /**
-   * Button the action occurred. Due to restrictions of the tracking protocols
+   * Button the action occured. Due to restrictions of the tracking protocols
   * it is not possible to report multiple buttons at once.
   * Wheel is treated as a button.
   * There are invalid combinations of buttons and actions possible
@@ -1,54 +0,0 @@
-/**
- * Copyright (c) 2026 The xterm.js authors. All rights reserved.
- * @license MIT
- */
-
-import { assert } from 'chai';
-import { DEFAULT_ATTR_DATA } from './buffer/BufferLine';
-import { updateWindowsModeWrappedState } from './WindowsMode';
-import { BufferService } from './services/BufferService';
-import { OptionsService } from './services/OptionsService';
-import { MockLogService } from './TestUtils.test';
-
-describe('WindowsMode', () => {
-  describe('updateWindowsModeWrappedState', () => {
-    it('should mark the next line wrapped when the previous line ends in a non-whitespace character', () => {
-      const bufferService = new BufferService(new OptionsService({ rows: 5, cols: 10 }), new MockLogService());
-      const buffer = bufferService.buffer;
-      const previousLine = buffer.lines.get(buffer.ybase)!;
-      for (let i = 0; i < bufferService.cols; i++) {
-        previousLine!.setCellFromCodepoint(i, 'a'.charCodeAt(0), 1, DEFAULT_ATTR_DATA);
-      }
-      buffer.y = 1;
-
-      updateWindowsModeWrappedState(bufferService);
-
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 1)!.isWrapped, true);
-    });
-
-    it('should not mark the next line wrapped when the previous line ends in whitespace', () => {
-      const bufferService = new BufferService(new OptionsService({ rows: 5, cols: 10 }), new MockLogService());
-      const buffer = bufferService.buffer;
-      const previousLine = buffer.lines.get(buffer.ybase)!;
-      for (let i = 0; i < bufferService.cols - 1; i++) {
-        previousLine!.setCellFromCodepoint(i, 'a'.charCodeAt(0), 1, DEFAULT_ATTR_DATA);
-      }
-      previousLine!.setCellFromCodepoint(bufferService.cols - 1, ' '.charCodeAt(0), 1, DEFAULT_ATTR_DATA);
-      buffer.y = 1;
-
-      updateWindowsModeWrappedState(bufferService);
-
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 1)!.isWrapped, false);
-    });
-
-    it('should not mark the next line wrapped when the previous line ends in a null cell', () => {
-      const bufferService = new BufferService(new OptionsService({ rows: 5, cols: 10 }), new MockLogService());
-      const buffer = bufferService.buffer;
-      buffer.y = 1;
-
-      updateWindowsModeWrappedState(bufferService);
-
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 1)!.isWrapped, false);
-    });
-  });
-});
@@ -233,27 +233,6 @@ describe('Buffer', () => {
          assert.equal(buffer.lines.length, INIT_ROWS + 10);
        });
      });
-
-      describe('Windows ConPTY', () => {
-        beforeEach(() => {
-          optionsService.options.windowsPty = { backend: 'conpty', buildNumber: 19000 };
-        });
-
-        it('should not adjust ybase or ydisp when growing rows', () => {
-          buffer.fillViewportRows();
-          for (let i = 0; i < 10; i++) {
-            buffer.lines.push(buffer.getBlankLine(DEFAULT_ATTR_DATA));
-          }
-          buffer.y = INIT_ROWS - 1;
-          buffer.ybase = 10;
-          buffer.ydisp = 10;
-          const linesBefore = buffer.lines.length;
-          buffer.resize(INIT_COLS, INIT_ROWS + 5);
-          assert.equal(buffer.ybase, 10);
-          assert.equal(buffer.ydisp, 10);
-          assert.equal(buffer.lines.length, linesBefore + 5);
-        });
-      });
    });

    describe('row and column increased', () => {
@@ -326,71 +305,6 @@ describe('Buffer', () => {
        assert.equal(buffer.lines.get(8)!.translateToString(), '     ');
        assert.equal(buffer.lines.get(9)!.translateToString(), '     ');
      });
-      it('should gate reflow on ConPTY buildNumber 21376', () => {
-        const prepareWrappedShrink = () => {
-          buffer.fillViewportRows();
-          buffer.resize(5, 10);
-          const firstLine = buffer.lines.get(0)!;
-          for (let i = 0; i < 5; i++) {
-            const code = 'a'.charCodeAt(0) + i;
-            firstLine.set(i, [0, String.fromCharCode(code), 1, code]);
-          }
-          buffer.y = 1;
-          buffer.resize(1, 10);
-        };
-
-        optionsService.options.windowsPty = { backend: 'conpty', buildNumber: 21375 };
-        prepareWrappedShrink();
-        assert.equal(buffer.lines.get(1)!.translateToString().trim(), '');
-
-        buffer = new TestBuffer(true, optionsService, bufferService, new MockLogService());
-        optionsService.options.windowsPty = { backend: 'conpty', buildNumber: 21376 };
-        prepareWrappedShrink();
-        assert.equal(buffer.lines.get(1)!.translateToString().trim(), 'b');
-        assert.ok(buffer.lines.get(1)!.isWrapped);
-      });
-      it('should unwrap lines on ConPTY builds with reflow support', () => {
-        optionsService.options.windowsPty = { backend: 'conpty', buildNumber: 21376 };
-        buffer.fillViewportRows();
-        buffer.resize(5, 10);
-        const firstLine = buffer.lines.get(0)!;
-        for (let i = 0; i < 5; i++) {
-          const code = 'a'.charCodeAt(0) + i;
-          firstLine.set(i, [0, String.fromCharCode(code), 1, code]);
-        }
-        buffer.y = 1;
-        buffer.resize(1, 10);
-        buffer.resize(5, 10);
-        assert.equal(buffer.lines.get(0)!.translateToString(), 'abcde');
-        assert.equal(buffer.lines.get(1)!.translateToString(), '     ');
-      });
-      it('should reflow wrapped lines containing the cursor when reflowCursorLine is enabled', () => {
-        optionsService.options.reflowCursorLine = true;
-        buffer.fillViewportRows();
-        buffer.resize(5, 10);
-        const firstLine = buffer.lines.get(0)!;
-        for (let i = 0; i < 5; i++) {
-          const code = 'a'.charCodeAt(0) + i;
-          firstLine.set(i, [0, String.fromCharCode(code), 1, code]);
-        }
-        buffer.resize(1, 10);
-        buffer.y = 2;
-        buffer.resize(5, 10);
-        assert.equal(buffer.lines.get(0)!.translateToString(), 'abcde');
-      });
-      it('should not reflow wrapped lines containing the cursor by default', () => {
-        buffer.fillViewportRows();
-        buffer.resize(5, 10);
-        const firstLine = buffer.lines.get(0)!;
-        for (let i = 0; i < 5; i++) {
-          const code = 'a'.charCodeAt(0) + i;
-          firstLine.set(i, [0, String.fromCharCode(code), 1, code]);
-        }
-        buffer.resize(1, 10);
-        buffer.y = 2;
-        buffer.resize(5, 10);
-        assert.notEqual(buffer.lines.get(0)!.translateToString(), 'abcde');
-      });
      it('should discard parts of wrapped lines that go out of the scrollback', () => {
        buffer.fillViewportRows();
        optionsService.options.scrollback = 1;
@@ -142,7 +142,7 @@ export class Buffer extends Disposable implements IBuffer {
  }

  /**
-   * Clears the buffer to its initial state, discarding all previous data.
+   * Clears the buffer to it's initial state, discarding all previous data.
   */
  public clear(): void {
    this._stringCache.clear();
@@ -197,7 +197,7 @@ export class Buffer extends Disposable implements IBuffer {
        for (let y = this._rows; y < newRows; y++) {
          if (this.lines.length < newRows + this.ybase) {
            if (this._optionsService.rawOptions.windowsPty.backend !== undefined || this._optionsService.rawOptions.windowsPty.buildNumber !== undefined) {
-              // Just add the new missing rows on Windows as conpty reprints the screen with its
+              // 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(this._stringCache, newCols, nullCell, false));
            } else {
@@ -73,9 +73,7 @@ export interface IBufferLineStringCache {
 */
 export class BufferLine implements IBufferLine {
  protected _data: Uint32Array;
-  /** Sparse cache; only read when `IS_COMBINED_MASK` is set in `_data`. */
  protected _combined: {[index: number]: string} = {};
-  /** Sparse cache; only read when `HAS_EXTENDED` is set in `_data`. */
  protected _extendedAttrs: {[index: number]: IExtendedAttrs | undefined} = {};
  protected _stringCacheEntryRef: WeakRef<IBufferLineStringCacheEntry> | undefined;
  public length: number;
@@ -207,15 +205,9 @@ export class BufferLine implements IBufferLine {
    cell.bg = this._data[$startIndex + Cell.BG];
    if (cell.content & Content.IS_COMBINED_MASK) {
      cell.combinedData = this._combined[index];
-    } else {
-      cell.combinedData = '';
    }
    if (cell.bg & BgFlags.HAS_EXTENDED) {
      cell.extended = this._extendedAttrs[index]!;
-    } else {
-      // Do not mutate cell.extended in place: it may still reference this line's map entry from a
-      // prior loadCell into a reused CellData (e.g. $workCell during insert/delete).
-      cell.extended = DEFAULT_ATTR_DATA.extended.clone();
    }
    return cell;
  }
@@ -467,7 +459,14 @@ export class BufferLine implements IBufferLine {
      this._data.set(line._data);
    }
    this.length = line.length;
-    this._copySparseMapsFrom(line);
+    this._combined = {};
+    for (const el in line._combined) {
+      this._combined[el] = line._combined[el];
+    }
+    this._extendedAttrs = {};
+    for (const el in line._extendedAttrs) {
+      this._extendedAttrs[el] = line._extendedAttrs[el];
+    }
    this.isWrapped = line.isWrapped;
  }

@@ -476,7 +475,12 @@ export class BufferLine implements IBufferLine {
    const newLine = new BufferLine(this._stringCache, 0, undefined, false);
    newLine._data = new Uint32Array(this._data);
    newLine.length = this.length;
-    newLine._copySparseMapsFrom(this);
+    for (const el in this._combined) {
+      newLine._combined[el] = this._combined[el];
+    }
+    for (const el in this._extendedAttrs) {
+      newLine._extendedAttrs[el] = this._extendedAttrs[el];
+    }
    newLine.isWrapped = this.isWrapped;
    return newLine;
  }
@@ -507,14 +511,27 @@ export class BufferLine implements IBufferLine {
        for (let i = 0; i < Constants.CELL_INDICIES; i++) {
          this._data[(destCol + cell) * Constants.CELL_INDICIES + i] = srcData[(srcCol + cell) * Constants.CELL_INDICIES + i];
        }
-        this._copyCellMapsFrom(src, srcCol + cell, destCol + cell);
+        if (srcData[(srcCol + cell) * Constants.CELL_INDICIES + Cell.BG] & BgFlags.HAS_EXTENDED) {
+          this._extendedAttrs[destCol + cell] = src._extendedAttrs[srcCol + cell];
+        }
      }
    } else {
      for (let cell = 0; cell < length; cell++) {
        for (let i = 0; i < Constants.CELL_INDICIES; i++) {
          this._data[(destCol + cell) * Constants.CELL_INDICIES + i] = srcData[(srcCol + cell) * Constants.CELL_INDICIES + i];
        }
-        this._copyCellMapsFrom(src, srcCol + cell, destCol + cell);
+        if (srcData[(srcCol + cell) * Constants.CELL_INDICIES + Cell.BG] & BgFlags.HAS_EXTENDED) {
+          this._extendedAttrs[destCol + cell] = src._extendedAttrs[srcCol + cell];
+        }
+      }
+    }
+
+    // Move any combined data over as needed, FIXME: repeat for extended attrs
+    const srcCombinedKeys = Object.keys(src._combined);
+    for (let i = 0; i < srcCombinedKeys.length; i++) {
+      const key = parseInt(srcCombinedKeys[i], 10);
+      if (key >= srcCol) {
+        this._combined[key - srcCol + destCol] = src._combined[key];
      }
    }
  }
@@ -603,24 +620,4 @@ export class BufferLine implements IBufferLine {
      cacheEntry.isTrimmed = false;
    }
  }
-
-  /** Copy sparse map entries for a single cell when `_data` flags require them. */
-  private _copyCellMapsFrom(src: BufferLine, srcCol: number, destCol: number): void {
-    const srcStart = srcCol * Constants.CELL_INDICIES;
-    if (src._data[srcStart + Cell.CONTENT] & Content.IS_COMBINED_MASK) {
-      this._combined[destCol] = src._combined[srcCol];
-    }
-    if (src._data[srcStart + Cell.BG] & BgFlags.HAS_EXTENDED) {
-      this._extendedAttrs[destCol] = src._extendedAttrs[srcCol];
-    }
-  }
-
-  /** Rebuild sparse maps from another line, keyed only by `_data` flags. */
-  private _copySparseMapsFrom(line: BufferLine): void {
-    this._combined = {};
-    this._extendedAttrs = {};
-    for (let i = 0; i < line.length; i++) {
-      this._copyCellMapsFrom(line, i, i);
-    }
-  }
 }
@@ -3,13 +3,10 @@
 * @license MIT
 */
 import { assert } from 'chai';
-import { CircularList } from '../CircularList';
 import { BufferLine } from './BufferLine';
 import { BufferLineStringCache } from './BufferLineStringCache';
-import { CellData } from './CellData';
 import { NULL_CELL_CHAR, NULL_CELL_WIDTH, NULL_CELL_CODE } from './Constants';
-import { reflowLargerGetLinesToRemove, reflowSmallerGetNewLineLengths } from './BufferReflow';
-import { IBufferLine } from './Types';
+import { reflowSmallerGetNewLineLengths } from './BufferReflow';

 const TEST_STRING_CACHE = new BufferLineStringCache();

@@ -96,31 +93,4 @@ describe('BufferReflow', () => {
      assert.deepEqual(reflowSmallerGetNewLineLengths([line], 4, 2), [2, 2], 'line: 汉, 语');
    });
  });
-  describe('reflowLargerGetLinesToRemove', () => {
-    const nullCell = CellData.fromCharData([0, NULL_CELL_CHAR, NULL_CELL_WIDTH, NULL_CELL_CODE]);
-
-    function createWrappedLines(chars: string): CircularList<IBufferLine> {
-      const lines = new CircularList<IBufferLine>(chars.length);
-      for (let i = 0; i < chars.length; i++) {
-        const line = new BufferLine(TEST_STRING_CACHE, 1);
-        line.set(0, [0, chars[i], 1, chars.charCodeAt(i)]);
-        line.isWrapped = i > 0;
-        lines.push(line);
-      }
-      return lines;
-    }
-
-    it('should skip reflow when the cursor is in a wrapped block and reflowCursorLine is false', () => {
-      const lines = createWrappedLines('abcde');
-      const skipped = reflowLargerGetLinesToRemove(lines, 1, 5, 2, nullCell, false);
-      const reflowed = reflowLargerGetLinesToRemove(lines, 1, 5, 2, nullCell, true);
-      assert.deepEqual(skipped, []);
-      assert.notDeepEqual(reflowed, []);
-    });
-
-    it('should reflow wrapped blocks when the cursor is outside the block', () => {
-      const lines = createWrappedLines('abcde');
-      assert.notDeepEqual(reflowLargerGetLinesToRemove(lines, 1, 5, 10, nullCell, false), []);
-    });
-  });
 });
@@ -24,7 +24,7 @@ export const NULL_CELL_CODE = 0;
 /**
 * Whitespace cell.
 * This is meant as a replacement for empty cells when needed
- * during rendering lines to preserve correct alignment.
+ * during rendering lines to preserve correct aligment.
 */
 export const WHITESPACE_CELL_CHAR = ' ';
 export const WHITESPACE_CELL_WIDTH = 1;
@@ -36,18 +36,18 @@ export const WHITESPACE_CELL_CODE = 32;
 export const enum Content {
  /**
   * bit 1..21    codepoint, max allowed in UTF32 is 0x10FFFF (21 bits taken)
-   *              read:   `codepoint = content & Content.CODEPOINT_MASK;`
-   *              write:  `content |= codepoint & Content.CODEPOINT_MASK;`
+   *              read:   `codepoint = content & Content.codepointMask;`
+   *              write:  `content |= codepoint & Content.codepointMask;`
   *                      shortcut if precondition `codepoint <= 0x10FFFF` is met:
   *                      `content |= codepoint;`
   */
  CODEPOINT_MASK = 0x1FFFFF,

  /**
-   * bit 22       flag indicating whether a cell contains combined content
-   *              read:   `isCombined = content & Content.IS_COMBINED_MASK;`
-   *              set:    `content |= Content.IS_COMBINED_MASK;`
-   *              clear:  `content &= ~Content.IS_COMBINED_MASK;`
+   * bit 22       flag indication whether a cell contains combined content
+   *              read:   `isCombined = content & Content.isCombined;`
+   *              set:    `content |= Content.isCombined;`
+   *              clear:  `content &= ~Content.isCombined;`
   */
  IS_COMBINED_MASK = 0x200000,  // 1 << 21

@@ -55,19 +55,19 @@ export const enum Content {
   * bit 1..22    mask to check whether a cell contains any string data
   *              we need to check for codepoint and isCombined bits to see
   *              whether a cell contains anything
-   *              read:   `isEmpty = !(content & Content.HAS_CONTENT_MASK)`
+   *              read:   `isEmpty = !(content & Content.hasContent)`
   */
  HAS_CONTENT_MASK = 0x3FFFFF,

  /**
   * bit 23..24   wcwidth value of cell, takes 2 bits (ranges from 0..2)
-   *              read:   `width = (content & Content.WIDTH_MASK) >> Content.WIDTH_SHIFT;`
-   *                      `hasWidth = content & Content.WIDTH_MASK;`
+   *              read:   `width = (content & Content.widthMask) >> Content.widthShift;`
+   *                      `hasWidth = content & Content.widthMask;`
   *                      as long as wcwidth is highest value in `content`:
-   *                      `width = content >> Content.WIDTH_SHIFT;`
-   *              write:  `content |= (width << Content.WIDTH_SHIFT) & Content.WIDTH_MASK;`
+   *                      `width = content >> Content.widthShift;`
+   *              write:  `content |= (width << Content.widthShift) & Content.widthMask;`
   *                      shortcut if precondition `0 <= width <= 3` is met:
-   *                      `content |= width << Content.WIDTH_SHIFT;`
+   *                      `content |= width << Content.widthShift;`
   */
  WIDTH_MASK = 0xC00000,   // 3 << 22
  WIDTH_SHIFT = 22
@@ -30,7 +30,7 @@ export class Marker implements IMarker {
    }
    this.isDisposed = true;
    this.line = -1;
-    // Emit before super.dispose such that dispose listeners get a chance to react
+    // Emit before super.dispose such that dispose listeners get a change to react
    this._onDispose.fire();
    dispose(this._disposables);
    this._disposables.length = 0;
@@ -46,7 +46,7 @@ export function evaluateKeyboardEvent(
    // Whether to cancel event propagation (NOTE: this may not be needed since the event is
    // canceled at the end of keyDown
    cancel: false,
-    // The new key event to emit
+    // The new key even to emit
    key: undefined
  };
  const modifiers = (ev.shiftKey ? 1 : 0) | (ev.altKey ? 2 : 0) | (ev.ctrlKey ? 4 : 0) | (ev.metaKey ? 8 : 0);
@@ -267,7 +267,7 @@ export class WriteBuffer extends Disposable {
         * this condition here, also the renderer has no way to spot nonsense updates either.
         * FIXME: A proper fix for this would track the FPS at the renderer entry level separately.
         *
-         * If favoring of FPS shows bad throughput impact, use the following instead. It favors
+         * If favoring of FPS shows bad throughtput impact, use the following instead. It favors
         * throughput by eval'ing `startTime` upfront pulling at least one more chunk into the
         * current microtask queue (executed before setTimeout).
         */
@@ -21,7 +21,7 @@ function identifier(id: IFunctionIdentifier): number {
      throw new Error('only one byte as prefix supported');
    }
    res = id.prefix.charCodeAt(0);
-    if (res < 0x3c || res > 0x3f) {
+    if (res && 0x3c > res || res > 0x3f) {
      throw new Error('prefix must be in range 0x3c .. 0x3f');
    }
  }
@@ -459,29 +459,4 @@ describe('ApcParser - async tests', () => {
      });
    });
  });
-  describe('reset', () => {
-    it('should abort active handlers with end(false) when reset during payload', () => {
-      const ident = identifier({ intermediates: '+', final: 'p' });
-      parser.registerHandler(ident, new TestHandler(reports, 'th'));
-      parser.start(ident);
-      let data = toUtf32('partial');
-      parser.put(data, 0, data.length);
-      parser.reset();
-      assert.deepEqual(reports, [
-        ['th', 'START'],
-        ['th', 'PUT', 'partial'],
-        ['th', 'END', false]
-      ]);
-      reports.length = 0;
-      parser.start(ident);
-      data = toUtf32('complete');
-      parser.put(data, 0, data.length);
-      parser.end(true);
-      assert.deepEqual(reports, [
-        ['th', 'START'],
-        ['th', 'PUT', 'complete'],
-        ['th', 'END', true]
-      ]);
-    });
-  });
 });
@@ -22,7 +22,7 @@ function identifier(id: IFunctionIdentifier): number {
      throw new Error('only one byte as prefix supported');
    }
    res = id.prefix.charCodeAt(0);
-    if (res < 0x3c || res > 0x3f) {
+    if (res && 0x3c > res || res > 0x3f) {
      throw new Error('prefix must be in range 0x3c .. 0x3f');
    }
  }
@@ -470,30 +470,4 @@ describe('DcsParser - async tests', () => {
      });
    });
  });
-  describe('reset', () => {
-    it('should abort active handlers with unhook(false) when reset during payload', () => {
-      const ident = identifier({ intermediates: '+', final: 'p' });
-      const params = Params.fromArray([1, 2, 3]);
-      parser.registerHandler(ident, new TestHandler(reports, 'th'));
-      parser.hook(ident, params);
-      let data = toUtf32('partial');
-      parser.put(data, 0, data.length);
-      parser.reset();
-      assert.deepEqual(reports, [
-        ['th', 'HOOK', [1, 2, 3]],
-        ['th', 'PUT', 'partial'],
-        ['th', 'UNHOOK', false]
-      ]);
-      reports.length = 0;
-      parser.hook(ident, params);
-      data = toUtf32('complete');
-      parser.put(data, 0, data.length);
-      parser.unhook(true);
-      assert.deepEqual(reports, [
-        ['th', 'HOOK', [1, 2, 3]],
-        ['th', 'PUT', 'complete'],
-        ['th', 'UNHOOK', true]
-      ]);
-    });
-  });
 });
@@ -243,7 +243,7 @@ describe('EscapeSequenceParser', () => {
      p = new TestEscapeSequenceParser(tansitions);
      assert.deepEqual(p.transitions, tansitions);
    });
-    it('initial states', () => {
+    it('inital states', () => {
      assert.equal(parser.initialState, ParserState.GROUND);
      assert.equal(parser.currentState, ParserState.GROUND);
      assert.equal(parser.osc, '');
@@ -2177,38 +2177,6 @@ describe('EscapeSequenceParser - async', () => {
      parser.reset();
      await parseP(parser, INPUT); // does not throw anymore
    });
-    it('reset during async pause continues at next codepoint in chunk', async () => {
-      const localParser = new TestEscapeSequenceParser();
-      const localStack: any[] = [];
-      localParser.setPrintHandler((data, start, end) => {
-        let result = '';
-        for (let i = start; i < end; ++i) {
-          result += stringFromCodePoint(data[i]);
-        }
-        localStack.push(['PRINT', result]);
-      });
-      localParser.registerCsiHandler({ final: 'm' }, async params => {
-        localStack.push(['SGR', params.toArray()]);
-        return new Promise<boolean>(resolve => setTimeout(() => resolve(true), 0));
-      });
-      const data = '\x1b[1mXY';
-      const container = new Uint32Array(data.length);
-      const decoder = new StringToUtf32();
-      const len = decoder.decode(data, container);
-
-      assert.ok(localParser.parse(container, len) instanceof Promise);
-      localParser.reset();
-      assert.equal(localParser.parseStack.state, ParserStackType.RESET);
-
-      let prev: boolean | undefined = true;
-      let result: void | Promise<boolean> | undefined;
-      while (result = localParser.parse(container, len, prev)) {
-        prev = await result;
-      }
-
-      assert.deepEqual(localStack, [['SGR', [1]], ['PRINT', 'XY']]);
-      assert.equal(localParser.parseStack.state, ParserStackType.NONE);
-    });
    it('correct result on awaited parse call', async () => {
      await parseP(parser, INPUT);
      assert.deepEqual(callstack, RESULT);
@@ -18,7 +18,7 @@ import { ApcParser } from './ApcParser';
 // @vt: #Y   ESC   CSI   "Control Sequence Introducer"   "ESC ["   "Start of a CSI sequence."
 // @vt: #Y   ESC   OSC   "Operating System Command"      "ESC ]"   "Start of an OSC sequence."
 // @vt: #Y   ESC   DCS   "Device Control String"         "ESC P"   "Start of a DCS sequence."
-// @vt: #Y   ESC   ST    "String Terminator"             "ESC \\"  "Terminator used for string type sequences."
+// @vt: #Y   ESC   ST    "String Terminator"             "ESC \"   "Terminator used for string type sequences."
 // @vt: #Y   ESC   PM    "Privacy Message"               "ESC ^"   "Start of a privacy message."
 // @vt: #Y   ESC   APC   "Application Program Command"   "ESC _"   "Start of an APC sequence."
 // @vt: #Y   C1    CSI   "Control Sequence Introducer"   "\x9B"    "Start of a CSI sequence."
@@ -243,7 +243,7 @@ export const VT500_TRANSITION_TABLE = (function (): TransitionTable {
 *
 * This parser is currently hardcoded to operate in ZDM (Zero Default Mode)
 * as suggested by the original parser, thus empty parameters are set to 0.
- * This is not in line with the latest ECMA-48 specification
+ * This this is not in line with the latest ECMA-48 specification
 * (ZDM was part of the early specs and got completely removed later on).
 *
 * Other than the original parser from vt100.net this parser supports
@@ -342,7 +342,7 @@ export class EscapeSequenceParser extends Disposable implements IEscapeSequenceP
        throw new Error('only one byte as prefix supported');
      }
      res = id.prefix.charCodeAt(0);
-      if (res < 0x3c || res > 0x3f) {
+      if (res && 0x3c > res || res > 0x3f) {
        throw new Error('prefix must be in range 0x3c .. 0x3f');
      }
    }
@@ -464,28 +464,4 @@ describe('OscParser - async tests', () => {
      });
    });
  });
-  describe('reset', () => {
-    it('should abort active handlers with end(false) when reset during payload', () => {
-      parser.registerHandler(1234, new TestHandler(1234, reports, 'th'));
-      parser.start();
-      let data = toUtf32('1234;partial');
-      parser.put(data, 0, data.length);
-      parser.reset();
-      assert.deepEqual(reports, [
-        ['th', 1234, 'START'],
-        ['th', 1234, 'PUT', 'partial'],
-        ['th', 1234, 'END', false]
-      ]);
-      reports.length = 0;
-      parser.start();
-      data = toUtf32('1234;complete');
-      parser.put(data, 0, data.length);
-      parser.end(true);
-      assert.deepEqual(reports, [
-        ['th', 1234, 'START'],
-        ['th', 1234, 'PUT', 'complete'],
-        ['th', 1234, 'END', true]
-      ]);
-    });
-  });
 });
@@ -199,13 +199,13 @@ describe('Params', () => {
    it('reject params lesser -1', () => {
      const params = new Params();
      params.addParam(-1);
-      assert.throws(() => params.addParam(-2), 'values less than -1 are not allowed');
+      assert.throws(() => params.addParam(-2), 'values lesser than -1 are not allowed');
    });
    it('reject subparams lesser -1', () => {
      const params = new Params();
      params.addParam(-1);
      params.addSubParam(-1);
-      assert.throws(() => params.addSubParam(-2), 'values less than -1 are not allowed');
+      assert.throws(() => params.addSubParam(-2), 'values lesser than -1 are not allowed');
      assert.deepEqual(params.toArray(), [-1, [-1]]);
    });
    it('clamp parsed params', () => {
@@ -162,7 +162,7 @@ export class Params implements IParams {
      return;
    }
    if (value < -1) {
-      throw new Error('values less than -1 are not allowed');
+      throw new Error('values lesser than -1 are not allowed');
    }
    this._subParamsIdx[this.length] = this._subParamsLength << 8 | this._subParamsLength;
    this.params[this.length++] = value > Constants.MAX_VALUE ? Constants.MAX_VALUE : value;
@@ -172,7 +172,7 @@ export class Params implements IParams {
   * Add a sub parameter value.
   * The sub parameter is automatically associated with the last parameter value.
   * Thus it is not possible to add a subparameter without any parameter added yet.
-   * `Params` only stores up to `maxSubParamsLength` sub parameters, any later
+   * `Params` only stores up to `subParamsLength` sub parameters, any later
   * sub parameter will be ignored.
   */
  public addSubParam(value: number): void {
@@ -185,7 +185,7 @@ export class Params implements IParams {
      return;
    }
    if (value < -1) {
-      throw new Error('values less than -1 are not allowed');
+      throw new Error('values lesser than -1 are not allowed');
    }
    this._subParams[this._subParamsLength++] = value > Constants.MAX_VALUE ? Constants.MAX_VALUE : value;
    this._subParamsIdx[this.length - 1]++;
@@ -234,8 +234,8 @@ export interface IApcParser extends ISubParser<IApcHandler, ApcFallbackHandlerTy

 /**
 * Interface to denote a specific ESC, CSI or DCS handler slot.
- * The values are used to create an integer representation during handler
- * registration before passed to the subparsers as `ident`.
+ * The values are used to create an integer respresentation during handler
+ * regristation before passed to the subparsers as `ident`.
 * The integer translation is made to allow a faster handler access
 * in `EscapeSequenceParser.parse`.
 */
@@ -1,137 +0,0 @@
-/**
- * Copyright (c) 2026 The xterm.js authors. All rights reserved.
- * @license MIT
- */
-
-import { assert } from 'chai';
-import { AttributeData } from '../buffer/AttributeData';
-import { BufferService } from './BufferService';
-import { OptionsService } from './OptionsService';
-import { MockLogService } from '../TestUtils.test';
-
-describe('BufferService', () => {
-  describe('scroll', () => {
-    const eraseAttr = new AttributeData();
-
-    it('should decrement ydisp when the buffer is full and the user has scrolled up', () => {
-      const optionsService = new OptionsService({ rows: 3, cols: 10, scrollback: 2 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-
-      while (!buffer.lines.isFull) {
-        bufferService.scroll(eraseAttr);
-      }
-      assert.strictEqual(buffer.lines.length, 5);
-
-      bufferService.isUserScrolling = true;
-      buffer.ydisp = 2;
-      const ybaseBefore = buffer.ybase;
-
-      bufferService.scroll(eraseAttr);
-
-      assert.strictEqual(buffer.ybase, ybaseBefore);
-      assert.strictEqual(buffer.ydisp, 1);
-    });
-
-    it('should not advance ydisp with ybase while the user has scrolled up and the buffer is not full', () => {
-      const optionsService = new OptionsService({ rows: 3, cols: 10, scrollback: 2 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-
-      bufferService.isUserScrolling = true;
-      buffer.ydisp = 0;
-      const ybaseBefore = buffer.ybase;
-
-      bufferService.scroll(eraseAttr);
-
-      assert.strictEqual(buffer.ybase, ybaseBefore + 1);
-      assert.strictEqual(buffer.ydisp, 0);
-    });
-
-    it('should follow ybase with ydisp when the user is not scrolling', () => {
-      const optionsService = new OptionsService({ rows: 3, cols: 10, scrollback: 2 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-
-      while (!buffer.lines.isFull) {
-        bufferService.scroll(eraseAttr);
-      }
-
-      bufferService.isUserScrolling = false;
-      bufferService.scroll(eraseAttr);
-
-      assert.strictEqual(buffer.ydisp, buffer.ybase);
-    });
-
-    it('should scroll within DECSTBM margins without affecting lines outside the region', () => {
-      const optionsService = new OptionsService({ rows: 5, cols: 10, scrollback: 10 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-
-      const markRow = (row: number, ch: string) => {
-        buffer.lines.get(buffer.ybase + row)!.setCellFromCodepoint(0, ch.charCodeAt(0), 1, eraseAttr);
-      };
-      markRow(0, 'A');
-      markRow(1, 'B');
-      markRow(2, 'C');
-      markRow(3, 'D');
-      markRow(4, 'E');
-      buffer.scrollTop = 1;
-      buffer.scrollBottom = 3;
-
-      bufferService.scroll(eraseAttr);
-
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 0)!.translateToString().trim(), 'A');
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 1)!.translateToString().trim(), 'C');
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 2)!.translateToString().trim(), 'D');
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 3)!.translateToString(true).trim(), '');
-      assert.strictEqual(buffer.lines.get(buffer.ybase + 4)!.translateToString().trim(), 'E');
-    });
-  });
-
-  describe('scrollLines', () => {
-    it('should move ydisp and set isUserScrolling when scrolling up', () => {
-      const optionsService = new OptionsService({ rows: 10, cols: 80, scrollback: 10 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-      buffer.ybase = 5;
-      buffer.ydisp = 5;
-
-      let scrollEvent: number | undefined;
-      bufferService.onScroll(e => { scrollEvent = e; });
-
-      bufferService.scrollLines(-2);
-
-      assert.strictEqual(buffer.ydisp, 3);
-      assert.strictEqual(bufferService.isUserScrolling, true);
-      assert.strictEqual(scrollEvent, 3);
-    });
-
-    it('should not scroll above the top of the buffer', () => {
-      const optionsService = new OptionsService({ rows: 10, cols: 80, scrollback: 10 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-      buffer.ybase = 5;
-      buffer.ydisp = 0;
-
-      bufferService.scrollLines(-1);
-
-      assert.strictEqual(buffer.ydisp, 0);
-      assert.strictEqual(bufferService.isUserScrolling, false);
-    });
-
-    it('should clear isUserScrolling when scrolling to the bottom', () => {
-      const optionsService = new OptionsService({ rows: 10, cols: 80, scrollback: 10 });
-      const bufferService = new BufferService(optionsService, new MockLogService());
-      const buffer = bufferService.buffer;
-      buffer.ybase = 5;
-      buffer.ydisp = 2;
-      bufferService.isUserScrolling = true;
-
-      bufferService.scrollLines(10);
-
-      assert.strictEqual(buffer.ydisp, 5);
-      assert.strictEqual(bufferService.isUserScrolling, false);
-    });
-  });
-});
@@ -1,43 +0,0 @@
-/**
- * Copyright (c) 2026 The xterm.js authors. All rights reserved.
- * @license MIT
- */
-
-import { assert } from 'chai';
-import { CHARSETS } from '../data/Charsets';
-import { CharsetService } from './CharsetService';
-
-describe('CharsetService', () => {
-  let service: CharsetService;
-
-  beforeEach(() => {
-    service = new CharsetService();
-  });
-
-  it('should not update active charset when designating an inactive glevel', () => {
-    service.setgCharset(1, CHARSETS['0']);
-    assert.strictEqual(service.glevel, 0);
-    assert.ok(service.charset === undefined);
-  });
-
-  it('should expose the designated charset after setgLevel', () => {
-    service.setgCharset(1, CHARSETS['0']);
-    service.setgLevel(1);
-    assert.strictEqual(service.charset, CHARSETS['0']);
-  });
-
-  it('should update active charset when designating the current glevel', () => {
-    service.setgLevel(1);
-    service.setgCharset(1, CHARSETS['0']);
-    assert.strictEqual(service.charset, CHARSETS['0']);
-  });
-
-  it('should reset glevel, charsets, and active charset', () => {
-    service.setgCharset(1, CHARSETS['0']);
-    service.setgLevel(1);
-    service.reset();
-    assert.strictEqual(service.glevel, 0);
-    assert.deepEqual(service.charsets, []);
-    assert.ok(service.charset === undefined);
-  });
-});
@@ -344,13 +344,13 @@ export type UnicodeCharWidth = 0 | 1 | 2;
 export const IUnicodeService = createDecorator<IUnicodeService>('UnicodeService');
 export interface IUnicodeService {
  serviceBrand: undefined;
-  /** Register a Unicode version provider. */
+  /** Register an Unicode version provider. */
  register(provider: IUnicodeVersionProvider): void;
  /** Registered Unicode versions. */
  readonly versions: string[];
  /** Currently active version. */
  activeVersion: string;
-  /** Event triggered when the active version changes. */
+  /** Event triggered, when activate version changed. */
  readonly onChange: IEvent<string>;

  /**
@@ -4,8 +4,8 @@
 */

 import { perfContext, before, RuntimeCase } from 'xterm-benchmark';
-import { DecorationService } from 'common/services/DecorationService';
-import { MockLogService, MockBufferService, MockOptionsService } from 'common/TestUtils.test';
+import { DecorationService } from '../../src/common/services/DecorationService';
+import { MockLogService, MockBufferService, MockOptionsService } from '../../src/common/TestUtils.test';
 const enum Constants {
  COLS = 80,
  ROWS = 30,
@@ -4,12 +4,12 @@
 */
 import { perfContext, before, beforeEach, ThroughputRuntimeCase } from 'xterm-benchmark';

-import { EscapeSequenceParser } from 'common/parser/EscapeSequenceParser';
-import { C0, C1 } from 'common/data/EscapeSequences';
-import { IDcsHandler, IOscHandler, IApcHandler, IParams } from 'common/parser/Types';
-import { OscHandler } from 'common/parser/OscParser';
-import { DcsHandler } from 'common/parser/DcsParser';
-import { ApcHandler } from 'common/parser/ApcParser';
+import { EscapeSequenceParser } from '../../src/common/parser/EscapeSequenceParser';
+import { C0, C1 } from '../../src/common/data/EscapeSequences';
+import { IDcsHandler, IOscHandler, IApcHandler, IParams } from '../../src/common/parser/Types';
+import { OscHandler } from '../../src/common/parser/OscParser';
+import { DcsHandler } from '../../src/common/parser/DcsParser';
+import { ApcHandler } from '../../src/common/parser/ApcParser';

 const SIZE = 5000000;

@@ -4,7 +4,7 @@
 */

 import { perfContext, before, RuntimeCase } from 'xterm-benchmark';
-import { Emitter } from 'common/Event';
+import { Emitter } from '../../src/common/Event';

 const ITERATIONS = 1_000_000;

@@ -6,8 +6,8 @@
 import { perfContext, before, ThroughputRuntimeCase } from 'xterm-benchmark';

 import { spawn } from 'node-pty';
-import { Utf8ToUtf32, stringFromCodePoint } from 'common/input/TextDecoder';
-import { CoreBrowserTerminal } from 'browser/CoreBrowserTerminal';
+import { Utf8ToUtf32, stringFromCodePoint } from '../../src/common/input/TextDecoder';
+import { CoreBrowserTerminal } from '../../src/browser/CoreBrowserTerminal';

 perfContext('Terminal: ls -lR /usr/lib', () => {
  let content = '';
@@ -11,11 +11,7 @@
    "moduleResolution": "nodenext",
    "strict": false,
    "target": "es2021",
-    "module": "nodenext",
-    "paths": {
-      "common/*": ["../../src/common/*"],
-      "browser/*": ["../../src/browser/*"]
-    }
+    "module": "nodenext"
  },
  "include": [
    "./**/*",
@@ -980,7 +980,7 @@ export function injectSharedRendererTests(ctx: ISharedRendererTestContext): void
  });

  test.describe('selectionInactiveBackground', async () => {
-    test('should render the inactive selection when not focused', async () => {
+    test('should render the the inactive selection when not focused', async () => {
      const theme: ITheme = {
        selectionBackground: '#FF000080',
        selectionInactiveBackground: '#0000FF80'
Author	SHA1	Message	Date
Daniel ImmsandCursor	fd6a339bcb	fix(browser): place imports before module const in TimeBasedDebouncer RENDER_DEBOUNCE_THRESHOLD_MS was declared above the import, which breaks ES module conventions and can confuse tooling. Move the import to the top of the module after the license header. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:28:12 +00:00
Daniel ImmsandCursor	a2552053f5	fix(browser): clear idle blink timer when handle is 0 _clearIdleTimer() used truthiness on _idleTimeout, so a setTimeout handle of 0 would not be cleared. Compare against undefined, consistent with RenderDebouncer rAF handling. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:28:12 +00:00
Daniel ImmsandCursor	f3783729aa	fix(browser): treat rAF id 0 as scheduled in RenderDebouncer dispose(), addRefreshCallback(), and refresh() used truthiness on _animationFrame, so a requestAnimationFrame handle of 0 would not be cancelled or deduplicated. Compare against undefined instead, matching other timer checks in the browser layer. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:28:12 +00:00
Daniel ImmsandCursor	a97b77af3f	fix(browser): cancel trailing debounce timeout on immediate refresh TimeBasedDebouncer can schedule a trailing setTimeout when refresh requests are throttled. If the debounce window later elapses and an immediate refresh runs, that pending timeout was left armed and could fire afterward, causing a duplicate _innerRefresh (e.g. extra live region updates for screen readers). Clear any pending timeout and reset _additionalRefreshRequested before running the immediate refresh path. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:28:11 +00:00
Daniel ImmsandCursor	275d6d8a80	fix(browser): reset TimeBasedDebouncer state on dispose dispose() cleared the pending timeout but left _refreshTimeoutID and _additionalRefreshRequested set. After dispose, a stale timeout id or trailing-refresh flag could confuse a new instance or tests. Match RenderDebouncer by clearing the timeout id and reset the trailing flag. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:27:58 +00:00
Daniel ImmsandCursor	8679eba407	refactor(browser): remove unreachable coords guard in getCoords getCoordsRelativeToElement always returns a two-element tuple, so the array is always truthy and if (!coords) could never run. hasValidCharSize already handles the only early-exit path before coordinates are computed. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:27:58 +00:00
Daniel ImmsandCursor	3960db39a9	fix(browser): parenthesize document fallback for CoreBrowserService Nullish coalescing binds tighter than the conditional operator, so the previous expression was parsed as (this._document ?? windowDefined) ? window.document : null. When this._document was set (e.g. parent ownerDocument or documentOverride), the ternary still returned window.document instead of the configured document. CoreBrowserService.mainDocument is used by AccessibilityManager, viewport styles, and decoration renderers; CharSizeService already received this._document directly. Add parentheses so the fallback applies only when _document is nullish. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:27:58 +00:00
Daniel ImmsandCursor	ff8a0d9c99	fix(browser): use startY in horizontalDirection row helper call horizontalDirection called moveToRequestedRow with targetX as the first argument, but that parameter is the starting row (startY). resetStartingRow and moveToRequestedCol already pass startY correctly; only this copy-paste mistake passed a column index where a row was expected. That could miscompute wrapped-row offsets and emit the wrong horizontal arrow key sequence when building selection/cursor movement strings. Co-authored-by: Cursor <cursoragent@cursor.com>	2026-06-01 15:27:58 +00:00