fix: handle directory targets according to specification

When a directory path is provided, it now recursively scans that
directory for all files by appending /**/* to the path pattern.
This ensures directory targets work as expected in scanning operations.

Also fixes a type annotation issue in getWorkspaceGlobs.

Cherry-picked from PR #794 (commit 5f78dfd)
Original author: Martin Torp <martin@socket.dev>

Co-Authored-By: Martin Torp <martin@socket.dev>

Author: jdalton

src/utils/fs/glob.mts

@@ -1,20 +1,25 @@
 import os from 'node:os'
 import path from 'node:path'
-import type { SocketYml } from '@socketsecurity/config'
+
+import fastGlob from 'fast-glob'
+import ignore from 'ignore'
+import micromatch from 'micromatch'
+import { parse as yamlParse } from 'yaml'
+
 import { NODE_MODULES } from '@socketsecurity/lib/constants/paths'
-import { safeReadFile } from '@socketsecurity/lib/fs'
+import { isDirSync, safeReadFile } from '@socketsecurity/lib/fs'
 import { defaultIgnore } from '@socketsecurity/lib/globs'
 import { readPackageJson } from '@socketsecurity/lib/packages'
 import { transform } from '@socketsecurity/lib/streams'
 import { isNonEmptyString } from '@socketsecurity/lib/strings'
-import type { SocketSdkSuccessResult } from '@socketsecurity/sdk'
-import type { Options as GlobOptions } from 'fast-glob'
-import fastGlob from 'fast-glob'
-import ignore from 'ignore'
-import micromatch from 'micromatch'
-import { parse as yamlParse } from 'yaml'
+
+
 import { PNPM } from '../../constants/agents.mjs'
+
 import type { Agent } from '../ecosystem/environment.mts'
+import type { SocketYml } from '@socketsecurity/config'
+import type { SocketSdkSuccessResult } from '@socketsecurity/sdk'
+import type { Options as GlobOptions } from 'fast-glob'
 
 const DEFAULT_IGNORE_FOR_GIT_IGNORE = defaultIgnore.filter(
   p => !p.endsWith('.gitignore'),
@@ -42,7 +47,7 @@ async function getWorkspaceGlobs(
   agent: Agent,
   cwd = process.cwd(),
 ): Promise<string[]> {
-  let workspacePatterns
+  let workspacePatterns: string[] | undefined
   if (agent === PNPM) {
     const workspacePath = path.join(cwd, 'pnpm-workspace.yaml')
     const yml = await safeReadFile(workspacePath)
@@ -305,6 +310,13 @@ export function pathsToGlobPatterns(
     // Expand tilde paths.
     const expanded = expandTildePath(p)
     // Convert current directory references to glob patterns.
-    return expanded === '.' || expanded === './' ? '**/*' : expanded
+    if (expanded === '.' || expanded === './') {
+      return '**/*'
+    }
+    // If the path is a directory, scan it recursively for all files.
+    if (isDirSync(expanded)) {
+      return `${expanded}/**/*`
+    }
+    return expanded
   })
 }

src/utils/fs/path-resolve.test.mts

@@ -1,19 +1,22 @@
 import path from 'node:path'
 import { fileURLToPath } from 'node:url'
-import { NODE_MODULES } from '@socketsecurity/lib/constants/paths'
-import { normalizePath } from '@socketsecurity/lib/path'
+
 import mockFs from 'mock-fs'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import {
-  PACKAGE_LOCK_JSON,
-  PNPM_LOCK_YAML,
-  YARN_LOCK,
-} from '../../constants/packages.mts'
+
+import { NODE_MODULES } from '@socketsecurity/lib/constants/paths'
+import { normalizePath } from '@socketsecurity/lib/path'
+
 import {
   findBinPathDetailsSync,
   findNpmDirPathSync,
   getPackageFilesForScan,
 } from './resolve.mts'
+import {
+  PACKAGE_LOCK_JSON,
+  PNPM_LOCK_YAML,
+  YARN_LOCK,
+} from '../../constants/packages.mts'
 
 const PACKAGE_JSON = 'package.json'
 
@@ -178,6 +181,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 .gitignore', async () => {
       mockTestFs({
         [`${mockFixturePath}/.gitignore`]: 'bar/*\n!bar/package.json',