Allowing custom folder name for plugin installation

Signed-off-by: Vacha Shah <[email protected]>

src/cli_plugin/install/index.js

@@ -64,6 +64,7 @@ export function installCommand(program) {
       'length of time before failing; 0 for never fail',
       parseMilliseconds
     )
+    .option('-o, --folderName', 'override the folder name for plugin installation')
     .description(
       'install a plugin',
       `Common examples:

src/cli_plugin/install/install.js

@@ -59,11 +59,14 @@ export async function install(settings, logger) {
 
     del.sync(settings.tempArchiveFile, { force: true });
 
+    settings.plugins[0].folderName = settings.override
+      ? settings.plugins[0].folderName
+      : settings.plugins[0].id;
     existingInstall(settings, logger);
 
     assertVersion(settings);
 
-    const targetDir = path.join(settings.pluginDir, kebabCase(settings.plugins[0].id));
+    const targetDir = path.join(settings.pluginDir, kebabCase(settings.plugins[0].folderName));
     await renamePlugin(settings.workingPath, targetDir);
 
     logger.log('Plugin installation complete');

src/cli_plugin/install/opensearch_dashboards.js

@@ -47,6 +47,16 @@ export function existingInstall(settings, logger) {
   } catch (e) {
     if (e.code !== 'ENOENT') throw e;
   }
+  try {
+    statSync(path.join(settings.pluginDir, kebabCase(settings.plugins[0].folderName)));
+
+    logger.error(
+      `Plugin ${settings.plugins[0].id} already exists with custom folder name, please remove before installing a new version`
+    );
+    process.exit(70);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+  }
 }
 
 export function assertVersion(settings) {

src/cli_plugin/install/opensearch_dashboards.test.js

@@ -55,6 +55,7 @@ describe('opensearchDashboards cli', function () {
       const settings = {
         workingPath: testWorkingPath,
         tempArchiveFile: tempArchiveFilePath,
+        override: false,
         plugin: 'test-plugin',
         version: '1.0.0',
         plugins: [{ id: 'foo' }],
@@ -81,6 +82,7 @@ describe('opensearchDashboards cli', function () {
           const settings = {
             workingPath: testWorkingPath,
             tempArchiveFile: tempArchiveFilePath,
+            override: false,
             plugin: 'test-plugin',
             version: '5.0.0-SNAPSHOT',
             plugins: [

src/cli_plugin/install/pack.test.js

@@ -120,6 +120,7 @@ describe('opensearchDashboards cli', function () {
         expect(settings.plugins).toMatchInlineSnapshot(`
           Array [
             Object {
+              "folderName": undefined,
               "id": "testPlugin",
               "opensearchDashboardsVersion": "1.0.0",
               "stripPrefix": "opensearch-dashboards/test-plugin",
@@ -134,6 +135,7 @@ describe('opensearchDashboards cli', function () {
         expect(settings.plugins).toMatchInlineSnapshot(`
           Array [
             Object {
+              "folderName": undefined,
               "id": "testPlugin",
               "opensearchDashboardsVersion": "5.0.1",
               "stripPrefix": "opensearch-dashboards/test-plugin",
@@ -148,16 +150,19 @@ describe('opensearchDashboards cli', function () {
         expect(settings.plugins).toMatchInlineSnapshot(`
           Array [
             Object {
+              "folderName": undefined,
               "id": "fungerPlugin",
               "opensearchDashboardsVersion": "1.0.0",
               "stripPrefix": "opensearch-dashboards/funger-plugin",
             },
             Object {
+              "folderName": undefined,
               "id": "pdf",
               "opensearchDashboardsVersion": "1.0.0",
               "stripPrefix": "opensearch-dashboards/pdf",
             },
             Object {
+              "folderName": undefined,
               "id": "testPlugin",
               "opensearchDashboardsVersion": "1.0.0",
               "stripPrefix": "opensearch-dashboards/test-plugin",
@@ -166,6 +171,21 @@ describe('opensearchDashboards cli', function () {
         `);
       });
 
+      it('populate settings.plugin.folderName', async () => {
+        await copyReplyFile('test_plugin_custom_folder.zip');
+        await getPackData(settings, logger);
+        expect(settings.plugins).toMatchInlineSnapshot(`
+          Array [
+            Object {
+              "folderName": "customPluginName",
+              "id": "testPlugin",
+              "opensearchDashboardsVersion": "1.0.0",
+              "stripPrefix": "opensearch-dashboards/test-plugin-custom-folder",
+            },
+          ]
+        `);
+      });
+
       it('throw an error if there is no opensearch-dashboards plugin', async () => {
         await copyReplyFile('test_plugin_no_opensearch_dashboards.zip');
         await expect(getPackData(settings, logger)).rejects.toThrowErrorMatchingInlineSnapshot(

src/cli_plugin/install/settings.js

@@ -62,6 +62,7 @@ export function parse(command, options, osdPackage) {
     quiet: options.quiet || false,
     silent: options.silent || false,
     config: options.config || '',
+    override: options.override || false,
     plugin: command,
     version: osdPackage.version,
     pluginDir: fromRoot('plugins'),

src/cli_plugin/install/settings.test.js

@@ -66,6 +66,7 @@ describe('parse function', function () {
     expect(parse(command, { ...defaultOptions }, osdPackage)).toMatchInlineSnapshot(`
       Object {
         "config": "",
+        "override": false,
         "plugin": "plugin name",
         "pluginDir": <absolute path>/plugins,
         "quiet": false,
@@ -87,12 +88,14 @@ describe('parse function', function () {
       quiet: true,
       silent: true,
       config: 'foo bar baz',
+      override: true,
       ...defaultOptions,
     };
 
     expect(parse(command, options, osdPackage)).toMatchInlineSnapshot(`
       Object {
         "config": "foo bar baz",
+        "override": true,
         "plugin": "plugin name",
         "pluginDir": <absolute path>/plugins,
         "quiet": true,

src/cli_plugin/install/zip.js

@@ -82,6 +82,7 @@ export function analyzeArchive(archive) {
             plugins.push({
               id: manifest.id,
               stripPrefix: match[1],
+              folderName: manifest.folderName,
 
               // Plugins must specify their version, and by default that version in the plugin
               // manifest should match the version of opensearch-dashboards down to the patch level. If these

src/cli_plugin/install/zip.test.js

@@ -63,6 +63,7 @@ describe('opensearchDashboards cli', function () {
         expect(packages).toMatchInlineSnapshot(`
           Array [
             Object {
+              "folderName": undefined,
               "id": "testPlugin",
               "opensearchDashboardsVersion": "1.0.0",
               "stripPrefix": "opensearch-dashboards/test-plugin",