[8.x] Add consistent-type-imports eslint rule in `obs-ux-infra_serv…

…ices-team` owned plugins (#204549) (#206047) # Backport This will backport the following commits from `main` to `8.x`: - [Add `consistent-type-imports` eslint rule in `obs-ux-infra_services-team` owned plugins (#204549)](#204549)  ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport)
elastic · Jan 9, 2025 · 2ea9fed · 2ea9fed
1 parent 1adc1ae
commit 2ea9fed
Show file tree

Hide file tree

Showing 2,101 changed files with 5,297 additions and 5,164 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -1028,6 +1028,20 @@ module.exports = {
         ],
       },
     },
+    {
+      files: [
+        'x-pack/solutions/observability/plugins/apm/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/apm_data_access/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/infra/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/inventory/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/metrics_data_access/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/profiling/**/*.{js,mjs,ts,tsx}',
+        'x-pack/solutions/observability/plugins/profiling_data_access/**/*.{js,mjs,ts,tsx}',
+      ],
+      rules: {
+        '@typescript-eslint/consistent-type-imports': 'error',
+      },
+    },
 
     /**
      * Fleet overrides

diff --git a/...k/solutions/observability/plugins/apm/common/agent_configuration/configuration_types.d.ts b/...k/solutions/observability/plugins/apm/common/agent_configuration/configuration_types.d.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import t from 'io-ts';
-import { agentConfigurationIntakeRt } from './runtime_types/agent_configuration_intake_rt';
+import type t from 'io-ts';
+import type { agentConfigurationIntakeRt } from './runtime_types/agent_configuration_intake_rt';
 
 export type AgentConfigurationIntake = t.TypeOf<typeof agentConfigurationIntakeRt>;
 

diff --git a/...ity/plugins/apm/common/agent_configuration/runtime_types/agent_configuration_intake_rt.ts b/...ity/plugins/apm/common/agent_configuration/runtime_types/agent_configuration_intake_rt.ts
@@ -7,7 +7,7 @@
 
 import * as t from 'io-ts';
 import { settingDefinitions } from '../setting_definitions';
-import { SettingValidation } from '../setting_definitions/types';
+import type { SettingValidation } from '../setting_definitions/types';
 
 // retrieve validation from config definitions settings and validate on the server
 const knownSettings = settingDefinitions.reduce<Record<string, SettingValidation>>(

diff --git a/...lutions/observability/plugins/apm/common/agent_configuration/runtime_types/duration_rt.ts b/...lutions/observability/plugins/apm/common/agent_configuration/runtime_types/duration_rt.ts
@@ -7,8 +7,10 @@
 
 import * as t from 'io-ts';
 import { either } from 'fp-ts/lib/Either';
-import moment, { unitOfTime } from 'moment';
-import { amountAndUnitToObject, AmountAndUnit } from '../amount_and_unit';
+import type { unitOfTime } from 'moment';
+import moment from 'moment';
+import type { AmountAndUnit } from '../amount_and_unit';
+import { amountAndUnitToObject } from '../amount_and_unit';
 import { getRangeTypeMessage } from './get_range_type_message';
 
 function toMilliseconds({ amount, unit }: AmountAndUnit) {

diff --git a/...ervability/plugins/apm/common/agent_configuration/setting_definitions/general_settings.ts b/...ervability/plugins/apm/common/agent_configuration/setting_definitions/general_settings.ts
@@ -10,7 +10,7 @@ import { captureBodyRt } from '../runtime_types/capture_body_rt';
 import { logLevelRt } from '../runtime_types/log_level_rt';
 import { logEcsReformattingRt } from '../runtime_types/log_ecs_reformatting_rt';
 import { traceContinuationStrategyRt } from '../runtime_types/trace_continuation_strategy_rt';
-import { RawSettingDefinition } from './types';
+import type { RawSettingDefinition } from './types';
 
 export const generalSettings: RawSettingDefinition[] = [
   // API Request Size

diff --git a/...ns/observability/plugins/apm/common/agent_configuration/setting_definitions/index.test.ts b/...ns/observability/plugins/apm/common/agent_configuration/setting_definitions/index.test.ts
@@ -7,8 +7,8 @@
 
 import { omit } from 'lodash';
 import { filterByAgent, settingDefinitions } from '.';
-import { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
-import { SettingDefinition } from './types';
+import type { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
+import type { SettingDefinition } from './types';
 
 describe('filterByAgent', () => {
   describe('when `excludeAgents` is dotnet and nodejs', () => {

diff --git a/...lutions/observability/plugins/apm/common/agent_configuration/setting_definitions/index.ts b/...lutions/observability/plugins/apm/common/agent_configuration/setting_definitions/index.ts
@@ -9,13 +9,13 @@ import * as t from 'io-ts';
 import { sortBy } from 'lodash';
 import { isRight } from 'fp-ts/lib/Either';
 import { PathReporter } from 'io-ts/lib/PathReporter';
-import { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
+import type { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
 import { booleanRt } from '../runtime_types/boolean_rt';
 import { getIntegerRt } from '../runtime_types/integer_rt';
 import { isRumOrMobileAgentName } from '../../agent_name';
 import { floatThreeDecimalPlacesRt } from '../runtime_types/float_three_decimal_places_rt';
 import { floatFourDecimalPlacesRt } from '../runtime_types/float_four_decimal_places_rt';
-import { RawSettingDefinition, SettingDefinition } from './types';
+import type { RawSettingDefinition, SettingDefinition } from './types';
 import { generalSettings } from './general_settings';
 import { javaSettings } from './java_settings';
 import { mobileSettings } from './mobile_settings';

diff --git a/...observability/plugins/apm/common/agent_configuration/setting_definitions/java_settings.ts b/...observability/plugins/apm/common/agent_configuration/setting_definitions/java_settings.ts
@@ -6,7 +6,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { RawSettingDefinition } from './types';
+import type { RawSettingDefinition } from './types';
 
 export const javaSettings: RawSettingDefinition[] = [
   {

diff --git a/...servability/plugins/apm/common/agent_configuration/setting_definitions/mobile_settings.ts b/...servability/plugins/apm/common/agent_configuration/setting_definitions/mobile_settings.ts
@@ -6,7 +6,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { RawSettingDefinition } from './types';
+import type { RawSettingDefinition } from './types';
 
 export const mobileSettings: RawSettingDefinition[] = [
   // Session sample rate

diff --git a/...tions/observability/plugins/apm/common/agent_configuration/setting_definitions/types.d.ts b/...tions/observability/plugins/apm/common/agent_configuration/setting_definitions/types.d.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import * as t from 'io-ts';
-import { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
+import type * as t from 'io-ts';
+import type { AgentName } from '../../../typings/es_schemas/ui/fields/agent';
 
 // TODO: is it possible to get rid of `any`?
 export type SettingValidation = t.Type<any, string, unknown>;

diff --git a/x-pack/solutions/observability/plugins/apm/common/anomaly_detection/apm_ml_job.ts b/x-pack/solutions/observability/plugins/apm/common/anomaly_detection/apm_ml_job.ts
@@ -4,8 +4,8 @@
  * 2.0; you may not use this file except in compliance with the Elastic License
  * 2.0.
  */
-import { DATAFEED_STATE, JOB_STATE } from '@kbn/ml-plugin/common';
-import { Environment } from '../environment_rt';
+import type { DATAFEED_STATE, JOB_STATE } from '@kbn/ml-plugin/common';
+import type { Environment } from '../environment_rt';
 
 export interface ApmMlJob {
   environment: Environment;

diff --git a/...ity/plugins/apm/common/anomaly_detection/get_preferred_service_anomaly_timeseries.test.ts b/...ity/plugins/apm/common/anomaly_detection/get_preferred_service_anomaly_timeseries.test.ts
@@ -6,10 +6,10 @@
  */
 import { v4 as uuidv4 } from 'uuid';
 import { ENVIRONMENT_ALL } from '../environment_filter_values';
-import { Environment } from '../environment_rt';
+import type { Environment } from '../environment_rt';
 import { AnomalyDetectorType } from './apm_ml_detectors';
 import { getPreferredServiceAnomalyTimeseries } from './get_preferred_service_anomaly_timeseries';
-import { ServiceAnomalyTimeseries } from './service_anomaly_timeseries';
+import type { ServiceAnomalyTimeseries } from './service_anomaly_timeseries';
 
 const PROD = 'production' as Environment;
 const DEV = 'development' as Environment;

diff --git a/...vability/plugins/apm/common/anomaly_detection/get_preferred_service_anomaly_timeseries.ts b/...vability/plugins/apm/common/anomaly_detection/get_preferred_service_anomaly_timeseries.ts
@@ -5,9 +5,9 @@
  * 2.0.
  */
 
-import { Environment } from '../environment_rt';
-import { AnomalyDetectorType } from './apm_ml_detectors';
-import { ServiceAnomalyTimeseries } from './service_anomaly_timeseries';
+import type { Environment } from '../environment_rt';
+import type { AnomalyDetectorType } from './apm_ml_detectors';
+import type { ServiceAnomalyTimeseries } from './service_anomaly_timeseries';
 
 export function getPreferredServiceAnomalyTimeseries({
   preferredEnvironment,

diff --git a/x-pack/solutions/observability/plugins/apm/common/anomaly_detection/index.ts b/x-pack/solutions/observability/plugins/apm/common/anomaly_detection/index.ts
@@ -9,7 +9,7 @@ import { i18n } from '@kbn/i18n';
 import { getSeverityType } from '@kbn/ml-anomaly-utils/get_severity_type';
 import { getSeverityColor as mlGetSeverityColor } from '@kbn/ml-anomaly-utils/get_severity_color';
 import { ML_ANOMALY_SEVERITY } from '@kbn/ml-anomaly-utils/anomaly_severity';
-import { ServiceHealthStatus } from '../service_health_status';
+import type { ServiceHealthStatus } from '../service_health_status';
 
 export interface ServiceAnomalyStats {
   transactionType?: string;

diff --git a/...olutions/observability/plugins/apm/common/anomaly_detection/service_anomaly_timeseries.ts b/...olutions/observability/plugins/apm/common/anomaly_detection/service_anomaly_timeseries.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import { Coordinate } from '../../typings/timeseries';
-import { AnomalyDetectorType } from './apm_ml_detectors';
+import type { Coordinate } from '../../typings/timeseries';
+import type { AnomalyDetectorType } from './apm_ml_detectors';
 
 export interface ServiceAnomalyTimeseries {
   jobId: string;

diff --git a/x-pack/solutions/observability/plugins/apm/common/connections.ts b/x-pack/solutions/observability/plugins/apm/common/connections.ts
@@ -4,8 +4,8 @@
  * 2.0; you may not use this file except in compliance with the Elastic License
  * 2.0.
  */
-import { AgentName } from '../typings/es_schemas/ui/fields/agent';
-import { Coordinate } from '../typings/timeseries';
+import type { AgentName } from '../typings/es_schemas/ui/fields/agent';
+import type { Coordinate } from '../typings/timeseries';
 
 export enum NodeType {
   service = 'service',

diff --git a/...s/observability/plugins/apm/common/correlations/failed_transactions_correlations/types.ts b/...s/observability/plugins/apm/common/correlations/failed_transactions_correlations/types.ts
@@ -5,9 +5,9 @@
  * 2.0.
  */
 
-import { FieldValuePair, HistogramItem } from '../types';
+import type { FieldValuePair, HistogramItem } from '../types';
 
-import { CORRELATIONS_IMPACT_THRESHOLD } from './constants';
+import type { CORRELATIONS_IMPACT_THRESHOLD } from './constants';
 
 export interface FailedTransactionsCorrelation extends FieldValuePair {
   doc_count: number;

diff --git a/x-pack/solutions/observability/plugins/apm/common/correlations/latency_correlations/types.ts b/x-pack/solutions/observability/plugins/apm/common/correlations/latency_correlations/types.ts
@@ -5,7 +5,7 @@
  * 2.0.
  */
 
-import { FieldValuePair, HistogramItem } from '../types';
+import type { FieldValuePair, HistogramItem } from '../types';
 
 export interface LatencyCorrelation extends FieldValuePair {
   correlation: number;

diff --git a/x-pack/solutions/observability/plugins/apm/common/correlations/types.ts b/x-pack/solutions/observability/plugins/apm/common/correlations/types.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import { QueryDslQueryContainer } from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
-import { Environment } from '../environment_rt';
+import type { QueryDslQueryContainer } from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
+import type { Environment } from '../environment_rt';
 
 export interface FieldValuePair {
   fieldName: string;

diff --git a/x-pack/solutions/observability/plugins/apm/common/critical_path/get_critical_path.test.ts b/x-pack/solutions/observability/plugins/apm/common/critical_path/get_critical_path.test.ts
@@ -4,10 +4,11 @@
  * 2.0; you may not use this file except in compliance with the Elastic License
  * 2.0.
  */
-import { apm, ApmFields, dedot } from '@kbn/apm-synthtrace-client';
+import type { ApmFields } from '@kbn/apm-synthtrace-client';
+import { apm, dedot } from '@kbn/apm-synthtrace-client';
 import { getWaterfall } from '../../public/components/app/transaction_details/waterfall_with_summary/waterfall_container/waterfall/waterfall_helpers/waterfall_helpers';
-import { Span } from '../../typings/es_schemas/ui/span';
-import { Transaction } from '../../typings/es_schemas/ui/transaction';
+import type { Span } from '../../typings/es_schemas/ui/span';
+import type { Transaction } from '../../typings/es_schemas/ui/transaction';
 import { getCriticalPath } from './get_critical_path';
 
 describe('getCriticalPath', () => {

diff --git a/x-pack/solutions/observability/plugins/apm/common/critical_path/get_critical_path.ts b/x-pack/solutions/observability/plugins/apm/common/critical_path/get_critical_path.ts
@@ -9,7 +9,7 @@ import type {
   IWaterfall,
   IWaterfallSpanOrTransaction,
 } from '../../public/components/app/transaction_details/waterfall_with_summary/waterfall_container/waterfall/waterfall_helpers/waterfall_helpers';
-import { CriticalPath, CriticalPathSegment } from './types';
+import type { CriticalPath, CriticalPathSegment } from './types';
 
 export function getCriticalPath(waterfall: IWaterfall): CriticalPath {
   const segments: CriticalPathSegment[] = [];

diff --git a/x-pack/solutions/observability/plugins/apm/common/custom_link/custom_link.test.ts b/x-pack/solutions/observability/plugins/apm/common/custom_link/custom_link.test.ts
@@ -5,7 +5,7 @@
  * 2.0.
  */
 import { extractTemplateVariableNames, getEncodedCustomLinkUrl } from '.';
-import { Transaction } from '../../typings/es_schemas/ui/transaction';
+import type { Transaction } from '../../typings/es_schemas/ui/transaction';
 
 describe('Custom link', () => {
   describe('extractTemplateVariableNames', () => {

diff --git a/x-pack/solutions/observability/plugins/apm/common/custom_link/custom_link_types.d.ts b/x-pack/solutions/observability/plugins/apm/common/custom_link/custom_link_types.d.ts
@@ -5,13 +5,13 @@
  * 2.0.
  */
 
-import {
+import type {
   SERVICE_NAME,
   SERVICE_ENVIRONMENT,
   TRANSACTION_NAME,
   TRANSACTION_TYPE,
 } from '../es_fields/apm';
-import { FILTER_OPTIONS } from './custom_link_filter_options';
+import type { FILTER_OPTIONS } from './custom_link_filter_options';
 
 export interface CustomLinkES {
   id?: string;

diff --git a/x-pack/solutions/observability/plugins/apm/common/custom_link/index.ts b/x-pack/solutions/observability/plugins/apm/common/custom_link/index.ts
@@ -9,7 +9,7 @@ import { i18n } from '@kbn/i18n';
 import { get } from 'lodash';
 import { set } from '@kbn/safer-lodash-set';
 import Mustache from 'mustache';
-import { Transaction } from '../../typings/es_schemas/ui/transaction';
+import type { Transaction } from '../../typings/es_schemas/ui/transaction';
 
 export const INVALID_LICENSE = i18n.translate('xpack.apm.settings.customLink.license.text', {
   defaultMessage:

diff --git a/x-pack/solutions/observability/plugins/apm/common/environment_filter_values.ts b/x-pack/solutions/observability/plugins/apm/common/environment_filter_values.ts
@@ -8,7 +8,7 @@
 import { i18n } from '@kbn/i18n';
 import { escapeKuery } from '@kbn/es-query';
 import { SERVICE_ENVIRONMENT } from './es_fields/apm';
-import { Environment } from './environment_rt';
+import type { Environment } from './environment_rt';
 
 export const ENVIRONMENT_ALL_VALUE = 'ENVIRONMENT_ALL' as const;
 const ENVIRONMENT_NOT_DEFINED_VALUE = 'ENVIRONMENT_NOT_DEFINED' as const;

diff --git a/x-pack/solutions/observability/plugins/apm/common/es_fields/es_fields.test.ts b/x-pack/solutions/observability/plugins/apm/common/es_fields/es_fields.test.ts
@@ -6,10 +6,10 @@
  */
 
 import { get } from 'lodash';
-import { AllowUnknownProperties } from '../../typings/common';
-import { APMError } from '../../typings/es_schemas/ui/apm_error';
-import { Span } from '../../typings/es_schemas/ui/span';
-import { Transaction } from '../../typings/es_schemas/ui/transaction';
+import type { AllowUnknownProperties } from '../../typings/common';
+import type { APMError } from '../../typings/es_schemas/ui/apm_error';
+import type { Span } from '../../typings/es_schemas/ui/span';
+import type { Transaction } from '../../typings/es_schemas/ui/transaction';
 import * as allApmFieldNames from './apm';
 import * as infraMetricsFieldNames from './infra_metrics';
 

diff --git a/x-pack/solutions/observability/plugins/apm/common/license_check.ts b/x-pack/solutions/observability/plugins/apm/common/license_check.ts
@@ -5,7 +5,7 @@
  * 2.0.
  */
 
-import { ILicense, LicenseType } from '@kbn/licensing-plugin/common/types';
+import type { ILicense, LicenseType } from '@kbn/licensing-plugin/common/types';
 
 function isActiveLicense(licenseType: LicenseType, license?: ILicense) {
   return license && license.isActive && license.hasAtLeast(licenseType);

diff --git a/x-pack/solutions/observability/plugins/apm/common/rules/schema.ts b/x-pack/solutions/observability/plugins/apm/common/rules/schema.ts
@@ -5,9 +5,10 @@
  * 2.0.
  */
 
-import { schema, TypeOf } from '@kbn/config-schema';
+import type { TypeOf } from '@kbn/config-schema';
+import { schema } from '@kbn/config-schema';
 import { ML_ANOMALY_SEVERITY } from '@kbn/ml-anomaly-utils/anomaly_severity';
-import { ApmRuleType } from '@kbn/rule-data-utils';
+import type { ApmRuleType } from '@kbn/rule-data-utils';
 import { AnomalyDetectorType } from '../anomaly_detection/apm_ml_detectors';
 import { AggregationType } from './apm_rule_types';
 

diff --git a/x-pack/solutions/observability/plugins/apm/common/service_health_status.ts b/x-pack/solutions/observability/plugins/apm/common/service_health_status.ts
@@ -6,7 +6,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { EuiTheme } from '@kbn/kibana-react-plugin/common';
+import type { EuiTheme } from '@kbn/kibana-react-plugin/common';
 import { ML_ANOMALY_SEVERITY } from '@kbn/ml-anomaly-utils/anomaly_severity';
 
 export enum ServiceHealthStatus {

diff --git a/x-pack/solutions/observability/plugins/apm/common/service_inventory.ts b/x-pack/solutions/observability/plugins/apm/common/service_inventory.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import { AgentName } from '@kbn/elastic-agent-utils';
-import { ServiceHealthStatus } from './service_health_status';
+import type { AgentName } from '@kbn/elastic-agent-utils';
+import type { ServiceHealthStatus } from './service_health_status';
 
 export interface ServiceListItem {
   serviceName: string;

diff --git a/x-pack/solutions/observability/plugins/apm/common/service_map.ts b/x-pack/solutions/observability/plugins/apm/common/service_map.ts
@@ -6,9 +6,9 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import cytoscape from 'cytoscape';
-import { Coordinate } from '../typings/timeseries';
-import { ServiceAnomalyStats } from './anomaly_detection';
+import type cytoscape from 'cytoscape';
+import type { Coordinate } from '../typings/timeseries';
+import type { ServiceAnomalyStats } from './anomaly_detection';
 
 // These should be imported, but until TypeScript 4.2 we're inlining them here.
 // All instances of "agent.name", "service.name", "service.environment", "span.type",

diff --git a/x-pack/solutions/observability/plugins/apm/common/span_links.ts b/x-pack/solutions/observability/plugins/apm/common/span_links.ts
@@ -5,8 +5,8 @@
  * 2.0.
  */
 
-import { AgentName } from '../typings/es_schemas/ui/fields/agent';
-import { Environment } from './environment_rt';
+import type { AgentName } from '../typings/es_schemas/ui/fields/agent';
+import type { Environment } from './environment_rt';
 
 export interface SpanLinkDetails {
   traceId: string;

diff --git a/x-pack/solutions/observability/plugins/apm/common/tutorial/tutorials.ts b/x-pack/solutions/observability/plugins/apm/common/tutorial/tutorials.ts
@@ -6,7 +6,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { CustomIntegration } from '@kbn/custom-integrations-plugin/common';
+import type { CustomIntegration } from '@kbn/custom-integrations-plugin/common';
 
 const APM_INTEGRATION_CATEGORIES = ['observability', 'apm'];
 

diff --git a/x-pack/solutions/observability/plugins/apm/common/utils/array_union_to_callable.ts b/x-pack/solutions/observability/plugins/apm/common/utils/array_union_to_callable.ts
@@ -5,7 +5,7 @@
  * 2.0.
  */
 
-import { ValuesType } from 'utility-types';
+import type { ValuesType } from 'utility-types';
 
 // work around a TypeScript limitation described in https://stackoverflow.com/posts/49511416
 

diff --git a/x-pack/solutions/observability/plugins/apm/common/utils/environment_query.ts b/x-pack/solutions/observability/plugins/apm/common/utils/environment_query.ts
@@ -5,7 +5,7 @@
  * 2.0.
  */
 
-import { QueryDslQueryContainer } from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
+import type { QueryDslQueryContainer } from '@elastic/elasticsearch/lib/api/typesWithBodyKey';
 import { SERVICE_ENVIRONMENT, SERVICE_NODE_NAME } from '../es_fields/apm';
 import { ENVIRONMENT_ALL, ENVIRONMENT_NOT_DEFINED } from '../environment_filter_values';
 import { SERVICE_NODE_NAME_MISSING } from '../service_nodes';
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,7 +5,7 @@ @@
      * 2.0.
      */
-    import { ValuesType } from 'utility-types';
+    import type { ValuesType } from 'utility-types';
     // work around a TypeScript limitation described in https://stackoverflow.com/posts/49511416
@@ Expand Down @@