remove-glob 0.3.6 → 0.3.7

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAWrD;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,GAAE,aAAkB,EAAE,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,KAAK,IAAI,qBA0ElF"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAWrD;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,IAAI,GAAE,aAAkB,EAAE,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,KAAK,IAAI,qBAyElF"}

package/dist/index.js

@@ -1,5 +1,5 @@
-import { existsSync, readdirSync, rmSync, statSync, unlinkSync } from 'node:fs';
-import { join, resolve } from 'node:path';
+import { existsSync, rmSync, statSync, unlinkSync } from 'node:fs';
+import { resolve } from 'node:path';
 import { globSync } from 'tinyglobby';
 /** Helper to throw or callback with error */
 function throwOrCallback(err, cb) {
@@ -23,7 +23,7 @@ export function removeSync(opts = {}, callback) {
         return;
     }
     if (paths.length && opts.glob) {
-        throwOrCallback(new Error('Providing both `--paths` and `--glob` pattern are not supported, you must provide only one of these options.'), cb);
+        throwOrCallback(new Error('Providing both `--paths` and `--glob` pattern at the same time is not supported, you must chose only one.'), cb);
         return;
     }
     let pathExists = false;
@@ -40,7 +40,9 @@ export function removeSync(opts = {}, callback) {
             ignore: ['**/.git/**', '**/node_modules/**'],
         });
     }
-    opts.stat && console.time('Duration');
+    if (opts.stat || opts.verbose) {
+        console.time('Duration');
+    }
     opts.dryRun && console.log('=== dry-run ===');
     paths.forEach(path => {
         // do we need to resolve file/dir from a different cwd?
@@ -55,9 +57,6 @@ export function removeSync(opts = {}, callback) {
                 }
                 else {
                     opts.verbose && console.log(`removing directory: ${path}`);
-                    readdirSync(path).forEach(name => {
-                        removeSync({ paths: join(path, name) }); // recursively remove content
-                    });
                     rmSync(path, { recursive: true, force: true, maxRetries: process.platform === 'win32' ? 10 : 0 });
                 }
             }
@@ -79,7 +78,6 @@ export function removeSync(opts = {}, callback) {
         console.timeEnd('Duration');
     }
     opts.dryRun && console.log('=== dry-run ===');
-    if (typeof cb === 'function')
-        cb();
+    typeof cb === 'function' && cb();
     return pathExists;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "remove-glob",
-  "version": "0.3.6",
+  "version": "0.3.7",
   "description": "A tiny utility to remove items or directories recursively, also supports glob",
   "bin": {
     "remove": "dist/cli.js"
@@ -17,7 +17,8 @@
   },
   "module": "./dist/index.js",
   "publishConfig": {
-    "access": "public"
+    "access": "public",
+    "registry": "https://registry.npmjs.org/"
   },
   "files": [
     "/dist",

package/src/index.ts

@@ -1,5 +1,5 @@
-import { existsSync, readdirSync, rmSync, statSync, unlinkSync } from 'node:fs';
-import { join, resolve } from 'node:path';
+import { existsSync, rmSync, statSync, unlinkSync } from 'node:fs';
+import { resolve } from 'node:path';
 import { globSync } from 'tinyglobby';
 
 import type { RemoveOptions } from './interfaces.js';
@@ -32,7 +32,7 @@ export function removeSync(opts: RemoveOptions = {}, callback?: (e?: Error) => v
   }
   if (paths.length && opts.glob) {
     throwOrCallback(
-      new Error('Providing both `--paths` and `--glob` pattern are not supported, you must provide only one of these options.'),
+      new Error('Providing both `--paths` and `--glob` pattern at the same time is not supported, you must chose only one.'),
       cb,
     );
     return;
@@ -51,7 +51,9 @@ export function removeSync(opts: RemoveOptions = {}, callback?: (e?: Error) => v
       ignore: ['**/.git/**', '**/node_modules/**'],
     });
   }
-  opts.stat && console.time('Duration');
+  if (opts.stat || opts.verbose) {
+    console.time('Duration');
+  }
   opts.dryRun && console.log('=== dry-run ===');
 
   paths.forEach(path => {
@@ -67,9 +69,6 @@ export function removeSync(opts: RemoveOptions = {}, callback?: (e?: Error) => v
           console.log(`would remove directory: ${path}`);
         } else {
           opts.verbose && console.log(`removing directory: ${path}`);
-          readdirSync(path).forEach(name => {
-            removeSync({ paths: join(path, name) }); // recursively remove content
-          });
           rmSync(path, { recursive: true, force: true, maxRetries: process.platform === 'win32' ? 10 : 0 });
         }
       } else {
@@ -90,6 +89,6 @@ export function removeSync(opts: RemoveOptions = {}, callback?: (e?: Error) => v
     console.timeEnd('Duration');
   }
   opts.dryRun && console.log('=== dry-run ===');
-  if (typeof cb === 'function') cb();
+  typeof cb === 'function' && cb();
   return pathExists;
 }