fix most vis type errors due to ts 4.9.5 upgrade

src/dev/code_coverage/ingest_coverage/__tests__/mocks/team_assign_mock.txt

@@ -61,9 +61,6 @@ x-pack/plugins/lens/public/datatable_visualization/expression.tsx kibana-app
 x-pack/plugins/lens/public/datatable_visualization/index.ts kibana-app
 x-pack/plugins/lens/public/datatable_visualization/visualization.test.tsx kibana-app
 x-pack/plugins/lens/public/datatable_visualization/visualization.tsx kibana-app
-x-pack/plugins/lens/public/debounced_component/debounced_component.test.tsx kibana-app
-x-pack/plugins/lens/public/debounced_component/debounced_component.tsx kibana-app
-x-pack/plugins/lens/public/debounced_component/index.ts kibana-app
 x-pack/plugins/lens/public/drag_drop/drag_drop.test.tsx kibana-app
 x-pack/plugins/lens/public/drag_drop/drag_drop.tsx kibana-app
 x-pack/plugins/lens/public/drag_drop/index.ts kibana-app

src/plugins/chart_expressions/expression_legacy_metric/public/__stories__/metric_renderer.stories.tsx

@@ -10,7 +10,7 @@ import React from 'react';
 import { storiesOf } from '@storybook/react';
 import { from } from 'rxjs';
 import { ExpressionValueVisDimension } from '@kbn/visualizations-plugin/common';
-import { Datatable, DatatableColumn } from '@kbn/expressions-plugin/common';
+import { Datatable, DatatableColumn, TextAlignment } from '@kbn/expressions-plugin/common';
 import { Render } from '@kbn/presentation-util-plugin/public/__stories__';
 import { ColorMode, CustomPaletteState } from '@kbn/charts-plugin/common';
 import { getFormatService } from '../__mocks__/format_service';
@@ -209,8 +209,11 @@ storiesOf('renderers/visMetric', module)
               },
               labels: {
                 show: false,
-                // @ts-expect-error upgrade typescript v4.9.5
-                style: { spec: { fontSize: '60px', align: 'left' }, type: 'style', css: '' },
+                style: {
+                  spec: { fontSize: '60px', textAlign: TextAlignment.LEFT },
+                  type: 'style',
+                  css: '',
+                },
                 position: LabelPosition.TOP,
               },
             },

src/plugins/chart_expressions/expression_legacy_metric/public/components/with_auto_scale.tsx

@@ -63,10 +63,13 @@ function hasAutoscaleProps<T>(props: T): props is T & AutoScaleProps {
   return false;
 }
 
-function getWrappedComponentProps<T>(props: T) {
+function getWrappedComponentProps<T>(props: T): T {
   if (hasAutoscaleProps(props)) {
-    const { autoScaleParams, renderComplete, ...rest } = props;
-    return rest;
+    return {
+      ...props,
+      autoScaleParams: undefined,
+      renderComplete: undefined,
+    };
   }
 
   return props;
@@ -132,8 +135,7 @@ export function withAutoScale<T>(WrappedComponent: ComponentType<T>) {
               : {}),
           }}
         >
-          {/* @ts-expect-error upgrade typescript v4.9.5*/}
-          <WrappedComponent {...(restProps as T)} />
+          <WrappedComponent {...restProps} />
         </div>
       </div>
     );

src/plugins/expressions/common/execution/execution_contract.ts

@@ -17,17 +17,19 @@ import { ExpressionAstExpression } from '../ast';
  * `ExecutionContract` is a wrapper around `Execution` class. It provides the
  * same functionality but does not expose Expressions plugin internals.
  */
-export class ExecutionContract<Input = unknown, Output = unknown, InspectorAdapters = unknown> {
+export class ExecutionContract<
+  Input = unknown,
+  Output = unknown,
+  InspectorAdapters extends Adapters = object
+> {
   public get isPending(): boolean {
     const { state, result } = this.execution.state.get();
     const finished = state === 'error' || (state === 'result' && !result?.partial);
     return !finished;
   }
 
-  // @ts-expect-error upgrade typescript v4.9.5
   protected readonly execution: Execution<Input, Output, InspectorAdapters>;
 
-  // @ts-expect-error upgrade typescript v4.9.5
   constructor(execution: Execution<Input, Output, InspectorAdapters>) {
     this.execution = execution;
   }
@@ -82,6 +84,5 @@ export class ExecutionContract<Input = unknown, Output = unknown, InspectorAdapt
    * Get Inspector adapters provided to all functions of expression through
    * execution context.
    */
-  // @ts-expect-error upgrade typescript v4.9.5
   inspect = (): Adapters => this.execution.inspectorAdapters;
 }

src/plugins/vis_types/table/public/components/table_vis_basic.test.tsx

@@ -13,6 +13,7 @@ import { FormattedColumn, TableVisConfig, TableVisUiState } from '../types';
 import { DatatableColumn } from '@kbn/expressions-plugin/common';
 import { createTableVisCell } from './table_vis_cell';
 import { createGridColumns } from './table_vis_columns';
+import { EuiDataGridProps } from '@elastic/eui';
 
 jest.mock('./table_vis_columns', () => ({
   createGridColumns: jest.fn(() => []),
@@ -107,8 +108,8 @@ describe('TableVisBasic', () => {
       undefined
     );
 
-    // @ts-expect-error upgrade typescript v4.9.5
-    const { onSort } = comp.find('EuiDataGrid').prop('sorting');
+    const { onSort } = comp.find('EuiDataGrid').prop<EuiDataGridProps['sorting']>('sorting')!;
+
     // sort the first column
     onSort([{ id: 'first', direction: 'asc' }]);
     expect(uiStateProps.setSort).toHaveBeenCalledWith({ columnIndex: 0, direction: 'asc' });

src/plugins/vis_types/timeseries/public/convert_to_lens/lib/convert/percentile.ts

@@ -49,19 +49,20 @@ export const convertToPercentileColumn = (
     return null;
   }
   const commonColumnParams = createColumn(series, metric, field, { reducedTimeRange, timeShift });
+  const meta: PercentileColumn['meta'] =
+    index !== undefined
+      ? {
+          reference: `${metric.id}.${index}`,
+          ...commonColumnParams.meta,
+        }
+      : commonColumnParams.meta;
+
   return {
     operationType: 'percentile',
     sourceField: field.name,
     ...commonColumnParams,
     params: { ...params, ...getFormat(series) },
-    // @ts-expect-error upgrade typescript v4.9.5
-    meta:
-      index !== undefined
-        ? {
-            reference: `${metric.id}.${index}`,
-            ...commonColumnParams.meta,
-          }
-        : commonColumnParams.meta,
+    meta,
   };
 };
 

src/plugins/vis_types/timeseries/public/convert_to_lens/lib/convert/percentile_rank.ts

@@ -19,6 +19,7 @@ import {
   Column,
   PercentileRanksColumnWithExtendedMeta,
   CommonColumnConverterArgs,
+  PercentileColumn,
 } from './types';
 
 export const isPercentileRanksColumnWithMeta = (
@@ -56,19 +57,19 @@ export const convertToPercentileRankColumn = (
   }
 
   const commonColumnParams = createColumn(series, metric, field, { reducedTimeRange, timeShift });
+  const meta: PercentileColumn['meta'] =
+    index !== undefined
+      ? {
+          reference: `${metric.id}.${index}`,
+          ...commonColumnParams.meta,
+        }
+      : commonColumnParams.meta;
   return {
     operationType: 'percentile_rank',
     sourceField: field.name,
     ...commonColumnParams,
     params: { ...params, ...getFormat(series) },
-    // @ts-expect-error upgrade typescript v4.9.5
-    meta:
-      index !== undefined
-        ? {
-            reference: `${metric.id}.${index}`,
-            ...commonColumnParams.meta,
-          }
-        : commonColumnParams.meta,
+    meta,
   };
 };
 

src/plugins/visualizations/public/vis.ts

@@ -57,7 +57,7 @@ const getSearchSource = async (inputSearchSource: ISearchSource, savedSearchId?:
 
 type PartialVisState = Assign<SerializedVis, { data: Partial<SerializedVisData> }>;
 
-export class Vis<TVisParams = VisParams> {
+export class Vis<TVisParams extends VisParams = VisParams> {
   public readonly type: BaseVisType<TVisParams>;
   public readonly id?: string;
   public title: string = '';
@@ -69,7 +69,6 @@ export class Vis<TVisParams = VisParams> {
 
   constructor(visType: string, visState: SerializedVis<TVisParams> = {} as any) {
     this.type = this.getType(visType);
-    // @ts-expect-error upgrade typescript v4.9.5
     this.params = this.getParams(visState.params);
     this.uiState = new PersistedState(visState.uiState);
     this.id = visState.id;
@@ -192,7 +191,6 @@ export class Vis<TVisParams = VisParams> {
       title: this.title,
       description: this.description,
       type: this.type.name,
-      // @ts-expect-error upgrade typescript v4.9.5
       params: cloneDeep(this.params),
       uiState: this.uiState.toJSON(),
       data: {

src/plugins/visualizations/public/vis_async.ts

@@ -9,7 +9,7 @@
 import type { SerializedVis } from './vis';
 import type { VisParams } from '../common';
 
-export const createVisAsync = async <TVisParams = VisParams>(
+export const createVisAsync = async <TVisParams extends VisParams = VisParams>(
   visType: string,
   visState: SerializedVis<TVisParams> = {} as any
 ) => {
@@ -20,7 +20,6 @@ export const createVisAsync = async <TVisParams = VisParams>(
   const { Vis } = await import('./vis');
   const vis = new Vis(visType, visState);
 
-  // @ts-expect-error upgrade typescript v4.9.5
   await vis.setState(visState);
   return vis;
 };

x-pack/plugins/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel.test.tsx

@@ -18,11 +18,6 @@ import {
 } from '../../../mocks';
 
 import { mockDataPlugin, mountWithProvider } from '../../../mocks';
-jest.mock('../../../debounced_component', () => {
-  return {
-    debouncedComponent: (fn: unknown) => fn,
-  };
-});
 
 import { WorkspacePanel } from './workspace_panel';
 import { ReactWrapper } from 'enzyme';

x-pack/plugins/lens/public/state_management/lens_slice.ts

@@ -1005,15 +1005,13 @@ export const makeLensReducer = (storeDeps: LensStoreDeps) => {
         activeVisualization.removeLayer &&
         state.visualization.state
       ) {
-        const updater = layerIds.reduce(
+        const updater = layerIds.reduce<unknown>(
           (acc, layerId) =>
-            // @ts-expect-error upgrade typescript v4.9.5
             activeVisualization.removeLayer ? activeVisualization.removeLayer(acc, layerId) : acc,
           state.visualization.state
         );
 
         state.visualization.state =
-          // @ts-expect-error upgrade typescript v4.9.5
           typeof updater === 'function' ? updater(current(state.visualization.state)) : updater;
       }
 

x-pack/plugins/lens/server/migrations/common_migrations.ts

@@ -502,7 +502,11 @@ export const commonMigratePartitionChartGroups = (
   }>
 ): LensDocShape850<{
   shape: string;
-  layers: Array<{ primaryGroups?: string[]; secondaryGroups?: string[] }>;
+  layers: Array<{
+    primaryGroups?: string[];
+    secondaryGroups?: string[];
+    [key: string]: unknown; // unknown carryover key/values
+  }>;
 }> => {
   if (
     attributes.state.visualization?.layers &&
@@ -514,7 +518,6 @@ export const commonMigratePartitionChartGroups = (
         ...attributes.state,
         visualization: {
           ...attributes.state.visualization,
-          // @ts-expect-error upgrade typescript v4.9.5
           layers: attributes.state.visualization.layers.map((l) => {
             const groups = l.groups;