fix bug where directory target were not handled according to the specification

Author: mtorp

src/utils/glob.mts

@@ -5,7 +5,7 @@ import ignore from 'ignore'
 import micromatch from 'micromatch'
 import { parse as yamlParse } from 'yaml'
 
-import { safeReadFile } from '@socketsecurity/registry/lib/fs'
+import { isDirSync, safeReadFile } from '@socketsecurity/registry/lib/fs'
 import { defaultIgnore } from '@socketsecurity/registry/lib/globs'
 import { readPackageJson } from '@socketsecurity/registry/lib/packages'
 import { transform } from '@socketsecurity/registry/lib/streams'
@@ -291,5 +291,14 @@ export function pathsToGlobPatterns(
   paths: string[] | readonly string[],
 ): string[] {
   // TODO: Does not support `~/` paths.
-  return paths.map(p => (p === '.' || p === './' ? '**/*' : p))
+  return paths.map(p => {
+    if (p === '.' || p === './') {
+      return '**/*'
+    }
+    // If the path is a directory, scan it recursively for all files.
+    if (isDirSync(p)) {
+      return `${p}/**/*`
+    }
+    return p
+  })
 }

src/utils/path-resolve.test.mts

@@ -110,6 +110,27 @@ describe('Path Resolve', () => {
       ])
     })
 
+    it('should handle a directory path input', async () => {
+      const subDirPath = normalizePath(path.join(mockFixturePath, 'subdir'))
+      mockTestFs({
+        [`${mockFixturePath}/package.json`]: '{}',
+        [`${subDirPath}/package.json`]: '{}',
+        [`${subDirPath}/nested/package.json`]: '{}',
+      })
+
+      const actual = await sortedGetPackageFilesFullScans(
+        [subDirPath],
+        globPatterns,
+        {
+          cwd: mockFixturePath,
+        },
+      )
+      expect(actual.map(normalizePath)).toEqual([
+        `${subDirPath}/nested/package.json`,
+        `${subDirPath}/package.json`,
+      ])
+    })
+
     it('should respect ignores from socket config', async () => {
       mockTestFs({
         [`${mockFixturePath}/bar/package-lock.json`]: '{}',