fix: Plugin resolution for relative paths (#1194)

Resolve plugin modules relative to the current path instead of the internal path.

Fixes #1188

Author: socsieng

src/lib/utils/plugins.ts

@@ -20,7 +20,7 @@ export class PluginHost extends AbstractComponent<Application> {
      */
     load(): boolean {
         const logger = this.application.logger;
-        const plugins = this.plugins.length ? this.plugins : this.discoverNpmPlugins();
+        const plugins = this.plugins.length ? this.resolvePluginPaths(this.plugins) : this.discoverNpmPlugins();
 
         if (plugins.some(plugin => plugin.toLowerCase() === 'none')) {
             return true;
@@ -133,4 +133,29 @@ export class PluginHost extends AbstractComponent<Application> {
             return false;
         }
     }
+
+    /**
+     * Resolves plugin paths to absolute paths from the current working directory
+     * (`process.cwd()`).
+     *
+     * ```txt
+     * ./plugin   -> resolve
+     * ../plugin  -> resolve
+     * plugin     -> don't resolve (module resolution)
+     * /plugin    -> don't resolve (already absolute path)
+     * c:\plugin  -> don't resolve (already absolute path)
+     * ```
+     *
+     * @param plugins
+     */
+    private resolvePluginPaths(plugins: string[]) {
+        const cwd = process.cwd();
+        return plugins.map(plugin => {
+            // treat plugins that start with `.` as relative, requiring resolution
+            if (plugin.startsWith('.')) {
+                return Path.resolve(cwd, plugin);
+            }
+            return plugin;
+        });
+    }
 }

src/test/plugin-host.test.ts

@@ -1,6 +1,7 @@
 import { Application } from '..';
 import Assert = require('assert');
 import * as mockery from 'mockery';
+import * as path from 'path';
 
 describe('PluginHost', function () {
   before (function () {
@@ -27,4 +28,27 @@ describe('PluginHost', function () {
       'typedoc-plugin-2'
     ]);
   });
+
+  it('loads a plugin with relative path', function () {
+    const app = new Application();
+    app.bootstrap({
+      plugin: ['./dist/test/plugins/relative']
+    });
+
+    Assert.deepEqual(app.plugins.plugins, [
+      './dist/test/plugins/relative'
+    ]);
+  });
+
+  it('loads a plugin with absolute path', function () {
+    const app = new Application();
+    const absolutePath = path.resolve(__dirname, './plugins/absolute');
+    app.bootstrap({
+      plugin: [absolutePath]
+    });
+
+    Assert.deepEqual(app.plugins.plugins, [
+        absolutePath
+    ]);
+  });
 });

src/test/plugins/absolute.ts

@@ -0,0 +1,4 @@
+import { Application } from '../..';
+
+module.exports = (pluginHost: Application) => {
+};

src/test/plugins/relative.ts

@@ -0,0 +1,4 @@
+import { Application } from '../..';
+
+module.exports = (pluginHost: Application) => {
+};