Fix lint nits

Author: jdalton

src/commands/fix/cmd-fix.test.mts

@@ -369,7 +369,7 @@ describe('socket fix', async () => {
     'should handle vulnerable dependencies fixture project',
     async cmd => {
       const { tempDir, cleanup } = await withTempFixture(
-        path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+        path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
       )
       cleanupFunctions.push(cleanup)
 
@@ -390,7 +390,7 @@ describe('socket fix', async () => {
     'should handle monorepo fixture project',
     async cmd => {
       const { tempDir, cleanup } = await withTempFixture(
-        path.join(fixtureBaseDir, 'pnpm/monorepo')
+        path.join(fixtureBaseDir, 'pnpm/monorepo'),
       )
       cleanupFunctions.push(cleanup)
 
@@ -541,7 +541,7 @@ describe('socket fix', async () => {
       'should handle PURL-based vulnerability identification',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -568,7 +568,7 @@ describe('socket fix', async () => {
       'should handle multiple vulnerability IDs in comma-separated format',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -597,7 +597,7 @@ describe('socket fix', async () => {
       'should handle multiple vulnerability IDs as separate flags',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -628,7 +628,7 @@ describe('socket fix', async () => {
       'should handle autopilot mode with JSON output and custom limit',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -658,7 +658,7 @@ describe('socket fix', async () => {
       'should handle monorepo with pin style and markdown output',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/monorepo')
+          path.join(fixtureBaseDir, 'pnpm/monorepo'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -767,7 +767,7 @@ describe('socket fix', async () => {
       'should handle non-existent GHSA IDs gracefully',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -792,7 +792,7 @@ describe('socket fix', async () => {
       'should show clear error when both json and markdown flags are used',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -828,7 +828,7 @@ describe('socket fix', async () => {
       'should handle malformed CVE IDs gracefully',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -861,7 +861,7 @@ describe('socket fix', async () => {
       'should handle unusually long tokens gracefully',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -886,7 +886,7 @@ describe('socket fix', async () => {
       'should handle mixed valid and invalid vulnerability IDs',
       async cmd => {
         const { tempDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps')
+          path.join(fixtureBaseDir, 'pnpm/vulnerable-deps'),
         )
         cleanupFunctions.push(cleanup)
 

src/commands/optimize/cmd-optimize-pnpm-versions.test.mts

@@ -32,14 +32,13 @@ describe('socket optimize - pnpm versions', { timeout: 60_000 }, async () => {
   })
 
   describe('pnpm v8', () => {
-
     it(
       'should optimize packages with pnpm v8',
       { timeout: 30_000 },
       async () => {
         // Create temp fixture for pnpm8.
         const { tempDir: pnpm8FixtureDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm8')
+          path.join(fixtureBaseDir, 'pnpm8'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -118,7 +117,7 @@ describe('socket optimize - pnpm versions', { timeout: 60_000 }, async () => {
       async () => {
         // Create temp fixture for pnpm8.
         const { tempDir: pnpm8FixtureDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm8')
+          path.join(fixtureBaseDir, 'pnpm8'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -178,14 +177,13 @@ describe('socket optimize - pnpm versions', { timeout: 60_000 }, async () => {
   })
 
   describe('pnpm v9', () => {
-
     it(
       'should optimize packages with pnpm v9',
       { timeout: 30_000 },
       async () => {
         // Create temp fixture for pnpm9.
         const { tempDir: pnpm9FixtureDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm9')
+          path.join(fixtureBaseDir, 'pnpm9'),
         )
         cleanupFunctions.push(cleanup)
 
@@ -260,7 +258,7 @@ describe('socket optimize - pnpm versions', { timeout: 60_000 }, async () => {
       async () => {
         // Create temp fixture for pnpm9.
         const { tempDir: pnpm9FixtureDir, cleanup } = await withTempFixture(
-          path.join(fixtureBaseDir, 'pnpm9')
+          path.join(fixtureBaseDir, 'pnpm9'),
         )
         cleanupFunctions.push(cleanup)
 

src/utils/test-fixtures.mts

@@ -14,7 +14,7 @@ import trash from 'trash'
  */
 export async function createTempFixture(
   fixturePath: string,
-  cleanupHook?: (cleanup: () => Promise<void>) => void
+  cleanupHook?: (cleanup: () => Promise<void>) => void,
 ): Promise<string> {
   // Create a unique temporary directory.
   const tempBaseDir = tmpdir()
@@ -25,7 +25,7 @@ export async function createTempFixture(
   await fs.cp(fixturePath, tempDir, {
     recursive: true,
     // Preserve file permissions and timestamps.
-    preserveTimestamps: true
+    preserveTimestamps: true,
   })
 
   // Register cleanup if hook provided.
@@ -51,12 +51,13 @@ export async function createTempFixture(
  */
 export async function createTempFixtures(
   fixtures: Record<string, string>,
-  cleanupHook?: (cleanup: () => Promise<void>) => void
+  cleanupHook?: (cleanup: () => Promise<void>) => void,
 ): Promise<Record<string, string>> {
-  const tempFixtures: Record<string, string> = { __proto__: null } as Record<string, string>
+  const tempFixtures = { __proto__: null } as unknown as Record<string, string>
   const tempDirs: string[] = []
 
   for (const [name, fixturePath] of Object.entries(fixtures)) {
+    // eslint-disable-next-line no-await-in-loop
     const tempDir = await createTempFixture(fixturePath)
     tempFixtures[name] = tempDir
     tempDirs.push(tempDir)
@@ -65,9 +66,13 @@ export async function createTempFixtures(
   // Register cleanup for all temp directories.
   if (cleanupHook) {
     cleanupHook(async () => {
-      await Promise.all(tempDirs.map(dir => trash(dir).catch(() => {
-        // Ignore cleanup errors.
-      })))
+      await Promise.all(
+        tempDirs.map(dir =>
+          trash(dir).catch(() => {
+            // Ignore cleanup errors.
+          }),
+        ),
+      )
     })
   }
 
@@ -96,4 +101,4 @@ export async function withTempFixture(fixturePath: string): Promise<{
   }
 
   return { tempDir, cleanup }
-}
+}