dotcom-tool-kit 1.0.0 → 1.0.2-beta.3

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,CA2FvG;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,6 +1,6 @@
 "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 = (0, tslib_1.__importDefault)(require("path"));
 const plugin_1 = require("./plugin");
@@ -20,7 +20,7 @@ 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 } = {}) {
+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));
@@ -53,7 +53,6 @@ async function validateConfig(config, { checkInstall = true } = {}) {
         }
     }
     const configuredHookTasks = (0, conflict_1.withoutConflicts)(Object.values(config.hookTasks));
-    const definedHooks = (0, conflict_1.withoutConflicts)(Object.values(config.hooks));
     const definedHookIds = new Set(Object.keys(config.hooks));
     const undefinedHookTasks = configuredHookTasks.filter((hookTask) => {
         // we only care about undefined hooks that were configured by the app, not default config from plugins
@@ -65,6 +64,13 @@ async function validateConfig(config, { checkInstall = true } = {}) {
         shouldThrow = true;
         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) => ({
         hook,
@@ -75,28 +81,34 @@ async function validateConfig(config, { checkInstall = true } = {}) {
         shouldThrow = true;
         error.details += (0, 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 += (0, messages_1.formatUninstalledHooks)(uninstalledHooks);
-        }
-    }
     if (shouldThrow) {
         throw error;
     }
     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 (0, 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 (0, config_1.loadConfig)();
+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()}
-${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.` : 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":"AAUA,wBAAsB,QAAQ,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsE7D;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 (0, 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,22 +23,28 @@ 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);
+            const task = new Task(logger, options);
             try {
-                await task.run();
+                await task.run(files);
             }
             catch (error) {
                 // allow subsequent hook tasks to run on error
@@ -50,16 +56,16 @@ ${availableHooks}`;
             }
         }
         if (errors.length > 0) {
-            const error = new error_1.ToolKitError(`error running tasks for ${messages_1.styles.hook(hook)}`);
+            const error = new error_1.ToolKitError(`error running tasks for ${logger_1.styles.hook(hook)}`);
             error.details = errors
-                .map(({ hook, task, error }) => `${messages_1.styles.heading(`${messages_1.styles.task(task)}:`)}
+                .map(({ hook, task, error }) => `${logger_1.styles.heading(`${logger_1.styles.task(task)}:`)}
 
 ${error.message}${error instanceof error_1.ToolKitError
                 ? `
 
 ${error.details}`
                 : ''}`)
-                .join(`\n${messages_1.styles.dim(messages_1.styles.ruler())}\n`);
+                .join(`${logger_1.styles.dim(logger_1.styles.ruler())}\n`);
             error.exitCode = errors.length + 1;
             throw error;
         }

package/lib/install.d.ts

@@ -1,3 +1,4 @@
+import type { Logger } from 'winston';
 import { Config } from './config';
-export default function installHooks(): Promise<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":"AACA,OAAO,EAAE,MAAM,EAAc,MAAM,UAAU,CAAA;AAE7C,wBAA8B,YAAY,IAAI,OAAO,CAAC,MAAM,CAAC,CA+B5D"}
+{"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,15 +1,20 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const error_1 = require("@dotcom-tool-kit/error");
+const options_1 = require("@dotcom-tool-kit/options");
 const config_1 = require("./config");
-async function installHooks() {
-    const config = await (0, config_1.loadConfig)({ checkInstall: false });
-    const tasks = Object.values(config.hooks).map((Hook) => async () => {
-        const hook = new Hook();
+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 {
@@ -26,7 +31,7 @@ async function installHooks() {
     }
     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;

package/lib/messages.d.ts

@@ -1,34 +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;
-    filepath: colours.StyleFunction;
-    app: colours.StyleFunction;
-    option: colours.StyleFunction;
-    makeTarget: 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;;;;;;;;;;;;oBAYW,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,MAWF,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,91 +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 = (0, 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,
-    filepath: ansi_colors_1.default.italic,
-    app: ansi_colors_1.default.green,
-    option: ansi_colors_1.default.blue,
-    makeTarget: ansi_colors_1.default.grey.italic,
-    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) => `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)}`).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 installed in this app.
 
 ${definedHooks.length > 0
-    ? `Hooks that are defined and available for tasks are: ${definedHooks.map(s.hook).join(', ')}`
+    ? `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

@@ -7,10 +7,13 @@ 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 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
     (0, lodash_mergewith_1.default)(config.hookTasks, hooks, 
@@ -72,7 +75,7 @@ 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];
@@ -80,7 +83,17 @@ async function loadPlugin(id, config, parent) {
     const root = parent ? parent.root : process.cwd();
     // load plugin relative to the parent plugin
     const pluginRoot = (0, resolve_from_1.default)(root, id);
-    const basePlugin = (0, import_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,
@@ -115,11 +128,11 @@ async function loadPlugin(id, config, parent) {
             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/package.json

@@ -1,13 +1,13 @@
 {
   "name": "dotcom-tool-kit",
-  "version": "1.0.0",
+  "version": "1.0.2-beta.3",
   "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,21 +17,24 @@
     "dotcom-tool-kit": "./bin/run"
   },
   "scripts": {
-    "test": "cd ../../ ; npx jest --silent --projects packages/cli"
+    "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",
-    "ts-node": "^8.10.2"
+    "ts-node": "^8.10.2",
+    "winston": "^3.5.1"
   },
   "dependencies": {
-    "@dotcom-tool-kit/error": "^1.0.0",
-    "@dotcom-tool-kit/wait-for-ok": "^1.0.0",
-    "ansi-colors": "^4.1.1",
+    "@dotcom-tool-kit/error": "^1.0.2-beta.3",
+    "@dotcom-tool-kit/hook": "^1.0.2-beta.3",
+    "@dotcom-tool-kit/logger": "^1.0.2-beta.3",
+    "@dotcom-tool-kit/options": "^1.0.2-beta.3",
+    "@dotcom-tool-kit/types": "^1.0.2-beta.3",
+    "@dotcom-tool-kit/wait-for-ok": "^1.0.2-beta.3",
     "cosmiconfig": "^7.0.0",
     "import-from": "^3.0.0",
     "lodash.mergewith": "^4.6.2",