Fix linter

Author: gohabereg

packages/collaboration-manager/src/CollaborationManager.ts

@@ -5,8 +5,7 @@ import {
   EventType,
   type ModelEvents,
   TextAddedEvent,
-  TextFormattedEvent, type TextFormattedEventData,
-  TextRemovedEvent,
+  TextFormattedEvent, TextRemovedEvent,
   TextUnformattedEvent
 } from '@editorjs/model';
 import { Operation, OperationType } from './Operation.js';

packages/collaboration-manager/src/Operation.spec.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-magic-numbers */
 import type { BlockNodeSerialized, DataKey, DocumentIndex } from '@editorjs/model';
 import { IndexBuilder } from '@editorjs/model';
-import { describe } from "@jest/globals";
+import { describe } from '@jest/globals';
 import { type InsertOrDeleteOperationData, type ModifyOperationData, Operation, OperationType } from './Operation.js';
 
 const createOperation = (
@@ -18,7 +18,7 @@ const createOperation = (
   if (Array.isArray(value)) {
     index.addBlockIndex(startIndex);
   } else {
-    index.addDataKey('text' as DataKey).addTextRange([ startIndex, startIndex ]);
+    index.addDataKey('text' as DataKey).addTextRange([startIndex, startIndex]);
   }
 
   const data: InsertOrDeleteOperationData | ModifyOperationData = {
@@ -33,7 +33,7 @@ const createOperation = (
   return new Operation(
     type,
     index.build(),
-    data,
+    data
   );
 };
 
@@ -102,15 +102,15 @@ describe('Operation', () => {
         const localOp = createOperation(OperationType.Insert, 0, 'abc');
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 6, 6 ]);
+        expect(transformedOp.index.textRange).toEqual([6, 6]);
       });
 
       it('should transform a received operation if it is at the same position as a local one', () => {
         const receivedOp = createOperation(OperationType.Modify, 0, 'abc');
         const localOp = createOperation(OperationType.Insert, 0, 'def');
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 3, 3 ]);
+        expect(transformedOp.index.textRange).toEqual([3, 3]);
       });
 
       it('should not change the text index if local op is a Block operation', () => {
@@ -121,7 +121,7 @@ describe('Operation', () => {
         } ]);
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 0, 0 ]);
+        expect(transformedOp.index.textRange).toEqual([0, 0]);
       });
 
       it('should not change the operation if local op is a Block operation after a received one', () => {
@@ -184,15 +184,15 @@ describe('Operation', () => {
         const localOp = createOperation(OperationType.Delete, 0, 'abc');
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 0, 0 ]);
+        expect(transformedOp.index.textRange).toEqual([0, 0]);
       });
 
       it('should transform a received operation if it is at the same position as a local one', () => {
         const receivedOp = createOperation(OperationType.Modify, 3, 'abc');
         const localOp = createOperation(OperationType.Delete, 3, 'def');
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 0, 0 ]);
+        expect(transformedOp.index.textRange).toEqual([0, 0]);
       });
 
       it('should not change the text index if local op is a Block operation', () => {
@@ -203,7 +203,7 @@ describe('Operation', () => {
         } ]);
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 1, 1 ]);
+        expect(transformedOp.index.textRange).toEqual([1, 1]);
       });
 
       it('should not change the text index if local op is a Block operation', () => {
@@ -214,7 +214,7 @@ describe('Operation', () => {
         } ]);
         const transformedOp = receivedOp.transform(localOp);
 
-        expect(transformedOp.index.textRange).toEqual([ 0, 0 ]);
+        expect(transformedOp.index.textRange).toEqual([0, 0]);
       });
 
       it('should not change the operation if local op is a Block operation after a received one', () => {
@@ -304,7 +304,8 @@ describe('Operation', () => {
       const op = createOperation(OperationType.Modify, 0, { bold: true }, { bold: false });
       const inverted = op.inverse();
 
-      expect(inverted.data).toEqual({ payload: { bold: false }, prevPayload: { bold: true } });
+      expect(inverted.data).toEqual({ payload: { bold: false },
+        prevPayload: { bold: true } });
     });
 
     it('should throw an error if unsupported operation type is provided', () => {

packages/collaboration-manager/src/Operation.ts

@@ -141,7 +141,7 @@ export class Operation<T extends OperationType = OperationType> {
           break;
         }
 
-        newIndexBuilder.addTextRange([ this.index.textRange![0] + payload.length, this.index.textRange![1] + payload.length ]);
+        newIndexBuilder.addTextRange([this.index.textRange![0] + payload.length, this.index.textRange![1] + payload.length]);
 
         break;
       }
@@ -155,7 +155,7 @@ export class Operation<T extends OperationType = OperationType> {
           break;
         }
 
-        newIndexBuilder.addTextRange([ this.index.textRange![0] - payload.length, this.index.textRange![1] - payload.length ]);
+        newIndexBuilder.addTextRange([this.index.textRange![0] - payload.length, this.index.textRange![1] - payload.length]);
 
         break;
       }
@@ -172,8 +172,9 @@ export class Operation<T extends OperationType = OperationType> {
   }
 
   /**
+   * Checks if operation needs to be transformed
    *
-   * @param indexToCompare
+   * @param indexToCompare - index of a relative operation
    */
   #shouldTransform(indexToCompare: Index): boolean {
     if (indexToCompare.isBlockIndex && this.index.blockIndex !== undefined) {

packages/model/src/entities/EditorDocument/EditorDocument.spec.ts

@@ -834,7 +834,7 @@ describe('EditorDocument', () => {
       const spy = jest.spyOn(document, 'insertText');
       const index = new IndexBuilder().addBlockIndex(blockIndex)
         .addDataKey(dataKey)
-        .addTextRange([ 0, 0 ])
+        .addTextRange([0, 0])
         .build();
 
       document.insertData(index, text);
@@ -879,7 +879,7 @@ describe('EditorDocument', () => {
       const index = new IndexBuilder()
         .addBlockIndex(blockIndex)
         .addDataKey(dataKey)
-        .addTextRange([ 0, rangeEnd ])
+        .addTextRange([0, rangeEnd])
         .build();
 
       document.removeData(index, 'hello');

packages/model/src/entities/EditorDocument/index.ts

@@ -49,9 +49,9 @@ export class EditorDocument extends EventBus {
    * @param [args.toolsRegistry] - ToolsRegistry instance for the current document. Defaults to a new ToolsRegistry instance.
    */
   constructor({
-                properties = {},
-                toolsRegistry = new ToolsRegistry(),
-              }: EditorDocumentConstructorParameters = {}) {
+    properties = {},
+    toolsRegistry = new ToolsRegistry(),
+  }: EditorDocumentConstructorParameters = {}) {
     super();
 
     this.#properties = properties;

packages/model/src/entities/Index/index.ts

@@ -2,18 +2,6 @@ import type { DocumentIndex, TextRange } from '../../EventBus/index.js';
 import type { DataKey } from '../BlockNode/index.js';
 import type { BlockTuneName } from '../BlockTune/index.js';
 
-interface IndexDelta {
-  /**
-   * The first number is a delta of the starts,
-   * the second number is a delta of the ends
-   */
-  textRangeDelta?: TextRange;
-
-  blockIndexDelta?: number;
-
-  isTheSameDataKey?: boolean;
-}
-
 /**
  * Class representing index in the document model tree
  */
@@ -64,7 +52,7 @@ export class Index {
     const index = new Index();
 
     for (const value of arrayIndex) {
-      const [ type, data ] = value.split('@');
+      const [type, data] = value.split('@');
 
       switch (type) {
         case 'doc':