fix: default options assignment for undefined values -uses merge-stra…

…tegies

package.json

@@ -95,6 +95,7 @@
     "find-up": "^5.0.0",
     "fs-extra": "^9.1.0",
     "glob": "^7.1.6",
+    "merge-strategies": "^0.2.0",
     "pipettes": "^0.1.3",
     "prefix-stream": "^1.0.1",
     "type-core": "^0.8.0"

src/bin/index.ts

@@ -4,6 +4,7 @@ import { log } from '../tasks/stdio/log';
 import { constants } from '../constants';
 import main from './main';
 import { NullaryFn } from 'type-core';
+import { shallow } from 'merge-strategies';
 import { attach, options as _options, resolver, add } from 'exits';
 
 export interface BinOptions {
@@ -22,14 +23,14 @@ export interface BinOptions {
  */
 export async function bin(options?: BinOptions): Promise<void> {
   try {
-    const opts = Object.assign(
+    const opts = shallow(
       {
         bin: constants.bin,
         file: constants.file,
         description: constants.description,
         version: constants.version
       },
-      options
+      options || undefined
     );
 
     const task = await main({ argv: process.argv.slice(2) }, opts);

src/tasks/filesystem/copy.ts

@@ -2,6 +2,7 @@ import { Task, Context } from '../../definitions';
 import { getPathPairs, usePair } from '../../helpers/paths';
 import { isCancelled } from '../../utils/is-cancelled';
 import { log } from '../stdio/log';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -29,9 +30,9 @@ export function copy(
   return async (ctx: Context): Promise<void> => {
     into(ctx, log('debug', 'Copy', paths, 'to', destination));
 
-    const opts = Object.assign(
+    const opts = shallow(
       { glob: false, single: false, strict: false, exists: 'error' },
-      options
+      options || undefined
     );
     const pairs = await getPathPairs(paths, destination, ctx, {
       glob: opts.glob,

src/tasks/filesystem/edit.ts

@@ -3,6 +3,7 @@ import { getPaths, useSource } from '../../helpers/paths';
 import { isCancelled } from '../../utils/is-cancelled';
 import { log } from '../stdio/log';
 import { Serial } from 'type-core';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -33,7 +34,7 @@ export function edit(
   return async (ctx: Context): Promise<void> => {
     into(ctx, log('debug', 'Edit:', paths));
 
-    const opts = Object.assign({ glob: false, strict: false }, options);
+    const opts = shallow({ glob: false, strict: false }, options || undefined);
     const sources = await getPaths(paths, ctx, {
       glob: opts.glob,
       strict: opts.strict

src/tasks/filesystem/mkdir.ts

@@ -2,6 +2,7 @@ import { Task, Context } from '../../definitions';
 import { getPaths } from '../../helpers/paths';
 import { isCancelled } from '../../utils/is-cancelled';
 import { log } from '../stdio/log';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -21,7 +22,7 @@ export function mkdir(
   return async (ctx: Context): Promise<void> => {
     into(ctx, log('debug', 'Create directories:', paths));
 
-    const opts = Object.assign({ ensure: false }, options);
+    const opts = shallow({ ensure: false }, options || undefined);
 
     const dirs = await getPaths(paths, ctx, { glob: false, strict: false });
 

src/tasks/filesystem/move.ts

@@ -2,6 +2,7 @@ import { Task, Context } from '../../definitions';
 import { getPathPairs, usePair } from '../../helpers/paths';
 import { isCancelled } from '../../utils/is-cancelled';
 import { log } from '../stdio/log';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -29,9 +30,9 @@ export function move(
   return async (ctx: Context): Promise<void> => {
     into(ctx, log('debug', 'Move', paths, 'to', destination));
 
-    const opts = Object.assign(
+    const opts = shallow(
       { glob: false, single: false, strict: false, exists: 'error' },
-      options
+      options || undefined
     );
     const pairs = await getPathPairs(paths, destination, ctx, {
       glob: opts.glob,

src/tasks/filesystem/remove.ts

@@ -2,6 +2,7 @@ import { Task, Context } from '../../definitions';
 import { getPaths, useSource } from '../../helpers/paths';
 import { isCancelled } from '../../utils/is-cancelled';
 import { log } from '../stdio/log';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -25,9 +26,9 @@ export function remove(
   return async (ctx: Context): Promise<void> => {
     into(ctx, log('debug', 'Remove:', paths));
 
-    const opts = Object.assign(
+    const opts = shallow(
       { glob: false, strict: false, exists: 'error' },
-      options
+      options || undefined
     );
     const sources = await getPaths(paths, ctx, {
       glob: opts.glob,

src/tasks/filesystem/watch.ts

@@ -5,6 +5,7 @@ import { clear } from '../stdio/clear';
 import { series } from '../aggregate/series';
 import { NullaryFn, UnaryFn, Empty } from 'type-core';
 import { into, combine } from 'pipettes';
+import { shallow } from 'merge-strategies';
 import chokidar from 'chokidar';
 import debounce from 'debounce';
 
@@ -51,7 +52,7 @@ export function watch(options: WatchOptions | Empty, task: Task): Task.Async {
         poll: -1,
         symlinks: false
       },
-      (defaults): Required<WatchOptions> => Object.assign(defaults, options),
+      (defaults) => shallow(defaults, options || undefined),
       ({ include, exclude, ...options }) => ({
         ...options,
         include: Array.isArray(include) ? include : [include],

src/tasks/filesystem/write.ts

@@ -2,6 +2,7 @@ import { Task, Context } from '../../definitions';
 import { useDestination } from '../../helpers/paths';
 import { log } from '../stdio/log';
 import { Serial } from 'type-core';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import fs from 'fs-extra';
 
@@ -31,7 +32,7 @@ export function write(
     await useDestination(
       path,
       ctx,
-      Object.assign({ exists: 'error' }, options),
+      shallow({ exists: 'error' }, options || undefined),
       (dest) => fs.writeFile(dest, data)
     );
   };

src/tasks/process/exec.ts

@@ -2,9 +2,10 @@ import { Task, Context } from '../../definitions';
 import { getPrefix } from '../../helpers/prefix';
 import { log } from '../stdio/log';
 import { Empty } from 'type-core';
+import { shallow } from 'merge-strategies';
+import transform from 'prefix-stream';
 import { WriteStream } from 'tty';
 import { into } from 'pipettes';
-import transform from 'prefix-stream';
 import execa from 'execa';
 
 export interface ExecOptions extends execa.Options {
@@ -24,7 +25,7 @@ export function exec(
   options?: ExecOptions | Empty,
   cb?: (ps: execa.ExecaChildProcess) => void
 ): Task.Async {
-  const opts = Object.assign({ extendEnv: true }, options || undefined);
+  const opts = shallow({ extendEnv: true }, options || undefined);
   return async (ctx: Context): Promise<void> => {
     const fullArgs = (args || []).concat(ctx.args || []);
 

src/tasks/reflection/lift.ts

@@ -7,6 +7,7 @@ import { constants } from '../../constants';
 import { select } from '../transform/select';
 import { write } from '../filesystem/write';
 import { print } from '../stdio/print';
+import { shallow } from 'merge-strategies';
 import { Members } from 'type-core';
 import { into } from 'pipettes';
 import chalk from 'chalk';
@@ -38,9 +39,9 @@ export interface LiftOptions {
  */
 export function lift(tasks: Task.Record, options?: LiftOptions): Task.Async {
   return async (ctx: Context): Promise<void> => {
-    const opts = Object.assign(
+    const opts = shallow(
       { purge: false, mode: 'default', bin: constants.bin },
-      options
+      options || undefined
     );
 
     const pkgPath = getAbsolutePath('package.json', ctx);

src/tasks/reflection/list.ts

@@ -3,6 +3,7 @@ import { parseToArray } from '../../helpers/parse';
 import { constants } from '../../constants';
 import { print } from '../stdio/print';
 import { Empty } from 'type-core';
+import { shallow } from 'merge-strategies';
 import { into } from 'pipettes';
 import table from 'as-table';
 import chalk from 'chalk';
@@ -25,7 +26,7 @@ export function list(
   map?: (name: string, route: string[]) => string[]
 ): Task.Sync {
   return (ctx: Context): void => {
-    const opts = Object.assign({ bin: constants.bin }, options);
+    const opts = shallow({ bin: constants.bin }, options || undefined);
     const items = parseToArray(tasks);
     const maxRouteLength = items.reduce(
       (acc, item) => (acc > item.route.length ? acc : item.route.length),