ts-repo-utils 7.8.1 → 7.9.0

package/src/functions/exec-async.test.mts

@@ -10,10 +10,12 @@ describe('exec-async', () => {
   const withSilentEcho = async <T,>(fn: () => Promise<T>): Promise<T> => {
     // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
     const originalEcho = (globalThis as any).echo;
+
     // eslint-disable-next-line functional/immutable-data, @typescript-eslint/no-unsafe-member-access
     (globalThis as any).echo = () => {
       // Silent implementation - no output
     };
+
     try {
       return await fn();
     } finally {
@@ -21,13 +23,16 @@ describe('exec-async', () => {
       (globalThis as any).echo = originalEcho;
     }
   };
+
   describe('basic command execution', () => {
     test('should execute simple command successfully', async () => {
       const result = await $('echo "hello world"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('hello world');
+
         expect(result.value.stderr).toBe('');
       }
     });
@@ -36,8 +41,10 @@ describe('exec-async', () => {
       const result = await $('echo "line1\nline2\nline3"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('line1\nline2\nline3');
+
         expect(result.value.stderr).toBe('');
       }
     });
@@ -46,8 +53,10 @@ describe('exec-async', () => {
       const result = await $('true', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout).toBe('');
+
         expect(result.value.stderr).toBe('');
       }
     });
@@ -58,8 +67,10 @@ describe('exec-async', () => {
       const result = await $('nonexistent_command_xyz', { silent: true });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
+
         expect(result.value.code).toBeDefined();
       }
     });
@@ -68,8 +79,10 @@ describe('exec-async', () => {
       const result = await $('exit 1', { silent: true });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
+
         expect(result.value.code).toBe(1);
       }
     });
@@ -80,6 +93,7 @@ describe('exec-async', () => {
       });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
       }
@@ -115,8 +129,9 @@ describe('exec-async', () => {
       const result = await $('echo "test"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
-        expect(typeof result.value.stdout).toBe('string');
+        expectTypeOf(result.value.stdout).toBeString();
       }
     });
 
@@ -127,8 +142,10 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(Buffer.isBuffer(result.value.stdout)).toBe(true);
+
         expect(Buffer.isBuffer(result.value.stderr)).toBe(true);
       }
     });
@@ -137,8 +154,10 @@ describe('exec-async', () => {
       const result = await $('echo "test"', { silent: true, encoding: null });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(Buffer.isBuffer(result.value.stdout)).toBe(true);
+
         expect(Buffer.isBuffer(result.value.stderr)).toBe(true);
       }
     });
@@ -150,8 +169,10 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
-        expect(typeof result.value.stdout).toBe('string');
+        expectTypeOf(result.value.stdout).toBeString();
+
         expect(result.value.stdout.trim()).toBe('test 日本語');
       }
     });
@@ -164,6 +185,7 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('hello world');
       }
@@ -173,6 +195,7 @@ describe('exec-async', () => {
       const result = await $('echo "first" && echo "second"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('first\nsecond');
       }
@@ -182,6 +205,7 @@ describe('exec-async', () => {
       const result = await $('echo "first"; echo "second"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('first\nsecond');
       }
@@ -208,6 +232,7 @@ describe('exec-async', () => {
       if (Result.isOk(result)) {
         // These assignments will fail at compile time if types don't match
         assertType<string>(result.value.stdout);
+
         assertType<string>(result.value.stderr);
       }
     });
@@ -234,6 +259,7 @@ describe('exec-async', () => {
 
       if (Result.isOk(result)) {
         assertType<Buffer>(result.value.stdout);
+
         assertType<Buffer>(result.value.stderr);
       }
     });
@@ -248,6 +274,7 @@ describe('exec-async', () => {
 
       if (Result.isOk(result)) {
         assertType<Buffer>(result.value.stdout);
+
         assertType<Buffer>(result.value.stderr);
       }
     });
@@ -257,10 +284,12 @@ describe('exec-async', () => {
         encoding: 'utf8',
         silent: true,
       });
+
       const _resultAscii = await $('echo "test"', {
         encoding: 'ascii',
         silent: true,
       });
+
       const _resultBase64 = await $('echo "test"', {
         encoding: 'base64',
         silent: true,
@@ -284,17 +313,21 @@ describe('exec-async', () => {
 
     test('should maintain type safety with const encoding values', async () => {
       const bufferEncoding = 'buffer';
+
       const nullEncoding = null;
+
       const utf8Encoding = 'utf8';
 
       const _resultBuffer = await $('echo "test"', {
         encoding: bufferEncoding,
         silent: true,
       });
+
       const _resultNull = await $('echo "test"', {
         encoding: nullEncoding,
         silent: true,
       });
+
       const _resultUtf8 = await $('echo "test"', {
         encoding: utf8Encoding,
         silent: true,
@@ -350,12 +383,15 @@ describe('exec-async', () => {
       // Type check for native exec with default options
       exec('echo "test"', (error, stdout, stderr) => {
         expectType<ExecException | null, typeof error>('=');
+
         expectType<string, typeof stdout>('=');
+
         expectType<string, typeof stderr>('=');
       });
 
       // The $ function should produce the same types wrapped in Result (suppressed for clean output)
       const _resultPromise = withSilentEcho(async () => $('echo "test"'));
+
       expectType<
         typeof _resultPromise,
         Promise<
@@ -368,7 +404,9 @@ describe('exec-async', () => {
       // Type check for native exec with buffer encoding
       exec('echo "test"', { encoding: 'buffer' }, (error, stdout, stderr) => {
         expectType<ExecException | null, typeof error>('=');
+
         expectType<Buffer, typeof stdout>('>=');
+
         expectType<Buffer, typeof stderr>('>=');
       });
 
@@ -376,6 +414,7 @@ describe('exec-async', () => {
       const _resultPromise = withSilentEcho(async () =>
         $('echo "test"', { encoding: 'buffer' }),
       );
+
       expectType<
         typeof _resultPromise,
         Promise<
@@ -388,7 +427,9 @@ describe('exec-async', () => {
       // Type check for native exec with null encoding
       exec('echo "test"', { encoding: null }, (error, stdout, stderr) => {
         expectType<ExecException | null, typeof error>('=');
+
         expectType<Buffer, typeof stdout>('>=');
+
         expectType<Buffer, typeof stderr>('>=');
       });
 
@@ -396,6 +437,7 @@ describe('exec-async', () => {
       const _resultPromise = withSilentEcho(async () =>
         $('echo "test"', { encoding: null }),
       );
+
       expectType<
         typeof _resultPromise,
         Promise<
@@ -408,7 +450,9 @@ describe('exec-async', () => {
       // Type check for native exec with utf8 encoding
       exec('echo "test"', { encoding: 'utf8' }, (error, stdout, stderr) => {
         expectType<ExecException | null, typeof error>('=');
+
         expectType<string, typeof stdout>('=');
+
         expectType<string, typeof stderr>('=');
       });
 
@@ -416,6 +460,7 @@ describe('exec-async', () => {
       const _resultPromise = withSilentEcho(async () =>
         $('echo "test"', { encoding: 'utf8' }),
       );
+
       expectType<
         typeof _resultPromise,
         Promise<
@@ -431,7 +476,9 @@ describe('exec-async', () => {
         { encoding: 'utf8', timeout: 5000 },
         (error, stdout, stderr) => {
           expectType<ExecException | null, typeof error>('=');
+
           expectType<string, typeof stdout>('=');
+
           expectType<string, typeof stderr>('=');
         },
       );
@@ -441,6 +488,7 @@ describe('exec-async', () => {
         encoding: 'utf8',
         silent: true,
       });
+
       expectType<
         typeof _resultPromise,
         Promise<
@@ -488,17 +536,21 @@ describe('exec-async', () => {
               // eslint-disable-next-line total-functions/no-unsafe-type-assertion
               stderr: undefined as unknown as T,
             };
+
             return emptyParams;
           };
 
           // Default encoding comparison
           const _execDefault = captureExecTypes<string>();
+
           const $Default = await $('echo "test"', { silent: true });
+
           if (Result.isOk($Default)) {
             expectType<
               typeof _execDefault.stdout,
               typeof $Default.value.stdout
             >('=');
+
             expectType<
               typeof _execDefault.stderr,
               typeof $Default.value.stderr
@@ -507,14 +559,17 @@ describe('exec-async', () => {
 
           // Buffer encoding comparison
           const _execBuffer = captureExecTypes<Buffer>('buffer');
+
           const $Buffer = await $('echo "test"', {
             encoding: 'buffer',
             silent: true,
           });
+
           if (Result.isOk($Buffer)) {
             expectType<typeof _execBuffer.stdout, typeof $Buffer.value.stdout>(
               '=',
             );
+
             expectType<typeof _execBuffer.stderr, typeof $Buffer.value.stderr>(
               '=',
             );
@@ -527,7 +582,9 @@ describe('exec-async', () => {
         });
       } finally {
         consoleLogSpy.mockRestore();
+
         consoleErrorSpy.mockRestore();
+
         stderrWriteSpy.mockRestore();
       }
     });

package/src/functions/format.mts

@@ -18,7 +18,7 @@ export const formatFiles = async (
   files: readonly string[],
   options?: Readonly<{
     silent?: boolean;
-    ignore?: (filePath: string) => boolean;
+    ignore?: false | ((filePath: string) => boolean);
     ignoreUnknown?: boolean;
   }>,
 ): Promise<Result<undefined, readonly unknown[]>> => {
@@ -28,6 +28,7 @@ export const formatFiles = async (
 
   if (files.length === 0) {
     conditionalEcho('No files to format');
+
     return Result.ok(undefined);
   }
 
@@ -45,6 +46,7 @@ export const formatFiles = async (
           } catch {
             // File doesn't exist, skip it
             conditionalEcho(`Skipping non-existent file: ${filePath}`);
+
             return Result.ok(undefined);
           }
 
@@ -60,10 +62,11 @@ export const formatFiles = async (
           });
 
           if (
-            fileInfo.ignored ||
-            (options?.ignore ?? defaultIgnoreFn)(filePath)
+            options?.ignore !== false &&
+            (fileInfo.ignored || (options?.ignore ?? defaultIgnoreFn)(filePath))
           ) {
             conditionalEcho(`Skipping ignored file: ${filePath}`);
+
             return Result.ok(undefined);
           }
 
@@ -73,6 +76,7 @@ export const formatFiles = async (
           ) {
             // Silently skip files with no parser
             conditionalEcho(`Skipping file (no parser): ${filePath}`);
+
             return Result.ok(undefined);
           }
 
@@ -87,6 +91,7 @@ export const formatFiles = async (
             conditionalEcho(`Unchanged: ${filePath}`);
           } else {
             await fs.writeFile(filePath, formatted, 'utf8');
+
             conditionalEcho(`Formatted: ${filePath}`);
           }
 
@@ -95,6 +100,7 @@ export const formatFiles = async (
           if (!silent) {
             console.error(`Error formatting ${filePath}:`, error);
           }
+
           return Result.err(error);
         }
       }),
@@ -102,6 +108,7 @@ export const formatFiles = async (
 
   if (results.every((r) => r.status === 'fulfilled')) {
     const fulfilled = results.map((r) => r.value);
+
     if (fulfilled.every(Result.isOk)) {
       return Result.ok(undefined);
     } else {
@@ -191,24 +198,28 @@ export const formatFilesGlob = async (
   options?: Readonly<{
     silent?: boolean;
     ignoreUnknown?: boolean;
-    ignore?: (filePath: string) => boolean;
+    ignore?: false | ((filePath: string) => boolean);
   }>,
 ): Promise<Result<undefined, unknown>> => {
   const silent = options?.silent ?? false;
+
   const ignoreUnknown = options?.ignoreUnknown ?? true;
+
   const ignore = options?.ignore;
+
   const conditionalEcho = silent ? () => {} : echo;
 
   try {
     // Find all files matching the glob
     const files = await glob(pathGlob, {
       absolute: true,
-      ignore: ['**/node_modules/**', '**/.git/**'],
+      ignore: ignore === false ? [] : ['**/node_modules/**', '**/.git/**'],
       dot: true,
     });
 
     if (files.length === 0) {
       conditionalEcho('No files found matching pattern:', pathGlob);
+
       return Result.ok(undefined);
     }
 
@@ -217,6 +228,7 @@ export const formatFilesGlob = async (
     if (!silent) {
       console.error('Error in formatFiles:', error);
     }
+
     return Result.err(error);
   }
 };
@@ -233,7 +245,7 @@ export const formatUncommittedFiles = async (
     staged?: boolean;
     silent?: boolean;
     ignoreUnknown?: boolean;
-    ignore?: (filePath: string) => boolean;
+    ignore?: false | ((filePath: string) => boolean);
   }>,
 ): Promise<
   Result<
@@ -262,6 +274,7 @@ export const formatUncommittedFiles = async (
           untrackedFilesResult.value,
         );
       }
+
       return untrackedFilesResult;
     }
 
@@ -275,6 +288,7 @@ export const formatUncommittedFiles = async (
       if (!silent) {
         console.error('Error getting changed files:', diffFilesResult.value);
       }
+
       return diffFilesResult;
     }
 
@@ -288,6 +302,7 @@ export const formatUncommittedFiles = async (
       if (!silent) {
         console.error('Error getting changed files:', stagedFilesResult.value);
       }
+
       return stagedFilesResult;
     }
 
@@ -322,7 +337,7 @@ export const formatDiffFrom = async (
     includeStaged?: boolean;
     silent?: boolean;
     ignoreUnknown?: boolean;
-    ignore?: (filePath: string) => boolean;
+    ignore?: false | ((filePath: string) => boolean);
   }>,
 ): Promise<
   Result<
@@ -355,10 +370,12 @@ export const formatDiffFrom = async (
     if (!silent) {
       console.error('Error getting changed files:', diffFromBaseResult.value);
     }
+
     return diffFromBaseResult;
   }
 
   const diffFiles = diffFromBaseResult.value;
+
   const mut_allFiles: string[] = diffFiles.slice();
 
   // If includeUntracked is true, also get untracked files
@@ -375,6 +392,7 @@ export const formatDiffFrom = async (
         if (!silent) {
           console.error(`Error getting ${type} files:`, filesResult.value);
         }
+
         return filesResult;
       }
 
@@ -408,6 +426,7 @@ export const formatDiffFrom = async (
 
   if (allFiles.length === 0) {
     conditionalEcho('No files to format');
+
     return Result.ok(undefined);
   }