@plugjs/plug 0.0.16 → 0.0.17

package/src/plugs/eslint.ts

@@ -1,5 +1,5 @@
+import { installForking } from '../fork'
 import { requireResolve } from '../paths'
-import { installForking } from '../pipe'
 
 export interface ESLintOptions {
   /** ESLint's own _current working directory_, where config files are. */

package/src/plugs/exec.ts

@@ -3,12 +3,12 @@ import reaadline from 'node:readline'
 
 import { spawn, SpawnOptions } from 'node:child_process'
 import { assert } from '../assert'
+import { requireContext } from '../async'
 import { Files } from '../files'
 import { $p, logOptions } from '../log'
 import { AbsolutePath, getCurrentWorkingDirectory, resolveDirectory } from '../paths'
 import { Context, install, PipeParameters, Plug } from '../pipe'
 import { parseOptions } from '../utils/options'
-import { requireContext } from '../async'
 
 /** Options for executing scripts */
 export interface ExecOptions {

package/src/plugs/mocha/runner.ts

@@ -1,9 +1,9 @@
 import RealMocha from 'mocha' // Mocha types pollute the global scope!
 
-import { failure } from '../../assert'
+import { BuildFailure } from '../../failure'
 import { Files } from '../../files'
 import { $wht, NOTICE } from '../../log'
-import { PipeParameters, Plug, Context } from '../../pipe'
+import { Context, PipeParameters, Plug } from '../../pipe'
 import { MochaOptions } from '../mocha'
 import { logSymbol, PlugReporter } from './reporter'
 
@@ -40,7 +40,7 @@ export default class Mocha implements Plug<void> {
     return new Promise((resolve, reject) => {
       try {
         mocha.run((failures) => {
-          if (failures) reject(failure())
+          if (failures) reject(new BuildFailure({ logged: true }))
           resolve(undefined)
         })
       } catch (error) {

package/src/plugs/mocha.ts

@@ -1,5 +1,5 @@
+import { installForking } from '../fork'
 import { requireResolve } from '../paths'
-import { installForking } from '../pipe'
 
 /** Options to construct our {@link Mocha} plug. */
 export interface MochaOptions {

package/src/plugs/rmf.ts

@@ -1,6 +1,6 @@
 import { Files } from '../files'
 import { $gry, $p } from '../log'
-import { install, PipeParameters, Plug, Context } from '../pipe'
+import { Context, install, PipeParameters, Plug } from '../pipe'
 import { rm } from '../utils/asyncfs'
 
 declare module '../pipe' {

package/src/plugs/tsc/runner.ts

@@ -1,6 +1,7 @@
-import ts from 'typescript'; // TypeScript does NOT support ESM modules
+import ts from 'typescript' // TypeScript does NOT support ESM modules
 
-import { failure } from '../../assert'
+import { assertPromises } from '../../assert'
+import { BuildFailure } from '../../failure'
 import { Files } from '../../files'
 import { $p, log } from '../../log'
 import { AbsolutePath, resolveFile } from '../../paths'
@@ -95,16 +96,12 @@ export default class Tsc implements Plug<Files> {
       promises.push(builder.write(fileName, code).then((file) => {
         log.trace('Written', $p(file))
       }).catch((error) => {
-        context.log.error('Error writing to', fileName, error)
-        throw failure() // no more logs!
+        throw new BuildFailure(`Error writing to ${fileName}`, [ error ])
       }))
     })
 
     /* Await for all files to be written and check */
-    const settlements = await Promise.allSettled(promises)
-    const failures = settlements
-        .reduce((failures, s) => failures + s.status === 'rejected' ? 1 : 0, 0)
-    if (failures) throw failure() // already logged above
+    await assertPromises(promises, 'Error writing files')
 
     /* Update report and fail on errors */
     updateReport(report, result.diagnostics, rootDir)

package/src/plugs/tsc.ts

@@ -1,7 +1,7 @@
 import type { CompilerOptions } from 'typescript'
 
+import { installForking } from '../fork'
 import { requireResolve } from '../paths'
-import { installForking } from '../pipe'
 
 declare module '../pipe' {
   export interface Pipe {

package/src/types.ts

@@ -44,7 +44,7 @@ export interface Task<T extends Result = Result> {
   /** The absolute file name where this {@link Task} was defined */
   readonly buildFile: AbsolutePath,
   /** Invoke a task from (possibly) a different {@link Context} */
-  call(state: State, taskName: string): Promise<T>
+  invoke(state: State, taskName: string): Promise<T>
 }
 
 /**
@@ -97,11 +97,11 @@ export type ThisBuild<D extends BuildDef> = {
     k extends string ?
       D[k] extends TaskDef<infer R> ?
         R extends Promise<undefined> | void | undefined ? () => Promise<undefined> :
-        R extends Pipe | Files ? () => Pipe & Promise<Files> :
+        R extends Pipe | Files ? () => Pipe :
         never :
       D[k] extends Task<infer R> ?
         R extends undefined ? () => Promise<undefined> :
-        R extends Files ? () => Pipe & Promise<Files> :
+        R extends Files ? () => Pipe :
         never :
       D[k] extends string ?
         string :

package/src/utils/asyncfs.ts

@@ -1,4 +1,5 @@
 import { constants } from 'node:fs'
+
 import fsp from 'node:fs/promises'
 
 type FsPromises = typeof fsp