fix(lint): disable noImportAssign for tests and fix duplicate beforeEach

- Add noImportAssign override for test files (needed for immutability tests)
- Fix duplicate beforeEach in test/stdio/footer.test.ts (convert second one to afterEach)
- Apply additional auto-fixes from biome

All lint checks now pass successfully.

Author: jdalton

biome.json

@@ -143,11 +143,22 @@
           },
           "suspicious": {
             "noExplicitAny": "off",
+            "noImportAssign": "off",
             "noShadowRestrictedNames": "off"
           }
         }
       }
     },
+    {
+      "includes": ["test/**/*.ts"],
+      "linter": {
+        "rules": {
+          "suspicious": {
+            "noImportAssign": "off"
+          }
+        }
+      }
+    },
     {
       "includes": ["registry/src/external/**/*.d.ts"],
       "linter": {

test/cacache.test.ts

@@ -2,7 +2,7 @@
  * @fileoverview Unit tests for cacache utility wrapper.
  */
 
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import {
   clear,

test/constants/agents.test.ts

@@ -10,7 +10,6 @@ import {
   BUN_LOCKB,
   NPM,
   NPM_BIN_PATH,
-  NPM_LOCK_JSON,
   NPM_REAL_EXEC_PATH,
   NPM_REGISTRY_URL,
   NPM_SHRINKWRAP_JSON,

test/constants/packages.test.ts

@@ -98,7 +98,7 @@ describe('constants/packages', () => {
     it('should be iterable with for-of', () => {
       const extensions = getPackageExtensions()
       let count = 0
-      for (const [key, value] of extensions) {
+      for (const [key, _value] of extensions) {
         expect(typeof key).toBe('string')
         count++
       }
@@ -172,7 +172,7 @@ describe('constants/packages', () => {
 
     it('should be mutable Map', () => {
       const cache = getPackumentCache()
-      const key = 'test-key-' + Date.now()
+      const key = `test-key-${Date.now()}`
       const value = { test: true }
 
       cache.set(key, value)

test/download-lock.test.ts

@@ -3,7 +3,7 @@
  */
 
 import { existsSync } from 'node:fs'
-import { mkdir, readdir, readFile, rm, writeFile } from 'node:fs/promises'
+import { mkdir, readdir, rm, writeFile } from 'node:fs/promises'
 import { join } from 'node:path'
 import { tmpdir } from 'node:os'
 import { downloadWithLock } from '@socketsecurity/lib/download-lock'

test/env/home.test.ts

@@ -111,7 +111,7 @@ describe('env/home', () => {
     })
 
     it('should handle very long paths', () => {
-      const longPath = '/home/' + 'a'.repeat(200)
+      const longPath = `/home/${'a'.repeat(200)}`
       setEnv('HOME', longPath)
       expect(getHome()).toBe(longPath)
     })

test/env/node-auth-token.test.ts

@@ -82,7 +82,7 @@ describe('env/node-auth-token', () => {
       expect(getNodeAuthToken()).toBe('test-token')
 
       clearEnv('NODE_AUTH_TOKEN')
-      const result = clearEnv('NODE_AUTH_TOKEN')
+      const _result = clearEnv('NODE_AUTH_TOKEN')
       expect(typeof getNodeAuthToken()).toMatch(/string|undefined/)
 
       setEnv('NODE_AUTH_TOKEN', 'new-token')

test/env/socket-cli-shadow.test.ts

@@ -79,7 +79,7 @@ describe('env/socket-cli-shadow', () => {
     })
 
     it('should handle long API token', () => {
-      const longToken = 'sock_' + 'a'.repeat(100)
+      const longToken = `sock_${'a'.repeat(100)}`
       setEnv('SOCKET_CLI_SHADOW_API_TOKEN', longToken)
       expect(getSocketCliShadowApiToken()).toBe(longToken)
     })

test/env/temp-dir.test.ts

@@ -75,7 +75,7 @@ describe('env/temp-dir', () => {
     })
 
     it('should handle long tmpdir path', () => {
-      const longPath = '/a'.repeat(100) + '/tmp'
+      const longPath = `${'/a'.repeat(100)}/tmp`
       setEnv('TMPDIR', longPath)
       expect(getTmpdir()).toBe(longPath)
     })

test/stdio/footer.test.ts

@@ -19,7 +19,7 @@ describe('stdio/footer', () => {
     dateNowSpy = vi.spyOn(Date, 'now').mockReturnValue(1_000_000)
   })
 
-  beforeEach(() => {
+  afterEach(() => {
     if (dateNowSpy) {
       dateNowSpy.mockRestore()
     }