dotcom-tool-kit 2.2.3 → 2.3.2

package/lib/config.d.ts

@@ -1,16 +1,16 @@
 import type { Logger } from 'winston';
 import type { HookTask } from './hook';
 import { Conflict } from './conflict';
-import { TaskClass, Hook, Plugin } from '@dotcom-tool-kit/types';
+import { TaskClass, Hook, Plugin, Validated } from '@dotcom-tool-kit/types';
 export interface PluginOptions {
     options: Record<string, unknown>;
     plugin: Plugin;
     forPlugin: Plugin;
 }
-export interface Config {
+export interface RawConfig {
     root: string;
     plugins: {
-        [id: string]: Plugin;
+        [id: string]: Validated<Plugin>;
     };
     resolvedPlugins: Set<Plugin>;
     tasks: {
@@ -26,7 +26,12 @@ export interface Config {
         [id: string]: Hook | Conflict<Hook>;
     };
 }
-export interface ValidConfig extends Config {
+export declare type ValidPluginsConfig = Omit<RawConfig, 'plugins'> & {
+    plugins: {
+        [id: string]: Plugin;
+    };
+};
+export interface ValidConfig extends ValidPluginsConfig {
     tasks: {
         [id: string]: TaskClass;
     };
@@ -40,13 +45,14 @@ export interface ValidConfig extends Config {
         [id: string]: Hook;
     };
 }
-export declare const createConfig: () => Config;
-export declare function validateConfig(config: Config): asserts config is ValidConfig;
+export declare const createConfig: () => RawConfig;
+export declare function validateConfig(config: ValidPluginsConfig): asserts config is ValidConfig;
+export declare function validatePlugins(config: RawConfig): Validated<ValidPluginsConfig>;
 export declare function checkInstall(config: ValidConfig): Promise<void>;
 export declare function loadConfig(logger: Logger, options?: {
     validate?: true;
 }): Promise<ValidConfig>;
 export declare function loadConfig(logger: Logger, options?: {
     validate?: false;
-}): Promise<Config>;
+}): Promise<RawConfig>;
 //# sourceMappingURL=config.d.ts.map

package/lib/config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAErC,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAEtC,OAAO,EAAE,QAAQ,EAA+C,MAAM,YAAY,CAAA;AAElF,OAAO,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAA;AAYhE,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAChC,MAAM,EAAE,MAAM,CAAA;IACd,SAAS,EAAE,MAAM,CAAA;CAClB;AAED,MAAM,WAAW,MAAM;IACrB,IAAI,EAAE,MAAM,CAAA;IACZ,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,CAAA;KAAE,CAAA;IACjC,eAAe,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAC5B,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAA;KAAE,CAAA;IACxD,SAAS,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,QAAQ,GAAG,QAAQ,CAAC,QAAQ,CAAC,CAAA;KAAE,CAAA;IAC1D,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,aAAa,GAAG,QAAQ,CAAC,aAAa,CAAC,GAAG,SAAS,CAAA;KAAE,CAAA;IAC9E,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAA;KAAE,CAAA;CAC/C;AAED,MAAM,WAAW,WAAY,SAAQ,MAAM;IACzC,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,CAAA;KAAE,CAAA;IAClC,SAAS,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,QAAQ,CAAA;KAAE,CAAA;IACrC,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,aAAa,CAAA;KAAE,CAAA;IACxC,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CAAA;CAC9B;AAID,eAAO,MAAM,YAAY,QAAO,MAQ9B,CAAA;AAQF,wBAAgB,cAAc,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,IAAI,WAAW,CAuF5E;AAED,wBAAsB,YAAY,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,CAWrE;AAED,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,IAAI,CAAA;CAAE,GAAG,OAAO,CAAC,WAAW,CAAC,CAAA;AAC/F,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,KAAK,CAAA;CAAE,GAAG,OAAO,CAAC,MAAM,CAAC,CAAA"}
+{"version":3,"file":"config.d.ts","sourceRoot":"","sources":["../src/config.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAErC,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAEtC,OAAO,EAAE,QAAQ,EAA+C,MAAM,YAAY,CAAA;AAElF,OAAO,EAAE,SAAS,EAAE,IAAI,EAAgB,MAAM,EAAmB,SAAS,EAAE,MAAM,wBAAwB,CAAA;AAY1G,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAChC,MAAM,EAAE,MAAM,CAAA;IACd,SAAS,EAAE,MAAM,CAAA;CAClB;AAED,MAAM,WAAW,SAAS;IACxB,IAAI,EAAE,MAAM,CAAA;IACZ,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,CAAC,MAAM,CAAC,CAAA;KAAE,CAAA;IAC5C,eAAe,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAC5B,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAA;KAAE,CAAA;IACxD,SAAS,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,QAAQ,GAAG,QAAQ,CAAC,QAAQ,CAAC,CAAA;KAAE,CAAA;IAC1D,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,aAAa,GAAG,QAAQ,CAAC,aAAa,CAAC,GAAG,SAAS,CAAA;KAAE,CAAA;IAC9E,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAA;KAAE,CAAA;CAC/C;AAED,oBAAY,kBAAkB,GAAG,IAAI,CAAC,SAAS,EAAE,SAAS,CAAC,GAAG;IAC5D,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,CAAA;KAAE,CAAA;CAClC,CAAA;AAED,MAAM,WAAW,WAAY,SAAQ,kBAAkB;IACrD,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,CAAA;KAAE,CAAA;IAClC,SAAS,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,QAAQ,CAAA;KAAE,CAAA;IACrC,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,aAAa,CAAA;KAAE,CAAA;IACxC,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CAAA;CAC9B;AAID,eAAO,MAAM,YAAY,QAAO,SAQ9B,CAAA;AAQF,wBAAgB,cAAc,CAAC,MAAM,EAAE,kBAAkB,GAAG,OAAO,CAAC,MAAM,IAAI,WAAW,CAuFxF;AAED,wBAAgB,eAAe,CAAC,MAAM,EAAE,SAAS,GAAG,SAAS,CAAC,kBAAkB,CAAC,CAKhF;AAED,wBAAsB,YAAY,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC,CAWrE;AAED,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,IAAI,CAAA;CAAE,GAAG,OAAO,CAAC,WAAW,CAAC,CAAA;AAC/F,wBAAgB,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,KAAK,CAAA;CAAE,GAAG,OAAO,CAAC,SAAS,CAAC,CAAA"}

package/lib/config.js

@@ -1,11 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadConfig = exports.checkInstall = exports.validateConfig = exports.createConfig = void 0;
+exports.loadConfig = exports.checkInstall = exports.validatePlugins = exports.validateConfig = exports.createConfig = void 0;
 const tslib_1 = require("tslib");
 const path_1 = (0, tslib_1.__importDefault)(require("path"));
 const plugin_1 = require("./plugin");
 const conflict_1 = require("./conflict");
 const error_1 = require("@dotcom-tool-kit/error");
+const types_1 = require("@dotcom-tool-kit/types");
 const messages_1 = require("./messages");
 const coreRoot = path_1.default.resolve(__dirname, '../');
 const createConfig = () => ({
@@ -88,6 +89,11 @@ function validateConfig(config) {
     }
 }
 exports.validateConfig = validateConfig;
+function validatePlugins(config) {
+    const validatedPlugins = (0, types_1.reduceValidated)(Object.entries(config.plugins).map(([id, plugin]) => (0, types_1.mapValidated)(plugin, (p) => [id, p])));
+    return (0, types_1.mapValidated)(validatedPlugins, (plugins) => ({ ...config, plugins: Object.fromEntries(plugins) }));
+}
+exports.validatePlugins = validatePlugins;
 async function checkInstall(config) {
     const definedHooks = (0, conflict_1.withoutConflicts)(Object.values(config.hooks));
     const uninstalledHooks = await asyncFilter(definedHooks, async (hook) => {
@@ -104,10 +110,23 @@ async function loadConfig(logger, { validate = true } = {}) {
     const config = (0, exports.createConfig)();
     // start loading config and child plugins, starting from the consumer app directory
     const rootPlugin = await (0, plugin_1.loadPlugin)('app root', config, logger);
+    if (!rootPlugin.valid) {
+        const error = new error_1.ToolKitError('root plugin was not valid!');
+        error.details = rootPlugin.reasons.join('\n\n');
+        throw error;
+    }
+    const validRootPlugin = rootPlugin.value;
+    const validatedPluginConfig = validatePlugins(config);
+    if (!validatedPluginConfig.valid) {
+        const error = new error_1.ToolKitError('config was not valid!');
+        error.details = validatedPluginConfig.reasons.join('\n\n');
+        throw error;
+    }
+    const validPluginConfig = validatedPluginConfig.value;
     // collate root plugin and descendent hooks, options etc into config
-    (0, plugin_1.resolvePlugin)(rootPlugin, config, logger);
+    (0, plugin_1.resolvePlugin)(validRootPlugin, validPluginConfig, logger);
     if (validate) {
-        validateConfig(config);
+        validateConfig(validPluginConfig);
     }
     return config;
 }

package/lib/index.js

@@ -76,8 +76,8 @@ exports.runTasks = runTasks;
 async function listPlugins(logger) {
     const config = await (0, config_1.loadConfig)(logger, { validate: false });
     const rootPlugin = config.plugins['app root'];
-    if (rootPlugin) {
-        logger.info((0, messages_1.formatPluginTree)(rootPlugin).join('\n'));
+    if (rootPlugin === null || rootPlugin === void 0 ? void 0 : rootPlugin.valid) {
+        logger.info((0, messages_1.formatPluginTree)(rootPlugin.value).join('\n'));
     }
 }
 exports.listPlugins = listPlugins;

package/lib/install.d.ts

@@ -1,4 +1,4 @@
 import type { Logger } from 'winston';
-import { Config } from './config';
-export default function installHooks(logger: Logger): Promise<Config>;
+import { ValidConfig } from './config';
+export default function installHooks(logger: Logger): Promise<ValidConfig>;
 //# sourceMappingURL=install.d.ts.map

package/lib/install.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"install.d.ts","sourceRoot":"","sources":["../src/install.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AACrC,OAAO,EAAE,MAAM,EAAc,MAAM,UAAU,CAAA;AAE7C,wBAA8B,YAAY,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAqC1E"}
+{"version":3,"file":"install.d.ts","sourceRoot":"","sources":["../src/install.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AACrC,OAAO,EAAc,WAAW,EAAE,MAAM,UAAU,CAAA;AAGlD,wBAA8B,YAAY,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC,CAqC/E"}

package/lib/plugin.d.ts

@@ -1,7 +1,7 @@
+import { Plugin, PluginModule, Validated } from '@dotcom-tool-kit/types';
 import type { Logger } from 'winston';
-import { Config } from './config';
-import { Plugin, PluginModule } from '@dotcom-tool-kit/types';
-export declare function validatePlugin(plugin: unknown): asserts plugin is PluginModule;
-export declare function loadPlugin(id: string, config: Config, logger: Logger, parent?: Plugin): Promise<Plugin>;
-export declare function resolvePlugin(plugin: Plugin, config: Config, logger: Logger): void;
+import { RawConfig, ValidPluginsConfig } from './config';
+export declare function validatePlugin(plugin: unknown): Validated<PluginModule>;
+export declare function loadPlugin(id: string, config: RawConfig, logger: Logger, parent?: Plugin): Promise<Validated<Plugin>>;
+export declare function resolvePlugin(plugin: Plugin, config: ValidPluginsConfig, logger: Logger): void;
 //# sourceMappingURL=plugin.d.ts.map

package/lib/plugin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAGrC,OAAO,EAAE,MAAM,EAAiB,MAAM,UAAU,CAAA;AAIhD,OAAO,EAAQ,MAAM,EAAE,YAAY,EAAQ,MAAM,wBAAwB,CAAA;AAezE,wBAAgB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,MAAM,IAAI,YAAY,CAmB9E;AASD,wBAAsB,UAAU,CAC9B,EAAE,EAAE,MAAM,EACV,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,EACd,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,MAAM,CAAC,CAmCjB;AAED,wBAAgB,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAsIlF"}
+{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AACA,OAAO,EAKL,MAAM,EACN,YAAY,EAIZ,SAAS,EACV,MAAM,wBAAwB,CAAA;AAG/B,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AACrC,OAAO,EAAiB,SAAS,EAAE,kBAAkB,EAAE,MAAM,UAAU,CAAA;AAmBvE,wBAAgB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,SAAS,CAAC,YAAY,CAAC,CAgDvE;AAoBD,wBAAsB,UAAU,CAC9B,EAAE,EAAE,MAAM,EACV,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,MAAM,EACd,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAyD5B;AAED,wBAAgB,aAAa,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,kBAAkB,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAsI9F"}

package/lib/plugin.js

@@ -2,12 +2,12 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolvePlugin = exports.loadPlugin = exports.validatePlugin = void 0;
 const tslib_1 = require("tslib");
+const logger_1 = require("@dotcom-tool-kit/logger");
+const types_1 = require("@dotcom-tool-kit/types");
+const lodash_isplainobject_1 = (0, tslib_1.__importDefault)(require("lodash.isplainobject"));
 const resolve_from_1 = (0, tslib_1.__importDefault)(require("resolve-from"));
 const conflict_1 = require("./conflict");
 const rc_file_1 = require("./rc-file");
-const error_1 = require("@dotcom-tool-kit/error");
-const types_1 = require("@dotcom-tool-kit/types");
-const lodash_isplainobject_1 = (0, tslib_1.__importDefault)(require("lodash.isplainobject"));
 function isDescendent(possibleAncestor, possibleDescendent) {
     if (!possibleDescendent.parent) {
         return false;
@@ -19,24 +19,61 @@ function isDescendent(possibleAncestor, possibleDescendent) {
         return isDescendent(possibleAncestor, possibleDescendent.parent);
     }
 }
+const indentReasons = (reasons) => reasons.replace(/\n/g, '\n  ');
 function validatePlugin(plugin) {
+    var _a, _b;
+    const errors = [];
     const rawPlugin = plugin;
-    if (rawPlugin.tasks &&
-        !(Array.isArray(rawPlugin.tasks) && rawPlugin.tasks.every((task) => task.prototype instanceof types_1.Task))) {
-        throw new error_1.ToolKitError('tasks are not valid');
+    if (rawPlugin.tasks) {
+        if (!Array.isArray(rawPlugin.tasks)) {
+            errors.push(`the exported ${logger_1.styles.code('tasks')} value from this plugin is not an array`);
+        }
+        else {
+            const validatedTasks = (0, types_1.reduceValidated)(rawPlugin.tasks.map((task) => (0, types_1.mapValidationError)(types_1.Task.isCompatible(task), (reasons) => [
+                `the task ${logger_1.styles.task(task.name)} is not a compatible instance of ${logger_1.styles.code('Task')}:\n  - ${reasons.join('\n  - ')}`
+            ])));
+            if (!validatedTasks.valid) {
+                errors.push(...validatedTasks.reasons);
+            }
+        }
+    }
+    if (rawPlugin.hooks) {
+        if (!(0, lodash_isplainobject_1.default)(rawPlugin.hooks)) {
+            errors.push(`the exported ${logger_1.styles.code('hooks')} value from this plugin is not an object`);
+        }
+        else {
+            const validatedHooks = (0, types_1.reduceValidated)(Object.entries(rawPlugin.hooks).map(([id, hook]) => (0, types_1.mapValidationError)(types_1.Hook.isCompatible(hook), (reasons) => [
+                `the hook ${logger_1.styles.hook(id)} is not a compatible instance of ${logger_1.styles.code('Hook')}:\n  - ${reasons.join('\n  - ')}`
+            ])));
+            if (!validatedHooks.valid) {
+                errors.push(...validatedHooks.reasons);
+            }
+        }
     }
-    if (rawPlugin.hooks &&
-        !((0, lodash_isplainobject_1.default)(rawPlugin.hooks) &&
-            Object.values(rawPlugin.hooks).every((hook) => hook.prototype instanceof types_1.Hook))) {
-        throw new error_1.ToolKitError('hooks are not valid');
+    if (errors.length > 0) {
+        return { valid: false, reasons: errors };
+    }
+    else {
+        const pluginModule = { tasks: (_a = rawPlugin.tasks) !== null && _a !== void 0 ? _a : [], hooks: (_b = rawPlugin.hooks) !== null && _b !== void 0 ? _b : {} };
+        return { valid: true, value: pluginModule };
     }
 }
 exports.validatePlugin = validatePlugin;
 async function importPlugin(pluginPath) {
-    // pluginPath is an absolute resolved path to a plugin module as found from its parent
-    const pluginModule = await Promise.resolve().then(() => (0, tslib_1.__importStar)(require(pluginPath)));
-    validatePlugin(pluginModule);
-    return pluginModule;
+    try {
+        // pluginPath is an absolute resolved path to a plugin module as found from its parent
+        const pluginModule = (await Promise.resolve().then(() => (0, tslib_1.__importStar)(require(pluginPath))));
+        return validatePlugin(pluginModule);
+    }
+    catch (e) {
+        const err = e;
+        return {
+            valid: false,
+            reasons: [
+                `an error was thrown when loading this plugin's entrypoint:\n  ${logger_1.styles.code(indentReasons(err.toString()))}`
+            ]
+        };
+    }
 }
 async function loadPlugin(id, config, logger, parent) {
     // don't load duplicate plugins
@@ -45,22 +82,44 @@ async function loadPlugin(id, config, logger, parent) {
     }
     // load plugin relative to the parent plugin
     const root = parent ? parent.root : process.cwd();
-    const pluginRoot = id === 'app root' ? root : (0, resolve_from_1.default)(root, id);
+    let pluginRoot;
+    try {
+        pluginRoot = id === 'app root' ? root : (0, resolve_from_1.default)(root, id);
+    }
+    catch (e) {
+        return { valid: false, reasons: [`could not find path for name ${logger_1.styles.filepath(id)}`] };
+    }
     const plugin = {
-        id,
-        root: pluginRoot,
-        parent
+        valid: true,
+        value: {
+            id,
+            root: pluginRoot,
+            parent
+        }
     };
     config.plugins[id] = plugin;
     // start loading rc file in the background
     const rcFilePromise = (0, rc_file_1.loadToolKitRC)(pluginRoot);
     // start loading module in the background
-    const pluginModulePromise = id === 'app root' ? Promise.resolve(undefined) : importPlugin(pluginRoot);
-    plugin.rcFile = await rcFilePromise;
+    const pluginModulePromise = id === 'app root' ? Promise.resolve({ valid: true, value: undefined }) : importPlugin(pluginRoot);
+    plugin.value.rcFile = await rcFilePromise;
     // start loading child plugins in the background
-    const childrenPromise = Promise.all(plugin.rcFile.plugins.map((child) => loadPlugin(child, config, logger, plugin)));
-    [plugin.module, plugin.children] = await Promise.all([pluginModulePromise, childrenPromise]);
-    return plugin;
+    const childrenPromise = Promise.all(plugin.value.rcFile.plugins.map((child) => loadPlugin(child, config, logger, plugin.value)));
+    // wait for pending promises concurrently
+    const [module, children] = await Promise.all([pluginModulePromise, childrenPromise]);
+    const validatedModule = (0, types_1.mapValidationError)(module, (reasons) => [
+        indentReasons(`plugin ${logger_1.styles.plugin(id)} failed to load because:\n- ${reasons.join('\n- ')}`)
+    ]);
+    const validatedChildren = (0, types_1.mapValidationError)((0, types_1.reduceValidated)(children), (reasons) => [
+        indentReasons(`some child plugins of ${logger_1.styles.plugin(id)} failed to load:\n- ${reasons.join('\n- ')}`)
+    ]);
+    return (0, types_1.mapValidated)((0, types_1.joinValidated)(validatedModule, validatedChildren), ([module, children]) => {
+        // avoid cloning the plugin value with an object spread as we do object
+        // reference comparisons in multiple places
+        plugin.value.module = module;
+        plugin.value.children = children;
+        return plugin.value;
+    });
 }
 exports.loadPlugin = loadPlugin;
 function resolvePlugin(plugin, config, logger) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dotcom-tool-kit",
-  "version": "2.2.3",
+  "version": "2.3.2",
   "description": "modern, maintainable, modular developer tooling for FT.com projects",
   "author": "FT.com Platforms Team <platforms-team.customer-products@ft.com>",
   "license": "MIT",
@@ -23,17 +23,17 @@
     "@jest/globals": "^27.4.6",
     "@types/lodash.merge": "^4.6.6",
     "@types/node": "^12.20.24",
-    "@dotcom-tool-kit/backend-app": "^2.0.5",
-    "@dotcom-tool-kit/heroku": "^2.0.4",
-    "@dotcom-tool-kit/webpack": "^2.1.2",
-    "@dotcom-tool-kit/babel": "^2.0.3",
-    "@dotcom-tool-kit/circleci": "^2.1.1",
-    "@dotcom-tool-kit/npm": "^2.0.4",
-    "@dotcom-tool-kit/circleci-heroku": "^2.0.5",
-    "@dotcom-tool-kit/frontend-app": "^2.1.3",
-    "@dotcom-tool-kit/eslint": "^2.1.2",
-    "@dotcom-tool-kit/mocha": "^2.1.0",
-    "@dotcom-tool-kit/n-test": "^2.0.4",
+    "@dotcom-tool-kit/backend-app": "^2.0.8",
+    "@dotcom-tool-kit/heroku": "^2.0.7",
+    "@dotcom-tool-kit/webpack": "^2.1.5",
+    "@dotcom-tool-kit/babel": "^2.0.6",
+    "@dotcom-tool-kit/circleci": "^2.1.4",
+    "@dotcom-tool-kit/npm": "^2.0.7",
+    "@dotcom-tool-kit/circleci-heroku": "^2.0.8",
+    "@dotcom-tool-kit/frontend-app": "^2.1.6",
+    "@dotcom-tool-kit/eslint": "^2.1.5",
+    "@dotcom-tool-kit/mocha": "^2.1.3",
+    "@dotcom-tool-kit/n-test": "^2.1.1",
     "chai": "^4.3.4",
     "globby": "^10.0.2",
     "ts-node": "^8.10.2",
@@ -41,9 +41,9 @@
   },
   "dependencies": {
     "@dotcom-tool-kit/error": "^2.0.0",
-    "@dotcom-tool-kit/logger": "^2.0.0",
-    "@dotcom-tool-kit/options": "^2.0.3",
-    "@dotcom-tool-kit/types": "^2.3.0",
+    "@dotcom-tool-kit/logger": "^2.1.1",
+    "@dotcom-tool-kit/options": "^2.0.6",
+    "@dotcom-tool-kit/types": "^2.5.1",
     "@dotcom-tool-kit/wait-for-ok": "^2.0.0",
     "cosmiconfig": "^7.0.0",
     "lodash.merge": "^4.6.2",