Remove legacy deprecation adapter (elastic#76753) (elastic#76835)

src/core/server/legacy/config/index.ts

@@ -19,4 +19,3 @@
 
 export { ensureValidConfiguration } from './ensure_valid_configuration';
 export { LegacyObjectToConfigAdapter } from './legacy_object_to_config_adapter';
-export { convertLegacyDeprecationProvider } from './legacy_deprecation_adapters';

src/core/server/legacy/legacy_service.test.ts

@@ -19,9 +19,7 @@
 
 jest.mock('../../../legacy/server/kbn_server');
 jest.mock('../../../cli/cluster/cluster_manager');
-jest.mock('./config/legacy_deprecation_adapters', () => ({
- convertLegacyDeprecationProvider: (provider: any) => Promise.resolve(provider),
-}));
+
 import {
  findLegacyPluginSpecsMock,
  logLegacyThirdPartyPluginDeprecationWarningMock,
@@ -446,46 +444,8 @@ describe('#discoverPlugins()', () => {
  expect(findLegacyPluginSpecs).toHaveBeenCalledWith(expect.any(Object), logger, env.packageInfo);
  });
 
- it(`register legacy plugin's deprecation providers`, async () => {
- findLegacyPluginSpecsMock.mockImplementation(
- (settings) =>
- Promise.resolve({
- pluginSpecs: [
- {
- getDeprecationsProvider: () => undefined,
- },
- {
- getDeprecationsProvider: () => 'providerA',
- },
- {
- getDeprecationsProvider: () => 'providerB',
- },
- ],
- pluginExtendedConfig: settings,
- disabledPluginSpecs: [],
- uiExports: {},
- navLinks: [],
- }) as any
- );
-
- const legacyService = new LegacyService({
- coreId,
- env,
- logger,
- configService: configService as any,
- });
-
- await legacyService.discoverPlugins();
- expect(configService.addDeprecationProvider).toHaveBeenCalledTimes(2);
- expect(configService.addDeprecationProvider).toHaveBeenCalledWith('', 'providerA');
- expect(configService.addDeprecationProvider).toHaveBeenCalledWith('', 'providerB');
- });
-
  it(`logs deprecations for legacy third party plugins`, async () => {
- const pluginSpecs = [
- { getId: () => 'pluginA', getDeprecationsProvider: () => undefined },
- { getId: () => 'pluginB', getDeprecationsProvider: () => undefined },
- ];
+ const pluginSpecs = [{ getId: () => 'pluginA' }, { getId: () => 'pluginB' }];
  findLegacyPluginSpecsMock.mockImplementation(
  (settings) =>
  Promise.resolve({

src/core/server/legacy/legacy_service.ts

@@ -21,15 +21,14 @@ import { combineLatest, ConnectableObservable, EMPTY, Observable, Subscription }
 import { first, map, publishReplay, tap } from 'rxjs/operators';
 
 import { CoreService } from '../../types';
-import { Config, ConfigDeprecationProvider } from '../config';
+import { Config } from '../config';
 import { CoreContext } from '../core_context';
 import { CspConfigType, config as cspConfig } from '../csp';
 import { DevConfig, DevConfigType, config as devConfig } from '../dev';
 import { BasePathProxyServer, HttpConfig, HttpConfigType, config as httpConfig } from '../http';
 import { Logger } from '../logging';
 import { PathConfigType } from '../path';
 import { findLegacyPluginSpecs, logLegacyThirdPartyPluginDeprecationWarning } from './plugins';
-import { convertLegacyDeprecationProvider } from './config';
 import {
  ILegacyInternals,
  LegacyServiceSetupDeps,
@@ -145,18 +144,6 @@ export class LegacyService implements CoreService {
  navLinks,
  };
 
- const deprecationProviders = await pluginSpecs
- .map((spec) => spec.getDeprecationsProvider())
- .reduce(async (providers, current) => {
- if (current) {
- return [...(await providers), await convertLegacyDeprecationProvider(current)];
- }
- return providers;
- }, Promise.resolve([] as ConfigDeprecationProvider[]));
- deprecationProviders.forEach((provider) =>
- this.coreContext.configService.addDeprecationProvider('', provider)
- );
-
  this.legacyRawConfig = pluginExtendedConfig;
 
  // check for unknown uiExport types

src/core/server/legacy/plugins/log_legacy_plugins_warning.test.ts

@@ -26,7 +26,6 @@ const createPluginSpec = ({ id, path }: { id: string; path: string }): LegacyPlu
  getId: () => id,
  getExpectedKibanaVersion: () => 'kibana',
  getConfigPrefix: () => 'plugin.config',
- getDeprecationsProvider: () => undefined,
  getPack: () => ({
  getPath: () => path,
  }),

src/core/server/legacy/types.ts

@@ -51,36 +51,6 @@ export interface LegacyConfig {
  set(config: LegacyVars): void;
 }
 
-/**
- * Representation of a legacy configuration deprecation factory used for
- * legacy plugin deprecations.
- *
- * @internal
- * @deprecated
- */
-export interface LegacyConfigDeprecationFactory {
- rename(oldKey: string, newKey: string): LegacyConfigDeprecation;
- unused(unusedKey: string): LegacyConfigDeprecation;
-}
-
-/**
- * Representation of a legacy configuration deprecation.
- *
- * @internal
- * @deprecated
- */
-export type LegacyConfigDeprecation = (settings: LegacyVars, log: (msg: string) => void) => void;
-
-/**
- * Representation of a legacy configuration deprecation provider.
- *
- * @internal
- * @deprecated
- */
-export type LegacyConfigDeprecationProvider = (
- factory: LegacyConfigDeprecationFactory
-) => LegacyConfigDeprecation[] | Promise<LegacyConfigDeprecation[]>;
-
 /**
  * @internal
  * @deprecated
@@ -97,7 +67,6 @@ export interface LegacyPluginSpec {
  getId: () => unknown;
  getExpectedKibanaVersion: () => string;
  getConfigPrefix: () => string;
- getDeprecationsProvider: () => LegacyConfigDeprecationProvider | undefined;
  getPack: () => LegacyPluginPack;
 }
 

src/core/server/server.api.md

@@ -2949,11 +2949,11 @@ export const validBodyOutput: readonly ["data", "stream"];
 // Warnings were encountered during analysis:
 //
 // src/core/server/http/router/response.ts:316:3 - (ae-forgotten-export) The symbol "KibanaResponse" needs to be exported by the entry point index.d.ts
-// src/core/server/legacy/types.ts:163:3 - (ae-forgotten-export) The symbol "VarsProvider" needs to be exported by the entry point index.d.ts
-// src/core/server/legacy/types.ts:164:3 - (ae-forgotten-export) The symbol "VarsReplacer" needs to be exported by the entry point index.d.ts
-// src/core/server/legacy/types.ts:165:3 - (ae-forgotten-export) The symbol "LegacyNavLinkSpec" needs to be exported by the entry point index.d.ts
-// src/core/server/legacy/types.ts:166:3 - (ae-forgotten-export) The symbol "LegacyAppSpec" needs to be exported by the entry point index.d.ts
-// src/core/server/legacy/types.ts:167:16 - (ae-forgotten-export) The symbol "LegacyPluginSpec" needs to be exported by the entry point index.d.ts
+// src/core/server/legacy/types.ts:132:3 - (ae-forgotten-export) The symbol "VarsProvider" needs to be exported by the entry point index.d.ts
+// src/core/server/legacy/types.ts:133:3 - (ae-forgotten-export) The symbol "VarsReplacer" needs to be exported by the entry point index.d.ts
+// src/core/server/legacy/types.ts:134:3 - (ae-forgotten-export) The symbol "LegacyNavLinkSpec" needs to be exported by the entry point index.d.ts
+// src/core/server/legacy/types.ts:135:3 - (ae-forgotten-export) The symbol "LegacyAppSpec" needs to be exported by the entry point index.d.ts
+// src/core/server/legacy/types.ts:136:16 - (ae-forgotten-export) The symbol "LegacyPluginSpec" needs to be exported by the entry point index.d.ts
 // src/core/server/plugins/types.ts:266:3 - (ae-forgotten-export) The symbol "KibanaConfigType" needs to be exported by the entry point index.d.ts
 // src/core/server/plugins/types.ts:266:3 - (ae-forgotten-export) The symbol "SharedGlobalConfigKeys" needs to be exported by the entry point index.d.ts
 // src/core/server/plugins/types.ts:268:3 - (ae-forgotten-export) The symbol "PathConfigType" needs to be exported by the entry point index.d.ts