dotcom-tool-kit 1.0.0-beta.9 → 1.0.2-beta.2

package/bin/run

@@ -1,34 +1,45 @@
 #!/usr/bin/env node
 
-const c = require('ansi-colors')
 const argv = require('minimist')(process.argv.slice(2), {
-   boolean: ['help', 'install']
+  boolean: ['help', 'install'],
+  '--': true
 })
 
 const { runTasks, showHelp, installHooks } = require('../lib')
-const { styles } = require('../lib/messages')
+const { rootLogger, styles } = require('@dotcom-tool-kit/logger')
 
 async function main() {
-   try {
-      if(argv.install) {
-         await installHooks()
-      } else if(argv.help || argv._.length === 0) {
-         await showHelp(argv._)
+  try {
+    if (argv.install) {
+      await installHooks(rootLogger)
+    } else if (argv.help || argv._.length === 0) {
+      await showHelp(rootLogger, argv._)
+    } else {
+      if (argv['--'].length > 0) {
+        // The `--` in a command such as `dotcom-tool-kit test:staged --`
+        // delineates between hooks and file patterns. For example, when the
+        // lint-staged task is run it will identify the files that are staged
+        // and match its glob patterns and append them to the command, so that
+        // the command becomes something like `dotcom-tool-kit test:staged --
+        // index.js`. When this command is executed it runs the configured task
+        // where the file path arguments would then be extracted.
+        await runTasks(rootLogger, argv._, argv['--'])
       } else {
-         await runTasks(argv._)
-      }
-   } catch(error) {
-      if(error.details) {
-         console.error(
-            styles.error(error.message) + '\n\n' +
-            error.details
-         )
-      } else {
-         console.error(error.stack)
+        await runTasks(rootLogger, argv._)
       }
+    }
+  } catch (error) {
+    if (error.details) {
+      rootLogger.error('', { skipformat: true })
+      rootLogger.error(error.message)
+      rootLogger.error(styles.ruler(), { skipformat: true })
+      rootLogger.error(error.details, { skipformat: true })
+    } else {
+      rootLogger.error(error.stack)
+    }
 
-      process.exitCode = error.exitCode || 1
-   }
+    process.exitCode = error.exitCode || 1
+  }
 }
 
 main()

package/lib/config.d.ts

@@ -1,7 +1,7 @@
-import type { TaskClass } from '@dotcom-tool-kit/task';
-import type { HookTask, HookClass } from './hook';
-import { Plugin } from './plugin';
+import type { Logger } from 'winston';
+import type { HookTask } from './hook';
 import { Conflict } from './conflict';
+import type { TaskClass, Hook, Plugin } from '@dotcom-tool-kit/types';
 export interface PluginOptions {
     options: Record<string, unknown>;
     plugin: Plugin;
@@ -22,7 +22,7 @@ export interface Config {
         [id: string]: PluginOptions | Conflict<PluginOptions> | undefined;
     };
     hooks: {
-        [id: string]: HookClass | Conflict<HookClass>;
+        [id: string]: Hook | Conflict<Hook>;
     };
 }
 export interface ValidConfig extends Config {
@@ -36,18 +36,15 @@ export interface ValidConfig extends Config {
         [id: string]: PluginOptions;
     };
     hooks: {
-        [id: string]: HookClass;
+        [id: string]: Hook;
     };
 }
-export declare function validateConfig(config: Config, { checkInstall }?: {
-    checkInstall?: boolean | undefined;
-}): Promise<ValidConfig>;
-export declare function loadConfig(options?: {
+export declare function validateConfig(config: Config): Promise<ValidConfig>;
+export declare function checkInstall(config: ValidConfig): Promise<void>;
+export declare function loadConfig(logger: Logger, options?: {
     validate?: true;
-    checkInstall?: boolean;
 }): Promise<ValidConfig>;
-export declare function loadConfig(options?: {
+export declare function loadConfig(logger: Logger, options?: {
     validate?: false;
-    checkInstall?: boolean;
 }): Promise<Config>;
 //# 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":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAA;AACtD,OAAO,KAAK,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AACjD,OAAO,EAAoB,MAAM,EAAE,MAAM,UAAU,CAAA;AACnD,OAAO,EAAE,QAAQ,EAAmC,MAAM,YAAY,CAAA;AAYtE,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,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,SAAS,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAA;KAAE,CAAA;CACzD;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,SAAS,CAAA;KAAE,CAAA;CACnC;AAmBD,wBAAsB,cAAc,CAAC,MAAM,EAAE,MAAM,EAAE,EAAE,YAAmB,EAAE;;CAAK,GAAG,OAAO,CAAC,WAAW,CAAC,CA0EvG;AAED,wBAAgB,UAAU,CAAC,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,IAAI,CAAC;IAAC,YAAY,CAAC,EAAE,OAAO,CAAA;CAAE,GAAG,OAAO,CAAC,WAAW,CAAC,CAAA;AACvG,wBAAgB,UAAU,CAAC,OAAO,CAAC,EAAE;IAAE,QAAQ,CAAC,EAAE,KAAK,CAAC;IAAC,YAAY,CAAC,EAAE,OAAO,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,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAA;AAYrE,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,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;AAmBD,wBAAsB,cAAc,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC,CAyFzE;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"}

package/lib/config.js

@@ -1,8 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadConfig = exports.validateConfig = void 0;
+exports.loadConfig = exports.checkInstall = exports.validateConfig = void 0;
 const tslib_1 = require("tslib");
-const path_1 = tslib_1.__importDefault(require("path"));
+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");
@@ -20,39 +20,56 @@ async function asyncFilter(items, predicate) {
     const results = await Promise.all(items.map(async (item) => ({ item, keep: await predicate(item) })));
     return results.filter(({ keep }) => keep).map(({ item }) => item);
 }
-async function validateConfig(config, { checkInstall = true } = {}) {
-    const hookTaskConflicts = conflict_1.findConflicts(Object.values(config.hookTasks));
-    const hookConflicts = conflict_1.findConflicts(Object.values(config.hooks));
-    const taskConflicts = conflict_1.findConflicts(Object.values(config.tasks));
-    const optionConflicts = conflict_1.findConflicts(Object.values(config.options));
+async function validateConfig(config) {
+    const hookTaskConflicts = (0, conflict_1.findConflicts)(Object.values(config.hookTasks));
+    const hookConflicts = (0, conflict_1.findConflicts)(Object.values(config.hooks));
+    const taskConflicts = (0, conflict_1.findConflicts)(Object.values(config.tasks));
+    const optionConflicts = (0, conflict_1.findConflicts)(Object.values(config.options));
+    const definedHookTaskConflicts = hookTaskConflicts.filter((conflict) => {
+        return conflict.conflicting[0].id in config.hooks;
+    });
     let shouldThrow = false;
-    const error = new error_1.ToolKitError('There are problems with your Tool Kit configuration.');
+    const error = new error_1.ToolKitConflictError('There are problems with your Tool Kit configuration.', hookTaskConflicts.map((conflict) => ({
+        hook: conflict.conflicting[0].id,
+        conflictingTasks: conflict.conflicting.flatMap((hook) => hook.tasks.map((task) => ({ task, plugin: hook.plugin.id })))
+    })));
     error.details = '';
     if (hookConflicts.length > 0 ||
-        hookTaskConflicts.length > 0 ||
+        definedHookTaskConflicts.length > 0 ||
         taskConflicts.length > 0 ||
         optionConflicts.length > 0) {
         shouldThrow = true;
         if (hookConflicts.length) {
-            error.details += messages_1.formatHookConflicts(hookConflicts);
+            error.details += (0, messages_1.formatHookConflicts)(hookConflicts);
         }
-        if (hookTaskConflicts.length) {
-            error.details += messages_1.formatHookTaskConflicts(hookTaskConflicts);
+        if (definedHookTaskConflicts.length) {
+            error.details += (0, messages_1.formatHookTaskConflicts)(definedHookTaskConflicts);
         }
         if (taskConflicts.length) {
-            error.details += messages_1.formatTaskConflicts(taskConflicts);
+            error.details += (0, messages_1.formatTaskConflicts)(taskConflicts);
         }
         if (optionConflicts.length) {
-            error.details += messages_1.formatOptionConflicts(optionConflicts);
+            error.details += (0, messages_1.formatOptionConflicts)(optionConflicts);
         }
     }
-    const configuredHookTasks = conflict_1.withoutConflicts(Object.values(config.hookTasks));
-    const definedHooks = conflict_1.withoutConflicts(Object.values(config.hooks));
+    const configuredHookTasks = (0, conflict_1.withoutConflicts)(Object.values(config.hookTasks));
     const definedHookIds = new Set(Object.keys(config.hooks));
-    const undefinedHookTasks = configuredHookTasks.filter((hook) => !definedHookIds.has(hook.id));
+    const undefinedHookTasks = configuredHookTasks.filter((hookTask) => {
+        // we only care about undefined hooks that were configured by the app, not default config from plugins
+        const fromApp = hookTask.plugin.root === process.cwd();
+        const hookDefined = definedHookIds.has(hookTask.id);
+        return fromApp && !hookDefined;
+    });
     if (undefinedHookTasks.length > 0) {
         shouldThrow = true;
-        error.details += messages_1.formatUndefinedHookTasks(undefinedHookTasks, Array.from(definedHookIds));
+        error.details += (0, messages_1.formatUndefinedHookTasks)(undefinedHookTasks, Array.from(definedHookIds));
+    }
+    const unusedOptions = Object.entries(config.options)
+        .filter(([, option]) => option && !(0, conflict_1.isConflict)(option) && !option.forPlugin && option.plugin.root === process.cwd())
+        .map(([id]) => id);
+    if (unusedOptions.length > 0) {
+        shouldThrow = true;
+        error.details += (0, messages_1.formatUnusedOptions)(unusedOptions, Object.keys(config.plugins));
     }
     const missingTasks = configuredHookTasks
         .map((hook) => ({
@@ -62,17 +79,7 @@ async function validateConfig(config, { checkInstall = true } = {}) {
         .filter(({ tasks }) => tasks.length > 0);
     if (missingTasks.length > 0) {
         shouldThrow = true;
-        error.details += messages_1.formatMissingTasks(missingTasks, Object.keys(config.tasks));
-    }
-    if (checkInstall) {
-        const uninstalledHooks = await asyncFilter(definedHooks, async (Hook) => {
-            const hook = new Hook();
-            return !(await hook.check());
-        });
-        if (uninstalledHooks.length > 0) {
-            shouldThrow = true;
-            error.details += messages_1.formatUninstalledHooks(uninstalledHooks);
-        }
+        error.details += (0, messages_1.formatMissingTasks)(missingTasks, Object.keys(config.tasks));
     }
     if (shouldThrow) {
         throw error;
@@ -80,12 +87,28 @@ async function validateConfig(config, { checkInstall = true } = {}) {
     return config;
 }
 exports.validateConfig = validateConfig;
-async function loadConfig({ validate = true, checkInstall = true } = {}) {
+async function checkInstall(config) {
+    const definedHooks = (0, conflict_1.withoutConflicts)(Object.values(config.hooks));
+    const uninstalledHooks = await asyncFilter(definedHooks, async (hook) => {
+        return !(await hook.check());
+    });
+    if (uninstalledHooks.length > 0) {
+        const error = new error_1.ToolKitError('There are problems with your Tool Kit installation.');
+        error.details = (0, messages_1.formatUninstalledHooks)(uninstalledHooks);
+        throw error;
+    }
+}
+exports.checkInstall = checkInstall;
+async function loadConfig(logger, { validate = true } = {}) {
     // start loading config and child plugins, starting from the consumer app directory
-    const config = await plugin_1.loadPluginConfig({
+    const config = await (0, plugin_1.loadPluginConfig)(logger, {
         id: 'app root',
         root: process.cwd()
     }, createConfig());
-    return validate ? validateConfig(config, { checkInstall }) : config;
+    return validate ? validateConfig(config) : config;
 }
 exports.loadConfig = loadConfig;
+// abstract class TestBase {}
+// class Test extends TestBase {}
+// const testBase = TestBase
+// const test = new testBase()

package/lib/conflict.d.ts

@@ -1,4 +1,4 @@
-import type { Plugin } from './plugin';
+import type { Plugin } from '@dotcom-tool-kit/types';
 export interface Conflict<T> {
     plugin: Plugin;
     conflicting: T[];

package/lib/conflict.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"conflict.d.ts","sourceRoot":"","sources":["../src/conflict.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,UAAU,CAAA;AAEtC,MAAM,WAAW,QAAQ,CAAC,CAAC;IACzB,MAAM,EAAE,MAAM,CAAA;IACd,WAAW,EAAE,CAAC,EAAE,CAAA;CACjB;AAED,wBAAgB,UAAU,CAAC,CAAC,EAAE,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,QAAQ,CAAC,CAAC,CAAC,CAElE;AAED,wBAAgB,aAAa,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,GAAG,QAAQ,CAAC,CAAC,CAAC,EAAE,CAU7E;AAED,wBAAgB,gBAAgB,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAUtE"}
+{"version":3,"file":"conflict.d.ts","sourceRoot":"","sources":["../src/conflict.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAA;AAEpD,MAAM,WAAW,QAAQ,CAAC,CAAC;IACzB,MAAM,EAAE,MAAM,CAAA;IACd,WAAW,EAAE,CAAC,EAAE,CAAA;CACjB;AAED,wBAAgB,UAAU,CAAC,CAAC,EAAE,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,QAAQ,CAAC,CAAC,CAAC,CAElE;AAED,wBAAgB,aAAa,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,GAAG,QAAQ,CAAC,CAAC,CAAC,EAAE,CAU7E;AAED,wBAAgB,gBAAgB,CAAC,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAUtE"}

package/lib/help.d.ts

@@ -1,2 +1,3 @@
-export default function showHelp(hooks: string[]): Promise<void>;
+import type { Logger } from 'winston';
+export default function showHelp(logger: Logger, hooks: string[]): Promise<void>;
 //# sourceMappingURL=help.d.ts.map

package/lib/help.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"help.d.ts","sourceRoot":"","sources":["../src/help.ts"],"names":[],"mappings":"AAGA,wBAA8B,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAqDrE"}
+{"version":3,"file":"help.d.ts","sourceRoot":"","sources":["../src/help.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAErC,wBAA8B,QAAQ,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAiErF"}

package/lib/help.js

@@ -1,43 +1,54 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const config_1 = require("./config");
-const messages_1 = require("./messages");
-async function showHelp(hooks) {
-    const config = await config_1.loadConfig({ checkInstall: false });
+const options_1 = require("@dotcom-tool-kit/options");
+const logger_1 = require("@dotcom-tool-kit/logger");
+async function showHelp(logger, hooks) {
+    const config = await (0, config_1.loadConfig)(logger);
     if (hooks.length === 0) {
         hooks = Object.keys(config.hooks).sort();
     }
+    for (const pluginOptions of Object.values(config.options)) {
+        if (pluginOptions.forPlugin) {
+            (0, options_1.setOptions)(pluginOptions.forPlugin.id, pluginOptions.options);
+        }
+    }
+    await (0, config_1.checkInstall)(config);
     const missingHooks = hooks.filter((hook) => !config.hooks[hook]);
-    console.log(`
-🧰 ${messages_1.styles.title(`welcome to ${messages_1.styles.app('Tool Kit')}!`)}
+    logger.info(`
+🧰 ${logger_1.styles.title(`welcome to ${logger_1.styles.app('Tool Kit')}!`)}
 
 Tool Kit is modern, maintainable & modular developer tooling for FT.com projects.
 
-${messages_1.styles.URL('https://github.com/financial-times/dotcom-tool-kit')}
+${logger_1.styles.URL('https://github.com/financial-times/dotcom-tool-kit')}
 
-${messages_1.styles.ruler()}
-${messages_1.styles.dim(hooks.length === 0
-        ? 'available hooks'
-        : `help for ${hooks.length - missingHooks.length} ${hooks.length - missingHooks.length > 1 ? 'hooks' : 'hook'}`)}:
+${logger_1.styles.ruler()}
+${Object.keys(config.hooks).length === 0
+        ? `there are no hooks available. you'll need to install plugins that define hooks to be able to run Tool Kit tasks.`
+        : logger_1.styles.dim(hooks.length === 0
+            ? 'available hooks'
+            : `help for ${hooks.length - missingHooks.length} ${hooks.length - missingHooks.length > 1 ? 'hooks' : 'hook'}`)}:
 `);
     for (const hook of hooks) {
         const Hook = config.hooks[hook];
         if (Hook) {
             const tasks = config.hookTasks[hook];
-            console.log(`${messages_1.styles.heading(hook)}
-${Hook.description ? Hook.description + '\n' : ''}
+            /* eslint-disable @typescript-eslint/no-explicit-any -- Object.constructor does not consider static properties */
+            logger.info(`${logger_1.styles.heading(hook)}
+${Hook.constructor.description ? Hook.constructor.description + '\n' : ''}
 ${tasks && tasks.tasks.length
                 ? `runs ${tasks.tasks.length > 1 ? 'these tasks' : 'this task'}:
 ${tasks.tasks
-                    .map((task) => `- ${messages_1.styles.task(task)} ${messages_1.styles.dim(config.tasks[task].description)}`)
+                    .map((task) => `- ${logger_1.styles.task(task)} ${logger_1.styles.dim(config.tasks[task].description)}`)
                     .join('\n')}`
-                : messages_1.styles.dim('no tasks configured to run on this hook.')}
-${messages_1.styles.ruler()}
+                : logger_1.styles.dim('no tasks configured to run on this hook.')}
+${logger_1.styles.ruler()}
 `);
+            /*eslint-enable @typescript-eslint/no-explicit-any */
         }
     }
     if (missingHooks.length) {
-        console.warn(messages_1.styles.warning(`no such ${missingHooks.length > 1 ? 'hooks' : 'hook'} ${missingHooks.map(messages_1.styles.hook).join(', ')}`));
+        logger.warn(`no such ${missingHooks.length > 1 ? 'hooks' : 'hook'} ${missingHooks.map(logger_1.styles.hook).join(', ')}`);
     }
 }
 exports.default = showHelp;

package/lib/hook.d.ts

@@ -1,17 +1,7 @@
-import type { Plugin } from './plugin';
+import type { Plugin } from '@dotcom-tool-kit/types';
 export interface HookTask {
     id: string;
     plugin: Plugin;
     tasks: string[];
 }
-export interface HookClass {
-    id?: string;
-    plugin?: Plugin;
-    description?: string;
-    new (): Hook;
-}
-export interface Hook {
-    check(): Promise<boolean>;
-    install(): Promise<void>;
-}
 //# sourceMappingURL=hook.d.ts.map

package/lib/hook.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"hook.d.ts","sourceRoot":"","sources":["../src/hook.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,UAAU,CAAA;AAEtC,MAAM,WAAW,QAAQ;IACvB,EAAE,EAAE,MAAM,CAAA;IACV,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,EAAE,CAAA;CAChB;AAED,MAAM,WAAW,SAAS;IACxB,EAAE,CAAC,EAAE,MAAM,CAAA;IACX,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,QAAQ,IAAI,CAAA;CACb;AAED,MAAM,WAAW,IAAI;IACnB,KAAK,IAAI,OAAO,CAAC,OAAO,CAAC,CAAA;IACzB,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC,CAAA;CACzB"}
+{"version":3,"file":"hook.d.ts","sourceRoot":"","sources":["../src/hook.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAA;AAEpD,MAAM,WAAW,QAAQ;IACvB,EAAE,EAAE,MAAM,CAAA;IACV,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,EAAE,CAAA;CAChB"}

package/lib/index.d.ts

@@ -1,4 +1,5 @@
-export declare function runTasks(hooks: string[]): Promise<void>;
+import type { Logger } from 'winston';
+export declare function runTasks(logger: Logger, hooks: string[], files?: string[]): Promise<void>;
 export { default as showHelp } from './help';
 export { default as installHooks } from './install';
 //# sourceMappingURL=index.d.ts.map

package/lib/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAIA,wBAAsB,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAqC7D;AAED,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAC5C,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,WAAW,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAQrC,wBAAsB,QAAQ,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,EAAE,KAAK,CAAC,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CA8E/F;AAED,OAAO,EAAE,OAAO,IAAI,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAC5C,OAAO,EAAE,OAAO,IAAI,YAAY,EAAE,MAAM,WAAW,CAAA"}

package/lib/index.js

@@ -6,10 +6,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.installHooks = exports.showHelp = exports.runTasks = void 0;
 const error_1 = require("@dotcom-tool-kit/error");
 const config_1 = require("./config");
-const messages_1 = require("./messages");
-async function runTasks(hooks) {
-    var _a;
-    const config = await config_1.loadConfig();
+const options_1 = require("@dotcom-tool-kit/options");
+const logger_1 = require("@dotcom-tool-kit/logger");
+async function runTasks(logger, hooks, files) {
+    const config = await (0, config_1.loadConfig)(logger);
     const availableHooks = Object.keys(config.hooks)
         .sort()
         .map((id) => `- ${id}`)
@@ -23,20 +23,51 @@ hooks that are available are:
 ${availableHooks}`;
         throw error;
     }
+    for (const pluginOptions of Object.values(config.options)) {
+        if (pluginOptions.forPlugin) {
+            (0, options_1.setOptions)(pluginOptions.forPlugin.id, pluginOptions.options);
+        }
+    }
+    await (0, config_1.checkInstall)(config);
     for (const hook of hooks) {
+        const errors = [];
         if (!config.hookTasks[hook]) {
-            console.warn(messages_1.styles.warning(`no task configured for ${hook}: skipping assignment...}`));
+            logger.warn(`no task configured for ${hook}: skipping assignment...`);
             continue;
         }
         const assignment = config.hookTasks[hook];
         for (const id of assignment.tasks) {
             const Task = config.tasks[id];
-            const options = Task.plugin ? (_a = config.options[Task.plugin.id]) === null || _a === void 0 ? void 0 : _a.options : {};
+            const options = Task.plugin ? (0, options_1.getOptions)(Task.plugin.id) : {};
             // `Task` is an abstract class. here we know it's a concrete subclass
             // but typescript doesn't, so cast it to any.
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            const task = new Task(options);
-            await task.run();
+            const task = new Task(logger, options);
+            try {
+                await task.run(files);
+            }
+            catch (error) {
+                // allow subsequent hook tasks to run on error
+                errors.push({
+                    hook,
+                    task: id,
+                    error
+                });
+            }
+        }
+        if (errors.length > 0) {
+            const error = new error_1.ToolKitError(`error running tasks for ${logger_1.styles.hook(hook)}`);
+            error.details = errors
+                .map(({ hook, task, error }) => `${logger_1.styles.heading(`${logger_1.styles.task(task)}:`)}
+
+${error.message}${error instanceof error_1.ToolKitError
+                ? `
+
+${error.details}`
+                : ''}`)
+                .join(`${logger_1.styles.dim(logger_1.styles.ruler())}\n`);
+            error.exitCode = errors.length + 1;
+            throw error;
         }
     }
 }

package/lib/install.d.ts

@@ -1,2 +1,4 @@
-export default function installHooks(): Promise<void>;
+import type { Logger } from 'winston';
+import { Config } from './config';
+export default function installHooks(logger: Logger): Promise<Config>;
 //# 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":"AAGA,wBAA8B,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC,CAyB1D"}
+{"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,CAmC1E"}

package/lib/install.js

@@ -1,28 +1,39 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const config_1 = require("./config");
 const error_1 = require("@dotcom-tool-kit/error");
-async function installHooks() {
-    const config = await config_1.loadConfig({ checkInstall: false });
-    const tasks = Object.values(config.hooks).map((Hook) => async () => {
-        const hook = new Hook();
+const options_1 = require("@dotcom-tool-kit/options");
+const config_1 = require("./config");
+async function installHooks(logger) {
+    const config = await (0, config_1.loadConfig)(logger);
+    const tasks = Object.values(config.hooks).map((hook) => async () => {
         if (!(await hook.check())) {
             await hook.install();
         }
     });
+    for (const pluginOptions of Object.values(config.options)) {
+        if (pluginOptions.forPlugin) {
+            (0, options_1.setOptions)(pluginOptions.forPlugin.id, pluginOptions.options);
+        }
+    }
     const errors = [];
     for (const task of tasks) {
         try {
             await task();
         }
         catch (err) {
-            errors.push(err);
+            if (err instanceof Error) {
+                errors.push(err);
+            }
+            else {
+                throw err;
+            }
         }
     }
     if (errors.length) {
         const error = new error_1.ToolKitError('could not automatically install hooks:');
-        error.details = errors.map((error) => error.message).join('\n\n');
+        error.details = errors.map((error) => `- ${error.message}`).join('\n');
         throw error;
     }
+    return config;
 }
 exports.default = installHooks;

package/lib/messages.d.ts

@@ -1,31 +1,18 @@
-import colours from 'ansi-colors';
 import type { PluginOptions } from './config';
 import type { Conflict } from './conflict';
-import type { HookTask, HookClass } from './hook';
-import type { TaskClass } from '@dotcom-tool-kit/task';
-declare const s: {
-    hook: colours.StyleFunction;
-    task: colours.StyleFunction;
-    plugin: colours.StyleFunction;
-    URL: colours.StyleFunction;
-    app: colours.StyleFunction;
-    heading: colours.StyleFunction;
-    dim: colours.StyleFunction;
-    title: colours.StyleFunction;
-    error: (string: string) => string;
-    warning: (string: string) => string;
-    ruler: () => string;
-};
-export { s as styles };
+import type { HookTask } from './hook';
+import type { Hook, TaskClass } from '@dotcom-tool-kit/types';
 export declare const formatTaskConflicts: (conflicts: Conflict<TaskClass>[]) => string;
-export declare const formatHookConflicts: (conflicts: Conflict<HookClass>[]) => string;
+export declare const formatHookConflicts: (conflicts: Conflict<Hook>[]) => string;
 export declare const formatHookTaskConflicts: (conflicts: Conflict<HookTask>[]) => string;
 export declare const formatOptionConflicts: (conflicts: Conflict<PluginOptions>[]) => string;
 export declare const formatUndefinedHookTasks: (undefinedHooks: HookTask[], definedHooks: string[]) => string;
-export declare const formatUninstalledHooks: (uninstalledHooks: HookClass[]) => string;
+export declare const formatUnusedOptions: (unusedOptions: string[], definedPlugins: string[]) => string;
+export declare const formatUninstalledHooks: (uninstalledHooks: Hook[]) => string;
 declare type Missing = {
     hook: HookTask;
     tasks: string[];
 };
 export declare const formatMissingTasks: (missingTasks: Missing[], tasks: string[]) => string;
+export {};
 //# sourceMappingURL=messages.d.ts.map

package/lib/messages.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"messages.d.ts","sourceRoot":"","sources":["../src/messages.ts"],"names":[],"mappings":"AAAA,OAAO,OAAO,MAAM,aAAa,CAAA;AAEjC,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAC7C,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAA;AAC1C,OAAO,KAAK,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AAEjD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAA;AAItD,QAAA,MAAM,CAAC;;;;;;;;;oBASW,MAAM,KAAG,MAAM;sBACb,MAAM,KAAG,MAAM;iBACtB,MAAM;CAClB,CAAA;AAED,OAAO,EAAE,CAAC,IAAI,MAAM,EAAE,CAAA;AAStB,eAAO,MAAM,mBAAmB,cAAe,SAAS,SAAS,CAAC,EAAE,KAAG,MAKE,CAAA;AASzE,eAAO,MAAM,mBAAmB,cAAe,SAAS,SAAS,CAAC,EAAE,KAAG,MAKE,CAAA;AAazE,eAAO,MAAM,uBAAuB,cAAe,SAAS,QAAQ,CAAC,EAAE,KAAG,MAQzE,CAAA;AAOD,eAAO,MAAM,qBAAqB,cAAe,SAAS,aAAa,CAAC,EAAE,KAAG,MAU5E,CAAA;AAMD,eAAO,MAAM,wBAAwB,mBACnB,QAAQ,EAAE,gBACZ,MAAM,EAAE,KACrB,MAOF,CAAA;AAED,eAAO,MAAM,sBAAsB,qBACf,SAAS,EAAE,KAC5B,MAKF,CAAA;AAED,aAAK,OAAO,GAAG;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,EAAE,CAAA;CAAE,CAAA;AAOlD,eAAO,MAAM,kBAAkB,iBACf,OAAO,EAAE,SAChB,MAAM,EAAE,KACd,MAOF,CAAA"}
+{"version":3,"file":"messages.d.ts","sourceRoot":"","sources":["../src/messages.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAC7C,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAA;AAC1C,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,QAAQ,CAAA;AAEtC,OAAO,KAAK,EAAU,IAAI,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAA;AASrE,eAAO,MAAM,mBAAmB,cAAe,SAAS,SAAS,CAAC,EAAE,KAAG,MAKE,CAAA;AASzE,eAAO,MAAM,mBAAmB,cAAe,SAAS,IAAI,CAAC,EAAE,KAAG,MAKO,CAAA;AAazE,eAAO,MAAM,uBAAuB,cAAe,SAAS,QAAQ,CAAC,EAAE,KAAG,MAQzE,CAAA;AAOD,eAAO,MAAM,qBAAqB,cAAe,SAAS,aAAa,CAAC,EAAE,KAAG,MAU5E,CAAA;AAMD,eAAO,MAAM,wBAAwB,mBACnB,QAAQ,EAAE,gBACZ,MAAM,EAAE,KACrB,MAWF,CAAA;AAED,eAAO,MAAM,mBAAmB,kBACf,MAAM,EAAE,kBACP,MAAM,EAAE,KACvB,MAWF,CAAA;AAED,eAAO,MAAM,sBAAsB,qBACf,IAAI,EAAE,KACvB,MAKF,CAAA;AAED,aAAK,OAAO,GAAG;IAAE,IAAI,EAAE,QAAQ,CAAC;IAAC,KAAK,EAAE,MAAM,EAAE,CAAA;CAAE,CAAA;AAOlD,eAAO,MAAM,kBAAkB,iBACf,OAAO,EAAE,SAChB,MAAM,EAAE,KACd,MAOF,CAAA"}

package/lib/messages.js

@@ -1,86 +1,82 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.formatMissingTasks = exports.formatUninstalledHooks = exports.formatUndefinedHookTasks = exports.formatOptionConflicts = exports.formatHookTaskConflicts = exports.formatHookConflicts = exports.formatTaskConflicts = exports.styles = void 0;
-const tslib_1 = require("tslib");
-const ansi_colors_1 = tslib_1.__importDefault(require("ansi-colors"));
-// consistent styling use cases for terminal colours
-// don't use ansi-colors directly, define a style please
-const s = {
-    hook: ansi_colors_1.default.magenta,
-    task: ansi_colors_1.default.blueBright,
-    plugin: ansi_colors_1.default.cyan,
-    URL: ansi_colors_1.default.cyan.underline,
-    app: ansi_colors_1.default.green,
-    heading: ansi_colors_1.default.bold,
-    dim: ansi_colors_1.default.grey,
-    title: ansi_colors_1.default.bold.underline,
-    error: (string) => `${ansi_colors_1.default.red.bold('‼︎')} ${s.title(string)}`,
-    warning: (string) => `${ansi_colors_1.default.yellow.bold('⚠︎')} ${s.title(string)}`,
-    ruler: () => s.dim('─'.repeat(process.stdout.columns / 2))
-};
-exports.styles = s;
-const formatTaskConflict = (conflict) => `- ${s.task(conflict.conflicting[0].id || 'unknown task')} ${s.dim('from plugins')} ${conflict.conflicting
-    .map((task) => s.plugin(task.plugin ? task.plugin.id : 'unknown plugin'))
-    .join(s.dim(', '))}`;
-const formatTaskConflicts = (conflicts) => `${s.heading('There are multiple plugins that include the same tasks')}:
+exports.formatMissingTasks = exports.formatUninstalledHooks = exports.formatUnusedOptions = exports.formatUndefinedHookTasks = exports.formatOptionConflicts = exports.formatHookTaskConflicts = exports.formatHookConflicts = exports.formatTaskConflicts = void 0;
+const logger_1 = require("@dotcom-tool-kit/logger");
+const formatTaskConflict = (conflict) => `- ${logger_1.styles.task(conflict.conflicting[0].id || 'unknown task')} ${logger_1.styles.dim('from plugins')} ${conflict.conflicting
+    .map((task) => logger_1.styles.plugin(task.plugin ? task.plugin.id : 'unknown plugin'))
+    .join(logger_1.styles.dim(', '))}`;
+const formatTaskConflicts = (conflicts) => `${logger_1.styles.heading('There are multiple plugins that include the same tasks')}:
 ${conflicts.map(formatTaskConflict).join('\n')}
 
 You must resolve this conflict by removing all but one of these plugins.`;
 exports.formatTaskConflicts = formatTaskConflicts;
-const formatHookConflict = (conflict) => `- ${s.hook(conflict.conflicting[0].id || 'unknown event')} ${s.dim('from plugins')} ${conflict.conflicting
-    .map((task) => s.plugin(task.plugin ? task.plugin.id : 'unknown plugin'))
-    .join(s.dim(', '))}`;
-const formatHookConflicts = (conflicts) => `${s.heading('There are multiple plugins that include the same hooks')}:
+const formatHookConflict = (conflict) => `- ${logger_1.styles.hook(conflict.conflicting[0].id || 'unknown event')} ${logger_1.styles.dim('from plugins')} ${conflict.conflicting
+    .map((task) => logger_1.styles.plugin(task.plugin ? task.plugin.id : 'unknown plugin'))
+    .join(logger_1.styles.dim(', '))}`;
+const formatHookConflicts = (conflicts) => `${logger_1.styles.heading('There are multiple plugins that include the same hooks')}:
 ${conflicts.map(formatHookConflict).join('\n')}
 
 You must resolve this conflict by removing all but one of these plugins.`;
 exports.formatHookConflicts = formatHookConflicts;
-const formatHookTaskConflict = (conflict) => `${s.hook(conflict.conflicting[0].id)}:
+const formatHookTaskConflict = (conflict) => `${logger_1.styles.hook(conflict.conflicting[0].id)}:
 ${conflict.conflicting
-    .map((hook) => `- ${hook.tasks.map(s.task).join(s.dim(', '))} ${s.dim('by plugin')} ${s.plugin(hook.plugin.id)}`)
+    .map((hook) => `- ${hook.tasks.map(logger_1.styles.task).join(logger_1.styles.dim(', '))} ${logger_1.styles.dim('by plugin')} ${logger_1.styles.plugin(hook.plugin.id)}`)
     .join('\n')}
 `;
-const formatHookTaskConflicts = (conflicts) => `${s.heading('These hooks are configured to run different tasks by multiple plugins')}:
+const formatHookTaskConflicts = (conflicts) => `${logger_1.styles.heading('These hooks are configured to run different tasks by multiple plugins')}:
 ${conflicts.map(formatHookTaskConflict).join('\n')}
-You must resolve this conflict by explicitly configuring which task to run for these hooks. See ${s.URL('https://github.com/financial-times/dotcom-tool-kit/tree/main/docs/resolving-hook-conflicts.md')} for more details.
+You must resolve this conflict by explicitly configuring which task to run for these hooks. See ${logger_1.styles.URL('https://github.com/financial-times/dotcom-tool-kit/tree/main/docs/resolving-hook-conflicts.md')} for more details.
 
 `;
 exports.formatHookTaskConflicts = formatHookTaskConflicts;
-const formatOptionConflict = (conflict) => `${s.plugin(conflict.conflicting[0].forPlugin.id)}, configured by:
-${conflict.conflicting.map((option) => `- ${s.plugin(option.plugin.id)}`)}`;
-const formatOptionConflicts = (conflicts) => `${s.heading('These plugins have conflicting options')}:
+const formatOptionConflict = (conflict) => `${logger_1.styles.plugin(conflict.conflicting[0].forPlugin.id)}, configured by:
+${conflict.conflicting.map((option) => `- ${logger_1.styles.plugin(option.plugin.id)}`)}`;
+const formatOptionConflicts = (conflicts) => `${logger_1.styles.heading('These plugins have conflicting options')}:
 
 ${conflicts.map(formatOptionConflict).join('\n')}
 
-You must resolve this conflict by providing options in your app's Tool Kit configuration for these plugins, or installing a use-case plugin that provides these options. See ${s.URL('https://github.com/financial-times/dotcom-tool-kit/tree/main/readme.md#options')} for more details.
+You must resolve this conflict by providing options in your app's Tool Kit configuration for these plugins, or installing a use-case plugin that provides these options. See ${logger_1.styles.URL('https://github.com/financial-times/dotcom-tool-kit/tree/main/readme.md#options')} for more details.
 
 `;
 exports.formatOptionConflicts = formatOptionConflicts;
-const formatPlugin = (plugin) => plugin.id === 'app root' ? s.app('your app') : `plugin ${s.plugin(plugin.id)}`;
+const formatPlugin = (plugin) => plugin.id === 'app root' ? logger_1.styles.app('your app') : `plugin ${logger_1.styles.plugin(plugin.id)}`;
 // TODO text similarity "did you mean...?"
-const formatUndefinedHookTasks = (undefinedHooks, definedHooks) => `These hooks don't exist, but are configured to run tasks:
+const formatUndefinedHookTasks = (undefinedHooks, definedHooks) => `Hooks must be defined by a plugin before you can configure a task to run for them. In your Tool Kit configuration you've configured hooks that aren't defined:
 
-${undefinedHooks.map((hook) => `- ${s.hook(hook.id)} assigned by ${formatPlugin(hook.plugin)}`).join('\n')}
+${undefinedHooks.map((hook) => `- ${logger_1.styles.hook(hook.id)}`).join('\n')}
 
-They could be misspelt, or defined by a Tool Kit plugin that isn't used by this app.
+They could be misspelt, or defined by a Tool Kit plugin that isn't installed in this app.
 
-Available hooks are: ${definedHooks.map(s.hook).join(', ')}.
+${definedHooks.length > 0
+    ? `Hooks that are defined and available for tasks are: ${definedHooks.map(logger_1.styles.hook).join(', ')}`
+    : `There are no hooks defined by this app's plugins. You probably need to install some plugins to define hooks.`}.
 `;
 exports.formatUndefinedHookTasks = formatUndefinedHookTasks;
+const formatUnusedOptions = (unusedOptions, definedPlugins) => `Options are defined in your Tool Kit configuration for plugins that don't exist:
+
+${unusedOptions.map((optionName) => `- ${logger_1.styles.plugin(optionName)}`).join('\n')}
+
+They could be misspelt, or defined by a Tool Kit plugin that isn't installed in this app.
+
+${definedPlugins.length > 0
+    ? `Plugins that are defined and can have options set are: ${definedPlugins.map(logger_1.styles.plugin).join(', ')}`
+    : `There are no plugins installed currently. You'll need to install some plugins before options can be set.`}.
+`;
+exports.formatUnusedOptions = formatUnusedOptions;
 const formatUninstalledHooks = (uninstalledHooks) => `These hooks aren't installed into your app:
 
-${uninstalledHooks.map((hook) => `- ${s.hook(hook.id || 'unknown event')}`).join('\n')}
+${uninstalledHooks.map((hook) => `- ${logger_1.styles.hook(hook.id || 'unknown event')}`).join('\n')}
 
-Run ${s.task('dotcom-tool-kit --install')} to install these hooks.
+Run ${logger_1.styles.task('dotcom-tool-kit --install')} to install these hooks.
 `;
 exports.formatUninstalledHooks = formatUninstalledHooks;
-const formatMissingTask = (missing) => `- ${missing.tasks.map(s.task).join(', ')} ${s.dim(`(assigned to ${s.hook(missing.hook.id)} by ${formatPlugin(missing.hook.plugin)})`)}`;
+const formatMissingTask = (missing) => `- ${missing.tasks.map(logger_1.styles.task).join(', ')} ${logger_1.styles.dim(`(assigned to ${logger_1.styles.hook(missing.hook.id)} by ${formatPlugin(missing.hook.plugin)})`)}`;
 const formatMissingTasks = (missingTasks, tasks) => `These tasks don't exist, but are configured to run from hooks:
 
 ${missingTasks.map(formatMissingTask).join('\n')}
 
 They could be misspelt, or defined by a Tool Kit plugin that isn't used by this app.
 
-Available tasks are: ${tasks.map(s.task).join(', ')}.
+Available tasks are: ${tasks.map(logger_1.styles.task).join(', ')}.
 `;
 exports.formatMissingTasks = formatMissingTasks;

package/lib/plugin.d.ts

@@ -1,16 +1,7 @@
-import type { TaskClass } from '@dotcom-tool-kit/task';
-import type { HookClass } from './hook';
+import type { Logger } from 'winston';
 import { Config } from './config';
-export interface Plugin {
-    id: string;
-    root: string;
-    parent?: Plugin;
-    tasks?: TaskClass[];
-    hooks?: {
-        [id: string]: HookClass;
-    };
-}
-export declare function loadPluginConfig(plugin: Plugin, config: Config): Promise<Config>;
-export declare function loadPlugin(id: string, config: Config, parent?: Plugin): Promise<Plugin>;
-export declare function loadPlugins(plugins: string[], config: Config, parent?: Plugin): Promise<Plugin[]>;
+import { Plugin } from '@dotcom-tool-kit/types';
+export declare function loadPluginConfig(logger: Logger, plugin: Plugin, config: Config): Promise<Config>;
+export declare function loadPlugin(logger: Logger, id: string, config: Config, parent?: Plugin): Promise<Plugin>;
+export declare function loadPlugins(logger: Logger, plugins: string[], config: Config, parent?: Plugin): Promise<Plugin[]>;
 //# 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":"AAIA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAA;AACtD,OAAO,KAAK,EAAY,SAAS,EAAE,MAAM,QAAQ,CAAA;AAEjD,OAAO,EAAE,MAAM,EAAiB,MAAM,UAAU,CAAA;AAGhD,MAAM,WAAW,MAAM;IACrB,EAAE,EAAE,MAAM,CAAA;IACV,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,KAAK,CAAC,EAAE,SAAS,EAAE,CAAA;IACnB,KAAK,CAAC,EAAE;QACN,CAAC,EAAE,EAAE,MAAM,GAAG,SAAS,CAAA;KACxB,CAAA;CACF;AAED,wBAAsB,gBAAgB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CA6FtF;AAED,wBAAsB,UAAU,CAAC,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CA2E7F;AAED,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAEjG"}
+{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAGrC,OAAO,EAAE,MAAM,EAAiB,MAAM,UAAU,CAAA;AAKhD,OAAO,EAA2B,MAAM,EAAa,MAAM,wBAAwB,CAAA;AAEnF,wBAAsB,gBAAgB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CA6FtG;AAED,wBAAsB,UAAU,CAC9B,MAAM,EAAE,MAAM,EACd,EAAE,EAAE,MAAM,EACV,MAAM,EAAE,MAAM,EACd,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,MAAM,CAAC,CAkFjB;AAED,wBAAgB,WAAW,CACzB,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,MAAM,EAAE,EACjB,MAAM,EAAE,MAAM,EACd,MAAM,CAAC,EAAE,MAAM,GACd,OAAO,CAAC,MAAM,EAAE,CAAC,CAEnB"}

package/lib/plugin.js

@@ -2,18 +2,21 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.loadPlugins = exports.loadPlugin = exports.loadPluginConfig = void 0;
 const tslib_1 = require("tslib");
-const import_from_1 = tslib_1.__importDefault(require("import-from"));
-const resolve_from_1 = tslib_1.__importDefault(require("resolve-from"));
-const lodash_mergewith_1 = tslib_1.__importDefault(require("lodash.mergewith"));
+const import_from_1 = (0, tslib_1.__importDefault)(require("import-from"));
+const resolve_from_1 = (0, tslib_1.__importDefault)(require("resolve-from"));
+const lodash_mergewith_1 = (0, tslib_1.__importDefault)(require("lodash.mergewith"));
 const conflict_1 = require("./conflict");
 const rc_file_1 = require("./rc-file");
-async function loadPluginConfig(plugin, config) {
-    const { plugins = [], hooks = {}, options = {} } = await rc_file_1.loadToolKitRC(plugin.root);
+const error_1 = require("@dotcom-tool-kit/error");
+const logger_1 = require("@dotcom-tool-kit/logger");
+const types_1 = require("@dotcom-tool-kit/types");
+async function loadPluginConfig(logger, plugin, config) {
+    const { plugins = [], hooks = {}, options = {} } = await (0, rc_file_1.loadToolKitRC)(plugin.root);
     // load any plugins requested by this plugin
-    await loadPlugins(plugins, config, plugin);
+    await loadPlugins(logger, plugins, config, plugin);
     // load plugin hook tasks. do this after loading child plugins, so
     // parent hooks get assigned after child hooks and can override them
-    lodash_mergewith_1.default(config.hookTasks, hooks, 
+    (0, lodash_mergewith_1.default)(config.hookTasks, hooks, 
     // handle conflicts between hooks from different plugins
     (existingHookTask, configHookTask, id) => {
         const newHookTask = {
@@ -30,7 +33,7 @@ async function loadPluginConfig(plugin, config) {
         // return a conflict either listing this hook and the siblings,
         // or merging in a previously-generated hook
         if (existingFromSibling) {
-            const conflicting = conflict_1.isConflict(existingHookTask) ? existingHookTask.conflicting : [existingHookTask];
+            const conflicting = (0, conflict_1.isConflict)(existingHookTask) ? existingHookTask.conflicting : [existingHookTask];
             const conflict = {
                 plugin,
                 conflicting: conflicting.concat(newHookTask)
@@ -43,7 +46,7 @@ async function loadPluginConfig(plugin, config) {
     });
     // merge options from this plugin's config with any options we've collected already
     // TODO this is almost the exact same code as for hooks, refactor
-    lodash_mergewith_1.default(config.options, options, (existingOptions, configOptions, id) => {
+    (0, lodash_mergewith_1.default)(config.options, options, (existingOptions, configOptions, id) => {
         const pluginOptions = {
             options: configOptions,
             plugin,
@@ -58,7 +61,7 @@ async function loadPluginConfig(plugin, config) {
         // return a conflict either listing these options and the sibling's,
         // or merging in previously-generated options
         if (existingFromSibling) {
-            const conflicting = conflict_1.isConflict(existingOptions) ? existingOptions.conflicting : [existingOptions];
+            const conflicting = (0, conflict_1.isConflict)(existingOptions) ? existingOptions.conflicting : [existingOptions];
             const conflict = {
                 plugin,
                 conflicting: conflicting.concat(pluginOptions)
@@ -72,15 +75,25 @@ async function loadPluginConfig(plugin, config) {
     return config;
 }
 exports.loadPluginConfig = loadPluginConfig;
-async function loadPlugin(id, config, parent) {
+async function loadPlugin(logger, id, config, parent) {
     // don't load duplicate plugins
     if (id in config.plugins) {
         return config.plugins[id];
     }
     const root = parent ? parent.root : process.cwd();
     // load plugin relative to the parent plugin
-    const pluginRoot = resolve_from_1.default(root, id);
-    const basePlugin = import_from_1.default(root, id);
+    const pluginRoot = (0, resolve_from_1.default)(root, id);
+    const rawPlugin = (0, import_from_1.default)(root, id);
+    let basePlugin;
+    try {
+        basePlugin = (0, types_1.instantiatePlugin)(rawPlugin, logger);
+    }
+    catch (error) {
+        if (error instanceof error_1.ToolKitError) {
+            error.details = `the package ${logger_1.styles.plugin(id)} at ${logger_1.styles.filepath(pluginRoot)} is not a valid plugin`;
+        }
+        throw error;
+    }
     const plugin = {
         ...basePlugin,
         id,
@@ -89,13 +102,13 @@ async function loadPlugin(id, config, parent) {
     };
     config.plugins[id] = plugin;
     // add plugin tasks to our task registry, handling any conflicts
-    lodash_mergewith_1.default(config.tasks, Object.fromEntries((plugin.tasks || []).map((task) => [task.name, task])), (existingTask, newTask, taskId) => {
+    (0, lodash_mergewith_1.default)(config.tasks, Object.fromEntries((plugin.tasks || []).map((task) => [task.name, task])), (existingTask, newTask, taskId) => {
         newTask.plugin = plugin;
         newTask.id = taskId;
         if (!existingTask) {
             return newTask;
         }
-        const conflicting = conflict_1.isConflict(existingTask) ? existingTask.conflicting : [existingTask];
+        const conflicting = (0, conflict_1.isConflict)(existingTask) ? existingTask.conflicting : [existingTask];
         return {
             plugin,
             conflicting: conflicting.concat(newTask)
@@ -103,23 +116,23 @@ async function loadPlugin(id, config, parent) {
     });
     // add hooks to the registry, handling any conflicts
     // TODO refactor with command conflict handler
-    lodash_mergewith_1.default(config.hooks, plugin.hooks, (existingHook, newHook, hookId) => {
+    (0, lodash_mergewith_1.default)(config.hooks, plugin.hooks, (existingHook, newHook, hookId) => {
         newHook.id = hookId;
         newHook.plugin = plugin;
         if (!existingHook) {
             return newHook;
         }
-        const conflicting = conflict_1.isConflict(existingHook) ? existingHook.conflicting : [existingHook];
+        const conflicting = (0, conflict_1.isConflict)(existingHook) ? existingHook.conflicting : [existingHook];
         return {
             plugin,
             conflicting: conflicting.concat(newHook)
         };
     });
-    await loadPluginConfig(plugin, config);
+    await loadPluginConfig(logger, plugin, config);
     return plugin;
 }
 exports.loadPlugin = loadPlugin;
-function loadPlugins(plugins, config, parent) {
-    return Promise.all(plugins.map((plugin) => loadPlugin(plugin, config, parent)));
+function loadPlugins(logger, plugins, config, parent) {
+    return Promise.all(plugins.map((plugin) => loadPlugin(logger, plugin, config, parent)));
 }
 exports.loadPlugins = loadPlugins;

package/lib/rc-file.d.ts

@@ -1,3 +1,10 @@
+export declare const explorer: {
+    readonly search: (searchFrom?: string | undefined) => Promise<import("cosmiconfig/dist/types").CosmiconfigResult>;
+    readonly load: (filepath: string) => Promise<import("cosmiconfig/dist/types").CosmiconfigResult>;
+    readonly clearLoadCache: () => void;
+    readonly clearSearchCache: () => void;
+    readonly clearCaches: () => void;
+};
 export interface RCFile {
     plugins: string[];
     hooks: {

package/lib/rc-file.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"rc-file.d.ts","sourceRoot":"","sources":["../src/rc-file.ts"],"names":[],"mappings":"AAIA,MAAM,WAAW,MAAM;IACrB,OAAO,EAAE,MAAM,EAAE,CAAA;IACjB,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,EAAE,CAAA;KAAE,CAAA;IAC1C,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;KAAE,CAAA;CACnD;AAED,wBAAsB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAKjE"}
+{"version":3,"file":"rc-file.d.ts","sourceRoot":"","sources":["../src/rc-file.ts"],"names":[],"mappings":"AAEA,eAAO,MAAM,QAAQ;;;;;;CAA6D,CAAA;AAElF,MAAM,WAAW,MAAM;IACrB,OAAO,EAAE,MAAM,EAAE,CAAA;IACjB,KAAK,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,GAAG,MAAM,EAAE,CAAA;KAAE,CAAA;IAC1C,OAAO,EAAE;QAAE,CAAC,EAAE,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;KAAE,CAAA;CACnD;AAED,wBAAsB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAKjE"}

package/lib/rc-file.js

@@ -1,10 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadToolKitRC = void 0;
+exports.loadToolKitRC = exports.explorer = void 0;
 const cosmiconfig_1 = require("cosmiconfig");
-const explorer = cosmiconfig_1.cosmiconfig('toolkit', { ignoreEmptySearchPlaces: false });
+exports.explorer = (0, cosmiconfig_1.cosmiconfig)('toolkit', { ignoreEmptySearchPlaces: false });
 async function loadToolKitRC(root) {
-    const result = await explorer.search(root);
+    const result = await exports.explorer.search(root);
     if (!result || !result.config)
         return { plugins: [], hooks: {}, options: {} };
     return result.config;

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "dotcom-tool-kit",
-  "version": "1.0.0-beta.9",
+  "version": "1.0.2-beta.2",
   "description": "modern, maintainable, modular developer tooling for FT.com projects",
   "author": "FT.com Platforms Team <platforms-team.customer-products@ft.com>",
   "license": "MIT",
   "repository": {
     "type": "git",
     "url": "https://github.com/financial-times/dotcom-tool-kit.git",
-    "directory": "packages/cli"
+    "directory": "core/cli"
   },
   "homepage": "https://github.com/financial-times/dotcom-tool-kit",
   "bugs": "https://github.com/financial-times/dotcom-tool-kit/issues",
@@ -17,22 +17,24 @@
     "dotcom-tool-kit": "./bin/run"
   },
   "scripts": {
-    "test": "jest --silent"
+    "test": "cd ../../ ; npx jest --silent --projects core/cli"
   },
   "devDependencies": {
-    "@dotcom-tool-kit/task": "file:../task",
-    "@jest/globals": "^26.6.2",
+    "@jest/globals": "^27.4.6",
     "@types/lodash.mergewith": "^4.6.6",
+    "@types/node": "^12.20.24",
     "chai": "^4.3.4",
     "globby": "^10.0.2",
-    "jest": "^26.6.3",
-    "ts-jest": "^26.5.6",
-    "ts-node": "^8.10.2"
+    "ts-node": "^8.10.2",
+    "winston": "^3.5.1"
   },
   "dependencies": {
-    "@dotcom-tool-kit/error": "^1.0.0-beta.9",
-    "@dotcom-tool-kit/wait-for-ok": "^1.0.0-beta.9",
-    "ansi-colors": "^4.1.1",
+    "@dotcom-tool-kit/error": "^1.0.2-beta.2",
+    "@dotcom-tool-kit/hook": "^1.0.2-beta.2",
+    "@dotcom-tool-kit/logger": "^1.0.2-beta.2",
+    "@dotcom-tool-kit/options": "^1.0.2-beta.2",
+    "@dotcom-tool-kit/types": "^1.0.2-beta.2",
+    "@dotcom-tool-kit/wait-for-ok": "^1.0.2-beta.2",
     "cosmiconfig": "^7.0.0",
     "import-from": "^3.0.0",
     "lodash.mergewith": "^4.6.2",
@@ -47,5 +49,8 @@
     "/bin",
     "/lib",
     "/npm-shrinkwrap.json"
-  ]
+  ],
+  "volta": {
+    "extends": "../../package.json"
+  }
 }