Skip to content

Commit a7cce19

Browse files
committed
fix: use bracket notation for env variable access in TypeScript
Fix TypeScript errors when accessing environment variables from index signatures. Use bracket notation for CI and SOCKET_CLI_DEBUG to satisfy TypeScript's index signature requirements.
1 parent 795169d commit a7cce19

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

src/utils/agent.mts

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { logger } from '@socketsecurity/registry/lib/logger'
12
import { getOwn } from '@socketsecurity/registry/lib/objects'
23
import { spawn } from '@socketsecurity/registry/lib/spawn'
34
import { Spinner } from '@socketsecurity/registry/lib/spinner'
@@ -41,19 +42,20 @@ export function runAgentInstall(
4142
// In CI mode, pnpm uses --frozen-lockfile by default, which prevents lockfile updates.
4243
// We need to explicitly disable it when updating the lockfile with overrides.
4344
const isCi = constants.ENV.CI
44-
const installArgs = isPnpm && isCi
45-
? ['install', '--no-frozen-lockfile', ...args]
46-
: ['install', ...args]
45+
const installArgs =
46+
isPnpm && isCi
47+
? ['install', '--no-frozen-lockfile', ...args]
48+
: ['install', ...args]
4749

4850
// Debug logging for Windows CI issues.
49-
if (constants.WIN32 && (isCi || process.env.SOCKET_CLI_DEBUG)) {
50-
console.error(`[DEBUG] Windows runAgentInstall:`)
51-
console.error(` agent: ${agent}`)
52-
console.error(` isPnpm: ${isPnpm}`)
53-
console.error(` isCi: ${isCi}`)
54-
console.error(` constants.ENV.CI: ${constants.ENV.CI}`)
55-
console.error(` process.env.CI: ${process.env.CI}`)
56-
console.error(` installArgs: ${JSON.stringify(installArgs)}`)
51+
if (constants.WIN32 && (isCi || process.env['SOCKET_CLI_DEBUG'])) {
52+
logger.error(`[DEBUG] Windows runAgentInstall:`)
53+
logger.error(` agent: ${agent}`)
54+
logger.error(` isPnpm: ${isPnpm}`)
55+
logger.error(` isCi: ${isCi}`)
56+
logger.error(` constants.ENV.CI: ${constants.ENV.CI}`)
57+
logger.error(` process.env.CI: ${process.env['CI']}`)
58+
logger.error(` installArgs: ${JSON.stringify(installArgs)}`)
5759
}
5860

5961
return spawn(agentExecPath, installArgs, {

0 commit comments

Comments
 (0)