chore(NA): upgrade typescript into v5.1.6

.buildkite/package.json

@@ -26,8 +26,8 @@
     "chai": "^4.3.10",
     "mocha": "^10.3.0",
     "nock": "^12.0.2",
-    "ts-node": "^10.7.0",
-    "typescript": "^4.6.4"
+    "ts-node": "^10.9.2",
+    "typescript": "^5.1.6"
   },
   "imports": {
     "#pipeline-utils": "./pipeline-utils/index.ts",

package.json

@@ -91,7 +91,8 @@
     "**/react-intl/**/@types/react": "^17.0.45",
     "**/remark-parse/trim": "1.0.1",
     "**/sharp": "0.32.6",
-    "**/typescript": "4.9.5",
+    "**/typescript": "5.1.6",
+    "@storybook/react-docgen-typescript-plugin": "1.0.6--canary.9.cd77847.0",
     "globby/fast-glob": "^3.2.11"
   },
   "dependencies": {
@@ -1424,7 +1425,6 @@
     "@storybook/node-logger": "^6.5.16",
     "@storybook/preview-web": "^6.5.16",
     "@storybook/react": "^6.5.16",
-    "@storybook/react-docgen-typescript-plugin": "^1.0.1",
     "@storybook/testing-react": "^1.3.0",
     "@storybook/theming": "^6.5.16",
     "@testing-library/jest-dom": "^5.16.5",
@@ -1742,8 +1742,8 @@
     "tough-cookie": "^4.1.4",
     "tree-kill": "^1.2.2",
     "ts-morph": "^15.1.0",
-    "tsd": "^0.20.0",
-    "typescript": "4.9.5",
+    "tsd": "^0.31.1",
+    "typescript": "5.1.6",
     "url-loader": "^2.2.0",
     "val-loader": "^1.1.1",
     "vinyl-fs": "^4.0.0",

packages/core/http/core-http-router-server-internal/src/versioned_router/core_versioned_route.ts

@@ -93,6 +93,7 @@ export class CoreVersionedRoute implements VersionedRoute {
       {
         path: this.path,
         validate: passThroughValidation,
+        // @ts-expect-error upgrade typescript v5.1.6
         options: this.getRouteConfigOptions(),
       },
       this.requestHandler,

packages/core/notifications/core-notifications-browser-internal/src/toasts/global_toast_list.tsx

@@ -30,6 +30,7 @@ interface State {
 
 const convertToEui = (toast: ToastWithRichTitle): EuiToast => ({
   ...toast,
+  // @ts-expect-error upgrade typescript v5.1.6
   title: toast.title instanceof Function ? <MountWrapper mount={toast.title} /> : toast.title,
   text: toast.text instanceof Function ? <MountWrapper mount={toast.text} /> : toast.text,
 });

packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_consumer_selection.tsx

@@ -34,7 +34,6 @@ type ComboBoxOption = EuiComboBoxOptionOption<RuleCreationValidConsumer>;
 
 export const RuleConsumerSelection = (props: RuleConsumerSelectionProps) => {
   const { validConsumers } = props;
-
   const { multiConsumerSelection, baseErrors } = useRuleFormState();
 
   const dispatch = useRuleFormDispatch();

packages/kbn-es-query/src/es_query/from_filters.ts

@@ -36,6 +36,7 @@ const filterNegate = (reverse: boolean) => (filter: Filter) => {
  * @return {Object} the query version of that filter
  */
 const translateToQuery = (filter: Partial<Filter>): estypes.QueryDslQueryContainer => {
+  // @ts-expect-error upgrade typescript v5.1.6
   return filter.query || filter;
 };
 

packages/kbn-eslint-plugin-i18n/rules/formatted_message_should_start_with_the_right_id.ts

@@ -45,6 +45,7 @@ export const FormattedMessageShouldStartWithTheRightId: Rule.RuleModule = {
           idAttribute.value.value;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);

packages/kbn-eslint-plugin-i18n/rules/i18n_translate_should_start_with_the_right_id.ts

@@ -47,6 +47,7 @@ export const I18nTranslateShouldStartWithTheRightId: Rule.RuleModule = {
           node.arguments[0].value;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);

packages/kbn-eslint-plugin-i18n/rules/strings_should_be_translated_with_formatted_message.ts

@@ -40,6 +40,7 @@ export const StringsShouldBeTranslatedWithFormattedMessage: Rule.RuleModule = {
         if (intent === false) return;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);
@@ -107,6 +108,7 @@ export const StringsShouldBeTranslatedWithFormattedMessage: Rule.RuleModule = {
         if (intent === false) return;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);

packages/kbn-eslint-plugin-i18n/rules/strings_should_be_translated_with_i18n.ts

@@ -41,6 +41,7 @@ export const StringsShouldBeTranslatedWithI18n: Rule.RuleModule = {
         if (intent === false) return;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);
@@ -105,6 +106,7 @@ export const StringsShouldBeTranslatedWithI18n: Rule.RuleModule = {
         if (intent === false) return;
 
         const i18nAppId = getI18nIdentifierFromFilePath(filename, cwd);
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);

packages/kbn-eslint-plugin-telemetry/rules/event_generating_elements_should_be_instrumented.ts

@@ -53,6 +53,7 @@ export const EventGeneratingElementsShouldBeInstrumented: Rule.RuleModule = {
         }
 
         const hasDataTestSubjProp = checkNodeForExistingDataTestSubjProp(parent, () =>
+          // @ts-expect-error upgrade typescript v5.1.6
           sourceCode.getScope(node as TSNode)
         );
 
@@ -69,6 +70,7 @@ export const EventGeneratingElementsShouldBeInstrumented: Rule.RuleModule = {
         const appName = getAppName(fileName, cwd);
 
         // 2. Component name
+        // @ts-expect-error upgrade typescript v5.1.6
         const functionDeclaration = sourceCode.getScope(node as TSNode)
           .block as TSESTree.FunctionDeclaration;
         const functionName = getFunctionName(functionDeclaration);

packages/kbn-esql-validation-autocomplete/src/definitions/functions.ts

@@ -1393,7 +1393,7 @@ const logDefinition: FunctionDefinition = {
     // do not really care here about the base and field
     // just need to check both values are not negative
     for (const arg of fnDef.args) {
-      if (isLiteralItem(arg) && arg.value < 0) {
+      if (isLiteralItem(arg) && Number(arg.value) < 0) {
         messages.push({
           type: 'warning' as const,
           code: 'logOfNegativeValue',
@@ -1446,7 +1446,7 @@ const log10Definition: FunctionDefinition = {
     // do not really care here about the base and field
     // just need to check both values are not negative
     for (const arg of fnDef.args) {
-      if (isLiteralItem(arg) && arg.value < 0) {
+      if (isLiteralItem(arg) && Number(arg.value) < 0) {
         messages.push({
           type: 'warning' as const,
           code: 'logOfNegativeValue',

packages/kbn-management/settings/components/field_row/description/description.tsx

@@ -58,6 +58,7 @@ export const FieldDescription = <T extends SettingType>({
          * Justification for dangerouslySetInnerHTML:
          * Setting description may contain formatting and links to documentation.
          */
+        /* @ts-expect-error upgrade typescript v5.1.6 */
         dangerouslySetInnerHTML={{ __html: content || '' }} // eslint-disable-line react/no-danger
       />
     );

packages/kbn-repo-linter/lib/lint_target.ts

@@ -56,11 +56,11 @@ export abstract class LintTarget {
     return this instanceof TsProjectLintTarget;
   }
 
-  getTsProject() {
+  getTsProject(): any {
     return this.isPackage() || this.isTsProject() ? this.tsProject : undefined;
   }
 
-  getPkg() {
+  getPkg(): any {
     return this.isPackage() ? this.pkg : undefined;
   }
 }

packages/kbn-telemetry-tools/src/tools/ts_program/ts_program.test.ts

@@ -21,7 +21,7 @@ describe('createKibanaProgram', () => {
             "fetch": Object {
               "typeDescriptor": Object {
                 "locale": Object {
-                  "kind": 152,
+                  "kind": 154,
                   "type": "StringKeyword",
                 },
               },

packages/kbn-telemetry-tools/src/tools/utils.ts

@@ -145,8 +145,16 @@ export function getResolvedModuleSourceFile(
   originalSource: ts.SourceFile,
   program: ts.Program,
   importedModuleName: string
-) {
-  const resolvedModule = (originalSource as any).resolvedModules.get(importedModuleName);
+): ts.SourceFile {
+  // Resolve the module name to get the resolved module name result
+  const resolvedModuleNameResult = ts.resolveModuleName(
+    importedModuleName,
+    originalSource.fileName,
+    program.getCompilerOptions(),
+    ts.sys
+  );
+
+  const resolvedModule = resolvedModuleNameResult.resolvedModule;
   if (!resolvedModule) {
     throw new Error(
       `Import for [${importedModuleName}] in [${originalSource.fileName}] could not be resolved by TypeScript`

packages/kbn-user-profile-components/src/user_profiles_selectable.tsx

@@ -265,10 +265,7 @@ export const UserProfilesSelectable = <Option extends UserProfileWithAvatar | nu
     <EuiSelectable
       data-test-subj={props['data-test-subj']}
       options={displayedOptions}
-      // @ts-expect-error: Type of `nextOptions` in EuiSelectable does not match what's actually being passed back so need to manually override it
-      onChange={(
-        nextOptions: Array<EuiSelectableOption<{ data: Partial<UserProfileWithAvatar> }>>
-      ) => {
+      onChange={(nextOptions: SelectableOption[]) => {
         if (!onChange) {
           return;
         }
@@ -330,7 +327,7 @@ export const UserProfilesSelectable = <Option extends UserProfileWithAvatar | nu
       noMatchesMessage={noMatchesMessage}
       emptyMessage={emptyMessage}
       errorMessage={errorMessage}
-      renderOption={(option, searchValue) => {
+      renderOption={(option: SelectableOption, searchValue) => {
         if (option.user) {
           const displayName = getUserDisplayName(option.user);
           return (

src/plugins/bfetch/public/test_helpers/xhr.ts

@@ -13,10 +13,15 @@ export const mockXMLHttpRequest = (): {
   XMLHttpRequest: typeof window.XMLHttpRequest;
 } => {
   class MockXMLHttpRequest implements XMLHttpRequest {
+    // @ts-expect-error upgrade typescript v5.1.6
     DONE = 0;
+    // @ts-expect-error upgrade typescript v5.1.6
     HEADERS_RECEIVED = 0;
+    // @ts-expect-error upgrade typescript v5.1.6
     LOADING = 0;
+    // @ts-expect-error upgrade typescript v5.1.6
     OPENED = 0;
+    // @ts-expect-error upgrade typescript v5.1.6
     UNSENT = 0;
     abort = jest.fn();
     addEventListener = jest.fn();
@@ -52,6 +57,7 @@ export const mockXMLHttpRequest = (): {
   const xhr = new MockXMLHttpRequest();
 
   return {
+    // @ts-expect-error upgrade typescript v5.1.6
     xhr,
     XMLHttpRequest: class {
       constructor() {

src/plugins/chart_expressions/expression_partition_vis/public/__stories__/shared/config.ts

@@ -59,7 +59,7 @@ export const config: PartitionChartProps['visConfig'] = {
 export const pieConfig: PartitionVisParams = {
   ...config,
   isDonut: false,
-  emptySizeRatio: 0,
+  emptySizeRatio: 0.3,
   distinctColors: false,
   nestedLegend: false,
   dimensions: {

src/plugins/data/common/search/tabify/tabify.test.ts

@@ -188,9 +188,9 @@ describe('tabifyAggResponse Integration', () => {
       ) {
         expect(typeof row).toBe('object');
 
-        asserts.forEach((assert, i: number) => {
+        asserts.forEach((a, i: number) => {
           if (row[`col-${i}`]) {
-            assert(row[`col-${i}`]);
+            a(row[`col-${i}`]);
           }
         });
       }
@@ -214,7 +214,7 @@ describe('tabifyAggResponse Integration', () => {
       // check for something like an average bytes result
       function expectAvgBytes(val: string | number) {
         expect(typeof val).toBe('number');
-        expect(val === 0 || val > 1000).toBeDefined();
+        expect(val === 0 || +val > 1000).toBe(true);
       }
 
       test('for non-hierarchical vis', () => {

src/plugins/data/public/search/session/sessions_mgmt/lib/get_expiration_status.ts

@@ -43,7 +43,7 @@ export const getExpirationStatus = (config: SearchSessionsConfigSchema, expires:
     });
   }
 
-  if (durationToExpire.valueOf() > 0 && expiresInDays <= sufficientDays) {
+  if (durationToExpire.asMilliseconds() > 0 && expiresInDays <= sufficientDays) {
     return { toolTipContent, statusContent };
   }
 };

src/plugins/expressions/common/test_helpers/expression_functions/sum.ts

@@ -17,7 +17,12 @@ export const sum: ExpressionFunctionDefinition<'sum', unknown[], {}, ExpressionV
   fn: (values) => {
     return {
       type: 'num',
-      value: Array.isArray(values) ? values.reduce((a, b) => a + b) : values,
+      value: Array.isArray(values)
+        ? values
+            .map(Number)
+            .filter(isFinite)
+            .reduce((a, b) => a + b, 0)
+        : values,
     };
   },
 };

src/plugins/field_formats/common/constants/base_formatters.ts

@@ -42,6 +42,7 @@ export const baseFormatters: FieldFormatInstanceType[] = [
   StaticLookupFormat,
   StringFormat,
   TruncateFormat,
+  // @ts-expect-error upgrade typescript v5.1.6
   UrlFormat,
   HistogramFormat,
 ];

src/plugins/field_formats/common/converters/color.tsx

@@ -46,6 +46,7 @@ export class ColorFormat extends FieldFormat {
         return findLast(this.param('colors'), ({ range }) => {
           if (!range) return;
           const [start, end] = range.split(':');
+          // @ts-expect-error upgrade typescript v5.1.6
           return val >= Number(start) && val <= Number(end);
         });
 

src/plugins/field_formats/common/utils/geo_utils.ts

@@ -12,6 +12,7 @@ import { i18n } from '@kbn/i18n';
 export const converter = new usng.Converter();
 
 export function withinRange(value: string | number, min: number, max: number) {
+  // @ts-expect-error upgrade typescript v5.1.6
   const isInvalid = value === '' || value > max || value < min;
   const error = isInvalid
     ? i18n.translate('fieldFormats.geoUtils.outOfRangeErrorMsg', {