feat: exclude only directories with enabled the onlyFiles option

README.md

@@ -472,7 +472,7 @@ fg.globSync('*', { onlyDirectories: true });  // ['src']
 * Type: `boolean`
 * Default: `true`
 
-Return only files.
+Return everything (file, socket, …) except directories.
 
 ```js
 fg.globSync('*', { onlyFiles: false }); // ['index.js', 'src']

src/providers/filters/entry.spec.ts

@@ -15,6 +15,7 @@ interface FilterOptions {
 }
 
 const FILE_ENTRY = tests.entry.builder().path('root/file.txt').file().build();
+const SOCKET_ENTRY = tests.entry.builder().path('/tmp/test.sock').socket().build();
 const DIRECTORY_ENTRY = tests.entry.builder().path('root/directory').directory().build();
 
 function getEntryFilterInstance(options?: Options): EntryFilter {
@@ -135,6 +136,13 @@ describe('Providers → Filters → Entry', () => {
 					options: { onlyFiles: true },
 				});
 			});
+
+			it('should accept a socket entry', () => {
+				accept(SOCKET_ENTRY, {
+					positive: ['**/*'],
+					options: { onlyFiles: true },
+				});
+			});
 		});
 
 		describe('options.onlyDirectories', () => {
@@ -145,6 +153,13 @@ describe('Providers → Filters → Entry', () => {
 				});
 			});
 
+			it('should reject a socket entry', () => {
+				reject(SOCKET_ENTRY, {
+					positive: ['**/*'],
+					options: { onlyDirectories: true },
+				});
+			});
+
 			it('should accept a directory entry', () => {
 				accept(DIRECTORY_ENTRY, {
 					positive: ['**/*'],

src/providers/filters/entry.ts

@@ -47,11 +47,13 @@ export default class EntryFilter {
 			return false;
 		}
 
-		if (this.#onlyFileFilter(entry) || this.#onlyDirectoryFilter(entry)) {
+		const isDirectory = entry.dirent.isDirectory();
+
+		if (this.#onlyFileFilter(isDirectory) || this.#onlyDirectoryFilter(isDirectory)) {
 			return false;
 		}
 
-		const isMatched = this.#isMatchToPatternsSet(filepath, pattens, entry.dirent.isDirectory());
+		const isMatched = this.#isMatchToPatternsSet(filepath, pattens, isDirectory);
 
 		if (this.#settings.unique && isMatched) {
 			this.#createIndexRecord(filepath);
@@ -68,12 +70,12 @@ export default class EntryFilter {
 		this.index.set(filepath, undefined);
 	}
 
-	#onlyFileFilter(entry: Entry): boolean {
-		return this.#settings.onlyFiles && !entry.dirent.isFile();
+	#onlyFileFilter(isDirectory: boolean): boolean {
+		return this.#settings.onlyFiles && isDirectory;
 	}
 
-	#onlyDirectoryFilter(entry: Entry): boolean {
-		return this.#settings.onlyDirectories && !entry.dirent.isDirectory();
+	#onlyDirectoryFilter(isDirectory: boolean): boolean {
+		return this.#settings.onlyDirectories && !isDirectory;
 	}
 
 	#isMatchToPatternsSet(filepath: string, patterns: PatternsRegexSet, isDirectory: boolean): boolean {

src/tests/utils/entry.ts

@@ -38,6 +38,12 @@ class EntryBuilder {
 		return this;
 	}
 
+	public socket(): this {
+		this.#entryType = DirentType.Socket;
+
+		return this;
+	}
+
 	public stats(): this {
 		this.#entry.stats = new Stats();