chore: performance improvements (#2319)

webapp/packages/core-blocks/src/Tree/TreeNode/TreeNodeIcon.m.css

@@ -1,6 +1,7 @@
 .treeNodeIcon {
   position: relative;
   box-sizing: border-box;
+  pointer-events: none;
   flex-shrink: 0;
   width: 16px;
   height: 16px;

webapp/packages/core-blocks/src/Tree/TreeNode/TreeNodeNested.tsx

@@ -13,7 +13,6 @@ import { useS } from '../../useS';
 import style from './TreeNodeNested.m.css';
 
 interface Props extends React.PropsWithChildren {
-  expanded?: boolean;
   root?: boolean;
   className?: string;
 }

webapp/packages/core-ui/src/DragAndDrop/useDNDData.ts

@@ -68,7 +68,7 @@ export function useDNDData(context: IDataContextProvider, options: IOptions = {}
         }
       }
 
-      state.isDragging = monitor.isDragging();
+      state.isDragging = dragging;
     },
   }));
 

webapp/packages/core-utils/src/TempMap.ts

@@ -23,7 +23,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
     return 'TempMap';
   }
 
-  private readonly deleted: TKey[];
+  private readonly deleted: Map<TKey, boolean>;
   private readonly temp: Map<TKey, TValue>;
   private flushTask: NodeJS.Timeout | null;
   private readonly keysTemp: ICachedValueObject<TKey[]>;
@@ -33,7 +33,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
   constructor(private readonly target: Map<TKey, TValue>, private readonly onSync?: () => void) {
     this.temp = new Map();
     this.flushTask = null;
-    this.deleted = [];
+    this.deleted = new Map();
     this.keysTemp = cacheValue();
     this.entriesTemp = cacheValue();
     this.valuesTemp = cacheValue();
@@ -45,7 +45,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
   }
 
   isDeleted(key: TKey): boolean {
-    return this.deleted.includes(key);
+    return this.deleted.get(key) || false;
   }
 
   /**
@@ -56,7 +56,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
       clearTimeout(this.flushTask);
       this.flushTask = null;
     }
-    this.deleted.splice(0, this.deleted.length);
+    this.deleted.clear();
     this.temp.clear();
     this.keysTemp.invalidate();
     this.valuesTemp.invalidate();
@@ -65,7 +65,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
 
   delete(key: TKey): boolean {
     this.temp.delete(key);
-    this.deleted.push(key);
+    this.deleted.set(key, true);
     this.scheduleFlush();
     return this.has(key);
   }
@@ -105,10 +105,7 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
   set(key: TKey, value: TValue): this {
     this.temp.set(key, value);
 
-    const indexOfDeleted = this.deleted.indexOf(key);
-    if (indexOfDeleted !== -1) {
-      this.deleted.splice(indexOfDeleted, 1);
-    }
+    this.deleted.delete(key);
 
     this.scheduleFlush();
     return this;
@@ -139,10 +136,10 @@ export class TempMap<TKey, TValue> implements Map<TKey, TValue> {
 
     this.flushTask = setTimeout(
       action(() => {
-        for (const deleted of this.deleted) {
+        for (const [deleted] of this.deleted) {
           this.target.delete(deleted);
         }
-        this.deleted.splice(0, this.deleted.length);
+        this.deleted.clear();
 
         for (const [key, value] of this.temp) {
           this.target.set(key, value);

webapp/packages/plugin-navigation-tree/src/NavigationTree/ElementsTree/NavigationTreeNode/useNavigationNode.ts

@@ -9,15 +9,13 @@ import React, { useContext, useEffect, useRef } from 'react';
 
 import { getComputed, useExecutor, useObjectRef } from '@cloudbeaver/core-blocks';
 import { useService } from '@cloudbeaver/core-di';
-import { SyncExecutor } from '@cloudbeaver/core-executor';
 import { EObjectFeature, type NavNode, NavNodeInfoResource } from '@cloudbeaver/core-navigation-tree';
 import { resourceKeyList } from '@cloudbeaver/core-resource';
 import type { IDNDData } from '@cloudbeaver/core-ui';
 
 import { useChildren } from '../../../NodesManager/useChildren';
 import { useNode } from '../../../NodesManager/useNode';
 import { ElementsTreeContext } from '../ElementsTreeContext';
-import type { IElementsTreeAction } from '../IElementsTreeAction';
 import type { NavTreeControlComponent } from '../NavigationNodeComponent';
 import type { IElementsTree } from '../useElementsTree';
 
@@ -94,7 +92,7 @@ export function useNavigationNode(node: NavNode, path: string[]): INavigationNod
   }, []);
 
   useExecutor({
-    executor: contextRef.context?.tree.actions || new SyncExecutor<IElementsTreeAction>(),
+    executor: contextRef.context?.tree.actions,
     handlers: [
       function refreshRoot({ type, nodeId }) {
         if (type === 'show' && nodeId === node.id) {