fix(imports): node-builtin — 7 files converted to named imports

Author: jdalton

packages/cli/src/constants/agents.mts

@@ -3,7 +3,7 @@
  * Functions for package manager version requirements and execution paths.
  */
 
-import fs from 'node:fs'
+import { existsSync } from 'node:fs'
 import path from 'node:path'
 
 import { whichReal } from '@socketsecurity/lib/bin'
@@ -63,7 +63,7 @@ export async function getNpmExecPath(): Promise<string> {
   // Check npm in the same directory as node.
   const nodeDir = path.dirname(process.execPath)
   const npmInNodeDir = path.join(nodeDir, NPM)
-  if (fs.existsSync(npmInNodeDir)) {
+  if (existsSync(npmInNodeDir)) {
     return npmInNodeDir
   }
   // Fall back to whichReal.

packages/cli/src/utils/cli/completion.mts

@@ -1,4 +1,4 @@
-import fs from 'node:fs'
+import { existsSync } from 'node:fs'
 import path from 'node:path'
 
 import { getSocketAppDataPath, rootPath } from '../../constants/paths.mts'
@@ -70,7 +70,7 @@ export function getCompletionSourcingCommand(): CResult<string> {
     'socket-completion.bash',
   )
 
-  if (!fs.existsSync(completionScriptPath)) {
+  if (!existsSync(completionScriptPath)) {
     return {
       ok: false,
       message: 'Tab Completion script not found',

packages/cli/src/utils/ecosystem/environment.mts

@@ -25,7 +25,7 @@
  * - Configuring concurrent execution limits
  */
 
-import fs from 'node:fs'
+import { existsSync } from 'node:fs'
 import path from 'node:path'
 
 import browserslist from 'browserslist'
@@ -287,7 +287,7 @@ export async function detectPackageEnvironment({
       )
     : await findUp(PACKAGE_JSON, { cwd })
   const pkgPath =
-    pkgJsonPath && fs.existsSync(pkgJsonPath)
+    pkgJsonPath && existsSync(pkgJsonPath)
       ? path.dirname(pkgJsonPath)
       : undefined
   const pkgJson = pkgPath ? await readPackageJson(pkgPath) : undefined
@@ -447,20 +447,20 @@ export async function getAgentExecPath(agent: Agent): Promise<string> {
   if (binName === NPM) {
     // Try to use getNpmExecPath() first, but verify it exists.
     const npmPath = preferWindowsCmdShim(await getNpmExecPath(), NPM)
-    if (fs.existsSync(npmPath)) {
+    if (existsSync(npmPath)) {
       return npmPath
     }
     // If getNpmExecPath() doesn't exist, try common locations.
     // Check npm in the same directory as node.
     const nodeDir = path.dirname(process.execPath)
     if (WIN32) {
       const npmCmdInNodeDir = path.join(nodeDir, `${NPM}.cmd`)
-      if (fs.existsSync(npmCmdInNodeDir)) {
+      if (existsSync(npmCmdInNodeDir)) {
         return npmCmdInNodeDir
       }
     }
     const npmInNodeDir = path.join(nodeDir, NPM)
-    if (fs.existsSync(npmInNodeDir)) {
+    if (existsSync(npmInNodeDir)) {
       return preferWindowsCmdShim(npmInNodeDir, NPM)
     }
     // Fall back to which.
@@ -473,7 +473,7 @@ export async function getAgentExecPath(agent: Agent): Promise<string> {
   if (binName === PNPM) {
     // Try to use getPnpmExecPath() first, but verify it exists.
     const pnpmPath = await getPnpmExecPath()
-    if (fs.existsSync(pnpmPath)) {
+    if (existsSync(pnpmPath)) {
       return pnpmPath
     }
     // Fall back to which.

packages/cli/src/utils/ecosystem/windows-shims.mts

@@ -9,7 +9,7 @@
  * when one exists (so child_process can spawn it without `shell: true`).
  */
 
-import fs from 'node:fs'
+import { existsSync,  readFileSync } from 'node:fs'
 import path from 'node:path'
 
 import { WIN32 } from '@socketsecurity/lib/constants/platform'
@@ -46,7 +46,7 @@ export function preferWindowsCmdShim(binPath: string, binName: string): string {
   }
 
   const cmdShim = path.join(path.dirname(binPath), `${binName}.cmd`)
-  return fs.existsSync(cmdShim) ? cmdShim : binPath
+  return existsSync(cmdShim) ? cmdShim : binPath
 }
 
 /**
@@ -60,12 +60,12 @@ export function preferWindowsCmdShim(binPath: string, binName: string): string {
  * `npm-cli.js` entry point so we can spawn them via Node directly.
  */
 export function resolveBinPathSync(binPath: string): string {
-  if (!fs.existsSync(binPath)) {
+  if (!existsSync(binPath)) {
     return binPath
   }
 
   try {
-    const content = fs.readFileSync(binPath, 'utf8')
+    const content = readFileSync(binPath, 'utf8')
     // Look for common shim patterns:
     //   node "C:\path\to\npm-cli.js" "$@"
     //   "%_prog%"  "%dp0%\node_modules\npm\bin\npm-cli.js" %*

packages/cli/src/utils/npm/paths.mts

@@ -1,4 +1,4 @@
-import fs from 'node:fs'
+import { existsSync } from 'node:fs'
 import Module from 'node:module'
 import path from 'node:path'
 
@@ -83,7 +83,7 @@ export function getNpmRequire(): NodeJS.Require {
     const npmNmPath = path.join(npmDirPath, `${NODE_MODULES}/npm`)
     _npmRequire = Module.createRequire(
       path.join(
-        fs.existsSync(npmNmPath) ? npmNmPath : npmDirPath,
+        existsSync(npmNmPath) ? npmNmPath : npmDirPath,
         '<placeholder-basename>',
       ),
     )

packages/cli/src/utils/process/os.mts

@@ -26,7 +26,8 @@
  * - File permission management
  */
 
-import fs from 'node:fs'
+import { existsSync, readFileSync } from 'node:fs'
+import { chmod } from 'node:fs/promises'
 
 import { errorMessage } from '@socketsecurity/lib/errors'
 import { getDefaultLogger } from '@socketsecurity/lib/logger'
@@ -143,8 +144,8 @@ export function detectMusl(): boolean {
 
   // Method 1: Check /etc/os-release for Alpine.
   try {
-    if (fs.existsSync('/etc/os-release')) {
-      const osRelease = fs.readFileSync('/etc/os-release', 'utf8')
+    if (existsSync('/etc/os-release')) {
+      const osRelease = readFileSync('/etc/os-release', 'utf8')
       if (osRelease.includes('Alpine') || osRelease.includes('alpine')) {
         cachedLibc = 'musl'
         return true
@@ -157,8 +158,8 @@ export function detectMusl(): boolean {
   // Method 2: Check if ldd references musl.
   try {
     if (
-      fs.existsSync('/lib/ld-musl-x86_64.so.1') ||
-      fs.existsSync('/lib/ld-musl-aarch64.so.1')
+      existsSync('/lib/ld-musl-x86_64.so.1') ||
+      existsSync('/lib/ld-musl-aarch64.so.1')
     ) {
       cachedLibc = 'musl'
       return true
@@ -169,8 +170,8 @@ export function detectMusl(): boolean {
 
   // Method 3: Check /proc/version for musl indicators.
   try {
-    if (fs.existsSync('/proc/version')) {
-      const version = fs.readFileSync('/proc/version', 'utf8')
+    if (existsSync('/proc/version')) {
+      const version = readFileSync('/proc/version', 'utf8')
       if (version.includes('musl')) {
         cachedLibc = 'musl'
         return true
@@ -249,7 +250,7 @@ export async function ensureExecutable(filePath: string): Promise<void> {
   }
 
   try {
-    await fs.promises.chmod(filePath, 0o755)
+    await chmod(filePath, 0o755)
     logger.log('Set executable permissions')
   } catch (e) {
     logger.warn(`Failed to set executable permissions: ${errorMessage(e)}`)

packages/cli/src/utils/socket/json.mts

@@ -18,7 +18,8 @@
  * - Supports both read and write operations
  */
 
-import fs from 'node:fs'
+import { existsSync, readFileSync } from 'node:fs'
+import { readFile, writeFile } from 'node:fs/promises'
 import path from 'node:path'
 
 import { debugDirNs, debugNs } from '@socketsecurity/lib/debug'
@@ -127,14 +128,14 @@ export async function readSocketJson(
   defaultOnError = false,
 ): Promise<CResult<SocketJson>> {
   const sockJsonPath = path.join(cwd, SOCKET_JSON)
-  if (!fs.existsSync(sockJsonPath)) {
+  if (!existsSync(sockJsonPath)) {
     debugNs('notice', `miss: ${SOCKET_JSON} not found at ${cwd}`)
     return { ok: true, data: getDefaultSocketJson() }
   }
 
   let json = undefined
   try {
-    json = await fs.promises.readFile(sockJsonPath, 'utf8')
+    json = await readFile(sockJsonPath, 'utf8')
   } catch (e) {
     if (defaultOnError) {
       logger.warn(`Failed to read ${SOCKET_JSON}, using default`)
@@ -188,13 +189,13 @@ export function readSocketJsonSync(
   defaultOnError = false,
 ): CResult<SocketJson> {
   const sockJsonPath = path.join(cwd, SOCKET_JSON)
-  if (!fs.existsSync(sockJsonPath)) {
+  if (!existsSync(sockJsonPath)) {
     debugNs('notice', `miss: ${SOCKET_JSON} not found at ${cwd}`)
     return { ok: true, data: getDefaultSocketJson() }
   }
   let jsonContent = undefined
   try {
-    jsonContent = fs.readFileSync(sockJsonPath, 'utf8')
+    jsonContent = readFileSync(sockJsonPath, 'utf8')
   } catch (e) {
     if (defaultOnError) {
       logger.warn(`Failed to read ${SOCKET_JSON}, using default`)
@@ -262,7 +263,7 @@ export async function writeSocketJson(
   }
 
   const filepath = path.join(cwd, SOCKET_JSON)
-  await fs.promises.writeFile(filepath, `${jsonContent}\n`, 'utf8')
+  await writeFile(filepath, `${jsonContent}\n`, 'utf8')
 
   return { ok: true, data: undefined }
 }