Optimize Functions and Renderers for Webpack builds

x-pack/plugins/canvas/canvas_plugin_src/functions/browser/index.ts

@@ -5,8 +5,9 @@
  */
 
 import { functions as commonFunctions } from '../common';
+import { functions as externalFunctions } from '../external';
 import { location } from './location';
 import { markdown } from './markdown';
 import { urlparam } from './urlparam';
 
-export const functions = [location, markdown, urlparam, ...commonFunctions];
+export const functions = [location, markdown, urlparam, ...commonFunctions, ...externalFunctions];

x-pack/plugins/canvas/canvas_plugin_src/functions/common/index.ts

@@ -48,10 +48,6 @@ import { rounddate } from './rounddate';
 import { rowCount } from './rowCount';
 import { repeatImage } from './repeatImage';
 import { revealImage } from './revealImage';
-import { savedLens } from './saved_lens';
-import { savedMap } from './saved_map';
-import { savedSearch } from './saved_search';
-import { savedVisualization } from './saved_visualization';
 import { seriesStyle } from './seriesStyle';
 import { shape } from './shape';
 import { sort } from './sort';
@@ -110,10 +106,6 @@ export const functions = [
   revealImage,
   rounddate,
   rowCount,
-  savedLens,
-  savedMap,
-  savedSearch,
-  savedVisualization,
   seriesStyle,
   shape,
   sort,

x-pack/plugins/canvas/canvas_plugin_src/functions/external/index.ts

@@ -0,0 +1,12 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+
+import { savedLens } from './saved_lens';
+import { savedMap } from './saved_map';
+import { savedSearch } from './saved_search';
+import { savedVisualization } from './saved_visualization';
+
+export const functions = [savedLens, savedMap, savedSearch, savedVisualization];

x-pack/plugins/canvas/canvas_plugin_src/renderers/core.js

@@ -0,0 +1,37 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+
+import { debug } from './debug';
+import { error } from './error';
+import { image } from './image';
+import { markdown } from './markdown';
+import { metric } from './metric';
+import { pie } from './pie';
+import { plot } from './plot';
+import { progress } from './progress';
+import { repeatImage } from './repeat_image';
+import { revealImage } from './reveal_image';
+import { shape } from './shape';
+import { table } from './table';
+import { text } from './text';
+
+export const renderFunctions = [
+  debug,
+  error,
+  image,
+  markdown,
+  metric,
+  pie,
+  plot,
+  progress,
+  repeatImage,
+  revealImage,
+  shape,
+  table,
+  text,
+];
+
+export const renderFunctionFactories = [];

x-pack/plugins/canvas/canvas_plugin_src/renderers/embeddable/index.ts

@@ -0,0 +1,9 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+import { embeddableRendererFactory } from './embeddable';
+
+export const renderFunctions = [];
+export const renderFunctionFactories = [embeddableRendererFactory];

x-pack/plugins/canvas/canvas_plugin_src/renderers/embeddable/input_type_to_expression/lens.test.ts

@@ -5,7 +5,7 @@
  */
 
 import { toExpression } from './lens';
-import { SavedLensInput } from '../../../functions/common/saved_lens';
+import { SavedLensInput } from '../../../functions/external/saved_lens';
 import { fromExpression, Ast } from '@kbn/interpreter/common';
 
 const baseEmbeddableInput = {

x-pack/plugins/canvas/canvas_plugin_src/renderers/embeddable/input_type_to_expression/lens.ts

@@ -4,7 +4,7 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { SavedLensInput } from '../../../functions/common/saved_lens';
+import { SavedLensInput } from '../../../functions/external/saved_lens';
 
 export function toExpression(input: SavedLensInput): string {
   const expressionParts = [] as string[];

x-pack/plugins/canvas/canvas_plugin_src/renderers/advanced_filter/component/advanced_filter.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/advanced_filter/component/advanced_filter.tsx

@@ -7,7 +7,7 @@
 import { EuiFlexGroup, EuiFlexItem } from '@elastic/eui';
 import PropTypes from 'prop-types';
 import React, { FunctionComponent } from 'react';
-import { ComponentStrings } from '../../../../i18n';
+import { ComponentStrings } from '../../../../../i18n';
 
 const { AdvancedFilter: strings } = ComponentStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/advanced_filter/index.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/advanced_filter/index.tsx

@@ -6,9 +6,9 @@
 
 import React from 'react';
 import ReactDOM from 'react-dom';
-import { RendererFactory } from '../../../types';
+import { RendererFactory } from '../../../../types';
 import { AdvancedFilter } from './component';
-import { RendererStrings } from '../../../i18n';
+import { RendererStrings } from '../../../../i18n';
 
 const { advancedFilter: strings } = RendererStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/dropdown_filter/component/dropdown_filter.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/dropdown_filter/component/dropdown_filter.tsx

@@ -7,7 +7,7 @@
 import { EuiIcon } from '@elastic/eui';
 import PropTypes from 'prop-types';
 import React, { ChangeEvent, FocusEvent, FunctionComponent } from 'react';
-import { ComponentStrings } from '../../../../i18n';
+import { ComponentStrings } from '../../../../../i18n';
 
 const { DropdownFilter: strings } = ComponentStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/dropdown_filter/index.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/dropdown_filter/index.tsx

@@ -8,10 +8,10 @@ import { fromExpression, toExpression, Ast } from '@kbn/interpreter/common';
 import { get } from 'lodash';
 import React from 'react';
 import ReactDOM from 'react-dom';
-import { syncFilterExpression } from '../../../public/lib/sync_filter_expression';
-import { RendererFactory } from '../../../types';
+import { syncFilterExpression } from '../../../../public/lib/sync_filter_expression';
+import { RendererFactory } from '../../../../types';
 import { DropdownFilter } from './component';
-import { RendererStrings } from '../../../i18n';
+import { RendererStrings } from '../../../../i18n';
 
 const { dropdownFilter: strings } = RendererStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/index.js

@@ -0,0 +1,12 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+
+import { advancedFilter } from './advanced_filter';
+import { dropdownFilter } from './dropdown_filter';
+import { timeFilterFactory } from './time_filter';
+
+export const renderFunctions = [advancedFilter, dropdownFilter];
+export const renderFunctionFactories = [timeFilterFactory];

x-pack/plugins/canvas/canvas_plugin_src/renderers/time_filter/components/index.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/time_filter/components/index.tsx

@@ -4,6 +4,4 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { TimeFilter } from './time_filter';
-
-export { TimeFilter };
+export { TimeFilter } from './time_filter';

x-pack/plugins/canvas/canvas_plugin_src/renderers/time_filter/components/time_filter.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/time_filter/components/time_filter.tsx

@@ -8,7 +8,7 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { get } from 'lodash';
 import { fromExpression } from '@kbn/interpreter/common';
-import { UnitStrings } from '../../../../i18n/units';
+import { UnitStrings } from '../../../../../i18n/units';
 
 const { quickRanges: strings } = UnitStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/time_filter/index.tsx → x-pack/plugins/canvas/canvas_plugin_src/renderers/filters/time_filter/index.tsx

@@ -7,14 +7,14 @@
 import ReactDOM from 'react-dom';
 import React from 'react';
 import { toExpression } from '@kbn/interpreter/common';
-import { UI_SETTINGS } from '../../../../../../src/plugins/data/public';
-import { syncFilterExpression } from '../../../public/lib/sync_filter_expression';
-import { RendererStrings } from '../../../i18n';
+import { UI_SETTINGS } from '../../../../../../../src/plugins/data/public';
+import { syncFilterExpression } from '../../../../public/lib/sync_filter_expression';
+import { RendererStrings } from '../../../../i18n';
 import { TimeFilter } from './components';
-import { StartInitializer } from '../../plugin';
-import { RendererHandlers } from '../../../types';
-import { Arguments } from '../../functions/common/timefilterControl';
-import { RendererFactory } from '../../../types';
+import { StartInitializer } from '../../../plugin';
+import { RendererHandlers } from '../../../../types';
+import { Arguments } from '../../../functions/common/timefilterControl';
+import { RendererFactory } from '../../../../types';
 
 const { timeFilter: strings } = RendererStrings;
 

x-pack/plugins/canvas/canvas_plugin_src/renderers/index.js

@@ -4,40 +4,21 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { advancedFilter } from './advanced_filter';
-import { debug } from './debug';
-import { dropdownFilter } from './dropdown_filter';
-import { embeddableRendererFactory } from './embeddable/embeddable';
-import { error } from './error';
-import { image } from './image';
-import { markdown } from './markdown';
-import { metric } from './metric';
-import { pie } from './pie';
-import { plot } from './plot';
-import { progress } from './progress';
-import { repeatImage } from './repeat_image';
-import { revealImage } from './reveal_image';
-import { shape } from './shape';
-import { table } from './table';
-import { text } from './text';
-import { timeFilterFactory } from './time_filter';
+import {
+  renderFunctions as embeddableFunctions,
+  renderFunctionFactories as embeddableFactories,
+} from './embeddable';
 
-export const renderFunctions = [
-  advancedFilter,
-  debug,
-  dropdownFilter,
-  error,
-  image,
-  markdown,
-  metric,
-  pie,
-  plot,
-  progress,
-  repeatImage,
-  revealImage,
-  shape,
-  table,
-  text,
-];
+import {
+  renderFunctions as filterFunctions,
+  renderFunctionFactories as filterFactories,
+} from './filters';
+
+import { renderFunctions as coreFunctions, renderFunctionFactories as coreFactories } from './core';
 
-export const renderFunctionFactories = [embeddableRendererFactory, timeFilterFactory];
+export const renderFunctions = [...coreFunctions, ...filterFunctions, ...embeddableFunctions];
+export const renderFunctionFactories = [
+  ...coreFactories,
+  ...embeddableFactories,
+  ...filterFactories,
+];

x-pack/plugins/canvas/i18n/functions/dict/saved_lens.ts

@@ -5,7 +5,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { savedLens } from '../../../canvas_plugin_src/functions/common/saved_lens';
+import { savedLens } from '../../../canvas_plugin_src/functions/external/saved_lens';
 import { FunctionHelp } from '../function_help';
 import { FunctionFactory } from '../../../types';
 

x-pack/plugins/canvas/i18n/functions/dict/saved_map.ts

@@ -5,7 +5,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { savedMap } from '../../../canvas_plugin_src/functions/common/saved_map';
+import { savedMap } from '../../../canvas_plugin_src/functions/external/saved_map';
 import { FunctionHelp } from '../function_help';
 import { FunctionFactory } from '../../../types';
 

x-pack/plugins/canvas/i18n/functions/dict/saved_search.ts

@@ -5,7 +5,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { savedSearch } from '../../../canvas_plugin_src/functions/common/saved_search';
+import { savedSearch } from '../../../canvas_plugin_src/functions/external/saved_search';
 import { FunctionHelp } from '../function_help';
 import { FunctionFactory } from '../../../types';
 

x-pack/plugins/canvas/i18n/functions/dict/saved_visualization.ts

@@ -5,7 +5,7 @@
  */
 
 import { i18n } from '@kbn/i18n';
-import { savedVisualization } from '../../../canvas_plugin_src/functions/common/saved_visualization';
+import { savedVisualization } from '../../../canvas_plugin_src/functions/external/saved_visualization';
 import { FunctionHelp } from '../function_help';
 import { FunctionFactory } from '../../../types';
 

x-pack/plugins/canvas/public/style/index.scss

@@ -56,10 +56,10 @@
 @import '../components/workpad_page/workpad_interactive_page/workpad_interactive_page';
 @import '../components/workpad_page/workpad_static_page/workpad_static_page';
 
-@import '../../canvas_plugin_src/renderers/advanced_filter/component/advanced_filter.scss';
-@import '../../canvas_plugin_src/renderers/dropdown_filter/component/dropdown_filter.scss';
+@import '../../canvas_plugin_src/renderers/filters/advanced_filter/component/advanced_filter.scss';
+@import '../../canvas_plugin_src/renderers/filters/dropdown_filter/component/dropdown_filter.scss';
 @import '../../canvas_plugin_src/renderers/embeddable/embeddable.scss';
 @import '../../canvas_plugin_src/renderers/plot/plot.scss';
 @import '../../canvas_plugin_src/renderers/reveal_image/reveal_image.scss';
-@import '../../canvas_plugin_src/renderers/time_filter/time_filter.scss';
+@import '../../canvas_plugin_src/renderers/filters/time_filter/time_filter.scss';
 @import '../../canvas_plugin_src/uis/arguments/image_upload/image_upload.scss';

x-pack/plugins/canvas/types/functions.ts

@@ -8,6 +8,7 @@ import { ExpressionFunctionDefinition } from 'src/plugins/expressions/common';
 import { functions as commonFunctions } from '../canvas_plugin_src/functions/common';
 import { functions as browserFunctions } from '../canvas_plugin_src/functions/browser';
 import { functions as serverFunctions } from '../canvas_plugin_src/functions/server';
+import { functions as externalFunctions } from '../canvas_plugin_src/functions/external';
 import { initFunctions } from '../public/functions';
 
 /**
@@ -87,6 +88,7 @@ export type FunctionFactory<FnFactory> =
 type CommonFunction = FunctionFactory<typeof commonFunctions[number]>;
 type BrowserFunction = FunctionFactory<typeof browserFunctions[number]>;
 type ServerFunction = FunctionFactory<typeof serverFunctions[number]>;
+type ExternalFunction = FunctionFactory<typeof externalFunctions[number]>;
 type ClientFunctions = FunctionFactory<
   ReturnType<typeof initFunctions> extends Array<infer U> ? U : never
 >;
@@ -95,7 +97,12 @@ type ClientFunctions = FunctionFactory<
  * A collection of all Canvas Functions.
  */
 
-export type CanvasFunction = CommonFunction | BrowserFunction | ServerFunction | ClientFunctions;
+export type CanvasFunction =
+  | CommonFunction
+  | BrowserFunction
+  | ServerFunction
+  | ExternalFunction
+  | ClientFunctions;
 
 /**
  * Represents a function called by the `case` Function.