[bin/kibana-plugin] support KP plugins instead (#74604)

* [cli/kibana-plugin] support KP plugins instead

* fix some Logger imports from cli/kibana_keystore

Co-authored-by: spalger <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>

Author: 3 people

package.json

@@ -276,7 +276,6 @@
     "url-loader": "2.2.0",
     "uuid": "3.3.2",
     "val-loader": "^1.1.1",
-    "validate-npm-package-name": "2.2.2",
     "vega": "^5.13.0",
     "vega-lite": "^4.13.1",
     "vega-schema-url-parser": "^1.1.0",

packages/kbn-pm/dist/index.js

@@ -97,8 +97,6 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _production__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(511);
 /* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "buildProductionProjects", function() { return _production__WEBPACK_IMPORTED_MODULE_1__["buildProductionProjects"]; });
 
-/* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "prepareExternalProjectDependencies", function() { return _production__WEBPACK_IMPORTED_MODULE_1__["prepareExternalProjectDependencies"]; });
-
 /* harmony import */ var _utils_projects__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(145);
 /* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "getProjects", function() { return _utils_projects__WEBPACK_IMPORTED_MODULE_2__["getProjects"]; });
 
@@ -59477,9 +59475,6 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _build_production_projects__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(512);
 /* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "buildProductionProjects", function() { return _build_production_projects__WEBPACK_IMPORTED_MODULE_0__["buildProductionProjects"]; });
 
-/* harmony import */ var _prepare_project_dependencies__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(748);
-/* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "prepareExternalProjectDependencies", function() { return _prepare_project_dependencies__WEBPACK_IMPORTED_MODULE_1__["prepareExternalProjectDependencies"]; });
-
 /*
  * Licensed to Elasticsearch B.V. under one or more contributor
  * license agreements. See the NOTICE file distributed with
@@ -59500,7 +59495,6 @@ __webpack_require__.r(__webpack_exports__);
  */
 
 
-
 /***/ }),
 /* 512 */
 /***/ (function(module, __webpack_exports__, __webpack_require__) {
@@ -90331,71 +90325,5 @@ NestedError.prototype.name = 'NestedError';
 module.exports = NestedError;
 
 
-/***/ }),
-/* 748 */
-/***/ (function(module, __webpack_exports__, __webpack_require__) {
-
-"use strict";
-__webpack_require__.r(__webpack_exports__);
-/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "prepareExternalProjectDependencies", function() { return prepareExternalProjectDependencies; });
-/* harmony import */ var _utils_package_json__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(164);
-/* harmony import */ var _utils_project__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(163);
-/*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-
-/**
- * All external projects are located within `./plugins/{plugin}` relative
- * to the Kibana root directory or `../kibana-extra/{plugin}` relative
- * to Kibana itself.
- */
-
-const isKibanaDep = depVersion => // For ../kibana-extra/ directory (legacy only)
-depVersion.includes('../../kibana/packages/') || // For plugins/ directory
-depVersion.includes('../../packages/');
-/**
- * This prepares the dependencies for an _external_ project.
- */
-
-
-async function prepareExternalProjectDependencies(projectPath) {
-  const project = await _utils_project__WEBPACK_IMPORTED_MODULE_1__["Project"].fromPath(projectPath);
-
-  if (!project.hasDependencies()) {
-    return;
-  }
-
-  const deps = project.allDependencies;
-
-  for (const depName of Object.keys(deps)) {
-    const depVersion = deps[depName]; // Kibana currently only supports `link:` dependencies on Kibana's own
-    // packages, as these are packaged into the `node_modules` folder when
-    // Kibana is built, so we don't need to take any action to enable
-    // `require(...)` to resolve for these packages.
-
-    if (Object(_utils_package_json__WEBPACK_IMPORTED_MODULE_0__["isLinkDependency"])(depVersion) && !isKibanaDep(depVersion)) {
-      // For non-Kibana packages we need to set up symlinks during the
-      // installation process, but this is not something we support yet.
-      throw new Error('This plugin is using `link:` dependencies for non-Kibana packages');
-    }
-  }
-}
-
 /***/ })
 /******/ ]);

packages/kbn-pm/src/index.ts

@@ -18,7 +18,7 @@
  */
 
 export { run } from './cli';
-export { buildProductionProjects, prepareExternalProjectDependencies } from './production';
+export { buildProductionProjects } from './production';
 export { getProjects } from './utils/projects';
 export { Project } from './utils/project';
 export { copyWorkspacePackages } from './utils/workspaces';

packages/kbn-pm/src/production/index.ts

@@ -18,4 +18,3 @@
  */
 
 export { buildProductionProjects } from './build_production_projects';
-export { prepareExternalProjectDependencies } from './prepare_project_dependencies';

packages/kbn-pm/src/production/prepare_project_dependencies.test.ts

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import Logger from '../cli_plugin/lib/logger';
+import { Logger } from '../cli_plugin/lib/logger';
 import { confirm, question } from '../legacy/server/utils';
 import { createPromiseFromStreams, createConcatStream } from '../legacy/utils';
 

packages/kbn-pm/src/production/prepare_project_dependencies.ts

@@ -41,7 +41,7 @@ import { PassThrough } from 'stream';
 
 import { Keystore } from '../legacy/server/keystore';
 import { add } from './add';
-import Logger from '../cli_plugin/lib/logger';
+import { Logger } from '../cli_plugin/lib/logger';
 import * as prompt from '../legacy/server/utils/prompt';
 
 describe('Kibana keystore', () => {

src/cli_keystore/add.js

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import Logger from '../cli_plugin/lib/logger';
+import { Logger } from '../cli_plugin/lib/logger';
 import { confirm } from '../legacy/server/utils';
 
 export async function create(keystore, command, options) {

src/cli_keystore/add.test.js

@@ -40,7 +40,7 @@ import sinon from 'sinon';
 
 import { Keystore } from '../legacy/server/keystore';
 import { create } from './create';
-import Logger from '../cli_plugin/lib/logger';
+import { Logger } from '../cli_plugin/lib/logger';
 import * as prompt from '../legacy/server/utils/prompt';
 
 describe('Kibana keystore', () => {