Update dependencies and cleanup unused code

Remove unused logger imports and variables from various utilities, update package dependencies, and minor code formatting improvements.

Author: jdalton

CLAUDE.md

@@ -102,6 +102,11 @@ Each command follows consistent pattern:
   - All tests: `pnpm run testu`
   - Single file: ✅ CORRECT: `pnpm run testu src/commands/specific/cmd-file.test.mts`
   - ❌ WRONG: `pnpm run testu -- src/commands/specific/cmd-file.test.mts`
+- **🚨 MANDATORY Coverage Requirements**: Before pushing commits, ensure test coverage is maintained or improved
+  - **Never decrease coverage**: All changes MUST maintain or increase existing coverage percentages
+  - **Check before push**: Run `pnpm run test` to verify coverage thresholds are met
+  - **Fix coverage drops**: If coverage decreases, add tests to restore or improve coverage before pushing
+  - **Rationale**: Declining coverage indicates untested code paths, which increases risk of bugs and regressions
 
 ### CI Testing Infrastructure
 - **🚨 MANDATORY**: Use `SocketDev/socket-registry/.github/workflows/ci.yml@<SHA>` with full commit SHA (not @main)

package.json

@@ -136,7 +136,7 @@
     "@types/js-yaml": "4.0.9",
     "@types/micromatch": "4.0.9",
     "@types/mock-fs": "4.13.4",
-    "@types/node": "24.5.2",
+    "@types/node": "^20.0.0",
     "@types/npm-package-arg": "6.1.4",
     "@types/npmcli__arborist": "6.3.1",
     "@types/npmcli__config": "6.0.3",
@@ -282,5 +282,8 @@
     "ignore-type-assertion": true,
     "ignore-files": "test/*",
     "strict": true
+  },
+  "dependencies": {
+    "lodash": "4.17.21"
   }
 }

pnpm-lock.yaml

@@ -50,14 +50,13 @@ const noLog =
   process.argv.includes('--no-log') || process.argv.includes('--noLog')
 if (noLog) {
   // Silence all logger methods
-  const noop = () => {}
+  const noop = () => logger
   logger.log = noop
   logger.info = noop
   logger.success = noop
   logger.warn = noop
   logger.error = noop
   logger.fail = noop
-  logger.debug = noop
 }
 
 void (async () => {

src/cli.mts

@@ -96,41 +96,41 @@ export function debugHttpError(error: unknown): void {
 
     // Request details
     if (err.request) {
-      errorInfo.method = err.request.method || 'GET'
-      errorInfo.url = err.request.url || err.request.uri || 'unknown'
+      errorInfo['method'] = err.request.method || 'GET'
+      errorInfo['url'] = err.request.url || err.request.uri || 'unknown'
     } else if (err.config) {
       // Axios-style error
-      errorInfo.method = err.config.method?.toUpperCase() || 'GET'
-      errorInfo.url = err.config.url || 'unknown'
+      errorInfo['method'] = err.config.method?.toUpperCase() || 'GET'
+      errorInfo['url'] = err.config.url || 'unknown'
     }
 
     // Response details
     if (err.response) {
-      errorInfo.status = err.response.status || err.response.statusCode
-      errorInfo.statusText = err.response.statusText || ''
+      errorInfo['status'] = err.response.status || err.response.statusCode
+      errorInfo['statusText'] = err.response.statusText || ''
 
       // Response body
       if (err.response.data) {
-        errorInfo.body = err.response.data
+        errorInfo['body'] = err.response.data
       } else if (err.response.body) {
-        errorInfo.body = err.response.body
+        errorInfo['body'] = err.response.body
       }
 
       // Cloudflare ray ID for debugging
       if (err.response.headers) {
         const cfRay =
           err.response.headers['cf-ray'] || err.response.headers['CF-RAY']
         if (cfRay) {
-          errorInfo.cfRay = cfRay
+          errorInfo['cfRay'] = cfRay
         }
       }
     } else if (err.statusCode) {
-      errorInfo.status = err.statusCode
+      errorInfo['status'] = err.statusCode
     }
 
     // Error message
     if (err.message) {
-      errorInfo.message = err.message
+      errorInfo['message'] = err.message
     }
   }
 

src/utils/debug.mts

@@ -1,5 +1,4 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import type { IncomingMessage } from 'node:http'
 
 // Mock node:http and node:https modules
 const mockRequest = vi.fn()
@@ -19,6 +18,8 @@ vi.mock('node:https', () => ({
 
 import { httpGetJson, httpGetText, httpRequest } from './http.mts'
 
+import type { IncomingMessage } from 'node:http'
+
 describe('HTTP utilities', () => {
   beforeEach(() => {
     vi.clearAllMocks()