refactor: remove unused imports and adopt safeMkdir utilities

- Remove unused fs import from github.mts
- Adopt safeMkdir in optimize test following lib pattern
- Add recursive option to safeMkdirSync in config test

Author: jdalton

packages/cli/src/commands/optimize/cmd-optimize.test.mts

@@ -5,6 +5,7 @@ import path from 'node:path'
 import { afterAll, afterEach, beforeAll, describe, expect } from 'vitest'
 
 import { NPM, PNPM } from '@socketsecurity/lib/constants/agents'
+import { safeMkdir } from '@socketsecurity/lib/fs'
 import { readPackageJson } from '@socketsecurity/lib/packages'
 import { spawn } from '@socketsecurity/lib/spawn'
 
@@ -67,7 +68,7 @@ async function _createTempFixture(sourceDir: string): Promise<string> {
   )
 
   // Copy the fixture files to the temp directory.
-  await promises.mkdir(tempDir, { recursive: true })
+  await safeMkdir(tempDir)
 
   // Copy package.json.
   const sourcePackageJson = path.join(sourceDir, PACKAGE_JSON)

packages/cli/src/utils/config.test.mts

@@ -60,7 +60,7 @@ describe('utils/config', () => {
 
       try {
         // Create nested directories.
-        safeMkdirSync(nestedDir)
+        safeMkdirSync(nestedDir, { recursive: true })
 
         // Create socket.yml in the tmpdir root.
         writeFileSync(

packages/cli/src/utils/git/github.mts

@@ -24,7 +24,7 @@
  * - Support for GitHub Actions environment variables
  */
 
-import { existsSync, promises as fs } from 'node:fs'
+import { existsSync } from 'node:fs'
 import path from 'node:path'
 
 import {