feat: remove unused exports, functions and files (#1750)

ydb-platform · Dec 12, 2024 · cdf9ebc · cdf9ebc
1 parent d40aea0
commit cdf9ebc
Show file tree

Hide file tree

Showing 65 changed files with 67 additions and 791 deletions.
diff --git a/package.json b/package.json
@@ -69,7 +69,7 @@
     "lint:styles": "stylelint 'src/**/*.{css,scss}'",
     "lint:other": "npm run prettier -- --check",
     "prettier": "prettier '**/*.{json,yaml,yml,md}'",
-    "unimported": "npx unimported --no-cache",
+    "unused": "npx knip --include dependencies,files,exports,types",
     "package": "rm -rf dist && tsc -p tsconfig.package.json && copyfiles -u 1 'src/**/*.{css,scss,svg}' dist",
     "test": "react-app-rewired test",
     "eject": "react-scripts eject",

diff --git a/src/components/Graph/Graph.tsx b/src/components/Graph/Graph.tsx
@@ -38,7 +38,7 @@ export function Graph<T>(props: GraphProps<T>) {
     return <div id={containerId} ref={containerRef} style={{overflow: 'auto'}} />;
 }
 
-export const renderExplainNode = (node: GraphNode): string => {
+const renderExplainNode = (node: GraphNode): string => {
     const parts = node.name.split('|');
     return parts.length > 1 ? parts[1] : node.name;
 };

diff --git a/src/components/GroupInfo/GroupInfo.tsx b/src/components/GroupInfo/GroupInfo.tsx
diff --git a/src/components/GroupInfo/i18n/en.json b/src/components/GroupInfo/i18n/en.json
diff --git a/src/components/GroupInfo/i18n/index.ts b/src/components/GroupInfo/i18n/index.ts
diff --git a/src/components/QuerySettingsDescription/index.ts b/src/components/QuerySettingsDescription/index.ts
diff --git a/src/components/Tablet/Tablet.scss b/src/components/Tablet/Tablet.scss
diff --git a/src/components/Tablet/Tablet.tsx b/src/components/Tablet/Tablet.tsx
diff --git a/src/components/Tablet/index.ts b/src/components/Tablet/index.ts
diff --git a/src/components/TabletsOverall/TabletsOverall.scss b/src/components/TabletsOverall/TabletsOverall.scss
diff --git a/src/components/TabletsOverall/TabletsOverall.tsx b/src/components/TabletsOverall/TabletsOverall.tsx
diff --git a/src/containers/ClusterModeGuard/index.ts b/src/containers/ClusterModeGuard/index.ts
diff --git a/src/containers/Node/Node.tsx b/src/containers/Node/Node.tsx
@@ -39,7 +39,7 @@ import './Node.scss';
 
 const b = cn('node');
 
-export const STORAGE_ROLE = 'Storage';
+const STORAGE_ROLE = 'Storage';
 
 interface NodeProps {
     additionalNodesProps?: AdditionalNodesProps;

diff --git a/src/containers/Nodes/columns/hooks.ts b/src/containers/Nodes/columns/hooks.ts
diff --git a/src/containers/Nodes/index.ts b/src/containers/Nodes/index.ts
diff --git a/src/containers/Operations/Operations.tsx b/src/containers/Operations/Operations.tsx
@@ -10,6 +10,7 @@ import {useAutoRefreshInterval} from '../../utils/hooks';
 
 import {OperationsControls} from './OperationsControls';
 import {getColumns} from './columns';
+import {OPERATIONS_SELECTED_COLUMNS_KEY} from './constants';
 import i18n from './i18n';
 import {b} from './shared';
 import {useOperationsQueryParams} from './useOperationsQueryParams';
@@ -62,6 +63,7 @@ export function Operations({database}: OperationsProps) {
                 {data ? (
                     <ResizeableDataTable
                         columns={getColumns({database, refreshTable: refetch})}
+                        columnsWidthLSKey={OPERATIONS_SELECTED_COLUMNS_KEY}
                         data={filteredOperations}
                         emptyDataMessage={i18n('title_empty')}
                     />

diff --git a/src/containers/PDiskPage/shared.ts b/src/containers/PDiskPage/shared.ts
diff --git a/src/containers/ReduxTooltip/ReduxTooltip.scss b/src/containers/ReduxTooltip/ReduxTooltip.scss
@@ -7,7 +7,6 @@
 }
 
 .node-tootltip,
-.tabletsOverall-tooltip,
 .histogram-tooltip {
     padding: 10px;
 
@@ -18,35 +17,6 @@
     }
 }
 
-.json-tooltip {
-    padding: 20px 20px 20px 0;
-
-    &__inspector {
-        @include mixins.json-tree-styles();
-
-        :not(.json-inspector__leaf_expanded).json-inspector__leaf_composite:before {
-            content: '';
-        }
-        .json-inspector__leaf_expanded.json-inspector__leaf_composite:before {
-            content: '';
-        }
-
-        & .json-inspector {
-            &__line:hover:after {
-                background: transparent;
-            }
-            &__show-original:hover:after,
-            &__show-original:hover:before {
-                color: transparent;
-            }
-
-            &__value_helper {
-                display: none;
-            }
-        }
-    }
-}
-
 .cell-tooltip {
     padding: 10px;