fix(types): resolve TypeScript errors in GitLab provider

Fixed TypeScript errors:
- Remove unused imports (RequestError, gitDeleteRemoteBranch, sleep)
- Fix GitBeaker import to use correct type path
- Use bracket notation for process.env access (GITLAB_HOST, GITLAB_TOKEN)
- Remove unused destructured variables (base, head)

Author: jdalton

packages/cli/src/commands/fix/pull-request.mts

@@ -1,5 +1,3 @@
-import { RequestError } from '@octokit/request-error'
-
 import { UNKNOWN_VALUE } from '@socketsecurity/lib/constants/core'
 import { debug, debugDir } from '@socketsecurity/lib/debug'
 import { isNonEmptyString } from '@socketsecurity/lib/strings'
@@ -17,7 +15,6 @@ import {
   GQL_PR_STATE_OPEN,
 } from '../../constants/github.mts'
 import { formatErrorWithDetail } from '../../utils/error/errors.mjs'
-import { gitDeleteRemoteBranch } from '../../utils/git/operations.mjs'
 import {
   cacheFetch,
   type GhsaDetails,
@@ -38,10 +35,6 @@ export type OpenSocketFixPrOptions = {
   retries?: number | undefined
 }
 
-async function sleep(ms: number): Promise<void> {
-  return new Promise(resolve => setTimeout(resolve, ms))
-}
-
 export async function openSocketFixPr(
   owner: string,
   repo: string,

packages/cli/src/utils/git/gitlab-provider.mts

@@ -15,10 +15,7 @@ import type {
   PrResponse,
   UpdatePrOptions,
 } from './provider.mts'
-import type {
-  MergeRequestSchema,
-  ProjectSchema,
-} from '@gitbeaker/rest/dist/types/resources/index.js'
+import type { MergeRequestSchema } from '@gitbeaker/rest'
 
 /**
  * GitLab provider for Merge Request operations.
@@ -30,7 +27,7 @@ export class GitLabProvider implements PrProvider {
 
   constructor() {
     const token = getGitLabToken()
-    const host = process.env.GITLAB_HOST || 'https://gitlab.com'
+    const host = process.env['GITLAB_HOST'] || 'https://gitlab.com'
 
     this.gitlab = new Gitlab({
       host,
@@ -103,7 +100,7 @@ export class GitLabProvider implements PrProvider {
   }
 
   async updatePr(options: UpdatePrOptions): Promise<void> {
-    const { base, head, owner, prNumber, repo } = options
+    const { owner, prNumber, repo } = options
 
     const projectId = `${owner}/${repo}`
 
@@ -337,7 +334,7 @@ async function sleep(ms: number): Promise<void> {
  */
 function getGitLabToken(): string {
   // Check environment variable.
-  const envToken = process.env.GITLAB_TOKEN
+  const envToken = process.env['GITLAB_TOKEN']
   if (envToken) {
     return envToken
   }

packages/cli/src/utils/git/provider-factory.mts

@@ -14,7 +14,7 @@ export function createPrProvider(): PrProvider {
   // Check for GitLab.
   if (
     remoteUrl.includes('gitlab.com') ||
-    process.env.GITLAB_HOST ||
+    process.env['GITLAB_HOST'] ||
     remoteUrl.includes('gitlab')
   ) {
     // Lazy load to avoid importing GitLab dependency if not needed.

packages/cli/test/fixtures/commands/patch/pnpm/.socket/manifest.json

@@ -11,7 +11,9 @@
       },
       "vulnerabilities": {
         "GHSA-76c9-3jph-rj3q": {
-          "cves": ["CVE-2025-7339"],
+          "cves": [
+            "CVE-2025-7339"
+          ],
           "summary": "on-headers is vulnerable to http response header manipulation",
           "severity": "LOW",
           "description": "### Impact\n\nA bug in on-headers versions `< 1.1.0` may result in response headers being inadvertently modified when an array is passed to `response.writeHead()`\n\n### Patches\n\nUsers should upgrade to `1.1.0`\n\n### Workarounds\n\nUses are encouraged to upgrade to `1.1.0`, but this issue can be worked around by passing an object to `response.writeHead()` rather than an array.",
@@ -20,4 +22,4 @@
       }
     }
   }
-}
+}

packages/cli/test/fixtures/commands/patch/pnpm/custom-patches/index.js

@@ -1,18 +1,23 @@
+/*!
+ * on-headers
+ * Copyright(c) 2014 Douglas Christopher Wilson
+ * MIT Licensed
+ */
+
+'use strict'
+
 /**
  * Module exports.
  * @public
  */
 
 module.exports = onHeaders
 
-var http = require('node:http')
+var http = require('http')
 
 // older node versions don't have appendHeader
-var isAppendHeaderSupported =
-  typeof http.ServerResponse.prototype.appendHeader === 'function'
-var set1dArray = isAppendHeaderSupported
-  ? set1dArrayWithAppend
-  : set1dArrayWithSet
+var isAppendHeaderSupported = typeof http.ServerResponse.prototype.appendHeader === 'function'
+var set1dArray = isAppendHeaderSupported ? set1dArrayWithAppend : set1dArrayWithSet
 
 /**
  * Create a replacement writeHead method.
@@ -22,11 +27,11 @@ var set1dArray = isAppendHeaderSupported
  * @private
  */
 
-function createWriteHead(prevWriteHead, listener) {
+function createWriteHead (prevWriteHead, listener) {
   var fired = false
 
   // return function with core name and argument list
-  return function writeHead(_statusCode) {
+  return function writeHead (statusCode) {
     // set headers from arguments
     var args = setWriteHeadHeaders.apply(this, arguments)
 
@@ -54,7 +59,7 @@ function createWriteHead(prevWriteHead, listener) {
  * @public
  */
 
-function onHeaders(res, listener) {
+function onHeaders (res, listener) {
   if (!res) {
     throw new TypeError('argument res is required')
   }
@@ -74,7 +79,7 @@ function onHeaders(res, listener) {
  * @private
  */
 
-function setHeadersFromArray(res, headers) {
+function setHeadersFromArray (res, headers) {
   if (headers.length && Array.isArray(headers[0])) {
     // 2D
     set2dArray(res, headers)
@@ -96,7 +101,7 @@ function setHeadersFromArray(res, headers) {
  * @private
  */
 
-function setHeadersFromObject(res, headers) {
+function setHeadersFromObject (res, headers) {
   var keys = Object.keys(headers)
   for (var i = 0; i < keys.length; i++) {
     var k = keys[i]
@@ -111,11 +116,15 @@ function setHeadersFromObject(res, headers) {
  * @private
  */
 
-function setWriteHeadHeaders(statusCode) {
+function setWriteHeadHeaders (statusCode) {
   var length = arguments.length
-  var headerIndex = length > 1 && typeof arguments[1] === 'string' ? 2 : 1
+  var headerIndex = length > 1 && typeof arguments[1] === 'string'
+    ? 2
+    : 1
 
-  var headers = length >= headerIndex + 1 ? arguments[headerIndex] : undefined
+  var headers = length >= headerIndex + 1
+    ? arguments[headerIndex]
+    : undefined
 
   this.statusCode = statusCode
 
@@ -136,7 +145,7 @@ function setWriteHeadHeaders(statusCode) {
   return args
 }
 
-function set2dArray(res, headers) {
+function set2dArray (res, headers) {
   var key
   for (var i = 0; i < headers.length; i++) {
     key = headers[i][0]
@@ -146,7 +155,7 @@ function set2dArray(res, headers) {
   }
 }
 
-function set1dArrayWithAppend(res, headers) {
+function set1dArrayWithAppend (res, headers) {
   for (var i = 0; i < headers.length; i += 2) {
     res.removeHeader(headers[i])
   }
@@ -160,7 +169,7 @@ function set1dArrayWithAppend(res, headers) {
   }
 }
 
-function set1dArrayWithSet(res, headers) {
+function set1dArrayWithSet (res, headers) {
   var key
   for (var i = 0; i < headers.length; i += 2) {
     key = headers[i]