share single data plugin bundle (#64549)

packages/kbn-optimizer/src/worker/webpack.config.ts

@@ -35,7 +35,7 @@ import { Bundle, WorkerConfig, parseDirPath, DisallowedSyntaxPlugin } from '../c
 const BABEL_PRESET_PATH = require.resolve('@kbn/babel-preset/webpack_preset');
 
 const STATIC_BUNDLE_PLUGINS = [
-  // { id: 'data', dirname: 'data' },
+  { id: 'data', dirname: 'data' },
   { id: 'kibanaReact', dirname: 'kibana_react' },
   { id: 'kibanaUtils', dirname: 'kibana_utils' },
   { id: 'esUiShared', dirname: 'es_ui_shared' },
@@ -58,13 +58,8 @@ function dynamicExternals(bundle: Bundle, context: string, request: string) {
     return;
   }
 
-  // don't allow any static bundle to rely on other static bundles
-  if (STATIC_BUNDLE_PLUGINS.some(p => bundle.id === p.id)) {
-    return;
-  }
-
-  // ignore requests that don't include a /data/public, /kibana_react/public, or
-  // /kibana_utils/public segment as a cheap way to avoid doing path resolution
+  // ignore requests that don't include a /{dirname}/public for one of our
+  // "static" bundles as a cheap way to avoid doing path resolution
   // for paths that couldn't possibly resolve to what we're looking for
   const reqToStaticBundle = STATIC_BUNDLE_PLUGINS.some(p =>
     request.includes(`/${p.dirname}/public`)

src/legacy/ui/ui_render/ui_render_mixin.js

@@ -142,8 +142,9 @@ export function uiRenderMixin(kbnServer, server, config) {
           // load these plugins first, they are "shared" and other bundles access their
           // public/index exports without considering topographic sorting by plugin deps (for now)
           'kibanaUtils',
-          'esUiShared',
           'kibanaReact',
+          'data',
+          'esUiShared',
           ...kbnServer.newPlatform.__internals.uiPlugins.public.keys()
         );