[8.15] chore(NA): upgrade typescript into v5.1.6 (#191794)

# Backport This will backport the following commits from `main` to `8.15`: - [chore(NA): upgrade typescript into v5.1.6 (#186437)](#186437) --------- Co-authored-by: kibanamachine <[email protected]>
elastic · Aug 30, 2024 · bc6ebf2 · bc6ebf2
1 parent d34d807
commit bc6ebf2
Show file tree

Hide file tree

Showing 255 changed files with 1,067 additions and 206 deletions.
diff --git a/.buildkite/package-lock.json b/.buildkite/package-lock.json
diff --git a/.buildkite/package.json b/.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",

diff --git a/package.json b/package.json
@@ -94,7 +94,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": {
@@ -1429,7 +1430,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",
@@ -1748,8 +1748,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",

diff --git a/...s/core/http/core-http-router-server-internal/src/versioned_router/core_versioned_route.ts b/...s/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,

diff --git a/...s/core/notifications/core-notifications-browser-internal/src/toasts/global_toast_list.tsx b/...s/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,
 });

diff --git a/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_alert_delay.tsx b/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_alert_delay.tsx
@@ -45,6 +45,7 @@ export const RuleAlertDelay = (props: RuleAlertDelayProps) => {
   return (
     <EuiFormRow
       fullWidth
+      // @ts-expect-error upgrade typescript v5.1.6
       isInvalid={errors.alertDelay?.length > 0}
       error={errors.alertDelay}
       data-test-subj="alertDelay"
@@ -57,6 +58,7 @@ export const RuleAlertDelay = (props: RuleAlertDelayProps) => {
         name="alertDelay"
         data-test-subj="alertDelayInput"
         prepend={[ALERT_DELAY_TITLE_PREFIX]}
+        // @ts-expect-error upgrade typescript v5.1.6
         isInvalid={errors.alertDelay?.length > 0}
         append={ALERT_DELAY_TITLE_SUFFIX}
         onChange={onAlertDelayChange}

diff --git a/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_consumer_selection.tsx b/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_consumer_selection.tsx
@@ -33,6 +33,7 @@ type ComboBoxOption = EuiComboBoxOptionOption<RuleCreationValidConsumer>;
 export const RuleConsumerSelection = (props: RuleConsumerSelectionProps) => {
   const { consumers, selectedConsumer, errors = {}, onChange } = props;
 
+  // @ts-expect-error upgrade typescript v5.1.6
   const isInvalid = (errors.consumer?.length || 0) > 0;
 
   const validatedSelectedConsumer = useMemo(() => {

diff --git a/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_schedule.tsx b/packages/kbn-alerts-ui-shared/src/rule_form/rule_definition/rule_schedule.tsx
@@ -57,6 +57,7 @@ export interface RuleScheduleProps {
 export const RuleSchedule = (props: RuleScheduleProps) => {
   const { interval, minimumScheduleInterval, errors = {}, onChange } = props;
 
+  // @ts-expect-error upgrade typescript v5.1.6
   const hasIntervalError = errors.interval?.length > 0;
 
   const intervalNumber = getDurationNumberInItsUnit(interval);
@@ -99,6 +100,7 @@ export const RuleSchedule = (props: RuleScheduleProps) => {
       data-test-subj="ruleSchedule"
       display="rowCompressed"
       helpText={helpText}
+      // @ts-expect-error upgrade typescript v5.1.6
       isInvalid={errors.interval?.length > 0}
       error={errors.interval}
     >
@@ -107,6 +109,7 @@ export const RuleSchedule = (props: RuleScheduleProps) => {
           <EuiFieldNumber
             fullWidth
             prepend={[SCHEDULE_TITLE_PREFIX]}
+            // @ts-expect-error upgrade typescript v5.1.6
             isInvalid={errors.interval?.length > 0}
             value={intervalNumber}
             name="interval"

diff --git a/packages/kbn-alerts-ui-shared/src/rule_form/rule_details/rule_details.tsx b/packages/kbn-alerts-ui-shared/src/rule_form/rule_details/rule_details.tsx
@@ -85,6 +85,7 @@ export const RuleDetails = (props: RuleDetailsProps) => {
       <EuiFormRow
         fullWidth
         label={RULE_NAME_INPUT_TITLE}
+        // @ts-expect-error upgrade typescript v5.1.6
         isInvalid={errors.name?.length > 0}
         error={errors.name}
       >
@@ -98,6 +99,7 @@ export const RuleDetails = (props: RuleDetailsProps) => {
       <EuiFormRow
         fullWidth
         label={RULE_TAG_INPUT_TITLE}
+        // @ts-expect-error upgrade typescript v5.1.6
         isInvalid={errors.tags?.length > 0}
         error={errors.tags}
       >

diff --git a/packages/kbn-es-query/src/es_query/from_filters.ts b/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;
 };
 

diff --git a/packages/kbn-eslint-plugin-i18n/rules/formatted_message_should_start_with_the_right_id.ts b/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);

diff --git a/packages/kbn-eslint-plugin-i18n/rules/i18n_translate_should_start_with_the_right_id.ts b/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);

diff --git a/packages/kbn-eslint-plugin-i18n/rules/strings_should_be_translated_with_formatted_message.ts b/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);

diff --git a/packages/kbn-eslint-plugin-i18n/rules/strings_should_be_translated_with_i18n.ts b/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);

diff --git a/...ges/kbn-eslint-plugin-telemetry/rules/event_generating_elements_should_be_instrumented.ts b/...ges/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);

diff --git a/packages/kbn-esql-validation-autocomplete/src/definitions/functions.ts b/packages/kbn-esql-validation-autocomplete/src/definitions/functions.ts
@@ -1361,7 +1361,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',
@@ -1414,7 +1414,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',

diff --git a/packages/kbn-management/settings/components/field_row/description/description.tsx b/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
       />
     );

diff --git a/packages/kbn-repo-linter/lib/lint_target.ts b/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;
   }
 }

diff --git a/packages/kbn-telemetry-tools/src/tools/ts_program/ts_program.test.ts b/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",
                 },
               },

diff --git a/packages/kbn-telemetry-tools/src/tools/utils.ts b/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`

diff --git a/packages/kbn-user-profile-components/src/user_profiles_selectable.tsx b/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 (

diff --git a/src/plugins/bfetch/public/test_helpers/xhr.ts b/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() {

diff --git a/src/plugins/chart_expressions/expression_partition_vis/public/__stories__/shared/config.ts b/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: {